Skip to content

Adopt ACM v1.1 PR governance in alt-asset-spec - #4

Merged
roblarsen merged 2 commits into
mainfrom
copilot/research-comparison-acm-standards
Sep 3, 2026
Merged

roblarsen merged 2 commits into
mainfrom
copilot/research-comparison-acm-standards

Conversation

Copilot AI commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

This PR aligns alt-asset-spec with the ACM standard by introducing PR-level boundary contracts and automated governance checks. It adds the minimum repository scaffolding needed to require and validate ACM content in pull requests.

  • PR governance template

    • Adds .github/PULL_REQUEST_TEMPLATE.md with:
      • required <!-- ACM-START --> ... <!-- ACM-END --> block
      • ACM v1.1 typed contracts and distributed primitives
      • reviewer sign-off checklist for unsupported/unknown boundaries and evidence verification
  • CI governance gate

    • Adds .github/workflows/acm-governance.yml to run on PR open/edit/sync/reopen
    • Validates PR body ACM content via @roblarsen/acm-cli
    • Posts/updates a governance audit comment and fails the workflow on invalid or missing ACM
  • Local workflow alignment

    • Updates package.json with:
      • acm:validate:file
      • acm:validate:pr-body
    • Ensures contributors can run the same ACM checks locally as CI
  • Contributor and agent documentation

    • Updates README.md to document ACM-required PR descriptions and governance gate behavior
    • Updates llm.md to require ACM v1.1 contract output for implementation/refactor work
{
  "scripts": {
    "acm:validate:file": "npx --yes @roblarsen/acm-cli validate --file",
    "acm:validate:pr-body": "npx --yes @roblarsen/acm-cli validate --pr-body"
  }
}

Copilot AI and others added 2 commits September 3, 2026 15:41
Co-authored-by: roblarsen <361421+roblarsen@users.noreply.github.com>
Co-authored-by: roblarsen <361421+roblarsen@users.noreply.github.com>

@roblarsen roblarsen left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

@roblarsen
roblarsen marked this pull request as ready for review September 3, 2026 15:47
@roblarsen
roblarsen merged commit b348329 into main Sep 3, 2026
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