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

推荐订阅源

Stack Overflow Blog
Stack Overflow Blog
云风的 BLOG
云风的 BLOG
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
Recent Announcements
Recent Announcements
Microsoft Security Blog
Microsoft Security Blog
Microsoft Azure Blog
Microsoft Azure Blog
J
Java Code Geeks
D
DataBreaches.Net
U
Unit 42
P
Proofpoint News Feed
I
InfoQ
Apple Machine Learning Research
Apple Machine Learning Research
Google DeepMind News
Google DeepMind News
博客园 - Franky
博客园_首页
IT之家
IT之家
博客园 - 叶小钗
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
酷 壳 – CoolShell
酷 壳 – CoolShell
博客园 - 【当耐特】
Hugging Face - Blog
Hugging Face - Blog
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
让小产品的独立变现更简单 - 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(skill-workshop): support live hook enablement · openc...
vincentkoc · 2026-04-23 · via Recent Commits to openclaw:main

@@ -13,10 +13,6 @@ export default definePluginEntry({

1313

description:

1414

"Captures repeatable workflows as workspace skills, with pending review and safe writes.",

1515

register(api) {

16-

const startupConfig = resolveConfig(api.pluginConfig);

17-

if (!startupConfig.enabled) {

18-

return;

19-

}

2016

const resolveCurrentConfig = () => {

2117

const runtimePluginConfig = resolveLivePluginConfigObject(

2218

api.runtime.config?.loadConfig,

@@ -49,97 +45,94 @@ export default definePluginEntry({

4945

};

5046

});

514752-

if (startupConfig.autoCapture && startupConfig.reviewMode !== "off") {

53-

api.on("agent_end", async (event, ctx) => {

54-

const config = resolveCurrentConfig();

55-

if (!config.enabled || !config.autoCapture || config.reviewMode === "off") {

56-

return;

57-

}

58-

if (!event.success) {

59-

return;

60-

}

61-

if (ctx.sessionId?.startsWith("skill-workshop-review-")) {

62-

return;

63-

}

64-

const agentId = ctx.agentId ?? resolveDefaultAgentId(api.config);

65-

const workspaceDir =

66-

ctx.workspaceDir || api.runtime.agent.resolveAgentWorkspaceDir(api.config, agentId);

67-

const store = createStoreForContext({ api, ctx: { ...ctx, workspaceDir }, config });

68-

const heuristicProposal = createProposalFromMessages({

69-

messages: event.messages,

70-

workspaceDir,

71-

agentId,

72-

sessionId: ctx.sessionId,

73-

});

74-

const heuristicEnabled =

75-

config.reviewMode === "heuristic" || config.reviewMode === "hybrid";

76-

if (heuristicEnabled && heuristicProposal) {

77-

try {

78-

const result = await applyOrStoreProposal({

79-

proposal: heuristicProposal,

80-

store,

81-

config,

82-

workspaceDir,

83-

});

84-

if (result.status === "applied") {

85-

api.logger.info(`skill-workshop: applied ${heuristicProposal.skillName}`);

86-

} else if (result.status === "quarantined") {

87-

api.logger.warn(`skill-workshop: quarantined ${heuristicProposal.skillName}`);

88-

} else {

89-

api.logger.info(`skill-workshop: queued ${heuristicProposal.skillName}`);

90-

}

91-

} catch (error) {

92-

api.logger.warn(`skill-workshop: heuristic capture skipped: ${String(error)}`);

93-

}

94-

}

95-96-

const llmEnabled = config.reviewMode === "llm" || config.reviewMode === "hybrid";

97-

if (!llmEnabled) {

98-

return;

99-

}

100-

const reviewState = await store.recordReviewTurn(countToolCalls(event.messages));

101-

const thresholdMet =

102-

reviewState.turnsSinceReview >= config.reviewInterval ||

103-

reviewState.toolCallsSinceReview >= config.reviewMinToolCalls;

104-

const shouldReview =

105-

thresholdMet || (config.reviewMode === "llm" && heuristicProposal !== undefined);

106-

if (!shouldReview) {

107-

return;

108-

}

109-

await store.markReviewed();

48+

api.on("agent_end", async (event, ctx) => {

49+

const config = resolveCurrentConfig();

50+

if (!config.enabled || !config.autoCapture || config.reviewMode === "off") {

51+

return;

52+

}

53+

if (!event.success) {

54+

return;

55+

}

56+

if (ctx.sessionId?.startsWith("skill-workshop-review-")) {

57+

return;

58+

}

59+

const agentId = ctx.agentId ?? resolveDefaultAgentId(api.config);

60+

const workspaceDir =

61+

ctx.workspaceDir || api.runtime.agent.resolveAgentWorkspaceDir(api.config, agentId);

62+

const store = createStoreForContext({ api, ctx: { ...ctx, workspaceDir }, config });

63+

const heuristicProposal = createProposalFromMessages({

64+

messages: event.messages,

65+

workspaceDir,

66+

agentId,

67+

sessionId: ctx.sessionId,

68+

});

69+

const heuristicEnabled = config.reviewMode === "heuristic" || config.reviewMode === "hybrid";

70+

if (heuristicEnabled && heuristicProposal) {

11071

try {

111-

const proposal = await reviewTranscriptForProposal({

112-

api,

72+

const result = await applyOrStoreProposal({

73+

proposal: heuristicProposal,

74+

store,

11375

config,

114-

messages: event.messages,

115-

ctx: {

116-

agentId,

117-

sessionId: ctx.sessionId,

118-

sessionKey: ctx.sessionKey,

119-

workspaceDir,

120-

modelProviderId: ctx.modelProviderId,

121-

modelId: ctx.modelId,

122-

messageProvider: ctx.messageProvider,

123-

channelId: ctx.channelId,

124-

},

76+

workspaceDir,

12577

});

126-

if (!proposal) {

127-

api.logger.debug?.("skill-workshop: reviewer found no update");

128-

return;

129-

}

130-

const result = await applyOrStoreProposal({ proposal, store, config, workspaceDir });

13178

if (result.status === "applied") {

132-

api.logger.info(`skill-workshop: applied ${proposal.skillName}`);

79+

api.logger.info(`skill-workshop: applied ${heuristicProposal.skillName}`);

13380

} else if (result.status === "quarantined") {

134-

api.logger.warn(`skill-workshop: quarantined ${proposal.skillName}`);

81+

api.logger.warn(`skill-workshop: quarantined ${heuristicProposal.skillName}`);

13582

} else {

136-

api.logger.info(`skill-workshop: queued ${proposal.skillName}`);

83+

api.logger.info(`skill-workshop: queued ${heuristicProposal.skillName}`);

13784

}

13885

} catch (error) {

139-

api.logger.warn(`skill-workshop: reviewer skipped: ${String(error)}`);

86+

api.logger.warn(`skill-workshop: heuristic capture skipped: ${String(error)}`);

14087

}

141-

});

142-

}

88+

}

89+90+

const llmEnabled = config.reviewMode === "llm" || config.reviewMode === "hybrid";

91+

if (!llmEnabled) {

92+

return;

93+

}

94+

const reviewState = await store.recordReviewTurn(countToolCalls(event.messages));

95+

const thresholdMet =

96+

reviewState.turnsSinceReview >= config.reviewInterval ||

97+

reviewState.toolCallsSinceReview >= config.reviewMinToolCalls;

98+

const shouldReview =

99+

thresholdMet || (config.reviewMode === "llm" && heuristicProposal !== undefined);

100+

if (!shouldReview) {

101+

return;

102+

}

103+

await store.markReviewed();

104+

try {

105+

const proposal = await reviewTranscriptForProposal({

106+

api,

107+

config,

108+

messages: event.messages,

109+

ctx: {

110+

agentId,

111+

sessionId: ctx.sessionId,

112+

sessionKey: ctx.sessionKey,

113+

workspaceDir,

114+

modelProviderId: ctx.modelProviderId,

115+

modelId: ctx.modelId,

116+

messageProvider: ctx.messageProvider,

117+

channelId: ctx.channelId,

118+

},

119+

});

120+

if (!proposal) {

121+

api.logger.debug?.("skill-workshop: reviewer found no update");

122+

return;

123+

}

124+

const result = await applyOrStoreProposal({ proposal, store, config, workspaceDir });

125+

if (result.status === "applied") {

126+

api.logger.info(`skill-workshop: applied ${proposal.skillName}`);

127+

} else if (result.status === "quarantined") {

128+

api.logger.warn(`skill-workshop: quarantined ${proposal.skillName}`);

129+

} else {

130+

api.logger.info(`skill-workshop: queued ${proposal.skillName}`);

131+

}

132+

} catch (error) {

133+

api.logger.warn(`skill-workshop: reviewer skipped: ${String(error)}`);

134+

}

135+

});

143136

},

144137

});

145138