Skip to content

fix: make agent reply cache lookup deterministic and fail closed#7

Merged
BobDLA merged 2 commits into
mainfrom
fix/reply-cache-determinism
Mar 24, 2026
Merged

fix: make agent reply cache lookup deterministic and fail closed#7
BobDLA merged 2 commits into
mainfrom
fix/reply-cache-determinism

Conversation

@BobDLA

@BobDLA BobDLA commented Mar 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • make Claude, Gemini, and Codex reply-cache lookup deterministic via direct hits plus metadata-backed alias resolution
  • fail closed for trusted-caller-only launches instead of probing unrelated agent caches
  • split desktop smoke coverage into portable and unix-only scenarios, and add regressions for trusted-caller-only and gemini pid-only launches

Validation

Ran

  • pnpm test -- src/app/hooks/useInitDocument.test.ts src/app/__tests__/useInitDocument.tauri.test.tsx
  • cargo test --manifest-path src-tauri/Cargo.toml -- --nocapture
  • pnpm build
  • pnpm tauri:build

Not run

  • pnpm test:e2e:desktop:unix
    • reason: package build was validated in this worktree, but the unix-only desktop smoke suite was not rerun after the final cleanup/build pass
  • pnpm lint
    • reason: this change was covered by targeted frontend tests, Rust tests, and production/package builds

@BobDLA BobDLA merged commit 362c0be into main Mar 24, 2026
1 check passed
@BobDLA BobDLA deleted the fix/reply-cache-determinism branch March 24, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant