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

推荐订阅源

G
Google Developers Blog
博客园 - 司徒正美
Last Week in AI
Last Week in AI
Recent Announcements
Recent Announcements
Y
Y Combinator Blog
博客园 - 聂微东
M
MIT News - Artificial intelligence
博客园_首页
Jina AI
Jina AI
博客园 - 叶小钗
酷 壳 – CoolShell
酷 壳 – CoolShell
H
Hackread – Cybersecurity News, Data Breaches, AI and More
J
Java Code Geeks
F
Fortinet All Blogs
aimingoo的专栏
aimingoo的专栏
小众软件
小众软件
Vercel News
Vercel News
The Cloudflare Blog
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
云风的 BLOG
云风的 BLOG
N
Netflix TechBlog - Medium
B
Blog
Google DeepMind News
Google DeepMind News
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More

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(plugins): warn on invalid install default choice (#71...
vincentkoc · 2026-04-24 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

@@ -9,6 +9,7 @@ Docs: https://docs.openclaw.ai

99

- Plugins/activation: expose activation plan reasons and a richer plan API so callers can inspect why a plugin was selected while preserving existing id-list activation behavior. (#70943) Thanks @vincentkoc.

1010

- Plugins/source metadata: expose normalized install-source facts on provider and channel catalogs so onboarding can explain npm pinning, integrity state, and local availability before runtime loads. (#70951) Thanks @vincentkoc.

1111

- Plugins/catalog: pin the official external WeCom channel source to an exact npm release plus dist integrity, with a guard that official external sources stay integrity-pinned. (#70997) Thanks @vincentkoc.

12+

- Plugins/source metadata: warn when `openclaw.install.defaultChoice` is invalid or points at a missing source, keeping catalog diagnostics explicit without breaking existing plugins. Thanks @vincentkoc.

1213

- Diagnostics/OTEL: add a lightweight diagnostic trace-context carrier for future span correlation without adding OTEL SDK state to core. Thanks @vincentkoc.

1314

- Diagnostics/OTEL: attach diagnostic trace context to exported OTEL logs so log records can correlate with future spans without adding retained process state. Thanks @vincentkoc.

1415

- Diagnostics/OTEL: pass immutable per-run diagnostic trace context through agent and tool hook contexts, and parent exported diagnostic spans from validated context without retaining global trace state. Thanks @vincentkoc.

Original file line numberDiff line numberDiff line change

@@ -888,10 +888,11 @@ Generated channel catalog entries and provider install catalog entries expose

888888

normalized install-source facts next to the raw `openclaw.install` block. The

889889

normalized facts identify whether the npm spec is an exact version or floating

890890

selector, whether expected integrity metadata is present, and whether a local

891-

source path is also available. Consumers should treat `installSource` as an

892-

additive optional field so older hand-built entries and compatibility shims do

893-

not have to synthesize it. This lets onboarding and diagnostics explain

894-

source-plane state without importing plugin runtime.

891+

source path is also available. They also warn when `defaultChoice` is invalid

892+

or points at a source that is not available. Consumers should treat

893+

`installSource` as an additive optional field so older hand-built entries and

894+

compatibility shims do not have to synthesize it. This lets onboarding and

895+

diagnostics explain source-plane state without importing plugin runtime.

895896
896897

Official external npm entries should prefer an exact `npmSpec` plus

897898

`expectedIntegrity`. Bare package names and dist-tags still work for

Original file line numberDiff line numberDiff line change

@@ -596,9 +596,10 @@ entries should pair exact specs with `expectedIntegrity` so update flows fail

596596

closed if the fetched npm artifact no longer matches the pinned release.

597597

Interactive onboarding still offers trusted registry npm specs, including bare

598598

package names and dist-tags, for compatibility. Catalog diagnostics can

599-

distinguish exact, floating, integrity-pinned, and missing-integrity sources.

600-

When `expectedIntegrity` is present, install/update flows enforce it; when it

601-

is omitted, the registry resolution is recorded without an integrity pin.

599+

distinguish exact, floating, integrity-pinned, missing-integrity, and invalid

600+

default-choice sources. When `expectedIntegrity` is present, install/update

601+

flows enforce it; when it is omitted, the registry resolution is recorded

602+

without an integrity pin.

602603
603604

Channel plugins should provide `openclaw.setupEntry` when status, channel list,

604605

or SecretRef scans need to identify configured accounts without loading the full

Original file line numberDiff line numberDiff line change

@@ -131,4 +131,50 @@ describe("describePluginInstallSource", () => {

131131

warnings: ["invalid-npm-spec"],

132132

});

133133

});

134+
135+

it("warns when defaultChoice is not a supported install source", () => {

136+

expect(

137+

describePluginInstallSource({

138+

npmSpec: "@vendor/demo@1.2.3",

139+

defaultChoice: "registry",

140+

} as never),

141+

).toEqual({

142+

npm: {

143+

spec: "@vendor/demo@1.2.3",

144+

packageName: "@vendor/demo",

145+

selector: "1.2.3",

146+

selectorKind: "exact-version",

147+

exactVersion: true,

148+

pinState: "exact-without-integrity",

149+

},

150+

warnings: ["invalid-default-choice", "npm-spec-missing-integrity"],

151+

});

152+

});

153+
154+

it("warns when defaultChoice points at a missing source", () => {

155+

expect(

156+

describePluginInstallSource({

157+

localPath: "extensions/demo",

158+

defaultChoice: "npm",

159+

}),

160+

).toEqual({

161+

defaultChoice: "npm",

162+

local: {

163+

path: "extensions/demo",

164+

},

165+

warnings: ["default-choice-missing-source"],

166+

});

167+

});

168+
169+

it("warns when defaultChoice points at an invalid npm source", () => {

170+

expect(

171+

describePluginInstallSource({

172+

npmSpec: "github:vendor/demo",

173+

defaultChoice: "npm",

174+

}),

175+

).toEqual({

176+

defaultChoice: "npm",

177+

warnings: ["invalid-npm-spec", "default-choice-missing-source"],

178+

});

179+

});

134180

});

