Skip to content

feat: add category_icon field to PluginSearchResult#288

Draft
christophkroeppl wants to merge 1 commit into
pop-os:masterfrom
christophkroeppl:claude/category_icon_per_PluginSearchResult
Draft

feat: add category_icon field to PluginSearchResult#288
christophkroeppl wants to merge 1 commit into
pop-os:masterfrom
christophkroeppl:claude/category_icon_per_PluginSearchResult

Conversation

@christophkroeppl

@christophkroeppl christophkroeppl commented Feb 28, 2026

Copy link
Copy Markdown
Contributor

Allow individual search results to optionally override the plugin-level category icon.

  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • [TODO_ADD_SCREENSHOTS] My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

@christophkroeppl christophkroeppl force-pushed the claude/category_icon_per_PluginSearchResult branch from 457b62d to a61f672 Compare February 28, 2026 06:42
Allow individual search results to optionally override the plugin-level
category icon. The service conversion logic now prefers a per-result
`category_icon` over the plugin config icon, falling back to the plugin
icon when no per-result override is set.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@christophkroeppl christophkroeppl force-pushed the claude/category_icon_per_PluginSearchResult branch from a61f672 to 947a24e Compare February 28, 2026 06:48
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