Repository navigation
Resync the Carried Instruction Set and Fleet Skills with the Hub - #1242
Conversation
Re-vendors the verbatim sections the hub audit at 9f3e03cf reports as matching a past hub revision, with carry.py apply-sections. The new "Editor and Tasks" rule has a repository shipping Python ignore .vscode/settings.json, so .gitignore gains that entry. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Re-vendors .github/skills from hub 9f3e03cf with carry.py apply: every stale skill file is refreshed, decision-triage is added, and the retired unattended-handoff skill is pruned. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The re-vendored "Editor and Tasks" section names .vscode/settings.json, which this repository tracked once and now ignores. The prose gate's dead-path rule reads that byte-locked mention as stale, so the root .prose-gate.json lists the path under hub_hosted, as NxWitness does. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The re-vendored skills include the hub's newer CODESTYLE.md "Command-Line Arguments" and env-file naming text, and pr-review-conduct's newer Merge Gate. This repository's copies of those carried lines trailed them, so they take the hub text at 9f3e03cf. The Disproved Claims ledger and the repository's own CODESTYLE sections are unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
A recorded local strict-review pass covers head |
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the
|
PR Summary by QodoResync fleet instructions and skills with the hub
AI Description
Diagram
High-Level Assessment
Files changed (31)
|
Code Review by Qodo
1. Merged promotions can miss their release
|
|
@qodo-free-for-open-source-projects please dismiss findings 1 and 2. Both are declined here as out of scope for this repository.
|
I can't dismiss findings 1 and 2 right now. The findings catalog for this PR is empty, so I can't match either finding to a stable ID, and dismissal requires one. Nothing has been changed. The published report does list both findings by title: "Merged promotions can miss their release" (finding 1) and "Answered questions remain open as work" (finding 2). If the review's findings aren't indexed yet, the dismissal can't go through. Please try again once the review has finished syncing. If it still fails, you can dismiss them in the Qodo portal or the PR review UI. |
There was a problem hiding this comment.
🟡 Changes recommended
The carried Python profile conflicts with this repository’s existing CI guidance on the Ruff version used for RegressionTests, leaving agents with contradictory instructions that must be reconciled.
1 open issue
What changed in this PR
This pull request resynchronizes PlexCleaner's carried instruction set and generated fleet skills with ProjectTemplate hub revision 9f3e03c. It also adds decision-triage, removes the retired unattended-handoff skill, and updates repository metadata for hub-hosted editor settings.
Changes:
- Refreshes carried governance, coding-style, workflow, review, and Copilot guidance.
- Updates the fleet skills tree and replaces retired handoff behavior with backlog-burndown away mode.
- Adds
.vscode/settings.jsonhandling to.gitignoreand.prose-gate.json. - Leaves one documentation consistency nit regarding conflicting Ruff version guidance in the Python profile and repository CI documentation.
This pull request is too complex or risky for automated approval.
| File | Description |
|---|---|
CODESTYLE.md |
Refreshes command-line argument and environment-file naming rules. |
AGENTS.md |
Updates the rules map and skill descriptions. |
.prose-gate.json |
Declares the hub-hosted VS Code settings path. |
.gitignore |
Ignores generated VS Code settings. |
.github/skills/workflow-ci-contract/SKILL.md |
Refreshes workflow contract guidance. |
.github/skills/workflow-ci-contract/references/test-methodology.md |
Updates workflow test scenarios. |
.github/skills/workflow-ci-contract/references/d-guarantees.md |
Refreshes workflow validation and guarantee definitions. |
.github/skills/upstream-contribution-workflow/SKILL.md |
Refreshes upstream contribution guidance. |
.github/skills/unattended-handoff/SKILL.md |
Removes the retired unattended-handoff skill. |
.github/skills/skill-lifecycle/SKILL.md |
Updates skill generation and validation rules. |
.github/skills/shell-codestyle/SKILL.md |
Refreshes shell and command-line guidance. |
.github/skills/session-handoff/SKILL.md |
Updates handoff and away-mode procedures. |
.github/skills/repo-worktree/SKILL.md |
Adds operational-repository worktree guidance. |
.github/skills/python-codestyle/SKILL.md |
Refreshes Python style and toolchain guidance. |
.github/skills/python-codestyle/references/profiles.md |
Updates Python profile and Ruff versioning guidance. |
.github/skills/pr-review-conduct/SKILL.md |
Refreshes review coverage and finding-handling rules. |
.github/skills/merge-and-release/SKILL.md |
Updates promotion, release, and cleanup procedures. |
.github/skills/local-strict-review/SKILL.md |
Updates review-report and recording mechanics. |
.github/skills/git-commit-conventions/SKILL.md |
Adds operational repository commit guidance. |
.github/skills/fleet-code-review/SKILL.md |
Refines class-sweep and reuse review guidance. |
.github/skills/drive-pr/SKILL.md |
Updates pull-request driving and merge mechanics. |
.github/skills/dotnet-codestyle/SKILL.md |
Refreshes .NET style guidance. |
.github/skills/decision-triage/SKILL.md |
Adds maintainer decision-triage procedures. |
.github/skills/copilot-instructions-keeper/SKILL.md |
Refreshes Copilot instruction synchronization guidance. |
.github/skills/comment-and-doc-style/SKILL.md |
Updates prose, sentence-length, and comment guidance. |
.github/skills/check-this-repo/SKILL.md |
Refreshes repository currency-check guidance. |
.github/skills/carried-instruction-file-guard/SKILL.md |
Updates carried-file overwrite protection guidance. |
.github/skills/branching-and-release-model/SKILL.md |
Adds operational repository commit and release rules. |
.github/skills/audit-a-repo/SKILL.md |
Refreshes repository audit routing. |
.github/skills/agent-conduct/SKILL.md |
Updates verification and review conduct guidance. |
.github/copilot-instructions.md |
Refreshes the Copilot review runbook. |
Review effort: Lite
|
@coderabbitai review |
Action not completed
|
The carried python-codestyle profile states CI runs ruff at the version the hub validator pins, and the validator at this repository's pin does. CODESTYLE.md and RegressionTests/README.md said CI runs the latest of every tool, so they now name the pinned ruff beside the latest mypy and coverage. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
A recorded local strict-review pass covers head |
|
Qodo re-keyed the same two findings on head aef4146 (merged promotions can miss their release, and answered questions remain open as work). Both stay declined as carried hub text, on the evidence in the earlier answer: |

