Replace hand-built board with XiaoBoard from @tscircuit/common - #39
Open
js360000 wants to merge 1 commit into
Open
Replace hand-built board with XiaoBoard from @tscircuit/common#39js360000 wants to merge 1 commit into
js360000 wants to merge 1 commit into
Conversation
…scircuit#2) Adopt the shared XIAO breakout from the new `@tscircuit/common` module instead of maintaining the bespoke `PinOutCircuit` stamp-board chip inline. The Waveshare RP2040-Zero is a XIAO-form-factor board, so the upstream component already encodes the correct edge pad layout, pin labels, schematic arrangement, and plated-hole geometry. Changes: - Add `@tscircuit/common@^0.0.42` to dependencies. - `index.tsx`: replace `<board>` + `<PinOutCircuit />` with `<XiaoBoard variant="RP2040" withPlatedHoles boardProps={…}>` and hold the remaining subcircuits (VoltageRegulator, LedCircuit, FlashCircuit, CrystalCircuit, RP2040Circuit) as children. Pass the existing `routingDisabled` + `schMaxTraceDistance` flags through `boardProps` so the previous board-level behaviour is preserved. - `__snapshots__/`: add PCB + schematic snapshots produced by `tsci snapshot index.tsx` so the new top-level can be regression- tested against future XiaoBoard upstream changes. `PinOutCircuit` is left in `lib/` untouched — keeping the file in the tree avoids breaking any out-of-tree imports while the migration settles, and the lib component is no longer instantiated from `index.tsx`. `bun run build` and `bunx tsci snapshot index.tsx` succeed locally. The two key-prop warnings under `XiaoBoardFootprint` and `RP2040Circuit` are pre-existing render-time warnings inside those components and are not introduced by this change. Closes tscircuit#2
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
feat: replace hand-built board with XiaoBoard from @tscircuit/common (#2)
Adopt the shared XIAO breakout from the new
@tscircuit/commonmoduleinstead of maintaining the bespoke
PinOutCircuitstamp-board chipinline. The Waveshare RP2040-Zero is a XIAO-form-factor board, so the
upstream component already encodes the correct edge pad layout, pin
labels, schematic arrangement, and plated-hole geometry.
Changes:
@tscircuit/common@^0.0.42to dependencies.index.tsx: replace<board>+<PinOutCircuit />with<XiaoBoard variant="RP2040" withPlatedHoles boardProps={…}>andhold the remaining subcircuits (VoltageRegulator, LedCircuit,
FlashCircuit, CrystalCircuit, RP2040Circuit) as children. Pass the
existing
routingDisabled+schMaxTraceDistanceflags throughboardPropsso the previous board-level behaviour is preserved.__snapshots__/: add PCB + schematic snapshots produced bytsci snapshot index.tsxso the new top-level can be regression-tested against future XiaoBoard upstream changes.
PinOutCircuitis left inlib/untouched — keeping the file in thetree avoids breaking any out-of-tree imports while the migration
settles, and the lib component is no longer instantiated from
index.tsx.bun run buildandbunx tsci snapshot index.tsxsucceed locally.The two key-prop warnings under
XiaoBoardFootprintandRP2040Circuitare pre-existing render-time warnings inside thosecomponents and are not introduced by this change.
Closes #2
Diff
Claim
/claim #2