Skip to content

feat(desktop): integrate cloud workspace skills into UI - #628

Merged
Ooooze merged 5 commits into
rel/xp-buildfrom
codex/desktop-workspace-skills
Oct 9, 2026
Merged

Ooooze merged 5 commits into
rel/xp-buildfrom
codex/desktop-workspace-skills

Conversation

@Ooooze

@Ooooze Ooooze commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Desktop previously read the boot-time skills registry without a session and a separate legacy CLI list, so cloud project skills such as openspec-explore were absent from the UI. Integrate #627 into rel/xp-build and use the selected cloud session's live workspace catalog for the Skills pane, command palette and slash completion, including before the first message.

Skills show their chosen source and disabled reasons. Detail controls can disable a name in this workspace or globally, and the toolbar can disable all project sources while preserving global skills. Submission rechecks the catalog before sending the ordinary skill.view instruction. Catalog, detail and setting responses cannot replace a newer session's view; background catalog reads update the relevant surface without rebuilding the composer. Choosing another startup folder starts a new cloud chat, so a conversation bound to A cannot be mislabeled as B. Local mode retains its legacy catalog and restart semantics.

HTTP skills/capabilities add explicit workspace=true preview for an unsaved new session in the server's startup directory. Optional sessionId selects an existing session. No renderer-supplied filesystem path is accepted; context-free API calls retain their previous behavior.

Validation:

  • Full Vitest: 1,129 files, 13,668 tests passed; 4 skipped, no quarantine exclusions.
  • Desktop unit tests: 180 passed.
  • Manual Electron check against an existing OpenSpec workspace: all six project skill commands appear before the first message.
  • Real Electron E2E (workspace-skills.drive.mjs): pre-chat discovery, duplicate/source selection, all three disable scopes, palette and slash completion, stale invocation after a CLI policy edit, real runtime skill.view then reply, isolation in a second workspace.
  • Windows smoke CI: both gates passed with the existing known-failure registry; x64 739 passed / 72 known failures, arm64 738 passed / 74 known failures, no unexpected failures.
  • Root lint, typecheck:tests (773 existing debt diagnostics, no new errors), imports:check, docs:check and build passed; desktop lint/build passed.

E2E inference uses a deterministic loopback fixture. A real provider/model evaluation and packaging/signing are not part of these checks.

@Ooooze
Ooooze merged commit 480b15a into rel/xp-build Oct 9, 2026
2 checks passed
sosidudku1 added a commit that referenced this pull request Oct 12, 2026
…rkspace skills) into the Atomic Core desktop branch
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