projecta-rrr 1.22.0 → 1.22.2

package/CHANGELOG.md

@@ -4,6 +4,78 @@ All notable changes to RRR will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [1.22.2] - 2026-04-19
+
+**Patch: dynamic routing now actually fires in Claude Code (Opus 4.7+).**
+
+### Fixed
+- **Phase 83 model-router matcher**: Claude Code (Opus 4.7+) emits subagent dispatches as `tool_name: "Agent"`, not `"Task"`. v1.22.0/.1 hardcoded `"Task"` in both:
+  1. `hooks/model-router.js` internal gate (`if (toolName !== "Task") return emitAllow()`)
+  2. `rrr/lib/install-hooks-wiring.js` matcher (`{ matcher: "Task", ... }`)
+  Result: hook never fired for any Agent dispatch in Opus 4.7+ sessions, so dynamic routing was a no-op even with `settings.rrr.model_router: "dynamic"`.
+
+  v1.22.2 accepts both `Task` and `Agent`. Live UAT verified end-to-end:
+  ```
+  {"ts":"2026-04-19T05:46:21Z","event":"dispatch","tool_name":"Agent",
+   "agent_name":"rrr-explore","tier":"haiku","reason":"dispatch"}
+  ```
+  Token usage on rrr-explore call: ~10.4k (haiku) vs ~13.2k baseline (general-purpose, sonnet/inherit) — ~21% reduction from tier downgrade alone, on a single dispatch.
+
+### Verified in UAT (v1.21 hosted webhook + Phase 80 App-level)
+- **Hosted Fly app redeployed** with v1.22 code (was running pre-Phase-80 image — `/webhooks/github-app` returned 404 until `fly deploy`).
+- **Phase 80 webhook end-to-end**:
+  - Ping: 200 OK
+  - Valid HMAC push (with `installation_id` lookup): 202 + `{queued:true, job_id}`
+  - Replay (same delivery_id): 200 + `{duplicate:true}` ✓ dedup
+  - Bad signature: 401 `signature_invalid`
+  - Non-default branch: 200 `{skipped:"non-default-branch"}`
+  - Unindexed repo (NULL `github_repo_id`): 200 `{skipped:"unindexed-repo"}`
+- **Neon migration 014 applied**: `installations` table populated with 5 rows from existing `repos.installation_id` data.
+- **`installations` table**: 5 rows under `team_id=pa-ai-team`. Largest installation: 125117864 (29 repos).
+
+### Known followups
+- **Backfill `repos.github_repo_id`** for all 62 repos via `gh api /repos/<full_name>`. Without this, App-level webhook deliveries hit "unindexed-repo" skip path. Operator-actionable.
+- **Postinstall `--hud-only`** still requires separate `npx projecta-rrr install` for full provision. v1.22.x followup.
+- **`tool-disallow` subagent exemption heuristic** uses `input.agent_name` — this field is NOT present in real Claude Code PreToolUse payload (UAT-verified payload schema has `tool_name`, `tool_input`, `session_id`, `transcript_path`, `cwd`, `permission_mode`, `tool_use_id`). Subagent context detection needs different mechanism.
+
+## [1.22.1] - 2026-04-19
+
+**Patch: fixes v1.22.0 fresh install regression.**
+
+### Fixed
+- **`scripts/register-mcp.js` postinstall** no longer breaks `npm install -g projecta-rrr`. Two fixes:
+  1. Fallback path corrected — looks at `<pkg>/rrr/mcp-server/index.js` (npm layout) before `<pkg>/mcp-server/index.js` (legacy). v1.22.0 only checked the legacy path, which doesnt exist in the npm tarball, causing the postinstall to exit 1 and abort the entire install.
+  2. Registration failures are now advisory (`process.exit(0)` + warn) instead of fatal (`process.exit(1)`). Re-run `node scripts/register-mcp.js` or `/rrr:doctor` after install if needed.
+
+Surfaced via UAT: fresh `HOME=tmp npm install -g projecta-rrr@1.22.0 --prefix=tmp` aborted before `node_modules/projecta-rrr` landed. v1.22.1 installs cleanly.
+
+### Verified in UAT
+- **Hosted MCP latency** (5 samples): 108–142ms (median ~128ms) on /health.
+- **Model-router PreToolUse latency** (10 samples): 30–40ms per Task spawn.
+- **Dynamic routing (Phase 83)** static = no rewrite (v1.21 bit-for-bit); dynamic = `rrr-explore→haiku` rewrite + telemetry row.
+- **Stage-aware routing**: `verify-work` + `rrr-verifier` = sonnet; `plan-phase` execution + `general-purpose` = inherit.
+- **Phase 84 escalation**: `rrr-debugger` iter-3 with state at `<project>/.rrr/state/debug-session.json` → opus + telemetry `reason:"debugger_iter_escalate", retry_count:3, prior_tier:"inherit"`.
+- **Phase 85 tool-disallow**: BLOCKs main-thread Read with exit 2 when flag on; allows when flag off; respects `RRR_TOK_DISALLOW=off` kill switch.
+- **Phase 85 edit-batching-nudge**: 2nd Edit within 30s → "edited foo.js 2× in ~30s" advisory; 3rd silent (lastNudgeMs prevents spam); state at `<project>/.planning/.cache/edit-batching-state.json`.
+- **PUSH-POLICY**: 10/10 RRR command skills include `git push`/`auto-push` reference.
+- **auto-push helper**: graceful failure on no-remote ("non-fatal — your work is committed locally"); `auto_push:false` → `{ran:false, skipped:"settings_disabled"}`.
+- **measure-token-delta + measure-opus-rate**: correctly return INSUFFICIENT_DATA verdict on partial telemetry (13 dispatches, need ≥100). Methodology + min_required documented in JSON output.
+- **/rrr:savings**: reads `<project>/.planning/observations*.jsonl` + `~/.rrr/telemetry/escalations.jsonl`.
+- **doctor**: clean on populated install ("Found 1 command root(s) ✅ active"); reports 0 + SAFE MODE on hud-only install (expected — full provision needs `npx projecta-rrr install`).
+- **Fresh install (v1.22.1)**: `HOME=tmp npm install -g projecta-rrr@1.22.1 --prefix=tmp` succeeds, package lands in node_modules.
+
+### Known UAT findings (followups, not blockers)
+- **edit-batching-nudge state location**: `<project>/.planning/.cache/edit-batching-state.json`. Hook is silent if CWD has no `.planning/` directory — by design (cant infer project context). Document in onboarding.
+- **/rrr:savings input path**: project-root `.planning/observations*.jsonl`, NOT HOME. Reports 0 if v1.19 observation logger never ran in current project.
+- **`tool-disallow` subagent exemption**: heuristic uses `input.agent_name` field; needs validation against real Claude Code PreToolUse payload schema (Phase 85 followup).
+- **Postinstall `--hud-only`**: full command/agent/script provision requires separate `npx projecta-rrr install`. Consider auto-bootstrapping in v1.22.x.
+- **`/metrics/freshness`** hosted endpoint returns 404 — by design; HUD reads freshness via `semantic_search` response payload.
+
+
+### Known limitations (followups)
+- `tool-disallow` subagent exemption uses `input.agent_name` heuristic; may need updating when Claude Code PreToolUse payload schema is fully nailed down.
+- Escalation requires per-project state file at `<project>/.rrr/state/debug-session.json` (not HOME). Increments are wired in v1.22 but operator should verify their /rrr:debug session writes to CWD.
+
 ## [1.22.0] - 2026-04-19
 
 **Webhook Live Reindex + Dynamic Routing + Token Diet v2.**

