Skip to content

Update tscircuit deps and remove Bun lockfile#237

Open
ar-amk wants to merge 1 commit into
tscircuit:mainfrom
ar-amk:codex/update-tscircuit-deps-lockfile-hygiene
Open

Update tscircuit deps and remove Bun lockfile#237
ar-amk wants to merge 1 commit into
tscircuit:mainfrom
ar-amk:codex/update-tscircuit-deps-lockfile-hygiene

Conversation

@ar-amk
Copy link
Copy Markdown

@ar-amk ar-amk commented May 21, 2026

/claim #79

Summary

  • Updated the tscircuit/circuit-json viewer and exporter stack to current published versions, including tscircuit, circuit-json, the PCB/schematic/3D viewers, and related export packages.
  • Removed the tracked bun.lock so the repo matches its existing bunfig.toml setting of [install.lockfile] save = false.
  • Added bun.lock and bun.lockb to .gitignore to prevent accidental lockfile reintroduction.

Validation

  • bun install completed and did not recreate bun.lock or bun.lockb.
  • bun run format
  • git diff --check
  • bun run build passed. Vite emitted the existing browser external/chunk-size warnings, but completed successfully.

Note

I also tried bunx tsc --noEmit; it currently fails on pre-existing docs/reference-implementation.tsx demo references such as ./use-store, src/parse-kicad-mod-to-circuit-json, convertCircuitJsonToTscircuit, and createSnippetUrl. I did not change that docs file because it is outside this dependency/build-hygiene scope.

Refs #79

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

@ar-amk is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

@ar-amk
Copy link
Copy Markdown
Author

ar-amk commented May 21, 2026

Vercel is showing a GitHub authorization failure URL rather than a build error on this fork preview. I verified the production build locally with bun run build; it completed successfully with only the existing dependency externalisation/chunk-size warnings noted in the PR body.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant