refactor: remove Node.js toolchain and rewrite plugin validation

Replace the Bun/TypeScript validate-plugin.ts with a shell script that
validates .claude-plugin/plugin.json. Remove package.json, bun.lock,
lefthook.yml, and node_modules since no plugins use Node.js anymore.
Simplify CI to only run plugin validation without bun setup.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: retsohuang

.github/workflows/ci.yml

@@ -5,28 +5,6 @@ on:
     branches: [main]
 
 jobs:
-  check:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - uses: oven-sh/setup-bun@v2
-        with:
-          bun-version: latest
-
-      - name: Install dependencies
-        run: bun install --frozen-lockfile
-
-      - name: Build
-        run: bun run build
-
-      - name: Type check
-        run: bun run typecheck
-
-      - name: Test
-        run: bun test
-
   validate-plugins:
     runs-on: ubuntu-latest
 
@@ -35,13 +13,6 @@ jobs:
         with:
           fetch-depth: 0
 
-      - uses: oven-sh/setup-bun@v2
-        with:
-          bun-version: latest
-
-      - name: Install dependencies
-        run: bun install --frozen-lockfile
-
       - name: Get modified plugins
         id: changed-plugins
         run: |
@@ -57,9 +28,9 @@ jobs:
           failed=0
           while IFS= read -r plugin_dir; do
             [ -z "$plugin_dir" ] && continue
-            if [ -f "${plugin_dir}/plugin.json" ]; then
+            if [ -f "${plugin_dir}/.claude-plugin/plugin.json" ]; then
               echo "Validating: $plugin_dir"
-              if ! bun scripts/validate-plugin.ts "$plugin_dir"; then
+              if ! bash scripts/validate-plugin.sh "$plugin_dir"; then
                 failed=1
               fi
             fi

.gitignore

@@ -12,12 +12,6 @@ Thumbs.db
 .mcp.json
 .worktreeinclude.yaml
 
-# Node modules (if plugins use npm)
-node_modules/
-npm-debug.log*
-yarn-debug.log*
-yarn-error.log*
-
 # Environment files
 .env
 .env.local
@@ -33,7 +27,5 @@ yarn-error.log*
 # Build outputs
 build/
 *.log
-*.bun-build
-*/dist/
 .spectra/
 openspec/.vector-search.db

CLAUDE.md

@@ -4,7 +4,7 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 ## Project Overview
 
-Retso Marketplace is a custom plugin marketplace for Claude Code. It's a Bun monorepo containing plugins and the marketplace registry.
+Retso Marketplace is a custom plugin marketplace for Claude Code, containing plugins and the marketplace registry.
 
 ## Version Bumping
 
@@ -24,13 +24,13 @@ plugins/
 ## Plugin Structure
 
 Each plugin contains:
-- `plugin.json` - Metadata: name, version, commands[], agents[], skills[]
+- `.claude-plugin/plugin.json` - Metadata: name, version, commands[], agents[], skills[]
 - `commands/*.md` - Slash commands with YAML frontmatter (description, allowed-tools, model)
 - `agents/*.md` - Sub-agents for parallel processing with JSON input/output contracts
 - `skills/*/SKILL.md` - Auto-invoked skills with their own scripts/assets
 
 ## Adding a New Plugin
 
-1. Create `plugins/<name>/` with `plugin.json`
+1. Create `plugins/<name>/` with `.claude-plugin/plugin.json`
 2. Add entry to `.claude-plugin/marketplace.json` plugins array
 3. Update root `README.md` Available Plugins table

README.md

