Skip to content

Mark PhotoCleaner Operational After the Promotion of the b09078e Resync - #739

Merged
ptr727 merged 5 commits into
developfrom
photocleaner-report-main-2026-08-15
Aug 15, 2026
Merged

Mark PhotoCleaner Operational After the Promotion of the b09078e Resync#739
ptr727 merged 5 commits into
developfrom
photocleaner-report-main-2026-08-15

Conversation

@ptr727

@ptr727 ptr727 commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Follow-up to #723 after ptr727/PhotoCleaner#51 promoted develop to main (f56178a), carrying ptr727/PhotoCleaner#50 and ptr727/PhotoCleaner#52.

spec/audit.py PhotoCleaner at hub b09078e reports clean at main (audit run 2026-08-15T18:43:20Z | hub b09078e) and at develop, including the two hub-only workflow findings that #736 settled. The report moves to operational: readme-structure and agent-instruction-set to pass, no defects, no drift, nothing in flight. It records the one-round dead-path block on the promotion (#721, fixed by #731, cleared by a re-run since the gate reads rules from hub develop) and closes both escalations (#721, #722). The registry entry is unchanged: status stays cataloged (the schema's only non-backlog value) and both driftNotes still describe reality (publish shape, private-for-now) and assert no outstanding work.

Report lints clean (markdownlint, prose_lint), and spec/validate.py passes.

🤖 Generated with Claude Code

The deterministic audit is clean at main (f56178a) and develop with the
hub at b09078e, so the report drops the promotion-in-flight framing,
moves readme-structure and agent-instruction-set to pass, records the
dead-path block and its hub fix, and closes the two escalations.
Copilot AI lite review requested due to automatic review settings August 15, 2026 18:44

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

Updates the PhotoCleaner audit report to reflect that develop has been promoted to main and the repository now audits as operational/clean under hub b09078e.

Changes:

  • Refreshes the audited branch SHA, verdict, and run stamp for the latest audit run.
  • Updates drift/convergence/escalation sections to reflect that promotion-related blockers are resolved and the audit is now clean.
Suppressed comments (2)

reports/photocleaner/audit.md:15

  • This line adds an inline Markdown link ([text](url)), but inline links are disallowed here per the repo's reference-style link rule (allowed inline-link exceptions are AGENTS.md, GOVERNANCE.md, OPERATIONS.md, and .github/copilot-instructions.md). Prefer a reference-style link or plain text reference (see .agents/skills/comment-and-doc-style/references/markdown-links.md:9-15,57-65).
The promotion was blocked for one round by the prose gate's `dead-path` rule flagging verbatim mentions of the retired `repo-config/configure.sh`, filed as [#721](https://github.com/ptr727/ProjectTemplate/issues/721) and fixed by #731. The gate fetches its rules from hub `develop` on a `develop`-targeted run, so a re-run cleared it with no repo change. Copilot read 18 of 19 changed files on the promotion in both rounds and raised no thread. Its one suppressed finding, a Mermaid edge in the hub-carried `WORKFLOW.md` diagram, was answered on the pull request as declined, since the diagram is hub content and the D1 text beside it already states the rule.

reports/photocleaner/audit.md:57

  • These list items use inline Markdown links ([text](url)), which violates the repo's reference-style link rule for Markdown files outside the four allowed exceptions. Use reference-style links (with definitions at the bottom) or plain text references instead (see .agents/skills/comment-and-doc-style/references/markdown-links.md:9-15,57-65).
