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

推荐订阅源

量子位
Recent Announcements
Recent Announcements
D
Docker
V
V2EX
阮一峰的网络日志
阮一峰的网络日志
Vercel News
Vercel News
Microsoft Security Blog
Microsoft Security Blog
The GitHub Blog
The GitHub Blog
U
Unit 42
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
月光博客
月光博客
腾讯CDC
B
Blog
博客园_首页
罗磊的独立博客
D
DataBreaches.Net
IT之家
IT之家
酷 壳 – CoolShell
酷 壳 – CoolShell
L
LangChain Blog
aimingoo的专栏
aimingoo的专栏
MongoDB | Blog
MongoDB | Blog
GbyAI
GbyAI
Stack Overflow Blog
Stack Overflow Blog
M
MIT News - Artificial intelligence

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
refactor(plugins): reuse optional string normalization · ...
vincentkoc · 2026-06-23 · via Recent Commits to openclaw:main

@@ -140,14 +140,6 @@ function splitRef(value: string): { base: string; ref?: string } {

140140

};

141141

}

142142143-

function toOptionalString(value: unknown): string | undefined {

144-

if (typeof value !== "string") {

145-

return undefined;

146-

}

147-

const trimmed = value.trim();

148-

return trimmed.length > 0 ? trimmed : undefined;

149-

}

150-151143

function normalizeEntrySource(

152144

raw: unknown,

153145

): { ok: true; source: MarketplaceEntrySource } | { ok: false; error: string } {

@@ -167,21 +159,21 @@ function normalizeEntrySource(

167159

}

168160169161

const rec = raw as Record<string, unknown>;

170-

const kind = toOptionalString(rec.type) ?? toOptionalString(rec.source);

162+

const kind = normalizeOptionalString(rec.type) ?? normalizeOptionalString(rec.source);

171163

if (!kind) {

172164

return { ok: false, error: 'plugin source object missing "type" or "source"' };

173165

}

174166175167

if (kind === "path") {

176-

const sourcePath = toOptionalString(rec.path);

168+

const sourcePath = normalizeOptionalString(rec.path);

177169

if (!sourcePath) {

178170

return { ok: false, error: 'path source missing "path"' };

179171

}

180172

return { ok: true, source: { kind: "path", path: sourcePath } };

181173

}

182174183175

if (kind === "github") {

184-

const repo = toOptionalString(rec.repo) ?? toOptionalString(rec.url);

176+

const repo = normalizeOptionalString(rec.repo) ?? normalizeOptionalString(rec.url);

185177

if (!repo) {

186178

return { ok: false, error: 'github source missing "repo"' };

187179

}

@@ -190,14 +182,17 @@ function normalizeEntrySource(

190182

source: {

191183

kind: "github",

192184

repo,

193-

path: toOptionalString(rec.path),

194-

ref: toOptionalString(rec.ref) ?? toOptionalString(rec.branch) ?? toOptionalString(rec.tag),

185+

path: normalizeOptionalString(rec.path),

186+

ref:

187+

normalizeOptionalString(rec.ref) ??

188+

normalizeOptionalString(rec.branch) ??

189+

normalizeOptionalString(rec.tag),

195190

},

196191

};

197192

}

198193199194

if (kind === "git") {

200-

const url = toOptionalString(rec.url) ?? toOptionalString(rec.repo);

195+

const url = normalizeOptionalString(rec.url) ?? normalizeOptionalString(rec.repo);

201196

if (!url) {

202197

return { ok: false, error: 'git source missing "url"' };

203198

}

@@ -206,15 +201,18 @@ function normalizeEntrySource(

206201

source: {

207202

kind: "git",

208203

url,

209-

path: toOptionalString(rec.path),

210-

ref: toOptionalString(rec.ref) ?? toOptionalString(rec.branch) ?? toOptionalString(rec.tag),

204+

path: normalizeOptionalString(rec.path),

205+

ref:

206+

normalizeOptionalString(rec.ref) ??

207+

normalizeOptionalString(rec.branch) ??

208+

normalizeOptionalString(rec.tag),

211209

},

212210

};

213211

}

214212215213

if (kind === "git-subdir") {

216-

const url = toOptionalString(rec.url) ?? toOptionalString(rec.repo);

217-

const sourcePath = toOptionalString(rec.path) ?? toOptionalString(rec.subdir);

214+

const url = normalizeOptionalString(rec.url) ?? normalizeOptionalString(rec.repo);

215+

const sourcePath = normalizeOptionalString(rec.path) ?? normalizeOptionalString(rec.subdir);

218216

if (!url) {

219217

return { ok: false, error: 'git-subdir source missing "url"' };

220218

}

@@ -227,13 +225,16 @@ function normalizeEntrySource(

227225

kind: "git-subdir",

228226

url,

229227

path: sourcePath,

230-

ref: toOptionalString(rec.ref) ?? toOptionalString(rec.branch) ?? toOptionalString(rec.tag),

228+

ref:

229+

normalizeOptionalString(rec.ref) ??

230+

normalizeOptionalString(rec.branch) ??

231+

normalizeOptionalString(rec.tag),

231232

},

232233

};

233234

}

234235235236

if (kind === "url") {

236-

const url = toOptionalString(rec.url);

237+

const url = normalizeOptionalString(rec.url);

237238

if (!url) {

238239

return { ok: false, error: 'url source missing "url"' };

239240

}

@@ -381,7 +382,7 @@ function parseMarketplaceManifest(

381382

return { ok: false, error: `invalid marketplace entry in ${sourceLabel}: expected object` };

382383

}

383384

const plugin = entry as Record<string, unknown>;

384-

const name = toOptionalString(plugin.name);

385+

const name = normalizeOptionalString(plugin.name);

385386

if (!name) {

386387

return { ok: false, error: `invalid marketplace entry in ${sourceLabel}: missing name` };

387388

}

@@ -394,17 +395,17 @@ function parseMarketplaceManifest(

394395

}

395396

plugins.push({

396397

name,

397-

version: toOptionalString(plugin.version),

398-

description: toOptionalString(plugin.description),

398+

version: normalizeOptionalString(plugin.version),

399+

description: normalizeOptionalString(plugin.description),

399400

source: normalizedSource.source,

400401

});

401402

}

402403403404

return {

404405

ok: true,

405406

manifest: {

406-

name: toOptionalString(rec.name),

407-

version: toOptionalString(rec.version),

407+

name: normalizeOptionalString(rec.name),

408+

version: normalizeOptionalString(rec.version),

408409

plugins,

409410

},

410411

};

@@ -430,7 +431,7 @@ async function readClaudeKnownMarketplaces(): Promise<Record<string, KnownMarket

430431

}

431432

const record = value as Record<string, unknown>;

432433

result[name] = {

433-

installLocation: toOptionalString(record.installLocation),

434+

installLocation: normalizeOptionalString(record.installLocation),

434435

source: record.source,

435436

};

436437

}