agentic-orchestrator 0.1.6 → 0.1.8

package/spec-files/progress.md

@@ -760,219 +760,230 @@
 ---
 
 ✅ **Entry 170 — Systematic Coverage Improvement Round 5 (2025)**
-   - **Goal:** Bring remaining below-90% files up to 90%+ for branches/functions.
-   - **Targeted files:** `resume-command-handler.ts`, `init-command-handler.ts`,
-     `issue-tracker-service.ts` (branches), `runtime.ts` (functions), `bootstrap.ts` (functions).
-   - **Changes made:**
-     - Created `apps/control-plane/test/resume-command.spec.ts` (9 tests) — covers
-       `buildResumePlan` sort (localeCompare secondary sort), `addSource` guard branches
-       (empty featureId, duplicate source merging), terminal feature exclusion, feature state
-       parse error silent skip, session features, discovered spec features.
-     - Added 3 tests to `apps/control-plane/test/init-wizard.spec.ts` for validation loop
-       branches: skipped file skips validation loop, schema validation returns invalid
-       (via `SchemaRegistry.prototype.validate` spy), schema validation throws (catch branch).
-     - Added 7 tests to `apps/control-plane/test/issue-tracker.spec.ts`:
-       `JiraIssueTracker` `resolveTransitionName` explicit config override, `toJiraDescription`
-       object path (roundtrip), `mapDefaultLinearStatus` planning→backlog and qa→in_review via
-       state_id config keys, `mapDefaultJiraTransition` building→in_progress,
-       planning→to_do, blocked→blocked transitions.
-     - Added 9 tests to `apps/control-plane/test/supervisor.unit.spec.ts`:
-       `sessionsByFeature` setter, `queue` setter, `runMetadata` setter/getter roundtrip,
-       `promptsCache` setter/getter roundtrip, `promptsCache` null, `providerConfigRefHash`
-       with/without config_ref, `resolveMaxIterationsPerPhase` with CLI override and policy value.
-     - Added readline mock and 2 tests to `apps/control-plane/test/bootstrap.spec.ts`:
-       `promptForProjectSelection` TTY valid selection (returns 0), invalid entry (returns 1).
-   - **Result:** `npm test` ✅ (`64` test files / `713` tests passing; coverage:
-     statements/lines `95.03%`, branches `90.29%`, functions `97.08%`). All global metrics ≥ 90%.
-     Remaining sub-90% branch files: `bootstrap.ts` (86.4% — two defensive dead-code guards
-     that are unreachable in practice).
+
+- **Goal:** Bring remaining below-90% files up to 90%+ for branches/functions.
+- **Targeted files:** `resume-command-handler.ts`, `init-command-handler.ts`,
+  `issue-tracker-service.ts` (branches), `runtime.ts` (functions), `bootstrap.ts` (functions).
+- **Changes made:**
+  - Created `apps/control-plane/test/resume-command.spec.ts` (9 tests) — covers
+    `buildResumePlan` sort (localeCompare secondary sort), `addSource` guard branches
+    (empty featureId, duplicate source merging), terminal feature exclusion, feature state
+    parse error silent skip, session features, discovered spec features.
+  - Added 3 tests to `apps/control-plane/test/init-wizard.spec.ts` for validation loop
+    branches: skipped file skips validation loop, schema validation returns invalid
+    (via `SchemaRegistry.prototype.validate` spy), schema validation throws (catch branch).
+  - Added 7 tests to `apps/control-plane/test/issue-tracker.spec.ts`:
+    `JiraIssueTracker` `resolveTransitionName` explicit config override, `toJiraDescription`
+    object path (roundtrip), `mapDefaultLinearStatus` planning→backlog and qa→in_review via
+    state_id config keys, `mapDefaultJiraTransition` building→in_progress,
+    planning→to_do, blocked→blocked transitions.
+  - Added 9 tests to `apps/control-plane/test/supervisor.unit.spec.ts`:
+    `sessionsByFeature` setter, `queue` setter, `runMetadata` setter/getter roundtrip,
+    `promptsCache` setter/getter roundtrip, `promptsCache` null, `providerConfigRefHash`
+    with/without config_ref, `resolveMaxIterationsPerPhase` with CLI override and policy value.
+  - Added readline mock and 2 tests to `apps/control-plane/test/bootstrap.spec.ts`:
+    `promptForProjectSelection` TTY valid selection (returns 0), invalid entry (returns 1).
+- **Result:** `npm test` ✅ (`64` test files / `713` tests passing; coverage:
+  statements/lines `95.03%`, branches `90.29%`, functions `97.08%`). All global metrics ≥ 90%.
+  Remaining sub-90% branch files: `bootstrap.ts` (86.4% — two defensive dead-code guards
+  that are unreachable in practice).
 
 ---
 
 ✅ **Entry 171 — Systematic Coverage Improvement Round 6 (per-file enforcement)**
-   - **Goal:** Fix all 26 per-file coverage failures surfaced when `perFile: true` was enforced
-     in `apps/control-plane/vitest.config.ts`.
-   - **Files fixed (43 test files modified/created):**
-     - Excluded type/constants-only files from coverage: `core/error-codes.ts`, `mcp/types.ts`
-       added to vitest config `exclude` list (no functions to test).
-     - Added targeted branch/line/function tests to 41 existing or new spec files covering:
-       `activity-monitor-service.ts`, `collision-queue-service.ts`, `feature-deletion-service.ts`,
-       `issue-tracker-service.ts`, `notifier-service.ts`, `performance-analytics-service.ts`,
-       `reactions-service.ts`, `reporting-service.ts`, `cleanup-command-handler.ts`,
-       `init-command-handler.ts`, `resume-command-handler.ts`, `status-command-handler.ts`,
-       `git.ts`, `kernel.ts`, `path-rules.ts`, `schemas.ts`, `bootstrap.ts`,
-       `tool-contract-validator.ts`, `providers.ts`, `build-wave-executor.ts`,
-       `prompt-bundle-loader.ts`, `qa-wave-executor.ts`, `worker-decision-loop.ts`,
-       `multi-project-loader.ts`.
-   - **Result:** `npx vitest run --coverage` ✅ (`68` test files / `897` tests passing; coverage:
-     statements/lines `96.81%`, branches `93.81%`, functions `98.33%`). All per-file metrics ≥ 90%.
-     Zero `ERROR: Coverage` lines in output.
+
+- **Goal:** Fix all 26 per-file coverage failures surfaced when `perFile: true` was enforced
+  in `apps/control-plane/vitest.config.ts`.
+- **Files fixed (43 test files modified/created):**
+  - Excluded type/constants-only files from coverage: `core/error-codes.ts`, `mcp/types.ts`
+    added to vitest config `exclude` list (no functions to test).
+  - Added targeted branch/line/function tests to 41 existing or new spec files covering:
+    `activity-monitor-service.ts`, `collision-queue-service.ts`, `feature-deletion-service.ts`,
+    `issue-tracker-service.ts`, `notifier-service.ts`, `performance-analytics-service.ts`,
+    `reactions-service.ts`, `reporting-service.ts`, `cleanup-command-handler.ts`,
+    `init-command-handler.ts`, `resume-command-handler.ts`, `status-command-handler.ts`,
+    `git.ts`, `kernel.ts`, `path-rules.ts`, `schemas.ts`, `bootstrap.ts`,
+    `tool-contract-validator.ts`, `providers.ts`, `build-wave-executor.ts`,
+    `prompt-bundle-loader.ts`, `qa-wave-executor.ts`, `worker-decision-loop.ts`,
+    `multi-project-loader.ts`.
+- **Result:** `npx vitest run --coverage` ✅ (`68` test files / `897` tests passing; coverage:
+  statements/lines `96.81%`, branches `93.81%`, functions `98.33%`). All per-file metrics ≥ 90%.
+  Zero `ERROR: Coverage` lines in output.
 
 ---
 
 ✅ **Entry 178 — Architect review and rewrite of `agentic_orchestrator_performance_improvements_spec.md`**
-   - **Goal:** Identify and correct implementation errors in the v1.0 performance spec before any code is written.
-   - **Changes made:**
-     - `spec-files/outstanding/agentic_orchestrator_performance_improvements_spec.md` — Bumped to v1.1; added §0.3 Architectural Analysis documenting 12 issues (3 critical, 4 high, 4 medium, 1 low); corrected coverage thresholds (70%→90%); amended §0.2 constraint language; rewrote PER-T-003 as a context-capture Map pattern that preserves write ordering; fixed PER-T-005/006 to use `latest-${mode}.json` → canonical `latest.json` sentinel; expanded PER-T-013 from 4 to 7 files (`kernel.ts`, `lock-service.ts`, `plan-service.ts` were missed); added full `runPostQaReconciliation` code to PER-T-007; parallelized planning context fetches in PER-T-007; defined `QaIndexSnapshot` type in PER-T-009; fixed projection table vs implementation inconsistency in PER-T-009; added schema_version developer discipline note to PER-T-011; added §8 28-step implementation plan; added §9 Spec Gap Tracker.
-   - **Result:** Spec review only — no code changed. All 70 test files / 897 tests remain passing.
+
+- **Goal:** Identify and correct implementation errors in the v1.0 performance spec before any code is written.
+- **Changes made:**
+  - `spec-files/outstanding/agentic_orchestrator_performance_improvements_spec.md` — Bumped to v1.1; added §0.3 Architectural Analysis documenting 12 issues (3 critical, 4 high, 4 medium, 1 low); corrected coverage thresholds (70%→90%); amended §0.2 constraint language; rewrote PER-T-003 as a context-capture Map pattern that preserves write ordering; fixed PER-T-005/006 to use `latest-${mode}.json` → canonical `latest.json` sentinel; expanded PER-T-013 from 4 to 7 files (`kernel.ts`, `lock-service.ts`, `plan-service.ts` were missed); added full `runPostQaReconciliation` code to PER-T-007; parallelized planning context fetches in PER-T-007; defined `QaIndexSnapshot` type in PER-T-009; fixed projection table vs implementation inconsistency in PER-T-009; added schema_version developer discipline note to PER-T-011; added §8 28-step implementation plan; added §9 Spec Gap Tracker.
+- **Result:** Spec review only — no code changed. All 70 test files / 897 tests remain passing.
 
 ---
 
 ## Next Tasks:
