Skip to content

fix(context): handle literal search topics safely (#145) - #146

Open
TheRealBecks wants to merge 1 commit into
neuledge:mainfrom
TheRealBecks:TheRealBecks/issues145
Open

fix(context): handle literal search topics safely (#145)#146
TheRealBecks wants to merge 1 commit into
neuledge:mainfrom
TheRealBecks:TheRealBecks/issues145

Conversation

@TheRealBecks

Copy link
Copy Markdown
Contributor

Fixes #145.

Search topics such as "ExecStart and AND could trigger SQLite FTS syntax errors. This change safely quotes literal search terms while preserving paired double quotes for phrase searches. Unmatched quotes are ignored, punctuation separates words, Unicode keywords are preserved, and empty or punctuation-only topics return no results.

Adds local fixtures built through the real ingestion pipeline to verify:

  • Expected documents appear within the top three results.
  • GDScript and C# queries rank the intended language first.
  • Results preserve source attribution and code examples within the token budget.

Updates CLI documentation and MCP guidance, and includes a patch changeset.

Validation: frozen dependency install, lint, build, and all 300 tests pass.

@changeset-bot

changeset-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: f176ded

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@neuledge/context Patch
@neuledge/registry Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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.

Handle literal search topics safely and add retrieval regression fixtures

1 participant