@event4u/agent-config 9.17.0 → 9.18.1

package/src/scripts/schemas/rule.schema.json

@@ -155,6 +155,16 @@
       },
       "description": "ADR-013 trust block. Vocabulary mirrored by scripts/lint_artefact_frontmatter.py."
     },
+    "collision_ok": {
+      "type": "object",
+      "additionalProperties": {"type": "string", "minLength": 8},
+      "description": "Per-trigger collision disposition: maps a trigger VALUE (case-insensitive) shared with >=1 other rule to a one-line reason why this rule also firing on it is correct. Enforced by scripts/lint_trigger_collisions.ts."
+    },
+    "precedence": {
+      "type": "object",
+      "additionalProperties": {"type": "integer"},
+      "description": "Alternative collision disposition: maps a shared trigger VALUE to an integer ordering; a collision is clean when ALL sharers declare pairwise-distinct integers for it. Enforced by scripts/lint_trigger_collisions.ts."
+    },
     "install": {
       "type": "object",
       "additionalProperties": false,

package/src/server/schemas/settings.ts

@@ -205,6 +205,24 @@ export const settingsSchema = z.object({
             'Optional planning horizon (weeks) the agent shows in roadmap framing ("next 4 weeks"). Set 0 to omit the horizon — most teams prefer to ship without a hardcoded window.',
         ),
     }),
+    planning: z.object({
+        challenge_on_create: z.boolean().default(true).describe(
+            'Gate C — plan-confidence gate before authoring. true (default) = a plan-artifact ask (/roadmap:create, roadmap-writing, /feature:plan, /feature:roadmap) first checks the four 95%-confidence conditions from /challenge-me vision; any gap routes into the interview (or the inline degrade protocol) before authoring, and a confident pass emits exactly one marker line. false = inert, plan asks author directly. An explicit user bypass always wins for that turn and is counted.',
+        ),
+        risk_review: z.boolean().default(true).describe(
+            'Gate R1 — plan-risk review. true (default) = every ready (non-draft) plan must carry a schema-valid "## Risk Register" section (ranked risks, mitigation + anchor per row, freshness marker, exact honest-null grammar), enforced by lint_plan_risk_register at pre-push + CI. false = escape hatch, the validator skips.',
+        ),
+        completion_review: z.boolean().default(true).describe(
+            'Gate R2 — completion review at 100% roadmap completion / pre-PR. true (default) = a findings-before-fixes review by a fresh reviewer context must exist for the current diff hash (or an exact honest-null / skip declaration) before fix commits and PR creation, enforced by check_completion_review at pre-push + CI (CI authoritative; a crashed validator warns and allows). false = escape hatch, the validator skips.',
+        ),
+    })
+        // `.default({})` is load-bearing, not cosmetic: without it the SECTION is
+        // required, so any settings payload that omits `planning` fails with
+        // `ZodError: planning Required` — which contradicts this gate family's own
+        // contract ("missing key = true" for all three) and reds every server
+        // test whose fixture predates the section. With the default, an absent
+        // section materialises as the three `true` leaves the contract promises.
+        .default({}),
     quality: z.object({
         local_auto_run: z.boolean().default(false).describe(
             'Run quality tools (linters, type-checks, formatters) and the local test suite autonomously after edits. Off by default — the agent never runs quality tools proactively and does not ask; the user runs them manually (e.g. /quality-fix) and remote CI is the authoritative gate. The agent only runs a quality tool on an explicit ask, a concrete CI failure, or the new-gate carve-out. Turn on to restore autonomous pipeline runs.',