@feng0531/dsh-code-review 0.76.0 → 0.76.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +5 -1
- package/CORE-ALIGNMENT-TASKS.md +2 -1
- package/RELEASE-NOTES.md +9 -9
- package/SVN-SUPPORT-MATRIX.md +13 -2
- package/package.json +1 -1
- package/src/service.mjs +1 -1
- package/src/svn-working-identity.mjs +9 -0
- package/src/svn.mjs +9 -2
package/CHANGELOG.md
CHANGED
|
@@ -2,9 +2,13 @@
|
|
|
2
2
|
|
|
3
3
|
本文件介绍对使用者有意义的变化。安装与参数见 [README](README.md),本次发布正文见 [发布说明](RELEASE-NOTES.md)。历史提交和安装包保留在 [GitHub Releases](https://github.com/f-e-n-g-0531/dsh-code-review/releases);本文不把开发日志当作功能介绍。
|
|
4
4
|
|
|
5
|
+
## 0.76.2 — 修复 SVN 普通变更来源身份遗漏
|
|
6
|
+
|
|
7
|
+
普通节点和根URL进入快照并末尾复核,切换祖先下的子文件明确阻断,正常预览后切换目录会拒绝执行;不以相同正文/revision代替来源身份。复制语义及中文转义兼容保持。共享来源预算的独立审查发现仍待修复。
|
|
8
|
+
|
|
5
9
|
## 0.76.0 — 独立显式 SVN 标准关键字上下文
|
|
6
10
|
|
|
7
|
-
未启用autoContext时可显式选择标准关键字正常文件;原始展开正文/hash/行号保持,BASE收缩与未展开正文独立核验,全部核验正文计入原显式来源预算。预览只披露映射元数据,自定义/固定宽度和不完整展开拒绝,自动依赖和调用方关键字仍未开放。
|
|
11
|
+
未启用autoContext时可显式选择标准关键字正常文件;原始展开正文/hash/行号保持,BASE收缩与未展开正文独立核验,全部核验正文计入原显式来源预算。预览只披露映射元数据,自定义/固定宽度和不完整展开拒绝,自动依赖和调用方关键字仍未开放。514项串行回归、9项显式/依赖专项、宿主/隔离安装和四矩阵CI通过;原始Actions/npm/Release包147,614字节逐字节一致,双哈希/清单/冻结说明通过。
|
|
8
12
|
|
|
9
13
|
## 0.75.1 — 修复 SVN 根级上下文祖先校验遗漏
|
|
10
14
|
|
package/CORE-ALIGNMENT-TASKS.md
CHANGED
|
@@ -32,7 +32,8 @@
|
|
|
32
32
|
|
|
33
33
|
## 当前验证结论
|
|
34
34
|
|
|
35
|
-
- 最近完整回归:
|
|
35
|
+
- 最近完整回归:514项通过;v0.76.0 显式/依赖组合9项通过。
|
|
36
|
+
- v0.76.0 已发布独立显式标准关键字上下文,BASE展开/未展开核验计入原来源预算,原始展开正文为证据,预览元数据披露;宿主/隔离安装及四矩阵CI通过,原始Actions/npm/Release包147,614字节逐字节一致,双哈希/清单/冻结说明通过。自动依赖/调用方关键字与全面SVN其余缺口继续。
|
|
36
37
|
- v0.75.1 已发布根级上下文祖先检查BUG修复,所有节点指纹绑定祖先属性,mergeinfo变化使旧身份过期,不安全根属性拒绝;宿主/隔离安装和四矩阵CI通过,原始Actions/npm/Release包146,408字节逐字节一致,双哈希/清单/冻结正文通过。纯BUG递增Z;全面SVN缺口继续。
|
|
37
38
|
- v0.75.0 已发布独立显式SVN工作副本上下文节点/属性/URL身份核验与EOL元数据,末尾复核且离线受限,拒绝不安全来源;宿主/隔离安装与四矩阵CI通过,原始Actions/npm/Release包145,650字节逐字节一致,双哈希/清单/冻结说明通过。关键字、特殊状态与分支来源等缺口继续。
|
|
38
39
|
- v0.74.0 已发布离线自动依赖/显式导航入口/命中调用方EOL元数据与normal属性候选漏选修复,关键字/special/非正常状态仍检查;宿主/隔离安装与四矩阵CI通过,原始Actions/npm/Release包144,494字节逐字节一致,双哈希/清单/冻结正文通过。独立无autoContext显式本地属性映射、关键字与其余SVN缺口继续。
|
package/RELEASE-NOTES.md
CHANGED
|
@@ -1,14 +1,14 @@
|
|
|
1
|
-
# v0.76.
|
|
1
|
+
# v0.76.2 发布说明
|
|
2
2
|
|
|
3
|
-
## SVN
|
|
3
|
+
## 修复 SVN 普通变更未冻结节点来源的问题
|
|
4
4
|
|
|
5
|
-
|
|
5
|
+
普通工作副本变更现在也记录并复核节点URL、仓库UUID/根、revision、类型及调度身份。预览披露来源,根URL进入快照,避免只核验正文和属性却遗漏来源变化。
|
|
6
6
|
|
|
7
|
-
-
|
|
8
|
-
-
|
|
9
|
-
-
|
|
10
|
-
-
|
|
7
|
+
- 切换目录下的修改子文件即使自身没有switched标志,也会因实际URL偏离所选仓库路径而以 SVN_SWITCHED_ANCESTOR 明确阻断。
|
|
8
|
+
- 普通节点在采集末尾重新核验,根节点身份也复核;复制来源仍保持独立信息与目标新增语义。
|
|
9
|
+
- 不新增网络访问,不修改工作副本;中文、空格与合法URL转义的等价表示仍兼容。
|
|
10
|
+
- 正常预览后切换目录,执行会拒绝旧预览且不发送模型。
|
|
11
11
|
|
|
12
|
-
|
|
12
|
+
本版是BUG补丁,不代表新增switched节点审查支持。另一个已复现的本地组合共享来源预算问题尚未修复,继续保留在 [支持矩阵](SVN-SUPPORT-MATRIX.md) 中。
|
|
13
13
|
|
|
14
|
-
|
|
14
|
+
使用见 [README](README.md)。
|
package/SVN-SUPPORT-MATRIX.md
CHANGED
|
@@ -31,11 +31,22 @@
|
|
|
31
31
|
| 历史两侧上下文与调用方 | 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 字节、双哈希/清单/发布说明通过;字面发现仍不证明完整调用覆盖 | 第六批:逐侧固定版本,不能回退至工作副本 |
|
|
32
32
|
| 真正 r7314→r7315 样本 | 用户未提供隔离副本路径;本地类似复现不是该样本验收 | 获得路径后只读生成预览,不自动发送模型 |
|
|
33
33
|
|
|
34
|
-
##
|
|
34
|
+
## 审查发现修复:普通变更来源身份(v0.76.2 待发布)
|
|
35
|
+
|
|
36
|
+
v0.76.1 标签CI `38718993310` 的Windows两组因新测试比较 `~` 与 `%7E` 原始URL字符串失败,Ubuntu两组通过;未触发正式发布,标签保留不移动。修正测试按等价URL比较后使用新补丁v0.76.2重新验收。
|
|
37
|
+
|
|
38
|
+
- 已复现切换祖先的修改子文件误收及不同分支同正文/revision快照碰撞。修复为普通节点冻结URL/UUID/仓库根/revision/kind/schedule,末尾及根身份复核,预览披露workingSource。
|
|
39
|
+
- `test/svn-working-identity.test.mjs` 真实仓库证明正常来源绑定、切换子节点SVN_SWITCHED_ANCESTOR阻断、无正文授权、旧预览execute拒绝且0模型发送;与普通采集组合16项通过。中文/空格/URL转义兼容修正,515项串行全量回归、16项采集专项、宿主/隔离安装通过,尚未发布。
|
|
40
|
+
- switched状态并未因此可审查;阻断项可同ID但无正文授权,不把其视为已审核成功。
|
|
41
|
+
- 独立待修BUG:本地contextPaths+callerScopePaths来源分别计费,实测1,140,102字节超过共享1MiB仍成功;本批不声称已解决。
|
|
42
|
+
|
|
43
|
+
## 独立显式标准关键字上下文(v0.76.0 已发布)
|
|
44
|
+
|
|
45
|
+
冻结提交 `ad439af93fd48a805cd85206e2138851078f05cb`;标签CI `38714424406`,唯一发布运行 `38715124806`,原工件 `11962002956`。原始Actions/npm/Release包147,614字节逐字节一致;SHA-256 `918c3b115329986f257917a2278152b0f75968f5cb6d1fa62d112d91ed046202`,SHA-512/清单/冻结正文核验通过。
|
|
35
46
|
|
|
36
47
|
- 仅 `captureSvnExplicitContext` 显式开启节点标准关键字能力,默认调用方/依赖安全检查仍拒绝keywords;正常状态、URL/UUID/祖先指纹核验保留。
|
|
37
48
|
- 原始工作正文保留;BASE展开收缩与cat --ignore-keywords核验,两个核验正文计入同一1MiB来源/30秒预算。预览contextKeywordMappings不含正文,模型使用原展开原行号。
|
|
38
|
-
- `test/svn-explicit-keywords.test.mjs` 离线真实复现通过:原文/映射、BASE来源计费、预览0发送、模型原文、status不变和稳定快照;自定义/固定宽度拒绝,默认节点与autoContext组合继续拒绝。与显式/依赖组合9项通过,514项串行全量回归、9
|
|
49
|
+
- `test/svn-explicit-keywords.test.mjs` 离线真实复现通过:原文/映射、BASE来源计费、预览0发送、模型原文、status不变和稳定快照;自定义/固定宽度拒绝,默认节点与autoContext组合继续拒绝。与显式/依赖组合9项通过,514项串行全量回归、9项显式/依赖专项、宿主/隔离安装、四矩阵标签和发布CI通过,正式发布工件核验完成。
|
|
39
50
|
- 自动关键字依赖/调用方、特殊状态和分支来源仍未闭环。
|
|
40
51
|
|
|
41
52
|
## BUG修复:根级上下文祖先检查(v0.75.1 已发布)
|
package/package.json
CHANGED
package/src/service.mjs
CHANGED
|
@@ -49,7 +49,7 @@ export function createReviewService(llm, { capture = captureSnapshot, now = Date
|
|
|
49
49
|
while (previews.size >= maxPreviews) previews.delete(previews.keys().next().value);
|
|
50
50
|
const previewId = randomUUID();
|
|
51
51
|
previews.set(previewId, { ...current, target, options, rounds, snapshotId: snapshot.id, expires: now() + ttlMs });
|
|
52
|
-
return { ...(options.businessRequirement!==undefined?{businessRequirement:{text:options.businessRequirement,hash:hash(options.businessRequirement),bytes:Buffer.byteLength(options.businessRequirement)}}:{}), previewId, plan, snapshotId: snapshot.id, repositoryRoot: snapshot.root, vcs: snapshot.vcs, ...(snapshot.history ? { history: snapshot.history } : {}), ...(snapshot.mergeSources ? {mergeSources:snapshot.mergeSources} : {}), model: current.route, modelSendingEnabled: allowModelSending, files: snapshot.files.map(f => ({ path: f.path, eligibility: f.eligibility, reason: f.reason, ...(f.eolMapping?{eolMapping:f.eolMapping,textNotice:f.textNotice}:{}), ...(f.keywordMapping?{keywordMapping:f.keywordMapping,textNotice:f.textNotice}:{}), ...(f.rawStatus==='replaced'&&snapshot.history?.vcs==='svn'?{oldKind:f.oldKind,newKind:f.newKind,nodeNotice:f.nodeNotice}:{}), ...(f.blockCode ? {blockCode:f.blockCode} : {}), ...(f.copySource ? {copySource:f.copySource,copyNotice:f.copyNotice} : {}) })), contextPaths: snapshot.context.map(c => c.path), ...(snapshot.context.some(c=>c.keywordMapping)?{contextKeywordMappings:snapshot.context.filter(c=>c.keywordMapping).map(c=>({path:c.path,keywordMapping:c.keywordMapping}))}:{}), ...(snapshot.context.some(c=>c.eolMapping)?{contextEolMappings:snapshot.context.filter(c=>c.eolMapping).map(c=>({path:c.path,eolMapping:c.eolMapping}))}:{}), ...(snapshot.history?.vcs==='svn'?{contextSources:snapshot.context.map(c=>({path:c.path,...(c.svnSource?{new:c.svnSource}:{}),...(c.oldSvnSource?{old:c.oldSvnSource}:{})}))}:{}), ...(options.oldContextPaths !== undefined ? {oldContextPaths:options.oldContextPaths,oldContextSources:snapshot.context.filter(c=>c.oldOnly).map(c=>({path:c.path,side:'context-old',revision:c.oldRevision,...(c.oldSvnSource?{svnSource:c.oldSvnSource}:{blobOid:c.oldBlobOid}),hash:c.oldHash}))} : {}), ...(snapshot.callerDiscovery ? {callerDiscovery:snapshot.callerDiscovery} : {}), ...(snapshot.autoContext ? { autoContext: snapshot.autoContext } : {}), rules: (snapshot.rules ?? []).map(({ path, hash }) => ({ path, hash })), notice: (snapshot.history?.vcs==='svn'&&options.autoContext===true?'SVN历史autoContext为精确依赖定位读取固定版本父目录有界清单,包含同目录非匹配名字元数据;仅匹配捕获正文发送模型,不扫描全仓,不用工作树替代历史版本。':'') + (options.oldContextPaths !== undefined ? 'oldContextPaths仅为baseline旧侧正文,无目标侧替代;选中变更复用原old身份,旧侧不能证明新侧行为。' : '') + (snapshot.callerDiscovery ? 'callerScopePaths已授权预览读取限定目录内tracked候选正文,包括不匹配文件;非匹配正文不发送模型,匹配捕获上下文会发送。仅字面引用导航,不是函数调用或完整调用覆盖。' : '') + (enableRiskPlanning ? '执行按plan阈值决定风险假设规划,小变更可跳过独立规划但仍主审并复核;计划不是证据,所有阶段共享原预算。' : '') + '执行会向上述模型提供方发送可审查文件两侧内容、预览列出的显式/自动上下文、所选项目规则全文及预览业务需求文本。模型可在该快照范围内多轮只读检索,不读取范围外文件;候选生成后会进行证据与反证复核,两阶段各最多3轮检索,共享每文件120秒和总模型调用100次预算;复核不能证明缺陷成立。plan.reviewRounds为独立审查工作量,非模型reasoning effort;后续轮不带首轮风险计划,所有轮共享原超时与预算。plan仅预检初始输入,ready不代表已审查;minimumCalls不含后续检索/复核/动态业务综合;多文件可先进行业务分组,分组失败保留原组并记录限制,动态综合数执行前未知,实际预算可能不足。请先向用户展示范围,获得确认后执行。' };
|
|
52
|
+
return { ...(options.businessRequirement!==undefined?{businessRequirement:{text:options.businessRequirement,hash:hash(options.businessRequirement),bytes:Buffer.byteLength(options.businessRequirement)}}:{}), previewId, plan, snapshotId: snapshot.id, repositoryRoot: snapshot.root, vcs: snapshot.vcs, ...(snapshot.history ? { history: snapshot.history } : {}), ...(snapshot.mergeSources ? {mergeSources:snapshot.mergeSources} : {}), model: current.route, modelSendingEnabled: allowModelSending, files: snapshot.files.map(f => ({ path: f.path,...(f.workingSource?{workingSource:f.workingSource}:{}), eligibility: f.eligibility, reason: f.reason, ...(f.eolMapping?{eolMapping:f.eolMapping,textNotice:f.textNotice}:{}), ...(f.keywordMapping?{keywordMapping:f.keywordMapping,textNotice:f.textNotice}:{}), ...(f.rawStatus==='replaced'&&snapshot.history?.vcs==='svn'?{oldKind:f.oldKind,newKind:f.newKind,nodeNotice:f.nodeNotice}:{}), ...(f.blockCode ? {blockCode:f.blockCode} : {}), ...(f.copySource ? {copySource:f.copySource,copyNotice:f.copyNotice} : {}) })), contextPaths: snapshot.context.map(c => c.path), ...(snapshot.context.some(c=>c.keywordMapping)?{contextKeywordMappings:snapshot.context.filter(c=>c.keywordMapping).map(c=>({path:c.path,keywordMapping:c.keywordMapping}))}:{}), ...(snapshot.context.some(c=>c.eolMapping)?{contextEolMappings:snapshot.context.filter(c=>c.eolMapping).map(c=>({path:c.path,eolMapping:c.eolMapping}))}:{}), ...(snapshot.history?.vcs==='svn'?{contextSources:snapshot.context.map(c=>({path:c.path,...(c.svnSource?{new:c.svnSource}:{}),...(c.oldSvnSource?{old:c.oldSvnSource}:{})}))}:{}), ...(options.oldContextPaths !== undefined ? {oldContextPaths:options.oldContextPaths,oldContextSources:snapshot.context.filter(c=>c.oldOnly).map(c=>({path:c.path,side:'context-old',revision:c.oldRevision,...(c.oldSvnSource?{svnSource:c.oldSvnSource}:{blobOid:c.oldBlobOid}),hash:c.oldHash}))} : {}), ...(snapshot.callerDiscovery ? {callerDiscovery:snapshot.callerDiscovery} : {}), ...(snapshot.autoContext ? { autoContext: snapshot.autoContext } : {}), rules: (snapshot.rules ?? []).map(({ path, hash }) => ({ path, hash })), notice: (snapshot.history?.vcs==='svn'&&options.autoContext===true?'SVN历史autoContext为精确依赖定位读取固定版本父目录有界清单,包含同目录非匹配名字元数据;仅匹配捕获正文发送模型,不扫描全仓,不用工作树替代历史版本。':'') + (options.oldContextPaths !== undefined ? 'oldContextPaths仅为baseline旧侧正文,无目标侧替代;选中变更复用原old身份,旧侧不能证明新侧行为。' : '') + (snapshot.callerDiscovery ? 'callerScopePaths已授权预览读取限定目录内tracked候选正文,包括不匹配文件;非匹配正文不发送模型,匹配捕获上下文会发送。仅字面引用导航,不是函数调用或完整调用覆盖。' : '') + (enableRiskPlanning ? '执行按plan阈值决定风险假设规划,小变更可跳过独立规划但仍主审并复核;计划不是证据,所有阶段共享原预算。' : '') + '执行会向上述模型提供方发送可审查文件两侧内容、预览列出的显式/自动上下文、所选项目规则全文及预览业务需求文本。模型可在该快照范围内多轮只读检索,不读取范围外文件;候选生成后会进行证据与反证复核,两阶段各最多3轮检索,共享每文件120秒和总模型调用100次预算;复核不能证明缺陷成立。plan.reviewRounds为独立审查工作量,非模型reasoning effort;后续轮不带首轮风险计划,所有轮共享原超时与预算。plan仅预检初始输入,ready不代表已审查;minimumCalls不含后续检索/复核/动态业务综合;多文件可先进行业务分组,分组失败保留原组并记录限制,动态综合数执行前未知,实际预算可能不足。请先向用户展示范围,获得确认后执行。' };
|
|
53
53
|
} finally { busy = false; }
|
|
54
54
|
},
|
|
55
55
|
async execute(args, exec) {
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
import {hash} from './content.mjs';
|
|
2
|
+
export function svnWorkingIdentity(node,rootInfo,name){
|
|
3
|
+
const expected=name==='.'?rootInfo.url:rootInfo.url.replace(/\/$/,'')+'/'+name.split('/').map(encodeURIComponent).join('/');
|
|
4
|
+
const sameUrl=(a,b)=>{try{const x=new URL(a),y=new URL(b);return x.protocol===y.protocol&&x.host===y.host&&x.username===y.username&&x.password===y.password&&decodeURIComponent(x.pathname)===decodeURIComponent(y.pathname)&&x.search===y.search&&x.hash===y.hash;}catch{return false;}};
|
|
5
|
+
if(!sameUrl(node?.url,expected))throw Object.assign(new Error('SVN node URL differs from selected working-copy path; switched ancestors are unsupported'),{code:'SVN_SWITCHED_ANCESTOR'});
|
|
6
|
+
if(!rootInfo.repository?.uuid||node.repository?.uuid!==rootInfo.repository.uuid||node.repository?.root!==rootInfo.repository.root||node['wc-info']?.['wcroot-abspath']!==rootInfo['wc-info']?.['wcroot-abspath'])throw Object.assign(new Error('SVN node repository or working-copy identity mismatch'),{code:'SVN_NODE_IDENTITY'});
|
|
7
|
+
const source={url:node.url,repositoryUuid:node.repository.uuid,repositoryRoot:node.repository.root,revision:String(node['@_revision']),kind:node['@_kind'],schedule:node['wc-info']?.schedule,copyFromUrl:node['wc-info']?.['copy-from-url']??null,copyFromRevision:node['wc-info']?.['copy-from-rev']??null};
|
|
8
|
+
return {source,fingerprint:hash(JSON.stringify(source))};
|
|
9
|
+
}
|
package/src/svn.mjs
CHANGED
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import {svnWorkingIdentity} from './svn-working-identity.mjs';
|
|
1
2
|
import { isSecretPath } from './secret-path.mjs';
|
|
2
3
|
import { svnStateIssue } from './svn-state.mjs';
|
|
3
4
|
import { snapshotMergeSources } from './svn-mergeinfo.mjs';
|
|
@@ -59,6 +60,7 @@ export async function captureSvn(cwd, options = {}) {
|
|
|
59
60
|
if (selectedPaths) for (const name of selectedPaths) if (name !== '.') relativePath(name);
|
|
60
61
|
const info = xml(await svn(cwd, ['info', '--xml', '--', '.@'], options)).info.entry[0];
|
|
61
62
|
const root = await realpath(info['wc-info']['wcroot-abspath']);
|
|
63
|
+
const rootInfo=xml(await svn(root,['info','--xml','--','.@'],options)).info.entry[0];
|
|
62
64
|
const statusArgs = ['status', '--xml', '--ignore-externals', '--', '.@'];
|
|
63
65
|
const before = await svn(root, statusArgs, options);
|
|
64
66
|
const originalEntries = xml(before).status.target.flatMap(t => t.entry ?? []);
|
|
@@ -87,6 +89,8 @@ export async function captureSvn(cwd, options = {}) {
|
|
|
87
89
|
if (status === 'unversioned') kind = (await lstat(path.join(root, name))).isDirectory() ? 'dir' : 'file';
|
|
88
90
|
else {
|
|
89
91
|
const node = xml(await svn(root, ['info', '--xml', '--', target(name)], options)).info.entry[0];
|
|
92
|
+
const identity=svnWorkingIdentity(node,rootInfo,name);
|
|
93
|
+
item.workingSource=identity.source;item.workingIdentity=identity.fingerprint;
|
|
90
94
|
kind = node['@_kind'];
|
|
91
95
|
if (state['@_copied'] === 'true' || node['wc-info']?.['copy-from-url']) {
|
|
92
96
|
let source = node['wc-info']?.['copy-from-url'], revision = node['wc-info']?.['copy-from-rev'];
|
|
@@ -145,7 +149,7 @@ export async function captureSvn(cwd, options = {}) {
|
|
|
145
149
|
}
|
|
146
150
|
}
|
|
147
151
|
} catch (error) {
|
|
148
|
-
signal?.throwIfAborted(); item.eligibility = 'blocked'; item.reason = error.message;
|
|
152
|
+
signal?.throwIfAborted(); item.eligibility = 'blocked'; item.reason = error.message;if(error.code)item.blockCode=error.code;
|
|
149
153
|
delete item.left; delete item.right;
|
|
150
154
|
}
|
|
151
155
|
}
|
|
@@ -160,6 +164,7 @@ export async function captureSvn(cwd, options = {}) {
|
|
|
160
164
|
}
|
|
161
165
|
}
|
|
162
166
|
for (const item of files.filter(f => f.eligibility === 'reviewable')) {
|
|
167
|
+
if(item.workingIdentity){const node=xml(await svn(root,['info','--xml','--',target(item.path)],options)).info.entry[0];if(svnWorkingIdentity(node,rootInfo,item.path).fingerprint!==item.workingIdentity)throw new Error('SVN node identity changed during capture');}
|
|
163
168
|
if (item.copySource) {
|
|
164
169
|
const node = xml(await svn(root, ['info', '--xml', '--', target(item.path)], options)).info.entry[0];
|
|
165
170
|
const identity = hash(JSON.stringify([node.url, node['@_kind'], node['@_revision'], node['wc-info']?.schedule, node['wc-info']?.['wcroot-abspath'], node['wc-info']?.['copy-from-url'] ?? null, node['wc-info']?.['copy-from-rev'] ?? null, node.repository]));
|
|
@@ -170,7 +175,9 @@ export async function captureSvn(cwd, options = {}) {
|
|
|
170
175
|
const right = ['deleted', 'unversioned'].includes(item.rawStatus) ? {} : await properties(root, item.path, false, options);
|
|
171
176
|
if (hash(JSON.stringify([left, right])) !== item.propertyIdentity) throw new Error('Properties changed during capture');
|
|
172
177
|
}
|
|
178
|
+
const finalRootInfo=xml(await svn(root,['info','--xml','--','.@'],options)).info.entry[0];
|
|
179
|
+
if(svnWorkingIdentity(finalRootInfo,rootInfo,'.').fingerprint!==svnWorkingIdentity(rootInfo,rootInfo,'.').fingerprint)throw new Error('SVN root identity changed during capture');
|
|
173
180
|
const mergeSources = snapshotMergeSources(files);
|
|
174
|
-
const snapshot = { schemaVersion: 1, vcs: 'svn', root, repositoryUuid:
|
|
181
|
+
const snapshot = { schemaVersion: 1, vcs: 'svn', root, repositoryUuid: rootInfo.repository.uuid, repositoryUrl:rootInfo.url, ...(mergeSources.length ? {mergeSources} : {}), files };
|
|
175
182
|
return { ...snapshot, id: hash(JSON.stringify(snapshot)) };
|
|
176
183
|
}
|