ci: fix tool-count smoke check after moduleRegistrar refactor

The Phase-2 refactor replaced direct server.registerTool() calls with a
register() wrapper returned by moduleRegistrar(), so the dist/*.js
output no longer contains 180 matches of the old pattern. The CI smoke
check was counting 2 (just the ones remaining inside registry.js) and
failing with "Expected 180+".

Switch the grep to count register("arcane_ — matches every tool
registration site in dist/tools/*.js exactly once (verified locally:
180).

Also updates the same stale verification snippet in CLAUDE.md.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: RandomSynergy17

.github/workflows/ci.yml

@@ -38,7 +38,10 @@ jobs:
 
       - name: Verify tool count
         run: |
-          TOTAL=$(grep -rc "registerTool(" dist/tools/*.js | awk -F: '{sum+=$2} END {print sum}')
+          # After the tool-filtering refactor each module registers via the
+          # `register("arcane_...", ...)` wrapper returned by moduleRegistrar,
+          # so we count those instead of the former `server.registerTool(`.
+          TOTAL=$(grep -rc 'register("arcane_' dist/tools/*.js | awk -F: '{sum+=$2} END {print sum}')
           echo "Found $TOTAL tool registrations"
           if [ "$TOTAL" -lt 180 ]; then echo "FAIL: Expected 180+"; exit 1; fi
 

CLAUDE.md

@@ -41,7 +41,7 @@ All tools use `server.registerTool()` with:
 3. Classify annotations: list/get = readOnly, delete/prune = destructive, external calls = openWorld
 4. Use `toolHandler()` wrapper — return a plain string, it handles the MCP response envelope
 5. Register in `src/tools/index.ts`
-6. Verify: `npm run build && grep -c "registerTool(" src/tools/*.ts | awk -F: '{sum+=$2} END {print sum}'`
+6. Verify: `npm run build && grep -rc 'register("arcane_' dist/tools/*.js | awk -F: '{sum+=$2} END {print sum}'` (expect 180)
 
 ### Resources & Prompts