test(ai-review): 補關閉所有 comment、部分 resolve 失敗、百分比邊界與 usageSection 排版測試

app/resolve.test.js

@@ -170,6 +170,39 @@ describe('reconcileConversations', () => {
     assert.deepEqual(result.carriedFindings, []);
   });
 
+  it('resolves every unresolved comment id, not just the first per path/line group', async () => {
+    const closedIds = [];
+    const deps = baseDeps();
+    deps.listComments = async () => [
+      { id: 10, path: 'a.js', position: 5, body: reviewBody('🔴 嚴重', 'Assassin', 'p', 's10') },
+      { id: 11, path: 'a.js', position: 5, body: reviewBody('🔴 嚴重', 'Mage', 'p', 's11') }, // 同 path|line → 同一組
+      { id: 12, path: '', position: 0, body: 'no path' },                                      // 無 path → 不分組但仍要關
+      { id: 13, path: 'b.js', position: 8, body: 'done', resolver: { login: 'dev' } },          // 已 resolve → 不關
+    ];
+    deps.resolveComment = async (id) => { closedIds.push(id); return { ok: true }; };
+    deps.judge = async (items) => items.map(it => ({ idx: it.idx, verdict: 'open' }));
+
+    const result = await reconcileConversations(deps);
+
+    // 同組的 10、11 都關，無 path 的 12 也關；已 resolve 的 13 不關
+    assert.deepEqual(closedIds.sort((a, b) => a - b), [10, 11, 12]);
+    assert.equal(result.closedCount, 3);
+  });
+
+  it('counts only successful closes when some resolve calls fail', async () => {
+    const deps = baseDeps();
+    // a.js(id1) 關閉成功、b.js(id2) 關閉失敗（c.js 已 resolved 略過）
+    deps.resolveComment = async (id) => { if (id === 2) throw new Error('403'); return { ok: true }; };
+    deps.judge = async (items) => items.map(it => ({ idx: it.idx, verdict: 'resolved' }));
+
+    const result = await reconcileConversations(deps);
+
+    assert.equal(result.closedCount, 1); // 僅 id1 成功關閉
+    // findings 分流不受 resolve 成敗影響：兩個都判 resolved
+    assert.equal(result.resolvedCount, 2);
+    assert.deepEqual(result.resolvedFindings.map(f => f.location).sort(), ['a.js:10', 'b.js:20']);
+  });
+
   it('carries open-verdict conversations into findings while still closing them', async () => {
     const closedIds = [];
     const deps = baseDeps();