@feng0531/dsh-code-review 0.74.0 → 0.75.0

package/CHANGELOG.md

@@ -2,9 +2,13 @@
 
 本文件介绍对使用者有意义的变化。安装与参数见 [README](README.md)，本次发布正文见 [发布说明](RELEASE-NOTES.md)。历史提交和安装包保留在 [GitHub Releases](https://github.com/f-e-n-g-0531/dsh-code-review/releases)；本文不把开发日志当作功能介绍。
 
+## 0.75.0 — SVN 手动上下文离线来源校验
+
+未启用autoContext时，未变更显式上下文也校验SVN节点/属性/URL及祖先身份并末尾复核，保留EOL元数据。已选变更复用原快照，不读取排除或删除来源。不安全来源明确拒绝，不再仅凭本地文件位置读取。只读离线，显式来源共享20名额/1MiB/30秒预算；其他SVN缺口继续。
+
 ## 0.74.0 — SVN 本地依赖/调用方换行信息与正常属性修复
 
-工作副本自动依赖、显式导航入口与命中调用方保留EOL元数据，预览披露而不替换原文。修复调用方索引把已提交正常属性误当作不可选的遗漏；候选仍经节点/属性/URL/祖先核验，关键字、特殊和非正常状态不放开。不新增网络或读取命令，完整调用覆盖不作保证。
+工作副本自动依赖、显式导航入口与命中调用方保留EOL元数据，预览披露而不替换原文。修复调用方索引把已提交正常属性误当作不可选的遗漏；候选仍经节点/属性/URL/祖先核验，关键字、特殊和非正常状态不放开。不新增网络或读取命令，完整调用覆盖不作保证。511项串行全量回归、11项本地导航安全专项、实际宿主/隔离安装和四矩阵CI通过；原始Actions/npm/Release包144,494字节逐字节一致，双哈希、清单及冻结正文通过。
 
 ## 0.73.0 — SVN 历史上下文换行来源信息
 

package/CORE-ALIGNMENT-TASKS.md

@@ -32,7 +32,8 @@
 
 ## 当前验证结论
 
-- 最近完整回归：510项通过；v0.73.0 最终上下文/检索兼容66项通过。
+- 最近完整回归：511项通过；v0.74.0 本地导航安全专项11项通过。
+- v0.74.0 已发布离线自动依赖/显式导航入口/命中调用方EOL元数据与normal属性候选漏选修复，关键字/special/非正常状态仍检查；宿主/隔离安装与四矩阵CI通过，原始Actions/npm/Release包144,494字节逐字节一致，双哈希/清单/冻结正文通过。独立无autoContext显式本地属性映射、关键字与其余SVN缺口继续。
 - v0.73.0 已发布固定历史上下文逐侧EOL来源元数据，显式/旧侧/依赖/调用方保持原身份，预览不含正文，篡改映射拒绝；宿主、隔离安装与四矩阵CI通过，原始Actions/npm/Release 143,452字节逐字节一致，双哈希/清单/冻结说明通过。工作副本关键字上下文及其余SVN缺口继续。
 - v0.72.0 已发布可操作使用指南与固定历史变更文件逐侧EOL元数据，缺失/目录侧明确null，原来源身份/正文保持；实际宿主、隔离安装与四矩阵CI通过；原始Actions/npm/Release包142,623字节逐字节一致，双哈希/清单/冻结发布说明通过。历史上下文EOL、关键字上下文、未提交替换和跨URL等仍未闭环。
 - v0.71.0 已发布工作副本 EOL 纯元数据映射，原文/原字节证据保持，单CR明确不套用规范化行号。实际宿主、隔离安装和四矩阵CI通过；原始Actions/npm/Release 139,804字节逐字节一致，双哈希、清单和冻结正文核验通过。历史EOL元数据、关键字上下文、未提交替换和跨URL等继续待闭环。

package/README.md

@@ -156,7 +156,7 @@ dsh plugin add https://github.com/f-e-n-g-0531/dsh-code-review.git
 
 这些文件和目录需要实际存在，路径相对于所选仓库：
 
-- `contextPaths`：手动加入辅助文件，不代表该文件本身完成了主审。
+- `contextPaths`：手动加入辅助文件，不代表该文件本身完成了主审。SVN工作副本的未变更文件须通过离线正常节点/属性及来源检查；关键字、特殊或不可确认来源会明确拒绝，已选择的可审查变更复用快照。
 - `autoContext:true`：自动补充最多三层文本依赖，不是全仓扫描或编译器分析。
 - `callerScopePaths`：在明确指定的非根目录中寻找字面引用。预览可能扫描该目录内不匹配的候选正文；只有捕获的匹配上下文随执行发送模型。不保证找到全部调用方。
 - `rulePaths`：最多四个 UTF-8 Markdown 或文本规则文件，不能同时列为上下文。

package/RELEASE-NOTES.md

@@ -1,14 +1,14 @@
-# v0.74.0 发布说明
+# v0.75.0 发布说明
 
-## SVN 工作副本依赖和调用方的换行信息
+## SVN 手动上下文的离线来源校验
 
-使用 `autoContext:true` 补充依赖，或用 `callerScopePaths` 寻找引用方时，安全正常节点的 `svn:eol-style` 现在会与原始上下文一起保留。预览披露已捕获上下文的换行样式、计数和视图哈希，不显示正文。
+工作副本审查中，单独使用 `contextPaths`（未启用自动依赖）加入未变更文件时，现在也会核对SVN节点、仓库URL、属性与祖先身份，并在采集末尾复核。正常EOL属性与原始正文一起保留，预览披露换行元数据。
 
-同时修复调用方索引遗漏已有正常属性文件的问题：SVN 将已提交属性标为 normal，而不是无属性的 none。两种正常状态现在都进入候选，正文读取前仍须通过节点、属性、URL及祖先检查；修改或冲突属性仍拒绝。
+- 不再仅凭文件位于本地目录就当作安全SVN上下文。未跟踪/排除变更、关键字和特殊节点等不可确认来源明确拒绝。
+- 已选入且可审查的变更文件继续复用原快照正文和工作侧EOL元数据，不恢复被排除或删除的文件。
+- 显式未变更来源共享20个名额、1MiB正文与30秒采集及身份复核预算；只读本地元数据，不连接远端。
+- 原文、原始字节哈希和行号保持，属性变化要求重新预览；不以换行映射证明行为等价。
 
-- 仅使用已有离线元数据和原文，不增加 SVN 命令或网络访问。
-- 自动依赖入口文件也保留元数据；属性变化使快照过期，执行前必须重新预览。
-- 原正文、字节哈希和行号不变；单CR不能套用规范化LF行号，换行视图不证明行为等价。
-- 关键字、特殊节点、切换/外部根和非正常属性继续受原安全检查约束，不把本修复视为完整调用覆盖。
+如果以前把不受支持的节点当作手动上下文加入，本版可能明确报错，而不是继续读取。工作副本关键字上下文、未提交替换、有效继承mergeinfo和跨URL比较仍未完成。
 
-本批不放开工作副本关键字上下文；未提交替换、跨URL分支及有效继承mergeinfo等仍待实现。日常操作见 [README](README.md)，具体限制与证据见 [SVN支持矩阵](SVN-SUPPORT-MATRIX.md)。
+操作示例见 [README](README.md)，状态与证据见 [SVN支持矩阵](SVN-SUPPORT-MATRIX.md)。

package/SVN-SUPPORT-MATRIX.md

@@ -31,11 +31,20 @@
 | 历史两侧上下文与调用方 | v0.63.0 已发布显式 contextPaths 固定双侧来源及目标 rulePaths，UUID/URL/peg/operative revision/原字节与正文 hash 验证，禁止脏工作树与排除项回流。477 项串行回归、宿主、隔离安装、四矩阵 CI 与原包/npm/Release 127,355 字节、双哈希/清单/发布说明通过。v0.64.0 已发布 oldContextPaths 旧侧独有正文、SVN 来源专用校验、预览披露与选中删除身份复用；479 项串行回归、宿主、隔离安装、四矩阵 CI 与原包/npm/Release 127,879 字节、双哈希/清单/发布说明通过；本地开发了注入固定版本 lookup/read 能力的三层历史依赖导航核心，逐侧共享512探测/20候选/1MiB来源与上下文名额；排除项、旧侧限定和规则隔离测试通过。v0.68.0 已发布三层逐侧历史 autoContext，固定父目录有界清单 lookup，清单成功证明缺失、访问失败直接传播，真实旧/新版本链和脏工作树隔离通过。496 项串行回归、宿主、隔离安装、四矩阵 CI 与原包/npm/Release 134,056 字节、双哈希/清单/发布说明通过；本地新增逐侧固定 revision 调用方核心能力接口：共享256索引/128扫描/1MiB正文/512侧探测，命中侧才捕获正文，规则/变更排除与old-only限制保留，同侧命名调用声明导航测试通过；本地已接入固定 SVN 显式目录递归清单/同侧正文、统一30秒/512命令/元数据/正文预算，caller 与依赖共享精确探测及来源上限，真实集成专项13项通过；v0.69.0 已发布历史限定调用方和autoContext组合；502项串行全量回归、组合模型输入隔离、宿主/隔离安装、四矩阵 CI 通过。原包/npm/Release 136,030 字节、双哈希/清单/发布说明通过；字面发现仍不证明完整调用覆盖 | 第六批：逐侧固定版本，不能回退至工作副本 |
 | 真正 r7314→r7315 样本 | 用户未提供隔离副本路径；本地类似复现不是该样本验收 | 获得路径后只读生成预览，不自动发送模型 |
 
-## 下一批：本地导航 EOL 与正常属性候选（v0.74.0 待发布）
+## 下一批：独立显式本地上下文（v0.75.0 待发布）
+
+- `src/svn-explicit-context.mjs` 在无autoContext模式核验未变更显式SVN节点与属性/URL/祖先身份，末尾复核；20名额/1MiB正文/30秒含核验，离线只读。
+- `snapshot.mjs` 复用显式捕获正文，已选可审查变更复用工作侧EOL；排除/删除/关键字/special等不回流。预览沿用contextEolMappings披露，不显示原文。
+- `test/svn-explicit-context.test.mjs` 真实离线复现：CRLF元数据、稳定身份、0预览发送、status不变、关键字拒绝、未跟踪排除、属性变化及取消拒绝；与既有SVN采集组合16项通过。512项串行全量回归、16项既有采集组合及最终3项安全组合、实际宿主/隔离安装通过，尚未发布。
+- 工作副本关键字、特殊状态、跨URL与有效继承mergeinfo仍未完成。
+
+## 本地导航 EOL 与正常属性候选（v0.74.0 已发布）
+
+冻结提交 `09241e046eaeff4f9c917102bc9dad97d91e81d9`；标签CI `38702380795`，唯一发布运行 `38703089522`，原工件 `11957397597`。原始Actions/npm/Release包144,494字节逐字节一致；SHA-256 `35946d5d536482cb8fd7061a9a291c030c49a564e9eeffacac82ecf206e37552`，SHA-512、清单及冻结正文核验通过。
 
 - `svnCallerNode` 返回既有已验证属性中的EOL样式，依赖链/显式导航入口/命中调用方根据原文生成映射，预览只披露元数据；不新增命令或网络范围。
 - 真实复现发现带已提交属性的调用方被索引 none-only 条件漏选；改为 none/normal，正文仍经既有关键字/special/externals/URL/祖先安全检查。modified/conflicted 等不放开。
-- `test/svn-working-context-eol.test.mjs` 离线真实仓库组合通过：入口→依赖和限定调用方映射、非匹配正文不捕获、预览0发送、模型原文、status不变、快照稳定及属性变化拒绝。既有关键字拒绝7项依赖专项通过；511项串行全量回归与11项本地导航安全专项、实际宿主/隔离安装通过，尚未发布。
+- `test/svn-working-context-eol.test.mjs` 离线真实仓库组合通过：入口→依赖和限定调用方映射、非匹配正文不捕获、预览0发送、模型原文、status不变、快照稳定及属性变化拒绝。既有关键字拒绝7项依赖专项通过；511项串行全量回归与11项本地导航安全专项、实际宿主/隔离安装、四矩阵标签及发布CI通过，发布工件核验完成。
 - 独立无autoContext的显式本地上下文未因本批新增属性映射；工作副本关键字上下文与其余矩阵缺口继续保留。
 
 ## 历史上下文 EOL（v0.73.0 已发布）

package/package.json

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

package/src/snapshot.mjs

@@ -1,4 +1,5 @@
 import {captureSvnDependencyChain} from './svn-dependency-chain.mjs';
+import {captureSvnExplicitContext} from './svn-explicit-context.mjs';
 import {captureDependencyChain} from './dependency-chain.mjs';
 import {captureSvnCallers} from './svn-caller-capture.mjs';
 import { captureCallers } from './caller-capture.mjs';
@@ -62,15 +63,20 @@ export async function captureSnapshot(cwd, options = {}) {
     const chain=await captureSvnDependencyChain(snapshot.root,snapshot.files,options);
     explicitSeeds=chain.explicitSeeds;context.push(...chain.context);autoContext=chain.autoContext;indexOptions=chain.indexOptions;verifyDependencies=chain.verify;
   }
+  let verifyExplicit;
+  if(type==='svn'&&options.autoContext!==true&&contextPaths.length){
+    const explicit=await captureSvnExplicitContext(snapshot.root,[...new Set(contextPaths)].filter(name=>!snapshot.files.some(f=>f.path===name)).sort(),options);
+    explicitSeeds=explicit.context;verifyExplicit=explicit.verify;indexOptions=explicit.options;
+  }
   for (const name of [...new Set(contextPaths)].sort()) {
     const changed = snapshot.files.find(f => f.path === name);
     if (changed) {
       if (changed.eligibility !== 'reviewable') throw new Error('Context path is an excluded or blocked change');
       if (changed.rightExists === false) throw new Error('Deleted path cannot be working context');
-      context.push({ path: name, ...changed.right });
+      context.push({ path: name, ...changed.right,...(changed.eolMapping?.new?{eolMapping:structuredClone(changed.eolMapping.new)}:{}) });
     } else {
-      const item={path:name,...await readLocal(snapshot.root,name,indexOptions)};
       const seed=explicitSeeds.find(c=>c.path===name);
+      const item=type==='svn'&&options.autoContext!==true&&seed?structuredClone(seed):{path:name,...await readLocal(snapshot.root,name,indexOptions)};
       if(seed&&seed.hash!==item.hash)throw new Error('Explicit dependency seed changed during capture');
       if(seed?.eolMapping)item.eolMapping=structuredClone(seed.eolMapping);
       context.push(item);
@@ -100,6 +106,7 @@ export async function captureSnapshot(cwd, options = {}) {
   if (index && (await (type === 'git' ? gitContextIndex : svnContextIndex)(snapshot.root, indexOptions)).fingerprint !== index.fingerprint) throw new Error('Tracked context index changed during capture');
   const verifiedRules = await captureProjectRules(snapshot.root, rulePaths, { signal: options.signal, files: snapshot.files });
   if (JSON.stringify(verifiedRules) !== JSON.stringify(rules)) throw new Error('Rules changed during capture');
+  if (verifyExplicit) await verifyExplicit();
   if (verifyDependencies) await verifyDependencies();
   if (verifyCallers) await verifyCallers();
   const result = { ...snapshot, context, ...(callerDiscovery ? {callerDiscovery} : {}), ...(autoContext ? { autoContext } : {}), ...(rules.length ? { rules } : {}) };

package/src/svn-explicit-context.mjs

@@ -0,0 +1,14 @@
+import {svnCallerNode} from './svn-caller-node.mjs';
+import {readLocal} from './content.mjs';
+import {svnEolMapping} from './svn-eol-mapping.mjs';
+export async function captureSvnExplicitContext(root,names,options={}){
+ const signal=AbortSignal.any([AbortSignal.timeout(30000),...(options.signal?[options.signal]:[])]),bounded={...options,signal,maxFileBytes:Math.min(options.maxFileBytes??256*1024,256*1024)};
+ if(!Array.isArray(names)||names.length>20||new Set(names).size!==names.length)throw new Error('Invalid SVN explicit context selection');
+ const context=[],identities=[];let bytes=0;
+ for(const name of names){
+  const node=await svnCallerNode(root,name,bounded),content=await readLocal(root,name,bounded);bytes+=content.bytes;if(bytes>1024*1024)throw new Error('SVN explicit context source byte limit exceeded');
+  context.push({path:name,...content,...(node.eolStyle?{eolMapping:svnEolMapping(content.text,node.eolStyle)}:{})});identities.push({path:name,fingerprint:node.fingerprint});
+ }
+ const verify=async()=>{for(const node of identities)if((await svnCallerNode(root,node.path,bounded)).fingerprint!==node.fingerprint)throw new Error('SVN explicit context identity changed during capture');signal.throwIfAborted();};
+ return {context,verify,options:bounded,sourceBytes:bytes};
+}