Skip to content

Hide Skills UI: remove Settings Skills section + Usage skills widget - #142

Merged
yikkuro merged 1 commit into
mainfrom
sunt-microsoft-feat-hide-skills-ui
Jul 30, 2026
Merged

Hide Skills UI: remove Settings Skills section + Usage skills widget#142
yikkuro merged 1 commit into
mainfrom
sunt-microsoft-feat-hide-skills-ui

Conversation

@yikkuro

@yikkuro yikkuro commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

Step 3 of the hide Skills feature. This is a renderer-only UI removal (branched off main, independent of the earlier steps). It removes the three user-facing Skills management surfaces so users never see the "Skills" concept.

Removed

  • Settings → Skills section in SettingsView.vue: the nav item + icon, the entire section template, skills from VALID_SECTIONS, the auto-refresh watcher branch, the onMounted loader, and all now-unused skill state/logic (built-in/custom lists, toggles, allowlist/managed-entry helpers, askAgent flow). Also dropped the now-unused router/useAgentStore.
  • Usage panel skills widget in UsagePanel.vue: the section, chips, empty state, and the "view more skills" button, plus the loadSkills/goSkillSettings logic and skill CSS.
  • i18n: the orphaned settings.* skills-management keys, settings.menu.skills, and usage.*Skills keys, removed from both en-US and zh-CN.

Intentionally kept (out of scope)

  • The skill-integrity / tamper-detection dialog in App.vue and all integrity.* keys — it is a security gate.
  • The window.openclaw.skills IPC surface (the integrity dialog still uses it).
  • sidebar.skills (misnamed — it maps to "Channels").
  • All backend / gateway / skill-scanning code.

Validation

  • Renderer vue-tsc -b + vite build: clean
  • Renderer vitest: 198/198 pass
  • Main-process tsc --noEmit: clean
  • Main-process vitest: all pass
  • ESLint on changed files: 0 errors (only pre-existing warnings)

Step 3 of the hide-Skills feature. Renderer-only removal of the three
user-facing Skills management surfaces:
- Settings > Skills section (nav item, section, and all skill list/toggle logic)
- Usage panel skills widget (chips, view-more button, and loaders)
- Orphaned settings.* and usage.* skills i18n keys in en-US and zh-CN

The skill-integrity / tamper-detection dialog and the window.openclaw.skills
IPC surface it depends on are intentionally kept.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 1fe75171-fea2-45ef-9401-bf8a4ea0f2ba
@yikkuro
yikkuro merged commit 9464d1c into main Jul 30, 2026
1 of 2 checks passed
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