@feng0531/dsh-code-review 0.75.0 → 0.75.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.75.1 — 修复 SVN 根级上下文祖先校验遗漏
+
+根目录直属上下文不再跳过根目录属性检查，节点指纹绑定祖先身份。仅mergeinfo变化仍可用但使旧快照过期；不安全根属性明确拒绝。保持根目录mergeinfo预览和路径保护，只读离线，不新增支持模式。
+
 ## 0.75.0 — SVN 手动上下文离线来源校验
 
-未启用autoContext时，未变更显式上下文也校验SVN节点/属性/URL及祖先身份并末尾复核，保留EOL元数据。已选变更复用原快照，不读取排除或删除来源。不安全来源明确拒绝，不再仅凭本地文件位置读取。只读离线，显式来源共享20名额/1MiB/30秒预算；其他SVN缺口继续。
+未启用autoContext时，未变更显式上下文也校验SVN节点/属性/URL及祖先身份并末尾复核，保留EOL元数据。已选变更复用原快照，不读取排除或删除来源。不安全来源明确拒绝，不再仅凭本地文件位置读取。只读离线，显式来源共享20名额/1MiB/30秒预算；其他SVN缺口继续。512项全量串行回归、16项采集组合、实际宿主/隔离安装和四矩阵CI通过；原始Actions/npm/Release包145,650字节逐字节一致，双哈希、清单及冻结正文通过。
 
 ## 0.74.0 — SVN 本地依赖/调用方换行信息与正常属性修复
 

package/CORE-ALIGNMENT-TASKS.md

@@ -32,7 +32,8 @@
 
 ## 当前验证结论
 
-- 最近完整回归：511项通过；v0.74.0 本地导航安全专项11项通过。
+- 最近完整回归：512项通过；v0.75.0 采集组合16项通过。
+- v0.75.0 已发布独立显式SVN工作副本上下文节点/属性/URL身份核验与EOL元数据，末尾复核且离线受限，拒绝不安全来源；宿主/隔离安装与四矩阵CI通过，原始Actions/npm/Release包145,650字节逐字节一致，双哈希/清单/冻结说明通过。关键字、特殊状态与分支来源等缺口继续。
 - 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等仍未闭环。

package/DEVELOPMENT.md

@@ -27,6 +27,8 @@
 
 ## 正式发布
 
+每次改动完成并验证通过后都发布正式版本，不止于提交推送。版本格式为 `vX.Y.Z`：X是大版本（重大或不兼容升级），Y是小版本/新增功能，Z是BUG修复；纯修复递增Z，不一律递增Y。已发布标签不移动，发布不自动安装、升级或重启宿主。
+
 1. 修改版本号与锁文件；更新 [变更记录](CHANGELOG.md) 和 [发布说明](RELEASE-NOTES.md)。
 2. 完成测试、差异检查和独立安装验证后，提交、推送，并创建对应 vX.Y.Z 标签。标签一旦进入发布流程就不再移动。
 3. 等待标签 CI 的 Ubuntu/Windows × Node.js 22/24 四组测试全部成功。

package/RELEASE-NOTES.md

@@ -1,14 +1,14 @@
-# v0.75.0 发布说明
+# v0.75.1 发布说明
 
-## SVN 手动上下文的离线来源校验
+## 修复 SVN 根目录直属上下文跳过祖先检查
 
-工作副本审查中，单独使用 `contextPaths`（未启用自动依赖）加入未变更文件时，现在也会核对SVN节点、仓库URL、属性与祖先身份，并在采集末尾复核。正常EOL属性与原始正文一起保留，预览披露换行元数据。
+根目录直属文件此前没有执行根目录属性安全检查，其节点指纹也没有包含祖先属性。现在所有本地导航上下文都核验父目录（包括根目录），并将祖先指纹绑定到节点身份。
 
-- 不再仅凭文件位于本地目录就当作安全SVN上下文。未跟踪/排除变更、关键字和特殊节点等不可确认来源明确拒绝。
-- 已选入且可审查的变更文件继续复用原快照正文和工作侧EOL元数据，不恢复被排除或删除的文件。
-- 显式未变更来源共享20个名额、1MiB正文与30秒采集及身份复核预算；只读本地元数据，不连接远端。
-- 原文、原始字节哈希和行号保持，属性变化要求重新预览；不以换行映射证明行为等价。
+- 根目录仅 mergeinfo 变化仍可读取安全上下文，但变化会改变身份，旧预览不能复用。
+- 根目录其他未提交属性变化、externals/special或不安全状态明确拒绝，不作为正常来源继续读取。
+- 不更新、切换或修改业务工作副本；检查仍只读离线，并受原超时与元数据限制约束。
+- 根目录mergeinfo本身的默认/指定文件预览能力保持，路径穿越保护不放宽。
 
