Repository navigation
Conversation
Emit a canonical SHA-256 provenance digest for audit integrity and finish slice-4 docs/test polish on 2.1.0.
This was referenced Nov 3, 2026
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.
Description
Slice 4 of #439: finish the wallet_screening v2 milestone with
provenance.report_hash(canonical SHA-256 over the report body) and final docs/test polish. Stays on 2.1.0 per #461 guidance (no version drift).Acceptance criteria → diff:
provenance.report_hash+generatorvia_attach_provenance/verify_report_hash→skills/finance/wallet_screening/skill.py; covered by execute success, offline-direct, Tornado vector, andtest_verify_report_hash_detects_tamperdocs/skills/finance/wallet_screening.mdinstructions.mdprovenance→tests/fixtures/card_ui_schema/finance__wallet_screening.json[Unreleased]note for slice 4Closes the #439 milestone after merge (Ross on #461).
Type of Change
skills/skills/skillware/core/loader, env, adaptersskillware/cli.py,docs/usage/cli.mdexamples/*.py, agent loops,examples/README.mdpyproject.toml,MANIFEST.inChecklist (all PRs)
Fixes #…orRefs #…)python -m black --check .,flake8, andruff check .pass locally (or CI-equivalent subset)pytest skills/andpytest tests/pass locally when relevantCHANGELOG.mdupdated under[Unreleased]when user-visible behavior changesexamples/README.mdupdated if this PR adds, renames, or removes a runnable scriptpytest tests/test_registry_docs.pywhen skills, examples index, or agent-loops matrix changedpytest tests/test_skill_docs.pywhen catalog Usage Examples or provider snippets changedNew or updated skill
Skip unless this PR adds or changes files under
skills/.Bundle and metadata
skills/<category>/<skill_name>/(fromtemplates/python_skill/or equivalent)manifest.yaml:name(full ID),version,description,parameters,constitution, realissuershort_description,issuer.github,issuer.org,requirements,env_varsEffect, Directive, Assurance
skill.py(Effect; no ad-hoc LLM-generated execution paths)instructions.md(Directive) explains when and how to use the skillcard.json(Presentation) issuer matches manifest when presenttest_skill.py(Assurance) covers execution and schema expectationsSkillLoader.load_skill("<category>/<skill_name>")succeeds (or deps documented)Documentation and catalog
docs/skills/<category>/<skill_name>.md, category hub row, row indocs/skills/README.md, anddocs/sitemap.mdConstitution and safety (skills only)
Read-only screening. Provenance hash is integrity metadata only — no signing or broadcasting. Hosts should halt on
DENYand require a person onREVIEW. Not legal advice.Related Issues
Fixes #439