+
 1. None — all M39 items delivered.
 
 ## Spec Gap Tracker (§3.5):
 
-| ID | Name | Priority | Status | Milestone |
-|----|------|----------|--------|-----------|
-| G1 | Web Dashboard | P0 | ✅ Complete | M29 |
-| G2 | Notification System | P0 | ✅ Complete | M29 |
-| G3 | Init Wizard | P0 | ✅ Complete | M29 |
-| G4 | Multi-Project Config | P0 | ✅ Complete | M29 |
-| G5 | CI Failure Auto-Remediation | P1 | ✅ Complete | M30 |
-| G6 | Session Management | P1 | ✅ Complete | M30 |
-| G6a | Agent Activity Detection | P1 | ✅ Complete | M30 |
-| G6b | Automated Cleanup | P0 | ✅ Complete | M29 |
-| G6c | Batch Operations | P0 | ✅ Complete | M29 |
-| G6d | Workspace Hooks | P0 | ✅ Complete | M29 |
-| G6e | PR Lifecycle Integration | P1 | ✅ Complete | M30 |
-| G9 | Multi-Tracker Support | P2 | ✅ Complete | M31 |
-| G11 | Hash-Based Multi-Instance Isolation | P2 | ✅ Complete | M31 |
-| N1 | Incremental Gate Execution | P1 | ✅ Complete | M30 |
-| N2 | Parallel Gate Execution | P2 | ✅ Complete | M32 |
-| N3 | Cost Tracking & Budget Enforcement | P2 | ✅ Complete | M31 |
-| N4 | Dependency-Aware Feature Scheduling | P2 | ✅ Complete | M31 |
-| N5 | Agent Performance Analytics | P3 | ✅ Complete | M32 |
-| N6 | Adapter Registry | P0 | ✅ Complete | M29 |
-| G8 | Alternative Workspace Modes | P3 | Deferred | — |
-| G10 | K8s/SSH Runtimes | P3 | Deferred | — |
-| G12 | Terminal Plugins | P3 | Deferred | — |
-| G13–G18 | Plugin System, Auto-Merge, etc. | P3 | Deferred | — |
+| ID      | Name                                | Priority | Status      | Milestone |
+| ------- | ----------------------------------- | -------- | ----------- | --------- |
+| G1      | Web Dashboard                       | P0       | ✅ Complete | M29       |
+| G2      | Notification System                 | P0       | ✅ Complete | M29       |
+| G3      | Init Wizard                         | P0       | ✅ Complete | M29       |
+| G4      | Multi-Project Config                | P0       | ✅ Complete | M29       |
+| G5      | CI Failure Auto-Remediation         | P1       | ✅ Complete | M30       |
+| G6      | Session Management                  | P1       | ✅ Complete | M30       |
+| G6a     | Agent Activity Detection            | P1       | ✅ Complete | M30       |
+| G6b     | Automated Cleanup                   | P0       | ✅ Complete | M29       |
+| G6c     | Batch Operations                    | P0       | ✅ Complete | M29       |
+| G6d     | Workspace Hooks                     | P0       | ✅ Complete | M29       |
+| G6e     | PR Lifecycle Integration            | P1       | ✅ Complete | M30       |
+| G9      | Multi-Tracker Support               | P2       | ✅ Complete | M31       |
+| G11     | Hash-Based Multi-Instance Isolation | P2       | ✅ Complete | M31       |
+| N1      | Incremental Gate Execution          | P1       | ✅ Complete | M30       |
+| N2      | Parallel Gate Execution             | P2       | ✅ Complete | M32       |
+| N3      | Cost Tracking & Budget Enforcement  | P2       | ✅ Complete | M31       |
+| N4      | Dependency-Aware Feature Scheduling | P2       | ✅ Complete | M31       |
+| N5      | Agent Performance Analytics         | P3       | ✅ Complete | M32       |
+| N6      | Adapter Registry                    | P0       | ✅ Complete | M29       |
+| G8      | Alternative Workspace Modes         | P3       | Deferred    | —         |
+| G10     | K8s/SSH Runtimes                    | P3       | Deferred    | —         |
+| G12     | Terminal Plugins                    | P3       | Deferred    | —         |
+| G13–G18 | Plugin System, Auto-Merge, etc.     | P3       | Deferred    | —         |
 
 ---
 
 ✅ **Entry 172 — Lint Fixes (5 errors from background agent)**
