Skip to content

docs: runbook for the local npm.ix publish path (PLAT-890) - #152

Merged
kreneskyp merged 1 commit into
mainfrom
plat-890-npm-ix-runbook
Nov 8, 2026
Merged

kreneskyp merged 1 commit into
mainfrom
plat-890-npm-ix-runbook

Conversation

@kreneskyp

Copy link
Copy Markdown
Contributor

Writes the local/manual npm.ix publish procedure as a README section next to the existing Development/make-target docs (README.md is where this repo keeps procedure; NFR-008 is a boundary spec and CONTRIBUTING.md a review checklist). Also corrects tools/quire-dist/README.md, which still said package-npm needs every catalog target.

Framing: repeatability only. The structural guarantee is package-npm deriving optionalDependencies from the artifacts present; make dist-verify-published is the second line. The section says so.

Not done: the runbook has NOT been walked end to end against npm.ix (no publish was run). Unverified: whether --@agent-ix:registry overrides the publishConfig.registry in the generated manifests; the runbook dry-runs first to read the target from npm output. The README section records that it is unwalked.

Gate: fmt-check, lint pass; cargo test: only it_144 fails (missing python jsonschema on this host).

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@kreneskyp
kreneskyp requested a review from a team as a code owner November 8, 2026 17:09

@kreneskyp kreneskyp left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lead review (diff only): README-only runbook for the local npm.ix publish path (7 steps, dry-run first, structural guarantee stated as living in package-npm with dist-verify-published as second line) plus a correction in tools/quire-dist/README.md. No code touched; gate green apart from host-only it_144 (missing python jsonschema). Note: runbook is unwalked and the publishConfig.registry override is unconfirmed; the section says so. No findings.

@kreneskyp
kreneskyp merged commit 7921bd6 into main Nov 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant