specrails-desktop 2.36.0 → 2.42.2

package/server/dist/repo-lock.js

@@ -1,25 +1,100 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.repositoryLockKey = repositoryLockKey;
 exports.withRepoLock = withRepoLock;
+exports.withRepoLocks = withRepoLocks;
 exports.__resetRepoLocks = __resetRepoLocks;
-/**
- * Process-local per-key async mutex. Serialises work for a given key by chaining:
- * a new call waits for the previous one on the same key before running. Used so
- * concurrent rail merge-backs never run `git merge`/`reset` on the SAME base repo
- * at the same time (two single-ticket rails launched together are concurrent
- * writers). Different keys run in parallel.
- */
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
 const chains = new Map();
-function withRepoLock(key, fn) {
-    const prev = chains.get(key) ?? Promise.resolve();
-    // Run after the previous holder settles (regardless of its outcome). The
-    // caller still sees fn's own result/rejection.
-    const run = prev.catch(() => undefined).then(() => fn());
-    // Keep the chain alive but swallow errors so one failure never poisons the lock.
-    chains.set(key, run.catch(() => undefined));
+function canonicalPath(value) {
+    try {
+        return fs.realpathSync.native(value);
+    }
+    catch {
+        return path.resolve(value);
+    }
+}
+/** A worktree, symlink and main checkout share the same Git object/ref store.
+ * Resolve it on every admission: a moved checkout must not inherit a stale key. */
+function repositoryLockKey(repositoryPath) {
+    if (!fs.existsSync(repositoryPath))
+        return canonicalPath(repositoryPath);
+    let current = canonicalPath(repositoryPath);
+    while (true) {
+        const dotGit = path.join(current, '.git');
+        try {
+            let gitDir = dotGit;
+            if (fs.statSync(dotGit).isFile()) {
+                const match = /^gitdir:\s*(.+)\s*$/m.exec(fs.readFileSync(dotGit, 'utf8'));
+                if (!match)
+                    break;
+                gitDir = path.resolve(current, match[1].trim());
+            }
+            const commonFile = path.join(gitDir, 'commondir');
+            if (fs.existsSync(commonFile))
+                gitDir = path.resolve(gitDir, fs.readFileSync(commonFile, 'utf8').trim());
+            return canonicalPath(gitDir);
+        }
+        catch { /* walk upward for paths inside a checkout */ }
+        const parent = path.dirname(current);
+        if (parent === current)
+            break;
+        current = parent;
+    }
+    return canonicalPath(repositoryPath);
+}
+function withKeyLock(key, fn) {
+    const previous = chains.get(key) ?? Promise.resolve();
+    const run = previous.catch(() => undefined).then(fn);
+    const tail = run.then(() => undefined, () => undefined);
+    chains.set(key, tail);
+    void tail.then(() => { if (chains.get(key) === tail)
+        chains.delete(key); });
     return run;
 }
-/** Test helper: forget all chains (avoids cross-test leakage). */
-function __resetRepoLocks() {
-    chains.clear();
+/** Process-local mutex over the canonical Git common directory. */
+function withRepoLock(repositoryPath, fn) {
+    return withKeyLock(repositoryLockKey(repositoryPath), fn);
+}
+/** Acquire each physical repository exactly once, in stable order. Callers must
+ * not re-enter withRepoLock from fn; compose effects under this one admission. */
+function withRepoLocks(repositoryPaths, fn) {
+    const keys = [...new Set(repositoryPaths.map(repositoryLockKey))].sort();
+    const enter = (index) => index === keys.length ? Promise.resolve().then(fn) : withKeyLock(keys[index], () => enter(index + 1));
+    return enter(0);
 }
+function __resetRepoLocks() { chains.clear(); }

package/server/dist/review-packet.js

@@ -9,23 +9,31 @@ const delivery_evidence_1 = require("./delivery-evidence");
 const rail_pr_store_1 = require("./rail-pr-store");
 const pr_body_1 = require("./pr-body");
 const ai_invocations_1 = require("./ai-invocations");
+const project_repository_provenance_1 = require("./project-repository-provenance");
 const TEST_FILE_RE = /(\.test\.|\.spec\.|(^|\/)__tests__\/)/;
-function churnForRuns(db, runIds) {
+function churnForRuns(db, runIds, repository) {
     const ids = [...new Set(runIds.filter((id) => typeof id === 'string' && id.length > 0))];
     if (ids.length === 0)
         return [];
     const placeholders = ids.map(() => '?').join(', ');
     try {
+        const scope = (0, project_repository_provenance_1.provenanceRepositoryFilter)(db, repository, 'p');
+        const hasRepository = db.prepare('PRAGMA table_info(file_provenance)').all().some(column => column.name === 'repository_id');
         return db.prepare(`
-      SELECT job_id, file_path, added_lines, removed_lines
-        FROM file_story_contributions
-       WHERE job_id IN (${placeholders})
-    `).all(...ids);
+      SELECT c.job_id, c.file_path, c.added_lines, c.removed_lines,
+             ${hasRepository ? 'p.repository_id' : 'NULL'} AS repository_id
+        FROM file_story_contributions c
+        LEFT JOIN file_provenance p ON p.id = c.provenance_id
+       WHERE c.job_id IN (${placeholders}) AND ${scope.sql}
+    `).all(...ids, ...scope.params);
     }
     catch {
         return [];
     }
 }
+function churnIdentity(row) {
+    return JSON.stringify([row.repository_id ?? null, row.file_path]);
+}
 /**
  * Variant selection reads ONLY durable outcomes. `no-changes` must win over
  * `success` (a run that changed nothing has nothing to show), and a mixed batch
@@ -57,8 +65,8 @@ exports.HUMAN_REVIEW_BAND = { min: 50, max: 69 };
 function buildProof(units, evidence, churn, confidence) {
     const proof = [];
     // ── Tier 1: what Specrails measured itself ────────────────────────────────
-    const testFiles = [...new Set(churn.filter((row) => TEST_FILE_RE.test(row.file_path)).map((row) => row.file_path))];
-    const totalFiles = new Set(churn.map((row) => row.file_path)).size;
+    const testFiles = [...new Set(churn.filter((row) => TEST_FILE_RE.test(row.file_path)).map(churnIdentity))];
+    const totalFiles = new Set(churn.map(churnIdentity)).size;
     const added = churn.reduce((sum, row) => sum + (row.added_lines || 0), 0);
     const removed = churn.reduce((sum, row) => sum + (row.removed_lines || 0), 0);
     if (totalFiles > 0) {
@@ -203,13 +211,14 @@ function computeDriftNudges(input) {
     }
     return nudges;
 }
-function composeReviewPacket({ db, row }) {
+function composeReviewPacket({ db, row, repositoryId = row.repository_id ?? undefined, includeLegacyProvenance = false }) {
+    const repository = repositoryId ? { repositoryId, includeLegacy: includeLegacyProvenance } : undefined;
     const ticketIds = safeParse(row.ticket_ids, []);
     const units = safeParse(row.branches, []);
     const runIds = safeParse(row.run_ids, []);
     const snapshot = (0, rail_pr_store_1.readSpecSnapshot)(row.spec_snapshot);
     const evidence = (0, delivery_evidence_1.readSettleEvidence)(row.settle_evidence);
-    const churn = churnForRuns(db, runIds);
+    const churn = churnForRuns(db, runIds, repository);
     const variant = selectVariant(row, units);
     const confidence = (evidence?.units ?? [])
         .map((unit) => unit.confidence)
@@ -236,10 +245,10 @@ function composeReviewPacket({ db, row }) {
             deliveryOutcome: unitRows[0]?.deliveryOutcome ?? null,
             changed: unitRows[0]?.changed ?? null,
             churn: unitChurn === null ? null : {
-                filesTouched: new Set(unitChurn.map((c) => c.file_path)).size,
+                filesTouched: new Set(unitChurn.map(churnIdentity)).size,
                 addedLines: unitChurn.reduce((sum, c) => sum + (c.added_lines || 0), 0),
                 removedLines: unitChurn.reduce((sum, c) => sum + (c.removed_lines || 0), 0),
-                testFilesTouched: [...new Set(unitChurn.filter((c) => TEST_FILE_RE.test(c.file_path)).map((c) => c.file_path))],
+                testFilesTouched: [...new Set(unitChurn.filter((c) => TEST_FILE_RE.test(c.file_path)).map((c) => !repositoryId && c.repository_id ? `${c.repository_id}:${c.file_path}` : c.file_path))],
             },
             runIds: unitRunIds,
         };
@@ -270,15 +279,16 @@ function composeReviewPacket({ db, row }) {
     // a revision touching mostly new files is the honest signal that the spec no
     // longer describes the work.
     const originalFiles = chain.length > 1
-        ? new Set(churnForRuns(db, safeParse(chain[0].run_ids, [])).map((entry) => entry.file_path))
+        ? new Set(churnForRuns(db, safeParse(chain[0].run_ids, []), repository).map(churnIdentity))
         : new Set();
     const driftNudges = computeDriftNudges({
         versions,
         originalFileSet: originalFiles,
-        currentFiles: new Set(churn.map((entry) => entry.file_path)),
+        currentFiles: new Set(churn.map(churnIdentity)),
     });
     return {
         schemaVersion: 1,
+        ...(repositoryId ? { repositoryId } : {}),
         prDeliveryId: row.id,
         railIndex: row.rail_index,
         variant,

package/server/dist/schemas/file-summary.v1.json

@@ -1,62 +1,62 @@
 {
-    "$schema": "https://json-schema.org/draft/2020-12/schema",
-    "$id": "https://raw.githubusercontent.com/fjpulidop/specrails-desktop/main/server/schemas/file-summary.v1.json",
-    "title": "specrails-desktop file summary (v1)",
-    "description": "On-disk JSON shape for an AI-generated file summary persisted under <project>/.specrails/file-summaries/<sha256-of-rel-path>.json.",
-    "type": "object",
-    "required": [
-        "schemaVersion",
-        "path",
-        "fileHash",
-        "summary",
-        "language",
-        "generatedAt",
-        "generatedBy",
-        "triggeredBy"
-    ],
-    "additionalProperties": false,
-    "properties": {
-        "schemaVersion": {
-            "const": 1
-        },
-        "path": {
-            "type": "string",
-            "minLength": 1
-        },
-        "fileHash": {
-            "type": "string",
-            "pattern": "^[0-9a-f]{64}$"
-        },
-        "summary": {
-            "type": "string"
-        },
-        "language": {
-            "type": "string",
-            "enum": ["en", "es"]
-        },
-        "generatedAt": {
-            "type": "string",
-            "format": "date-time"
-        },
-        "generatedBy": {
-            "type": "object",
-            "required": ["model", "promptVersion"],
-            "additionalProperties": false,
-            "properties": {
-                "model": { "type": "string", "minLength": 1 },
-                "promptVersion": { "const": 1 },
-                "truncated": { "type": "boolean" }
-            }
-        },
-        "triggeredBy": {
-            "type": "object",
-            "required": ["kind", "id", "ticketId"],
-            "additionalProperties": false,
-            "properties": {
-                "kind": { "type": "string", "enum": ["job", "user"] },
-                "id": { "type": "string", "minLength": 1 },
-                "ticketId": { "type": ["integer", "null"] }
-            }
-        }
+  "$schema": "https://json-schema.org/draft/2020-12/schema",
+  "$id": "https://raw.githubusercontent.com/fjpulidop/specrails-desktop/main/server/schemas/file-summary.v1.json",
+  "title": "specrails-desktop file summary (v1)",
+  "description": "On-disk JSON shape for an AI-generated file summary persisted under <project>/.specrails/file-summaries/<sha256-of-rel-path>.json.",
+  "type": "object",
+  "required": [
+    "schemaVersion",
+    "path",
+    "fileHash",
+    "summary",
+    "language",
+    "generatedAt",
+    "generatedBy",
+    "triggeredBy"
+  ],
+  "additionalProperties": false,
+  "properties": {
+    "schemaVersion": {
+      "const": 1
+    },
+    "path": {
+      "type": "string",
+      "minLength": 1
+    },
+    "fileHash": {
+      "type": "string",
+      "pattern": "^[0-9a-f]{64}$"
+    },
+    "summary": {
+      "type": "string"
+    },
+    "language": {
+      "type": "string",
+      "enum": ["en", "es"]
+    },
+    "generatedAt": {
+      "type": "string",
+      "format": "date-time"
+    },
+    "generatedBy": {
+      "type": "object",
+      "required": ["model", "promptVersion"],
+      "additionalProperties": false,
+      "properties": {
+        "model": { "type": "string", "minLength": 1 },
+        "promptVersion": { "type": "integer", "minimum": 1 },
+        "truncated": { "type": "boolean" }
+      }
+    },
+    "triggeredBy": {
+      "type": "object",
+      "required": ["kind", "id", "ticketId"],
+      "additionalProperties": false,
+      "properties": {
+        "kind": { "type": "string", "enum": ["job", "user"] },
+        "id": { "type": "string", "minLength": 1 },
+        "ticketId": { "type": ["integer", "null"] }
+      }
     }
+  }
 }

package/server/dist/schemas/profile.v1.json

@@ -1,150 +1,150 @@
 {
-    "$schema": "https://json-schema.org/draft/2020-12/schema",
-    "$id": "https://raw.githubusercontent.com/fjpulidop/specrails-desktop/main/server/schemas/profile.v1.json",
-    "title": "specrails agent profile (v1)",
-    "description": "Declarative configuration for the specrails implement pipeline. Describes which agents participate, which models they run with, and how tasks are routed to specialists. Consumed by `implement.md` and produced by tools like specrails-desktop. This is specrails-desktop's derivative of specrails-core's canonical v1 profile schema: it adds the optional top-level `provider` field and relaxes `modelAlias` to any non-empty string (provider-catalog validation happens in profile-manager.validateStructural). Per core's schema-identity contract, a diverging vendored copy must carry its own $id — only byte-identical copies may share the canonical core $id.",
-    "type": "object",
-    "required": ["schemaVersion", "name", "orchestrator", "agents", "routing"],
-    "additionalProperties": false,
-    "properties": {
-        "schemaVersion": {
-            "const": 1,
-            "description": "Profile schema version. v1 is the only supported value at this time."
-        },
-        "name": {
-            "type": "string",
-            "pattern": "^[a-z0-9][a-z0-9-]*$",
-            "minLength": 1,
-            "maxLength": 64,
-            "description": "Human-readable identifier for this profile (kebab-case)."
-        },
-        "description": {
-            "type": "string",
-            "maxLength": 512,
-            "description": "Optional short summary of when to use this profile."
+  "$schema": "https://json-schema.org/draft/2020-12/schema",
+  "$id": "https://raw.githubusercontent.com/fjpulidop/specrails-desktop/main/server/schemas/profile.v1.json",
+  "title": "specrails agent profile (v1)",
+  "description": "Declarative configuration for the specrails implement pipeline. Describes which agents participate, which models they run with, and how tasks are routed to specialists. Consumed by `implement.md` and produced by tools like specrails-desktop. This is specrails-desktop's derivative of specrails-core's canonical v1 profile schema: it adds the optional top-level `provider` field and relaxes `modelAlias` to any non-empty string (provider-catalog validation happens in profile-manager.validateStructural). Per core's schema-identity contract, a diverging vendored copy must carry its own $id — only byte-identical copies may share the canonical core $id.",
+  "type": "object",
+  "required": ["schemaVersion", "name", "orchestrator", "agents", "routing"],
+  "additionalProperties": false,
+  "properties": {
+    "schemaVersion": {
+      "const": 1,
+      "description": "Profile schema version. v1 is the only supported value at this time."
+    },
+    "name": {
+      "type": "string",
+      "pattern": "^[a-z0-9][a-z0-9-]*$",
+      "minLength": 1,
+      "maxLength": 64,
+      "description": "Human-readable identifier for this profile (kebab-case)."
+    },
+    "description": {
+      "type": "string",
+      "maxLength": 512,
+      "description": "Optional short summary of when to use this profile."
+    },
+    "provider": {
+      "type": "string",
+      "pattern": "^[a-z0-9][a-z0-9-]*$",
+      "description": "Optional registered provider id (for example `claude`, `codex`, `gemini`, or `kimi`). When set, the profile's model values are validated according to that provider adapter: normally against its catalog, while adapters that support configured aliases (such as Kimi) accept any schema-valid non-empty alias. When omitted, validation falls back to the project's resolved provider."
+    },
+    "orchestrator": {
+      "type": "object",
+      "required": ["model"],
+      "additionalProperties": false,
+      "properties": {
+        "model": {
+          "$ref": "#/$defs/modelAlias",
+          "description": "Model used to run the top-level implement orchestrator. The JSON schema accepts any non-empty string; provider-specific validation runs at the structural-check layer (see profile-manager.validateStructural)."
+        }
+      }
+    },
+    "agents": {
+      "type": "array",
+      "minItems": 1,
+      "items": { "$ref": "#/$defs/agentEntry" },
+      "description": "Ordered chain of agents that participate in the pipeline when this profile is active. Every profile (default + custom) must include the baseline trio (sr-architect, sr-developer, sr-reviewer) — the pipeline depends on all three. Custom profiles add optional agents on top of the baseline. sr-merge-resolver is an optional utility agent and is no longer required.",
+      "allOf": [
+        {
+          "description": "Required baseline agent: sr-architect",
+          "contains": {
+            "type": "object",
+            "properties": { "id": { "const": "sr-architect" } },
+            "required": ["id"]
+          }
         },
-        "provider": {
-            "type": "string",
-            "pattern": "^[a-z0-9][a-z0-9-]*$",
-            "description": "Optional registered provider id (for example `claude`, `codex`, `gemini`, or `kimi`). When set, the profile's model values are validated according to that provider adapter: normally against its catalog, while adapters that support configured aliases (such as Kimi) accept any schema-valid non-empty alias. When omitted, validation falls back to the project's resolved provider."
+        {
+          "description": "Required baseline agent: sr-developer",
+          "contains": {
+            "type": "object",
+            "properties": { "id": { "const": "sr-developer" } },
+            "required": ["id"]
+          }
         },
-        "orchestrator": {
+        {
+          "description": "Required baseline agent: sr-reviewer",
+          "contains": {
             "type": "object",
-            "required": ["model"],
-            "additionalProperties": false,
-            "properties": {
-                "model": {
-                    "$ref": "#/$defs/modelAlias",
-                    "description": "Model used to run the top-level implement orchestrator. The JSON schema accepts any non-empty string; provider-specific validation runs at the structural-check layer (see profile-manager.validateStructural)."
-                }
-            }
+            "properties": { "id": { "const": "sr-reviewer" } },
+            "required": ["id"]
+          }
+        }
+      ]
+    },
+    "routing": {
+      "type": "array",
+      "minItems": 0,
+      "description": "Ordered routing rules. The first rule whose `tags` intersects the task's tag set wins. If a `default: true` entry is present it MUST be the last element. Custom profiles may omit routing entirely — tasks then fall through to the first developer-shaped agent in the chain.",
+      "items": { "$ref": "#/$defs/routingRule" }
+    }
+  },
+  "$defs": {
+    "modelAlias": {
+      "type": "string",
+      "minLength": 1,
+      "description": "Accepted model alias. JSON schema accepts any non-empty string without truncating provider-configured aliases; provider-specific catalog enforcement runs at runtime (see profile-manager.validateStructural)."
+    },
+    "agentEntry": {
+      "type": "object",
+      "required": ["id"],
+      "additionalProperties": false,
+      "properties": {
+        "id": {
+          "type": "string",
+          "pattern": "^(sr|custom)-[a-z0-9][a-z0-9-]*$",
+          "description": "Agent identifier. MUST correspond to the selected provider's provider-native role artifact. File-based providers use their own agents catalog (for example `.claude/agents/<id>.md`); Kimi uses the directly discoverable `.kimi-code/skills/<id>/SKILL.md` layout."
         },
-        "agents": {
-            "type": "array",
-            "minItems": 1,
-            "items": { "$ref": "#/$defs/agentEntry" },
-            "description": "Ordered chain of agents that participate in the pipeline when this profile is active. Every profile (default + custom) must include the baseline trio (sr-architect, sr-developer, sr-reviewer) — the pipeline depends on all three. Custom profiles add optional agents on top of the baseline. sr-merge-resolver is an optional utility agent and is no longer required.",
-            "allOf": [
-                {
-                    "description": "Required baseline agent: sr-architect",
-                    "contains": {
-                        "type": "object",
-                        "properties": { "id": { "const": "sr-architect" } },
-                        "required": ["id"]
-                    }
-                },
-                {
-                    "description": "Required baseline agent: sr-developer",
-                    "contains": {
-                        "type": "object",
-                        "properties": { "id": { "const": "sr-developer" } },
-                        "required": ["id"]
-                    }
-                },
-                {
-                    "description": "Required baseline agent: sr-reviewer",
-                    "contains": {
-                        "type": "object",
-                        "properties": { "id": { "const": "sr-reviewer" } },
-                        "required": ["id"]
-                    }
-                }
-            ]
+        "model": {
+          "$ref": "#/$defs/modelAlias",
+          "description": "Model override for this agent when this profile is active. When omitted, the agent's frontmatter `model:` is used as a fallback."
         },
-        "routing": {
-            "type": "array",
-            "minItems": 0,
-            "description": "Ordered routing rules. The first rule whose `tags` intersects the task's tag set wins. If a `default: true` entry is present it MUST be the last element. Custom profiles may omit routing entirely — tasks then fall through to the first developer-shaped agent in the chain.",
-            "items": { "$ref": "#/$defs/routingRule" }
+        "required": {
+          "type": "boolean",
+          "default": false,
+          "description": "If true, the agent cannot be routed past. Baseline agents (sr-architect, sr-developer, sr-reviewer) SHOULD be marked required. sr-merge-resolver is optional and defaults to required: false."
         }
+      }
     },
-    "$defs": {
-        "modelAlias": {
-            "type": "string",
-            "minLength": 1,
-            "description": "Accepted model alias. JSON schema accepts any non-empty string without truncating provider-configured aliases; provider-specific catalog enforcement runs at runtime (see profile-manager.validateStructural)."
-        },
-        "agentEntry": {
-            "type": "object",
-            "required": ["id"],
-            "additionalProperties": false,
-            "properties": {
-                "id": {
-                    "type": "string",
-                    "pattern": "^(sr|custom)-[a-z0-9][a-z0-9-]*$",
-                    "description": "Agent identifier. MUST correspond to the selected provider's provider-native role artifact. File-based providers use their own agents catalog (for example `.claude/agents/<id>.md`); Kimi uses the directly discoverable `.kimi-code/skills/<id>/SKILL.md` layout."
-                },
-                "model": {
-                    "$ref": "#/$defs/modelAlias",
-                    "description": "Model override for this agent when this profile is active. When omitted, the agent's frontmatter `model:` is used as a fallback."
-                },
-                "required": {
-                    "type": "boolean",
-                    "default": false,
-                    "description": "If true, the agent cannot be routed past. Baseline agents (sr-architect, sr-developer, sr-reviewer) SHOULD be marked required. sr-merge-resolver is optional and defaults to required: false."
-                }
+    "routingRule": {
+      "oneOf": [
+        {
+          "type": "object",
+          "required": ["tags", "agent"],
+          "additionalProperties": false,
+          "properties": {
+            "tags": {
+              "type": "array",
+              "minItems": 1,
+              "items": {
+                "type": "string",
+                "pattern": "^[a-z0-9][a-z0-9-]*$"
+              },
+              "description": "Task tags that trigger this rule. Rule wins if any tag intersects."
+            },
+            "agent": {
+              "type": "string",
+              "pattern": "^(sr|custom)-[a-z0-9][a-z0-9-]*$",
+              "description": "Agent to route the task to when this rule wins."
             }
+          }
         },
-        "routingRule": {
-            "oneOf": [
-                {
-                    "type": "object",
-                    "required": ["tags", "agent"],
-                    "additionalProperties": false,
-                    "properties": {
-                        "tags": {
-                            "type": "array",
-                            "minItems": 1,
-                            "items": {
-                                "type": "string",
-                                "pattern": "^[a-z0-9][a-z0-9-]*$"
-                            },
-                            "description": "Task tags that trigger this rule. Rule wins if any tag intersects."
-                        },
-                        "agent": {
-                            "type": "string",
-                            "pattern": "^(sr|custom)-[a-z0-9][a-z0-9-]*$",
-                            "description": "Agent to route the task to when this rule wins."
-                        }
-                    }
-                },
-                {
-                    "type": "object",
-                    "required": ["default", "agent"],
-                    "additionalProperties": false,
-                    "properties": {
-                        "default": {
-                            "const": true,
-                            "description": "Marks this rule as the terminal catch-all. Exactly one entry with `default: true` is allowed, and it MUST be the last element of `routing`."
-                        },
-                        "agent": {
-                            "type": "string",
-                            "pattern": "^(sr|custom)-[a-z0-9][a-z0-9-]*$",
-                            "description": "Agent to route the task to when no earlier rule matched."
-                        }
-                    }
-                }
-            ]
+        {
+          "type": "object",
+          "required": ["default", "agent"],
+          "additionalProperties": false,
+          "properties": {
+            "default": {
+              "const": true,
+              "description": "Marks this rule as the terminal catch-all. Exactly one entry with `default: true` is allowed, and it MUST be the last element of `routing`."
+            },
+            "agent": {
+              "type": "string",
+              "pattern": "^(sr|custom)-[a-z0-9][a-z0-9-]*$",
+              "description": "Agent to route the task to when no earlier rule matched."
+            }
+          }
         }
+      ]
     }
+  }
 }