Skip to content

fix(cmdk): Restore Zendesk index to help search#114368

Merged
JonasBa merged 2 commits intomasterfrom
jb/cmdk/zendesk
Apr 29, 2026
Merged

fix(cmdk): Restore Zendesk index to help search#114368
JonasBa merged 2 commits intomasterfrom
jb/cmdk/zendesk

Conversation

@JonasBa
Copy link
Copy Markdown
Member

@JonasBa JonasBa commented Apr 29, 2026

Summary

  • Restores zendesk_sentry_articles search index to the CMDK help search, which was dropped during the JSX refactor
  • Adds limit={4} prop to the help action, consistent with other CMDK actions
  • Removes hardcoded .slice(0, 3) per-index cap in favor of the limit prop

Test plan

  • Open Command+K, type a help query, and verify Zendesk article results appear
  • Verify docs and develop results still appear
  • Verify result count is capped at 4

The CMDK help search was only querying docs and develop indexes,
dropping zendesk_sentry_articles and blog that the old HelpSource
included. Restore all four indexes and add a limit prop to the help
action consistent with other CMDK actions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@JonasBa JonasBa requested a review from a team as a code owner April 29, 2026 20:49
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 29, 2026
Blog results are not useful in the command palette help search.
Keep only docs, zendesk_sentry_articles, and develop indexes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@JonasBa JonasBa merged commit 07d1d83 into master Apr 29, 2026
67 checks passed
@JonasBa JonasBa deleted the jb/cmdk/zendesk branch April 29, 2026 21:16
cleptric pushed a commit that referenced this pull request May 5, 2026
## Summary
- Restores `zendesk_sentry_articles` search index to the CMDK help
search, which was dropped during the JSX refactor
- Adds `limit={4}` prop to the help action, consistent with other CMDK
actions
- Removes hardcoded `.slice(0, 3)` per-index cap in favor of the `limit`
prop

## Test plan
- [ ] Open Command+K, type a help query, and verify Zendesk article
results appear
- [ ] Verify docs and develop results still appear
- [ ] Verify result count is capped at 4

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants