chore: delete the canton-barebones folder - #120
Open
gabitoesmiapodo wants to merge 2 commits into
Open
Conversation
The folder was a husk: a package.json keeping it a workspace member, a README for a stack that moved out, a superseded .env.example, two unreferenced splice config files, and a prebuilt DAR. The LocalNet is the external @bootnodedev/canton-barebones now, and it ships its own overrides. .dockerignore existed for the wallet-service image build; no Dockerfile remains anywhere in the repo. pnpm build-dar reproduces the DAR and dev-stack.sh builds it before deploying, so PROVENANCE.md no longer lists it as a vendored artifact. .gitleaksignore keeps its canton-barebones lines: they are commit fingerprints for history gitleaks still scans on the outgoing range.
The Subprojects and Documentation Distribution tables each carried a canton-barebones row, and architecture.md still called it the local participant wrapper. Both the LocalNet and wallet-service are dependencies reached from outside the repository now, so the Subprojects trailer says so for the two of them together. The app-agnostic rules drop the folder from the list of trees a library may not import from.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
9 tasks
This was referenced Aug 28, 2026
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
1 participant
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
Closes #119
The folder was a husk after PR #118 moved its scripts to root
scripts/and rewired the loop to the external LocalNet tool. Twenty-two doc references still described it as a live subproject.Changes
canton-barebones/is gone, prebuilt DAR and dead splice config included.dockerignoreis gone; nothing in the repo builds an imagedapp/daml/PROVENANCE.mdrecords thatpnpm build-darproduces the DARReview guide
Nothing here needs close reading. The judgement calls are two: the
PROVENANCE.mdwording, which moved the build-produced DAR out of the Imported row, and the rows dropped from rootCLAUDE.md's two tables.Worth one check: that nothing outside the deleted folder still points at it, and that
.gitleaksignorewas left untouched — its entries are commit fingerprints the pre-push hook still needs.Deviations
@bootnodedev/canton-barebonestool and itsinit/start/resetcommands stay inREADME.md,CLAUDE.mdandarchitecture.md: those are the LocalNet setup instructions, so the criterion is met as no reference to the deleted folderPROVENANCE.mdrecords the build-produced DAR in the paragraph above the table rather than inside the "Imported" row, which now lists only what was vendoredAcceptance criteria
canton-barebones/deleted in full, prebuilt DAR included.dockerignoredeleted; no Dockerfile remains anywhere in the repodapp/daml/PROVENANCE.md"Imported" row records thatpnpm build-darproduces the DARpnpm-workspace.yaml,biome.jsonand.lintstagedrc.format.mjsno longer name canton-barebones.gitleaksignoreuntouchedCLAUDE.md,README.md,architecture.md,canton-connect/CLAUDE.md,canton-connect/architecture.md,canton-dappbooster/CLAUDE.mdCLAUDE.mdSubprojects and Documentation Distribution tables drop the canton-barebones rowpnpm lint,typecheck,test,build,knip,docs:checkandcheck:anatomyall passTest plan
Automated tests
pnpm installpnpm lint,pnpm typecheck,pnpm test,pnpm build,pnpm knip,pnpm docs:checkandpnpm run check:anatomyManual verification
pnpm installcanton-barebones/directory and no.dockerignorein the treenpx @bootnodedev/canton-barebones init, then setvalidators.appUser.uiandsv.scanUItotrue./scripts/dev-stack.sh up <dir>, thenpnpm build-dar,pnpm deploy-dar -- <dar>andpnpm run bootstrappnpm app:devand connect a CIP-0103 wallet at http://localhost:3012Breaking changes
None.
Checklist
Screenshots
None.