docs: document the recall tool - #417
Merged
Merged
Conversation
ada-sen
marked this pull request as ready for review
September 26, 2026 03:04
…g '-' claim FTS5 has no leading-'-' exclusion: 'here -foo' fails with 'no such column: foo'. The tool description and the syntax-error hint both advertised '-' as the exclusion syntax. They now name NOT (e.g. 'deploy NOT staging') and say a leading '-' does not exclude. Tests: 'here NOT foo' excludes the foo row, and the 'here -foo' error hint no longer advertises '-' and points at NOT.
…on list - NOT is the exclusion operator; a leading '-' fails as column syntax. - read.context is a radius on each side of the target. - Surrounding tool calls cap at 10,000 characters with full: true. - The verbatim journal line is missing while a session is still being backfilled, not because the event predates the journal table. - thread caps each event at about 4,000 bytes and returns a clipped event as a truncated JSON string. - read reads transcript files; thread reads the event journal. - List the token prefixes redaction actually matches (only sk-proj- OpenAI keys).
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
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Documents the built-in
recalltool (actions, params, defaults, caps, indexed event types, redaction list, and error behaviors) atdocs/features/recall.md, matching the style ofdocs/features/reminders.md. PRI-3257.Review fixes (jc)
-exclusion (here -foofails withno such column: foo). The doc, the tool description, and the syntax-error hint inrecall.tsnow nameNOT(e.g.deploy NOT staging). New tests pin thathere NOT fooexcludes and that thehere -foohint points atNOT. This PR now changes tool text inpackages/agent, not only docs.read.contextis described as events on each side of the target.full: true, not uncapped.threadcaps each event at about 4,000 bytes; a clipped event comes back as a truncated JSON string.readreads transcript files;threadreads the verbatim event journal.sk-proj-OpenAI keys are covered.