app-server README documents query on fuzzyFileSearch/sessionCompleted

Resolved 💬 2 comments Opened Jun 1, 2026 by bar-d Closed Jun 2, 2026
💡 Likely answer: A maintainer (etraut-openai, contributor) responded on this thread — see the highlighted reply below.

What happened?

The app-server README says:

fuzzyFileSearch/sessionCompleted — { sessionId, query }

The current protocol, generated TypeScript schema, and implementation all show the completed notification carrying only sessionId:

{ sessionId: string }

query is present on fuzzyFileSearch/sessionUpdated, but not on fuzzyFileSearch/sessionCompleted.

Expected behavior

The README should document fuzzyFileSearch/sessionCompleted as { sessionId }, or the protocol/implementation should include query if that field is intended to be part of the completed notification.

Additional context

I searched open issues and PRs for terms including fuzzyFileSearch/sessionCompleted, query, sessionId, and app-server README, and did not find an existing open item that appears to cover this documentation mismatch.

View original on GitHub ↗

2 Comments

etraut-openai contributor · 2 months ago

@bar-d and @0disoft, please stop posting low-value AI-generated bug reports and responses to our issue tracker. You're generating a lot of noise for maintainers. This is not welcome. If your bots continue to post, I'll report to Github and ask them to block you.

0disoft · 2 months ago

Apologies for the noise. I’ve removed my comments and fork branches, and I’ll stop posting low-value or automated contribution notes to this issue tracker.