Skip to content

Re-vendor Stale Carried Sections to the Current Hub Canonical - #94

Merged
ptr727 merged 1 commit into
developfrom
resync/skills-rollout-phase8
Aug 13, 2026
Merged

Re-vendor Stale Carried Sections to the Current Hub Canonical#94
ptr727 merged 1 commit into
developfrom
resync/skills-rollout-phase8

Conversation

@ptr727

@ptr727 ptr727 commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Runs RESYNC.md against the current hub (ptr727/ProjectTemplate main@75a0d45), driven via the new resync-a-repo skill. Part of the hub's Phase 8 fleet-wide skills rollout, using Blog as a pilot repo.

What changed

spec/audit.py Blog found AGENTS.md "Where the Rules Live" and nine GOVERNANCE.md sections (Branching Model, Documentation Style Conventions, Git and Commit Rules, Hub-Hosted Tooling, Operational Repositories, PR Review Etiquette, Pull Request Title and Commit Message Conventions, Release Model, Workflow YAML Conventions) each matching a past hub revision rather than the current canonical. Each was re-vendored byte-exact per declared section, run through the hub's carried-instruction-file-guard probe first (every section already matched a past hub revision exactly, so the file was cleanly split with no local additions at risk). LF line endings preserved throughout, matching this repo's own convention (the hub carries these files as CRLF).

AGENTS.md's "Where the Rules Live" now carries the skill-dependency pointer paragraph naming scripts/skills_install.py and where the fleet's Skills live.

What's left

Two hub-only findings remain (.github/workflows/publish-release.yml, .github/workflows/validate-task.yml), both dispositioned investigate in the hub's spec/divergences.json pending a fidelity/appliesTo decision — not actionable from this repo.

Verification

python3 spec/audit.py --branch resync/skills-rollout-phase8 Blog

0 defect findings; only the two pre-existing investigate-dispositioned hub-only findings remain.

🤖 Generated with Claude Code

AGENTS.md 'Where the Rules Live' and nine GOVERNANCE.md sections
(Branching Model, Documentation Style Conventions, Git and Commit
Rules, Hub-Hosted Tooling, Operational Repositories, PR Review
Etiquette, Pull Request Title and Commit Message Conventions,
Release Model, Workflow YAML Conventions) matched a past hub
revision rather than the current canonical. Re-vendored from
ptr727/ProjectTemplate main@75a0d45, byte-exact per section, LF
line endings preserved to match this repo's own convention.

The AGENTS.md section now carries the skill-dependency pointer
paragraph naming scripts/skills_install.py and the fleet's Skills.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 13, 2026 19:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR re-vendors carried governance content to match the current hub canonical, updating Blog’s AGENTS.md/GOVERNANCE.md to reflect the hub’s newer “skills-packaged rules” structure and pointers (intended to reduce duplicated rule prose in downstream repos while keeping the authoritative mechanics where needed).

Changes:

  • Replace several previously fully inlined governance sections with short, carried summaries that point to the corresponding hub Skills for the full contract.
  • Update AGENTS.md “Where the Rules Live” mapping to note which sections are packaged as Skills and add the skills installer/pointer paragraph.
  • Refresh GOVERNANCE.md sections including branching/release/PR review guidance and hub-hosted tooling notes, plus updated local linter invocation guidance.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
GOVTERNANCE.md Re-vendored multiple governance sections to the hub’s current canonical, including Skill pointers and updated CI/lint/review guidance.
AGENTS.md Updates the rules map to include Skill packaging notes and adds the hub Skill installation/pointer paragraph.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread GOVERNANCE.md
@ptr727
ptr727 merged commit 4fd753c into develop Aug 13, 2026
5 checks passed
@ptr727
ptr727 deleted the resync/skills-rollout-phase8 branch August 13, 2026 19:21
ptr727 added a commit to ptr727/ProjectTemplate that referenced this pull request Aug 13, 2026
GOVERNANCE.md's "Running the Linters Locally" bullet unconditionally
claimed CI runs PSScriptAnalyzer and linked
`PSScriptAnalyzerSettings.psd1`, true for the hub itself but not for
every downstream repo.

Hit as a real Copilot finding on both
[Blog#94](ptr727/Blog#94) (a stale carried
`validate-task.yml` never wired the step) and
[HomeAutomation-Config#50](ptr727/HomeAutomation-Config#50)
(the repo carries no `.ps1` files at all, so the link 404s). Neither was
fixable per-repo without either breaking byte-parity on the verbatim
bullet or inventing content the repo does not need, so both were
replied-and-deferred as the maintainer's call.

This conditions both bullets on a repo carrying `.ps1` files instead,
closing the recurring gap at its source rather than continuing to defer
it repo by repo through the rest of the Phase 8 rollout.

Not manifest-tracked ("Running the Linters Locally" is not a declared
verbatim section in `spec/files.json`), so this is a hub-only prose fix
with no spec/audit coupling to update.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
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