feat(scripts): skill catalog generator + drift gate (soc-vuu6.4 slice 1 #catalog-generator)#379
Merged
Merged
Conversation
… 1 #catalog-generator) Slice 1 of soc-vuu6.4: ship the generator, schema, drift gate, and committed catalog.json. Follow-up beads filed for the Go CLI subcommands (slice 2) and showcase consumer wiring (slice 3). Files: - schemas/skill-catalog.schema.json — JSON Schema (envelope + per-skill shape) - scripts/generate-skill-catalog.sh — awk-based YAML frontmatter parser → catalog.json - scripts/check-skill-catalog-drift.sh — CI-gate wrapper - skills/catalog.json — generated, committed, 79 skills Parser handles the SKILL.md frontmatter subset in use. Regression test covers the 'last context_rel entry dropped on scalar transition' bug found during dev. ## Test 14 bats tests, all passing. Self-dogfood against real tree generates 79-skill catalog cleanly. Closes-scenario: soc-vuu6.4#catalog-generator Bounded-context: BC1-Corpus Evidence: scripts/generate-skill-catalog.sh Evidence: scripts/check-skill-catalog-drift.sh Evidence: tests/scripts/generate-skill-catalog.bats
boshu2
force-pushed
the
feat/skill-catalog-soc-vuu6-4
branch
from
54da228 to
ccc0b46
Compare
boshu2
added a commit
that referenced
this pull request
May 21, 2026
…e-2-of-3) (#393) ## Why soc-dspz asks for 3 advisory I0 jobs to wire in. Two of the three source scripts are already on `main` (after PRs #373, #379 merged); the third (`scripts/lint-skill-frontmatter.sh` from #376) is still pending merge. Per the new agile invariant (soc-5qit, PR #392), the right move is to ship the unblocked part and leave the parent bead open for the third when #376 lands — instead of waiting for everything. ## What changed | Surface | Change | |---|---| | `.github/workflows/validate.yml` | Added 2 stanzas: `lint-evidence-lines-advisory` (PR-event triggered) and `check-skill-catalog-drift-advisory` (skills/CI path-filtered). Both `continue-on-error: true` so they emit `::warning::` annotations without blocking merges. | | `.github/workflows/validate.yml` summary job | `needs:` extended with both new advisory jobs. | | `docs/contracts/ci-jobs.yaml` | 2 manifest entries, alphabetically slotted. | | `AGENTS-CI.md` | Regenerated via `scripts/generate-ci-jobs-table.sh --write` (69 rows). | ## Promotion path Per soc-dspz acceptance: I0 advisory → T2 required after one clean week of runs. File a follow-up bead to flip `continue-on-error: true` off and promote both jobs to required. ## What's NOT in this PR - `lint-skill-frontmatter-advisory` — depends on PR #376 (open) adding `scripts/lint-skill-frontmatter.sh`. soc-dspz stays open after this merges; a follow-up small PR will add that 3rd stanza once #376 lands. ## Validation - `python3 -c 'import yaml; yaml.safe_load(...)'` parses both YAML files clean - `scripts/validate-ci-policy-parity.sh`: `CI_JOBS_TABLE: PASS (69 rows)` Closes-scenario: soc-dspz#wire-2-of-3 Bounded-context: BC4-CI Evidence: scripts/validate-ci-policy-parity.sh
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
1 participant
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.
Why
registry.json(47KB) is too coarse — agents grepping for "which skill produces a ratchet?" have to scan markdown. Hand-curated /skills tables drift. Futureao skill newscaffolders need a queryable inventory. This is slice 1 of soc-vuu6.4 (the generator + schema + CI gate). Go CLI subcommands and showcase consumer are filed as follow-up beads.What
Three artifacts:
schemas/skill-catalog.schema.json— JSON Schema for the catalog. Required fields: name, description, hexagonal_role, consumes, produces, context_rel, user_invocable, references_count, codex_override_present. hexagonal_role enum mirrors soc-e8pj/PR feat(scripts): lint-skill-frontmatter — enforce required keys on SKILL.md frontmatter (soc-e8pj #skill-md-frontmatter-linter) #376.scripts/generate-skill-catalog.sh— walks everyskills/*/SKILL.md, parses YAML frontmatter via awk, emitsskills/catalog.jsonwith all 79 skills. Modes: default (write),--check(CI drift gate, exits 1 on drift),--stdout,--out PATH.scripts/check-skill-catalog-drift.sh— thin wrapper over--checkso the future workflow has a stable, named entry point.skills/catalog.json— generated artifact, committed. 79 skills, 40KB.The parser handles the SKILL.md frontmatter subset we actually use: scalar key:value, lists (
- item), andcontext_relnested objects (- kind:/with:). A regression test covers the "last context_rel entry dropped on transition to next scalar" parser bug caught during development.Test
14 bats tests, all passing. Cover: envelope fields, name/description extraction, list round-trip, context_rel parsing (incl. the regression for the dropped-last-entry bug), user_invocable boolean, references_count, codex_override_present, default file-write,
--checkpass + drift detection, drift wrapper exit codes, empty-skills case, unknown-flag error.Self-dogfood: real skills/ tree → 79 skills generated, all 4 keys validated by lint-skill-frontmatter (#376), drift check green.
Follow-up beads filed
ao skills list/consumers/producers/graph(Go CLI subcommands, blocks on this PR)Closes-scenario: soc-vuu6.4#catalog-generator
Bounded-context: BC1-Corpus
Evidence: scripts/generate-skill-catalog.sh
Evidence: scripts/check-skill-catalog-drift.sh
Evidence: tests/scripts/generate-skill-catalog.bats