Merge remote-tracking branch 'origin/develop' into fb_graphviz

Author: labkey-adam

.claude/commands/review-pr.md

@@ -7,6 +7,8 @@ Steps:
 2. Run `gh pr diff $ARGUMENTS` to get the full diff.
 3. For each file changed, if you need more context than the diff provides, read the relevant file(s).
 
+**IMPORTANT — Line Numbers**: Do NOT use line numbers from the diff output file (e.g., from a saved tool result). Those are offsets within the diff text, not actual source line numbers. To cite an accurate line number in a finding, read the actual source file and find the line there. If you cannot confirm a line number, omit it and reference the code by method or function name instead.
+
 Then perform a thorough review in this exact order:
 
 ---

.claude/settings.json

@@ -6,7 +6,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "cd \"$(git rev-parse --show-toplevel)\" && if command -v python3 >/dev/null 2>&1; then python3 .claude/hooks/check-dangerous-commands.py; else python .claude/hooks/check-dangerous-commands.py; fi"
+            "command": "cd \"$CLAUDE_PROJECT_DIR\" && if command -v python3 >/dev/null 2>&1; then python3 .claude/hooks/check-dangerous-commands.py; else python .claude/hooks/check-dangerous-commands.py; fi"
           }
         ]
       },
@@ -15,7 +15,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "cd \"$(git rev-parse --show-toplevel)\" && if command -v python3 >/dev/null 2>&1; then python3 .claude/hooks/check-secrets-file.py; else python .claude/hooks/check-secrets-file.py; fi"
+            "command": "cd \"$CLAUDE_PROJECT_DIR\" && if command -v python3 >/dev/null 2>&1; then python3 .claude/hooks/check-secrets-file.py; else python .claude/hooks/check-secrets-file.py; fi"
           }
         ]
       }