Skip to content

Harden generated artefact ownership boundaries and transient ignore rules - #10

Merged
goldjg merged 2 commits into
mainfrom
copilot/harden-generated-artifact-boundaries
Jun 30, 2026
Merged

Harden generated artefact ownership boundaries and transient ignore rules#10
goldjg merged 2 commits into
mainfrom
copilot/harden-generated-artifact-boundaries

Conversation

Copilot AI commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

This PR clarifies ownership boundaries for generated artefacts so shard data, pack manifests, provider-op enrichment, and demo screenshots are treated as pipeline-managed outputs rather than hand-edited files. It also tightens ignore behavior for local transient generation outputs while keeping generated-offline runtime assets committed.

  • Scope + governance alignment

    • Updated .github/carl/current-pr-contract.md to reflect this task’s scope, forbidden changes, and ownership/validation boundaries for generated artefacts.
  • Ignore policy hardening

    • Updated .gitignore to ignore transient local outputs from data/sweep pipelines (inventory zips, timestamped sweep CSV/WebM/GIF, provider-op sweep dumps).
    • Added explicit comments documenting generated-but-committed paths that must remain tracked and pipeline-owned.
  • Contributor-facing ownership rules

    • Added a dedicated Generated Artefact Ownership (Do Not Hand-Edit) section in CONTRIBUTING.md.
    • Added a matching Generated Artefact Ownership section in README.md to make pipeline ownership explicit in top-level project docs.
# Generated but committed (pipeline-owned): do NOT ignore these paths.
# - extension/data/shards/
# - extension/data/manifest.json
# - extension/data/azure-provider-ops.json
# - demos/apispy-*.png

@goldjg
goldjg marked this pull request as ready for review June 30, 2026 22:54
@goldjg
goldjg merged commit d622d65 into main Jun 30, 2026
1 check passed
@goldjg
goldjg deleted the copilot/harden-generated-artifact-boundaries branch June 30, 2026 22:55
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