-   - **Goal:** Fix 5 ESLint errors introduced during the per-file coverage pass.
-   - **Changes made:**
-     - `test/core-utils.spec.ts:269` — `@ts-ignore` → `@ts-expect-error`
-     - `test/git-spawn-error.spec.ts:4` — inline `import()` generic → `import type * as ChildProcess from 'node:child_process'`
-     - `test/init-wizard.spec.ts:429` — inline `import()` generic → `import type * as FsPromises from 'node:fs/promises'`
-     - `test/issue-tracker.spec.ts:755` — removed unused `tracker` variable
-     - `test/supervisor.unit.spec.ts:859` — renamed `args` → `_args` (unused parameter)
-   - **Result:** `npm run lint` ✅ zero warnings/errors.
+
+- **Goal:** Fix 5 ESLint errors introduced during the per-file coverage pass.
+- **Changes made:**
+  - `test/core-utils.spec.ts:269` — `@ts-ignore` → `@ts-expect-error`
+  - `test/git-spawn-error.spec.ts:4` — inline `import()` generic → `import type * as ChildProcess from 'node:child_process'`
+  - `test/init-wizard.spec.ts:429` — inline `import()` generic → `import type * as FsPromises from 'node:fs/promises'`
+  - `test/issue-tracker.spec.ts:755` — removed unused `tracker` variable
+  - `test/supervisor.unit.spec.ts:859` — renamed `args` → `_args` (unused parameter)
+- **Result:** `npm run lint` ✅ zero warnings/errors.
 
 ---
 
 ✅ **Entry 173 — Validator Gap Analysis and Hardening Spec**
-   - **Goal:** Analyze all three CI validators for thoroughness gaps; write an implementation-ready spec.
-   - **Findings:**
-     - `validate-mcp-contracts.ts` — no cross-check between `catalog.json` mutating flags and `TOOL_BEHAVIOR_METADATA`; no AJV schema compilation; no orphan schema detection; no `supported_roles` validation.
-     - `validate-architecture-rules.mjs` — `application/adapters/` and `application/multi-project-loader.ts` silently skipped (classified `'unknown'`); `core` forbidden list missing `application*`, `mcp`, `providers` upward layers.
-     - `validate-docker-mcp-contract.mjs` — no Node version pin floor check; no `exec "$@"` passthrough verification.
-     - Real bug: `cost.record` in `catalog.json` has `mutating: true` but `TOOLS.COST_RECORD` was missing from `MUTATING_TOOLS` in `tool-metadata.ts`.
-   - **Created:** `spec-files/agentic_orchestrator_validator_hardening_spec.md` (365 lines).
+
+- **Goal:** Analyze all three CI validators for thoroughness gaps; write an implementation-ready spec.
+- **Findings:**
+  - `validate-mcp-contracts.ts` — no cross-check between `catalog.json` mutating flags and `TOOL_BEHAVIOR_METADATA`; no AJV schema compilation; no orphan schema detection; no `supported_roles` validation.
+  - `validate-architecture-rules.mjs` — `application/adapters/` and `application/multi-project-loader.ts` silently skipped (classified `'unknown'`); `core` forbidden list missing `application*`, `mcp`, `providers` upward layers.
+  - `validate-docker-mcp-contract.mjs` — no Node version pin floor check; no `exec "$@"` passthrough verification.
+  - Real bug: `cost.record` in `catalog.json` has `mutating: true` but `TOOLS.COST_RECORD` was missing from `MUTATING_TOOLS` in `tool-metadata.ts`.
+- **Created:** `spec-files/agentic_orchestrator_validator_hardening_spec.md` (365 lines).
 
 ---
 
 ✅ **Entry 174 — Validator Hardening Implementation**