-如果以前把不受支持的节点当作手动上下文加入，本版可能明确报错，而不是继续读取。工作副本关键字上下文、未提交替换、有效继承mergeinfo和跨URL比较仍未完成。
+本版为BUG修复，采用补丁版本 v0.75.1，不新增SVN支持模式。关键字上下文、未提交替换、有效继承mergeinfo和跨URL等剩余项见 [支持矩阵](SVN-SUPPORT-MATRIX.md)。
 
-操作示例见 [README](README.md)，状态与证据见 [SVN支持矩阵](SVN-SUPPORT-MATRIX.md)。
+使用方式见 [README](README.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 样本 | 用户未提供隔离副本路径；本地类似复现不是该样本验收 | 获得路径后只读生成预览，不自动发送模型 |
 
-## 下一批：独立显式本地上下文（v0.75.0 待发布）
+## 下一批BUG修复：根级上下文祖先检查（v0.75.1 待发布）
+
+- 新真实离线复现证明：根目录直属文件在根mergeinfo变化后节点指纹不变，`svnCallerNode` 跳过了父目录为根的检查。修复前测试明确失败。
+- `svnCallerAncestors` 支持严格根目录scope，避免重复根节点；`svnCallerNode` 始终核验父目录并把祖先指纹纳入节点身份。普通相对路径限制不放宽。
+- `test/svn-root-context-safety.test.mjs` 修复后通过：根mergeinfo改变指纹、已捕获上下文verify拒绝、其他根属性与externals明确阻断；与优先根预览/mergeinfo祖先/显式上下文组合7项通过。513项串行全量回归、7项根级安全/优先预览组合与宿主/隔离安装通过，尚未发布。
+- 本批纯BUG修复使用v0.75.1，不当作新增能力；其余SVN缺口保留。
+
+## 独立显式本地上下文（v0.75.0 已发布）
+
+冻结提交 `d08cb354709a2fb2051a7769b853cb98ae1619cd`；标签CI `38706343061`，唯一发布运行 `38707025423`，原工件 `11958951726`。原始Actions/npm/Release包145,650字节逐字节一致；SHA-256 `7d251aa209a7430c8b72a018472cc6d5d2e1e24ea9282623f19b5fd58896917e`，SHA-512/清单/冻结发布正文核验通过。
 
 - `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项安全组合、实际宿主/隔离安装通过，尚未发布。
+- `test/svn-explicit-context.test.mjs` 真实离线复现：CRLF元数据、稳定身份、0预览发送、status不变、关键字拒绝、未跟踪排除、属性变化及取消拒绝；与既有SVN采集组合16项通过。512项串行全量回归、16项既有采集组合及最终3项安全组合、实际宿主/隔离安装、四矩阵标签及发布CI通过，正式发布工件核验完成。
 - 工作副本关键字、特殊状态、跨URL与有效继承mergeinfo仍未完成。
 
 ## 本地导航 EOL 与正常属性候选（v0.74.0 已发布）

package/package.json

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

package/src/svn-caller-node.mjs

@@ -6,6 +6,6 @@ import path from 'node:path';
 import {svnCallerAncestors} from './svn-caller-safety.mjs';
 const canonical=v=>Array.isArray(v)?v.map(canonical):v&&typeof v==='object'?Object.fromEntries(Object.keys(v).sort().map(k=>[k,canonical(v[k])])):v;
 // Body authorization requires regular local node and offline SVN identity/properties.
-export async function svnCallerNode(root,name,options={}){relativePath(name);const parent=path.posix.dirname(name);if(parent!=='.')await svnCallerAncestors(root,parent,options);const stat=await lstat(path.join(root,name));if(stat.isSymbolicLink()||!stat.isFile())throw new Error('SVN caller source is not a regular file');const run=async args=>xml(await checked('svn',['--non-interactive',...args,'--',name+'@'],{...options,cwd:root,maxBytes:64*1024}));const entries=(await run(['status','--xml','--verbose','--ignore-externals','--depth','empty'])).status?.target?.flatMap(t=>t.entry??[])??[];if(entries.length!==1||entries[0]['@_path'].replaceAll('\\','/')!==name)throw new Error('SVN caller source 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 source state');const info=(await run(['info','--xml'])).info?.entry?.[0];const rootInfo=xml(await checked('svn',['--non-interactive','info','--xml','--','.@'],{...options,cwd:root,maxBytes:64*1024})).info?.entry?.[0];if(info?.['@_kind']!=='file'||info['wc-info']?.['copy-from-url']||info.repository?.uuid!==rootInfo.repository?.uuid||info['wc-info']?.['wcroot-abspath']!==rootInfo['wc-info']?.['wcroot-abspath'])throw new Error('Unsafe SVN caller source identity');const expected=rootInfo.url.replace(/\/$/,'')+'/'+name.split('/').map(encodeURIComponent).join('/');if(info.url!==expected)throw new Error('Unsafe SVN caller switched source URL');const props=(await run(['proplist','--xml','--verbose'])).properties?.target?.[0]?.property??[];if(props.some(p=>['svn:special','svn:keywords','svn:externals'].includes(p['@_name'])))throw new Error('Unsafe SVN caller source properties');const eol=props.filter(p=>p['@_name']==='svn:eol-style');if(eol.length>1||eol.some(p=>p['@_encoding']||!['native','LF','CRLF','CR'].includes(p['#text'])))throw new Error('Unsafe SVN caller EOL properties');return {...(eol.length?{eolStyle:eol[0]['#text']}:{}),revision:s['@_revision'],fingerprint:hash(JSON.stringify(canonical({status:entries[0],info,props})))};}
+export async function svnCallerNode(root,name,options={}){relativePath(name);const parent=path.posix.dirname(name);const ancestors=await svnCallerAncestors(root,parent,options);const stat=await lstat(path.join(root,name));if(stat.isSymbolicLink()||!stat.isFile())throw new Error('SVN caller source is not a regular file');const run=async args=>xml(await checked('svn',['--non-interactive',...args,'--',name+'@'],{...options,cwd:root,maxBytes:64*1024}));const entries=(await run(['status','--xml','--verbose','--ignore-externals','--depth','empty'])).status?.target?.flatMap(t=>t.entry??[])??[];if(entries.length!==1||entries[0]['@_path'].replaceAll('\\','/')!==name)throw new Error('SVN caller source 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 source state');const info=(await run(['info','--xml'])).info?.entry?.[0];const rootInfo=xml(await checked('svn',['--non-interactive','info','--xml','--','.@'],{...options,cwd:root,maxBytes:64*1024})).info?.entry?.[0];if(info?.['@_kind']!=='file'||info['wc-info']?.['copy-from-url']||info.repository?.uuid!==rootInfo.repository?.uuid||info['wc-info']?.['wcroot-abspath']!==rootInfo['wc-info']?.['wcroot-abspath'])throw new Error('Unsafe SVN caller source identity');const expected=rootInfo.url.replace(/\/$/,'')+'/'+name.split('/').map(encodeURIComponent).join('/');if(info.url!==expected)throw new Error('Unsafe SVN caller switched source URL');const props=(await run(['proplist','--xml','--verbose'])).properties?.target?.[0]?.property??[];if(props.some(p=>['svn:special','svn:keywords','svn:externals'].includes(p['@_name'])))throw new Error('Unsafe SVN caller source properties');const eol=props.filter(p=>p['@_name']==='svn:eol-style');if(eol.length>1||eol.some(p=>p['@_encoding']||!['native','LF','CRLF','CR'].includes(p['#text'])))throw new Error('Unsafe SVN caller EOL properties');return {...(eol.length?{eolStyle:eol[0]['#text']}:{}),revision:s['@_revision'],fingerprint:hash(JSON.stringify(canonical({status:entries[0],info,props,ancestorFingerprint:ancestors.fingerprint})))};}
 // Existing untracked/linked/directory paths compete too; absence grants no body read.
 export async function svnCallerProbes(root,names,options={}){if(!Array.isArray(names)||names.length>512||new Set(names).size!==names.length)throw new Error('Invalid SVN caller probes');const existing=[],identities=[];for(const name of [...names].sort()){relativePath(name);if(name.split('/').length>32)throw new Error('SVN caller probe depth limit exceeded');options.signal?.throwIfAborted();let cursor=root,identity=[];for(const part of name.split('/')){cursor=path.join(cursor,part);try{const s=await lstat(cursor);identity.push({part,kind:s.isSymbolicLink()?'link':s.isDirectory()?'dir':s.isFile()?'file':'other'});if(s.isSymbolicLink()){existing.push(name);break;}if(cursor===path.join(root,name))existing.push(name);if(!s.isDirectory())break;}catch(e){if(e.code!=='ENOENT'&&e.code!=='ENOTDIR')throw e;identity.push({part,kind:'missing'});break;}}identities.push({name,identity});if(Buffer.byteLength(JSON.stringify(identities))>1024*1024)throw new Error('SVN caller probe metadata limit exceeded');}return {paths:existing,fingerprint:hash(JSON.stringify(identities))};}

package/src/svn-caller-safety.mjs

@@ -15,4 +15,4 @@ export function mergeinfoOnlyAncestorProperties(oldProps,newProps){
  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','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)))};}
+export async function svnCallerAncestors(root,scope,options={}){if(scope!=='.')relativePath(scope);if(scope.split('/').length>32)throw new Error('SVN caller ancestor depth limit exceeded');const canonical=await realpath(root),names=scope==='.'?['.']:['.',...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)))};}