fix(release): Skip a named PR merged elsewhere; mark a published draft

[mpaulosky]

Summary

Two flaws in the release pipeline (#172), found by Copilot's review of its port to atelier-store (mpaulosky/atelier-store#90). This keeps IssueTracker, the reference implementation, in step.

• A manual run naming a PR merged elsewhere: release_queue.py queued the named PR without checking its base branch, so a PR merged into another branch failed the run with UnknownMergeOrder (its merge commit isn't on main). Only a named PR whose base is main is queued now.
• Publishing a draft Release: the release job published a draft for its tag without touching its body. The planner and the ordering guard recognise a Release only by its Source PR: #n marker, so a draft without one stayed "owed" forever and blocked every later PR. The decision now lives in a tested helper, .github/scripts/draft_release.py: it appends the release notes unless the draft already names this PR on a line of its own (LF or CRLF), keeping the draft's own text. A draft that names a different PR is inconsistent, so the step stops with an error naming it rather than publish a Release with two sources. Markers count anywhere in a line, the pattern release_post.source_pr_of() uses, so a - Source PR: #5 list item is caught too.

Testing

• New test_main_skips_a_manually_named_pr_merged_into_another_branch (it failed before the fix).
• New test_draft_release.py (17 tests): no marker, an empty draft, this PR's line (also with CRLF endings), this PR only inside a longer line, another PR (alone, in a list item, or beside this PR's line), #5 against #15 and #50, agreement with source_pr_of(), and the CLI's output and stderr error.
• All script tests: 158 passed, 1 skipped. A failing helper stops the step under bash -e, checked locally.
• yamllint, actionlint and zizmor are clean, and the pre-push gate passed.

Refs #144

🤖 Generated with Claude Code

[Copilot]

Copilot review overview

🟡 Changes recommended

Conflicting source-PR metadata can publish an inconsistent release that remains queued or bypasses ordering checks.

Review effort: Balanced
Findings: 1

Open (1)

• Prevent duplicate conflicting Source PR markers · New

What changed in this PR

Updates release handling to avoid queueing PRs merged outside main and to annotate draft releases before publication.

Changes:

• Filters manually named PRs by base branch.
• Adds regression coverage for non-main PRs.
• Preserves draft notes while appending release metadata.

File	Description
.github/​workflows/​release.yml	Adds metadata before publishing drafts.
.github/​scripts/​release_queue.py	Excludes named PRs not targeting main.
.github/​scripts/​tests/​test_release_queue.py	Tests manual dispatch with another base branch.

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

Test Results Summary

483 tests  ±0   483 ✅ ±0   20s ⏱️ -1s
  9 suites ±0     0 💤 ±0 
  9 files   ±0     0 ❌ ±0 

Results for commit 4413e1e. ± Comparison against base commit 3feeb2f.

♻️ This comment has been updated with latest results.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.40%. Comparing base (3feeb2f) to head (4413e1e).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #184   +/-   ##
=======================================
  Coverage   85.40%   85.40%           
=======================================
  Files          77       77           
  Lines        1596     1596           
  Branches      150      150           
=======================================
  Hits         1363     1363           
  Misses        189      189           
  Partials       44       44           

[Copilot]

Copilot review overview

🟡 Changes recommended

Draft conflict detection does not recognize every source marker accepted by the planner.

Review effort: Balanced
Findings: 1

Open (1)

• Conflict check misses planner-recognized Source PR markers · New

Resolved since last review (1)

• Prevent duplicate conflicting Source PR markers

[Copilot]

Copilot review overview

🟡 Changes recommended

The new draft-release behavior lacks automated regression coverage.

Review effort: Balanced
Findings: 1

Open (1)

• Add regression tests for draft-body marker extraction and preservation · New

Resolved since last review (1)

• Conflict check misses planner-recognized Source PR markers

[Copilot]

Copilot review overview

🟡 Changes recommended

The marker check mishandles CRLF release bodies and unnecessarily duplicates release metadata.

Review effort: Balanced
Findings: 1

Open (1)

• Handle CRLF line endings in whole-line metadata check · New

Resolved since last review (1)

• Add regression tests for draft-body marker extraction and preservation

[Copilot]

Copilot review overview

🟢 Approval recommended

The release fixes are focused, consistent with existing parsing behavior, and comprehensively tested.

Review effort: Balanced
Findings: None

Resolved since last review (1)

• Handle CRLF line endings in whole-line metadata check