pdurlej/kan-ductor

Fork 0

feat: detect manual projection drift #39

Merged

pdurlej merged 1 commit from codex/projection-manual-kan-drift into codex/web-blocked-policy-cues

2026-05-10 13:46:24 +02:00

codex commented

2026-05-10 01:12:26 +02:00

Collaborator

Canary status: missing — fire 3+3 review before merge

Summary

Adds a read-only projection drift path for Leviathan/Swarmheart-style clients. The new detectProjectionDrift Integration API route and detect_projection_drift MCP tool compare expected card columns against current Kan state and classify results as in_sync, manual_drift, agent_diverged, or unknown_drift.

Closes #15.

Scope

Adds repository helpers for latest card move activity and latest agent move audit.
Adds agent.detectProjectionDrift at POST /agent/projection/drift.
Adds MCP client/tool wrapper detect_projection_drift.
Documents projection drift semantics.
Adds integration coverage for manual drift and audited agent divergence.

Non-goals

No auto-repair.
No projection writes.
No schema migration.
No analytics dashboard.
No changes to production tokens or deploy config.

Spec sources read

docs/agent-mcp-contract.md
Existing get_recent_activity actor/audit matching behavior
Existing card move and audit write paths
Issue #15 acceptance criteria

Tests / smoke

pnpm --filter @kan/db build
pnpm --filter @kan/db typecheck
pnpm --filter @kan/api typecheck
pnpm --filter @kan/api build
pnpm --filter @kan/mcp typecheck
pnpm --filter @kan/api test -- integration-tests/agent.integration.test.ts agent.test.ts
pnpm exec prettier --check packages/db/src/repository/agent.repo.ts packages/api/src/routers/agent.ts packages/api/integration-tests/agent.integration.test.ts packages/mcp/src/client.ts packages/mcp/src/index.ts docs/agent-mcp-contract.md
git diff --check

Rollback

Remove/disable the detectProjectionDrift route and MCP tool. No data migration or persistent state is introduced.

Owner gates

None for this PR. Future auto-repair or projection write behavior requires an explicit Piotr checkpoint.

Canary status: missing — fire 3+3 review before merge ## Summary Adds a read-only projection drift path for Leviathan/Swarmheart-style clients. The new `detectProjectionDrift` Integration API route and `detect_projection_drift` MCP tool compare expected card columns against current Kan state and classify results as `in_sync`, `manual_drift`, `agent_diverged`, or `unknown_drift`. Closes #15. ## Scope - Adds repository helpers for latest card move activity and latest agent move audit. - Adds `agent.detectProjectionDrift` at `POST /agent/projection/drift`. - Adds MCP client/tool wrapper `detect_projection_drift`. - Documents projection drift semantics. - Adds integration coverage for manual drift and audited agent divergence. ## Non-goals - No auto-repair. - No projection writes. - No schema migration. - No analytics dashboard. - No changes to production tokens or deploy config. ## Spec sources read - `docs/agent-mcp-contract.md` - Existing `get_recent_activity` actor/audit matching behavior - Existing card move and audit write paths - Issue #15 acceptance criteria ## Tests / smoke - `pnpm --filter @kan/db build` - `pnpm --filter @kan/db typecheck` - `pnpm --filter @kan/api typecheck` - `pnpm --filter @kan/api build` - `pnpm --filter @kan/mcp typecheck` - `pnpm --filter @kan/api test -- integration-tests/agent.integration.test.ts agent.test.ts` - `pnpm exec prettier --check packages/db/src/repository/agent.repo.ts packages/api/src/routers/agent.ts packages/api/integration-tests/agent.integration.test.ts packages/mcp/src/client.ts packages/mcp/src/index.ts docs/agent-mcp-contract.md` - `git diff --check` ## Rollback Remove/disable the `detectProjectionDrift` route and MCP tool. No data migration or persistent state is introduced. ## Owner gates None for this PR. Future auto-repair or projection write behavior requires an explicit Piotr checkpoint.

codex added 1 commit

