@feng0531/dsh-code-review 0.66.0 → 0.66.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.66.1 — SVN 合并记录变化下的工作副本导航修复
+
+修复根目录 mergeinfo 条目在自动依赖和反向调用方候选中被当作非法源码路径的失败；仅目录身份根节点可跳过源码候选。祖先属性 modified 仅在双侧属性比较确认只改变 svn:mergeinfo 时允许导航，旧/新属性仍入身份指纹；其他属性变化、externals/special、switched 或冲突继续拒绝。离线依赖和限定调用方集成验证通过，不扩展历史自动导航。
+
 ## 0.66.0 — SVN 历史显式 mergeinfo 来源导航
 
-对已批准历史变更的 svn:mergeinfo 属性增加有界来源路径/区间导航，保留非继承标记，不展开 revision、不读取分支源码。预览和报告 history 保留旧/新导航；无效或超限元数据明确标记且不输出部分导航，不阻断原属性审查。不是有效继承 mergeinfo、源码归属或已审查提交证明；跨 URL 比较仍待实现。
+对已批准历史变更的 svn:mergeinfo 属性增加有界来源路径/区间导航，保留非继承标记，不展开 revision、不读取分支源码。预览和报告 history 保留旧/新导航；无效或超限元数据明确标记且不输出部分导航，不阻断原属性审查。不是有效继承 mergeinfo、源码归属或已审查提交证明；跨 URL 比较仍待实现。485 项串行回归、宿主、隔离安装及四矩阵 CI 通过；原产物/npm/Release 129,866 字节逐字节一致，双哈希、清单与冻结发布说明通过。
 
 ## 0.65.0 — SVN 历史文件/目录类型替换
 

package/CORE-ALIGNMENT-TASKS.md

@@ -32,7 +32,8 @@
 
 ## 当前验证结论
 
-- 最近完整回归：482 项通过。
+- 最近完整回归：485 项通过。
+- v0.66.0 已交付批准历史 mergeinfo 属性的有界来源/区间导航，不展开 revision、不读取来源，也不证明有效继承/归属/审查覆盖。485 项串行回归、宿主、隔离安装及四矩阵 CI 通过；原产物/npm/Release 129,866 字节逐字节一致，双哈希、清单与冻结发布说明通过。跨 URL 分支比较、自动历史链/调用方等仍未完成。
 - v0.65.0 已交付固定 SVN 历史文件/目录类型替换，各侧节点类型/正文/属性与来源独立绑定，预览披露且后代覆盖保护保留。482 项串行回归、宿主、隔离安装和四矩阵 CI 通过；原产物/npm/Release 128,792 字节逐字节一致，双哈希、清单与冻结发布说明通过。未提交替换、自动历史链/调用方和其他全面 SVN 缺口仍待实现。
 - v0.64.1 已修复已提交 SVN 历史替换的同路径增删记录，真实目标旧/新正文和属性逐侧读取。481 项串行回归、宿主、隔离安装和四矩阵 CI 通过；原产物/npm/Release 128,279 字节逐字节一致，双哈希、清单与冻结发布说明通过。未提交工作副本替换和类型变化等仍未支持，全面目标继续。
 - v0.64.0 已交付 SVN 历史 oldContextPaths，仅旧侧正文及来源身份，不合成目标，选中旧变更复用原身份，排除项拒绝回流。479 项串行回归、宿主、隔离安装和四矩阵 CI 通过；原产物/npm/Release 127,879 字节逐字节一致，双哈希、清单与冻结发布说明通过。自动历史链、调用方及全面 SVN 矩阵仍未闭环。

package/RELEASE-NOTES.md

@@ -1,14 +1,15 @@
-# v0.66.0 发布说明
+# v0.66.1 发布说明
 
-## SVN 历史显式 mergeinfo 来源导航
+## SVN 合并记录变化下的工作副本导航修复
 
-从已批准固定历史变更中的 svn:mergeinfo 属性生成来源路径和 revision 区间导航，预览及报告 history 保存两侧结果。
+修复根目录仅有 svn:mergeinfo 属性变化时，自动依赖与限定调用方发现可能报 Unsafe repository path 或祖先状态拒绝的问题。
 
-- 保留非继承 `*` 标记，不枚举区间内每个 revision，不增加来源正文读取或网络请求。
-- 导航限 16KiB、100 来源路径、500 区间；无效/超限时明确标记，不静默裁剪或展示部分结果。
-- 已排除/阻断变更不进入导航，原属性正文仍按既有证据流程审查。
-- 来源路径不代表有效继承 mergeinfo、提交归属、审查覆盖，也不授权读取分支源码。
+- 明确目录身份的根节点 `.` 不作为源码依赖入口或调用目标；普通非法路径仍拒绝。
+- 祖先属性 modified 只有在离线读取 BASE/当前属性并确认仅 mergeinfo 不同后，才允许导航。
+- 旧/新属性仍入身份指纹；mergeinfo 修改触发指纹变化，不能绕过执行复核。
+- 其他属性变化、encoded/duplicate 属性、externals/special、冲突及来源身份变化继续拒绝。
+- 独立离线 SVN 测试验证依赖与限定调用方捕获，不新增联网或模型发送。
 
-跨 URL 分支比較、自动历史链、调用方、工作副本替换等仍未完成，详见 [SVN 支持矩阵](SVN-SUPPORT-MATRIX.md)。本版本不把导航作为缺陷证据或完整合并审计。
+本版仅修复工作副本合并记录场景，历史自动链/调用方、未提交替换及其他全面 SVN 项仍在 [支持矩阵](SVN-SUPPORT-MATRIX.md) 中，目标尚未完成。
 
-用法见 [README](README.md)。
+使用见 [README](README.md)。

package/SVN-SUPPORT-MATRIX.md

@@ -26,8 +26,8 @@
 | switched、externals、稀疏/混合 revision | 多数拒绝；调用方/依赖链要求祖先 normal | 第四批：明确仓库边界与独立来源，禁止隐式跨仓读取 |
 | 固定 revision 单提交 | v0.62.0 已发布 svnRevision：r-1/r 固定对象、UUID/URL/peg/operative revision 逐侧绑定，正文不读工作树；远端显式 svnAllowNetwork。472 项串行回归、宿主、隔离安装、四矩阵 CI 及原包/npm/Release 125,486 字节、双哈希/清单/发布说明通过。历史上下文/规则/调用方尚未支持 | 第五批：新增明确 SVN 参数与冻结身份 |
 | 固定 revision 区间 | v0.62.0 已发布 svnBaseRevision/svnTargetRevision，固定树端点 summary+逐侧 cat/proplist，目录后代与属性纳入预算，不建临时工作树；同上述完整发布验证。分支间不同 URL 和历史上下文尚未支持 | 第五批：固定树端点与逐路径差异 |
-| 分支复制、分支间比较、合并来源 | 本地新增批准历史 mergeinfo 属性的来源路径/区间导航，保留非继承标记、有界不展开 revision、不读取分支源码；invalid/limited 明确且不裁剪。待 v0.66.0 发布；有效继承 mergeinfo、跨 URL 比较、源码归属/合并提交覆盖仍未支持 | 第六批：URL/peg/operative revision 与 mergeinfo 导航披露 |
-| 当前三层依赖与限定目录调用方 | 已有安全 normal 节点的离线有界测试；根属性修改可能使祖先检查拒绝，需单独验证 | 第四批补祖先/混合状态组合 |
+| 分支复制、分支间比较、合并来源 | v0.66.0 已发布批准历史 mergeinfo 属性的来源路径/区间导航，保留非继承标记、有界不展开 revision、不读取分支源码；invalid/limited 明确且不裁剪。485 项串行回归、宿主、隔离安装、四矩阵 CI 与原包/npm/Release 129,866 字节、双哈希/清单/发布说明通过；有效继承 mergeinfo、跨 URL 比较、源码归属/合并提交覆盖仍未支持 | 第六批：URL/peg/operative revision 与 mergeinfo 导航披露 |
+| 当前三层依赖与限定目录调用方 | 本地补根目录仅 mergeinfo 变化下的自动依赖/限定调用方：根目录不作源码候选，祖先双侧属性仅 mergeinfo 差异可通过且入指纹；其他属性/unsafe 状态仍拒绝，离线集成通过，待 v0.66.1 发布。其他祖先/混合状态继续待验证 | 第四批补祖先/混合状态组合 |
 | 历史两侧上下文与调用方 | 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 字节、双哈希/清单/发布说明通过；自动三层历史链与历史调用方仍未实现 | 第六批：逐侧固定版本，不能回退至工作副本 |
 | 真正 r7314→r7315 样本 | 用户未提供隔离副本路径；本地类似复现不是该样本验收 | 获得路径后只读生成预览，不自动发送模型 |
 

package/package.json

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

package/src/caller-candidates.mjs

@@ -31,6 +31,7 @@ export function callerCandidates(files, scanned, resolutionPaths, {includeBindin
   const resolved = new Set(resolutionPaths);
   const changed = new Set(), targets = new Set();
   for (const file of files) {
+    if(file.path==='.'&&file.kind==='dir'){changed.add(file.path);continue;}
     relativePath(file.path); changed.add(file.path);
     if (file.oldPath) { relativePath(file.oldPath); changed.add(file.oldPath); }
     if (file.eligibility === 'reviewable' && file.rightExists !== false && !isSecretPath(file.path)) targets.add(file.path);

package/src/context-candidates.mjs

@@ -7,6 +7,7 @@ export function contextCandidates(files, trackedPaths, {maxCandidates=20,onProbe
  const changed=new Set(files.flatMap(f=>[f.path,...(f.oldPath?[f.oldPath]:[])]));
  const found=new Map();const add=(p,from,reason)=>{if(!tracked.has(p)||(!includeChanged&&changed.has(p)))return;if(!found.has(p))found.set(p,{path:p,fromPaths:[],reasons:[],confidence:'navigation-only'});const v=found.get(p);if(!v.fromPaths.includes(from))v.fromPaths.push(from);if(!v.reasons.includes(reason))v.reasons.push(reason);};
  for(const f of files){
+  if(f.path==='.'&&f.kind==='dir')continue; // Root properties have no source dependency syntax.
   relativePath(f.path);if(f.eligibility!=='reviewable'||f.rightExists===false)continue;
   const text=f.right?.text;if(typeof text!=='string'||Buffer.byteLength(text)>256*1024)continue;
   if(text.includes('/*')||text.includes(String.fromCharCode(96))||text.includes('R"')||/\\\r?\n/.test(text))continue;

package/src/svn-caller-safety.mjs

@@ -4,5 +4,15 @@ import {relativePath,hash} from './content.mjs';
 import {lstat,realpath} from 'node:fs/promises';
 import path from 'node:path';
 const canonicalValue=v=>Array.isArray(v)?v.map(canonicalValue):v&&typeof v==='object'?Object.fromEntries(Object.keys(v).sort().map(k=>[k,canonicalValue(v[k])])):v;
+function propertyMap(props){
+ const result=Object.create(null);
+ for(const p of props){if(typeof p['@_name']!=='string'||p['@_encoding']||Object.hasOwn(result,p['@_name']))throw new Error('Unsafe SVN caller ancestor property representation');result[p['@_name']]=p['#text']??'';}
+ return result;
+}
+export function mergeinfoOnlyAncestorProperties(oldProps,newProps){
+ const old=propertyMap(oldProps),current=propertyMap(newProps);
+ for(const name of ['svn:externals','svn:special'])if(Object.hasOwn(old,name)||Object.hasOwn(current,name))throw new Error('Unsafe SVN caller ancestor properties');
+ if([...new Set([...Object.keys(old),...Object.keys(current)])].some(name=>name!=='svn:mergeinfo'&&old[name]!==current[name]))throw new Error('Unsafe SVN caller ancestor property change');
+}
 // Offline metadata on each explicit ancestor. Never follows external/switched roots.
-export async function svnCallerAncestors(root,scope,options={}){relativePath(scope);if(scope.split('/').length>32)throw new Error('SVN caller ancestor depth limit exceeded');const canonical=await realpath(root),names=['.',...scope.split('/').map((_,i)=>scope.split('/').slice(0,i+1).join('/'))],identities=[];let bytes=0,rootInfo;for(const name of names){options.signal?.throwIfAborted();const local=name==='.'?canonical:path.join(canonical,name);const stat=await lstat(local);if(stat.isSymbolicLink()||!stat.isDirectory())throw new Error('SVN caller scope requires real directories');const run=async args=>{const b=await checked('svn',['--non-interactive',...args,'--',name+'@'],{...options,cwd:canonical,maxBytes:64*1024});bytes+=b.length;if(bytes>256*1024)throw new Error('SVN caller ancestor metadata limit exceeded');return xml(b);};const status=await run(['status','--xml','--verbose','--ignore-externals','--depth','empty']);const entries=(status.status?.target??[]).flatMap(t=>t.entry??[]);if(entries.length!==1||entries[0]['@_path'].replaceAll('\\','/')!==name)throw new Error('SVN caller ancestor status missing');const s=entries[0]['wc-status'];if(s?.['@_item']!=='normal'||!['none','normal'].includes(s['@_props'])||['@_switched','@_copied','@_file-external','@_tree-conflicted'].some(k=>s[k]==='true'))throw new Error('Unsafe SVN caller ancestor state');const info=(await run(['info','--xml'])).info?.entry?.[0];if(!info||info['@_kind']!=='dir'||info['wc-info']?.['copy-from-url'])throw new Error('Unsafe SVN caller ancestor identity');rootInfo??=info;if(info.repository?.uuid!==rootInfo.repository?.uuid||info['wc-info']?.['wcroot-abspath']!==rootInfo['wc-info']?.['wcroot-abspath'])throw new Error('SVN caller ancestor external working root');if(name!=='.'){const expected=rootInfo.url.replace(/\/$/,'')+'/'+name.split('/').map(encodeURIComponent).join('/');if(info.url!==expected)throw new Error('Unsafe SVN caller switched ancestor URL');}const properties=await run(['proplist','--xml','--verbose']);const props=properties.properties?.target?.[0]?.property??[];if(props.some(p=>['svn:externals','svn:special'].includes(p['@_name'])))throw new Error('Unsafe SVN caller ancestor properties');identities.push({path:name,status:entries[0],info,properties:props});}return {bytes,fingerprint:hash(JSON.stringify(canonicalValue(identities)))};}
+export async function svnCallerAncestors(root,scope,options={}){relativePath(scope);if(scope.split('/').length>32)throw new Error('SVN caller ancestor depth limit exceeded');const canonical=await realpath(root),names=['.',...scope.split('/').map((_,i)=>scope.split('/').slice(0,i+1).join('/'))],identities=[];let bytes=0,rootInfo;for(const name of names){options.signal?.throwIfAborted();const local=name==='.'?canonical:path.join(canonical,name);const stat=await lstat(local);if(stat.isSymbolicLink()||!stat.isDirectory())throw new Error('SVN caller scope requires real directories');const run=async args=>{const b=await checked('svn',['--non-interactive',...args,'--',name+'@'],{...options,cwd:canonical,maxBytes:64*1024});bytes+=b.length;if(bytes>256*1024)throw new Error('SVN caller ancestor metadata limit exceeded');return xml(b);};const status=await run(['status','--xml','--verbose','--ignore-externals','--depth','empty']);const entries=(status.status?.target??[]).flatMap(t=>t.entry??[]);if(entries.length!==1||entries[0]['@_path'].replaceAll('\\','/')!==name)throw new Error('SVN caller ancestor status missing');const s=entries[0]['wc-status'];if(s?.['@_item']!=='normal'||!['none','normal','modified'].includes(s['@_props'])||['@_switched','@_copied','@_file-external','@_tree-conflicted'].some(k=>s[k]==='true'))throw new Error('Unsafe SVN caller ancestor state');const info=(await run(['info','--xml'])).info?.entry?.[0];if(!info||info['@_kind']!=='dir'||info['wc-info']?.['copy-from-url'])throw new Error('Unsafe SVN caller ancestor identity');rootInfo??=info;if(info.repository?.uuid!==rootInfo.repository?.uuid||info['wc-info']?.['wcroot-abspath']!==rootInfo['wc-info']?.['wcroot-abspath'])throw new Error('SVN caller ancestor external working root');if(name!=='.'){const expected=rootInfo.url.replace(/\/$/,'')+'/'+name.split('/').map(encodeURIComponent).join('/');if(info.url!==expected)throw new Error('Unsafe SVN caller switched ancestor URL');}const properties=await run(['proplist','--xml','--verbose']);const props=properties.properties?.target?.[0]?.property??[];if(props.some(p=>['svn:externals','svn:special'].includes(p['@_name'])))throw new Error('Unsafe SVN caller ancestor properties');let baselineProps;if(s['@_props']==='modified'){baselineProps=(await run(['proplist','--xml','--verbose','-r','BASE'])).properties?.target?.[0]?.property??[];mergeinfoOnlyAncestorProperties(baselineProps,props);}identities.push({path:name,status:entries[0],info,properties:props,...(baselineProps?{baselineProperties:baselineProps}: {})});}return {bytes,fingerprint:hash(JSON.stringify(canonicalValue(identities)))};}