chore(scaffold): bump @civitai/blocks-react pin to ^0.46.0 - #523
Merged
Conversation
pins-vs-published is a REQUIRED check and is currently red on every open PR in
this repo: npm published @civitai/blocks-react 0.46.0, the page-money template
pins ^0.45.0, and a pre-1.0 caret locks the minor, so the pin no longer admits
published latest. Every app scaffolded from the template is born stale. The
automation that normally handles this (bump-scaffold-pins) is itself failing on
main, so the bump did not land on its own.
Produced by the repo's own tool, `go run ./internal/scaffold/cmd/bump-pins` —
only @civitai/blocks-react moved, across the four places that mirror it (the
template, its README, the scaffold_test assertion, and the design-tokens
fixture header). The token count is unchanged at 32.
🔴 NOT a rubber stamp, because this package has form: 0.44 INVERTED the workflow
error contract (estimate()/submit() went from resolving a failure snapshot to
THROWING WorkflowEstimateError/WorkflowSubmitError, whose developer-facing
.message then renders to viewers). 0.45 was cleared only by a tarball diff, and
0.46 was cleared the same way here:
- internal/transport.js, internal/liveHost.js, internal/mockHost.js,
hooks/useBuzzWorkflow.js, hooks/useAppWorkflows.js — all BYTE-IDENTICAL
between 0.45.0 and 0.46.0;
- every dist file mentioning WorkflowEstimateError/WorkflowSubmitError —
byte-identical;
- the entire delta is additive: four new files (ui/ReportButton.*) plus one
re-export line in ui/index.js. Nothing else in dist changed.
Verified: TestScaffoldPinsSatisfyPublished passes locally with
CIVITAI_CHECK_PUBLISHED_PINS=1 armed — PASS, not SKIP, so it really reached npm
rather than skipping on an unreachable registry. Full suite 21/21 packages.
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.
Why this is urgent
pins-vs-publishedis a required check and is currently red on every open PR in this repo. npm published@civitai/blocks-react@0.46.0; thepage-moneytemplate pins^0.45.0, and a pre-1.0 caret locks the minor — so the pin no longer admits published latest and every app scaffolded from the template is born stale.The automation that normally handles this,
bump-scaffold-pins, is itself failing on main (bf4db4b), which is why the bump did not land on its own. That is a separate problem and is not addressed here.What this is
Output of the repo's own tool,
go run ./internal/scaffold/cmd/bump-pins. Only@civitai/blocks-reactmoved, across the four places that mirror the pin — template, its README, thescaffold_testassertion, and the design-tokens fixture header. Token count unchanged at 32.🔴 Not a rubber stamp — this package has form
0.44 inverted the workflow error contract:
estimate()/submit()went from resolving a failure snapshot to throwingWorkflowEstimateError/WorkflowSubmitError, whose.messagethe SDK documents as developer-facing and never for a viewer. Blocks that bumped across it silently began rendering internal text to users, and fixtures mocking a resolved failure kept passing straight through the change. 0.45 was cleared only by a tarball diff, so 0.46 got the same treatment.npm packed both versions and diffeddist/recursively:internal/transport.jsinternal/liveHost.jsinternal/mockHost.jshooks/useBuzzWorkflow.jshooks/useAppWorkflows.jsWorkflowEstimateError/WorkflowSubmitErrorThe entire delta is additive: four new files (
ui/ReportButton.*) and one re-export line inui/index.js. Nothing else indistchanged, in either direction.Verification
TestScaffoldPinsSatisfyPublishedpasses locally withCIVITAI_CHECK_PUBLISHED_PINS=1armed — and reports PASS, not SKIP, so it actually reached npm rather than skipping on an unreachable registry (the workflow itself notes a skip would be a green that says nothing).pinned ^0.45.0 (does NOT admit the published version) / published 0.46.0.Follow-on
This unblocks #522, which is otherwise green (12 of 13 checks passing) and blocked solely by this gate.