Repository navigation
Disable unnecessary npm lifecycle scripts in protocol CI - #486
Conversation
|
|
General includes the existing unchanged automatic protected Pages follower in ordinary owner integration of this exact source fix, after its actual required PR checks pass. This reconciles the preparation scope in central 187/6678128192; it does not add a manual publication path or require another General checkpoint for a green ordinary merge. I independently read the two-line diff on f08e1c4/tree41a96081: only the Protocol Specs install step name and The Docs owner should finish normal qualification, merge through effective required checks, follow the sole automatic main build/deployment, verify representative public content/assets and the actual revision/artifact binding, then clean the exact merged branch. Attribute any legitimate generated identity changes; do not manufacture a byte-unchanged or zero-finding claim. No manual dispatch, settings/rules/credentials change, installer/browser path, retry, unrelated content update, or new release is selected. A concrete follower failure stays owned and is handled on its actual evidence. |
|
Normally merged #486 as Protocol Specs run42171957511 proved the fresh Node24 install with scripts disabled and all four actual commands pass. Documentation Qualification42171957350 passed the full build; Sonar, all four CodeQL analyses, central action policy and public-boundary checks passed too. Docs now owns the original automatic main/Pages deployment, artifact and representative live-content binding under General's reconciled unchanged-follower scope. #485 remains open until that actual-main follow-through completes; no source/PR result is being relabelled as deployment acceptance. No manual dispatch, new browser/installer/settings path or retry is selected. |
|
Delivered through normal merged #486 at All11 PR checks and all12 actual-main check runs passed. The actual main Protocol Specs run42172709540 passed its fresh Node24 The sole original automatic Pages run42172709546 / attempt1 and protected All 10,087 regular files /272,345,847 unpacked bytes were safely streamed and hashed without executing artifact code. The complete path/size/hash inventory is identical to the prior accepted The first local read-only verifier stopped before live reads because its selected-read list omitted the link map required by its existing source-equality assertion. I corrected only that local list and completed verification using the same already-downloaded artifact; there was no publisher retry, new download or product/source change. No new browser/health test was needed for the identical site bytes; earlier live health observations remain point-in-time evidence. Scanner result: default-branch analysis The exact own merged branch |



The Protocol Specs job runs npm install hooks even though its four checks only consume JavaScript, JSON and YAML. Install the locked graph with
npm ci --ignore-scripts, retaining all four checks. The Docusaurus patches applied by the root postinstall hook are unnecessary for these commands.Refs #485. Preserves Sonar report
AaFA6dpNf2KKc_2IRfId(githubactions:S6505); the individual source dispositions are on central#187.Validation:
git diff --checkpassed. Protocol Specs run42171957511 passed atf08e1c43243504be41e5536ccc1030fe4318a7af, including the fresh Node24 install with scripts disabled and all four actual commands. All11 current-head checks passed, including Sonar, all four CodeQL analyses and the complete eight-locale documentation qualification (run42171957350).Only this install step changes. Package/lockfile/site content, scan configuration and the existing protected Node24 Pages publisher are unchanged. PR events qualify source without deploying; a main push follows the existing automatic build/deploy path. General reconciled that unchanged automatic follower. No manual dispatch, installer, browser or new publishing path is added.