2026-05-10 01:12:26 +02:00

feat(projection): detect manual kan drift ab82946907

codex reviewed

2026-05-10 01:12:31 +02:00

codex left a comment

Opened as #39. This is read-only drift detection only: no projection writes, no auto-repair, no schema migration.

Validation run:

pnpm --filter @kan/db build
pnpm --filter @kan/db typecheck
pnpm --filter @kan/api typecheck
pnpm --filter @kan/api build
pnpm --filter @kan/mcp typecheck
pnpm --filter @kan/api test -- integration-tests/agent.integration.test.ts agent.test.ts
pnpm exec prettier --check ...
git diff --check

Opened as #39. This is read-only drift detection only: no projection writes, no auto-repair, no schema migration. Validation run: - `pnpm --filter @kan/db build` - `pnpm --filter @kan/db typecheck` - `pnpm --filter @kan/api typecheck` - `pnpm --filter @kan/api build` - `pnpm --filter @kan/mcp typecheck` - `pnpm --filter @kan/api test -- integration-tests/agent.integration.test.ts agent.test.ts` - `pnpm exec prettier --check ...` - `git diff --check`

codex reviewed

2026-05-10 01:12:36 +02:00

codex left a comment

Linked implementation for issue #15: #39 adds read-only detectProjectionDrift / detect_projection_drift, classifies manual drift separately from audited agent divergence, and keeps auto-repair out of scope.

Linked implementation for issue #15: #39 adds read-only `detectProjectionDrift` / `detect_projection_drift`, classifies manual drift separately from audited agent divergence, and keeps auto-repair out of scope.

codex reviewed

2026-05-10 01:12:40 +02:00

codex left a comment

Correction: this PR body contains Closes #15, so Forgejo has the issue linkage in the merge metadata. No production-impacting action was taken.

Correction: this PR body contains `Closes #15`, so Forgejo has the issue linkage in the merge metadata. No production-impacting action was taken.

codex commented

2026-05-10 08:48:14 +02:00

Author

Collaborator

3+3 Review Result

Status: NOT READY

Review lanes completed: tech-ollama-1, tech-ollama-2, tech-codex, product-ollama-1, product-ollama-2, product-claude-opus.

Deterministic checks on current stack tip:

PASS: @kan/db typecheck/build
PASS: @kan/shared typecheck/build
PASS: @kan/mcp typecheck/build
PASS: @kan/api typecheck/build and focused agent tests: 28/28
FAIL: targeted web ESLint currently reports apps/web/src/views/card/components/ActivityList.tsx:374 unused isAdmin
FAIL, known broader baseline: @kan/web typecheck still has existing bootstrap/workspace typing errors

Canary gate: current automated canary is not green for the draft stack; treat this as a review/checkpoint result, not merge approval.

Review notes:

Tech blocker: manual drift detection can classify a human move as agent_diverged when a matching agent audit exists within the time window. Use exact audit/activity lineage or audit result activityPublicId; add human-after-agent-within-window regression test.

## 3+3 Review Result Status: **NOT READY** Review lanes completed: tech-ollama-1, tech-ollama-2, tech-codex, product-ollama-1, product-ollama-2, product-claude-opus. Deterministic checks on current stack tip: - PASS: `@kan/db` typecheck/build - PASS: `@kan/shared` typecheck/build - PASS: `@kan/mcp` typecheck/build - PASS: `@kan/api` typecheck/build and focused agent tests: 28/28 - FAIL: targeted web ESLint currently reports `apps/web/src/views/card/components/ActivityList.tsx:374` unused `isAdmin` - FAIL, known broader baseline: `@kan/web typecheck` still has existing bootstrap/workspace typing errors Canary gate: current automated canary is **not green** for the draft stack; treat this as a review/checkpoint result, not merge approval. Review notes: - Tech blocker: manual drift detection can classify a human move as agent_diverged when a matching agent audit exists within the time window. Use exact audit/activity lineage or audit result activityPublicId; add human-after-agent-within-window regression test.