agentic-orchestrator 0.1.2 → 0.1.4

package/spec-files/progress.md

@@ -1,8 +1,9 @@
 # Progress
 
-- Current milestone: `M28` spec-delta closure for supervisor orchestration completeness + deployment contract gaps.
-- Current task: track and implement remaining cross-spec capability deltas identified by full spec/codebase audit.
-- Completed since last update:
+- Re-audit found remaining integration gaps against `agentic_orchestrator_feature_gaps_closure_spec.md` (several items previously marked complete are only partially integrated).
+- Last updated: 2026-03-03T16:57:47Z
+
+## Completed Tasks:
 
 1. Built Nx monorepo scaffolding with Vitest test target (`apps/control-plane/project.json`,
    `apps/control-plane/vitest.config.ts`, root `package.json`, `nx.json`).
@@ -401,52 +402,480 @@
      `npm run validate:docker-mcp -- --smoke` successfully validated compose config, built `docker-aop-mcp`, and ran
      containerized CLI status via MCP transport against `/repo` bind mount.
 
-- Next tasks:
-
-1. No open implementation tasks remain for the previously tracked M28 backlog.
-
-- Open blockers/risks:
-
-1. No active blockers identified for the completed M28 backlog scope.
-
-- Handoff summary:
-
-1. MCP boundary formalization has been implemented with registry contracts, auth claim verification, idempotent mutation
-   ledgering, MCP adapter endpoints, and generated tool docs.
-2. `SupervisorRuntime` is now a composition facade over coordinator/services with typed ports and preserved
-   behavior-level tests.
-3. CLI orchestration now routes through dedicated command handlers and a composition bootstrap while preserving existing
-   command/flag behavior and output contracts.
-4. Step 6 and Step 7 of the MCP formalization backlog are now complete: default transport is `mcp` with documented
-   `inprocess` fallback, and CI-oriented protocol/registry consistency validation is enforced via loader checks, tests,
-   and workflow automation.
-5. Nx test execution is stabilized for this environment via repository-level daemon/plugin-isolation safe defaults, and
-   `nx run control-plane:test` now succeeds through the workspace launcher path; shell environments can opt in for raw
-   `npx nx` by sourcing `scripts/dev-shell-env.sh`.
-6. Decomposition verification now includes deterministic retry/error-envelope coverage across `PlanService`,
-   `FeatureStateService`, `GateService`, `MergeService`, and idempotent retry replay behavior for `ToolRuntime`.
-7. M18 rollout evidence is now explicit and linked (`spec-files/m18_rollout_checklist.md`) with parity scope, fallback
-   policy, rollback triggers, and sign-off.
-8. OOP-M8 architecture layering constraints are now mechanically enforced in CI (`npm run validate:architecture`) rather
-   than convention-only.
-9. Current baseline remains stable across all closure gates: `typecheck`, MCP contract pin validation, architecture
-   validation, and full test+coverage run.
-10. Runtime artifact ownership is now split as specified: source-managed contracts remain in `agentic/orchestrator/**`
-    while generated runtime/session artifacts are written under `.aop/**`, with deterministic legacy migration and
-    explicit precedence semantics validated by tests.
-11. CLI `delete` is now a first-class command routed through MCP/in-process tool clients and a mutating
-    `feature.delete` contract, with preview-by-default safety semantics and deterministic cleanup summaries.
-12. Planner self-correction architecture is now active: post-QA reconciliation runs every supervisor iteration,
-    deterministically decides `no_update` vs `plan_update`, and logs auditable correction intent into feature decisions.
-13. Supervisor iteration control is now schema-governed via `policy.supervisor.max_iterations_per_phase` (default `6`)
-    and consumed directly by runtime startup.
-14. Merge completion now performs deterministic post-accept cleanup by deleting `.aop/features/<feature_id>` and
-    removing residual queue/session/index references.
-15. Regression coverage now explicitly enforces post-QA correction behavior, edge-case checklist inclusion, and
-    iteration-loop configuration semantics.
-16. Previously open M28 capability deltas are now implemented and validated: queued-feature promotion lifecycle,
-    worker decision-loop integration, true resume restart semantics, and Dockerized MCP deployment contract artifacts.
-17. Dockerized MCP contract validation now includes static + smoke coverage (`validate:docker-mcp -- --smoke`) with
-    successful container build/run evidence.
-
-- Last updated: 2026-03-02T02:09:00Z
+132. Deep-dive analysis against ComposioHQ source: mapped AOP capability gaps across 11 features (G1–G11) and 6
+     infrastructure items (N1–N6); expanded M29-M31 backlog to 17 deliverables; documented AOP's code-driven
+     supervisor as intentional (determinism over meta-agent flexibility); updated roadmap and testing strategy.
+
+133. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G1] by implementing Web Dashboard:
+     - Created `packages/web-dashboard/` Next.js 14 package with SSE event stream, review API
+       (`POST /api/features/:id/review` — approve/deny/request_changes), checkout API
+       (`POST /api/features/:id/checkout`), dark-themed Kanban board UI; `aop dashboard [--port 3000]` CLI command;
+       `dashboard` policy config; 3 tests in `apps/control-plane/test/dashboard-command.spec.ts`.
+
+134. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G2] by implementing Notification System:
+     - Created `apps/control-plane/src/application/services/notifier-service.ts` with 4 channels (desktop, Slack,
+       webhook, in-process), 4-tier priority routing (critical/warning/info/debug), throttle/batch support;
+       event types: `gate_failed`, `collision_detected`, `ready_to_merge`, `feature_merged`;
+       8 tests in `apps/control-plane/test/notifier-service.spec.ts`.
+
+135. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G3] by implementing Init Wizard:
+     - Created `aop init [--auto] [--force]` command via `init-command-handler.ts`; git root detection, template
+       generation for `policy.yaml`, `gates.yaml`, `agents.yaml`, and system prompts; `--auto` mode skips prompts;
+       8 tests in `apps/control-plane/test/init-wizard.spec.ts`.
+
+136. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G4] by implementing Multi-Project Config:
+     - Created `multi-project-loader.ts` parsing `agentic/orchestrator/multi-project.yaml`; added `--project` flag to
+       CLI commands, `--all` flag to `aop status`, per-project run lease isolation;
+       6 tests in `apps/control-plane/test/multi-project.spec.ts`.
+
+137. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G6b] by implementing Automated Cleanup:
+     - Created `aop cleanup [--dry-run] [--yes]` command via `cleanup-command-handler.ts`; grace period enforcement,
+       orphan worktree detection; 5 tests in `apps/control-plane/test/cleanup-command.spec.ts`.
+
+138. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G6c] by implementing Batch Operations:
+     - Extended `aop run` with `--batch` flag for multi-feature deduplication and summary reporting; added
+       `aop status --summary`; 6 tests in `apps/control-plane/test/batch-operations.spec.ts`.
+
+139. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G6d] by implementing Workspace Hooks:
+     - Added `post_create` commands and `symlinks` to worktree config in `agents.yaml`; non-fatal failure handling;
+       5 tests in `apps/control-plane/test/workspace-hooks.spec.ts`.
+
+140. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.4 N6] by implementing Adapter Registry:
+     - Created `apps/control-plane/src/application/services/adapter-registry.ts` with `AdapterRegistry` class,
+       `agent-provider` slot migration from hardcoded union; `adapters.schema.json` validation;
+       6 tests in `apps/control-plane/test/adapter-registry.spec.ts`.
+
+141. Re-verified full M29 gates: `npm run lint`, `npm run typecheck`, `npm test` (45 files / 302 tests passing;
+     coverage above thresholds: lines 91.97%, branches 87.31%, functions 94.42%), `npm run validate:mcp-contracts`
+     (tools=28), `npm run validate:architecture` (79 files checked, 0 violations), `npm run validate:docker-mcp`.
+
+142. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.2 G5] by implementing CI Failure
+     Auto-Remediation (Reactions):
+     - Added `reactions` config to `agentic/orchestrator/policy.yaml` (5 reaction types: `gate_failed`,
+       `agent_stuck`, `collision_detected`, `ready_to_merge`, `changes_requested`)
+     - Extended `policy.schema.json` and `state.schema.json` with `reactions`, `gate_retry_count`, `last_retry_at`
+     - Created `apps/control-plane/src/application/services/reactions-service.ts` with `shouldRetry`,
+       `buildRepairPrompt`, `recordRetry`, `shouldEscalate`, `notifyEscalation`
+     - Wired retry loop into `QaWaveExecutor` and `BuildWaveExecutor`; added `aop retry --feature-id <id> [--force]`
+     - 9 tests in `apps/control-plane/test/reactions.spec.ts`
+
+143. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.2 G6a] by implementing Agent Activity
+     Detection & Health Monitoring:
+     - Created `apps/control-plane/src/application/services/activity-monitor-service.ts` with states
+       `active|idle|waiting_input|blocked|exited|unknown`, heartbeat-based detection
+     - Registered `activity-detector` adapter slot in `adapter-registry.ts`
+     - Added `policy.supervisor.agent_idle_threshold_ms: 300000`; wired into `WorkerDecisionLoop` and `aop status`
+     - 10 tests in `apps/control-plane/test/activity-monitor.spec.ts`
+
+144. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.2 G6] by implementing Session Management
+     Commands:
+     - Extended `WorkerProvider` interface with `sendMessage?`; added `feature.send_message` MCP tool with catalog
+       entry, input/output schemas, RBAC; created `aop send --feature-id <id> --message <msg>`
+     - 8 tests in `apps/control-plane/test/session-management.spec.ts`
+
+145. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.2 G6e] by implementing PR Lifecycle
+     Integration:
+     - Created `apps/control-plane/src/application/services/pr-monitor-service.ts` with injectable `GhRunner`,
+       PR detection, CI/review status mapping, merge score (CI=40, review=40, no-conflicts=15, no-threads=5)
+     - Registered `scm-provider` adapter slot; extended feature state schema with optional `pr` object
+     - Wired into `RunCoordinator` (post-`ready_to_merge` check) and `aop status` output
+     - 10 tests in `apps/control-plane/test/pr-monitor.spec.ts`
+
+146. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.5 N1] by implementing Incremental Gate
+     Execution:
+     - Created `apps/control-plane/src/application/services/gate-interpolation-service.ts` with
+       `interpolateGateCommands()` and `isIncrementalMode()`
+     - `fast` mode applies `{base_branch}`/`{feature_id}` substitution; `full`/`merge` modes unaffected
+     - 7 tests in `apps/control-plane/test/incremental-gates.spec.ts`
+
+147. Resolved architecture boundary violations: extracted `ToolCaller`, `FeatureStateFrontMatter`,
+     `FeatureStatePayload`, `RuntimeRole` into `apps/control-plane/src/core/tool-caller.ts`; updated imports across
+     three new M30 services; `supervisor/types.ts` re-exports for backward compat.
+
+148. Re-verified full M30 gates: `npm run lint`, `npm run typecheck`, `npm run validate:mcp-contracts` (tools=29),
+     `npm run validate:architecture` (86 files, 0 violations), `npm run validate:docker-mcp`, and
+     `npx vitest run` (50 test files / 346 tests passing).
+
+149. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.3 G11] by implementing Hash-Based
+     Multi-Instance Isolation:
+     - Derived `instance_id` from SHA256 of config path (12 hex chars) via `stableHash()` in `core/fs.ts`
+     - Namespaced run-lease file path: `.aop/runtime/<instanceId>/run-lease.json` with migration fallback
+     - Created `InstanceIsolationService` with `computeInstanceId()` and `defaultConfigPath()`
+     - Updated `RunLeaseService`, `FeatureDeletionService`, and CLI bootstrap to use instance-scoped paths
+     - 7 tests in `apps/control-plane/test/instance-isolation.spec.ts`
+
+150. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.5 N4] by implementing
+     Dependency-Aware Feature Scheduling:
+     - Added `DEPENDENCY_UNRESOLVED` and `DEPENDENCY_CIRCULAR` error codes
+     - Created `DependencySchedulerService` with `getUnresolvedDeps()`, `detectCircularDependency()` (DFS),
+       `addToDepBlocked()`, `resolveDepBlocked()`
+     - Hooked `checkDependencies()` into `PlanService.planSubmit()`; added optional `dep_blocked` to index schema
+     - On merge, `resolveDepBlocked()` auto-promotes waiting dependents from blocked queue
+     - 14 tests in `apps/control-plane/test/dependency-scheduler.spec.ts`
+
+151. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.5 N3] by implementing
+     Cost Tracking & Budget Enforcement:
+     - Created `CostTrackingService` with per-feature `cost.json` at `.aop/features/<id>/cost.json`
+     - Added `STATUS.PAUSED_BUDGET`, `cost.record`/`cost.get` MCP tools (catalog now 31 tools), RBAC entries
+     - Added `budget:` section to `policy.yaml` (`per_feature_limit_usd: 50.00`, `alert_threshold: 0.8`)
+     - Added `pauseOverBudgetFeatures()` in `RunCoordinator`; extended `reportDashboard()` with `cost:` field
+     - 8 tests in `apps/control-plane/test/cost-tracking.spec.ts`
+
+152. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.3 G9] by implementing
+     Multi-Tracker Support (GitHub, Linear, Jira):
+     - Created `IssueTrackerService` with `IssueTracker` interface, `GitHubIssueTracker` (gh CLI),
+       `LinearIssueTracker` (stub), `JiraIssueTracker` (stub), and `createIssueTracker()` factory
+     - Added `issue_tracker:` section to `policy.yaml` and schema
+     - Wired into `RunCoordinator`: status sync (comment on transition, close on merge/fail) and spec enrichment
+       (issue context stored in state frontmatter when feature enters planning for first time)
+     - 15 tests in `apps/control-plane/test/issue-tracker.spec.ts`
+
+153. Re-verified full M31 gates: `npm run lint`, `npm run typecheck`, `npm run validate:mcp-contracts` (tools=31),
+     `npm run validate:architecture` (90 files, 0 violations), `npm run validate:docker-mcp`, and
+     `npx vitest run` (54 test files / 390 tests passing).
+
+154. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.5 N2] by implementing Parallel Gate
+     Execution:
+     - Extended `GateStep` interface with optional `parallel_group` and `depends_on` fields
+     - Updated `agentic/orchestrator/schemas/gates.schema.json` with new step properties
+     - Added `buildExecutionWaves()` graph algorithm: groups steps into dependency-ordered waves,
+       steps sharing a `parallel_group` (or with no unmet deps) run concurrently via `Promise.allSettled()`
+     - Extracted `GateExecutionService.executeStep()` private method for per-step retry logic
+     - Backward compatible: steps without `parallel_group`/`depends_on` execute sequentially as before
+     - Added timing metadata (`started_at`, `finished_at`, `parallel_group`) to step results
+     - Failed dependency steps are skipped with `skip_reason: 'dependency_failed'`
+     - Added `parallel_example` profile to `agentic/orchestrator/gates.yaml` demonstrating parallel groups
+     - 10 tests in `apps/control-plane/test/parallel-gates.spec.ts`
+     - Re-verified: `npm run lint`, `npm run typecheck`, `npm run validate:mcp-contracts` (tools=31),
+       `npm run validate:architecture` (90 files, 0 violations), `npx vitest run` (88 files / 607+ tests passing)
+
+155. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.5 N5] by implementing Agent Performance
+     Analytics:
+     - New `PerformanceAnalyticsService` with port-based DI pattern (like `CostTrackingService`)
+     - Records per-feature outcomes: provider, model, status, gate_pass, retry_count, duration_ms, cost_usd
+     - Precomputes aggregates (success_rate, avg_retry_count, avg_duration_ms, avg_cost_usd) by provider+model
+     - Storage: `.aop/analytics/performance.json` (atomic writes via `atomicWriteJson`)
+     - 2 new MCP tools: `performance.record_outcome` (mutating) and `performance.get_analytics` (read-only)
+     - 4 new JSON schemas (input/output for each tool)
+     - Kernel integration: port methods, tool handler registration, RBAC in policy.yaml
+     - 8 tests in `apps/control-plane/test/performance-analytics.spec.ts`
+     - Verified: `npm run typecheck`, `npm run lint`, `npm run validate:mcp-contracts` (tools=33),
+       `npm run validate:architecture` (91 files, 0 violations), `npx vitest run` (89 files / 616+ tests passing)
+
+156. Completed all feature gap closure milestones (M29–M32) from `agentic_orchestrator_feature_gaps_closure_spec.md`.
+     21 deliverables across 4 phases:
+
+     **Phase 1: Critical UX Improvements (M29, P0)** — entries 133–141
+     - [G1] Web Dashboard — `packages/web-dashboard/` Next.js 14 package with SSE, review API, checkout API, Kanban UI
+     - [G2] Notification System — `notifier-service.ts`, 4 channels, 4-tier routing, throttling
+     - [G3] Init Wizard — `aop init [--auto] [--force]`, git detection, template generation
+     - [G4] Multi-Project Config — `multi-project-loader.ts`, `--project` flag, `--all` status, lease isolation
+     - [G6b] Automated Cleanup — `aop cleanup [--dry-run] [--yes]`, grace period, orphan worktree detection
+     - [G6c] Batch Operations — `aop run --batch`, deduplication, `aop status --summary`
+     - [G6d] Workspace Hooks — `post_create` commands, `symlinks`, non-fatal failure handling
+     - [N6] Adapter Registry — `AdapterRegistry`, `agent-provider` slot migration, adapters.schema.json
+
+     **Phase 2: Autonomous Operations & Observability (M30, P1)** — entries 142–148
+     - [G5] CI Failure Auto-Remediation — `reactions-service.ts`, retry loop, dual escalation, `aop retry` command
+     - [G6] Session Management — `aop send`, `feature.send_message` MCP tool, `WorkerProvider.sendMessage()`
+     - [G6a] Agent Activity Detection — `activity-monitor-service.ts`, 6 states, idle threshold, `aop status` integration
+     - [G6e] PR Lifecycle Integration — `pr-monitor-service.ts`, CI/review tracking, merge score, `changes_requested` reaction
+     - [N1] Incremental Gate Execution — `gate-interpolation-service.ts`, `{base_branch}` interpolation, fast-mode only
+
+     **Phase 3: Ecosystem Integration (M31, P2)** — entries 149–153
+     - [G9] Multi-Tracker Support — `issue-tracker-service.ts`, GitHub (gh CLI), Linear/Jira stubs, status sync, spec enrichment
+     - [G11] Hash-Based Multi-Instance Isolation — SHA256 instance IDs, namespaced run-lease paths, migration fallback
+     - [N3] Cost Tracking & Budget Enforcement — `cost.json` per feature, `PAUSED_BUDGET` status, `cost.record`/`cost.get` MCP tools
+     - [N4] Dependency-Aware Feature Scheduling — `dep_blocked` queue, DFS circular detection, auto-promotion on merge
+
+     **Phase 4: Performance & Analytics (M32, P2/P3)** — entries 154–155
+     - [N2] Parallel Gate Execution — `buildExecutionWaves()`, `parallel_group`/`depends_on`, `Promise.allSettled()`, timing metadata
+     - [N5] Agent Performance Analytics — `PerformanceAnalyticsService`, `performance.record_outcome`/`performance.get_analytics` MCP tools
+
+     Final stats: 89 test files / 616+ tests passing, 33 MCP tools, 91 architecture files validated.
+
+     **Deferred by design (P3, architectural misalignment or low ROI):**
+     G8 (alt workspace modes), G10 (K8s/SSH runtimes), G12 (terminal plugins), G13–G18 (plugin system, auto-merge, etc.)
+
+157. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G3] init-wizard closure items from
+     `Next Tasks`:
+     - Added non-`--auto` interactive wizard prompts in `apps/control-plane/src/cli/init-command-handler.ts` for:
+       worktree branch, max parallel gate runs, dashboard port, notification channels, and framework selection.
+     - Preserved deterministic zero-prompt `--auto` behavior via explicit prompt-session gating.
+     - Added template asset propagation during init:
+       `agentic/orchestrator/prompts/*.md` + `agentic/orchestrator/schemas/*.json` copied/generated into target repo.
+     - Added/updated wizard coverage in `apps/control-plane/test/init-wizard.spec.ts` for interactive + auto modes and
+       generated asset assertions.
+
+158. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.5 Quality Gate] missing integration
+     coverage from `Next Tasks` item 20:
+     - Added dashboard API integration suite `apps/control-plane/test/dashboard-api.integration.spec.ts` covering:
+       review approve/request-changes actions, checkout+restore flow, `/api/status?project=...` switching, and
+       `/api/projects` discovery.
+     - Expanded session-management coverage in `apps/control-plane/test/session-management.spec.ts` with
+       `AttachCommandHandler` attach/reattach/no-session paths.
+     - Expanded cleanup coverage in `apps/control-plane/test/cleanup-command.spec.ts` for
+       `cleanup.auto_after_merge` merged-residue eligibility behavior.
+     - Added positional send/attach parser assertions in `apps/control-plane/test/cli-helpers.spec.ts`.
+
+159. Re-verified closure gates after integration completion:
+     - `npm run lint` ✅
+     - `npm run typecheck` ✅
+     - `npm test` ✅ (`61` test files / `583` tests passing; coverage above configured thresholds).
+
+160. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.2 G1/G2/G4/G5/G6a/G6e; §3.5 N6] all
+     remaining `Next Tasks` items (1–9):
+     - [G1] Dashboard mapping/runtime closure:
+       YAML frontmatter parsing for nested fields in `packages/web-dashboard/src/lib/aop-client.ts`,
+       phase derivation hardening (active/blocked_queue/merged precedence), and production-first
+       `aop dashboard` launch path (`npm run build` + `npm run start`) with explicit `--dev` mode in
+       `apps/control-plane/src/cli/dashboard-command-handler.ts` and CLI parser/bootstrap wiring.
+     - [G2] Notification taxonomy/context closure in `apps/control-plane/src/application/services/notifier-service.ts`:
+       canonical `critical|warning|info` routing with bridge fallback for legacy `urgent|action`,
+       dashboard deep links, and richer serialized context payloads in Slack notifications.
+     - [G4] Multi-project behavior closure in `apps/control-plane/src/application/multi-project-loader.ts`
+       and `apps/control-plane/src/interfaces/cli/bootstrap.ts`:
+       omitted `--project` auto-selection (`cwd`, `defaults.default_project`, interactive TTY chooser),
+       and per-project/default `policy`/`gates` override resolution in kernel/runtime construction.
+     - [G5] Remediation/retry closure across `build-wave-executor.ts`, `qa-wave-executor.ts`,
+       `reactions-service.ts`, and `retry-command-handler.ts`:
+       configured `retry_delay_ms` application (`waitBeforeRetry()`), failure-history payloads on escalation,
+       non-force retries executing `gates.run`, and `--force` performing reset + immediate forced gate execution.
+     - [G6a] Provider-specific activity detector closure in
+       `apps/control-plane/src/application/services/activity-monitor-service.ts` with Claude/Codex/heuristic
+       detector paths, waiting-input derivation, and persisted activity fields surfaced in status/dashboard outputs.
+     - [G6e/G7] PR lifecycle correctness closure in
+       `apps/control-plane/src/application/services/pr-monitor-service.ts`,
+       `apps/control-plane/src/supervisor/run-coordinator.ts`, and dashboard routes/UI:
+       branch candidate fallback matching (`<id>`, `feature/<id>`), continuous polling for existing PR metadata,
+       and dashboard exposure of PR metadata + merge score.
+     - [N6] Typed adapter-registry closure in `apps/control-plane/src/core/kernel.ts`
+       and `apps/control-plane/src/supervisor/runtime.ts`:
+       validated adapter resolution at boot and typed registry-based resolution for
+       `notification-channel`, `activity-detector`, and `scm-provider`.
+
+161. Fixed post-closure test drift in `apps/control-plane/test/supervisor-collaborators.spec.ts` by updating
+     `ReactionsService` mocks to include `retryDelayMs()` so collaborator branch coverage matches the
+     current remediation contract.
+
+162. Re-verified full repository gates after final closure/fixes:
+     - `npm run lint` ✅
+     - `npm run typecheck` ✅
+     - `npm test` ✅ (`62` test files / `595` tests passing; coverage:
+       statements/lines `94.53%`, branches `89.43%`, functions `95.88%`).
+
+163. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G6d] remaining Workspace Hooks closure:
+     - Enhanced `apps/control-plane/src/core/workspace-hooks.ts` to emit structured warnings for symlink and
+       `post_create` failures (including non-zero command exits), and to create parent directories for nested
+       symlink targets.
+     - Wired non-fatal warning logging into `apps/control-plane/src/core/kernel.ts` `repoEnsureWorktree(...)`
+       via `formatWorkspaceHookWarning(...)`.
+     - Expanded coverage in `apps/control-plane/test/workspace-hooks.spec.ts` for warning emission and
+       nested symlink directory creation behavior.
+
+164. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.1 G6b; §3.2 G6; §3.3 G9]
+     targeted residual closure work on previously partial rows:
+     - [G6b] Auto-cleanup execution in merge flow:
+       `apps/control-plane/src/application/services/merge-service.ts` now performs best-effort merged artifact
+       cleanup (feature dir + worktree + branch) immediately when `cleanup.auto_after_merge=true` and
+       `grace_period_seconds<=0`, and schedules delayed cleanup when grace is positive.
+       Added coverage in `apps/control-plane/test/merge-service.spec.ts`.
+     - [G6] Session-send idle wait:
+       `apps/control-plane/src/core/kernel.ts` `featureSendMessage(...)` now performs bounded best-effort
+       `getSessionInfo` polling before delivery when provider session introspection is available.
+       Added coverage in `apps/control-plane/test/session-management.spec.ts`.
+     - [G9] Issue tracker status synchronization:
+       `apps/control-plane/src/supervisor/run-coordinator.ts` now syncs tracker status on every
+       status transition (not terminal-only) and persists richer issue context (`id/title/body/status/url`)
+       during planning enrichment. Added coverage in `apps/control-plane/test/run-coordinator.spec.ts`.
+
+165. Re-verified repository gates after residual partial-row implementation:
+     - `npm run lint` ✅
+     - `npm run typecheck` ✅
+     - `npm test` ✅ (`62` test files / `601` tests passing; coverage:
+       statements/lines `94.36%`, branches `89.45%`, functions `95.78%`).
+
+166. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.2 G6]
+     provider-specific interactive attach streaming closure:
+     - Implemented provider command execution path in
+       `apps/control-plane/src/providers/providers.ts` via `NodeProviderCommandRunner`
+       and command templates for supported providers (`codex`, `claude`) so
+       `attachToSession(...)` launches interactive CLI attach and `sendMessage(...)`
+       routes message delivery to provider CLI commands.
+     - Added `agent_config` command-template override support (`attach_command`/`attach_args`,
+       `send_command`/`send_args`, or nested `attach`/`send` objects) for custom
+       provider command wiring without changing code.
+     - Updated `apps/control-plane/src/cli/attach-command-handler.ts` to return deterministic
+       unsupported-provider errors when interactive attach is unavailable.
+     - Added coverage in `apps/control-plane/test/providers.spec.ts` and
+       `apps/control-plane/test/session-management.spec.ts` for supported attach/send
+       command execution and unsupported attach behavior.
+
+167. Completed [Spec: agentic_orchestrator_feature_gaps_closure_spec.md §3.3 G9]
+     non-stub Linear/Jira adapter closure:
+     - Replaced stub implementations in
+       `apps/control-plane/src/application/services/issue-tracker-service.ts` with:
+       - `LinearIssueTracker` API-backed GraphQL calls for
+         `getIssue` / `updateIssueStatus` / `addComment`.
+       - `JiraIssueTracker` API-backed REST calls for
+         `getIssue` / `updateIssueStatus` / `addComment`.
+       - shared injectable `HttpRunner` + default `fetch` runner for deterministic tests.
+     - Added `issue_tracker.enabled` gating to `createIssueTracker(...)`.
+     - Added policy configuration examples for Linear/Jira mappings in
+       `agentic/orchestrator/policy.yaml`.
+     - Expanded integration-style coverage in `apps/control-plane/test/issue-tracker.spec.ts`
+       for request payloads, mappings, fallback behavior, and factory wiring.
+
+168. Re-verified repository gates after G6/G9 closure implementation:
+     - `npm run typecheck` ✅
+     - `npm run lint` ✅
+     - `npm test` ✅ (`62` test files / `598` tests passing; coverage:
+       statements/lines `92.97%`, branches `88.49%`, functions `94.93%`).
+
+169. Systematic coverage improvement pass to bring all files above 90% thresholds:
+     - Added `HelpCommandHandler` tests (subcommand detail, no-subcommand, unknown-subcommand,
+       command-without-flags) to `apps/control-plane/test/cli-helpers.spec.ts`.
+     - Added `RetryCommandHandler` branch tests (terminal-status without force, blocked+forced_retry:fast,
+       blocked+force+full-mode, blocked+force+merge-mode) to `apps/control-plane/test/cli-helpers.spec.ts`.
+     - Added `AttachCommandHandler` branch tests (missing/invalid feature_id, qa/building status,
+       unassigned session fallback to orchestrator) to `apps/control-plane/test/session-management.spec.ts`.
+     - Added `formatWorkspaceHookWarning` tests and `runWorktreePostCreate` throw branch to
+       `apps/control-plane/test/workspace-hooks.spec.ts`.
+     - Added `CleanupCommandHandler` branch tests (active lease skips, grace period not elapsed,
+       auto_after_merge merged residue) to `apps/control-plane/test/cleanup-command.spec.ts`.
+     - Added `ReactionsService` branch tests (retryDelayMs/maxRetries configured values, invalid delay,
+       waitBeforeRetry zero/positive delay, buildRepairPrompt empty history) to
+       `apps/control-plane/test/reactions.spec.ts`.
+     - Added `JiraIssueTracker` branch tests (null payload, no matching transition, no base_url,
+       bearer token auth) to `apps/control-plane/test/issue-tracker.spec.ts`.
+     - Added `NullWorkerProvider.executeCommand` error branch tests (exitCode 127, ENOENT,
+       generic non-zero code) and `reattachSession` null/unknown/valid branches to
+       `apps/control-plane/test/providers.spec.ts`.
+     - Added multi-project selection branch tests (single project auto-select, cwd match,
+       default_project fallback, non-TTY multiple projects error, project not found, help variants)
+       to `apps/control-plane/test/bootstrap.spec.ts`.
+     - `npm test` ✅ (`63` test files / `683` tests passing; coverage:
+       statements/lines `94.41%`, branches `90.04%`, functions `96.07%`). All metrics ≥ 90%.
+
+---
+
+✅ **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).
+
+---
+
+✅ **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.
+
+---
+
+## Next Tasks:
+1. None from this spec slice; all coverage metrics now at or above 90% per-file.
+
+## 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 | — |
+
+---
+
+✅ **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.
+
+---
+
+✅ **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).
+
+---
+
+✅ **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).
+
+Open blockers:
+- None identified for implemented items in this pass.