fix(ci): correct nself release tarball extraction in quarterly-doc-audit - #29
Merged
Merged
Conversation
…arterly-doc-audit
The CLI release tarballs (nself-${VERSION}-${OS}-${ARCH}.tar.gz) nest
the binary inside a nself-${VERSION}-${OS}-${ARCH}/ directory. The
audit-tool install step extracted flat into /tmp and then did
'mv /tmp/nself ...', which no longer finds the binary since the
release packaging added the wrapping directory. Add
--strip-components=1 (matching the fix already applied in
nchat/nclaw/nfamily/clawde/homebrew-nself) and fail soft instead of
hard-erroring the whole audit if the download/extract ever fails.
Verified: current v1.2.5 linux-amd64 tarball extracts /tmp/nself
correctly with this flag; confirmed live failures in this repo's
quarterly-doc-audit runs before this fix (mv: cannot stat '/tmp/nself').
3 tasks
acamarata
added a commit
that referenced
this pull request
Aug 16, 2026
…d-paths (#31) peter-evans/create-pull-request's git add treats multiple pathspec arguments atomically: since this repo has zero .mdx files, the **/*.mdx pathspec always fails with 'pathspec did not match any files', which aborts the whole git add — so none of the 44 legitimately modified .md wiki files get staged either, and the commit step then fails with 'no changes added to commit'. Confirmed via gh run view --log-failed on run 31956740627 (head_sha d999eda, current main HEAD) and reproduced locally: an unmatched pathspec element in a multi-pathspec git add aborts the entire command. This is unrelated to the prior --strip-components=1 CLI tarball fix (#29), which is already correctly in place. find . -iname '*.mdx' confirms this repo has no mdx files, so drop the pattern entirely rather than working around it.
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 nself CLI release tarballs nest the binary inside a
nself-${VERSION}-${OS}-${ARCH}/directory. The quarterly-doc-audit install step assumed a flat extraction and didmv /tmp/nself ..., which fails since the release packaging added the wrapping dir. Adds--strip-components=1(matching the already-working fix in nchat/nclaw/nfamily/clawde/homebrew-nself) and fails soft instead of hard-erroring the whole audit.Verified locally: downloaded the current v1.2.5 linux-amd64 release tarball and confirmed
--strip-components=1extracts/tmp/nselfcorrectly. Confirmed this repo's quarterly-doc-audit workflow_dispatch runs were failing withmv: cannot stat '/tmp/nself': No such file or directorybefore this fix.