fix(web): return graceful error for unknown search context

[rachit367]

Fixes #1312

Problem

Searching with context:<value> (e.g. Helpers context:0) crashes the request when no search context with that name exists.

context: resolves a named search context. In packages/web/src/features/search/parser.ts, onExpandSearchContext looks the name up and, when it's missing, throws a plain Error. The surrounding catch only maps SyntaxError to a ServiceErrorException; every other error is re-thrown as-is, so this one escapes the search action and surfaces as a server crash instead of a clean error.

Fix

Throw a ServiceErrorException with the existing SEARCH_CONTEXT_NOT_FOUND error code (HTTP 404) instead of a plain Error. sew() already serializes ServiceErrorException into a proper service-error response, so the user now gets a clear "Search context not found" message rather than a crashed request. All imports needed were already present.

Tests

Added parser.test.ts:

• unknown context → rejects with a ServiceErrorException carrying SEARCH_CONTEXT_NOT_FOUND (previously a plain Error / crash);
• existing context → expands into its repo set.

Both pass. tsc --noEmit is clean for the web package.

Summary by CodeRabbit

• Bug Fixes

  • Searching with a non-existent search context no longer crashes; the system now returns a graceful error message instead.

• Tests

  • Added test coverage for search context error handling to ensure proper error responses.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e3716733-03f8-49d7-ae27-0becfe833e9c

📥 Commits

Reviewing files that changed from the base of the PR and between 889e2b1 and 2f09e68.

📒 Files selected for processing (3)

• CHANGELOG.md
• packages/web/src/features/search/parser.test.ts
• packages/web/src/features/search/parser.ts

---

Walkthrough

In parser.ts, onExpandSearchContext now throws a ServiceErrorException (HTTP 404, SEARCH_CONTEXT_NOT_FOUND) instead of a generic Error when a referenced search context is not found. Two Vitest tests are added covering the error path and the successful repo-expansion path. CHANGELOG.md records the fix.

Search Context Error Handling

Layer / File(s)	Summary
Typed error throw and unit tests packages/web/src/features/search/parser.ts, packages/web/src/features/search/parser.test.ts, CHANGELOG.md	onExpandSearchContext throws ServiceErrorException with StatusCodes.NOT_FOUND and ErrorCode.SEARCH_CONTEXT_NOT_FOUND instead of a generic Error; tests assert the error code on a null Prisma lookup and verify correct repo IR on a successful lookup; CHANGELOG notes the crash fix.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: fixing graceful error handling for unknown search contexts instead of crashes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Caution

Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted.

Error details

{}