@feng0531/dsh-code-review 0.58.0 → 0.58.1

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.58.1 — SVN 根目录属性预览修复
+
+修复 SVN 合并产生根目录 mergeinfo 属性时，默认预览和指定文件预览均报 Invalid relation file 的问题。保留根目录属性变更和覆盖统计，仅允许有目录身份的根节点 `.`，普通路径穿越仍拒绝；上下文竞争条目保留目录身份。独立 SVN 三文件变更与根 mergeinfo 场景验证预览零模型发送和工作副本状态不变。全面 SVN 扩展另按支持矩阵分批推进，尚未宣称历史区间或所有特殊状态已支持。
+
 ## 0.58.0 — 本地评测入口可靠性整理
 
-新增无需 Docker 的实验性 ReviewBench 本地入口与使用说明。要求显式本地仓库、模型与端点；复用严格 JSON 校验，拒绝截断和夹杂文本。未完成审查、不可映射的旧侧发现及超限结果明确失败并保存原报告，不伪装为空发现；同一行的不同问题不再误合并。mock 输出与可评分候选隔离，运行结果不覆盖。正常 DSH 工具行为、权限与预算不变；不代表真实模型质量已得到验证。
+新增无需 Docker 的实验性 ReviewBench 本地入口与使用说明。要求显式本地仓库、模型与端点；复用严格 JSON 校验，拒绝截断和夹杂文本。未完成审查、不可映射的旧侧发现及超限结果明确失败并保存原报告，不伪装为空发现；同一行的不同问题不再误合并。mock 输出与可评分候选隔离，运行结果不覆盖。正常 DSH 工具行为、权限与预算不变；不代表真实模型质量已得到验证。454 项串行回归、实际宿主、隔离安装与四组 CI 通过；npm、Release 与原产物 113,697 字节逐字节一致，双哈希、清单与冻结发布说明核验通过。
 
 ## 0.57.0 — 对齐结论与边界收尾
 

package/CORE-ALIGNMENT-TASKS.md

@@ -25,14 +25,15 @@
 
 - 继承、原型链与动态分派：需要语言解析器或类型检查器做语义绑定，超出「只读批准快照、不放宽预算」的保守边界。
 - 完整语义调用方、别名与跨目录实例上下文：需要语义解析器；保守的命名/默认/命名空间成员/类实例方法导航已交付，其余不猜测。
-- SVN 历史区间等旧侧独有来源：历史/旧侧三层链、oldContextPaths、删除旧正文复用已交付；SVN 历史区间按设计不支持。
+- SVN 全面支持扩展：用户已要求将工作副本特殊状态与固定 revision 历史审查纳入新目标。当前历史/旧侧三层链、oldContextPaths 和删除旧正文复用仅是 Git 已交付能力，不能算作 SVN 支持。SVN 当前缺口、批次与验收见 [SVN 支持矩阵](SVN-SUPPORT-MATRIX.md)。
 - 真实模型质量评测：目标明确后置，保存真实输出、人工裁决、分开统计，不作为对齐前提。
 
 各版交付结论见「当前验证结论」与 [版本变化](CHANGELOG.md)。
 
 ## 当前验证结论
 
-- 最近完整回归：449 项通过。
+- 最近完整回归：454 项通过。
+- v0.58.0 已正式交付实验性本地评测入口及可靠性整理。454 项串行回归、实际宿主、隔离安装与四组 CI 通过；npm、Release 与原包 113,697 字节逐字节一致，SHA-256、SHA-512、清单与冻结发布说明核验通过。正常 DSH 审查行为与预算不变；尚无真实模型基准评分。
 - v0.57.0 已正式交付对齐结论与边界收尾。449 项回归与四组 CI 通过；npm、Release 与原包 108,451 字节一致，SHA-256、SHA-512、清单与发布说明核验通过；不改变审查行为或参数。
 - v0.56.0 已正式交付依赖清单与配置文件专用指导。449 项回归、实际宿主、隔离安装与四组 CI 通过；npm、Release 与原包 108,081 字节一致，SHA-256、SHA-512、清单与发布说明核验通过；不改变审查行为或参数。
 - v0.55.0 已正式交付模板引擎与基础设施即代码专用指导。446 项回归、实际宿主、隔离安装与四组 CI 通过；npm、Release 与原包 106,969 字节一致，SHA-256、SHA-512、清单与发布说明核验通过；不改变审查行为或参数。

package/RELEASE-NOTES.md

@@ -1,16 +1,16 @@
-# v0.58.0 发布说明
+# v0.58.1 发布说明
 
-## 本地评测入口可靠性整理
+## SVN 根目录属性预览修复
 