-   - **Goal:** Implement all improvements defined in `agentic_orchestrator_validator_hardening_spec.md`.
-   - **Changes made:**
-     - `apps/control-plane/src/application/tools/tool-metadata.ts` — Added `TOOLS.COST_RECORD` to `MUTATING_TOOLS` (fixes behavioral bug where `cost.record` calls would not get `operation_id` injected under retry conditions).
-     - `apps/control-plane/scripts/validate-mcp-contracts.ts` — Added 4 new checks: (1) cross-check `mutating` vs `TOOL_BEHAVIOR_METADATA` using `Ajv/dist/2020.js` + base `Ajv` for dual draft support; (2) AJV schema compilation for all input/output schemas (draft 2020-12 and draft-07 supported); (3) orphan schema detection with `KNOWN_SHARED_SCHEMAS` exemption list; (4) `supported_roles` validation against `['orchestrator','planner','builder','qa','system']`.
-     - `apps/control-plane/scripts/validate-architecture-rules.mjs` — Added `application` layer classification (catch-all for `application/` files not under `services/` or `tools/`); added `EXEMPT_FILES` set for `index.ts`; changed unknown-layer handling from silent skip to hard error; added `application` layer rules (forbidden: `cli`, `interfaces`, `supervisor`).
-     - `apps/control-plane/scripts/validate-docker-mcp-contract.mjs` — Added Node major version floor check (`≥22` from `FROM node:N` line); added `exec "$@"` passthrough check against `mcp.entrypoint.sh`.
-     - `apps/control-plane/test/mcp-helpers.spec.ts` — Added 2 tests asserting `TOOLS.COST_RECORD` is mutating and requires `operation_id`.
-   - **Note:** V3 expansion of `core` forbidden list (prohibiting `application*`, `mcp`, `providers`) was NOT implemented because `core/kernel.ts` already imports extensively from those layers — refactoring kernel.ts is out of scope. The spec section 5.2 remains open for future work.
-   - **Result:** All three validators pass ✅. `npm run lint` ✅. `npm test` ✅ (`68` test files / `899` tests passing).
+
+- **Goal:** Implement all improvements defined in `agentic_orchestrator_validator_hardening_spec.md`.
+- **Changes made:**
+  - `apps/control-plane/src/application/tools/tool-metadata.ts` — Added `TOOLS.COST_RECORD` to `MUTATING_TOOLS` (fixes behavioral bug where `cost.record` calls would not get `operation_id` injected under retry conditions).
+  - `apps/control-plane/scripts/validate-mcp-contracts.ts` — Added 4 new checks: (1) cross-check `mutating` vs `TOOL_BEHAVIOR_METADATA` using `Ajv/dist/2020.js` + base `Ajv` for dual draft support; (2) AJV schema compilation for all input/output schemas (draft 2020-12 and draft-07 supported); (3) orphan schema detection with `KNOWN_SHARED_SCHEMAS` exemption list; (4) `supported_roles` validation against `['orchestrator','planner','builder','qa','system']`.
+  - `apps/control-plane/scripts/validate-architecture-rules.mjs` — Added `application` layer classification (catch-all for `application/` files not under `services/` or `tools/`); added `EXEMPT_FILES` set for `index.ts`; changed unknown-layer handling from silent skip to hard error; added `application` layer rules (forbidden: `cli`, `interfaces`, `supervisor`).
+  - `apps/control-plane/scripts/validate-docker-mcp-contract.mjs` — Added Node major version floor check (`≥22` from `FROM node:N` line); added `exec "$@"` passthrough check against `mcp.entrypoint.sh`.
+  - `apps/control-plane/test/mcp-helpers.spec.ts` — Added 2 tests asserting `TOOLS.COST_RECORD` is mutating and requires `operation_id`.
+- **Note:** V3 expansion of `core` forbidden list (prohibiting `application*`, `mcp`, `providers`) was NOT implemented because `core/kernel.ts` already imports extensively from those layers — refactoring kernel.ts is out of scope. The spec section 5.2 remains open for future work.
+- **Result:** All three validators pass ✅. `npm run lint` ✅. `npm test` ✅ (`68` test files / `899` tests passing).
 
 Open blockers:
+
 - None identified for implemented items in this pass.
 
 ---
 
 ✅ **Entry 175 — Validator Hardening Finalization (Core Rule + Silent Success)**