@@ -126,7 +126,8 @@ A typical plugin in this marketplace should follow this structure:
 
 ```
 plugins/my-plugin/
-├── plugin.json          # Plugin metadata (optional if strict: false)
+├── .claude-plugin/
+│   └── plugin.json      # Plugin metadata
 ├── commands/            # Slash commands
 │   └── my-command.md
 ├── agents/              # AI agents

bun.lock

@@ -1,36 +1,38 @@
 ## Context
 
-The `cli/` directory contains a React + Ink terminal UI (`plugin-kit`) for interactive plugin installation. It was the only way to install plugins before Claude Code added native `--plugin-dir` and marketplace support. The CLI is now unused but still referenced in workspaces, CI, README, and project docs.
-
-The `cli/scripts/validate-plugin.ts` script is independently useful — it validates `plugin.json` manifests in CI — and must be preserved.
+The `cli/` directory has been removed (phase 1 complete). However, the CI `check` job now fails because `bun run --filter '*' build` finds no matching workspaces. More broadly, the entire Node.js/Bun toolchain is dead weight — no plugins use node/bun, and the validate script checks for `plugin.json` at the plugin root which no longer exists. Plugins now store metadata at `.claude-plugin/plugin.json`.
 
 ## Goals / Non-Goals
 
 **Goals:**
 
-- Remove all CLI installer code and its dependencies
-- Preserve the plugin validation script for CI use
-- Clean up all references to the CLI across project files
+- Fix the CI build failure
+- Remove all Node.js/Bun toolchain files (`package.json`, `bun.lock`, `lefthook.yml`, `node_modules/`)
+- Replace the TypeScript validation script with a shell script that validates `.claude-plugin/plugin.json`
+- Update documentation to reflect the current plugin structure
 
 **Non-Goals:**
 
 - Changing any plugin structure or marketplace behavior
-- Modifying the validation logic itself
-- Removing CLI-related dependencies from root `package.json` (only workspace removal)
+- Adding new CI checks beyond plugin validation
 
 ## Decisions
 
-### Inline the Zod schema into validate-plugin.ts
+### Replace validate-plugin.ts with shell script
+
+The TypeScript validation script requires Bun and Zod. Since we're removing the entire Node.js toolchain, rewrite it as `scripts/validate-plugin.sh` using `jq` (pre-installed on Ubuntu CI runners). The new script validates `.claude-plugin/plugin.json` instead of root `plugin.json`.
+
+**Alternative**: Keep Bun just for the validate script. Rejected because maintaining an entire runtime for one small script is excessive.
 
-The validate script imports `PluginManifestSchema` from `cli/src/types/plugin.ts`. Rather than keeping a separate types file, inline the schema directly into `scripts/validate-plugin.ts`. The schema is small (~15 lines) and the validate script is its only remaining consumer.
+### Remove the check CI job entirely
 
-**Alternative**: Create `scripts/types/plugin.ts` as a separate file. Rejected because it adds unnecessary file structure for a single small schema.
+The `check` job ran `build`, `typecheck`, and `test` — all delegating to workspaces via `--filter '*'`. No workspaces exist anymore. Remove the job entirely rather than trying to make it work.
 
-### Delete cli/ entirely before relocating
+### Remove lefthook pre-commit hooks
 
-Delete the entire `cli/` directory first, then create the new `scripts/validate-plugin.ts` with inlined schema. This is cleaner than a partial move.
+The lefthook hooks only trigger on `plugins/*/scripts/**/*.{ts,js}` files which no longer exist. Remove `lefthook.yml` entirely.
 
 ## Risks / Trade-offs
 
-- [Removing install method] → Users who relied on `plugin-kit` CLI lose that path. Mitigated by `--plugin-dir` and marketplace being strictly better alternatives.
-- [CI breakage during transition] → If CI runs between deleting `cli/` and updating the workflow path. Mitigated by making all changes in a single commit.
+- [No pre-commit hooks] → Losing lint/format checks on commit. Acceptable because no JS/TS plugin code exists to lint.
+- [jq dependency in CI] → `jq` is pre-installed on GitHub Actions ubuntu runners, so no additional setup needed.

lefthook.yml

@@ -4,11 +4,11 @@ The CLI installer (`plugin-kit` / `pk`) was built before Claude Code supported `
 
 ## What Changes
 
