惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
月光博客
月光博客
爱范儿
爱范儿
The Cloudflare Blog
Y
Y Combinator Blog
B
Blog RSS Feed
Stack Overflow Blog
Stack Overflow Blog
博客园 - 叶小钗
G
Google Developers Blog
J
Java Code Geeks
P
Proofpoint News Feed
美团技术团队
Engineering at Meta
Engineering at Meta
腾讯CDC
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
博客园_首页
WordPress大学
WordPress大学
博客园 - 聂微东
雷峰网
雷峰网
有赞技术团队
有赞技术团队
L
LangChain Blog
N
Netflix TechBlog - Medium
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
博客园 - 【当耐特】

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(sandbox): move registry file migration to doctor · op...
steipete · 2026-05-03 · via Recent Commits to openclaw:main

@@ -53,6 +53,26 @@ type RegistryFile = {

5353

entries: RegistryEntryPayload[];

5454

};

555556+

type LegacyRegistryKind = "containers" | "browsers";

57+58+

type LegacyRegistryTarget = {

59+

kind: LegacyRegistryKind;

60+

registryPath: string;

61+

shardedDir: string;

62+

};

63+64+

export type LegacySandboxRegistryInspection = LegacyRegistryTarget & {

65+

exists: boolean;

66+

valid: boolean;

67+

entries: number;

68+

};

69+70+

export type LegacySandboxRegistryMigrationResult = LegacyRegistryTarget & {

71+

status: "missing" | "migrated" | "removed-empty" | "quarantined-invalid";

72+

entries: number;

73+

quarantinePath?: string;

74+

};

75+5676

const RegistryEntrySchema = z

5777

