Enforce validate:release in CI; add x402 references, docs, and fix validator bug#21
Merged
Merged
Conversation
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.
Motivation
x402protocol spec and make policy and onboarding guidance explicit for release reviewers.Description
npm run validate:releaseinto the existing.github/workflows/validate.ymlpipeline as a required step and switch installs tonpm ciwithcache: npmfor both jobs.x402compatibility references toREADME.mdandSPEC.mdthat treat x402 as an external protocol surface and note commercial flows may reference x402 payment/settlement artifacts.SECURITY.mdwith a concrete reporting address (dev@commandlayer.org), supported reporting path, disclosure process, response targets, scope, and out-of-scope items.scripts/archive/is historical-only inONBOARDING.mdand rewriteCHANGELOG.mdto explicitly document thev1.0.0→v1.1.0migration (added/changed/removed/flattened/moved/legacy/migration notes).validateLegacyLinefunction definition inscripts/validate-cards.mjsto resolve a syntax error so validation scripts run.Testing
npm cicompleted successfully in the environment.npm run validate(local current-line and checksum checks) completed successfully.npm run validate:releaseexecuted but network-based upstream schema fetches failed in this environment due to externalraw.githubusercontent.comresolution being blocked (observed403 CONNECT tunnel failed), so external URL resolution checks did not succeed here.npm run validate:legacycompleted successfully.Codex Task