use_claude_code() (again)

Author: jennybc

.claude/CLAUDE.md

@@ -44,6 +44,9 @@ air format .
 - All new code should have an accompanying test.
 - If there are existing tests, place new tests next to similar existing tests.
 - Strive to keep your tests minimal with few comments.
+- Avoid `expect_true()` and `expect_false()` in favour of a specific expectation which will give a better failure message. A few expectations in newer releases that you might not know about are `expect_all_true()`, `expect_all_equal()`, and `expect_r6_class()`.
+- When testing errors and warnings, don't us `expect_error()` or `expect_warning()`. Instead, use `expect_snapshot(error = TRUE)` for errors and `expect_snapshot()` for warnings because these allow the user to review the full text of the output.
+- Avoid the `.package` argument to `local_mocked_bindings()`; this modifies the namespace of another package which is not good practice. Instead create a mockable version of the function in the current package. See `?local_mocked_bindings` for more details.
 
 ### Documentation
 

.claude/settings.json

@@ -1,8 +1,7 @@
 {
-  
+  "$schema": "https://json.schemastore.org/claude-code-settings.json",
   "permissions": {
-    "$schema": "https://json.schemastore.org/claude-code-settings.json",
-    "defaultMode": "acceptEdits",  
+    "defaultMode": "acceptEdits",
     "allow": [
       "Bash(air:*)",
       "Bash(cat:*)",
@@ -19,13 +18,13 @@
       "Bash(rm:*)",
       "Bash(Rscript:*)",
       "Bash(sed:*)",
-      "Skill(*)",
+      "Skill",
       "WebFetch(domain:cran.r-project.org)",
       "WebFetch(domain:github.com)",
       "WebFetch(domain:raw.githubusercontent.com)"
     ],
     "deny": [
-      "Read(.Renviron)", 
+      "Read(.Renviron)",
       "Read(.env)"
     ]
   }

.claude/skills/tidy-argument-checking/SKILL.md

@@ -73,9 +73,9 @@ These functions are exported by rlang.
 
   Note that partial matching is an error, unlike `base::match.arg()`. 
 
-- `check_exclusive()` ensures only one of two arguments can be supplied. Supplying both together (i.e. both of them are non-`NULL`) is an error. Use `.require = TRUE` if both can be omitted.
+- `check_exclusive()` ensures only one of two arguments can be supplied. Supplying both together (i.e. both of them are non-`NULL`) is an error. Use `.require = FALSE` if both can be omitted.
 
-- `check_required()`: Nice error message if required argument is not supplied.
+- `check_required()`: Nice error message if required argument (i.e. can't be `missing()`) is not supplied.
 
 ## `call` and `arg` arguments