@feng0531/dsh-code-review 0.81.0 → 0.81.1
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 +4 -0
- package/DEVELOPMENT.md +7 -7
- package/README.md +2 -2
- package/RELEASE-NOTES.md +11 -11
- package/package.json +1 -1
- package/src/caller-capture.mjs +2 -2
- package/src/capture-signal.mjs +2 -0
- package/src/dependency-chain.mjs +2 -1
- package/src/history-caller-capture.mjs +2 -1
- package/src/history-dependency-chain.mjs +2 -1
- package/src/process.mjs +3 -3
- package/src/project-review-service.mjs +6 -6
- package/src/review.mjs +4 -3
- package/src/service.mjs +1 -1
- package/src/svn-caller-capture.mjs +2 -2
- package/src/svn-capture-budget.mjs +3 -2
- package/src/svn-dependency-chain.mjs +2 -1
- package/src/svn-explicit-context.mjs +2 -1
- package/src/svn-history.mjs +3 -2
- package/src/svn-navigation-budget.mjs +2 -1
- package/src/svn-working-mergeinfo.mjs +3 -2
package/CHANGELOG.md
CHANGED
|
@@ -2,6 +2,10 @@
|
|
|
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.81.1 — 取消默认内部超时和计数配额
|
|
6
|
+
|
|
7
|
+
修复外层延长仍被采集/导航30秒、模型120秒等内部截止终止的问题。默认不限制采集命令数或总模型调用数,遵循用户/宿主取消。保留来源、内存与模型输入保护,不声称解除宿主/提供方限制。
|
|
8
|
+
|
|
5
9
|
## 0.81.0 — 多会话独立审查与全项目当前源码
|
|
6
10
|
|
|
7
11
|
不同会话不再被全局busy阻塞,预览/锁/取消/模型/预算独立。新增reviewMode=project,跟踪当前源码分批检查并保持准确覆盖;不是差异审查,不把空旧侧当历史基线。项目模式有明确总量和共享调用限制,未完成不冒充成功。
|
package/DEVELOPMENT.md
CHANGED
|
@@ -16,19 +16,19 @@
|
|
|
16
16
|
|
|
17
17
|
服务按DSH session对象隔离缓存和锁,同一会话changes/project两种模式共用互斥锁;不同会话可并行,模型调用与取消由各请求生命周期约束。宿主dispose终止全部会话服务。全项目是跟踪文件的当前源码分批,不是多仓库汇总或跨批次语义分析。
|
|
18
18
|
|
|
19
|
-
项目捕获最多10000清单节点、1000选择节点、16MiB
|
|
19
|
+
项目捕获最多10000清单节点、1000选择节点、16MiB,不设默认总模型调用次数或执行秒表,初次捕获/复核均遵循宿主与用户取消。SVN项目模式拒绝不支持节点属性,不隐式继承固定历史审查的网络权限。
|
|
20
20
|
|
|
21
21
|
## 审查预算
|
|
22
22
|
|
|
23
23
|
除 README 中的文件和输入上限外:
|
|
24
24
|
|
|
25
|
-
-
|
|
25
|
+
- 采集、导航、命令及模型审查默认不设插件内部秒表;内部API调用者仍可显式指定timeoutMs(0表示无内部截止)。用户/宿主取消仍传播并等待清理。
|
|
26
26
|
- 每阶段最多三轮检索;全局最多 50 次读取请求,返回内容累计 256 KiB,单次最多 64 KiB,读取最多 200 行。
|
|
27
|
-
- Git 工作区依赖链最多三层、512 个精确路径查询、20 个候选、手动入口与自动依赖共享 1 MiB
|
|
28
|
-
- SVN
|
|
29
|
-
- SVN 依赖链最多三层,以精确本地路径和安全正常节点捕获;512 路径、20 候选和上下文。工作副本显式上下文、自动依赖、调用方扫描(含非命中)及关键字BASE核验共享1 MiB
|
|
30
|
-
- Git 历史依赖链同样最多三层,两侧共享 512 个路径、20 个候选和上下文、1 MiB
|
|
31
|
-
- 引用方发现最多四个不重叠的非根目录;128 个候选正文共 1 MiB,目录索引最多 256 条/64 KiB,竞争路径最多 512
|
|
27
|
+
- Git 工作区依赖链最多三层、512 个精确路径查询、20 个候选、手动入口与自动依赖共享 1 MiB 正文;文件读取与最终复核共享宿主/用户取消信号。
|
|
28
|
+
- SVN工作副本全采集记录命令次数、共享1MiB元数据内存保护,不设默认命令次数或时间截止,含初始/重复采集、导航、属性和末尾验证;cat输出另受16MiB总硬上限约束。历史采集也有独立的元数据与正文内存保护,不以命令次数或时间中断。
|
|
29
|
+
- SVN 依赖链最多三层,以精确本地路径和安全正常节点捕获;512 路径、20 候选和上下文。工作副本显式上下文、自动依赖、调用方扫描(含非命中)及关键字BASE核验共享1 MiB来源内存账本与宿主/用户取消信号,不能切换阶段重新获得额度;切换、外部与不支持属性节点拒绝。
|
|
30
|
+
- Git 历史依赖链同样最多三层,两侧共享 512 个路径、20 个候选和上下文、1 MiB 正文内存保护和取消信号;只读取固定对象。
|
|
31
|
+
- 引用方发现最多四个不重叠的非根目录;128 个候选正文共 1 MiB,目录索引最多 256 条/64 KiB,竞争路径最多 512 个。发现与复核共享宿主/用户取消信号。
|
|
32
32
|
|
|
33
33
|
达到上限必须阻断或说明未完成,不能静默扩大范围、丢弃证据或重新计时。
|
|
34
34
|
|
package/README.md
CHANGED
|
@@ -225,7 +225,7 @@ Git / SVN 历史审查也能使用这些参数;依赖和调用方按对应版
|
|
|
225
225
|
|
|
226
226
|
项目模式只接受reviewMode、repositoryPath和可选selectedPaths,不与历史revision、autoContext、规则或调用方参数混用。默认全部跟踪节点;不读取未跟踪/忽略文件、子模块、外部仓库及链接目标。目录只作清单记录,源码按批次审查;跨批次不宣称完整调用图或全程序语义分析。
|
|
227
227
|
|
|
228
|
-
清单上限10000节点,单次选择最多1000节点/16MiB,12节点一批,单文件256KiB
|
|
228
|
+
清单上限10000节点,单次选择最多1000节点/16MiB,12节点一批,单文件256KiB;超限明确拒绝,缩小选择后必须重新预览。单次采集共享8MiB元数据内存保护,不按命令条数、插件秒表或模型调用次数中断;耗时受宿主/模型提供方限制,用户可随时取消。SVN为本地已观察清单,稀疏/外部/链接及不支持属性显示限制;不证明仓库远端完整覆盖。
|
|
229
229
|
|
|
230
230
|
报告分别给出总数、completed、excluded、blocked、pending和每批快照。completed只表示所选捕获文件已处理,不代表代码正确;排除与阻断从不算完成。代码或清单变化会拒绝旧预览,运行后无法复核则标记outdated。
|
|
231
231
|
|
|
@@ -285,7 +285,7 @@ Git / SVN 历史审查也能使用这些参数;依赖和调用方按对应版
|
|
|
285
285
|
| 全部快照 | 4 MiB |
|
|
286
286
|
| 额外上下文 | 手动与自动来源共 20 个文件 |
|
|
287
287
|
| 单次模型输入 | 约 96 KiB |
|
|
288
|
-
|
|
|
288
|
+
| 内部时间/总调用次数 | 不设默认截止;遵循宿主与用户取消 |
|
|
289
289
|
|
|
290
290
|
大变更可按保留原行号的差异窗口分批分析;无法容纳的部分会明确标为未完成。建议先按功能或模块选择一小批文件。详细读取和时间预算见 [开发与发布指南](DEVELOPMENT.md)。
|
|
291
291
|
|
package/RELEASE-NOTES.md
CHANGED
|
@@ -1,17 +1,17 @@
|
|
|
1
|
-
# v0.81.
|
|
1
|
+
# v0.81.1 发布说明
|
|
2
2
|
|
|
3
|
-
##
|
|
3
|
+
## 修复预览被内部秒表中断
|
|
4
4
|
|
|
5
|
-
|
|
5
|
+
移除Git/SVN预览、上下文/依赖/调用方导航、mergeinfo、项目模式的默认内部时间截止。单条命令不再默认30秒被终止,模型审查不再默认120秒被取消。延长宿主外层预算后,不会再被这些插件秒表提前截断。
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
同时移除采集命令次数和审查总模型调用次数的默认硬配额;只统计,不按固定条数终止。内部API显式指定timeoutMs/maxCalls时仍可设置限制,默认timeoutMs=0。
|
|
8
8
|
|
|
9
|
-
|
|
9
|
+
用户/宿主取消、停用插件、只读权限、预览确认、来源复核全部保留。等待长任务时可以取消;宿主或模型提供方自己的超时不由本插件解除。
|
|
10
10
|
|
|
11
|
-
|
|
12
|
-
- 分批保留来源哈希,全部批次共享本会话100次模型调用和10分钟。清单最多10000节点,单次选择最多1000节点/16MiB,超限明确拒绝;单文件上限256KiB。
|
|
13
|
-
- 报告区分完成、排除、阻断及未完成路径;没有完成的批次保留pending/partial,来源变化拒绝旧预览。
|
|
14
|
-
- 不包含未跟踪/忽略/子模块/外部内容或链接目标;SVN是本地已观察范围,不证明远端完整清单。跨批次不提供全程序语义调用图。
|
|
15
|
-
- 项目模式仅支持reviewMode、repositoryPath、selectedPaths,不与历史、规则或自动上下文参数混用。不支持的SVN状态和属性明确阻断。
|
|
11
|
+
## 仍保留的资源保护
|
|
16
12
|
|
|
17
|
-
|
|
13
|
+
本次不是无限内存或无限单次模型输入:单文件、内存快照、清单大小、模型输入字节、检索轮次和来源范围保护仍存在,超限明确报告,不静默截断后声称通过。具体数值见 [README](README.md)。这些限制与本次的时间截止分开,不宣称已取消所有限制。
|
|
14
|
+
|
|
15
|
+
## 验证与事实边界
|
|
16
|
+
|
|
17
|
+
真实31秒子进程在不覆盖timeoutMs的情况下正常结束;固定计时器注入检查证明采集不创建内部计时器,显式超时和用户取消仍能终止子进程。没有取得用户C3_Framework隔离副本路径,未声称该样本已经成功预览或生成审查报告。正式交付前需完成统一回归、宿主、隔离安装和四矩阵。
|
package/package.json
CHANGED
package/src/caller-capture.mjs
CHANGED
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {gitCallerIndex} from './caller-index.mjs';
|
|
2
3
|
import {gitCallerProbes} from './caller-probes.mjs';
|
|
3
4
|
import {callerCandidates} from './caller-candidates.mjs';
|
|
@@ -6,8 +7,7 @@ import {isSecretPath} from './secret-path.mjs';
|
|
|
6
7
|
|
|
7
8
|
// Called only by preview capture, never by a model retrieval request.
|
|
8
9
|
export async function captureCallers(root, files, scopes, options={}) {
|
|
9
|
-
const
|
|
10
|
-
const signal=options.signal?AbortSignal.any([options.signal,deadline]):deadline;
|
|
10
|
+
const signal=captureSignal(options.signal);
|
|
11
11
|
const bounded={...options,signal,maxFileBytes:Math.min(options.maxFileBytes??256*1024,256*1024)};
|
|
12
12
|
const index=await gitCallerIndex(root,scopes,bounded);
|
|
13
13
|
const changed=new Set(files.flatMap(f=>[f.path,...(f.oldPath?[f.oldPath]:[])]));
|
package/src/dependency-chain.mjs
CHANGED
|
@@ -1,9 +1,10 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {contextCandidates} from './context-candidates.mjs';
|
|
2
3
|
import {gitContextIndex} from './tracked-context.mjs';
|
|
3
4
|
import {readLocal} from './content.mjs';
|
|
4
5
|
// Fixed-depth literal navigation, not symbol binding. Never enumerate the repository.
|
|
5
6
|
export async function captureDependencyChain(root,files,options={}){
|
|
6
|
-
const signal=
|
|
7
|
+
const signal=captureSignal(options.signal);
|
|
7
8
|
const scoped={...options,maxFileBytes:Math.min(options.maxFileBytes??256*1024,256*1024),signal},probes=new Set(),visited=new Set(files.map(f=>f.path)),context=[],candidates=[];
|
|
8
9
|
const reserved=new Set([...(options.contextPaths??[]),...(options.rulePaths??[])]),limit=Math.min(options.maxContextFiles??20,20);
|
|
9
10
|
let frontier=[...files],index,bytes=0,truncated=false,depth=0;
|
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {historyCallerIndex,historyCallerProbes} from './history-caller-index.mjs';
|
|
2
3
|
import {callerCandidates} from './caller-candidates.mjs';
|
|
3
4
|
import {checked} from './process.mjs';
|
|
@@ -5,7 +6,7 @@ import {decode,hash} from './content.mjs';
|
|
|
5
6
|
import {isSecretPath} from './secret-path.mjs';
|
|
6
7
|
// Both sides share one capture lifecycle and all discovery limits. Never called by model readers.
|
|
7
8
|
export async function captureHistoryCallers(root,files,history,scopes,options={}){
|
|
8
|
-
const
|
|
9
|
+
const signal=captureSignal(options.signal),bounded={...options,signal};
|
|
9
10
|
const changed=new Set(files.map(f=>f.path)),rules=new Set(options.rulePaths??[]),scanned=[],candidates=[],indexes=[],contexts=new Map();let scanBytes=0,indexBytes=0,indexRecords=0,probeBytes=0;const allProbes=new Set();
|
|
10
11
|
for(const [side,commit] of [['old',history.base],['new',history.target]]){if(!commit)continue;const index=await historyCallerIndex(root,commit,scopes,bounded);indexBytes+=index.bytes;indexRecords+=index.records;if(indexBytes>64*1024||indexRecords>256)throw new Error('Historical caller shared index limit exceeded');const sources=[];
|
|
11
12
|
for(const entry of index.entries){signal.throwIfAborted();if(changed.has(entry.path)||rules.has(entry.path)||isSecretPath(entry.path)||!/\.(mjs|cjs|js|jsx|ts|tsx|c|cc|cpp|cxx|h|hh|hpp|hxx|shader|hlsl|glsl|cginc|hlsli|compute)$/i.test(entry.path))continue;if(scanned.length>=128)throw new Error('Historical caller shared scan file limit exceeded');const content=decode(await checked('git',['--no-optional-locks','cat-file','blob',entry.oid],{...bounded,cwd:root,maxBytes:Math.min(options.maxFileBytes??256*1024,256*1024)}));scanBytes+=content.bytes;if(scanBytes>1024*1024)throw new Error('Historical caller shared scan byte limit exceeded');const source={path:entry.path,...content,blobOid:entry.oid,revision:commit,side};scanned.push(source);sources.push(source);}
|
|
@@ -1,10 +1,11 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {contextCandidates} from './context-candidates.mjs';
|
|
2
3
|
import {checked} from './process.mjs';
|
|
3
4
|
import {decode,hash,relativePath} from './content.mjs';
|
|
4
5
|
import {assertNonSecretPath} from './secret-path.mjs';
|
|
5
6
|
// Immutable object reads only. Both version sides share every navigation budget.
|
|
6
7
|
export async function captureHistoryDependencyChain(root,files,history,context,options={}){
|
|
7
|
-
const signal=
|
|
8
|
+
const signal=captureSignal(options.signal);
|
|
8
9
|
const bounded={...options,cwd:root,signal},probes=new Set(),visited={old:new Set(files.map(f=>f.path)),new:new Set(files.map(f=>f.path))};
|
|
9
10
|
const maps={old:new Map(),new:new Map()},candidates=new Map(),reserved=new Set([...(options.rulePaths??[]),...(options.oldContextPaths??[]),...(options.contextPaths??[])]);let bytes=0,truncated=false;
|
|
10
11
|
const regular=e=>e?.type==='blob'&&['100644','100755'].includes(e.mode);
|
package/src/process.mjs
CHANGED
|
@@ -1,10 +1,10 @@
|
|
|
1
1
|
import { spawn } from 'node:child_process';
|
|
2
2
|
|
|
3
3
|
/** Run only adapter-owned commands. No shell, prompts, or unbounded capture. */
|
|
4
|
-
export function run(command, args, { cwd, signal, timeoutMs =
|
|
4
|
+
export function run(command, args, { cwd, signal, timeoutMs = 0, maxBytes = 8 * 1024 * 1024, env = {} } = {}) {
|
|
5
5
|
return new Promise((resolve, reject) => {
|
|
6
6
|
signal?.throwIfAborted();
|
|
7
|
-
|
|
7
|
+
if(!Number.isSafeInteger(timeoutMs)||timeoutMs<0||!Number.isSafeInteger(maxBytes)||maxBytes<1)throw new Error('Invalid command limit');
|
|
8
8
|
const childEnv = { ...process.env, ...env };
|
|
9
9
|
for (const key of Object.keys(childEnv)) if (/^GIT_/i.test(key)) delete childEnv[key];
|
|
10
10
|
Object.assign(childEnv, { GIT_TERMINAL_PROMPT: '0', GIT_OPTIONAL_LOCKS: '0', GIT_NO_REPLACE_OBJECTS: '1', GIT_NO_LAZY_FETCH: '1' });
|
|
@@ -19,7 +19,7 @@ export function run(command, args, { cwd, signal, timeoutMs = 30_000, maxBytes =
|
|
|
19
19
|
const abort = () => stop(signal.reason ?? new Error('Cancelled'));
|
|
20
20
|
signal?.addEventListener('abort', abort, { once: true });
|
|
21
21
|
if (signal?.aborted) abort();
|
|
22
|
-
const timer = setTimeout(() => stop(new Error('Command timeout')), timeoutMs);
|
|
22
|
+
const timer = timeoutMs>0?setTimeout(() => stop(new Error('Command timeout')), timeoutMs):undefined;
|
|
23
23
|
const capture = chunks => data => {
|
|
24
24
|
bytes += data.length;
|
|
25
25
|
if (bytes > maxBytes) stop(new Error('Command output limit exceeded'));
|
|
@@ -7,11 +7,11 @@ import {projectInventory,captureProjectBatch} from './project-snapshot.mjs';
|
|
|
7
7
|
import {reviewSnapshot,markdownReport} from './review.mjs';
|
|
8
8
|
import {createDshModel} from './dsh-model.mjs';
|
|
9
9
|
const owner=exec=>{const a=exec?.agent;if(!a?.session?.header?.cwd||!a.options?.provider||!a.options?.model)throw new Error('Active Agent with cwd and model required');return {agent:a,cwd:a.session.header.cwd,route:{provider:a.options.provider,model:a.options.model,...(a.options.reasoningEffort?{reasoningEffort:a.options.reasoningEffort}:{})}};};
|
|
10
|
-
export function createProjectReviewService(llm,{allowModelSending=false,authorize=async()=>false,now=Date.now,ttlMs=300000,maxPreviews=8,projectMaxCalls=
|
|
11
|
-
if(!Number.isSafeInteger(projectMaxCalls)||projectMaxCalls<1
|
|
10
|
+
export function createProjectReviewService(llm,{allowModelSending=false,authorize=async()=>false,now=Date.now,ttlMs=300000,maxPreviews=8,projectMaxCalls=Number.MAX_SAFE_INTEGER}={}){
|
|
11
|
+
if(!Number.isSafeInteger(projectMaxCalls)||projectMaxCalls<1)throw new Error('Invalid project model call limit');
|
|
12
12
|
const previews=new Map(),lifetime=new AbortController();let busy=false;
|
|
13
13
|
async function capture(cwd,args,signal){
|
|
14
|
-
let commands=0,metadataBytes=0;const captureBudget={signal,reserve(){signal.throwIfAborted();
|
|
14
|
+
let commands=0,metadataBytes=0;const captureBudget={signal,reserve(){signal.throwIfAborted();commands++;},charge(size){metadataBytes+=size;if(metadataBytes>8*1024*1024)throw Object.assign(new Error('Project shared metadata budget exceeded'),{code:'PROJECT_CAPTURE_BUDGET'});}};
|
|
15
15
|
const captureOptions={signal,svnCaptureBudget:captureBudget};
|
|
16
16
|
const root=await realpath(await resolveWorkspaceRepository(cwd,args.repositoryPath)),vcs=await detectVcs(root),inventory=await projectInventory(root,vcs,captureOptions);
|
|
17
17
|
let paths=inventory.records.map(r=>r.path);if(args.selectedPaths!==undefined){if(!Array.isArray(args.selectedPaths)||args.selectedPaths.length>1000||new Set(args.selectedPaths).size!==args.selectedPaths.length||args.selectedPaths.some(p=>!paths.includes(p)))throw new Error('Invalid tracked project selection');paths=paths.filter(p=>args.selectedPaths.includes(p));}
|
|
@@ -21,10 +21,10 @@ export function createProjectReviewService(llm,{allowModelSending=false,authoriz
|
|
|
21
21
|
for(const b of batches)if((await captureProjectBatch(verified,b.files.map(f=>f.path),captureOptions)).id!==b.id)throw new Error('Project source changed during capture');
|
|
22
22
|
const result={root,vcs,inventoryId:inventory.id,inventoryComplete:inventory.vcs==='git',totalTrackedNodes:inventory.records.length,selectedFiles:paths.length,selectionComplete:paths.length===inventory.records.length,notice:inventory.notice,batches};return {...result,id:hash(JSON.stringify(result))};
|
|
23
23
|
}
|
|
24
|
-
const signalFor=
|
|
24
|
+
const signalFor=exec=>AbortSignal.any([lifetime.signal,...(exec.signal?[exec.signal]:[])]);
|
|
25
25
|
return {owns:id=>previews.has(id),
|
|
26
|
-
async preview(args,exec){if(busy)throw new Error('Review service busy');if(!args||typeof args!=='object')throw new Error('Project preview parameters required');if(Object.keys(args).some(k=>!['reviewMode','repositoryPath','selectedPaths'].includes(k))||args.reviewMode!=='project')throw new Error('Project mode supports only reviewMode/repositoryPath/selectedPaths; change/history context parameters are not project parameters');const current=owner(exec);busy=true;try{const s=await capture(current.cwd,args,signalFor(exec
|
|
27
|
-
async execute(args,exec){if(busy)throw new Error('Review service busy');if(!allowModelSending)throw new Error('Model sending disabled');if(args.confirmed!==true||typeof args.previewId!=='string'||Object.keys(args).some(k=>!['previewId','confirmed'].includes(k)))throw new Error('Confirmed preview required');const p=previews.get(args.previewId),current=owner(exec);if(!p||p.expires<=now()||p.agent!==current.agent||p.cwd!==current.cwd||JSON.stringify(p.route)!==JSON.stringify(current.route))throw new Error('Preview missing expired or owner/model/directory changed');previews.delete(args.previewId);busy=true;const signal=signalFor(exec
|
|
26
|
+
async preview(args,exec){if(busy)throw new Error('Review service busy');if(!args||typeof args!=='object')throw new Error('Project preview parameters required');if(Object.keys(args).some(k=>!['reviewMode','repositoryPath','selectedPaths'].includes(k))||args.reviewMode!=='project')throw new Error('Project mode supports only reviewMode/repositoryPath/selectedPaths; change/history context parameters are not project parameters');const current=owner(exec);busy=true;try{const s=await capture(current.cwd,args,signalFor(exec));for(const [id,p]of previews)if(p.expires<=now())previews.delete(id);while(previews.size>=maxPreviews)previews.delete(previews.keys().next().value);const previewId=randomUUID();previews.set(previewId,{...current,args:structuredClone(args),snapshotId:s.id,expires:now()+ttlMs});return {previewId,snapshotId:s.id,reviewMode:'project',repositoryRoot:s.root,vcs:s.vcs,model:current.route,modelSendingEnabled:allowModelSending,totalTrackedNodes:s.totalTrackedNodes,selectedFiles:s.selectedFiles,selectionComplete:s.selectionComplete,inventoryComplete:s.inventoryComplete,batches:s.batches.length,files:s.batches.flatMap(b=>b.files.map(f=>({path:f.path,eligibility:f.eligibility,reason:f.reason}))),notice:s.notice+' Full current source audit, not regression-only diff. Shared session execution budget: no internal elapsed-time or model-call cutoff, 16MiB capture memory protection. Blocked/excluded/pending files are not reviewed successfully. No untracked/ignored/external bodies; no complete semantic call graph. Show scope and model before confirmed execute.'};}finally{busy=false;}},
|
|
27
|
+
async execute(args,exec){if(busy)throw new Error('Review service busy');if(!allowModelSending)throw new Error('Model sending disabled');if(args.confirmed!==true||typeof args.previewId!=='string'||Object.keys(args).some(k=>!['previewId','confirmed'].includes(k)))throw new Error('Confirmed preview required');const p=previews.get(args.previewId),current=owner(exec);if(!p||p.expires<=now()||p.agent!==current.agent||p.cwd!==current.cwd||JSON.stringify(p.route)!==JSON.stringify(current.route))throw new Error('Preview missing expired or owner/model/directory changed');previews.delete(args.previewId);busy=true;const signal=signalFor(exec);try{const s=await capture(current.cwd,p.args,signal);if(s.id!==p.snapshotId)throw new Error('Project changed since preview');if(await authorize({snapshot:s,route:p.route,exec})!==true)throw new Error('Model sending denied');if(owner(exec).cwd!==p.cwd||JSON.stringify(owner(exec).route)!==JSON.stringify(p.route))throw new Error('Agent changed during approval');signal.throwIfAborted();
|
|
28
28
|
let calls=0;const adapter=createDshModel(llm,p.route),model=request=>{if(calls>=projectMaxCalls)throw new Error('Shared project model call limit exceeded');calls++;return adapter(request);};const results=[];
|
|
29
29
|
for(const b of s.batches){if(signal.aborted||calls>=projectMaxCalls){results.push({snapshotId:b.id,status:'partial',files:b.files.map(f=>({fileId:f.id,path:f.path,status:f.eligibility==='reviewable'?'pending':f.eligibility,reason:'Session project budget exhausted or cancelled'})),findings:[],limitations:[]});continue;}results.push(await reviewSnapshot(b,model,{signal,maxCalls:projectMaxCalls-calls,enableRetrieval:true,enableVerification:true}));}
|
|
30
30
|
const files=results.flatMap(r=>r.files.map(f=>({...f,batchSnapshotId:r.snapshotId}))),coverage={total:s.selectedFiles,completed:files.filter(f=>f.status==='completed').length,excluded:files.filter(f=>f.status==='excluded').length,blocked:files.filter(f=>f.status==='blocked').length,pending:files.filter(f=>!['completed','excluded','blocked'].includes(f.status)).length};const findings=results.flatMap(r=>r.findings.map(f=>({...f,batchSnapshotId:r.snapshotId})));const report={schemaVersion:1,mode:'project',snapshotId:s.id,status:signal.aborted?'cancelled':coverage.total===0||coverage.completed===0?'partial':results.every(r=>r.status==='completed')&&files.every(f=>['completed','excluded'].includes(f.status))?'completed':'partial',repositoryRoot:s.root,vcs:s.vcs,model:p.route,modelCalls:calls,maxModelCalls:projectMaxCalls,totalTrackedNodes:s.totalTrackedNodes,selectionComplete:s.selectionComplete,inventoryComplete:s.inventoryComplete,coverage,files,findings,batches:results,notice:s.notice+' Coverage counts processing of selected tracked files only, not proof of correctness. Excluded and blocked are never completed. No cross-batch whole-program semantic coverage is claimed.'};
|
package/src/review.mjs
CHANGED
|
@@ -64,7 +64,7 @@ async function invoke(model, request, timeoutMs, parentSignal) {
|
|
|
64
64
|
const abort = () => controller.abort(parentSignal.reason);
|
|
65
65
|
parentSignal?.addEventListener('abort', abort, { once: true });
|
|
66
66
|
if (parentSignal?.aborted) abort();
|
|
67
|
-
const timer = setTimeout(() => controller.abort(Object.assign(new Error('Model timeout'), { code: 'MODEL_TIMEOUT' })), timeoutMs);
|
|
67
|
+
const timer = timeoutMs>0?setTimeout(() => controller.abort(Object.assign(new Error('Model timeout'), { code: 'MODEL_TIMEOUT' })), timeoutMs):undefined;
|
|
68
68
|
try {
|
|
69
69
|
controller.signal.throwIfAborted();
|
|
70
70
|
// Executors must honor cancellation; never release ownership before cleanup settles.
|
|
@@ -79,9 +79,10 @@ async function invoke(model, request, timeoutMs, parentSignal) {
|
|
|
79
79
|
|
|
80
80
|
export async function reviewSnapshot(snapshot, model, options = {}) {
|
|
81
81
|
const rounds = reviewRounds(options.reviewRounds);
|
|
82
|
-
const { signal, maxInputBytes = 96 * 1024, maxCalls =
|
|
82
|
+
const { signal, maxInputBytes = 96 * 1024, maxCalls = Number.MAX_SAFE_INTEGER, timeoutMs = 0 } = options;
|
|
83
83
|
if (typeof model !== 'function') throw new Error('A model executor is required');
|
|
84
|
-
for
|
|
84
|
+
for(const n of [maxInputBytes,maxCalls])if(!Number.isSafeInteger(n)||n<1)throw new Error('Invalid review limit');
|
|
85
|
+
if(!Number.isSafeInteger(timeoutMs)||timeoutMs<0)throw new Error('Invalid review limit');
|
|
85
86
|
// Clone once: callers cannot change the input while a model call is pending.
|
|
86
87
|
const input = structuredClone(snapshot);
|
|
87
88
|
const instructions=input.projectScope?PROJECT_REVIEW_INSTRUCTIONS:REVIEW_INSTRUCTIONS;
|
package/src/service.mjs
CHANGED
|
@@ -51,7 +51,7 @@ function createSessionReviewService(llm, { capture = captureSnapshot, now = Date
|
|
|
51
51
|
while (previews.size >= maxPreviews) previews.delete(previews.keys().next().value);
|
|
52
52
|
const previewId = randomUUID();
|
|
53
53
|
previews.set(previewId, { ...current, target, options, rounds, snapshotId: snapshot.id, expires: now() + ttlMs });
|
|
54
|
-
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.workingScope?{workingScope:snapshot.workingScope}:{}), ...(snapshot.history ? { history: snapshot.history } : {}), ...(snapshot.mergeSources ? {mergeSources:snapshot.mergeSources} : {}),...(snapshot.effectiveMergeSources?{effectiveMergeSources:snapshot.effectiveMergeSources}:{}), model: current.route, modelSendingEnabled: allowModelSending, files: snapshot.files.map(f => ({ path: f.path,...(f.workingSource?{workingSource:f.workingSource}:{}),...(f.replacementSource?{replacementSource:f.replacementSource,contentNotice:f.contentNotice}:{}), eligibility: f.eligibility, reason: f.reason,...(f.specialData?{specialData:f.specialData,contentNotice:f.contentNotice}:{}), ...(f.eolMapping?{eolMapping:f.eolMapping,textNotice:f.textNotice}:{}), ...(f.keywordMapping?{keywordMapping:f.keywordMapping,textNotice:f.textNotice}:{}), ...(f.rawStatus==='replaced'&&(snapshot.history?.vcs==='svn'||f.replacementSource)?{oldKind:f.oldKind,newKind:f.newKind,nodeNotice:f.nodeNotice}:{}), ...(f.blockCode ? {blockCode:f.blockCode,...(svnDiagnostic(f.blockCode)?{diagnostic:svnDiagnostic(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
|
|
54
|
+
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.workingScope?{workingScope:snapshot.workingScope}:{}), ...(snapshot.history ? { history: snapshot.history } : {}), ...(snapshot.mergeSources ? {mergeSources:snapshot.mergeSources} : {}),...(snapshot.effectiveMergeSources?{effectiveMergeSources:snapshot.effectiveMergeSources}:{}), model: current.route, modelSendingEnabled: allowModelSending, files: snapshot.files.map(f => ({ path: f.path,...(f.workingSource?{workingSource:f.workingSource}:{}),...(f.replacementSource?{replacementSource:f.replacementSource,contentNotice:f.contentNotice}:{}), eligibility: f.eligibility, reason: f.reason,...(f.specialData?{specialData:f.specialData,contentNotice:f.contentNotice}:{}), ...(f.eolMapping?{eolMapping:f.eolMapping,textNotice:f.textNotice}:{}), ...(f.keywordMapping?{keywordMapping:f.keywordMapping,textNotice:f.textNotice}:{}), ...(f.rawStatus==='replaced'&&(snapshot.history?.vcs==='svn'||f.replacementSource)?{oldKind:f.oldKind,newKind:f.newKind,nodeNotice:f.nodeNotice}:{}), ...(f.blockCode ? {blockCode:f.blockCode,...(svnDiagnostic(f.blockCode)?{diagnostic:svnDiagnostic(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轮检索,不设插件内部默认时间或总模型调用次数截止,遵循宿主/用户取消;输入字节和检索轮次保护仍保留;复核不能证明缺陷成立。plan.reviewRounds为独立审查工作量,非模型reasoning effort;后续轮不带首轮风险计划,所有轮共享原超时与预算。plan仅预检初始输入,ready不代表已审查;minimumCalls不含后续检索/复核/动态业务综合;多文件可先进行业务分组,分组失败保留原组并记录限制,动态综合数执行前未知,实际预算可能不足。请先向用户展示范围,获得确认后执行。' };
|
|
55
55
|
} finally { busy = false; }
|
|
56
56
|
},
|
|
57
57
|
async execute(args, exec) {
|
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {svnCallerIndex} from './svn-caller-index.mjs';
|
|
2
3
|
import {svnCallerNode,svnCallerProbes} from './svn-caller-node.mjs';
|
|
3
4
|
|
|
@@ -9,8 +10,7 @@ import {createSvnNavigationBudget} from './svn-navigation-budget.mjs';
|
|
|
9
10
|
|
|
10
11
|
// Called only by preview capture, never by a model retrieval request.
|
|
11
12
|
export async function captureSvnCallers(root, files, scopes, options={}) {
|
|
12
|
-
const
|
|
13
|
-
const signal=options.signal?AbortSignal.any([options.signal,deadline]):deadline;
|
|
13
|
+
const signal=captureSignal(options.signal);
|
|
14
14
|
const bounded={...options,signal,svnNavigationBudget:options.svnNavigationBudget??createSvnNavigationBudget(signal),maxFileBytes:Math.min(options.maxFileBytes??256*1024,256*1024)};
|
|
15
15
|
const index=await svnCallerIndex(root,scopes,bounded);
|
|
16
16
|
const changed=new Set(files.flatMap(f=>[f.path,...(f.oldPath?[f.oldPath]:[])]));
|
|
@@ -1,6 +1,7 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
// Shared across initial capture, navigation, mergeinfo and final verification.
|
|
2
3
|
export function createSvnCaptureBudget(signal){
|
|
3
|
-
const deadline=
|
|
4
|
+
const deadline=captureSignal(signal);let commands=0,metadataBytes=0,bodyBytes=0;
|
|
4
5
|
const error=message=>Object.assign(new Error(message),{code:'SVN_CAPTURE_BUDGET'});
|
|
5
|
-
return Object.freeze({signal:deadline,reserve(){deadline.throwIfAborted();
|
|
6
|
+
return Object.freeze({signal:deadline,reserve(){deadline.throwIfAborted();commands++;},charge(size,body=false){deadline.throwIfAborted();if(!Number.isSafeInteger(size)||size<0)throw error('Invalid SVN capture byte charge');if(body){bodyBytes+=size;if(bodyBytes>16*1024*1024)throw error('SVN capture shared verification body budget exceeded');}else{metadataBytes+=size;if(metadataBytes>1024*1024)throw error('SVN capture shared metadata budget exceeded');}},usage(){return {commands,metadataBytes,bodyBytes};}});
|
|
6
7
|
}
|
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {contextCandidates} from './context-candidates.mjs';
|
|
2
3
|
import {svnCallerNode,svnCallerProbes} from './svn-caller-node.mjs';
|
|
3
4
|
import {captureSvnExplicitContext} from './svn-explicit-context.mjs';
|
|
@@ -5,7 +6,7 @@ import {createSvnNavigationBudget} from './svn-navigation-budget.mjs';
|
|
|
5
6
|
|
|
6
7
|
// Exact local path probes only. Every body requires safe offline SVN node identity.
|
|
7
8
|
export async function captureSvnDependencyChain(root,files,options={}){
|
|
8
|
-
const signal=
|
|
9
|
+
const signal=captureSignal(options.signal),bounded={...options,signal,svnNavigationBudget:options.svnNavigationBudget??createSvnNavigationBudget(signal),maxFileBytes:Math.min(options.maxFileBytes??256*1024,256*1024)};
|
|
9
10
|
const visited=new Set(files.map(f=>f.path)),reserved=new Set([...(options.contextPaths??[]),...(options.rulePaths??[])]),probes=new Set(),context=[],candidates=[],nodes=[];let frontier=[...files],bytes=0,truncated=false,index;
|
|
10
11
|
const limit=Math.min(options.maxContextFiles??20,20);if(new Set(options.contextPaths??[]).size>limit)throw new Error('Context file limit exceeded');
|
|
11
12
|
const capture=async name=>{const value=await captureSvnExplicitContext(root,[name],bounded);bytes+=value.sourceBytes;if(bytes>1024*1024)throw new Error('SVN dependency source byte limit exceeded');return {item:value.context[0],node:value.identities[0]};};
|
|
@@ -1,10 +1,11 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {svnCallerNode} from './svn-caller-node.mjs';
|
|
2
3
|
import {readLocal,decode} from './content.mjs';
|
|
3
4
|
import {checked} from './process.mjs';
|
|
4
5
|
import {svnKeywordMapping} from './svn-keyword-mapping.mjs';
|
|
5
6
|
import {svnEolMapping} from './svn-eol-mapping.mjs';
|
|
6
7
|
export async function captureSvnExplicitContext(root,names,options={}){
|
|
7
|
-
const signal=
|
|
8
|
+
const signal=captureSignal(options.signal),bounded={...options,allowExplicitKeywords:true,signal,maxFileBytes:Math.min(options.maxFileBytes??256*1024,256*1024)};
|
|
8
9
|
if(!Array.isArray(names)||names.length>20||new Set(names).size!==names.length)throw new Error('Invalid SVN explicit context selection');
|
|
9
10
|
const context=[],identities=[];let bytes=0;
|
|
10
11
|
for(const name of names){
|
package/src/svn-history.mjs
CHANGED
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {sameSvnUrl,canonicalSvnUrl} from './svn-url.mjs';
|
|
2
3
|
import {svnHistoryEndpoints} from './svn-history-endpoints.mjs';
|
|
3
4
|
import {realpath} from 'node:fs/promises';
|
|
@@ -62,11 +63,11 @@ export async function captureSvnHistory(cwd,options={}) {
|
|
|
62
63
|
for(const [value,cap]of [[maxFiles,200],[maxFileBytes,256*1024],[maxSnapshotBytes,4*1024*1024]])if(!Number.isSafeInteger(value)||value<1||value>cap)throw new Error('Invalid SVN history limit');
|
|
63
64
|
if(selectedPaths!==undefined && (!Array.isArray(selectedPaths)||selectedPaths.length>200||new Set(selectedPaths).size!==selectedPaths.length))throw new Error('Invalid SVN history selection');
|
|
64
65
|
for(const name of selectedPaths??[])if(name!=='.')relativePath(name);
|
|
65
|
-
const signal=
|
|
66
|
+
const signal=captureSignal(options.signal);
|
|
66
67
|
let commands=0,metadataBytes=0,sourceBytes=0;
|
|
67
68
|
const budgetError = message => Object.assign(new Error(message), {code:'SVN_HISTORY_BUDGET'});
|
|
68
69
|
const run=async(args,body=false)=>{
|
|
69
|
-
signal.throwIfAborted();if(
|
|
70
|
+
signal.throwIfAborted();commands++;if(metadataBytes>1024*1024 || sourceBytes>maxSnapshotBytes)throw budgetError('SVN historical shared budget exceeded');
|
|
70
71
|
const bytes=await checked('svn',['--non-interactive',...args],{cwd,signal,maxBytes:body?maxFileBytes:1024*1024});
|
|
71
72
|
if(body){sourceBytes+=bytes.length;if(sourceBytes>maxSnapshotBytes)throw budgetError('SVN historical source budget exceeded');}
|
|
72
73
|
else{metadataBytes+=bytes.length;if(metadataBytes>1024*1024)throw budgetError('SVN historical metadata budget exceeded');}
|
|
@@ -1,5 +1,6 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
// One source ledger/deadline per working-copy navigation capture, never per stage.
|
|
2
3
|
export function createSvnNavigationBudget(signal){
|
|
3
|
-
const sharedSignal=
|
|
4
|
+
const sharedSignal=captureSignal(signal);let bytes=0;
|
|
4
5
|
return Object.freeze({signal:sharedSignal,charge(size){sharedSignal.throwIfAborted();if(!Number.isSafeInteger(size)||size<0)throw new Error('Invalid SVN navigation source size');bytes+=size;if(bytes>1024*1024)throw Object.assign(new Error('SVN shared navigation source byte limit exceeded'),{code:'SVN_NAVIGATION_BUDGET'});},usage(){return bytes;}});
|
|
5
6
|
}
|
|
@@ -1,3 +1,4 @@
|
|
|
1
|
+
import {captureSignal} from './capture-signal.mjs';
|
|
1
2
|
import {checked} from './process.mjs';
|
|
2
3
|
import {xml} from './svn.mjs';
|
|
3
4
|
import {hash} from './content.mjs';
|
|
@@ -5,8 +6,8 @@ import {svnWorkingIdentity} from './svn-working-identity.mjs';
|
|
|
5
6
|
import {captureEffectiveMergeinfo} from './svn-effective-mergeinfo.mjs';
|
|
6
7
|
|
|
7
8
|
export async function workingEffectiveMergeinfo(root,rootInfo,files,options={}){
|
|
8
|
-
const signal=
|
|
9
|
-
const run=async args=>{signal.throwIfAborted();
|
|
9
|
+
const signal=captureSignal(options.signal);let commands=0,bytes=0;const cache=new Map();
|
|
10
|
+
const run=async args=>{signal.throwIfAborted();commands++;const data=await checked('svn',['--non-interactive',...args],{...options,cwd:root,signal,maxBytes:256*1024});bytes+=data.length;if(bytes>1024*1024)throw new Error('SVN working mergeinfo metadata budget exceeded');return xml(data);};
|
|
10
11
|
const read=async(name,side,revision)=>{
|
|
11
12
|
const key=JSON.stringify([name,side]);if(cache.has(key)){const value=cache.get(key);if(side==='old'&&value.source.revision!==revision)throw new Error('Mixed BASE revisions cannot prove effective mergeinfo');return value;}
|
|
12
13
|
const node=(await run(['info','--xml','--',name+'@'])).info?.entry?.[0];const identity=svnWorkingIdentity(node,rootInfo,name);
|