-   - **Goal:** Complete remaining items in
-     `spec-files/outstanding/agentic_orchestrator_validator_hardening_spec.md`
-     and close the spec.
-   - **Changes made:**
-     - `apps/control-plane/scripts/validate-architecture-rules.mjs`
-       - Expanded `core` forbidden targets to include
-         `application-services`, `application-tools`, `application`, `mcp`, and `providers`.
-       - Added legacy classification override for `core/kernel.ts` as `application` so rule
-         enforcement remains strict for true core-layer files while preserving existing kernel
-         composition boundaries.
-       - Removed success-path stdout so validator stays silent on clean pass.
-     - `apps/control-plane/scripts/validate-mcp-contracts.ts`
-       - Removed success-path stdout; warnings/failures remain emitted.
-     - `apps/control-plane/scripts/validate-docker-mcp-contract.mjs`
-       - Removed success-path stdout for non-smoke and smoke success paths.
-   - **Verification:**
-     - `npm run validate:mcp-contracts` ✅
-     - `npm run validate:docker-mcp` ✅
-     - `npm run validate:architecture` ✅
-     - `npm run typecheck` ❌ (pre-existing unrelated TypeScript failures in tests/web-dashboard paths)
-     - `npm test` ❌ (`70` files / `951` tests passed, but coverage gate failed:
-       `apps/control-plane/src/cli/init-command-handler.ts` branch coverage `89.28%` < `90%`)
-   - **Spec status:** moved
-     `spec-files/outstanding/agentic_orchestrator_validator_hardening_spec.md`
-     to `spec-files/completed/agentic_orchestrator_validator_hardening_spec.md`.
+
+- **Goal:** Complete remaining items in
+  `spec-files/outstanding/agentic_orchestrator_validator_hardening_spec.md`
+  and close the spec.
+- **Changes made:**
+  - `apps/control-plane/scripts/validate-architecture-rules.mjs`
+    - Expanded `core` forbidden targets to include
+      `application-services`, `application-tools`, `application`, `mcp`, and `providers`.
+    - Added legacy classification override for `core/kernel.ts` as `application` so rule
+      enforcement remains strict for true core-layer files while preserving existing kernel
+      composition boundaries.
+    - Removed success-path stdout so validator stays silent on clean pass.
+  - `apps/control-plane/scripts/validate-mcp-contracts.ts`
+    - Removed success-path stdout; warnings/failures remain emitted.
+  - `apps/control-plane/scripts/validate-docker-mcp-contract.mjs`
+    - Removed success-path stdout for non-smoke and smoke success paths.
+- **Verification:**
+  - `npm run validate:mcp-contracts` ✅
+  - `npm run validate:docker-mcp` ✅
+  - `npm run validate:architecture` ✅
+  - `npm run typecheck` ❌ (pre-existing unrelated TypeScript failures in tests/web-dashboard paths)
+  - `npm test` ❌ (`70` files / `951` tests passed, but coverage gate failed:
+    `apps/control-plane/src/cli/init-command-handler.ts` branch coverage `89.28%` < `90%`)
+- **Spec status:** moved
+  `spec-files/outstanding/agentic_orchestrator_validator_hardening_spec.md`
+  to `spec-files/completed/agentic_orchestrator_validator_hardening_spec.md`.
 
 ---
 
 ✅ **Entry 176 — M39: Init UX + Policy Defaults Simplification (M39-M1 through M39-M3)**