-新增无需 Docker 的实验性 ReviewBench 本地评测入口和说明，复用现有审查核心。配置、模型调用、结果映射分开测试；模型和端点显式指定，不改变 DSH 宿主配置。
+修复 SVN 分支合并产生根目录 svn:mergeinfo 属性变更时，插件预览报 `Invalid relation file` 的问题。原问题既影响默认预览，也影响仅选择源码文件的预览，因为根目录条目仍保留在快照中。
 
-- 复用严格 JSON 校验，拒绝截断响应和夹杂文字；HTTP 错误不输出正文。
-- 未完成审查、无法表达为 head 行号的发现以及超限结果明确失败，保存完整原报告，不静默丢弃或假装成功。
-- 同一行的不同问题保留，只去除完全相同的导出记录；旧结果不会被覆盖。
-- mock 只生成冒烟文件，不生成可评分候选。
+- 保留根目录属性变化与覆盖统计，不删除条目掩盖问题。
+- 仅允许明确目录身份的根节点 `.`；普通文件的 `.`、`..` 路径仍被拒绝。
+- 上下文规划保留目录身份，避免再次触发关系校验错误。
+- 独立 SVN 测试覆盖三源码文件与根 mergeinfo 变更，验证两种预览均零模型发送、工作副本状态不变。
 
-## 使用与边界
+## SVN 后续支持
 
-正常 DSH 插件工具、权限和预算保持不变。实验入口采用单次直审，不启用检索、复核或业务分组；其结果不能代表完整插件配置。尚未完成真实模型、真实基准 PR 或 judge 评分验证。
+按新的全面 SVN 目标，复制/移动/替换、关键字与换行、固定 revision 历史审查及历史上下文将分批实现。现阶段并非所有 SVN 状态都可审查；见 [支持矩阵](SVN-SUPPORT-MATRIX.md)。
 
-本地入口说明见 [本地评测指南](evaluation/REVIEWBENCH-LOCAL.md)，插件日常使用见 [README](README.md)。
+本版本未对用户 r7315 的实际临时副本进行审查，不宣称该提交已由插件通过。日常使用见 [README](README.md)。

package/SVN-SUPPORT-MATRIX.md

@@ -0,0 +1,43 @@
+# SVN 审查支持矩阵与实施顺序
+
+本矩阵按用户新的「全面支持 SVN」要求扩展目标。此前把 SVN 历史审查列为设计不支持，不再作为此任务的完成理由。支持状态须由实际测试确认，不以规划、发布数量或版本号代替验收。
+
+## 不变的约束
+
+- 不对用户工作副本执行 checkout/update/switch/merge/revert/cleanup/commit；集成测试在独立本地仓库和临时工作副本运行。
+- 预览不发送源码，执行必须绑定同一预览、模型、工作目录与快照身份。
+- 本地工作副本模式默认离线；新增远端历史读取必须显式选择并在预览中披露网络访问和来源。
+- 历史来源固定仓库 UUID、规范 URL、peg revision、operative revision、节点路径与内容哈希；禁止把较新工作树当作历史正文。
+- 文件/字节/上下文/精确探测/调用/超时预算继续共享。超限、不可解码、不可定位和权限失败明确保留未完成状态。
+
+## 当前盘点（不是全部支持）
+
+| 场景 | 当前证据/限制 | 实施批次 |
+|---|---|---|
+| 常规文件修改、普通新增、删除 | 已有 BASE/工作字节与属性捕获测试 | 维持并扩充组合验收 |
+| 子目录属性变更 | 已有独立属性审查测试 | 第一批 |
+| 根目录属性/mergeinfo | 新测试复现 `Invalid relation file`；本地补丁通过指定三文件和默认预览，0 模型发送，456 项回归通过；尚未发布 | 第一批先发布此修复 |
+| 复制/移动、copyfrom 及目录结构变更 | 现有采集拒绝复制和目录结构变更；不能说已支持 | 第二批：明确旧侧来源和后代覆盖 |
+| 替换、删除再新增、文件/目录类型变化 | 当前 replaced 被拒绝 | 第二批：区分 BASE 旧节点与目标节点 |
+| 缺失、文本/属性/树冲突、阻塞 | 当前多数仅统一 blocked | 第三批：明确状态诊断；不能合成不存在的工作侧或声称审核通过 |
+| svn:keywords、svn:eol-style | keywords 当前阻断；换行保留原文但无规范化映射 | 第三批：双侧可验证映射，保留原字节和定位 |
+| svn:special/链接 | 当前阻断，不跟随链接取正文 | 第三批：如支持只审链接目标数据；不沿链接读取文件 |
+| switched、externals、稀疏/混合 revision | 多数拒绝；调用方/依赖链要求祖先 normal | 第四批：明确仓库边界与独立来源，禁止隐式跨仓读取 |
+| 固定 revision 单提交 | 未实现；Git commit 参数不能冒充 SVN revision | 第五批：新增明确 SVN 参数与冻结身份 |
+| 固定 revision 区间 | 未实现；不能通过临时重建工作树冒充原生历史能力 | 第五批：固定树端点与逐路径差异 |
+| 分支复制、分支间比较、合并来源 | mergeinfo 当前仅作为属性文本；不证明源码归属或被合并提交均已审查 | 第六批：URL/peg/operative revision 与 mergeinfo 导航披露 |
+| 当前三层依赖与限定目录调用方 | 已有安全 normal 节点的离线有界测试；根属性修改可能使祖先检查拒绝，需单独验证 | 第四批补祖先/混合状态组合 |
+| 历史两侧上下文与调用方 | 未实现 | 第六批：逐侧固定版本，不能回退至工作副本 |
+| 真正 r7314→r7315 样本 | 用户未提供隔离副本路径；本地类似复现不是该样本验收 | 获得路径后只读生成预览，不自动发送模型 |
+
+## 每批发布门禁
+
+1. 独立测试副本复现，确认 capture→关系推断→预览→执行的覆盖和拒绝路径。
+2. 专项、全量串行回归、实际宿主、隔离安装通过。
+3. 提交/推送/冻结版本标签；四矩阵 CI 通过后唯一 Actions 发布。
+4. 保存原始 Actions 产物，核验 npm/Release 逐字节、SHA-256、SHA-512、清单与冻结发布说明。
+5. 写入该批验证结论；更新本矩阵，剩余项继续保留未完成。
+
+## 收尾标准
+
+每行均有明确支持模式、失败处理、源码位置和测试证据。诊断未完成不等于场景已经可审查；任何保守拒绝须单列原因，不能把拒绝范围自动从目标中删除。真实模型质量另行评测，不由采集/发布测试推断。

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@feng0531/dsh-code-review",
-  "version": "0.58.0",
+  "version": "0.58.1",
   "private": false,
   "license": "SEE LICENSE IN LICENSE",
   "publishConfig": { "access": "public", "registry": "https://registry.npmjs.org/" },
   "main": "./index.mjs",
   "exports": { ".": "./index.mjs", "./package.json": "./package.json", "./locale/*.json": "./locale/*.json" },
-  "files": ["scripts/review-bench-adapter.mjs", "evaluation/REVIEWBENCH-LOCAL.md", "index.mjs", "cordis.patch.yml", "src/", "locale/", "README.md", "CHANGELOG.md", "RELEASE-NOTES.md", "DEVELOPMENT.md", "CORE-ALIGNMENT-TASKS.md", "CORE-SOURCE-INVENTORY.md", "LICENSE", "LICENSES/"],
+  "files": ["scripts/review-bench-adapter.mjs", "evaluation/REVIEWBENCH-LOCAL.md", "index.mjs", "cordis.patch.yml", "src/", "locale/", "README.md", "CHANGELOG.md", "RELEASE-NOTES.md", "DEVELOPMENT.md", "CORE-ALIGNMENT-TASKS.md", "CORE-SOURCE-INVENTORY.md", "SVN-SUPPORT-MATRIX.md", "LICENSE", "LICENSES/"],
   "dsh": { "bundle": { "patch": "./cordis.patch.yml" } },
   "repository": { "type": "git", "url": "https://github.com/f-e-n-g-0531/dsh-code-review.git" },
   "type": "module",

package/src/context-relations.mjs

@@ -22,7 +22,7 @@ export function inferContextRelations(input, file) {
   const primary = { ...file, id: 'primary' };
   const entries = contexts.map((c, i) => ({ id: 'context-' + i, path: c.path, eligibility: 'reviewable', left: { text: '' }, right: { text: c.text } }));
   // Other selected paths participate as opaque competitors, not context.
-  const competitors = input.files.filter(f => f.id !== file.id).map((f, i) => ({ id: 'selected-' + i, path: f.path, eligibility: 'blocked' }));
+  const competitors = input.files.filter(f => f.id !== file.id).map((f, i) => ({ id: 'selected-' + i, path: f.path, ...(f.kind === 'dir' ? {kind: 'dir'} : {}), eligibility: 'blocked' }));
   const all = [primary, ...entries, ...competitors];
   if (all.length > 200) return [];
   const byId = new Map(all.map(f => [f.id, f.path]));

package/src/cpp-relations.mjs

@@ -5,7 +5,7 @@ export function inferCppRelations(files) {
  if (!Array.isArray(files) || files.length > 200) throw new Error('Invalid C++ relation input');
  const byPath = new Map(), ids = new Set();
  for (const file of files) {
-  if (!file || typeof file.id !== 'string' || !file.id || ids.has(file.id) || typeof file.path !== 'string' || !file.path || file.path.includes('\\') || file.path.includes(':') || file.path.startsWith('/') || file.path.split('/').some(s => !s || s === '.' || s === '..') || byPath.has(file.path) || !['reviewable','excluded','blocked'].includes(file.eligibility)) throw new Error('Invalid C++ relation file');
+  if (!file || typeof file.id !== 'string' || !file.id || ids.has(file.id) || typeof file.path !== 'string' || !file.path || file.path.includes('\\') || file.path.includes(':') || file.path.startsWith('/') || (!(file.path === '.' && file.kind === 'dir') && file.path.split('/').some(s => !s || s === '.' || s === '..')) || byPath.has(file.path) || !['reviewable','excluded','blocked'].includes(file.eligibility)) throw new Error('Invalid C++ relation file');
   ids.add(file.id); byPath.set(file.path,file);
  }
  const edges = [];

package/src/file-relations.mjs

@@ -11,7 +11,7 @@ export function inferTestRelations(files) {
   if (!Array.isArray(files) || files.length > 200) throw new Error('Invalid relation input');
   const paths = new Set(), ids = new Set(), implementations = new Map(), tests = [];
   for (const file of files) {
-    if (!file || typeof file.id !== 'string' || !file.id || ids.has(file.id) || typeof file.path !== 'string' || !file.path || file.path.includes('\\') || file.path.split('/').some(p => !p || p === '.' || p === '..') || paths.has(file.path) || !['reviewable', 'blocked', 'excluded'].includes(file.eligibility)) throw new Error('Invalid relation file');
+    if (!file || typeof file.id !== 'string' || !file.id || ids.has(file.id) || typeof file.path !== 'string' || !file.path || file.path.includes('\\') || (!(file.path === '.' && file.kind === 'dir') && file.path.split('/').some(p => !p || p === '.' || p === '..')) || paths.has(file.path) || !['reviewable', 'blocked', 'excluded'].includes(file.eligibility)) throw new Error('Invalid relation file');
     ids.add(file.id); paths.add(file.path);
     if (file.eligibility !== 'reviewable') continue;
     const match = /^(.*)\.(mjs|cjs|js|jsx|ts|tsx)$/.exec(file.path);

package/src/import-relations.mjs

@@ -12,7 +12,7 @@ export function inferImportRelations(files) {
   if (!Array.isArray(files) || files.length > 200) throw new Error('Invalid import relation input');
   const byPath = new Map(), ids = new Set();
   for (const file of files) {
-    if (!file || typeof file.id !== 'string' || !file.id || ids.has(file.id) || typeof file.path !== 'string' || !file.path || file.path.includes('\\') || file.path.startsWith('/') || file.path.includes(':') || file.path.split('/').some(p => !p || p === '.' || p === '..') || byPath.has(file.path) || !['reviewable', 'blocked', 'excluded'].includes(file.eligibility)) throw new Error('Invalid import relation file');
+    if (!file || typeof file.id !== 'string' || !file.id || ids.has(file.id) || typeof file.path !== 'string' || !file.path || file.path.includes('\\') || file.path.startsWith('/') || file.path.includes(':') || (!(file.path === '.' && file.kind === 'dir') && file.path.split('/').some(p => !p || p === '.' || p === '..')) || byPath.has(file.path) || !['reviewable', 'blocked', 'excluded'].includes(file.eligibility)) throw new Error('Invalid import relation file');
     ids.add(file.id); byPath.set(file.path, file);
   }
   const edges = [];

package/src/svn.mjs

@@ -38,7 +38,7 @@ export async function captureSvn(cwd, options = {}) {
     signal?.throwIfAborted();
     const name = entry['@_path'].replaceAll('\\', '/');
     const state = entry['wc-status'], status = state['@_item'];
-    const item = { id: hash(name), path: name, rawStatus: status, baseRevision: state['@_revision'], eligibility: 'reviewable', properties: [] };
+    const item = { id: hash(name), path: name, ...(name === '.' ? {kind: 'dir'} : {}), rawStatus: status, baseRevision: state['@_revision'], eligibility: 'reviewable', properties: [] };
     files.push(item);
     if (isSecretPath(name)) { item.eligibility='excluded'; item.reason='Credential path excluded'; continue; }
     if ((selectedPaths && !selectedPaths.includes(name)) || (status === 'unversioned' && !selectedPaths?.includes(name))) {