Summary
Resyncs the carried instruction-set sections and the fleet skills tree from
ptr727/ProjectTemplateat hub9f3e03c, steps 1 and 3 of the hub'sRESYNC.md. The two land together because the skills carry generated include regions copied from these sameGOVERNANCE.mdandAGENTS.mdsections. Also, Copilot reviews with the head branch'sfleet-code-reviewskill.AGENTS.md: re-vendors the "Where the Rules Live" verbatim section.GOVERNANCE.md: re-vendors the verbatim sections the audit reports as matching a past hub revision. They are "Repository Boundaries and Write Safety", "Representative Data in Agent-Authored Text", "Git and Commit Rules", "Operational Repositories", "Hub-Hosted Tooling", "Documentation Style Conventions", "Verification Discipline", "Design Before Code", "PR Review Etiquette", "Communicating with the User", "Workflow YAML Conventions", and "Editor and Tasks"..github/skills/: refreshes every stale carried skill, addsdecision-triage, and prunesunattended-handoff, which the hub retired intobacklog-burndown's away mode.CODESTYLE.mdand.github/copilot-instructions.md: the carried lines the refreshed skills include or restate take the hub text. Those are the General "Command-Line Arguments" and env-file naming bullets, and the runbook's Merge Gate paragraph. The repository's own CODESTYLE sections and its Disproved Claims entries are unchanged.CODESTYLE.md"PlexCleaner Python Conventions" andRegressionTests/README.md: say CI runs ruff at the version the hub validator pins, beside the latest mypy and coverage, which is what the validator at this repository's pin runs and what the refreshedpython-codestyleprofile states..gitignore: lists.vscode/settings.json, which the re-vendored "Editor and Tasks" section requires of a repository shipping Python..prose-gate.json: lists.vscode/settings.jsonunderhub_hosted. This repository tracked that file once, so the prose gate'sdead-pathrule reads the byte-locked "Editor and Tasks" mention as stale. The hub-side defect is recorded on GOVERNANCE.md Editor and Tasks Names .vscode/settings.json to Forbid It, Which Every Carrier That Ever Deleted That File Reads as a Dead Path ProjectTemplate#1612, whose reopening is the maintainer's call.Verification
spec/audit.py --branch develop PlexCleaner(audit run 2026-11-10T01:35:50Z, hub9f3e03cf) classified every re-vendored section and skill file as stale, so none holds a local edit. The pruned skill is byte-identical to a past hub revision.carry.py apply-sectionstouched only the declared verbatim regions and asserted the rest of each file byte-identical.carry.py applyreports the skills tree digest equal to the hub's, anddiff -rqagainst the hub's.github/skillsis empty..github/skillspath named by.github/copilot-instructions.mdchanges.mainand at thevalidate-task.ymlpin. The pre-commit markdownlint and CSpell runs are clean.The rest of the intent-fidelity drift the audit reports, in
AUDIT.mdand.editorconfig-checker.json, is left for a separate change.Pre-existing findings, filed
RegressionTests/synthetic/inject_cc_sei.pyandinject_hdr10plus_sei.py, correctness: neither answers--helpnor refuses an unknown argument, per the carried "Command-Line Arguments" rule. Filed as Answer --help and Refuse Unknown Arguments in the Synthetic SEI Inject Scripts #1241.Pre-existing findings, filed at the hub
The local passes raised these against carried hub skill text this repository cannot edit. All three are filed as ptr727/ProjectTemplate#4023.
.github/skills/session-handoff/SKILL.mdstep 0, carried-content gap: a session's fresh hub clone under the worktrees directory has no removal step..github/skills/decision-triage/SKILL.mdand.github/skills/backlog-burndown/SKILL.md, carried-content gap: adeferredissue whose condition is checkable state is never re-checked by any seat..github/skills/backlog-burndown/SKILL.mdaway scopedevelop, carried-content gap: a run ending on a failed dispatch or a session end opens no promotion pull request and reports nothing about it.🤖 Generated with Claude Code