fix: clear reset skills snapshot (#78873) · openclaw/open...

Recent Commits to openclaw:main

Evizero · 2026-05-07 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -141,6 +141,7 @@ Docs: https://docs.openclaw.ai`
`141`	`141`
`142`	`142`	`### Fixes`
`143`	`143`
	`144`	+- Gateway/sessions: clear cached skills snapshots during `/new` and `sessions.reset` so long-lived channel sessions rebuild the visible skill list after skills change. (#78873) Thanks @Evizero.
`144`	`145`	`- fix(auto-reply): gate inline skill tool dispatch [AI]. (#78517) Thanks @pgondhi987.`
`145`	`146`	- Canvas plugin: keep legacy root `canvasHost` configs valid until `openclaw doctor --fix` migrates them into `plugins.entries.canvas.config.host`, move Canvas/A2UI clients to gateway protocol v4 plugin surfaces, and refresh the generated A2UI bundle hash so normal builds stay clean.
`146`	`147`	`- feishu: honor config write policy for dynamic agents [AI]. (#78520) Thanks @pgondhi987.`

Original file line number	Diff line number	Diff line change
`@@ -729,6 +729,54 @@ describe("initSessionState RawBody", () => {`
`729`	`729`	`expect(result.triggerBodyNormalized).toBe("/NEW KeepThisCase");`
`730`	`730`	`});`
`731`	`731`
	`732`	`+it("drops cached skills snapshot when /new rotates an existing session", async () => {`
	`733`	`+const root = await makeCaseDir("openclaw-rawbody-reset-skills-");`
	`734`	`+const storePath = path.join(root, "sessions.json");`
	`735`	`+const sessionKey = "agent:main:signal:direct:uuid:reset-skills";`
	`736`	`+const existingSessionId = "session-with-stale-skills";`
	`737`	`+`
	`738`	`+await writeSessionStoreFast(storePath, {`
	`739`	`+[sessionKey]: {`
	`740`	`+sessionId: existingSessionId,`
	`741`	`+updatedAt: Date.now(),`
	`742`	`+systemSent: true,`
	`743`	`+skillsSnapshot: {`
	`744`	`+prompt: "<available_skills><skill><name>stale</name></skill></available_skills>",`
	`745`	`+skills: [{ name: "stale" }],`
	`746`	`+version: 0,`
	`747`	`+},`
	`748`	`+},`
	`749`	`+});`
	`750`	`+`
	`751`	`+const cfg = {`
	`752`	`+session: {`
	`753`	`+store: storePath,`
	`754`	`+resetTriggers: ["/new"],`
	`755`	`+},`
	`756`	`+} as OpenClawConfig;`
	`757`	`+`
	`758`	`+const result = await initSessionState({`
	`759`	`+ctx: {`
	`760`	`+RawBody: "/new continue",`
	`761`	`+ChatType: "direct",`
	`762`	`+SessionKey: sessionKey,`
	`763`	`+},`
	`764`	`+ cfg,`
	`765`	`+commandAuthorized: true,`
	`766`	`+});`
	`767`	`+`
	`768`	`+expect(result.isNewSession).toBe(true);`
	`769`	`+expect(result.resetTriggered).toBe(true);`
	`770`	`+expect(result.sessionId).not.toBe(existingSessionId);`
	`771`	`+expect(result.sessionEntry.skillsSnapshot).toBeUndefined();`
	`772`	`+`
	`773`	`+const store = JSON.parse(await fs.readFile(storePath, "utf-8")) as Record<`
	`774`	`+string,`
	`775`	`+{ skillsSnapshot?: unknown }`
	`776`	`+>;`
	`777`	`+expect(store[sessionKey]?.skillsSnapshot).toBeUndefined();`
	`778`	`+});`
	`779`	`+`
`732`	`780`	`it("drains stale system events when /new rotates an existing session", async () => {`
`733`	`781`	`const root = await makeCaseDir("openclaw-rawbody-reset-system-events-");`
`734`	`782`	`const storePath = path.join(root, "sessions.json");`

Original file line number	Diff line number	Diff line change
`@@ -768,6 +768,9 @@ export async function initSessionState(params: {`
`768`	`768`	`sessionEntry.outputTokens = undefined;`
`769`	`769`	`sessionEntry.estimatedCostUsd = undefined;`
`770`	`770`	`sessionEntry.contextTokens = undefined;`
	`771`	`+// Skills snapshots are prompt/runtime caches. Do not preserve a stale`
	`772`	`+// snapshot through /new; the next turn must rebuild the visible skill list.`
	`773`	`+sessionEntry.skillsSnapshot = undefined;`
`771`	`774`	`}`
`772`	`775`	`// Preserve per-session overrides while resetting compaction state on /new.`
`773`	`776`	`sessionStore[sessionKey] = { ...sessionStore[sessionKey], ...sessionEntry };`

Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,46 @@ test("sessions.reset recomputes model from defaults instead of stale runtime mod`
`50`	`50`	`await expect(fs.stat(reset.payload?.entry.sessionFile as string)).resolves.toBeTruthy();`
`51`	`51`	`});`
`52`	`52`
	`53`	`+test("sessions.reset drops cached skills snapshot so /new rebuilds visible skills", async () => {`
	`54`	`+const { storePath } = await createSessionStoreDir();`
	`55`	`+testState.agentConfig = {`
	`56`	`+model: {`
	`57`	`+primary: "openai/gpt-test-a",`
	`58`	`+},`
	`59`	`+};`
	`60`	`+`
	`61`	`+await writeSessionStore({`
	`62`	`+entries: {`
	`63`	`+main: sessionStoreEntry("sess-stale-skills", {`
	`64`	`+skillsSnapshot: {`
	`65`	`+prompt: "<available_skills><skill><name>stale</name></skill></available_skills>",`
	`66`	`+skills: [{ name: "stale" }],`
	`67`	`+version: 0,`
	`68`	`+},`
	`69`	`+}),`
	`70`	`+},`
	`71`	`+});`
	`72`	`+`
	`73`	`+const reset = await directSessionReq<{`
	`74`	`+ok: true;`
	`75`	`+key: string;`
	`76`	`+entry: {`
	`77`	`+sessionId: string;`
	`78`	`+skillsSnapshot?: unknown;`
	`79`	`+};`
	`80`	`+}>("sessions.reset", { key: "main" });`
	`81`	`+`
	`82`	`+expect(reset.ok).toBe(true);`
	`83`	`+expect(reset.payload?.entry.sessionId).not.toBe("sess-stale-skills");`
	`84`	`+expect(reset.payload?.entry.skillsSnapshot).toBeUndefined();`
	`85`	`+`
	`86`	`+const store = JSON.parse(await fs.readFile(storePath, "utf-8")) as Record<`
	`87`	`+string,`
	`88`	`+{ skillsSnapshot?: unknown }`
	`89`	`+>;`
	`90`	`+expect(store["agent:main:main"]?.skillsSnapshot).toBeUndefined();`
	`91`	`+});`
	`92`	`+`
`53`	`93`	`test("sessions.reset preserves legacy explicit model overrides without modelOverrideSource", async () => {`
`54`	`94`	`const { storePath } = await createSessionStoreDir();`
`55`	`95`	`testState.agentConfig = {`

Original file line number	Diff line number	Diff line change
`@@ -623,7 +623,10 @@ export async function performGatewaySessionReset(params: {`
`623`	`623`	`lastTo: currentEntry?.lastTo,`
`624`	`624`	`lastAccountId: currentEntry?.lastAccountId,`
`625`	`625`	`lastThreadId: currentEntry?.lastThreadId,`
`626`		`-skillsSnapshot: currentEntry?.skillsSnapshot,`
	`626`	`+// Do not carry the cached skills catalog across /new. Long-lived channel`
	`627`	`+// sessions (Signal DMs/groups in particular) otherwise keep advertising a`
	`628`	`+// stale <available_skills> block even after reset/restart, because the`
	`629`	`+// skills snapshot version is runtime-local and may reset to 0.`
`627`	`630`	`acp: currentEntry?.acp,`
`628`	`631`	`inputTokens: 0,`
`629`	`632`	`outputTokens: 0,`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main