Skip to content

docs: freeze forge pointer and harness snapshot join contract - #57

Open
HofniGartner wants to merge 2 commits into
RHEcosystemAppEng:mainfrom
HofniGartner:feat/join-contract-doc
Open

docs: freeze forge pointer and harness snapshot join contract#57
HofniGartner wants to merge 2 commits into
RHEcosystemAppEng:mainfrom
HofniGartner:feat/join-contract-doc

Conversation

@HofniGartner

Copy link
Copy Markdown

Summary

Freeze the cross-project join contract for harness snapshot / forge pointers, and extend Provenance so the typed report matches that field list.

Changes

  • Add Docs/provenance_forge_pointers.md (ownership matrix, field mapping, snapshot lifecycle, MLflow-first later read)
  • Extend Provenance with optional join fields (repository_url, change_id, trace_id, session_id, eval_run_id, harness_fingerprint, forge_platform)
  • Wire matching --* flags in scripts/analyze.py
  • Add passthrough test for forge join fields

Test plan

  • uv run pytest tests/test_analyze.py -k provenance (or project venv equivalent)
  • Review Docs/provenance_forge_pointers.md for field/ownership accuracy

Related

  • APPENG-4985 / APPENG-5370 (join / MLflow observer context)
  • Follow-up: agent runtime ADR + eval-engine MLflow consumer (separate repos)

Made with Cursor

HofniGartner and others added 2 commits July 22, 2026 11:47
Define ownership matrix, field mapping, and snapshot lifecycle so
runtimes and eval engines share one join interface.

Co-authored-by: Cursor <cursoragent@cursor.com>
Align typed fields and analyze CLI flags with the frozen join contract
in Docs/provenance_forge_pointers.md.

Co-authored-by: Cursor <cursoragent@cursor.com>

@GuyZivRH GuyZivRH left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great direction! do you extend this PR or open a following PR with the rest of the work?

@HofniGartner

Copy link
Copy Markdown
Author

Thanks Guy!

This PR is intentionally the schema / contract slice only:

  • freeze Docs/provenance_forge_pointers.md (ownership, fields, snapshot lifecycle)
  • extend typed Provenance + analyze CLI flags to match

Following PRs (separate repos / issues), not stacked here:

Happy to open a follow-up in ABEvalFlow for Phase C observer wiring once the producer + AEH tags/artifact path are in — that should stay out of this PR so the join contract can land first.

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