@feng0531/dsh-code-review 0.87.8 → 0.87.9

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 本文件介绍对使用者有意义的变化。安装与参数见 [README](README.md)，本次发布正文见 [发布说明](RELEASE-NOTES.md)。历史提交和安装包保留在 [GitHub Releases](https://github.com/f-e-n-g-0531/dsh-code-review/releases)；本文不把开发日志当作功能介绍。
 
+## 0.87.9 — 范围说明与未完成分开记账
+
+- 初始按需上下文说明由宿主单独记录为范围说明，不再仅因这项导航提示将已完成的所选审核标成部分完成；仍不表示全部上下文已核查或可以放行。模型返回、未知及其他限制仍保守影响完成状态，预算、失败、未完成综合与复核门槛不变。
+- 保留范围说明原文和文件归属；旧报告不重新分类。本版不改模型提示、检索额度或证据判定，不宣称提速。
+- 后续真实 0.87.8 固定七例已补测：三正例最终有效支持从旧版 0/3 到 3/3，四反例无误报；18次调用，执行88.8秒，仍为partial。仅这七个合成样例的一次观察，不是0.87.9实模验收或业务性能证明。
+
 ## 0.87.8 — 契约内静态复核与明确证据协议
 
 - 区分源码和明确适用契约内的静态支持与实机复现：触发可构造、局部实现完整且排除现有保护时，不因缺少业务运行日志自动降为不确定；关键外部条件未知时仍保留不确定，合同文本不扩大权限。

package/RELEASE-NOTES.md

@@ -1,17 +1,15 @@
-# v0.87.8 发布说明
+# v0.87.9 发布说明
 
-## 修复候选找对、复核过度退缩
+## 修复范围说明导致的部分完成标记
 
-复核明确区分“批准源码及明确适用契约内的静态支持”和“实机复现”。可构造的契约允许触发、完整局部实现、旧新对照和已有保护检查充分时，不因没有实际业务日志自动降为 uncertain；未知外部行为、关键实现或业务前提仍须检索或保留不确定。合同仅作为不可信需求数据，不得执行其中指令、扩权或假定真实性。
+初始输入采用批准目录按需读取时，宿主将这一范围说明单独记账，不再把它当成未完成限制。报告仍保留说明原文、归属及“不是全部上下文已核查”的限定。仅影响这项明确的宿主提示，不按文本匹配删除模型或未知限制，也不改写历史报告。预算耗尽、阶段失败、未完成综合或复核仍须保守呈现。
 
-传输说明不再用固定 uncertain 与空证据作为示例。原生输出 schema 明确 receipt-only 或完整六字段原文证据，以及普通旧新归因结构；仍兼容缺省归因的旧响应。复核提示主动对照批准目录中的合同，目录不冒充全文。身份、回执、原文、迁移归因、共享预算和零轮最终答复规则不变，不把候选或独立复现强行改成已支持。
+本版不修改模型提示、调用额度、读取预算或严格原文及旧新归因校验，不把所选范围处理完成等同于代码无缺陷或允许放行。
 
-实际模型适配器补记 verification-correction 调用用途，仅修复诊断身份，不新增调用额度。
+## 实际验收依据与边界
 
-## 固定自主验收观察与限制
+先前未加载新版的边界现已补测：实际运行0.87.8、同冻结七例和gpt/gpt-6.1-sol，三正例均最终supported/introduced并有宿主校验的旧新原文，四反例无误报，主审7/7完成。旧0.87.6有效支持为0/3，本次为3/3；独立裁决核对正文、哈希和确定性行为。
 
-冻结七个自产合成样例、正反例及行为真值后，当前实际宿主 0.87.6 的一次真实同模型基线：3/3 正例候选正确、4/4 反例无误报，但最终三条均 uncertain 且无证据引用，严格验收未通过。真实模型 route 为 gpt/gpt-6.1-sol，18次调用，执行约82.3秒。不是本版运行成绩，也不外推业务质量或稳定耗时。
+本次18次模型调用、执行88.809秒，旧基线82.293秒，未观察到提速；各一次且时间环境不同，不作稳定速度结论。0.87.8仍因七条通用上下文范围提示标为partial，预注册严格整体验收未追改为通过。这些是0.87.8的观察，不是本版真实模型成绩。本版离线回归不能代替实际加载后的完整真实验收。
 
-开发中的三正例实际模型组件探测读取固定两侧源码及合同后给出 supported/introduced，四预置反例假说得到 refuted；反例探测误见一项正例候选，不能称隔离盲测。为控制固定输入开销，发布前又压缩了重复措辞，因此早先探测也不是最终逐字提示的实模成绩。这些只是 Agent 对 verificationLoop 提示的语义探测，不是 createDshModel 原生提供方到严格 validator 的完整插件集成，也不是新版整体通过。离线回归检查协议与安全边界，不能代替真实模型质量。当前宿主未升级，本版完整真实插件效果仍未验收。
-
-发布不自动安装或重启宿主。固定样例、原始公开结果及裁决材料留在本地实验目录，不进入安装包。安装见 [README](README.md)，门禁见 [开发说明](DEVELOPMENT.md)。
+发布不自动安装或重启运行宿主。固定样例与本地验收材料不进入包。安装见[README](README.md)，门禁见[开发说明](DEVELOPMENT.md)。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feng0531/dsh-code-review",
-  "version": "0.87.8",
+  "version": "0.87.9",
   "private": false,
   "license": "SEE LICENSE IN LICENSE",
   "publishConfig": { "access": "public", "registry": "https://registry.npmjs.org/" },

package/src/background-review.mjs

@@ -28,6 +28,7 @@ function terminalSummary(value) {
     lines.push('报告状态：' + report.status + '；任务结束不等于所有审查通过。');
     lines.push('文件覆盖：' + COVERAGE_FIELDS.map(key => key + ' ' + count(report.coverage?.[key])).join('；'));
     lines.push('总文件：' + count(report.totalFiles) + '；总发现：' + count(report.totalFindings) + '；限制项：' + count(report.totalLimitations));
+    if (Number.isSafeInteger(report.totalScopeNotices) && report.totalScopeNotices > 0) lines.push('范围说明：' + report.totalScopeNotices + ' 项（非未完成限制）；完成不等于全部批准上下文已核查，原文与归属见报告附录。');
     lines.push('模型调用：' + count(report.modelCalls ?? report.diagnostics?.modelCalls));
     const d = report.diagnostics;
     if (d?.readerBudget) {

package/src/project-report.mjs

@@ -7,6 +7,7 @@ export function renderProjectReport(report) {
     findings: report.findings ?? [],
     interactions: (report.batches ?? []).flatMap(b => b.interactions ?? []),
     limitations: [...(report.limitations ?? []), ...(report.batches ?? []).flatMap(b => b.limitations ?? [])],
+    scopeNotices: report.scopeNotices ?? (report.batches ?? []).flatMap(b => b.scopeNotices ?? []),
   };
   return markdownReport(merged).replace('# Code Review 报告', '# 全项目代码审查报告');
 }

package/src/project-review-service.mjs

@@ -204,7 +204,7 @@ export function createProjectReviewService(llm, { allowModelSending = false, aut
             : 'partial',
           repositoryRoot: s.root, vcs: s.vcs, model: p.route, modelCalls: calls, maxModelCalls: projectMaxCalls,
           totalTrackedNodes: s.totalTrackedNodes, selectionComplete: s.selectionComplete, inventoryComplete: s.inventoryComplete, inventoryScope: s.inventoryScope, inventoryNodes: s.inventoryNodes,
-          coverage, files, findings, artifacts: results.flatMap(result => result.artifacts ?? []), batches: results, resume: { reusedFiles: files.filter(file => file.reused).length },
+          coverage, files, findings, scopeNotices: results.flatMap(result => result.scopeNotices ?? []), artifacts: results.flatMap(result => result.artifacts ?? []), batches: results, resume: { reusedFiles: files.filter(file => file.reused).length },
           privateArtifacts: { enabled: typeof exec.onModelArtifact === 'function', sourceBearing: true, maxArtifactBytes: 1048576, maxJobBytes: 8388608, maxArtifacts: 4096, notice: 'Bounded private untrusted model outputs; raw access requires explicit consent and is not review evidence.' },
           execution: { nativeOutputTools, enableGroupPrimary, batchConcurrency: projectConcurrency, maxRetrievalRounds },
           notice: s.notice + REPORT_NOTICE_SUFFIX,

package/src/report-readability.mjs

@@ -95,7 +95,7 @@ function collect(report) {
     return { record, reasons: fileReasons(record, findings, associated), paths: record?.path != null ? [String(record.path)] : paths(record) };
   }).filter(item => item.reasons.length);
   const interactions = rows(report?.interactions).filter(record => record?.status !== 'completed').map(record => ({ record, reasons: [reason(record)], paths: paths(record) }));
-  return { unfinished, interactions, units, limitations: rows(report?.limitations), paths };
+  return { unfinished, interactions, units, limitations: rows(report?.limitations), scopeNotices: rows(report?.scopeNotices), paths };
 }
 
 function grouped(items) {
@@ -183,6 +183,15 @@ export function limitationDetailsMarkdown(report = {}) {
     output.push('以下仅为状态与原因元数据；源码、候选及其他完整字段见完整源报告 JSON。');
     originalDetails(output, packed(items.map(item => Object.fromEntries(keys.filter(key => Object.hasOwn(item.record ?? {}, key)).map(key => [key, item.record[key]])))));
   }
+  if (data.scopeNotices.length) {
+    output.push('', '### 范围说明（' + data.scopeNotices.length + ' 条记录）', '以下为上下文提供方式说明，不计为未完成限制；处理完成不等于全部批准上下文已核查。');
+    for (const record of data.scopeNotices) {
+      const paths = data.paths(record);
+      output.push('- 范围：' + (paths.length ? paths.map(escape).join('、') : '未标明文件身份'));
+    }
+    // Preserve complete original values, order, duplicates and file ownership.
+    originalDetails(output, data.scopeNotices);
+  }
   if (data.units.length) {
     output.push('', '### 未完成阶段续审元数据', '按 continuation.units 原始索引保留对应阶段及依赖身份；record 为原始状态元数据，resolvedReason 仅为展示解释，不覆盖原始 pendingReason。');
     originalDetails(output, data.units.map(item => ({ position: item.position, record: unitMetadata(item.record), paths: item.paths, resolvedReason: reason(item.evidence), dependencies: item.dependencies })));

package/src/report-summary.mjs

@@ -83,6 +83,7 @@ export function reportSummaryData(report = {}) {
     coverage: { total, loaded: files.length, detailsComplete, unit: project ? 'files' : 'change-entries', ...coverage },
     unfinished, crossFile, stages, unverifiedCandidates,
     limitations: rows(report.limitations).length,
+    scopeNotices: rows(report.scopeNotices).length,
     reused: findings.some(finding => finding?.reused || finding?.priorRun),
     currentSourceOnly: Boolean(project || findings.some(finding => finding?.attribution?.causality === 'current-source-only')),
   };
@@ -113,6 +114,7 @@ export function reportSummaryMarkdown(report = {}) {
   if (data.unverifiedCandidates) output.push('- 候选复核未完成：' + data.unverifiedCandidates + ' 项。');
   if (data.crossFile.total) output.push('- 跨文件综合：完成 ' + data.crossFile.completed + '/' + data.crossFile.total + ' 项；未完成 ' + data.crossFile.pending + ' 项。');
   if (data.limitations) output.push('- 已记录限制：' + data.limitations + ' 项，见限制明细。');
+  if (data.scopeNotices) output.push('- 范围说明：' + data.scopeNotices + ' 项，见附录；不计为未完成限制，完成不等于全部批准上下文已核查。');
   const qualifications = ['完成仅指处理进度，空发现不证明无缺陷'];
   if (counts.total) qualifications.push('按候选记录而非去重根因计数，复核支持及引用校验不是事实或因果证明');
   if (data.currentSourceOnly) qualifications.push('当前源码模式不评估历史新增归因');

package/src/review-jobs.mjs

@@ -78,7 +78,7 @@ export function createReviewJobs({directory}={}){
   },
   async resume(id,exec){const r=await owned(id,exec);if(jobs.get(id)&&!jobs.get(id).settled||['queued','running'].includes(r.status)||!r.resumeState)throw new Error('No terminal completed-chain resume state available');return structuredClone(r.resumeState);},
   async artifacts(id,exec,page){const r=await owned(id,exec);return artifacts.read(id,r.owner,page);},
-  async status(id,exec,page){const r=structuredClone(await owned(id,exec));delete r.resumeState;const active=jobs.get(id);if(active&&!active.settled&&!['queued','running'].includes(r.status)){r.status='running';r.stage='finalizing';delete r.result;delete r.finishedAt;}r.recovery=recoveryCoverage(r);r.progress=jobProgress(r);r.waitRevision=jobs.get(id)?.savedRevision??r.checkpoints.length;r.nextAction=['queued','running'].includes(r.status)?'Call code_review_wait with waitFor=terminal; reuse the exact returned waitRevision as after, never guess increments. No code_review_status call is needed after each event. Do not end the user request while task runs.':r.result?.previewId?'Inspect preview against current explicit user authorization. For a matching explicit review request, execute with confirmed=true without asking again. Do not execute preview-only or explicit wait-for-confirmation requests; ask for authorization if scope/model/destination exceeds authorization or is unclear. Preserve host permissions and snapshot/identity checks.':'Collect final report and unfinished scope, then provide one final answer.';if(!page)return r;const {offset=0,limit=20,checkpointIndex}=page;if(!Number.isSafeInteger(offset)||offset<0||!Number.isSafeInteger(limit)||limit<1||limit>100)throw new Error('Invalid result page');if(checkpointIndex!==undefined){if(!Number.isSafeInteger(checkpointIndex)||checkpointIndex<0||checkpointIndex>=r.checkpoints.length)throw new Error('Invalid checkpoint index');return {jobId:r.jobId,status:r.status,waitRevision:r.waitRevision,progress:r.progress,nextAction:r.nextAction,checkpointIndex,checkpoint:pageCheckpoint(r.checkpoints[checkpointIndex],offset,limit),page:{offset,limit}};}const result=r.result;const report=result?.report;return {...r,result:undefined,recovery:{...r.recovery,unfinishedStages:r.recovery.unfinishedStages.slice(offset,offset+limit),files:r.recovery.files.slice(offset,offset+limit),windows:r.recovery.windows.slice(offset,offset+limit),suggestedPaths:r.recovery.suggestedPaths.slice(offset,offset+limit),totalObservedFiles:r.recovery.files.length},checkpoints:r.checkpoints.slice(offset,offset+limit).map(c=>pageCheckpoint(c,0,limit)),checkpointPage:{offset,limit,total:r.checkpoints.length},...(result?{resultSummary:report?{status:report.status,coverage:report.coverage,model:report.model,snapshotId:report.snapshotId,diagnostics:report.diagnostics,execution:report.execution,modelCalls:report.modelCalls,totalFiles:report.files?.length,totalFindings:report.findings?.length,totalLimitations:report.limitations?.length}:({...result,files:result.files?.slice(offset,offset+limit),filePage:result.files?{offset,limit,total:result.files.length}:undefined}),resultPage:report?{files:report.files?.slice(offset,offset+limit),findings:report.findings?.slice(offset,offset+limit),totalFiles:report.files?.length,totalFindings:report.findings?.length}:undefined}: {})};},
+  async status(id,exec,page){const r=structuredClone(await owned(id,exec));delete r.resumeState;const active=jobs.get(id);if(active&&!active.settled&&!['queued','running'].includes(r.status)){r.status='running';r.stage='finalizing';delete r.result;delete r.finishedAt;}r.recovery=recoveryCoverage(r);r.progress=jobProgress(r);r.waitRevision=jobs.get(id)?.savedRevision??r.checkpoints.length;r.nextAction=['queued','running'].includes(r.status)?'Call code_review_wait with waitFor=terminal; reuse the exact returned waitRevision as after, never guess increments. No code_review_status call is needed after each event. Do not end the user request while task runs.':r.result?.previewId?'Inspect preview against current explicit user authorization. For a matching explicit review request, execute with confirmed=true without asking again. Do not execute preview-only or explicit wait-for-confirmation requests; ask for authorization if scope/model/destination exceeds authorization or is unclear. Preserve host permissions and snapshot/identity checks.':'Collect final report and unfinished scope, then provide one final answer.';if(!page)return r;const {offset=0,limit=20,checkpointIndex}=page;if(!Number.isSafeInteger(offset)||offset<0||!Number.isSafeInteger(limit)||limit<1||limit>100)throw new Error('Invalid result page');if(checkpointIndex!==undefined){if(!Number.isSafeInteger(checkpointIndex)||checkpointIndex<0||checkpointIndex>=r.checkpoints.length)throw new Error('Invalid checkpoint index');return {jobId:r.jobId,status:r.status,waitRevision:r.waitRevision,progress:r.progress,nextAction:r.nextAction,checkpointIndex,checkpoint:pageCheckpoint(r.checkpoints[checkpointIndex],offset,limit),page:{offset,limit}};}const result=r.result;const report=result?.report;return {...r,result:undefined,recovery:{...r.recovery,unfinishedStages:r.recovery.unfinishedStages.slice(offset,offset+limit),files:r.recovery.files.slice(offset,offset+limit),windows:r.recovery.windows.slice(offset,offset+limit),suggestedPaths:r.recovery.suggestedPaths.slice(offset,offset+limit),totalObservedFiles:r.recovery.files.length},checkpoints:r.checkpoints.slice(offset,offset+limit).map(c=>pageCheckpoint(c,0,limit)),checkpointPage:{offset,limit,total:r.checkpoints.length},...(result?{resultSummary:report?{status:report.status,coverage:report.coverage,model:report.model,snapshotId:report.snapshotId,diagnostics:report.diagnostics,execution:report.execution,modelCalls:report.modelCalls,totalFiles:report.files?.length,totalFindings:report.findings?.length,totalLimitations:report.limitations?.length,totalScopeNotices:report.scopeNotices?.length??0}:({...result,files:result.files?.slice(offset,offset+limit),filePage:result.files?{offset,limit,total:result.files.length}:undefined}),resultPage:report?{files:report.files?.slice(offset,offset+limit),findings:report.findings?.slice(offset,offset+limit),totalFiles:report.files?.length,totalFindings:report.findings?.length}:undefined}: {})};},
   async wait(id,exec,{after,waitFor='progress',timeoutMs=60000,signal}={}){
    const r=await owned(id,exec);
    if(!Number.isSafeInteger(timeoutMs)||timeoutMs<1||timeoutMs>60000)throw new Error('Invalid wait duration');

package/src/review.mjs

@@ -286,7 +286,6 @@ export async function reviewSnapshot(snapshot, model, options = {}) {
     const prepared = preparePrimaryInput(input, file, { instructions, scope, grouping: report.grouping, maxInputBytes, riskPlanningMode: options.riskPlanningMode, focusPrimaryContext: options.focusPrimaryContext === true, enableRiskPlanning: options.enableRiskPlanning, skipSmallRiskPlans: options.skipSmallRiskPlans });
     const { changes, payload } = prepared;
     Object.assign(state, prepared.metadata);
-    if (state.contextMode === 'approved-retrieval') result.limitations.push({ fileId: file.id, text: '初始输入仅提供聚焦上下文或身份目录，其他批准全文须按需读取；不声明所有上下文已核查。' });
     if (prepared.synthesis) state.synthesisStatus = prepared.synthesis.status === 'ready' ? 'pending' : 'blocked';
     if (prepared.synthesis?.status === 'blocked') result.limitations.push({ fileId: file.id, text: (prepared.synthesis.reason === 'batch-limit' ? '跨窗口综合批次数超过上限（20），未发送；全部主窗口仍独立审查' : '跨窗口综合输入超限，未发送') + '；单窗口完成不表示交互影响已检查' });
     if (state.sourceMode === 'change-windows') state.windowCoverage = { completed: [], pending: [...state.windowIds] };
@@ -822,6 +821,9 @@ export async function reviewSnapshot(snapshot, model, options = {}) {
   if (checkpointFailed) throw checkpointFailure;
   for (const result of fileResults) if (result?.state.continuationPending === false) { const state = result.state; state.resumeEligible = state.status === 'completed' && !state.stageFailures?.length && !state.reviewPasses?.some(pass => pass.status !== 'completed') && !state.windowCoverage?.pending?.length && !state.retrievalLimited && result.findings.every(finding => finding.verification?.status === 'completed') && report.interactions.filter(item => item.fileIds?.includes(state.fileId)).every(item => item.status === 'completed'); }
   report.files = fileResults.map(result => result.state);
+  // Derive only from host-prepared state, including reused checkpoint state. Never
+  // reclassify model or historical limitations by text or model-supplied tags.
+  report.scopeNotices = report.files.filter(file => file.contextMode === 'approved-retrieval').map(file => ({ fileId: file.fileId, text: '初始输入仅提供聚焦上下文或身份目录，其他批准全文须按需读取；不声明所有上下文已核查。' }));
   report.findings = fileResults.flatMap(result => result.findings);
   report.limitations.push(...fileResults.flatMap(result => result.limitations));
   report.execution = { fileConcurrency, maxRetrievalRounds };