docs: add pypi-axi to community catalog - #128
Conversation
|
Speaking as Kun's firstmate: independent source review of the proposed catalog add. Pin inspected: https://github.com/mstuart/pypi-axi at f2533b98f3a105c71b3ca724fbe2b413f0565d9a (current main HEAD; catalog.yaml has no pin field, matching existing schema). I did not execute the package. Source actually read: README.md, package.json, src/cli.ts, src/home.ts, src/args.ts (assertKnownFlags), src/commands/view.ts, src/commands/deps.ts, src/commands/setup.ts. Observations from that source, not the PR body: real AXI CLI on axi-sdk-js (runAxiCli). Read-only public PyPI JSON API; no auth. Commands: view, versions, deps, downloads, search, setup hooks. Truncation with --full. Unknown flags fail loud. Metadata matches (pypi-axi, PyPI, mstuart). Not official, empty, malware, or a name-squat. VISION.md per-rule verdict:
I am not merging yet. The PR is CONFLICTING against current main. I could not push a file-only resolve to mstuart/axi. Please rebase onto current main, append the pypi-axi row to today's catalog.yaml, run docs:gen, and push. Once CI is green and conflict-free I will squash-merge. |
kunchenguid
left a comment
There was a problem hiding this comment.
Speaking as Kun's firstmate: source review at f2533b98 passes. Approving the catalog add. Merge waits on a rebase against current main.
|
Speaking as Kun's firstmate. This catalog entry landed via last-resort port #154 (squash merge |
What Changed
pypi-axientry tocatalog.yaml's community list (name, url, author, domain, description).README.mdanddocs/index.htmlto include the newpypi-axirow.Risk Assessment
✅ Low: Purely additive documentation change (one catalog.yaml entry plus its generated README.md and docs/index.html rows) with no source code, build, or release-process changes; content and table formatting are internally consistent with existing entries.
Testing
This is a docs-only change adding the pypi-axi CLI to the community catalog (catalog.yaml, README.md, docs/index.html). After installing workspace dependencies (pnpm via corepack, gitignored and removed afterward),
docs:checkpassed, confirming no drift between catalog.yaml and the generated markdown/HTML regions for the new entry, and the generator's existing unit tests passed unchanged. No code paths beyond doc generation were touched, so no further evidence is needed.Evidence: docs:check output confirming generated regions match catalog.yaml
Pipeline
Updates from git push no-mistakes
⏭️ **intent** - skipped
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
✅ **Review** - passed
✅ No issues found.
✅ **Test** - passed
✅ No issues found.
node scripts/generate-docs.mjs --check(docs:check) — validates generated README.md/docs/index.html community-catalog regions match catalog.yaml source of truth, including the new pypi-axi entrynode --test scripts/generate-docs.test.mjs(docs:test) — unit tests for the catalog HTML/Markdown generation logic used to render the new entry✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.