chore: resync sync-brand-numbers.mjs with the source copy - #17
Merged
Conversation
The script is vendored byte-for-byte from BlockRunAI/blockrun:brand/, and this copy had fallen 24 lines behind it. Three fixes landed upstream after the initial distribution and never reached here: - skip nested repos. Walking into a submodule rewrites its markers from the WRONG repo's snapshot and reports drift that belongs to its CI. - do not follow symlinked directories. blockrun's docs/ -> awesome-blockrun/docs is the live case; a link pointing at an ancestor would also recurse forever. - scan .txt, for llms.txt — the file agents read to learn what BlockRun serves, which could not be bound at all before. None of this was detectable: nothing compared the copies. blockrun CI gains a job that does, in the same shape as its existing clawrouter-install-sync guard.
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.
The script is vendored byte-for-byte from
BlockRunAI/blockrun:brand/sync-brand-numbers.mjs. This copy had fallen 24 lines behind it.Three fixes landed upstream after the initial distribution and never reached here:
blockrun/docs -> awesome-blockrun/docsis the live case — it edited a submodule behind the nested-repo skip. A link pointing at an ancestor would also recurse forever.txtllms.txtis the file agents read to learn what BlockRun serves, and could not be bound at all beforeWhy nobody noticed
Nothing compared the copies. A file described as byte-for-byte identical across five repos had no check that it actually was — which is the same failure mode the whole brand-numbers system exists to prevent, reproduced in the mechanism itself.
blockrun CI gains a job that diffs every vendored copy against the source, in the same shape as its existing
clawrouter-install-syncguard. That job lands after these four PRs, so it starts green.--checkstill passes here with the new script.