chore: retire the single-record bundled Skill provenance schema - #3738
Open
rbalachandar wants to merge 1 commit into
Open
chore: retire the single-record bundled Skill provenance schema#3738rbalachandar wants to merge 1 commit into
rbalachandar wants to merge 1 commit into
Conversation
provenance.json held one record and had no consumers; the bundled Skill catalog reads only <id>/SKILL.md and the sole tree reference was docs/code-origin-audit.md. The computer-use Skill's origin, AI-assistance, and review facts now live in the audit's Bundled Skills section — outside SKILL.md so provenance edits never change installed Skill content hashes. Closes apache#3424 Generated-by: opencode
rbalachandar
force-pushed
the
chore/retire-bundled-skill-provenance
branch
from
August 24, 2026 20:35
dadd63a to
4854b5b
Compare
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.
Summary
packages/runtime/resources/bundled-skills/provenance.jsonheld one record and had no consumers:scripts/gen-bundled-skill-catalog.mjsreads only<id>/SKILL.md, and the sole reference in the tree wasdocs/code-origin-audit.md. The schema was built for the batch that #2695 resolved — 29 Skills went, one stayed.This PR deletes the file and moves the
computer-useSkill's origin, AI-assistance, and review facts into the "Bundled Skills" section ofdocs/code-origin-audit.md, which already had to be touched to remove theprovenance.jsonreference.The issue's two constraints are preserved:
SKILL.md, so provenance edits never change installed Skill content hashes — the property the deleted file existed to hold.docs/computer-use-provenance.md, which registers third-party material; this Skill isorigin: independently-authored.Every fact is carried over verbatim from the deleted JSON (author Haoqing Wang / @hqhq1025, OpenAI Codex drafting assistance, inputs list, introduction via #2147 at
2fb83e20, contributor confirmation 2026-08-11, independent review in #2676 at3c7683b9on 2026-08-12 with its scope caveat), anchored to fixed commits so the historical audit report is not turned into mutable living state.Fixes #3424
Verification
npm run generate:bundled-skills -- --check— catalog unaffected (the generator never read the JSON)npm run format:check,npm run typecheck,npm run check:asf-headersprovenance.jsonnpm run lint,npm run build, knip — no code changed; CI covers themReview focus
One tradeoff worth flagging:
docs/code-origin-audit.mdis a dated evidence report. Folding the record in means any future bundled-Skill provenance update edits a historical document; if more Skills are bundled later, a dedicated provenance home may be worth revisiting.AI use
Select exactly one:
Tool(s) and scope: opencode explored the repository, drafted both the doc change and this description, and proposed the record's new home (selected by the human contributor of record, who reviewed every fact against the deleted JSON before submitting). The commit carries a
Generated-by: opencodetrailer.Checklist
Does this PR entail a change in behavior?