feat(pr): move shared description restoration into Go

Problem

Part of #588 and the restoration portion of #325. It belongs to agentrepo pr (alongside #323), not agentrepo checks: restoring a local description does not watch or diagnose checks.

On origin/main at 591fd70b46a5cca7174521d72eace58039a95444, fix-pr and merge-pr carry byte-identical populate-description.sh scripts (packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:1, packages/agents/source/skills/merge-pr/shared/scripts/populate-description.sh:1). Both rebuild PR_AGENTDESC.md from an open PR. The nonempty-draft guard is at populate-description.sh:54 (before target discovery and the metadata read), the single metadata read at :69, the direct write that can truncate a draft on decode failure at :73 and :104, and the false suffix-removal guidance at :107. No Go restoration exists: internal/agentrepo/command.go registers no pr command.

Coverage on main: no test exercises either script. tests/skills/fix-pr/scripts/test_diagnose.py no longer exists (removed when diagnosis moved to Go, #832/#959/#960); tests/skills/fix-pr/scripts/diagnose.bats covers only the diagnose wrapper's dispatch. docs/script-inventory.md:195 still names the removed file and says fix-pr has no Bats suite; this PR corrects that line.

Overlapping work, with the disposition decided here:

Open PRs fix(fix-pr): make unattended repair paths deterministic #349 and fix(fix-pr): make unattended repair paths deterministic #696 (near-duplicates; same branch family, same 18 files) also edit populate-description.sh and add tests/skills/fix-pr/populate-description.bats. They drop the guard at :54, read metadata for a nonempty draft, and report current/hold/refreshed with status 75 for a differing draft, to satisfy fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296. fix(fix-pr): make unattended repair paths deterministic #349 has CHANGES_REQUESTED; fix(fix-pr): make unattended repair paths deterministic #696 has no review or checks.
Disposition: this issue ports the contract on main and does not adopt that behavior. fix(fix-pr): make unattended repair paths deterministic #349 and fix(fix-pr): make unattended repair paths deterministic #696 are neither blockers nor incorporated; their populate-description changes are deferred to a Go follow-up on the new command (see Scope), and their other changes (fix-pr names the operator scripts for republishing instead of the agent-pr route #294, fix-pr asks through AskUserQuestion and has no default for a headless session #295) are independent of this issue. If one lands first, this PR rebases and its AC2/AC6 oracle becomes the landed contract; that is a scope amendment on this issue, not a native blocker.
fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296 stays open. This PR does not close it. The follow-up closes it.

Other restoration owners, with the disposition decided here (verified against the bodies of #873, #874 and #584 and origin/main at 591fd70b46a5cca7174521d72eace58039a95444):

feat(pr): move shared description restoration into Go #795 owns the single restoration command, agentrepo pr populate-description, and the wrapper cutover of both populate-description.sh copies (fix-pr and merge-pr). No agentrepo merge description command is created.
A pull request description draft is rebuilt from the published pull request in Go #873 (child 13 of Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324) specifies agentrepo merge description for the merge-pr copy only, blocked by Ship the Go binaries as release assets installed through mise's github backend #333, keeping the closing message about suffix removal that the script does not honor (its AC2 and Non-goal). That contradicts AC9 here and duplicates this command, so A pull request description draft is rebuilt from the published pull request in Go #873 is superseded by this issue.
The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874 (child 14 of Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324) names populate-description.sh among four scripts it routes to agentrepo merge (its Problem, AC1, AC2, AC8, Scope, Size, and A pull request description draft is rebuilt from the published pull request in Go #873 in its Dependencies). Its restoration portion is superseded by this issue; The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874 keeps lint-squash-msg.sh, squash-message.sh and review.sh (and the three agent-merge-pr counterparts). The merge-pr populate-description.sh (62 code lines, per Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324's table) leaves The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874's 351-line count, leaving 289.
Consolidate the duplicated populate-description.sh into one shared source #584 (relocate the byte-identical populate-description.sh into packages/agents/source/lib/) is independent and not a blocker. After this change the two copies are a dispatch shim and a frozen populate-description-legacy.sh, both still byte-identical across the skills, so Consolidate the duplicated populate-description.sh into one shared source #584 stays valid but shrinks to relocating those two files, and the legacy copy is deleted after the N-2 window. It is not incorporated here. Whichever lands second rebases onto the other; if Consolidate the duplicated populate-description.sh into one shared source #584 lands first this PR edits the shared source instead of the two skill sources.
Required reconciliation before build starts (issue-body edits, no code): close A pull request description draft is rebuilt from the published pull request in Go #873 as superseded by feat(pr): move shared description restoration into Go #795; narrow The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874 to drop the restoration script from its Problem, ACs, Scope, Size and Dependencies; strike child 13 in Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324's Children list and record that child 14 no longer covers populate-description.sh. No native blocked-by results: nothing here waits on those issues.

Acceptance criteria

AC1: When restoration is requested through either the fix-pr or merge-pr entry point, the result is the same as agentrepo pr populate-description for the same PR and local draft state.
AC2: Given a nonempty local draft and no --force, with GitHub access present, the draft is unchanged, the existing left-alone guidance is printed, the status is 0, and no PR metadata is read.
AC3: Given no local draft, an empty draft, or --force, a successful restoration writes the reconstructed draft at the repository root.
AC4: An explicit number or the current branch's open PR selects the target; missing repository context, missing GitHub access or no discoverable PR produces the existing refusal and nonzero status.
AC5: Non-functional: the reconstructed properties, title and body all come from one PR metadata read.
AC6: Given readable PR metadata, the draft keeps the existing field order and values, title and body transformations and single trailing newline.
AC7: Given any PR review-request state, the reconstructed frontmatter has an empty reviewers list.
AC8: Given failed retrieval, decoding or draft writing, restoration exits 1 and leaves any existing draft byte-identical, with no partial new draft (a deliberate difference from the frozen shell).
AC9: Successful restoration reports the draft and source PR with status 0, and its guidance does not claim a title suffix was removed (a deliberate difference from the frozen shell).
AC10: Non-functional: restoration changes only the requested local draft: no remote write, review request or Git-state change, whatever GH_REPO, GH_HOST or CDPATH hold.
AC11: Non-functional: restoration gives the same outcomes on Linux, macOS and Windows.
AC12: Both entry points only dispatch: they forward arguments, output and status unchanged to the Go command when it exists, run the unchanged frozen script throughout the N-2 window when it doesn't, never fall back after a supported run fails, and add no other shell logic.

Scope and non-goals

Moves into agentrepo pr populate-description: the draft restoration both skills duplicate, preserving the contract at the commit above (flag order [--force] [number], repository-root execution, access checks before the draft guard, field order base, draft, labels, reviewers, assignees, milestone, a blank line before the level-one title and body, CRLF to LF, outer whitespace stripped, existing title text kept). Go-path differences from the frozen shell are AC8 and AC9 only; they are documented in docs/agent-skills-migration.md.

Non-goal: the #296 classification (current/hold/refreshed, status 75), the removal of the AC2 guard, and any other behavior change. File a follow-up issue for it (Problem and numbered ACs), built on this command, list it under PR Related, and link #349/#696 and #296 from it.

Non-goals: an agentrepo merge description command (#873) and the restoration portion of #874; relocating the script source (#584); watching or diagnosing checks; description validation, review, publication or merging (#323, #324); new frontmatter properties, reviewer replay or title-suffix changes; guard or authority logic; deleting the N-2 fallback.

Approach

Capture old-script fixtures for both skills first, then add the Go command with table-driven tests against a fake gh and disposable repositories. Mirror the existing diagnose pattern: each skill's populate-description.sh becomes a dispatch shim (probe agentrepo pr populate-description --help, exec on support, otherwise run populate-description-legacy.sh, the unchanged old script). Edit the skill sources, then mise run generate-skills, mise run skill-tokens and apm install. Restoration stays explicit; it is not added to any preflight.

Dependencies

No native blocker. #333 and #544 deliver the binary to consumers; the probe-and-fallback shim lets this land first. #324 is affected only through the child reconciliation listed in Problem: #873 is superseded and #874 narrowed, so neither blocks nor is blocked by this issue. #323 is unaffected. #349, #696 and #584: see Problem.

Size

M, one PR, estimated 450 to 650 code lines counting tests and generated copies (Go command about 130, Go tests about 280, shim tests about 100, shims and generated copies about 120, from the 62 code lines of each current script and the roughly 10-line diagnose.sh shim pattern; renamed legacy scripts count as renames). Measure with git diff --stat before publishing. If it exceeds 800 code lines, split into: (1) feat(pr): add agentrepo pr populate-description (Go command and tests); (2) refactor(skills): route populate-description through the Go command (shims, generated output, docs, shim tests).

Verify plan

Red first: command regressions for the missing subcommand, draft preservation on failure and truthful guidance fail before the port and pass after.
AC1: the same fixtures through the Go command and both entry points, and against both pinned shell scripts; compare draft bytes, stdout, stderr and status.
AC2: nonempty draft without force, including a nonexistent explicit number; assert unchanged bytes, guidance, status 0 and zero metadata reads; missing access still refuses first.
AC3: missing, empty and forced-nonempty drafts, run from a nested directory; only the root draft is written.
AC4: explicit and inferred PRs; absent repository, tool, authentication or open PR; assert refusals, statuses and an untouched draft.
AC5: count exactly one metadata read; vary a hypothetical second response and assert every field comes from the first.
AC6, AC7: byte-compare against baseline for draft true/false, empty and multiple labels and assignees, present and absent milestone, empty, whitespace and CRLF bodies, title whitespace and a literal (#N) suffix, and PRs with unfinished and completed review requests (reviewers: []).
AC8: failed retrieval, malformed metadata and write failure with absent, empty, nonempty and forced drafts; assert status 1, original bytes retained and no partial file.
AC9: assert the success message and status 0, with a suffixed title kept intact and no removal claim.
AC10: fake gh rejects any mutating call; compare repository content, index and refs before and after; run with GH_REPO, GH_HOST and CDPATH set.
AC11: Go tests on the Linux, macOS and Windows jobs; mise run check, mise run cover-go-check, mise run cover-py-check; mise run check-generated-skills after generate, token and apm steps.
AC12 (including no other shell logic; review the shell diff for dispatch and renames only): for both entry points test a supporting binary, no binary, a binary lacking the subcommand, and a supported run that fails; assert forwarding, fallback only on missing capability and no second run after an error.
Delivery (D1 to D10): per the definition of done in the refinement comment; cite each AC id and its evidence in the PR.

Risks

A supported failure retried through the fallback would write twice (AC12). A forced run that fails could destroy a draft (AC8). A change reaching only one caller or only some generated copies would leave them diverged (AC1, mise run check-generated-skills). If #349 or #696 lands first the oracle shifts; rebase and amend the ACs. Undo: revert this PR and run mise run generate-skills; the frozen scripts return to their old names and behavior.

Operator authority

None

Refinement: NOT READY — resolve the competing existing-draft contract

Checked

Read feat(pr): move shared description restoration into Go #795, parent Move watch-pr and fix-pr scripts into agentrepo checks #325, Epic Replace complex shell with tested Go, starting with gates and merging #588, siblings feat(checks): move pull request watching into Go #793/feat(checks): move pull request failure diagnosis into Go #794, transaction epics Move the pr, write-pr-description and agent-pr scripts into agentrepo pr #323/Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324, distribution dependencies Ship the Go binaries as release assets installed through mise's github backend #333/feat(distribution): migrate consumer tool pins to release assets #544, and protection work build(agents): protect replacement Go enforcement paths #753. Accepted the reported mechanical PASS; did not rerun the gate or any tests.
Verified current origin/main against remote main at 591fd70b46a5cca7174521d72eace58039a95444. Both restoration scripts remain identical to the issue's pinned baseline. The overwrite guard, single metadata read, direct-write failure risk and misleading title guidance are confirmed at packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:54, packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:69, packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:104 and packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:107.
AC1–AC12 each have an explicit Verify-plan entry. The criteria cover observable behavior, refusal statuses, serialization, preservation on failure, one metadata snapshot, local-only writes, environment isolation, platform parity and compatibility dispatch. Command names and wrapper boundaries describe the public contract. The documented Go-path exceptions distinguish intentional changes from parity cases.
The shell-freeze approach is appropriate: tested Go behavior, thin capability/forwarding wrappers and an unchanged legacy fallback. No new restoration algorithm belongs in shell. N-2 rollout and no fallback after a supported-command failure are covered.
One shared restoration operation with both callers is plausibly one S/M PR below 800 code lines; confirm the actual count including tests before publication. Risks and revert are specified. Operator authority: None fits this local reconstruction scope and grants no publication or merge authority. Native blocked-by is empty; the listed broader ports and distribution rollout are described as context or later retirement prerequisites, rather than landing blockers.
No merged Go restoration implementation was found, and internal/agentrepo/command.go:85 does not register a PR command. Sibling diagnosis has shipped through feat(checks): diagnose failures in Go #832/test(checks): verify diagnosis report parity #959/feat(checks): route diagnosis through the Go command #960; that does not deliver restoration. Open PRs fix(fix-pr): make unattended repair paths deterministic #349 and fix(fix-pr): make unattended repair paths deterministic #696 do overlap this operation.

Gaps

Resolve overlapping work before implementation. Both open PR fix(fix-pr): make unattended repair paths deterministic #349 and open PR fix(fix-pr): make unattended repair paths deterministic #696 change the canonical fix-pr restoration script and add restoration tests for fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296. Their diffs remove the early guard at packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:54, read metadata even for a nonempty draft, and report current/hold/refreshed, including status 75 for a differing draft. This directly conflicts with AC2's metadata-free status-0 guard and with the assumption that both callers share an unchanged contract. Neither PR is acknowledged in feat(pr): move shared description restoration into Go #795. Record which work is superseded, deferred or incorporated, and reconcile fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296 with AC2 and the parity oracle. If an earlier change must land first, add its issue as a native blocked-by dependency; if it is superseded or deferred, state that disposition without inventing a blocker. Keep any adopted behavioral changes in Go under Replace complex shell with tested Go, starting with gates and merging #588's freeze, and revisit size if this expands scope. The Go migration itself is not already delivered, but these competing contracts cannot both remain the intended result.

The historical diagnosis-test reference is also stale on current main: #960 removed tests/skills/fix-pr/scripts/test_diagnose.py. Refresh that evidence when revising the issue; this is not an additional readiness blocker.

Split

No split required for the stated restoration-only scope. Resolve the overlap above before deciding whether adopting #296's classification behavior warrants a separate Go follow-up.

Definition of done

Checked boxes mean the issue's ACs and Verify plan cover the requirement, not that implementation or delivery has occurred.

Round: 1 of 3

Refinement: NOT READY — reconcile the second restoration owner and wrapper cutover

Checked

Accepted the reported mechanical PASS; did not rerun bin/request check, Go tests or any full suite. Read feat(pr): move shared description restoration into Go #795 and its newest earlier Refinement comment, parent Move watch-pr and fix-pr scripts into agentrepo checks #325, Epic Replace complex shell with tested Go, starting with gates and merging #588, siblings feat(checks): move pull request watching into Go #793/feat(checks): move pull request failure diagnosis into Go #794, transaction epics Move the pr, write-pr-description and agent-pr scripts into agentrepo pr #323/Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324, distribution work Ship the Go binaries as release assets installed through mise's github backend #333/feat(distribution): migrate consumer tool pins to release assets #544, fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296, overlapping issues Consolidate the duplicated populate-description.sh into one shared source #584/A pull request description draft is rebuilt from the published pull request in Go #873/The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874 and PRs fix(fix-pr): make unattended repair paths deterministic #349/fix(fix-pr): make unattended repair paths deterministic #696.
Verified origin/main against remote main at 591fd70b46a5cca7174521d72eace58039a95444. Both canonical restoration scripts are byte-identical. Access checks precede the metadata-free nonempty-draft guard at packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:51 and packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:54. The single metadata read, direct-write failure exposure and false suffix guidance remain at packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:69, packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:104 and packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:107.
Round 1's gap is resolved: fix(fix-pr): make unattended repair paths deterministic #349/fix(fix-pr): make unattended repair paths deterministic #696 are explicitly deferred for restoration, fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296 remains open for a Go follow-up with Problem and numbered ACs, AC2 retains main's guard, and landing either competing PR first requires a scope amendment. Both PR diffs remove that guard and introduce current/hold/refreshed, including status 75. Neither delivers Go restoration. The disposition is now explicit and consistent with the verified baseline.
Round 1's stale test reference is corrected: main contains tests/skills/fix-pr/scripts/diagnose.bats, not the removed Python test. The obsolete inventory claim remains at docs/script-inventory.md:195, and this issue includes its correction. The previous Split advice is addressed by retaining restoration-only scope and deferring fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296 to a separate Go follow-up.
No Go restoration is registered at internal/agentrepo/command.go:85; related-PR searches found no merged or open PR delivering it. Diagnosis PRs feat(checks): diagnose failures in Go #832/test(checks): verify diagnosis report parity #959/feat(checks): route diagnosis through the Go command #960 are merged and deliver different behavior. The compatibility dispatch pattern is visible at packages/agents/source/skills/fix-pr/shared/scripts/diagnose.sh:5.
AC1–AC12 each have explicit Verify entries covering observable results, refusals, serialization, one metadata snapshot, empty reviewer replay, preservation on failure, truthful guidance, local-only effects, environment isolation, platform parity and N-2 dispatch. The deliberate Go-path differences are identified. The approach respects Replace complex shell with tested Go, starting with gates and merging #588: tested Go, thin dispatch and unchanged frozen fallback, with no fallback after a supported run fails.
Restoration-only scope plausibly fits one M PR below 800 code lines; actual measurement and a contingency split are specified. Risks and revert are concrete. Operator authority: None fits local reconstruction. Native blocked-by is empty, consistent with capability fallback and no prerequisite distribution rollout, subject to resolving ownership below.

Gaps

Resolve the other active restoration implementation and cutover. The claim that Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324 is unaffected conflicts with its open children. A pull request description draft is rebuilt from the published pull request in Go #873 explicitly replaces packages/agents/source/skills/merge-pr/shared/scripts/populate-description.sh:1 with agentrepo merge description, preserves the misleading closing message, and waits on Ship the Go binaries as release assets installed through mise's github backend #333. The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874 includes this same script in its wrapper cutover, routes it to agentrepo merge, and depends on A pull request description draft is rebuilt from the published pull request in Go #873. feat(pr): move shared description restoration into Go #795 owns both callers, routes them to agentrepo pr populate-description, corrects the message under AC9 and lands with fallback independently of Ship the Go binaries as release assets installed through mise's github backend #333. These deliverables overlap and specify incompatible command and guidance contracts. Amend feat(pr): move shared description restoration into Go #795's Problem, Scope and Dependencies with an explicit disposition for A pull request description draft is rebuilt from the published pull request in Go #873 and the restoration portion of The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874; reconcile those linked issue bodies and Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324's child ownership so there is one command and one wrapper owner. Also state whether Consolidate the duplicated populate-description.sh into one shared source #584's source consolidation is superseded or remains independently useful: it relocates the same two maintained sources this issue converts to shims. If any work must land first, record its issue as native blocked-by; otherwise defer or supersede it explicitly without manufacturing a blocker. Keep the resolution in Go under Replace complex shell with tested Go, starting with gates and merging #588 and reassess size if scope grows.

Split

No implementation split is required for the stated restoration-only scope. The size contingency is reasonable. Settle #873/#874/#584 ownership before coding; retain the separately specified Go follow-up for #296.

Definition of done

Checked boxes mean the ACs and Verify plan cover the requirement, including the body's explicit adoption of D1–D10 from the previous refinement comment. They do not mean implementation or delivery has occurred.

Round: 2 of 3

Refinement: NOT READY — finish the recorded restoration ownership reconciliation

Checked

Accepted the reported mechanical PASS without rerunning it. Read feat(pr): move shared description restoration into Go #795 and both earlier refinement comments, parent Move watch-pr and fix-pr scripts into agentrepo checks #325, Epic Replace complex shell with tested Go, starting with gates and merging #588, siblings feat(checks): move pull request watching into Go #793/feat(checks): move pull request failure diagnosis into Go #794, transaction epics Move the pr, write-pr-description and agent-pr scripts into agentrepo pr #323/Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324, fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296, Ship the Go binaries as release assets installed through mise's github backend #333/feat(distribution): migrate consumer tool pins to release assets #544, Consolidate the duplicated populate-description.sh into one shared source #584/A pull request description draft is rebuilt from the published pull request in Go #873/The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874, and open PRs fix(fix-pr): make unattended repair paths deterministic #349/fix(fix-pr): make unattended repair paths deterministic #696, including their restoration patches. No tests or full suites ran.
Verified origin/main against remote main at 591fd70. Both maintained restoration scripts are byte-identical. Access checks precede the existing-draft guard at packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:51 and packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:54. The single metadata read, truncation exposure and false suffix-removal guidance are confirmed at packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:69, packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:104 and packages/agents/source/skills/fix-pr/shared/scripts/populate-description.sh:107.
No PR restoration command is registered at internal/agentrepo/command.go:85, and related-PR searches found no merged or open PR delivering this Go restoration. fix(fix-pr): make unattended repair paths deterministic #349/fix(fix-pr): make unattended repair paths deterministic #696 still overlap the shell operation, but the issue explicitly defers their current/hold/refreshed contract to a Go follow-up and keeps fix-pr lacks a decision rule for an existing PR_AGENTDESC.md #296 open. That resolves round 1's contract decision.
AC1–AC12 each have a named Verify entry. They specify testable observable behavior, serialization, one metadata snapshot, draft preservation, truthful guidance, local-only effects, environment isolation, platform parity and compatibility dispatch. The command and entry-point names define the public boundary; the implementation approach is separate. AC8/AC9 identify the intentional differences from frozen-shell parity.
The approach respects Replace complex shell with tested Go, starting with gates and merging #588's freeze: tested Go, thin dispatch only, unchanged legacy bodies, and no fallback after a supported run fails. The existing precedent is packages/agents/source/skills/fix-pr/shared/scripts/diagnose.sh:5. The stale inventory evidence at docs/script-inventory.md:195 is acknowledged and included for correction.
The stated M scope plausibly fits one PR below 800 code lines, with tests and generated copies counted, an actual-size check and a contingency split. Risks and undo are concrete. Operator authority: None fits this local reconstruction scope. Native blocked-by is empty, confirmed through issueDependenciesSummary and the dependency endpoint; Ship the Go binaries as release assets installed through mise's github backend #333/feat(distribution): migrate consumer tool pins to release assets #544 are correctly unnecessary for landing with the frozen fallback.
Round 2's decision is now explicit in feat(pr): move shared description restoration into Go #795: it owns both callers, supersedes A pull request description draft is rebuilt from the published pull request in Go #873, removes restoration from The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874, and leaves Consolidate the duplicated populate-description.sh into one shared source #584 independently useful. Parent Move watch-pr and fix-pr scripts into agentrepo checks #325 also names one shared agentrepo pr restoration command. However, the required linked-record reconciliation has not happened.

Gaps

Complete the reconciliation already required before build starts. Live A pull request description draft is rebuilt from the published pull request in Go #873 remains OPEN and still specifies agentrepo merge description, preserves the misleading closing message and waits on Ship the Go binaries as release assets installed through mise's github backend #333. Live The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874 still includes populate-description.sh in its Problem, AC1/AC2, Scope, 351-line Size and Dependencies on A pull request description draft is rebuilt from the published pull request in Go #873. Live Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324 still lists A pull request description draft is rebuilt from the published pull request in Go #873 as an active child and requires all 27 children to merge. These continue to assign the same merge-pr entry point a second command and wrapper owner, conflicting with feat(pr): move shared description restoration into Go #795 AC1/AC9/AC12. Recording the intended disposition only in feat(pr): move shared description restoration into Go #795 does not satisfy round 2's request to reconcile the linked bodies. Close A pull request description draft is rebuilt from the published pull request in Go #873 as superseded, narrow The lint, populate-description, squash-message and review scripts become wrappers over agentrepo merge #874 consistently (including size and native dependency relationships), and update Move the merge-pr, merge policy and agent-merge-pr scripts into agentrepo merge #324's Children and Done when so a superseded child is no longer required to merge. feat(pr): move shared description restoration into Go #795 already lists these as required pre-build issue edits; perform them before calling the item ready. No new code or invented blocker is needed. Consolidate the duplicated populate-description.sh into one shared source #584's independent disposition is sufficiently explained.

Split

No additional implementation split is needed for the current restoration-only scope; retain the documented over-800-line contingency and separate Go follow-up for #296. This third NOT READY exhausts the three refinement rounds: the owner must rethink the remaining ownership coordination and reconcile the records before building, rather than start a fourth unchanged refinement round or escalate a non-reserved choice to the operator.

Definition of done

Ticks mean the ACs and Verify plan cover the requirement, including the explicit adoption of D1–D10 from the refinement comment. They do not claim completed implementation, passing delivery checks or independent approval of a future head.

Round: 3 of 3

Navigation Menu

Repository navigation

feat(pr): move shared description restoration into Go #795

Problem

Acceptance criteria

Scope and non-goals

Approach

Dependencies

Size

Verify plan

Risks

Operator authority

Checked

Gaps

Split

Definition of done

Checked

Gaps

Split

Definition of done

Checked

Gaps

Split

Definition of done

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

feat(pr): move shared description restoration into Go #795

Description

Problem

Acceptance criteria

Scope and non-goals

Approach

Dependencies

Size

Verify plan

Risks

Operator authority

Activity

tbhb-dev-reviews commented on Oct 31, 2026

Checked

Gaps

Split

Definition of done

tbhb-dev-reviews commented on Oct 31, 2026

Checked

Gaps

Split

Definition of done

tbhb-dev-reviews commented on Oct 31, 2026

Checked

Gaps

Split

Definition of done

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions