nyrn
/
openclaw
mirror of https://github.com/openclaw/openclaw.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: dedupe post compaction legacy fallback checks

This commit is contained in:
Peter Steinberger 2026-03-13 22:17:42 +00:00
parent 91c94c8b95
commit 07b3f5233e
1 changed files with 24 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
src/auto-reply/reply/post-compaction-context.test.ts
View File

@ -15,6 +15,28 @@ describe("readPostCompactionContext", () => {
fs.rmSync(tmpDir, { recursive: true, force: true }); fs.rmSync(tmpDir, { recursive: true, force: true });
}); });
async function expectLegacySectionFallback(
postCompactionSections: string[],
expectDefaultProse = false,
) {
const content = `## Every Session\n\nDo startup things.\n\n## Safety\n\nBe safe.\n`;
fs.writeFileSync(path.join(tmpDir, "AGENTS.md"), content);
const cfg = {
agents: {
defaults: {
compaction: { postCompactionSections },
},
},
} as OpenClawConfig;
const result = await readPostCompactionContext(tmpDir, cfg);
expect(result).not.toBeNull();
expect(result).toContain("Do startup things");
expect(result).toContain("Be safe");
if (expectDefaultProse) {
expect(result).toContain("Run your Session Startup sequence");
}
}
it("returns null when no AGENTS.md exists", async () => { it("returns null when no AGENTS.md exists", async () => {
const result = await readPostCompactionContext(tmpDir); const result = await readPostCompactionContext(tmpDir);
expect(result).toBeNull(); expect(result).toBeNull();
@ -339,36 +361,11 @@ Read WORKFLOW.md on startup.
// Older AGENTS.md templates use "Every Session" / "Safety" instead of // Older AGENTS.md templates use "Every Session" / "Safety" instead of
// "Session Startup" / "Red Lines". Explicitly setting the defaults should // "Session Startup" / "Red Lines". Explicitly setting the defaults should
// still trigger the legacy fallback — same behavior as leaving the field unset. // still trigger the legacy fallback — same behavior as leaving the field unset.
const content = `## Every Session\n\nDo startup things.\n\n## Safety\n\nBe safe.\n`; await expectLegacySectionFallback(["Session Startup", "Red Lines"]);
fs.writeFileSync(path.join(tmpDir, "AGENTS.md"), content);
const cfg = {
agents: {
defaults: {
compaction: { postCompactionSections: ["Session Startup", "Red Lines"] },
},
},
} as OpenClawConfig;
const result = await readPostCompactionContext(tmpDir, cfg);
expect(result).not.toBeNull();
expect(result).toContain("Do startup things");
expect(result).toContain("Be safe");
}); });
it("falls back to legacy sections when default sections are configured in a different order", async () => { it("falls back to legacy sections when default sections are configured in a different order", async () => {
const content = `## Every Session\n\nDo startup things.\n\n## Safety\n\nBe safe.\n`; await expectLegacySectionFallback(["Red Lines", "Session Startup"], true);
fs.writeFileSync(path.join(tmpDir, "AGENTS.md"), content);
const cfg = {
agents: {
defaults: {
compaction: { postCompactionSections: ["Red Lines", "Session Startup"] },
},
},
} as OpenClawConfig;
const result = await readPostCompactionContext(tmpDir, cfg);
expect(result).not.toBeNull();
expect(result).toContain("Do startup things");
expect(result).toContain("Be safe");
expect(result).toContain("Run your Session Startup sequence");
}); });
it("custom section names are matched case-insensitively", async () => { it("custom section names are matched case-insensitively", async () => {