Original file line numberDiff line numberDiff line change

@@ -4,6 +4,8 @@ import type { PluginPackageInstall } from "./manifest.js";

44
55

export type PluginInstallSourceWarning =

66

| "invalid-npm-spec"

7+

| "invalid-default-choice"

8+

| "default-choice-missing-source"

79

| "npm-spec-floating"

810

| "npm-spec-missing-integrity";

911

@@ -44,18 +46,23 @@ function resolveNpmPinState(params: {

4446

return params.hasIntegrity ? "floating-with-integrity" : "floating-without-integrity";

4547

}

4648
49+

function resolveDefaultChoice(value: unknown): PluginPackageInstall["defaultChoice"] | undefined {

50+

return value === "npm" || value === "local" ? value : undefined;

51+

}

52+
4753

export function describePluginInstallSource(

4854

install: PluginPackageInstall,

4955

): PluginInstallSourceInfo {

5056

const npmSpec = normalizeOptionalString(install.npmSpec);

5157

const localPath = normalizeOptionalString(install.localPath);

52-

const defaultChoice =

53-

install.defaultChoice === "npm" || install.defaultChoice === "local"

54-

? install.defaultChoice

55-

: undefined;

58+

const defaultChoice = resolveDefaultChoice(install.defaultChoice);

5659

const warnings: PluginInstallSourceWarning[] = [];

5760

let npm: PluginInstallNpmSourceInfo | undefined;

5861
62+

if (install.defaultChoice !== undefined && !defaultChoice) {

63+

warnings.push("invalid-default-choice");

64+

}

65+
5966

if (npmSpec) {

6067

const parsed = parseRegistryNpmSpec(npmSpec);

6168

if (parsed) {

@@ -81,6 +88,12 @@ export function describePluginInstallSource(

8188

warnings.push("invalid-npm-spec");

8289

}

8390

}

91+

if (defaultChoice === "npm" && !npm) {

92+

warnings.push("default-choice-missing-source");

93+

}

94+

if (defaultChoice === "local" && !localPath) {

95+

warnings.push("default-choice-missing-source");

96+

}

8497
8598

return {

8699

...(defaultChoice ? { defaultChoice } : {}),