Repository navigation
ci: fix affected dependency preparation and documentation scope - #83
Merged
Merged
Conversation
NWYLZW
marked this pull request as ready for review
September 9, 2026 15:03
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.
Chatroom guide edits currently select the full product gate, while sensitive deletions mixed with prose and renames into Markdown can accidentally skip it. Classify deleted and old/new rename paths with NUL records, keep owner guides lightweight, and retain complete checks for maintenance rules, shared configuration, and package metadata. A scope failure falls back to the complete gate.
Affected typecheck/build-test jobs now retain npm lifecycle scripts: the exact Git Host dependency needs preparation to supply
cordisx/contractsand JSX exports. An ignore-scripts installation reproduced missing-module failures locally. Format and configuration-only jobs keep their lightweight installation. Shared quality checks use the PR head SHA and cache npm downloads.Validation: 13 real-Git/workflow regression tests, actionlint 1.7.12 (external shellcheck/pyflakes disabled), and whitespace checks passed. After installing prepared Git dependencies,
npm run check(including 378 tests), full-source ESLint, and changed-file dprint passed locally. No product runtime or public contract changed.Full jobs use
!cancelled()instead of job-levelalways()so superseded PR runs release their concurrency slot while scope failures still select complete validation. Evidence cleanup steps retainalways(). See GitHub cancellation semantics.