Skip to content

Remove deprecated convert CLI command in favour of export#3216

Draft
rushabhcodes wants to merge 1 commit into
tscircuit:mainfrom
rushabhcodes:codex/remove-deprecated-convert-command
Draft

Remove deprecated convert CLI command in favour of export#3216
rushabhcodes wants to merge 1 commit into
tscircuit:mainfrom
rushabhcodes:codex/remove-deprecated-convert-command

Conversation

@rushabhcodes
Copy link
Copy Markdown
Contributor

@rushabhcodes rushabhcodes commented Jun 5, 2026

Summary

Removes the deprecated tsci convert command from the CLI in favour of tsci export and cleans up the direct dependencies that were only used by that command.

Changes

  • removes registerConvert from the CLI entrypoint
  • deletes the deprecated cli/convert/register.ts command implementation
  • deletes the dedicated convert CLI test
  • removes circuit-json-to-tscircuit from direct dependencies
  • removes kicad-component-converter from direct dependencies
  • updates CLI help output in the README to no longer mention convert

Why

The convert command is deprecated and no longer needed in the CLI surface. Removing it also lets this package drop the direct dependencies that only existed to support that command.

Validation

  • ran bunx tsc --noEmit
  • ran bun run build
  • confirmed bun ./cli/main.ts --help no longer lists convert
  • confirmed circuit-json-to-tscircuit is no longer referenced anywhere in this repo

Notes

  • kicad-component-converter may still appear in bun.lock as a transitive dependency through tscircuit, even though it is no longer a direct dependency of this package

@rushabhcodes rushabhcodes changed the title [codex] Remove deprecated convert CLI command and unused direct dependencies Remove deprecated convert CLI command in favour of export Jun 5, 2026
@rushabhcodes rushabhcodes marked this pull request as ready for review June 5, 2026 04:25
Copilot AI review requested due to automatic review settings June 5, 2026 04:25
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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@rushabhcodes rushabhcodes marked this pull request as draft June 5, 2026 04:29
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