prioritize PR overview by project board and fix stale changelog placement#2210
Open
maybeec wants to merge 1 commit into
Open
prioritize PR overview by project board and fix stale changelog placement#2210maybeec wants to merge 1 commit into
maybeec wants to merge 1 commit into
Conversation
…ment report-pr-overview: read the IDEasy board (project devonfw#5) so PRs are grouped by pipeline column and ordered by board ownership, priority and size. sync-prs: audit the changelog placement of every PR, not only conflicting ones, and re-home entries stranded under an already-released section. Resolve the current release from the live remote instead of the local checkout, which reports bogus results on a stale clone. Mirror both into .github/prompts for GitHub Copilot.
Collaborator
Coverage Report for CI Build 30200137633Coverage increased (+0.03%) to 72.594%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats💛 - Coveralls |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implemented changes:
Two agent instruction workflows were producing incomplete or wrong results. Both are fixed here, and both are mirrored into
.github/prompts/so GitHub Copilot and Claude Code stay on the same instructions.report-pr-overview— prioritize by the project boardThe report was built purely from the GitHub PR view, which ignores the workflow actually documented in
project-board.adoc. It now reads the IDEasy board (project #5) and uses it to prioritize:Team Review(peer review) from👀 In review(final review by the PO), and PRs parked in non-review columns are surfaced as misplaced.reviewer = assignees − author) determines review ownership, so a PR with no reviewer assigned is reported as a first-class finding instead of looking healthy.PriorityandSizedrive ordering (high priority + small size first), and a board-hygiene section reports missing metadata, wrong columns, PRs absent from the board, and reviewer load per person.COMMENTEDreview thread left unresolved while the author pushed on; the previous spec only recognisedCHANGES_REQUESTEDand silently missed these.sync-prs— fix changelog entries stranded under released sectionsPlacement was only ever checked while resolving a conflict. But a PR opened while
2026.07.001was current keeps its bullet under== 2026.07.001; once that release ships the entry is stranded, and because the bullet sits in a different region of the file thanmain's additions this usually produces no merge conflict at all — so the sweep never looked at it. Real examples among currently open PRs: #1878 has its bullet under== 2026.05.001(three releases back) while its milestone says2026.08.001, and #2053 sits under the already-released== 2026.07.001.==heading — it does not assume the bullet is near the top of the file..mvn/maven.config(-Drevision=<version>-SNAPSHOT, the authoritative next open release per the DoD), cross-checked against the changelog's first heading and the earliest-due open milestone. Disagreement stops the run rather than guessing.== <version>heading is explicitly out of scope — that is the release manager's commit.gh pr editcommand rather than written silently, since PR metadata is outside this workflow's remit.Both — resolve the current release from the live remote
Both workflows resolved the current release by reading
CHANGELOG.adocfrom the local checkout. On a stale clone that yields a wrong answer: it reported "release2026.07.002shipped with no changelog section" when livemainwas perfectly fine. Compounding it,git fetchcan fail silently whileghstill works, so a cleangit statusproves nothing about freshness. Both now readCHANGELOG.adocand.mvn/maven.configfrom the live remote viagh api, and only report a mismatch when the live files actually disagree.Also recorded along the way: cheaper data-gathering (board pagination, one-call reviewer counts) and environment gotchas that cost real calls — quoting
'@me'in PowerShell, preferring bash for--jqwith embedded quotes, and stripping non-ASCII before printing board field names.Testing instructions
These are agent instruction files only — no product code, no build impact. Verify by running the commands:
/report-pr-overview(Claude Code) or thereport-pr-overviewprompt (Copilot). Confirm the output groups PRs by board column, flags PRs with no reviewer assigned, and reports board hygiene gaps. Cross-check a few rows against the board.-Drevision=in.mvn/maven.configonmainand the first== <version>heading inCHANGELOG.adoconmain— and that it stays correct even with a deliberately outdated local clone./sync-prs. Confirm it reports the resolved target section plus the sources it checked, and that it flags #1695: Clone settings to temporary directory, analyse, and then move #1878 and #1933 Added GUI console pane #2053 as changelog entries needing re-homing. Verify it never proposes writing into a released section or creating a new==heading.Checklist for this PR
mvn clean testlocally all tests pass and build is successful — not applicable, documentation/instruction files only, no code or build files touched#«issue-id»: «brief summary»(e.g.#921: fixed setup.bat). If no issue ID exists, title only. — no issue exists for this; title onlyIn Progressand assigned to you or there is no issue (might happen for very small PRs) — there is no issueinternal— no changelog entry: the changelog reflects only public changes relevant for end-users (DoD.adoc), and these are internal agent instructions