.object({

5878

containerName: z.string(),

@@ -103,7 +123,6 @@ async function readLegacyRegistryFile(registryPath: string): Promise<RegistryFil

103123

}

104124105125

export async function readRegistry(): Promise<SandboxRegistry> {

106-

await migrateMonolithicIfNeeded(SANDBOX_REGISTRY_PATH, SANDBOX_CONTAINERS_DIR);

107126

const entries = await readShardedEntries<SandboxRegistryEntry>(SANDBOX_CONTAINERS_DIR);

108127

return {

109128

entries: entries.map((entry) => normalizeSandboxRegistryEntry(entry)),

@@ -197,57 +216,115 @@ async function readShardedEntries<T extends RegistryEntry>(dir: string): Promise

197216

);

198217

}

199218200-

async function quarantineLegacyRegistry(registryPath: string): Promise<void> {

219+

async function quarantineLegacyRegistry(registryPath: string): Promise<string> {

201220

const quarantinePath = `${registryPath}.invalid-${Date.now()}`;

202221

await fs.rename(registryPath, quarantinePath).catch(async (error) => {

203222

const code = (error as { code?: string } | null)?.code;

204223

if (code !== "ENOENT") {

205224

await fs.rm(registryPath, { force: true });

206225

}

207226

});

227+

return quarantinePath;

208228

}

209229210-

async function migrateMonolithicIfNeeded(registryPath: string, shardedDir: string): Promise<void> {

230+

async function migrateMonolithicIfNeeded(

231+

target: LegacyRegistryTarget,

232+

): Promise<LegacySandboxRegistryMigrationResult> {

233+

const { registryPath, shardedDir } = target;

211234

try {

212235

await fs.access(registryPath);

213236

} catch (error) {

214237

const code = (error as { code?: string } | null)?.code;

215238

if (code === "ENOENT") {

216-

return;

239+

return { ...target, status: "missing", entries: 0 };

217240

}

218241

throw error;

219242

}

220243221-

await withRegistryLock(registryPath, async () => {

244+

return await withRegistryLock(registryPath, async () => {

222245

const registry = await readLegacyRegistryFile(registryPath);

223246

if (!registry) {

224-

await quarantineLegacyRegistry(registryPath);

225-

return;

247+

const quarantinePath = await quarantineLegacyRegistry(registryPath);

248+

return { ...target, status: "quarantined-invalid", entries: 0, quarantinePath };

226249

}

227250

if (registry.entries.length === 0) {

228251

await fs.rm(registryPath, { force: true });

229-

return;

252+

return { ...target, status: "removed-empty", entries: 0 };

230253

}

231254

await fs.mkdir(shardedDir, { recursive: true });

232255

for (const entry of registry.entries) {

233256

await withEntryLock(shardedDir, entry.containerName, async () => {

234-

await writeShardedEntry(shardedDir, entry);

257+

const existing = await readShardedEntry(shardedDir, entry.containerName);

258+

if (!existing) {

259+

await writeShardedEntry(shardedDir, entry);

260+

}

235261

});

236262

}

237263

await fs.rm(registryPath, { force: true });

264+

return { ...target, status: "migrated", entries: registry.entries.length };

238265

});

239266

}

240267268+

function legacyRegistryTargets(): LegacyRegistryTarget[] {

269+

return [

270+

{

271+

kind: "containers",

272+

registryPath: SANDBOX_REGISTRY_PATH,

273+

shardedDir: SANDBOX_CONTAINERS_DIR,

274+

},

275+

{

276+

kind: "browsers",

277+

registryPath: SANDBOX_BROWSER_REGISTRY_PATH,

278+

shardedDir: SANDBOX_BROWSERS_DIR,

279+

},

280+

];

281+

}

282+283+

export async function inspectLegacySandboxRegistryFiles(): Promise<

284+

LegacySandboxRegistryInspection[]

285+

> {

286+

const inspections: LegacySandboxRegistryInspection[] = [];

287+

for (const target of legacyRegistryTargets()) {

288+

try {

289+

await fs.access(target.registryPath);

290+

} catch (error) {

291+

const code = (error as { code?: string } | null)?.code;

292+

if (code === "ENOENT") {

293+

inspections.push({ ...target, exists: false, valid: true, entries: 0 });

294+

continue;

295+

}

296+

throw error;

297+

}

298+299+

const registry = await readLegacyRegistryFile(target.registryPath);

300+

inspections.push({

301+

...target,

302+

exists: true,

303+

valid: Boolean(registry),

304+

entries: registry?.entries.length ?? 0,

305+

});

306+

}

307+

return inspections;

308+

}

309+310+

export async function migrateLegacySandboxRegistryFiles(): Promise<

311+

LegacySandboxRegistryMigrationResult[]

312+

> {

313+

const results: LegacySandboxRegistryMigrationResult[] = [];

314+

for (const target of legacyRegistryTargets()) {

315+

results.push(await migrateMonolithicIfNeeded(target));

316+

}

317+

return results;

318+

}

319+241320

export async function readRegistryEntry(

242321

containerName: string,

243322

): Promise<SandboxRegistryEntry | null> {

244-

await migrateMonolithicIfNeeded(SANDBOX_REGISTRY_PATH, SANDBOX_CONTAINERS_DIR);

245323

const entry = await readShardedEntry<SandboxRegistryEntry>(SANDBOX_CONTAINERS_DIR, containerName);

246324

return entry ? normalizeSandboxRegistryEntry(entry) : null;

247325

}

248326249327

export async function updateRegistry(entry: SandboxRegistryEntry) {

250-

await migrateMonolithicIfNeeded(SANDBOX_REGISTRY_PATH, SANDBOX_CONTAINERS_DIR);

251328

await withEntryLock(SANDBOX_CONTAINERS_DIR, entry.containerName, async () => {

252329

const existing = await readShardedEntry<SandboxRegistryEntry>(

253330

SANDBOX_CONTAINERS_DIR,

@@ -266,19 +343,16 @@ export async function updateRegistry(entry: SandboxRegistryEntry) {

266343

}

267344268345

export async function removeRegistryEntry(containerName: string) {

269-

await migrateMonolithicIfNeeded(SANDBOX_REGISTRY_PATH, SANDBOX_CONTAINERS_DIR);

270346

await withEntryLock(SANDBOX_CONTAINERS_DIR, containerName, async () => {

271347

await removeShardedEntry(SANDBOX_CONTAINERS_DIR, containerName);

272348

});

273349

}

274350275351

export async function readBrowserRegistry(): Promise<SandboxBrowserRegistry> {

276-

await migrateMonolithicIfNeeded(SANDBOX_BROWSER_REGISTRY_PATH, SANDBOX_BROWSERS_DIR);

277352

return { entries: await readShardedEntries<SandboxBrowserRegistryEntry>(SANDBOX_BROWSERS_DIR) };

278353

}

279354280355

export async function updateBrowserRegistry(entry: SandboxBrowserRegistryEntry) {

281-

await migrateMonolithicIfNeeded(SANDBOX_BROWSER_REGISTRY_PATH, SANDBOX_BROWSERS_DIR);

282356

await withEntryLock(SANDBOX_BROWSERS_DIR, entry.containerName, async () => {

283357

const existing = await readShardedEntry<SandboxBrowserRegistryEntry>(

284358

SANDBOX_BROWSERS_DIR,

@@ -294,7 +368,6 @@ export async function updateBrowserRegistry(entry: SandboxBrowserRegistryEntry)

294368

}

295369296370

export async function removeBrowserRegistryEntry(containerName: string) {

297-

await migrateMonolithicIfNeeded(SANDBOX_BROWSER_REGISTRY_PATH, SANDBOX_BROWSERS_DIR);

298371

await withEntryLock(SANDBOX_BROWSERS_DIR, containerName, async () => {

299372

await removeShardedEntry(SANDBOX_BROWSERS_DIR, containerName);

300373

});