fix(publish): default PR body to Closes #N when the brief has an issue - empty PR bodies left issues dangling after merge (auto-close needs the keyword); apply the same default to the release composite path; explicit --body still wins; regression test pins the POST body

Author: 吴红02

lib/domains/publish.mjs

@@ -6,7 +6,11 @@ import { pushAndOpenPr } from '../steps.mjs'
 
 export async function planData(r, o) {
   const n = name(o), b = brief(r, n), q = repo(r)
-  return { name: n, repository: repository(b, r), branch: b.data.branch || q.branch, baseBranch: b.data.baseBranch || 'main', title: o.title || n, body: o.body || '', head: q.head, brief: b.data }
+  // PR 正文缺省携带 Closes #N：brief 有关联 issue 时，PR 合入即自动关闭该 issue
+  // （此前缺省空正文导致 issue 悬空）；显式 --body 仍优先，plan/execute 同源保持哈希一致
+  const issue = b.data.github && b.data.github.issue
+  const body = o.body || (issue ? `Closes #${issue}\n\n完整 brief：shadow-docs/changes/${n}/brief.md` : '')
+  return { name: n, repository: repository(b, r), branch: b.data.branch || q.branch, baseBranch: b.data.baseBranch || 'main', title: o.title || n, body, head: q.head, brief: b.data }
 }
 
 export async function execute(r, o, x, b) {

lib/domains/release.mjs

@@ -6,12 +6,15 @@ import { commitStep, pushAndOpenPr } from '../steps.mjs'
 
 export async function planData(r, o) {
   const n = name(o), b = brief(r, n), q = repo(r), w = b.data.workflow.release || {}
+  // 与 publish 同源：PR 正文缺省携带 Closes #N（brief 有关联 issue 时合并即自动关闭）
+  const issue = b.data.github && b.data.github.issue
+  const body = o.body ?? w.body ?? (issue ? `Closes #${issue}\n\n完整 brief：shadow-docs/changes/${n}/brief.md` : '')
   return {
     name: n,
     files: fileList(o.files ?? w.files),
     message: o.message ?? w.message ?? null,
     title: o.title ?? w.title ?? n,
-    body: o.body ?? w.body ?? '',
+    body,
     repository: repository(b, r),
     branch: b.data.branch || q.branch,
     baseBranch: b.data.baseBranch || 'main',

test/cli.test.mjs

@@ -673,6 +673,28 @@ test('publish pushes normally and creates a PR when none exists', () => {
   } finally { api.close() }
 })
 
+test('publish defaults the PR body to Closes #N when the brief has an issue', () => {
+  const root = fixture()
+  addOrigin(root)
+  execFileSync('git', ['switch', '-c', 'feat/sample'], { cwd: root })
+  writeFileSync(join(root, 'feature.txt'), 'feature\n')
+  execFileSync('git', ['add', '--', 'feature.txt'], { cwd: root })
+  execFileSync('git', ['commit', '-m', 'feature'], { cwd: root })
+  updateBrief(root, data => { data.github.repository = 'owner/repo'; data.branch = 'feat/sample'; data.github.issue = 5 })
+  const api = apiStub([
+    { method: 'GET', path: '/repos/owner/repo/pulls?', body: [] },
+    { method: 'POST', path: '/repos/owner/repo/pulls', body: { number: 9, html_url: 'https://github.test/pulls/9' } },
+  ])
+  try {
+    const args = ['--name', 'sample', '--title', 'Feature']
+    const planned = run(['publish', 'plan', ...args, '--json'], root)
+    const result = run(['publish', 'execute', ...args, '--plan-hash', JSON.parse(planned.stdout).planHash, '--confirm', '--json'], root, { GITHUB_TOKEN: 'token', SHADOW_GITHUB_API_URL: api.url })
+    assert.equal(result.status, 0, result.stderr)
+    const created = api.requests().find(request => request.method === 'POST')
+    assert.match(JSON.parse(created.body).body, /Closes #5/)
+  } finally { api.close() }
+})
+
 test('publish reuses an existing open PR', () => {
   const root = fixture()
   addOrigin(root)