-- **Remove** the entire `cli/` directory (React + Ink terminal UI, build scripts, install script)
-- **Relocate** `cli/scripts/validate-plugin.ts` to a top-level `scripts/` directory (still used by CI)
-- **Inline** the `PluginManifestSchema` into the relocated validate script (eliminating the dependency on `cli/src/types/plugin.ts`)
-- **Update** project configuration: `package.json` workspaces, `.gitignore`, CI workflow path, README, CLAUDE.md
-- **Regenerate** `bun.lock` without CLI dependencies
+- **Remove** the entire `cli/` directory (React + Ink terminal UI, build scripts, install script) *(done)*
+- **Remove** the entire Node.js/Bun toolchain: `package.json`, `bun.lock`, `node_modules/`, `lefthook.yml`
+- **Replace** `scripts/validate-plugin.ts` (Bun + Zod) with `scripts/validate-plugin.sh` (shell + jq), validating `.claude-plugin/plugin.json` instead of root `plugin.json`
+- **Rewrite** CI workflow: remove `check` job, simplify `validate-plugins` to use shell script without bun
+- **Update** documentation: README Plugin Structure section, CLAUDE.md, `.gitignore`
 
 ## Capabilities
 
@@ -22,6 +22,6 @@ The CLI installer (`plugin-kit` / `pk`) was built before Claude Code supported `
 
 ## Impact
 
-- Affected code: `cli/` (deleted), `scripts/validate-plugin.ts` (new location), `.github/workflows/ci.yml`, `package.json`, `README.md`, `CLAUDE.md`, `.gitignore`, `bun.lock`
-- Dependencies removed: `ink`, `react`, `ink-text-input`, `yaml` (CLI-only dependencies)
+- Affected code: `cli/` (deleted), `scripts/validate-plugin.ts` → `scripts/validate-plugin.sh`, `.github/workflows/ci.yml`, `package.json` (deleted), `bun.lock` (deleted), `lefthook.yml` (deleted), `README.md`, `CLAUDE.md`, `.gitignore`
+- All npm dependencies removed — project no longer uses Node.js/Bun runtime
 - No plugin functionality affected — marketplace and `--plugin-dir` workflows remain unchanged

openspec/changes/delete-cli-installation/design.md

@@ -3,3 +3,7 @@
 ### Requirement: CLI plugin installer
 **Reason**: Replaced by Claude Code's native `--plugin-dir` flag and marketplace installation
 **Migration**: Use `claude --plugin-dir ./plugins/<name>` for local testing, or `/plugin marketplace add` for marketplace installation
+
+#### Scenario: CLI binary no longer available
+- **WHEN** user attempts to run `plugin-kit` or `pk`
+- **THEN** the command SHALL NOT be available as the binary is no longer built or distributed

openspec/changes/delete-cli-installation/proposal.md

@@ -20,4 +20,33 @@
 
 - [x] 4.1 Run `bun run typecheck` — should pass
 - [x] 4.2 Run `bun test` — should pass
-- [x] 4.3 Commit all changes: `refactor: remove CLI installer in favor of native plugin-dir support`
+- [x] 4.3 Commit all changes: `refactor: remove CLI plugin installer in favor of native plugin-dir support`
+
+## 5. Replace validate-plugin.ts with shell script
+
+- [x] 5.1 Delete `scripts/validate-plugin.ts`
+- [x] 5.2 Create `scripts/validate-plugin.sh` using jq to validate `.claude-plugin/plugin.json` (check required fields: name matches directory, description, version semver, referenced commands/agents files exist)
+- [x] 5.3 Verify `bash scripts/validate-plugin.sh plugins/github-tools` runs successfully
+
+## 6. Remove Node.js toolchain
+
+- [x] 6.1 Delete `package.json`
+- [x] 6.2 Delete `bun.lock`
+- [x] 6.3 Remove lefthook pre-commit hooks — delete `lefthook.yml`
+- [x] 6.4 Delete `node_modules/` directory
+- [x] 6.5 Clean up `.gitignore` — remove `node_modules/` and npm/yarn log entries
+
+## 7. Rewrite CI workflow
+
+- [x] 7.1 Remove the check CI job entirely from `.github/workflows/ci.yml`
+- [x] 7.2 Update `validate-plugins` job: remove bun setup and `bun install`, use `bash scripts/validate-plugin.sh` instead
+
+## 8. Update documentation
+
+- [x] 8.1 Update Plugin Structure section in `README.md` — change `plugin.json` at root to `.claude-plugin/plugin.json`
+- [x] 8.2 Update `CLAUDE.md` — remove any remaining bun/node references
+
+## 9. Verification
+
+- [ ] 9.1 Push and confirm CI passes on PR #17
+- [x] 9.2 Commit all changes