Skip to content

Add VS code marketplace extension#5

Merged
deb-adarsh merged 2 commits into
mainfrom
add-vscode-extension
May 17, 2026
Merged

Add VS code marketplace extension#5
deb-adarsh merged 2 commits into
mainfrom
add-vscode-extension

Conversation

@deb-adarsh
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings May 17, 2026 21:53
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a Visual Studio Code Marketplace extension for AI Stack Kit, alongside web-UI improvements (module-type filtering, URL-synced filters, toast feedback, getting-started panel, Docs link) and supporting CLI/schema changes that make the headless extension flow possible.

Changes:

  • New VS Code extension surface (commands, modules/outputs trees, catalog webview) plus CI workflow to package/publish it.
  • Web Skill browser re-themed as a Module browser with module-type filter, URL-synced filter state, copy toasts, getting-started steps, and empty state.
  • Core changes: SpecFileSchema no longer requires at least one entry (enables empty init), apply-pipeline gains forceReinstall, and new pipeline/spec-loader tests.
  • Docs (USER_GUIDE.md) and template (templates/spec.yaml) updated to describe the extension and the built-in cursor | copilot | claude adapters.

Reviewed changes

Copilot reviewed 61 out of 69 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
web/src/index.css Styles for new hero docs link, getting-started block, empty state, toast, and module-type tag.
web/src/App.tsx Adds URL-synced filters, module-type filter, copy toasts, getting-started UI, and Docs link.
USER_GUIDE.md Documents the new Marketplace extension, UI areas, settings, and dev workflow.
templates/spec.yaml Updates the client.type comment to reflect built-in adapters.
src/validation/schema.ts Removes the "at least one skill/module" superRefine to allow empty starter specs.
src/pipeline/apply-pipeline.ts Adds forceReinstall option that wipes the install dir before reinstall.
src/pipeline/tests/spec-loader.test.ts New tests for loadSpec including empty-skills and missing-client cases.
src/pipeline/tests/apply-dry-run.test.ts New dry-run test for the apply pipeline.
Files not reviewed (1)
  • extension/webview-ui/package-lock.json: Language not supported
Comments suppressed due to low confidence (2)

web/src/App.tsx:292

  • The stats line and the empty-state message now both say "modules", but the input's aria-label is still "Search skills" (line 277, just above this block) and the filter select for module type uses the select-client CSS class designed for the client dropdown. Update the accessible label to "Search modules" and/or rename the CSS class to keep the terminology and styling intent consistent now that the UI is a module browser, not a skill browser.
    web/src/App.tsx:236
  • The header still reads "Skill browser" (line 235), and the search input's aria-label is "Search skills", but the rest of the page has been re-themed around "modules" (stats line, empty-state, module-type filter, module-type tag). The mismatched terminology will be confusing to users and screen-reader users alike. Consider updating the sub-title and aria-label to "Module browser"/"Search modules" for consistency.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Comment thread web/src/App.tsx
Comment thread web/src/App.tsx
Comment thread web/src/App.tsx
Comment thread src/validation/schema.ts
Comment thread src/pipeline/apply-pipeline.ts
Comment thread web/src/App.tsx
@deb-adarsh deb-adarsh merged commit dc84cba into main May 17, 2026
1 check failed
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.

2 participants