Skip to content

Add select-all checkbox for tool category groups in MCP key form - #14

Merged
ruibaby merged 1 commit into
mainfrom
feat/select-all-tools-by-category
Oct 10, 2026
Merged

ruibaby merged 1 commit into
mainfrom
feat/select-all-tools-by-category

Conversation

@ruibaby

@ruibaby ruibaby commented Oct 10, 2026

Copy link
Copy Markdown
Member

What this PR does?

Adds a checkbox to each tool category header in the MCP access key form, allowing users to select or deselect all tools within a category at once. The checkbox shows an indeterminate state when only part of the category is selected.

The change applies to both the creation and editing modals, as they share the same AccessKeyForm component.

Why we need it?

Categories like post management contain many tools. Previously users had to toggle each tool individually; now a whole category can be granted or revoked with one click.

How to test it?

  1. Open the "Create MCP key" modal in Console.
  2. Click a category checkbox (e.g. 文章管理) and verify all tools in that category are selected, and the count updates accordingly.
  3. Deselect one tool in the category and verify the category checkbox shows the indeterminate state.
  4. Submit the form and verify the saved key's allowedTools matches the selection.
  5. Repeat in the edit modal for an existing key.

pnpm check (Prettier, Oxlint/ESLint, vue-tsc, Vitest) passes locally.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-10T09:19:10.434166Z fa0c30d PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@ruibaby
ruibaby merged commit 4213634 into main Oct 10, 2026
1 check passed
@ruibaby
ruibaby deleted the feat/select-all-tools-by-category branch October 10, 2026 13:58
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