From 810fc9de224a5d23abcc07a9f317db6a74ee4d84 Mon Sep 17 00:00:00 2001 From: Pieter Viljoen Date: Thu, 3 Sep 2026 10:24:18 -0700 Subject: [PATCH 1/6] Point python-codestyle testing.md at D1.6's pytest-cov obligation WORKFLOW.md D1.6 requires a build-profile Python repo with tests to declare pytest-cov as a dev dependency, select the coverage source via an addopts --cov= entry, and leave the report at the repo root as coverage.xml. testing.md said nothing about this, so an agent authoring a pyproject.toml learned of the obligation from a red CI run instead. Add a short pointer naming the obligation and citing D1.6 as the authority, per the no-restatement rule for carried instruction content. Regenerated .github/skills/** and .claude-plugin/** via scripts/build_dist.py. Fixes #1220 Co-Authored-By: Claude Opus 5 (1M context) --- .agents/skills/python-codestyle/references/testing.md | 3 +++ .claude-plugin/fleet-skills/.source-digest | 2 +- .../skills/python-codestyle/references/testing.md | 3 +++ .github/skills/python-codestyle/references/testing.md | 3 +++ reports/canonical-review.json | 8 ++++++++ reports/canonical-review.md | 8 ++------ 6 files changed, 20 insertions(+), 7 deletions(-) diff --git a/.agents/skills/python-codestyle/references/testing.md b/.agents/skills/python-codestyle/references/testing.md index 0dae3fd4..2148446d 100644 --- a/.agents/skills/python-codestyle/references/testing.md +++ b/.agents/skills/python-codestyle/references/testing.md @@ -7,6 +7,9 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. +**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. Read +D1.6 before writing `pyproject.toml`. + - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. - Use fixtures (defined in `conftest.py` for shared ones, or per-test for narrowly-scoped) instead diff --git a/.claude-plugin/fleet-skills/.source-digest b/.claude-plugin/fleet-skills/.source-digest index 39c23d5d..c6d0e75f 100644 --- a/.claude-plugin/fleet-skills/.source-digest +++ b/.claude-plugin/fleet-skills/.source-digest @@ -1 +1 @@ -48c6cd114e4cc195 +e7e289575f6d6925 diff --git a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md index 0dae3fd4..2148446d 100644 --- a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md +++ b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md @@ -7,6 +7,9 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. +**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. Read +D1.6 before writing `pyproject.toml`. + - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. - Use fixtures (defined in `conftest.py` for shared ones, or per-test for narrowly-scoped) instead diff --git a/.github/skills/python-codestyle/references/testing.md b/.github/skills/python-codestyle/references/testing.md index 0dae3fd4..2148446d 100644 --- a/.github/skills/python-codestyle/references/testing.md +++ b/.github/skills/python-codestyle/references/testing.md @@ -7,6 +7,9 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. +**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. Read +D1.6 before writing `pyproject.toml`. + - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. - Use fixtures (defined in `conftest.py` for shared ones, or per-test for narrowly-scoped) instead diff --git a/reports/canonical-review.json b/reports/canonical-review.json index 4eed92ee..f6074ae4 100644 --- a/reports/canonical-review.json +++ b/reports/canonical-review.json @@ -193,6 +193,14 @@ "hubCommit": "5eb14540421d630293807288845284a78f8ecc7d", "stamp": "2026-09-02T03:51:13Z" }, + { + "unit": ".agents/skills/python-codestyle/references/testing.md", + "digest": "sha256:c4cc0be5dea91f803b9949f6e391af6e9ef7dab88a6dfae0b2971273983888c3", + "reviewer": "agent-skill", + "findings": 0, + "hubCommit": "6525cb888406c7f6a1f43c5f9b21fb041200c601", + "stamp": "2026-09-03T17:24:11Z" + }, { "unit": ".agents/skills/repo-worktree/SKILL.md > Creating a Worktree", "digest": "sha256:5d8b1dff2f47416ac8aa52b612837e9836316a6a374522956df42124c07235c5", diff --git a/reports/canonical-review.md b/reports/canonical-review.md index 153fa9eb..ccab5c25 100644 --- a/reports/canonical-review.md +++ b/reports/canonical-review.md @@ -7,9 +7,9 @@ A unit is what a reviewer reads whole, decided by the carry manifest rather than ## Coverage - units: 288 -- covered: 59 +- covered: 60 - stale: 0 -- never read here: 229 +- never read here: 228 ## Burn-down @@ -241,10 +241,6 @@ A unit is what a reviewer reads whole, decided by the carry manifest rather than - **Two profiles: full specification** - never - **Versioning** - never -### .agents/skills/python-codestyle/references/testing.md - -- **(whole file)** - never - ### .agents/skills/repo-worktree/SKILL.md - **(preamble)** - never From 6748afdbabf14201dab6f6ee4e58ec0482952955 Mon Sep 17 00:00:00 2001 From: Pieter Viljoen Date: Thu, 3 Sep 2026 10:29:38 -0700 Subject: [PATCH 2/6] Name WORKFLOW.md where the pointer's second sentence stands alone Round-1 review on PR #1230 found the "Read D1.6" sentence, taken on its own line, doesn't name where D1.6 lives to a reader who hasn't already seen the first sentence. Keep the WORKFLOW.md/D1.6 pairing on one line so the pointer reads correctly regardless of markdown reflow. Co-Authored-By: Claude Opus 5 (1M context) --- .agents/skills/python-codestyle/references/testing.md | 4 ++-- .claude-plugin/fleet-skills/.source-digest | 2 +- .../skills/python-codestyle/references/testing.md | 4 ++-- .github/skills/python-codestyle/references/testing.md | 4 ++-- reports/canonical-review.json | 6 +++--- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/.agents/skills/python-codestyle/references/testing.md b/.agents/skills/python-codestyle/references/testing.md index 2148446d..e1019648 100644 --- a/.agents/skills/python-codestyle/references/testing.md +++ b/.agents/skills/python-codestyle/references/testing.md @@ -7,8 +7,8 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. Read -D1.6 before writing `pyproject.toml`. +**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. +Read D1.6 before writing `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.claude-plugin/fleet-skills/.source-digest b/.claude-plugin/fleet-skills/.source-digest index c6d0e75f..6d82e1be 100644 --- a/.claude-plugin/fleet-skills/.source-digest +++ b/.claude-plugin/fleet-skills/.source-digest @@ -1 +1 @@ -e7e289575f6d6925 +db71754533b4a667 diff --git a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md index 2148446d..e1019648 100644 --- a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md +++ b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md @@ -7,8 +7,8 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. Read -D1.6 before writing `pyproject.toml`. +**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. +Read D1.6 before writing `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.github/skills/python-codestyle/references/testing.md b/.github/skills/python-codestyle/references/testing.md index 2148446d..e1019648 100644 --- a/.github/skills/python-codestyle/references/testing.md +++ b/.github/skills/python-codestyle/references/testing.md @@ -7,8 +7,8 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. Read -D1.6 before writing `pyproject.toml`. +**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. +Read D1.6 before writing `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/reports/canonical-review.json b/reports/canonical-review.json index f6074ae4..2f317937 100644 --- a/reports/canonical-review.json +++ b/reports/canonical-review.json @@ -195,11 +195,11 @@ }, { "unit": ".agents/skills/python-codestyle/references/testing.md", - "digest": "sha256:c4cc0be5dea91f803b9949f6e391af6e9ef7dab88a6dfae0b2971273983888c3", + "digest": "sha256:b9bd1b97ac9953aa9d34435938cb093570d1e33f51380bf1f4cd2ff73c2dfc40", "reviewer": "agent-skill", "findings": 0, - "hubCommit": "6525cb888406c7f6a1f43c5f9b21fb041200c601", - "stamp": "2026-09-03T17:24:11Z" + "hubCommit": "810fc9de224a5d23abcc07a9f317db6a74ee4d84", + "stamp": "2026-09-03T17:29:24Z" }, { "unit": ".agents/skills/repo-worktree/SKILL.md > Creating a Worktree", From 8ea4bf3032dc31dbbbf2885ba4c1676517080715 Mon Sep 17 00:00:00 2001 From: Pieter Viljoen Date: Thu, 3 Sep 2026 10:31:49 -0700 Subject: [PATCH 3/6] Cover modifying pyproject.toml, not only creating it CodeRabbit's round-1 review on PR #1230 found the trigger only named writing a new pyproject.toml, missing an agent editing an existing one. Co-Authored-By: Claude Opus 5 (1M context) --- .agents/skills/python-codestyle/references/testing.md | 2 +- .claude-plugin/fleet-skills/.source-digest | 2 +- .../skills/python-codestyle/references/testing.md | 2 +- .github/skills/python-codestyle/references/testing.md | 2 +- reports/canonical-review.json | 6 +++--- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/.agents/skills/python-codestyle/references/testing.md b/.agents/skills/python-codestyle/references/testing.md index e1019648..9b864504 100644 --- a/.agents/skills/python-codestyle/references/testing.md +++ b/.agents/skills/python-codestyle/references/testing.md @@ -8,7 +8,7 @@ Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocati `uv run pytest`. **Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. -Read D1.6 before writing `pyproject.toml`. +Read D1.6 before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.claude-plugin/fleet-skills/.source-digest b/.claude-plugin/fleet-skills/.source-digest index 6d82e1be..d3bc9fb1 100644 --- a/.claude-plugin/fleet-skills/.source-digest +++ b/.claude-plugin/fleet-skills/.source-digest @@ -1 +1 @@ -db71754533b4a667 +a1f3f742323c72ae diff --git a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md index e1019648..9b864504 100644 --- a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md +++ b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md @@ -8,7 +8,7 @@ Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocati `uv run pytest`. **Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. -Read D1.6 before writing `pyproject.toml`. +Read D1.6 before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.github/skills/python-codestyle/references/testing.md b/.github/skills/python-codestyle/references/testing.md index e1019648..9b864504 100644 --- a/.github/skills/python-codestyle/references/testing.md +++ b/.github/skills/python-codestyle/references/testing.md @@ -8,7 +8,7 @@ Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocati `uv run pytest`. **Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. -Read D1.6 before writing `pyproject.toml`. +Read D1.6 before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/reports/canonical-review.json b/reports/canonical-review.json index 2f317937..d3e5694d 100644 --- a/reports/canonical-review.json +++ b/reports/canonical-review.json @@ -195,11 +195,11 @@ }, { "unit": ".agents/skills/python-codestyle/references/testing.md", - "digest": "sha256:b9bd1b97ac9953aa9d34435938cb093570d1e33f51380bf1f4cd2ff73c2dfc40", + "digest": "sha256:74cc0b107cc85145502ffa587113133127434dce21b26d00ba14625692869ae6", "reviewer": "agent-skill", "findings": 0, - "hubCommit": "810fc9de224a5d23abcc07a9f317db6a74ee4d84", - "stamp": "2026-09-03T17:29:24Z" + "hubCommit": "6748afdbabf14201dab6f6ee4e58ec0482952955", + "stamp": "2026-09-03T17:31:29Z" }, { "unit": ".agents/skills/repo-worktree/SKILL.md > Creating a Worktree", From d5ca8b2a7bbbf0902d363c8f10e6f166d16186d7 Mon Sep 17 00:00:00 2001 From: Pieter Viljoen Date: Thu, 3 Sep 2026 10:38:53 -0700 Subject: [PATCH 4/6] Backtick the WORKFLOW.md reference and drop its possessive Copilot's round-3 review on PR #1230 found "WORKFLOW.md's" unbacktickled and possessive, inconsistent with the surrounding pyproject.toml and references/profiles.md path styling in the same file. Co-Authored-By: Claude Opus 5 (1M context) --- .agents/skills/python-codestyle/references/testing.md | 2 +- .claude-plugin/fleet-skills/.source-digest | 2 +- .../skills/python-codestyle/references/testing.md | 2 +- .github/skills/python-codestyle/references/testing.md | 2 +- reports/canonical-review.json | 6 +++--- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/.agents/skills/python-codestyle/references/testing.md b/.agents/skills/python-codestyle/references/testing.md index 9b864504..9cb1fb9d 100644 --- a/.agents/skills/python-codestyle/references/testing.md +++ b/.agents/skills/python-codestyle/references/testing.md @@ -7,7 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. +**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations. Read D1.6 before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. diff --git a/.claude-plugin/fleet-skills/.source-digest b/.claude-plugin/fleet-skills/.source-digest index d3bc9fb1..153fa0f4 100644 --- a/.claude-plugin/fleet-skills/.source-digest +++ b/.claude-plugin/fleet-skills/.source-digest @@ -1 +1 @@ -a1f3f742323c72ae +417493b59ae60695 diff --git a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md index 9b864504..9cb1fb9d 100644 --- a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md +++ b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md @@ -7,7 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. +**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations. Read D1.6 before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. diff --git a/.github/skills/python-codestyle/references/testing.md b/.github/skills/python-codestyle/references/testing.md index 9b864504..9cb1fb9d 100644 --- a/.github/skills/python-codestyle/references/testing.md +++ b/.github/skills/python-codestyle/references/testing.md @@ -7,7 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes WORKFLOW.md's D1.6 coverage obligations. +**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations. Read D1.6 before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. diff --git a/reports/canonical-review.json b/reports/canonical-review.json index d3e5694d..87060cb2 100644 --- a/reports/canonical-review.json +++ b/reports/canonical-review.json @@ -195,11 +195,11 @@ }, { "unit": ".agents/skills/python-codestyle/references/testing.md", - "digest": "sha256:74cc0b107cc85145502ffa587113133127434dce21b26d00ba14625692869ae6", + "digest": "sha256:357dab20264a34ba01815ae078365b206de5e5f1ccf50ae6f7b36585a8b5df93", "reviewer": "agent-skill", "findings": 0, - "hubCommit": "6748afdbabf14201dab6f6ee4e58ec0482952955", - "stamp": "2026-09-03T17:31:29Z" + "hubCommit": "8ea4bf3032dc31dbbbf2885ba4c1676517080715", + "stamp": "2026-09-03T17:38:42Z" }, { "unit": ".agents/skills/repo-worktree/SKILL.md > Creating a Worktree", From e56d99db54bd0d89f44904b7e76de366ca41c124 Mon Sep 17 00:00:00 2001 From: Pieter Viljoen Date: Thu, 3 Sep 2026 10:44:05 -0700 Subject: [PATCH 5/6] Collapse the pointer to one self-contained sentence Round-4 review on PR #1230 found the second sentence unlocatable when read or quoted apart from the first. Merge both into one sentence naming WORKFLOW.md and D1.6 together, so no partial quote loses the reference. Co-Authored-By: Claude Opus 5 (1M context) --- .agents/skills/python-codestyle/references/testing.md | 3 +-- .claude-plugin/fleet-skills/.source-digest | 2 +- .../skills/python-codestyle/references/testing.md | 3 +-- .github/skills/python-codestyle/references/testing.md | 3 +-- reports/canonical-review.json | 6 +++--- 5 files changed, 7 insertions(+), 10 deletions(-) diff --git a/.agents/skills/python-codestyle/references/testing.md b/.agents/skills/python-codestyle/references/testing.md index 9cb1fb9d..8c498cc7 100644 --- a/.agents/skills/python-codestyle/references/testing.md +++ b/.agents/skills/python-codestyle/references/testing.md @@ -7,8 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations. -Read D1.6 before creating or modifying `pyproject.toml`. +**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.claude-plugin/fleet-skills/.source-digest b/.claude-plugin/fleet-skills/.source-digest index 153fa0f4..e1032c20 100644 --- a/.claude-plugin/fleet-skills/.source-digest +++ b/.claude-plugin/fleet-skills/.source-digest @@ -1 +1 @@ -417493b59ae60695 +6a599187aaac248e diff --git a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md index 9cb1fb9d..8c498cc7 100644 --- a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md +++ b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md @@ -7,8 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations. -Read D1.6 before creating or modifying `pyproject.toml`. +**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.github/skills/python-codestyle/references/testing.md b/.github/skills/python-codestyle/references/testing.md index 9cb1fb9d..8c498cc7 100644 --- a/.github/skills/python-codestyle/references/testing.md +++ b/.github/skills/python-codestyle/references/testing.md @@ -7,8 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations. -Read D1.6 before creating or modifying `pyproject.toml`. +**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations before creating or modifying `pyproject.toml`. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/reports/canonical-review.json b/reports/canonical-review.json index 87060cb2..6c9c6722 100644 --- a/reports/canonical-review.json +++ b/reports/canonical-review.json @@ -195,11 +195,11 @@ }, { "unit": ".agents/skills/python-codestyle/references/testing.md", - "digest": "sha256:357dab20264a34ba01815ae078365b206de5e5f1ccf50ae6f7b36585a8b5df93", + "digest": "sha256:102d7d639db9503617243122be11be8642df35b791aed80dd7af3c682012f8d2", "reviewer": "agent-skill", "findings": 0, - "hubCommit": "8ea4bf3032dc31dbbbf2885ba4c1676517080715", - "stamp": "2026-09-03T17:38:42Z" + "hubCommit": "d5ca8b2a7bbbf0902d363c8f10e6f166d16186d7", + "stamp": "2026-09-03T17:43:54Z" }, { "unit": ".agents/skills/repo-worktree/SKILL.md > Creating a Worktree", From 1bf1add3cb5b79c886fe73ad15f40d9ebb19dba2 Mon Sep 17 00:00:00 2001 From: Pieter Viljoen Date: Thu, 3 Sep 2026 10:49:00 -0700 Subject: [PATCH 6/6] Fix the ordering claim to be an instruction, not a precondition Round-5 review on PR #1230 found "owes ... obligations before creating" reads as though D1.6's obligations must be met before the file exists, when they are actually implemented in pyproject.toml itself. Reword as an instruction to read D1.6 before touching the file. Co-Authored-By: Claude Opus 5 (1M context) --- .agents/skills/python-codestyle/references/testing.md | 2 +- .claude-plugin/fleet-skills/.source-digest | 2 +- .../skills/python-codestyle/references/testing.md | 2 +- .github/skills/python-codestyle/references/testing.md | 2 +- reports/canonical-review.json | 6 +++--- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/.agents/skills/python-codestyle/references/testing.md b/.agents/skills/python-codestyle/references/testing.md index 8c498cc7..49a867c4 100644 --- a/.agents/skills/python-codestyle/references/testing.md +++ b/.agents/skills/python-codestyle/references/testing.md @@ -7,7 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations before creating or modifying `pyproject.toml`. +**Coverage.** Before creating or modifying `pyproject.toml`, read `WORKFLOW.md` D1.6 for the coverage obligations a build-profile repo with tests owes. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.claude-plugin/fleet-skills/.source-digest b/.claude-plugin/fleet-skills/.source-digest index e1032c20..ef524e2a 100644 --- a/.claude-plugin/fleet-skills/.source-digest +++ b/.claude-plugin/fleet-skills/.source-digest @@ -1 +1 @@ -6a599187aaac248e +d96477900c9c348b diff --git a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md index 8c498cc7..49a867c4 100644 --- a/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md +++ b/.claude-plugin/fleet-skills/skills/python-codestyle/references/testing.md @@ -7,7 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations before creating or modifying `pyproject.toml`. +**Coverage.** Before creating or modifying `pyproject.toml`, read `WORKFLOW.md` D1.6 for the coverage obligations a build-profile repo with tests owes. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/.github/skills/python-codestyle/references/testing.md b/.github/skills/python-codestyle/references/testing.md index 8c498cc7..49a867c4 100644 --- a/.github/skills/python-codestyle/references/testing.md +++ b/.github/skills/python-codestyle/references/testing.md @@ -7,7 +7,7 @@ are in `references/profiles.md`. Use `pytest` with configuration in `[tool.pytest.ini_options]`. Default invocation: `uv run pytest`. -**Coverage.** A build-profile repo with tests owes `WORKFLOW.md` D1.6's coverage obligations before creating or modifying `pyproject.toml`. +**Coverage.** Before creating or modifying `pyproject.toml`, read `WORKFLOW.md` D1.6 for the coverage obligations a build-profile repo with tests owes. - One test file per module under test, named `test_.py`. - Test functions named `test__`, descriptive and not numbered. diff --git a/reports/canonical-review.json b/reports/canonical-review.json index 6c9c6722..1d999bda 100644 --- a/reports/canonical-review.json +++ b/reports/canonical-review.json @@ -195,11 +195,11 @@ }, { "unit": ".agents/skills/python-codestyle/references/testing.md", - "digest": "sha256:102d7d639db9503617243122be11be8642df35b791aed80dd7af3c682012f8d2", + "digest": "sha256:58de79a2634bd61f5d3d411b550b99537bba437a23a36f0e6995dd74ea67b1c2", "reviewer": "agent-skill", "findings": 0, - "hubCommit": "d5ca8b2a7bbbf0902d363c8f10e6f166d16186d7", - "stamp": "2026-09-03T17:43:54Z" + "hubCommit": "e56d99db54bd0d89f44904b7e76de366ca41c124", + "stamp": "2026-09-03T17:48:49Z" }, { "unit": ".agents/skills/repo-worktree/SKILL.md > Creating a Worktree",