Adopt the profiled repository contract - #1
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Introduces a repository-level contract for the “builder context” shape, including a declarative repo.toml, documentation describing canonical vs non-canonical artifacts, and a single make check entrypoint that is enforced in CI.
Changes:
- Adds
repo.tomlandscripts/check_context.pyto validate required paths plus theskillfoundry.tomlrepository/frontdoor declarations. - Adds
make checkand wires it into a least-privilege GitHub Actions workflow. - Documents the contract and transition exceptions via
AGENTS.md,CLAUDE.md,docs/architecture.md, and a README update; adds Dependabot for GitHub Actions.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/check_context.py | Implements contract validation for required repo structure and pinned front-door files. |
| repo.toml | Declares repository schema/version, shape, and authoritative artifact roots. |
| README.md | Adds lifecycle/verification section pointing to make check and architecture doc. |
| Makefile | Provides make check entrypoint for contract validation and patch hygiene checks. |
| docs/architecture.md | Documents artifact roles (canon vs non-canon) and dated transition exceptions. |
| CLAUDE.md | Adds Claude adapter front-door instruction pointer to AGENTS + pinned paths. |
| AGENTS.md | Defines the builder-context contribution/contract expectations and make check guidance. |
| .github/workflows/check.yml | Adds CI workflow to run make check with pinned action SHAs. |
| .github/dependabot.yml | Enables monthly updates for GitHub Actions dependencies. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+20
to
+22
| for path in context.get("frontdoor", {}).get("pinned_paths", []): | ||
| if not (ROOT / path).is_file(): | ||
| raise SystemExit(f"missing pinned front-door path: {path}") |
| @printf '%s\n' 'make check Validate context declaration, pinned front door, and patch hygiene' | ||
| check: | ||
| python3 scripts/check_context.py | ||
| git diff --check |
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.
Adds the ADR-0050 repository declaration, universal make check interface, architecture/front-door documentation, pinned least-privilege CI, and Dependabot configuration. Shape-specific checks preserve existing domain boundaries. Dated exceptions remain explicit, so this does not claim full conformance before prompt-eval, typing, runtime-path, dependency-lock, or host-containment gaps close.