Skip to content

🗑️ chore: delete CLI installer and Node.js toolchain - #18

Merged
retsohuang merged 1 commit into
mainfrom
feat/delete-cli-installtion
Mar 1, 2026
Merged

🗑️ chore: delete CLI installer and Node.js toolchain#18
retsohuang merged 1 commit into
mainfrom
feat/delete-cli-installtion

Conversation

@retsohuang

Copy link
Copy Markdown
Owner

Summary

This PR completes the removal of the CLI installer and entire Node.js/Bun toolchain from the project. After deleting the CLI in PR #17, this change removes all remaining Node.js dependencies and dead weight.

Changes

  • 🗑️ Deleted package.json, bun.lock, and node_modules directory
  • 🗑️ Removed lefthook.yml pre-commit hook configuration
  • 🗑️ Removed unused scripts/validate-plugin.ts (Bun-based)
  • ✨ Created scripts/validate-plugin.sh (shell+jq alternative)
  • ✨ Rewrote .github/workflows/ci.yml to validate .claude-plugin/plugin.json
  • 📝 Updated README.md Plugin Structure documentation
  • 📝 Updated CLAUDE.md project architecture documentation
  • 🙈 Cleaned up .gitignore to remove Node.js-related entries
  • 🗑️ Removed orphaned config files (.oxlintrc.json, .prettierrc)

Completed Work

All 30 OpenSpec tasks have been completed and the change has been archived:

  • ✅ All plugin validation logic migrated from TypeScript+Zod to shell+jq
  • ✅ CI pipeline simplified (removed build/typecheck/test job)
  • ✅ Plugin metadata validation now checks .claude-plugin/plugin.json structure
  • ✅ Project documentation updated to reflect new plugin locations
  • ✅ Verification and optimization suggestions implemented

Related


🤖 Generated with Claude Code

Archived the completed OpenSpec change that removed the CLI installation tool and Node.js/Bun toolchain from the project:

- Deleted package.json, bun.lock, and node_modules
- Removed lefthook.yml pre-commit configuration
- Removed unused config files (.oxlintrc.json, .prettierrc)
- Replaced TypeScript+Zod validation with shell+jq script
- Updated .github/workflows/ci.yml to validate .claude-plugin/plugin.json
- Updated documentation to reflect plugin structure changes
- All 30 tasks completed successfully

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 1, 2026 13:13
@retsohuang
retsohuang merged commit 6b5e723 into main Mar 1, 2026
3 checks passed
@retsohuang
retsohuang deleted the feat/delete-cli-installtion branch March 1, 2026 13:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an OpenSpec archive entry documenting the completed removal of the legacy CLI installer and associated Node.js/Bun toolchain work, capturing the proposal/design/spec and task checklist for future reference.

Changes:

  • Added archived OpenSpec documentation for the “delete CLI installation” initiative (proposal/design/spec/tasks).
  • Marked the archived change with archived_by / archived_at metadata.

Reviewed changes

Copilot reviewed 1 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
openspec/changes/archive/2026-03-01-delete-cli-installation/tasks.md Archived execution checklist for the CLI/toolchain removal work.
openspec/changes/archive/2026-03-01-delete-cli-installation/specs/cli-removal/spec.md Archived spec describing the removed CLI installer requirement and migration guidance.
openspec/changes/archive/2026-03-01-delete-cli-installation/proposal.md Archived proposal summarizing the motivation and intended repo changes.
openspec/changes/archive/2026-03-01-delete-cli-installation/design.md Archived design notes capturing decisions and trade-offs.
openspec/changes/archive/2026-03-01-delete-cli-installation/.openspec.yaml Archive metadata for the change record.

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