Skip to content

Fix conversation discovery results and shared paging (JVNAUTOSCI-2754) - #644

Merged
witbrock merged 1 commit into
mainfrom
codex/von-254d4dbec332e931
Sep 12, 2026
Merged

witbrock merged 1 commit into
mainfrom
codex/von-254d4dbec332e931

Conversation

@witbrock

Copy link
Copy Markdown
Member

Conversation discovery currently rejects normal conversation_list output because the strict gateway schema omits the producer's ordering field. Accept that existing field without relaxing unknown-field validation.

Also preserve a continuation into the accepted-shared stream when the final owned page exactly fills the requested limit. Previously that boundary could silently omit shared candidates and report complete coverage.

Validation: 51 focused tests pass across discovery gateway, conversation management, search and turn evidence. The new parametrised list/search replay uses real producers and InternalMCPGateway.invoke with trusted actor context and fixture storage, then the ordinary-turn evidence compactor and cursor resolver. It traverses an empty named-only page, an unnamed owned result, and a shared result; verifies decoded source positions and eventual complete coverage; and rejects cross-actor cursors and prior-turn evidence. Existing naming, membership, duplicate and partial-coverage controls pass.

Cursor diagnosis: source keyset positions advance before visible-row filtering; therefore bounded empty pages can be expected. The fixture demonstrates progression, not merely changing signed cursor strings. No loop was demonstrated, and no pager was added. Existing turn evidence preserves coverage and exact cursor recovery without production changes. This does not establish a live corpus census or explain the precise contents of the historical 12 pages.

Merge decision: ready on Tier 1 evidence. No model requests, live data changes, or deployment performed by this worker. After merge the controller is to deploy the exact current-main SHA and verify public and local worker/backend activation, retaining rollback. Native task reporting is owned by the controller.

Tracking: JVNAUTOSCI-2754 / #V#task_agent_47da4c2d72f4b7f31ce71faf1cfbb0a4.

@witbrock
witbrock merged commit 77aeceb into main Sep 12, 2026
2 checks passed
@witbrock
witbrock deleted the codex/von-254d4dbec332e931 branch September 12, 2026 18:20
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