1. [#721](https://github.com/ptr727/ProjectTemplate/issues/721): `prose_lint.py dead-path` could not recognize a hub-hosted path in a repo that retired the file, so verbatim text naming `repo-config/configure.sh` failed a downstream promotion gate. Fixed by #731.
2. [#722](https://github.com/ptr727/ProjectTemplate/issues/722): the hub's `.github/copilot-instructions.md` linked `GOVERNANCE.md#every-finding-ends-in-an-action`, an anchor that left `GOVERNANCE.md` when PR Review Etiquette was packaged as a Skill. PhotoCleaner re-pointed its copy in #50 after Copilot raised it, and #52 carried the canonical wording #730 landed.

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

Comment thread reports/photocleaner/audit.md Outdated
Inline links are reserved for the four named files, and the sibling
reports already use the owner/repo#N autolink form.
Copilot AI review requested due to automatic review settings August 15, 2026 18:48
@ptr727

ptr727 commented Aug 15, 2026

Copy link
Copy Markdown
Owner Author

Answering the two suppressed findings (reports/photocleaner/audit.md:15 and :57, inline Markdown links outside the four permitted files): fixed in 8dd6c99 together with the threaded one, by converting every inline link in the report to a bare owner/repo#N or #N reference.

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (2)

reports/photocleaner/audit.md:15

  • Grammar: "raised no thread" should be plural, since it refers to the absence of any review threads.
The promotion was blocked for one round by the prose gate's `dead-path` rule flagging verbatim mentions of the retired `repo-config/configure.sh`, filed as #721 and fixed by #731. The gate fetches its rules from hub `develop` on a `develop`-targeted run, so a re-run cleared it with no repo change. Copilot read 18 of 19 changed files on the promotion in both rounds and raised no thread. Its one suppressed finding, a Mermaid edge in the hub-carried `WORKFLOW.md` diagram, was answered on the pull request as declined, since the diagram is hub content and the D1 text beside it already states the rule.

reports/photocleaner/audit.md:57

  • Grammar: "the canonical wording #730 landed" is missing a connector word and reads ungrammatically; add "that" (or similar) to clarify the reference.
2. #722: the hub's `.github/copilot-instructions.md` linked `GOVERNANCE.md#every-finding-ends-in-an-action`, an anchor that left `GOVERNANCE.md` when PR Review Etiquette was packaged as a Skill. PhotoCleaner re-pointed its copy in #50 after Copilot raised it, and #52 carried the canonical wording #730 landed.

@ptr727

ptr727 commented Aug 15, 2026

Copy link
Copy Markdown
Owner Author

Answering the two suppressed findings on head 8dd6c99 (reports/photocleaner/audit.md:15 "raised no thread" and :57 "the canonical wording #730 landed"): both accepted and fixed in cf7044d, "raised no threads" and "the canonical wording that #730 landed".

Copilot AI review requested due to automatic review settings August 15, 2026 18:51

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

reports/photocleaner/audit.md:54

  • This section says "Two hub findings from this pass" but the report above states the current audit run is clean (no findings). These escalations appear to be historical carry-forward from the earlier snapshot, so the wording is misleading.
Two hub findings from this pass, filed rather than patched per repo, both closed since:

@ptr727

ptr727 commented Aug 15, 2026

Copy link
Copy Markdown
Owner Author

Answering the suppressed finding on head cf7044d (reports/photocleaner/audit.md:54, "Two hub findings from this pass" reading as current findings): accepted and fixed in 4b1882c, the sentence now says the two escalations were surfaced by the resync the report measures and closed before this run, so neither is a finding of the current audit.

Copilot AI review requested due to automatic review settings August 15, 2026 18:54

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (4)

reports/photocleaner/audit.md:19

  • These unqualified #49/#50/#52 references will auto-link to issues/PRs in this repository, but they appear to be PhotoCleaner PR numbers (the surrounding text is about PhotoCleaner resync/promotions). Use ptr727/PhotoCleaner#… so the references resolve correctly.
Judged at `main`. Type-dimension checks are hand-judged per AUDIT.md section 4 and are unchanged from the 2026-08-01 snapshot, since #49, #50, and #52 touched governance and configuration only.

reports/photocleaner/audit.md:46

  • Unqualified #50/#52/#51 will auto-link to ProjectTemplate issues/PRs, but this line is summarizing PhotoCleaner PR state. Use ptr727/PhotoCleaner#… so the links point to the correct repository.
None. #50, #52 (to `develop`) and #51 (`develop -> main`) are merged.

reports/photocleaner/audit.md:57

  • #50 and #52 here will auto-link to ProjectTemplate issues/PRs, but this sentence is describing actions taken in the PhotoCleaner repository. Use explicit ptr727/PhotoCleaner#… references so readers land on the right PRs.
2. #722: the hub's `.github/copilot-instructions.md` linked `GOVERNANCE.md#every-finding-ends-in-an-action`, an anchor that left `GOVERNANCE.md` when PR Review Etiquette was packaged as a Skill. PhotoCleaner re-pointed its copy in #50 after Copilot raised it, and #52 carried the canonical wording that #730 landed.

reports/photocleaner/audit.md:13

  • Unqualified #51 will auto-link to ProjectTemplate issue/PR 51, but this sentence is describing a PhotoCleaner promotion PR. Use an explicit owner/repo#N reference so the link resolves to the intended repository.

This issue also appears in the following locations of the same file:

  • line 19
  • line 46
  • line 57
`develop` vs `main`: identical content, since #51 merged `develop` at `ab23034` and nothing has landed since. The audit at `develop` reports zero findings.

A bare #N in a hub file autolinks to the hub's own issue or pull
request, so each PhotoCleaner reference now carries owner/repo.
@ptr727

ptr727 commented Aug 15, 2026

Copy link
Copy Markdown
Owner Author

Answering the four suppressed findings on head 4b1882c (reports/photocleaner/audit.md:13, :19, :46, :57, bare #N PhotoCleaner pull request numbers autolinking into this repository): accepted and fixed in ef1bf5d, every PhotoCleaner reference in the report now reads ptr727/PhotoCleaner#N, including the older ones the earlier snapshot carried.

Copilot AI review requested due to automatic review settings August 15, 2026 18:57

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@ptr727
ptr727 merged commit 1c3e80f into develop Aug 15, 2026
7 checks passed
@ptr727
ptr727 deleted the photocleaner-report-main-2026-08-15 branch August 15, 2026 19:12
ptr727 added a commit that referenced this pull request Aug 15, 2026
…leaner Report (#746)

Promote `develop` to `main`, carrying two changes:

- #744 Host the Merge-Bot in the Hub and Measure Fleet Workflow Reuse:
the hub-hosted reusable-workflow design (`docs/reusable-workflows.md`),
the fleet workflow-reuse measurement (`spec/workflow_reuse.py`,
`reports/workflow-reuse.md`), the first hub task
(`.github/workflows/merge-bot-task.yml`) with the hub's own caller stub,
the manifest contract change for `merge-bot-pull-request.yml`, and the
TODO cluster and adoption sweep.
- #739 Mark PhotoCleaner Operational After the Promotion of the b09078e
Resync: the PhotoCleaner audit report refresh.

The release that follows this promotion is the first tag naming
`merge-bot-task.yml`, which is the pin the downstream caller stubs use,
so the merge-bot adoption sweep and the catalog snippet both wait on it.
A Dependabot pull request against `main` after this merge is the
`--merge` half of the live proof of the callee.

Refs #521 (hub half shipped in #744, sweep half is the adoption). No
issue closes on this promotion.
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