feat(slack): show thinking level in reply footer (#240)

Show the per-turn thinking level in the finalized Slack reply footer
instead of the active trace ID, and carry that footer through both
normal reply execution and resumed reply delivery.

The footer is a user-visible artifact, so the selected reasoning bucket
is more useful than an internal tracing identifier. This update also
tightens the footer helper to the existing thinking-level contract and
aligns the Slack footer specs with the metadata we actually render.

I considered leaving the stale trace-oriented assertions and broader
footer input in place, but that would keep the tests and public contract
looser than the intended behavior. The follow-up test changes here are
narrowly scoped to environment and module-state isolation around the
touched suites.

---------

Co-authored-by: OpenAI Codex <codex@openai.com>
Co-authored-by: David Cramer <dcramer@gmail.com>
Co-authored-by: GPT-5 <noreply@openai.com>

Author: 3 people

packages/junior/src/chat/runtime/reply-executor.ts

@@ -3,7 +3,6 @@ import type { SlackAdapter } from "@chat-adapter/slack";
 import { botConfig } from "@/chat/config";
 import { getSlackMessageTs } from "@/chat/slack/message";
 import {
-  getActiveTraceId,
   logException,
   logInfo,
   logWarn,
@@ -447,7 +446,7 @@ export function createReplyToThread(deps: ReplyExecutorDeps) {
           const replyFooter = buildSlackReplyFooter({
             conversationId,
             durationMs: reply.diagnostics.durationMs,
-            traceId: getActiveTraceId(),
+            thinkingLevel: reply.diagnostics.thinkingLevel,
             usage: reply.diagnostics.usage,
           });
           const shouldUseSlackFooter =

packages/junior/src/chat/slack/footer.ts

@@ -1,3 +1,4 @@
+import type { TurnThinkingSelection } from "@/chat/services/turn-thinking-level";
 import type { AgentTurnUsage } from "@/chat/usage";
 
 interface SlackMrkdwnTextObject {
@@ -17,7 +18,7 @@ interface SlackContextBlock {
 
 export type SlackMessageBlock = SlackSectionBlock | SlackContextBlock;
 
-export interface SlackReplyFooterItem {
+interface SlackReplyFooterItem {
   label: string;
   value: string;
 }
@@ -83,7 +84,7 @@ function resolveTotalTokens(
 export function buildSlackReplyFooter(args: {
   conversationId?: string;
   durationMs?: number;
-  traceId?: string;
+  thinkingLevel?: TurnThinkingSelection["thinkingLevel"];
   usage?: AgentTurnUsage;
 }): SlackReplyFooter | undefined {
   const items: SlackReplyFooterItem[] = [];
@@ -112,11 +113,10 @@ export function buildSlackReplyFooter(args: {
     });
   }
 
-  const traceId = args.traceId?.trim();
-  if (traceId) {
+  if (args.thinkingLevel) {
     items.push({
-      label: "Trace",
-      value: traceId,
+      label: "Thinking",
+      value: args.thinkingLevel,
     });
   }
 

packages/junior/src/chat/slack/resume.ts

@@ -1,6 +1,5 @@
 import { botConfig } from "@/chat/config";
 import type { ChannelConfigurationService } from "@/chat/configuration/types";
-import { getActiveTraceId } from "@/chat/logging";
 import {
   generateAssistantReply,
   type AssistantReply,
@@ -226,7 +225,7 @@ export async function resumeSlackTurn(args: ResumeSlackTurnArgs) {
     const footer = buildSlackReplyFooter({
       conversationId: args.replyContext?.correlation?.conversationId ?? lockKey,
       durationMs: reply.diagnostics.durationMs,
-      traceId: getActiveTraceId(),
+      thinkingLevel: reply.diagnostics.thinkingLevel,
       usage: reply.diagnostics.usage,
     });
     await postSlackApiReplyPosts({

packages/junior/tests/integration/example-build-discovery.test.ts

@@ -25,6 +25,7 @@ function getExamplePluginPackages(): string[] {
 function buildJuniorPackage(): void {
   const env: NodeJS.ProcessEnv = {
     ...process.env,
+    CI: "true",
     JUNIOR_SKIP_SNAPSHOT: "1",
   };
   delete env.JUNIOR_EXTRA_PLUGIN_ROOTS;

packages/junior/tests/integration/slack/attachment-media-behavior.test.ts

@@ -15,6 +15,7 @@ async function createRuntime(
 ) {
   process.env = {
     ...ORIGINAL_ENV,
+    AI_VISION_MODEL: "",
     SLACK_BOT_TOKEN: "",
     SLACK_BOT_USER_TOKEN: "",
     ...env,

packages/junior/tests/integration/slack/bot-image-hydration.test.ts

@@ -16,6 +16,7 @@ async function createRuntime(
 ) {
   process.env = {
     ...ORIGINAL_ENV,
+    AI_VISION_MODEL: "",
     SLACK_BOT_TOKEN: "",
     SLACK_BOT_USER_TOKEN: "",
     ...env,

packages/junior/tests/integration/slack/message-changed-reply-contract.test.ts

@@ -165,10 +165,6 @@ describe("Slack contract: edited-message reply delivery", () => {
                     "*ID:* slack:D12345:1700000100.000100",
                   ),
                 }),
-                expect.objectContaining({
-                  type: "mrkdwn",
-                  text: expect.stringContaining("*Trace:* "),
-                }),
               ]),
             },
           ],

packages/junior/tests/integration/slack/outbound-normalization-contract.test.ts

@@ -44,7 +44,7 @@ describe("Slack contract: outbound normalization", () => {
   it("passes block payloads with a top-level fallback text", async () => {
     const footer = buildSlackReplyFooter({
       conversationId: "slack:C123:1700000000.000100",
-      traceId: "trace_123",
+      thinkingLevel: "low",
     });
 
     await postSlackMessage({
@@ -75,7 +75,7 @@ describe("Slack contract: outbound normalization", () => {
                 },
                 {
                   type: "mrkdwn",
-                  text: "*Trace:* trace_123",
+                  text: "*Thinking:* low",
                 },
               ],
             },

packages/junior/tests/unit/services/turn-checkpoint.test.ts

@@ -1,24 +1,32 @@
-import { afterEach, beforeEach, describe, expect, it } from "vitest";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import type { AgentMessage } from "@mariozechner/pi-agent-core";
-import { persistAuthPauseCheckpoint } from "@/chat/services/turn-checkpoint";
-import { disconnectStateAdapter } from "@/chat/state/adapter";
-import {
-  getAgentTurnSessionCheckpoint,
-  upsertAgentTurnSessionCheckpoint,
-} from "@/chat/state/turn-session-store";
+
+const ORIGINAL_ENV = { ...process.env };
 
 describe("persistAuthPauseCheckpoint", () => {
   beforeEach(async () => {
-    process.env.JUNIOR_STATE_ADAPTER = "memory";
+    process.env = {
+      ...ORIGINAL_ENV,
+      JUNIOR_STATE_ADAPTER: "memory",
+    };
+    vi.resetModules();
+    const { disconnectStateAdapter } = await import("@/chat/state/adapter");
     await disconnectStateAdapter();
   });
 
   afterEach(async () => {
+    const { disconnectStateAdapter } = await import("@/chat/state/adapter");
     await disconnectStateAdapter();
-    delete process.env.JUNIOR_STATE_ADAPTER;
+    vi.resetModules();
+    process.env = { ...ORIGINAL_ENV };
   });
 
   it("reuses the latest stored transcript when the auth pause captured no messages", async () => {
+    const { persistAuthPauseCheckpoint } =
+      await import("@/chat/services/turn-checkpoint");
+    const { getAgentTurnSessionCheckpoint, upsertAgentTurnSessionCheckpoint } =
+      await import("@/chat/state/turn-session-store");
+
     const priorMessages: AgentMessage[] = [
       {
         role: "user",

packages/junior/tests/unit/slack/footer.test.ts

@@ -10,7 +10,7 @@ describe("buildSlackReplyFooter", () => {
       buildSlackReplyFooter({
         conversationId: "slack:C123:1700000000.000100",
         durationMs: 842,
-        traceId: "0123456789abcdef0123456789abcdef",
+        thinkingLevel: "medium",
         usage: {
           totalTokens: 1234,
         },
@@ -30,8 +30,8 @@ describe("buildSlackReplyFooter", () => {
           value: "842ms",
         },
         {
-          label: "Trace",
-          value: "0123456789abcdef0123456789abcdef",
+          label: "Thinking",
+          value: "medium",
         },
       ],
     });
@@ -83,7 +83,7 @@ describe("buildSlackReplyBlocks", () => {
     const footer = buildSlackReplyFooter({
       conversationId: "slack:C123:1700000000.000100",
       durationMs: 1250,
-      traceId: "trace_123",
+      thinkingLevel: "high",
       usage: {
         inputTokens: 400,
         outputTokens: 250,
@@ -115,7 +115,7 @@ describe("buildSlackReplyBlocks", () => {
           },
           {
             type: "mrkdwn",
-            text: "*Trace:* trace_123",
+            text: "*Thinking:* high",
           },
         ],
       },