Repository navigation
chore(bootstrap): share native CLI test fixtures - #48
Merged
Merged
Conversation
egdev6
marked this pull request as ready for review
October 10, 2026 09:21
This was referenced Oct 10, 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
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
Validation
args.join(" ")collision limits, and platform boundaries.npm pack --ignore-scripts: exactly 64 regular members, all byte-identical to the selected source. Package SHA256:0385ac5248003191a4a7e4b73ed1b5ff9364b6dc9c29dfeb5c2e2ec7031115b6.testpasses for this PR head (automatic CI pending at submission).Independent technical verification reproduced source counts, preserved assertions, LF/CRLF checks, security scans, and package bytes. Windows C# launcher code is byte-identical to the prior launcher, but Windows/macOS/hosted Node 20 execution remains for automatic CI. No skill-load, live GitHub/Projects, stored-secret, sandbox, or LLM proof is claimed.
Linked issue
Refs #15
This is a nonclosing reference. D1 supplies test infrastructure only; remaining package/migration/activation acceptance, later characterization/refactor/API units, and release decisions are not completed by this PR.
PR type
type:chore)Changes
skills/github-repository-bootstrap/tests/lib.test.mjsskills/github-repository-bootstrap/tests/helpers/native-cli-fixtures.mjsdocs/odd/b01-native-cli-fixtures.md284 changed lines = 201 additions + 83 deletions, counting helper moves twice. No test/assertion removal, code golf, deferred essential tests/docs, product module changes, schema/default/flag/version changes, or cleanup.
Verification boundaries and process record
The first complete candidate was measured in RAM before writing. One line-count-neutral expected-log fix was re-measured after, rather than before, its write; that process deviation is retained in local evidence and not retro-authorized. Subsequent documentation and single-declaration fixes had complete pre-write measurements.
The mechanical extraction was characterized functionally. The new fixture-data guard was tested after implementation; an owned negative control changing unknown-request exit 1 to 0 makes the new test fail by assertion. This is post-hoc safety refutation, not historical test-first RED. The initial incorrect double-bracket expectation is not reported as a bug RED.
The stub's data is locally supplied and trusted. JSON values are written to stdout without eval or request-derived executable code interpolation; it is not a security sandbox, universal argv filter, credential validator, or GraphQL/stdin/pagination implementation.
Contributor checklist
Refs #15, no automatic closure.type:chore.Co-Authored-Bytrailer.Published as a draft. No assistant ready/merge, workflow dispatch/rerun, issue closure, new tag/release, activation, or cleanup is authorized.