fix: address PR review comments on Claude config

Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>

Author: joanagmaia

.claude/agents/code-standards-enforcer.md

@@ -114,6 +114,25 @@ After auditing all files, provide:
 
 ## Output Format
 
+Return findings as a JSON array first, then a brief markdown summary.
+
+**Part 1 — JSON findings** (used by `/review-pr` Phase 5 for filtering and compilation):
+
+```json
+[
+  {
+    "file": "backend/src/api/members.ts",
+    "line": 12,
+    "severity": "CRITICAL|SHOULD_FIX|NIT",
+    "rule": "<source>:<section>",
+    "message": "...",
+    "suggestion": "..."
+  }
+]
+```
+
+**Part 2 — Markdown summary** (human-readable):
+
 ```text
 ## Code Standards Audit Report
 
@@ -126,20 +145,6 @@ After auditing all files, provide:
 ### Protected Files
 - [List any protected files that were modified, or "None modified"]
 
-### Violations Found
-
-#### CRITICAL
-[Violations that break core patterns or introduce security/data issues]
-
-#### SHOULD FIX
-[Deviations from documented conventions]
-
-#### NIT
-[Minor style issues, protected-file awareness]
-
-### Correctly Followed
-[Notable rules that were correctly followed]
-
 ### Verdict: PASS / FAIL
-[Summary and priority fixes]
+[Total violations by severity. Top 3 priority fixes.]
 ```

.claude/hooks/guard-protected-files.sh

@@ -82,7 +82,7 @@ fi
 case "$FILE_PATH" in
   tsconfig*.json|*/tsconfig*.json)
     warn "TypeScript configuration — changes affect compilation across the monorepo." ;;
-  .eslintrc*|*/eslintrc*|eslint.config*|*/eslint.config*)
+  .eslintrc*|*/.eslintrc*|eslint.config*|*/eslint.config*)
     warn "ESLint configuration — changes affect code quality rules for the project." ;;
   .prettierrc*|*/.prettierrc*)
     warn "Prettier configuration — changes affect code formatting standards." ;;

.claude/settings.json

@@ -6,7 +6,7 @@
   "hooks": {
     "PreToolUse": [
       {
-        "matcher": "Edit|Write",
+        "matcher": "Edit|Write|MultiEdit",
         "hooks": [
           {
             "type": "command",
@@ -34,11 +34,17 @@
       "Bash(pnpm lint*)",
       "Bash(pnpm format-check*)",
       "Bash(pnpm tsc-check*)",
-      "Bash(npm run lint*)"
+      "Bash(pnpm format*)",
+      "Bash(pnpm test*)",
+      "Bash(npm run lint*)",
+      "Bash(git commit*)",
+      "Bash(git rebase*)",
+      "Bash(git cherry-pick*)",
+      "Bash(git push*)"
     ],
     "deny": [
       "Bash(git reset --hard*)",
-      "Bash(git push --force *)",
+      "Bash(git push --force*)",
       "Bash(rm -rf*)"
     ]
   }

.claude/skills/preflight/SKILL.md

@@ -42,12 +42,12 @@ git diff --name-only origin/main...HEAD
 
 Group changed files into workspaces:
 
-| Workspace | Path prefix | Lint command | Format check | Type check | Test |
+| Workspace | Path prefix | Lint | Format check | Type check | Test |
 |---|---|---|---|---|---|
-| Backend | `backend/` | `cd backend && pnpm lint` | `pnpm format-check` | `pnpm tsc-check` | `pnpm test` (needs Docker) |
+| Backend | `backend/` | `cd backend && pnpm lint` | `cd backend && pnpm format-check` | `cd backend && pnpm tsc-check` | `cd backend && pnpm test` (needs Docker) |
 | Frontend | `frontend/` | `cd frontend && npm run lint` | — | — | — |
-| Each service | `services/apps/<name>/` | `pnpm lint` | `pnpm format-check` | `pnpm tsc-check` | `pnpm test` if vitest present |
-| Each lib | `services/libs/<name>/` | `pnpm lint` | `pnpm format-check` | `pnpm tsc-check` | `pnpm test` if vitest present |
+| Each service | `services/apps/<name>/` | `cd services/apps/<name> && pnpm lint` | `cd services/apps/<name> && pnpm format-check` | `cd services/apps/<name> && pnpm tsc-check` | `cd services/apps/<name> && pnpm test` if vitest present |
+| Each lib | `services/libs/<name>/` | `cd services/libs/<name> && pnpm lint` | `cd services/libs/<name> && pnpm format-check` | `cd services/libs/<name> && pnpm tsc-check` | `cd services/libs/<name> && pnpm test` if vitest present |
 
 Run checks only for the workspaces that have changed files.