package/hooks/model-router.js

@@ -85,9 +85,12 @@ function main() {
   try { input = JSON.parse(raw); }
   catch (_e) { return emitAllow('router: stdin not JSON'); }
 
-  // Only act on the Task tool — every other tool is pass-through.
+  // Only act on the subagent-dispatch tool — every other tool is pass-through.
+  // Claude Code uses either "Task" (Anthropic API name, most builds) or "Agent"
+  // (Opus 4.7+ builds) for subagent dispatch. Accept both — confirmed via
+  // v1.22.2 UAT live-session capture (/tmp/uat-hook-log.jsonl).
   const toolName = input.tool_name;
-  if (toolName !== 'Task') return emitAllow();
+  if (toolName !== 'Task' && toolName !== 'Agent') return emitAllow();
 
   const toolInput = (input.tool_input && typeof input.tool_input === 'object') ? input.tool_input : {};
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "projecta-rrr",
-  "version": "1.22.0",
+  "version": "1.22.2",
   "description": "A meta-prompting, context engineering and spec-driven development system for Claude Code by Projecta.ai",
   "bin": {
     "projecta-rrr": "bin/install.js",

package/rrr/lib/install-hooks-wiring.js

@@ -210,7 +210,10 @@ function wireToolRedirect({ repoHooksDir, userHooksDir, userSettingsPath, dryRun
  */
 function buildModelRouterEntry(hookAbsPath) {
   return {
-    matcher: 'Task',
+    // Claude Code uses either "Task" (most builds) or "Agent" (Opus 4.7+) as
+    // the subagent dispatch tool name. Match both via regex alternation —
+    // confirmed via v1.22.2 UAT live-session capture.
+    matcher: 'Task|Agent',
     hooks: [
       { type: 'command', command: `node ${hookAbsPath}` },
     ],

package/scripts/register-mcp.js

@@ -41,8 +41,14 @@ function getMCPServerPath() {
     return installedPath;
   }
 
-  // Fallback to relative path from script (for source/development)
-  // scripts/register-mcp.js -> ../mcp-server/index.js
+  // Fallback to package-relative path. Try both layouts: published npm package
+  // ships the server at <pkg>/rrr/mcp-server/index.js, but the dev repo also
+  // has a top-level <pkg>/mcp-server (legacy v1.17 layout).
+  // scripts/register-mcp.js → __dirname is <pkg>/scripts
+  const npmLayout = path.resolve(__dirname, '..', 'rrr', 'mcp-server', 'index.js');
+  if (fs.existsSync(npmLayout)) {
+    return npmLayout;
+  }
   return path.resolve(__dirname, '..', 'mcp-server', 'index.js');
 }
 
@@ -274,13 +280,20 @@ if (require.main === module) {
           console.log(`Registered ${MCP_SERVER_ID} with Claude Code (method: ${result.method})`);
           process.exit(0);
         } else {
-          console.error(`Failed to register: ${result.error}`);
-          process.exit(1);
+          // Advisory only — don't fail npm postinstall on registration failure.
+          // The MCP server can be registered later via /rrr:doctor or by re-running
+          // `node scripts/register-mcp.js`. v1.22.0 shipped exit(1) here, which
+          // broke fresh `npm install -g projecta-rrr` (the package never landed
+          // because postinstall errored). v1.22.1 makes this advisory.
+          console.warn(`[mcp] Registration skipped: ${result.error}`);
+          console.warn(`[mcp] Run 'node scripts/register-mcp.js' or '/rrr:doctor' after install to retry.`);
+          process.exit(0);
         }
       })
       .catch(err => {
-        console.error(`Error: ${err.message}`);
-        process.exit(1);
+        console.warn(`[mcp] Registration error: ${err.message}`);
+        console.warn(`[mcp] Continuing — re-run registration manually.`);
+        process.exit(0);
       });
   }
 }