remove outdated TODOs

R/directory.R

@@ -188,7 +188,3 @@ filter_interfaces <- function(x) {
 
   x[recognized]
 }
-
-
-# TODO: make a test directory in tests/testthat that can be relied on
-# for consistent output

R/doc-pal-cli.R

@@ -19,8 +19,6 @@
 #'
 #' @section Cost:
 #'
-# TODO: make this a template that takes in the token counts and prices as input
-#'
 #' The system prompt from a pal includes something like 4,000 tokens.
 #' Add in (a generous) 100 tokens for the code that's actually highlighted
 #' and also sent off to the model and you're looking at 4,100 input tokens.

R/pal-add-remove.R

@@ -14,7 +14,6 @@
 #' Cand only contain letters and numbers.
 #' @param prompt A single string giving the system prompt. In most cases, this
 #' is a rather long string, containing several newlines.
-# TODO: only add prefix when not supplied one
 #' @param interface One of `"replace"`, `"prefix"`, or `"suffix"`, describing
 #' how the pal will interact with the selection. For example, the
 #' [cli pal][pal_cli] `"replace"`s the selection, while the

tests/testthat/test-pal-add-remove.R

@@ -29,9 +29,6 @@ test_that("pal addition with bad inputs", {
     .pal_add(role = identity, prompt = "hey")
   )
 
-  # TODO: decide on `prompt` interface and test
-
-
   expect_snapshot(
     error = TRUE,
     .pal_add(role = "sillyhead", prompt = "hey", interface = "no")