Skip to content

feat(catalog): add bws-axi community entry - #189

Merged
kunchenguid merged 4 commits into
kunchenguid:mainfrom
masculinecache:add-bws-axi-to-catalog
Sep 11, 2026
Merged

kunchenguid merged 4 commits into
kunchenguid:mainfrom
masculinecache:add-bws-axi-to-catalog

Conversation

@masculinecache

@masculinecache masculinecache commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor

What Changed

  • Added @masculinecache/bws-axi to the community AXI catalog with its Bitwarden Secrets Manager metadata.
  • Added the pinned admission record for revision e3a122cf5e24790e93279760e3f32a2d552beb0b, including reviewed components, source observations, and the version-probing exception.
  • Regenerated the community catalog entries in README.md and docs/index.html.

Pipeline

Updates from git push no-mistakes

✅ **intent** - passed

✅ No issues found.

✅ **Rebase** - passed

✅ No issues found. Merged upstream/main (five newer catalog entries) into the branch; catalog.yaml and docs/index.html auto-merged, README conflict resolved by regeneration (scripts/generate-docs.mjs --check green at head 17b0aff).

✅ **Review** - passed

✅ No issues found.

✅ **Test** - passed (1 info finding)
  • ℹ️ The focused semantic catalog check could not run because the ad hoc Node import of the yaml package failed with ERR_MODULE_NOT_FOUND; the repository's own docs generation checks did run successfully and provide equivalent consumer-level validation.
✅ **Document** - passed

✅ No issues found.

✅ **Lint** - passed

✅ No issues found.

✅ **Push** - passed

✅ No issues found.

✅ **PR** - passed

✅ Pull request updated: #189

✅ **CI** - passed

✅ All required checks green on head a8356626: Generated files must not be hand-edited, PR must be raised via no-mistakes, drift, Greptile Review (runs 34140448101, 34187077093, 34140447989).

opencode@InspironOne added 2 commits September 6, 2026 14:46
…ME.md, catalog.yaml, and docs/index.html; all generated docs in sync, lint and format checks pass.
@greptile-apps

greptile-apps Bot commented Sep 6, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge.

Reviews (4) · Last reviewed commit: "Merge remote-tracking branch 'upstream/m..."

Comment thread catalog.yaml
@kunchenguid

Copy link
Copy Markdown
Owner

Speaking as Kun's firstmate:

Triage — waiting on author (catalog admission incomplete)

Contract-class: opt-in (community catalog add; no default product path change).

VISION.md per-rule verdict

Rule Verdict Evidence
AXI principles (conservative evolution) aligns (with notes) Independent review of published @masculinecache/bws-axi@10.1. (npm tarball; gitHead e3a122cf5e24790e93279760e3f32a2d552beb0b; sha256 e8cbfab67f826a0cf3a6e977bffe248b931072d64fc200d9e255e326a3a4266). Inspected bin/bws-axi, bin/common.mjs, skill/SKILL.md, package.json. Read-only SM CLI; TOON-ish list/kv; exit 0/1/2 correctly mapped; values redacted unless --full. Missing -v/-V/--version fast path (principle 10) — would be an admission exception if otherwise ready.
Catalog — independent source review at pin cannot tell / incomplete for admission Catalog entry has no admission block and no pinned reviewed_revision. Linked repo https://github.com/masculinecache/bws-axi returns 404 / not found (git ls-remote + GitHub API). npm package exists and was inspectable; VISION still requires a durable pin + admission record in catalog.yaml before a positive admission verdict. Contributor claims / npm metadata alone are not enough to merge.
Catalog — official catalog untouched aligns Diff only adds a community entry (+ regenerated docs).
SDKs aligns (N/A) No SDK changes.

Diff notes (this PR)

  • catalog.yaml entry is metadata-only (name/url/author/domain/description) — Greptile correctly blocked on missing pin/admission.
  • Stray findings.json is introduced at repo root; it is not on main and should not ship with a catalog PR — please remove it.
  • Guard + docs-check are green on head f4f6499. Greptile Review is red (admission). Require no-mistakes failed on the opened body (run 34065018548) and has not shown a green check on the latest head — please re-raise / edit via git push no-mistakes so the body attestation lands on the current SHA.

What we need from you before we can admit/merge

  1. Make https://github.com/masculinecache/bws-axi public (or update url to the real public source), at a commit we can pin.
  2. Remove findings.json from the PR.
  3. After (1), we (or you) add an admission block with reviewed_revision, reviewed_components, source_observations, and any exception (at least principle 10 version probing). Do not merge until that lands.
  4. Clear no-mistakes on the current head.

This is waiting on you, not a captain hold. Ping when the repo is public and the PR is cleaned up and we'll finish admission + merge if still opt-in and green.

@masculinecache

Copy link
Copy Markdown
Contributor Author

All four triage items addressed on a835662:

  1. Repo public — https://github.com/masculinecache/bws-axi is now PUBLIC (visibility flipped; same source, no secrets in tree — verified against the 12-category secret scan on the published tarball earlier today).
  2. findings.json dropped from the PR branch.
  3. admission block added to the catalog.yaml entry: reviewed_revision e3a122cf5e24790e93279760e3f32a2d552beb0b (npm @masculinecache/bws-axi@1.0.1 gitHead, sha256 e8cbfab67f826a0cf3a6e977bffe248b931072d64fc200d9e255e326a3a4266), reviewed_components bin/bws-axi, bin/common.mjs, skill/SKILL.md, package.json; source_observations covering read-only semantics, token via env/file only, redaction default, exit 0/2/1 mapping; exception citing the principle-10 gap (no -v/-V short probing).
  4. Guard + drift green on a835662; Greptile re-review pending on the new head.

Ready for re-review.

@kunchenguid

Copy link
Copy Markdown
Owner

Speaking as Kun's firstmate: re-triage of #189 — admission/source review at pin e3a122cf is complete (contract-class opt-in; VISION aligns with principle-10 exception). Still blocked: PR body has no no-mistakes Pipeline attestation (open failure run 34065018548; compare #184). Please git push no-mistakes so an edited body-compliance check goes green. Waiting on you, not a captain hold.

@masculinecache masculinecache changed the title feat: add @masculinecache/bws-axi to catalog feat(catalog): add bws-axi community entry Sep 8, 2026
@kunchenguid
kunchenguid merged commit 8a3c8d8 into kunchenguid:main Sep 11, 2026
4 checks passed
@kunchenguid

Copy link
Copy Markdown
Owner

Speaking as Kun's firstmate: this is merged. Thank you @masculinecache — really appreciate you taking the time on this.

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.

2 participants