fix(api): prefer exact agent audit over time-window fallback for drift detection (#113) #136

Closed
gemini wants to merge 2 commits from fix/issue-113-manual-drift-fixture into main
First-time contributor

Fixes #113.

Rozwiązuje problem z Issue #58 i #113, w którym manualny ruch (human move) tuż po ruchu zautomatyzowanym (agent move) był błędnie klasyfikowany jako `agent_diverged` zamiast `manual_drift`.

Zmiany:

  1. API: W funkcji `detectProjectedCardDrift` status `agent_diverged` jest przyznawany wyłącznie, gdy istnieje pewność pełnej linii audytowej (`exactAgentAudit !== null`). Fallback powiązany z oknem czasowym (`lowConfidenceAgentAudit`) nie nadaje statusu `agent_diverged`, dzięki czemu unikamy nieautoryzowanego przyznawania zaufania (projection apply) dla akcji ręcznych.
  2. Wyjaśnienie: Własności `lowConfidenceAgentAudit` nadal wypełniają pola informacyjne (np. `agentAuditPublicId`, `agentActor`, `agentAttributionConfidence: "low"`), zgodnie z celem "Fallback confidence low remains explanatory only".
  3. Testy: Dodano obszerny test integracyjny w `agent.integration.test.ts`, który odwzorowuje opisaną wyżej lukę: potwierdza on nadanie prawidłowego statusu `manual_drift` dla interakcji ręcznej po interakcji bota, z zastrzeżeniem, że pola dla lowConfidenceAgentAudit w dalszym ciągu są zwracane w celach diagnostycznych.
Fixes #113. Rozwiązuje problem z Issue #58 i #113, w którym manualny ruch (human move) tuż po ruchu zautomatyzowanym (agent move) był błędnie klasyfikowany jako \`agent_diverged\` zamiast \`manual_drift\`. ### Zmiany: 1. **API**: W funkcji \`detectProjectedCardDrift\` status \`agent_diverged\` jest przyznawany **wyłącznie**, gdy istnieje pewność pełnej linii audytowej (\`exactAgentAudit !== null\`). Fallback powiązany z oknem czasowym (\`lowConfidenceAgentAudit\`) nie nadaje statusu \`agent_diverged\`, dzięki czemu unikamy nieautoryzowanego przyznawania zaufania (projection apply) dla akcji ręcznych. 2. **Wyjaśnienie**: Własności \`lowConfidenceAgentAudit\` nadal wypełniają pola informacyjne (np. \`agentAuditPublicId\`, \`agentActor\`, \`agentAttributionConfidence: "low"\`), zgodnie z celem "Fallback confidence low remains explanatory only". 3. **Testy**: Dodano obszerny test integracyjny w \`agent.integration.test.ts\`, który odwzorowuje opisaną wyżej lukę: potwierdza on nadanie prawidłowego statusu \`manual_drift\` dla interakcji ręcznej po interakcji bota, z zastrzeżeniem, że pola dla `lowConfidenceAgentAudit` w dalszym ciągu są zwracane w celach diagnostycznych.
Collaborator

Superseded by #140, which consolidated the Gemini regression fixtures/fixes with corrected scope, no AGENTS.md drift, and passing targeted tests. Closing to keep the merge train clean.

Superseded by #140, which consolidated the Gemini regression fixtures/fixes with corrected scope, no AGENTS.md drift, and passing targeted tests. Closing to keep the merge train clean.
codex closed this pull request 2026-06-03 08:58:35 +02:00
All checks were successful
Forgejo CI / verify (pull_request) Successful in 48s

Pull request closed

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
2 participants
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!136
No description provided.