@feng0531/dsh-code-review 0.20.1 → 0.20.2

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # 变更记录
 
+## 0.20.2
+
+- 可选主文件唯一全行原文锚纠正，仅同侧且重叠原精确编辑，保留原完整候选/状态，仍经原验证，不加调用权限。
+- 歧义/无引用/不重叠拒绝，有界CPU及取消；注册/跨文件纠正未开启，全量250通过。
+
 ## 0.20.1
 
 - SVN自动上下文使用离线正常节点索引，排除外部/切换/复制/冲突/属性变化，双读及规范化完整状态指纹一致性验证。

package/README.md

@@ -4,7 +4,7 @@
 
 ## 当前状态
 
-当前代码版本为 0.20.1
+当前代码版本为 0.20.2
 
 可在预览传入 `autoContext: true`，显式选择有界Git/SVN直接import/include上下文捕获；默认关闭。候选、阻塞、截断及实际路径在预览可见，正文仅在确认后发送，执行重新验证快照。仅当前侧导航非历史/调用图，SVN使用离线正常节点索引支持同样导航；不会自动引入未跟踪或排除变更。，包含有界覆盖预检、显式项目规则及候选证据与反证复核；发布状态以 npm 和 GitHub Release 为准。已通过真实临时 Git/SVN 仓库测试，以及隔离进程中的真实 DSH ToolRuntime 注册验证。当前开发会话的实际预览已返回0.6.0的plan/rules字段，空变更预览与默认关闭外发已验证；已进行经宿主审批的六次小规模真实试跑，累计12次模型调用；四组合成正负例与历史修复对照符合预期，历史引入方向检出候选但引用复核未完成（partial）。不是独立盲测或整体质量证明。详情见 [试跑结果](evaluation/PILOT-RESULTS.md)。
 

package/package.json

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

package/src/anchor-correction.mjs

@@ -0,0 +1,18 @@
+// Pure, single-attempt exact correction. Never searches another file or source side.
+export function correctAnchor(raw, file, changes, {signal}={}) {
+ signal?.throwIfAborted();const a=raw?.anchor;
+ const result=(status,reason)=>({status,reason,originalCandidate:structuredClone(raw)});
+ if(!raw||raw.fileId!==file.id||a?.kind!=='line'||!['old','new'].includes(a.side)||!Number.isSafeInteger(a.start)||!Number.isSafeInteger(a.end)||a.start<1||a.end<a.start||typeof a.snippet!=='string'||!a.snippet.trim()||a.snippet.length>8000)return result('not-applicable','invalid-anchor');
+ const text=(a.side==='old'?file.left:file.right)?.text;if(typeof text!=='string'||Buffer.byteLength(text)>256*1024)return result('not-applicable','source-limit');
+ const source=text.replaceAll('\r\n','\n').split('\n'),snippet=a.snippet.replaceAll('\r\n','\n').split('\n');
+ if(source.slice(a.start-1,a.end).join('\n')===snippet.join('\n'))return result('unchanged','already-exact');
+ if(changes?.status!=='changed'||changes.precision!=='exact'||!Array.isArray(raw.attribution?.editIds)||!raw.attribution.editIds.length)return result('rejected','missing-exact-edit');
+ const edits=changes.edits.filter(e=>raw.attribution.editIds.includes(e.id));if(edits.length!==raw.attribution.editIds.length)return result('rejected','unknown-or-duplicate-edit');
+ // Bound CPU even for a long multiline snippet; no silent partial search acceptance.
+ if(source.length*snippet.length>2000000)return result('rejected','search-work-limit');
+ let match;for(let i=0;i+snippet.length<=source.length;i++){signal?.throwIfAborted();if(!snippet.every((line,j)=>source[i+j]===line))continue;if(match!==undefined)return result('rejected','ambiguous-exact-text');match=i;}
+ if(match===undefined)return result('rejected','no-exact-text');
+ const start=match+1,end=match+snippet.length;
+ if(!edits.some(e=>e[a.side].count>0&&start<e[a.side].start+e[a.side].count&&end>=e[a.side].start))return result('rejected','does-not-overlap-referenced-edit');
+ const candidate=structuredClone(raw);candidate.anchor={...a,start,end};return {...result('corrected','unique-exact-text-overlaps-edit'),candidate};
+}

package/src/review.mjs

@@ -1,3 +1,4 @@
+import { correctAnchor } from './anchor-correction.mjs';
 import { BUSINESS_GROUP_INSTRUCTIONS, prepareBusinessGroups, validateBusinessGroups } from './business-groups.mjs';
 import { RISK_PLAN_INSTRUCTIONS, validateRiskPlan } from './risk-plan.mjs';
 import { prepareInteractionInput } from './interaction-input.mjs';
@@ -167,7 +168,19 @@ export async function reviewSnapshot(snapshot, model, options = {}) {
         }
         const request = { ...enclosingRequest, input: batchPayload };
         const auditStart = state.retrievalAudit?.length ?? 0;
-        const response = await executor(request);
+        let response = await executor(request);
+        if (options.enableAnchorCorrection === true && !batch.interaction) {
+          if (Buffer.byteLength(typeof response === 'string' ? response : JSON.stringify(response)) > 128 * 1024) throw new Error('Review response too large');
+          const decoded = typeof response === 'string' ? JSON.parse(response) : structuredClone(response);
+          if (Array.isArray(decoded?.findings) && decoded.findings.length <= 50) {
+            decoded.findings = decoded.findings.map(raw => {
+              const correction = correctAnchor(raw, file, changes, { signal: request.signal });
+              if (!['unchanged','not-applicable'].includes(correction.status)) (state.anchorCorrections ??= []).push(correction);
+              return correction.status === 'corrected' ? correction.candidate : raw;
+            });
+            response = decoded;
+          }
+        }
         const generated = batch.interaction ? validateInteractionFindings(response, input, batch.interaction, (state.retrievalAudit ?? []).slice(auditStart), scope.catalog()) : validateResponse(response, file, changes);
         if (batch.synthesis) validateSynthesisReads(generated.findings, changes, (state.retrievalAudit ?? []).slice(auditStart), scope.catalog(), file.id, input.id);
         if (!batch.interaction && state.sourceMode === 'change-windows') validateWindowFindings(generated.findings, batch.synthesis ? prepared.synthesis.windows : JSON.parse(batch.payload).windows, batch.synthesis === true);