FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Invalidate managed read cache after writes by mostlydev · Pull Request #52 · mostlydev/cllama · GitHub

Invalidate managed read cache after writes - #52

Open
mostlydev wants to merge 2 commits into
masterfrom
fix-managed-tool-read-after-write
Open

Invalidate managed read cache after writes#52
mostlydev wants to merge 2 commits into
masterfrom
fix-managed-tool-read-after-write

Conversation

Copy link
Copy Markdown
Owner

Summary

  • invalidate cached GET/HEAD/OPTIONS managed-tool results after a successful mutation
  • also invalidate after an ambiguous mutating failure that may have committed
  • preserve duplicate suppression for mutations and for repeated reads when no write intervenes

Why

A live get_trigger -> update_trigger -> get_trigger sequence replayed the pre-update cached GET in the final round. PostgreSQL and the update receipt had current state, but the model saw stale state and reported an apparent API revert. Read-after-write must execute again.

Verification

  • go test ./internal/proxy
  • go test ./...
  • regression test proves the first post-mutation read executes and subsequent duplicates replay the refreshed result

Observed live intervention: duplicate_managed_tool_call:trading-api.get_trigger immediately after managed_tool_schema_rejected:trading-api.update_trigger.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL