Skip to content

fix v4.1: restore the Search with AI option on the Community Cloud tab - #601

Merged
lightbulbman merged 1 commit into
core-betafrom
fix/restore-ai-cloud-search/core
Oct 19, 2026
Merged

lightbulbman merged 1 commit into
core-betafrom
fix/restore-ai-cloud-search/core

Conversation

@lightbulbman

Copy link
Copy Markdown
Contributor

Summary

The "Search with AI" option disappeared from the search method dropdown on the Community Cloud tab. A later merge kept an older version of CloudSearch.tsx, which removed the option and put back an onChange handler that only accepts term and codevault. The rest of the feature was unaffected: parseSearchMethod, the searchMethod=ai request parameter, and the REST mapping to s_method=ai.

Changes

  • Restore the "Search with AI" option and pass the selected value through parseSearchMethod.
  • Write any search method other than keyword to the by query parameter, so an AI search survives a page reload. Before this change only codevault was written.
  • Add an end-to-end test that selects Search with AI, checks that the request sends searchMethod=ai and the URL holds by=ai, then reloads and checks that the method is kept.

Verification

  • The new test fails before the fix (the option is missing) and passes after it.
  • Playwright: code-snippets-community-featured, code-snippets-pro-pages and cloud-download-eligibility pass (44 tests).
  • ESLint passes on the changed files. TypeScript reports no new errors.
  • Not verified against the live cloud AI search endpoint. The test mocks the search response.

@lightbulbman lightbulbman changed the title fix: restore the Search with AI option on the Community Cloud tab fix v4.1: restore the Search with AI option on the Community Cloud tab Oct 16, 2026
@lightbulbman
lightbulbman merged commit 9179336 into core-beta Oct 19, 2026
11 checks passed
@lightbulbman
lightbulbman deleted the fix/restore-ai-cloud-search/core branch October 19, 2026 13:21
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