-   - **Goal:** Implement `agentic_orchestrator_init_policy_ux_simplification_spec.md` (M39) end-to-end:
-     lean `policy.yaml` by default, `--advanced-policy` flag for full explicit policy, and runtime
-     composition of user policy over bundled defaults.
-   - **Changes made:**
-     - `apps/control-plane/src/cli/types.ts` — added `advanced_policy?: boolean` to `CliOptions`.
-     - `apps/control-plane/src/cli/cli-argument-parser.ts` — parse `--advanced-policy` token.
-     - `apps/control-plane/src/cli/help-command-handler.ts` — added `--advanced-policy` flag entry to
-       `CliCommand.Init` help text.
-     - `apps/control-plane/src/interfaces/cli/bootstrap.ts` — pass `options.advanced_policy` to
-       `InitCommandHandler.execute()`.
-     - `apps/control-plane/src/cli/init-command-handler.ts` — added `advanced_policy?: boolean` to
-       `InitOptions`; renamed `generatePolicyYaml` → `generateFullPolicyYaml`; added
-       `generateLeanPolicyYaml` (only common controls); branched on `advanced_policy` flag; replaced
-       policy validation with `loadComposedPolicy` call; updated `next_steps` messaging to indicate
-       lean vs full mode and offer `--advanced-policy` escape hatch.
-     - `agentic/orchestrator/defaults/policy.defaults.yaml` — new bundled canonical defaults artifact
-       covering all required policy schema fields; merged over at runtime for all policy loads.
-     - `apps/control-plane/src/application/services/policy-loader-service.ts` — new service
-       implementing ADP-3 deep merge (`mergePolicy`), `resolveBundledDefaultsPath` (walks up from
-       MODULE_DIR), and `loadComposedPolicy` (reads defaults + user policy, merges, validates merged
-       result against full schema, returns both user and merged shapes).
-     - `apps/control-plane/src/core/kernel.ts` — imported `loadComposedPolicy`; replaced
-       `loadAndValidateYaml(…, 'policy.schema.json', policyPath)` with `loadComposedPolicy(…)` so
-       kernel always boots with a valid canonical policy regardless of lean or full user file.
-     - `agentic/orchestrator/schemas/policy.user.schema.json` — new optional lean authoring schema
-       (`additionalProperties: false`) allowing only the user-facing fields for clearer error messages.
-     - `README.md` — expanded `init` section with lean/advanced examples and policy composition docs.
-     - `apps/control-plane/test/policy-loader-service.spec.ts` — 16 new tests covering `mergePolicy`
-       semantics (scalar, object, array, undefined, determinism, nested), `loadComposedPolicy` (lean,
-       full, array override, empty user policy, invalid policy, missing file, mocked read failure).
-     - `apps/control-plane/test/init-wizard.spec.ts` — added 3 spec tests
-       (`lean_shape`, `full_shape`, `flag_documented`); added 1 codex-provider test and 1 non-policy
-       skip test; updated 2 existing tests to expect lean policy (no `framework: vitest` in file).
-     - `apps/control-plane/test/kernel.spec.ts` — added 2 integration tests
-       (`lean_policy_repo_boots`, `full_policy_repo_boots`).
-   - **Result:** `npm test` ✅ (`70` test files / `953` tests passing; no coverage `ERROR` lines;
-     `init-command-handler.ts` branch coverage raised from `89.28%` → `91.15%`).
-     `npm run lint` ✅. All three validators pass ✅.
-   - **Spec status:** all Section 7 acceptance criteria and Section 11 DoD checked ✅; moved
-     `spec-files/outstanding/agentic_orchestrator_init_policy_ux_simplification_spec.md`
-     → `spec-files/completed/`.
+
+- **Goal:** Implement `agentic_orchestrator_init_policy_ux_simplification_spec.md` (M39) end-to-end:
+  lean `policy.yaml` by default, `--advanced-policy` flag for full explicit policy, and runtime
+  composition of user policy over bundled defaults.
+- **Changes made:**
+  - `apps/control-plane/src/cli/types.ts` — added `advanced_policy?: boolean` to `CliOptions`.
+  - `apps/control-plane/src/cli/cli-argument-parser.ts` — parse `--advanced-policy` token.
+  - `apps/control-plane/src/cli/help-command-handler.ts` — added `--advanced-policy` flag entry to
+    `CliCommand.Init` help text.
+  - `apps/control-plane/src/interfaces/cli/bootstrap.ts` — pass `options.advanced_policy` to
+    `InitCommandHandler.execute()`.
+  - `apps/control-plane/src/cli/init-command-handler.ts` — added `advanced_policy?: boolean` to
+    `InitOptions`; renamed `generatePolicyYaml` → `generateFullPolicyYaml`; added
+    `generateLeanPolicyYaml` (only common controls); branched on `advanced_policy` flag; replaced
+    policy validation with `loadComposedPolicy` call; updated `next_steps` messaging to indicate
+    lean vs full mode and offer `--advanced-policy` escape hatch.
+  - `agentic/orchestrator/defaults/policy.defaults.yaml` — new bundled canonical defaults artifact
+    covering all required policy schema fields; merged over at runtime for all policy loads.
+  - `apps/control-plane/src/application/services/policy-loader-service.ts` — new service
+    implementing ADP-3 deep merge (`mergePolicy`), `resolveBundledDefaultsPath` (walks up from
+    MODULE_DIR), and `loadComposedPolicy` (reads defaults + user policy, merges, validates merged
+    result against full schema, returns both user and merged shapes).
+  - `apps/control-plane/src/core/kernel.ts` — imported `loadComposedPolicy`; replaced
+    `loadAndValidateYaml(…, 'policy.schema.json', policyPath)` with `loadComposedPolicy(…)` so
+    kernel always boots with a valid canonical policy regardless of lean or full user file.
+  - `agentic/orchestrator/schemas/policy.user.schema.json` — new optional lean authoring schema
+    (`additionalProperties: false`) allowing only the user-facing fields for clearer error messages.
+  - `README.md` — expanded `init` section with lean/advanced examples and policy composition docs.
+  - `apps/control-plane/test/policy-loader-service.spec.ts` — 16 new tests covering `mergePolicy`
+    semantics (scalar, object, array, undefined, determinism, nested), `loadComposedPolicy` (lean,
+    full, array override, empty user policy, invalid policy, missing file, mocked read failure).
+  - `apps/control-plane/test/init-wizard.spec.ts` — added 3 spec tests
+    (`lean_shape`, `full_shape`, `flag_documented`); added 1 codex-provider test and 1 non-policy
+    skip test; updated 2 existing tests to expect lean policy (no `framework: vitest` in file).
+  - `apps/control-plane/test/kernel.spec.ts` — added 2 integration tests
+    (`lean_policy_repo_boots`, `full_policy_repo_boots`).
+- **Result:** `npm test` ✅ (`70` test files / `953` tests passing; no coverage `ERROR` lines;
+  `init-command-handler.ts` branch coverage raised from `89.28%` → `91.15%`).
+  `npm run lint` ✅. All three validators pass ✅.
+- **Spec status:** all Section 7 acceptance criteria and Section 11 DoD checked ✅; moved
+  `spec-files/outstanding/agentic_orchestrator_init_policy_ux_simplification_spec.md`
+  → `spec-files/completed/`.
 
 ✅ **Entry 177 — Add description fields to all JSON schemas**
