Skip to content

fix(ingress): emit required commit metadata - #161

Open
scbrown wants to merge 3 commits into
mainfrom
grant/yqfk-commit-metadata-20261023
Open

scbrown wants to merge 3 commits into
mainfrom
grant/yqfk-commit-metadata-20261023

Conversation

@scbrown

@scbrown scbrown commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

The git provenance adapter now emits sha and in_repo from its existing git-record inputs. This supplies the shared commit metadata without changing commit identity or work-item ownership. The tests cover quoted repository names and commits with and without paths, and the ingress reference describes the fields.

Validation: just check and just test (1,294 tests passed), with RDFLib 7.6.0 and the existing CI dependencies. Guarded HOME surfaces remained unchanged.

Demo transcript, actual emitter on eight synthetic input combinations, parsed as RDF:

before_required_fields: 0
 after_required_fields: 2
only_expected_triples_added: true
removed_facts: 0
live_writes: 0

Reproduce the source regression with python3 tests/test_ingest_git_provenance.py. The paired control covered ordinary/quoted repository names, empty/nonempty paths and empty/nonempty item links. This is a fixture transcript; no public recording was uploaded.

Historical stored-data repair and restoring required-field rejection remain separate acceptance steps. Rollback is a normal source revert of these three paths.

This branch has not been deployed

No deployments
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.

1 participant