From 38d5a5b8c2f2a74a5a77e995fd29683bc0d4b44e Mon Sep 17 00:00:00 2001
From: Devansh Jain <31609257+devanshrj@users.noreply.github.com>
Date: Thu, 12 Mar 2026 14:40:27 -0700
Subject: [PATCH] chore: Remove agent-info shared reminder for subagents
 [LET-7974] (#1368)

---
 src/reminders/catalog.ts                  | 7 +------
 src/tests/reminders/catalog.test.ts       | 4 ++--
 src/tests/reminders/engine-parity.test.ts | 6 +++---
 3 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/src/reminders/catalog.ts b/src/reminders/catalog.ts
index 8e69c65..ff71091 100644
--- a/src/reminders/catalog.ts
+++ b/src/reminders/catalog.ts
@@ -32,12 +32,7 @@ export const SHARED_REMINDER_CATALOG: ReadonlyArray<SharedReminderDefinition> =
     {
       id: "agent-info",
       description: "Agent identity (ID, name, server, memory dir)",
-      modes: [
-        "interactive",
-        "headless-one-shot",
-        "headless-bidirectional",
-        "subagent",
-      ],
+      modes: ["interactive", "headless-one-shot", "headless-bidirectional"],
     },
     {
       id: "permission-mode",
diff --git a/src/tests/reminders/catalog.test.ts b/src/tests/reminders/catalog.test.ts
index 9fa24d8..b105cd4 100644
--- a/src/tests/reminders/catalog.test.ts
+++ b/src/tests/reminders/catalog.test.ts
@@ -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) =>
       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", () => {
diff --git a/src/tests/reminders/engine-parity.test.ts b/src/tests/reminders/engine-parity.test.ts
index 75fc567..a889473 100644
--- a/src/tests/reminders/engine-parity.test.ts
+++ b/src/tests/reminders/engine-parity.test.ts
@@ -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) {
       providerMap[reminderId] = async () => reminderId;
     }
@@ -114,7 +114,7 @@ describe("shared reminder parity", () => {
     });
 
     expect(subagent.appliedReminderIds).toEqual(reminderIdsForMode("subagent"));
-    expect(subagent.appliedReminderIds).toEqual(["agent-info"]);
-    expect(subagent.parts.map((part) => part.text)).toEqual(["agent-info"]);
+    expect(subagent.appliedReminderIds).toEqual([]);
+    expect(subagent.parts.map((part) => part.text)).toEqual([]);
   });
 });