Repository navigation
feat(recall): share contextual ranking and budgeted display profiles - #150
Merged
Merged
Conversation
Prepare source-bound automatic recall fragments through InformationView, include guidance in the rendered budget, and fence cancelled or superseded preparations. Keep unconfirmed host assembly out of the disclosure ledger and document the remaining final-delivery boundary. Validation: targeted lifecycle regressions and all 16 blocking agent:verify checks passed; product suite passed 2077 tests with 2 skipped.
Reorder session-bound retrieval pages with bounded recent user context while preserving candidate membership and scores. Share compact and full automatic statement views through InformationView and expose daemon-local session controls. Validate source-bound previews, topic isolation and CLI round trips without live providers.
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
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.
Change summary
What (what changed):
Automatic recall now shares source-bound InformationView preparation across Pi, OpenCode, DSH and WorkBuddy. Session-bound search reorders its existing candidate page using the current request and bounded recent user context, independently of display mode. Automatic display defaults to at most ten candidates with 160-code-point statement previews; full mode retains statement bodies within the shared context budget.
Why (why it changed / what problem it addresses):
Unrelated long engineering memories can crowd out the current conversation. Shared ranking and display profiles reduce that exposure without deleting retrieval candidates or adding provider calls. Automatic adapters also stop recording disclosure before host delivery, and historical DSH UI snapshots cannot become a fresh turn's injected recall.
Changes (key changes, by file or module):
nmg recall display --session-id ID [--mode compact|full]; settings are daemon-local and session-specific.agent:verify; product tests: 2,082 passed, 2 skipped, zero failures. An isolated resident daemon verified CLI reads, mode changes and session isolation.Documentation owners:
docs/decisions/implemented/2026-10-07-shared-information-output.{md,zh-CN.md}.docs/design/state-spaces.{md,zh-CN.md},docs/design/recall-context-and-display.{md,zh-CN.md,terms.yaml},docs/design/session-memory-lifecycle-hooks-2026-08-10/automatic-recall-budget.{md,zh-CN.md}, and the feature-register/shared-display-options document groups underdocs/design/hidden-features-registry/.skills/nmg-memory/references/; adapter README updates indsh/,opencode-plugin/andworkbuddy-plugin/.Not verified
Not verified (what was not verified):
No live LLM/embedding benchmark or natural multi-turn relevance/answer-quality evaluation was run. Keeping all retrieved IDs preserves whole-page membership; it does not prove recall at a smaller cutoff or improvement in the first ten results. Production daemon deployment and live host GUI behavior were not exercised. Research/chaos advisory suites were not run. The existing production daemon was not restarted.
Completion checklist
Local quality checks
verify:staticcomponent checks passed via the 16-checkagent:verifyroute (build, package, type, tests types, lock, lint, format, docs, context routes, complexity, packages, glossary, policy words, RTM and mutation anchors).npm run test:product: 2,082 passed, 2 skipped, zero failures.check:lockpassed.verify:packages.docs:checkpassed; the existing unrelated embedding-deadline bilingual-link warning remains.RCP (Repository Control Plane)
agent:verify; local evidence in.nmg/verification/latest.json.nmg-rcp forge-status --pr <number>.CI completion confirmation
All checks passedfrom RCP forge observation.