Skip to content

Fix scan config page - #42

Open
chzh12 wants to merge 4 commits into
mainfrom
Fix-Scan-Config-page
Open

Fix scan config page#42
chzh12 wants to merge 4 commits into
mainfrom
Fix-Scan-Config-page

Conversation

@chzh12

@chzh12 chzh12 commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

execsumo added a commit to execsumo/skill-manager that referenced this pull request Jul 24, 2026
Cherry-picks two unmerged upstream fixes from mode-io's open PR mode-io#42
(Fix-Scan-Config-page), adapted for our fork's layout:

- McpHarnessLogoStack: show writable-but-disabled harnesses (dimmed)
  so MCP server cards mirror the Skills in-use coverage UI.
- SkillInUseCard: replace the read-only HarnessChipStack with an
  interactive HarnessToggleStack so harnesses can be enabled/disabled
  directly from the card.
- Scan config delete confirmation: replace window.confirm with the
  app's ConfirmActionDialog pattern. Re-targeted at
  ScanConfigSection.tsx (under Settings) since our fork moved scan
  config there after 9eca1a9; upstream's patch still targeted the
  old standalone ScanConfigPage.tsx.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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