docs(agent): clarify projection drift explanations #147

Merged
pdurlej merged 1 commit from codex/projection-drift-operator-copy into main 2026-06-03 21:31:22 +02:00
Collaborator

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

Summary

Fixes #128 by making projection drift explanations clearer for Piotr + Iskra without changing drift semantics.

Scope

  • Updates runtime detect_projection_drift explanations for:
    • in_sync,
    • manual_drift, including low-confidence nearby agent audit cases,
    • agent_diverged,
    • unknown_drift.
  • Updates MCP tool description to frame manual moves as operator overrides and unknown drift as review-needed.
  • Adds operator-facing status explanation tables to the MCP and Leviathan contracts.
  • Adds regression coverage for the low-confidence manual drift explanation.

Non-goals

  • No drift status/schema change.
  • No production deploy or restart.
  • No DB schema or migration.
  • No projection apply behavior change.
  • No auto-repair or new write path.

Spec sources read

  • #128
  • docs/agent-mcp-contract.md
  • docs/leviathan-projection-contract.md
  • packages/api/src/routers/agent.ts
  • packages/mcp/src/index.ts

Tests / smoke

  • pnpm --filter @kan/api exec vitest integration-tests/agent.integration.test.ts --run -t "reports human move after agent move" — passed.
  • pnpm --filter @kan/api exec vitest integration-tests/agent.integration.test.ts --run — passed, 15 tests.
  • pnpm --filter @kan/api typecheck — passed.
  • pnpm --filter @kan/mcp typecheck — passed.
  • pnpm --filter @kan/mcp test — passed.
  • pnpm exec prettier --check packages/api/src/routers/agent.ts packages/api/integration-tests/agent.integration.test.ts packages/mcp/src/index.ts docs/agent-mcp-contract.md docs/leviathan-projection-contract.md — passed.

Rollback

Revert 7187e9c0. No data migration.

Owner gates

No production-impacting action taken. Merge decision remains with Piotr.

Canary status: missing — fire 3+3 review before merge ## Summary Fixes #128 by making projection drift explanations clearer for Piotr + Iskra without changing drift semantics. ## Scope - Updates runtime `detect_projection_drift` explanations for: - `in_sync`, - `manual_drift`, including low-confidence nearby agent audit cases, - `agent_diverged`, - `unknown_drift`. - Updates MCP tool description to frame manual moves as operator overrides and unknown drift as review-needed. - Adds operator-facing status explanation tables to the MCP and Leviathan contracts. - Adds regression coverage for the low-confidence manual drift explanation. ## Non-goals - No drift status/schema change. - No production deploy or restart. - No DB schema or migration. - No projection apply behavior change. - No auto-repair or new write path. ## Spec sources read - #128 - `docs/agent-mcp-contract.md` - `docs/leviathan-projection-contract.md` - `packages/api/src/routers/agent.ts` - `packages/mcp/src/index.ts` ## Tests / smoke - `pnpm --filter @kan/api exec vitest integration-tests/agent.integration.test.ts --run -t "reports human move after agent move"` — passed. - `pnpm --filter @kan/api exec vitest integration-tests/agent.integration.test.ts --run` — passed, 15 tests. - `pnpm --filter @kan/api typecheck` — passed. - `pnpm --filter @kan/mcp typecheck` — passed. - `pnpm --filter @kan/mcp test` — passed. - `pnpm exec prettier --check packages/api/src/routers/agent.ts packages/api/integration-tests/agent.integration.test.ts packages/mcp/src/index.ts docs/agent-mcp-contract.md docs/leviathan-projection-contract.md` — passed. ## Rollback Revert `7187e9c0`. No data migration. ## Owner gates No production-impacting action taken. Merge decision remains with Piotr.
docs(agent): clarify projection drift explanations (#128)
All checks were successful
Forgejo CI / verify (pull_request) Successful in 39s
7187e9c0b3
pdurlej deleted branch codex/projection-drift-operator-copy 2026-06-03 21:31:22 +02:00
Sign in to join this conversation.
No reviewers
No labels
3plus3-followup
agent/claude-code
agent/codex
agent/hermes
agent/iskra
agent/ollama
agent/patchwarden
analytics
api
cockpit
dependency/blocked
dependency/blocks-others
dependency/cross-repo
dependency/needs-confirmation
docs
domain:agents
domain:ci
domain:docs
domain:forgejo
domain:infra
domain:memory
domain:runtime
domain:signal
domain:ux
flow/architecture
flow/blocked
flow/deployed
flow/done
flow/implementation
flow/intake
flow/maintained
flow/observed
flow/ready
flow/refining
flow/retired
flow/review
gemini-flash
judge/codex-candidate
judge/hermes-candidate
judge/low-confidence
judge/needs-refinement
judge/operator-needed
judge/p0
judge/p1
judge/p2
judge/p3
judge/park
judge/patchwarden-candidate
judge/stale-priority
kind/adr
kind/bug
kind/chore
kind/feature
kind/infra
kind/ops
kind/refactor
kind/research
leviathan
mcp
merge/auto
merge/manual
merge/manual-dependency-conflict
merge/manual-failing-tests
merge/manual-merge-conflict
merge/manual-missing-review
merge/manual-operator-preference
merge/manual-red-zone
merge/manual-security-sensitive
merge/manual-unclear-scope
merge/manual-unknown
mode:operator-only
mode:patchwarden-iskra-approved
mode:safe-auto
observed/erroring
observed/needs-followup
observed/pending
observed/retire-candidate
observed/unused
observed/used
ops
priority:p0
priority:p1
priority:p2
priority:p3
review:claude-reviewed
review:codex-reviewed
review:dziadek-reviewed
review:needs-human
safety
safety:external-write
safety:no-prod-mutation
safety:prod-impact
safety:secret-touch
scout
security
size/large
size/medium
size/small
size/tiny
size/unknown
small-task
source/adr
source/agent-generated
source/manual
source/operator-chat
source/voice-note
status:blocked
status:codex-ready
status:merged:pending-evidence
status:needs-evidence
status:operator-needed
status:parked
tests
tier:0-platform-substrate
tier:1-iskra-value-layer
tier:2-tools-products-modules
type:bug
type:chore
type:docs
type:feat
type:policy
type:research
ui
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pdurlej/kan-ductor!147
No description provided.