Fimeg/letta-code
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

chore: Remove agent-info shared reminder for subagents [LET-7974] (#1368)

Browse Source
This commit is contained in:
Devansh Jain
2026-03-12 14:40:27 -07:00
committed by GitHub
parent 8ce5b0ed76
commit 38d5a5b8c2
3 changed files with 6 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/reminders/catalog.ts
View File

@@ -32,12 +32,7 @@ export const SHARED_REMINDER_CATALOG: ReadonlyArray<SharedReminderDefinition> =
{ {
id: "agent-info", id: "agent-info",
description: "Agent identity (ID, name, server, memory dir)", description: "Agent identity (ID, name, server, memory dir)",
modes: [ modes: ["interactive", "headless-one-shot", "headless-bidirectional"],
"interactive",
"headless-one-shot",
"headless-bidirectional",
"subagent",
],
}, },
{ {
id: "permission-mode", id: "permission-mode",

4
src/tests/reminders/catalog.test.ts
View File

@@ -30,11 +30,11 @@ describe("shared reminder catalog", () => {
} }
}); });
test("subagent mode only has agent-info reminder", () => { test("subagent mode has no shared reminders", () => {
const subagentReminders = SHARED_REMINDER_CATALOG.filter((entry) => const subagentReminders = SHARED_REMINDER_CATALOG.filter((entry) =>
entry.modes.includes("subagent"), entry.modes.includes("subagent"),
); );
expect(subagentReminders.map((entry) => entry.id)).toEqual(["agent-info"]); expect(subagentReminders.map((entry) => entry.id)).toEqual([]);
}); });
test("command and toolset reminders are interactive-only", () => { test("command and toolset reminders are interactive-only", () => {

6
src/tests/reminders/engine-parity.test.ts
View File

@@ -88,7 +88,7 @@ describe("shared reminder parity", () => {
); );
}); });
test("subagent mode produces only agent-info reminder", async () => { test("subagent mode produces no shared reminders", async () => {
for (const reminderId of SHARED_REMINDER_IDS) { for (const reminderId of SHARED_REMINDER_IDS) {
providerMap[reminderId] = async () => reminderId; providerMap[reminderId] = async () => reminderId;
} }
@@ -114,7 +114,7 @@ describe("shared reminder parity", () => {
}); });
expect(subagent.appliedReminderIds).toEqual(reminderIdsForMode("subagent")); expect(subagent.appliedReminderIds).toEqual(reminderIdsForMode("subagent"));
expect(subagent.appliedReminderIds).toEqual(["agent-info"]); expect(subagent.appliedReminderIds).toEqual([]);
expect(subagent.parts.map((part) => part.text)).toEqual(["agent-info"]); expect(subagent.parts.map((part) => part.text)).toEqual([]);
}); });
}); });