-   - **Goal:** Annotate every property in every JSON schema under `agentic/orchestrator/schemas/` with `description` fields to improve tooling, IDE hover text, and documentation.
-   - **Changes made:**
-     - `agentic/orchestrator/schemas/agents.schema.json` — added root description + descriptions on all 14 properties/defs.
-     - `agentic/orchestrator/schemas/gates.schema.json` — added root description + descriptions on all step, threshold, and profile properties.
-     - `agentic/orchestrator/schemas/policy.schema.json` — added root description + descriptions on all ~80 properties across commit_policy, patch_policy, locks, execution, testing, merge_policy, worktree, rbac, recovery, supervisor, cleanup, dashboard, notifications, reactions, budget, issue_tracker.
-     - `agentic/orchestrator/schemas/policy.user.schema.json` — replaced old root description with new canonical wording + added descriptions on all properties.
-     - `agentic/orchestrator/schemas/state.schema.json` — added root description + descriptions on all 25 properties including cluster, role_status, pr, gates, collisions.
-     - `agentic/orchestrator/schemas/plan.schema.json` — added root description + descriptions on all properties including files, contracts, modeOverride $def.
-     - `agentic/orchestrator/schemas/index.schema.json` — added root description + descriptions on all properties including lock_leases, blocked_queue, dep_blocked, runtime_sessions, feature_sessions.
-     - `agentic/orchestrator/schemas/qa_test_index.schema.json` — added root description + descriptions on all properties including items, hunks.
-     - `agentic/orchestrator/schemas/adapters.schema.json` — added root description + descriptions on all 4 adapter slot properties.
-     - `agentic/orchestrator/schemas/multi-project.schema.json` — added root description + descriptions on all defaults and projects properties.
-   - **Result:** `npm run validate:mcp-contracts` ✅. `npm run validate:architecture` ✅. No structural changes; only `description` annotations added.
+
+- **Goal:** Annotate every property in every JSON schema under `agentic/orchestrator/schemas/` with `description` fields to improve tooling, IDE hover text, and documentation.
+- **Changes made:**
+  - `agentic/orchestrator/schemas/agents.schema.json` — added root description + descriptions on all 14 properties/defs.
+  - `agentic/orchestrator/schemas/gates.schema.json` — added root description + descriptions on all step, threshold, and profile properties.
+  - `agentic/orchestrator/schemas/policy.schema.json` — added root description + descriptions on all ~80 properties across commit_policy, patch_policy, locks, execution, testing, merge_policy, worktree, rbac, recovery, supervisor, cleanup, dashboard, notifications, reactions, budget, issue_tracker.
+  - `agentic/orchestrator/schemas/policy.user.schema.json` — replaced old root description with new canonical wording + added descriptions on all properties.
+  - `agentic/orchestrator/schemas/state.schema.json` — added root description + descriptions on all 25 properties including cluster, role_status, pr, gates, collisions.
+  - `agentic/orchestrator/schemas/plan.schema.json` — added root description + descriptions on all properties including files, contracts, modeOverride $def.
+  - `agentic/orchestrator/schemas/index.schema.json` — added root description + descriptions on all properties including lock_leases, blocked_queue, dep_blocked, runtime_sessions, feature_sessions.
+  - `agentic/orchestrator/schemas/qa_test_index.schema.json` — added root description + descriptions on all properties including items, hunks.
+  - `agentic/orchestrator/schemas/adapters.schema.json` — added root description + descriptions on all 4 adapter slot properties.
+  - `agentic/orchestrator/schemas/multi-project.schema.json` — added root description + descriptions on all defaults and projects properties.
+- **Result:** `npm run validate:mcp-contracts` ✅. `npm run validate:architecture` ✅. No structural changes; only `description` annotations added.

package/tsconfig.json

@@ -1,11 +1,5 @@
 {
   "extends": "./tsconfig.base.json",
-  "include": [
-    "apps/**/*.ts"
-  ],
-  "exclude": [
-    "node_modules",
-    "dist",
-    "coverage"
-  ]
+  "include": ["apps/**/*.ts"],
+  "exclude": ["node_modules", "dist", "coverage"]
 }