fix: clean up slack threading landings (#1360) (thanks @SocialNerd42069)

src/agents/pi-embedded-subscribe.subscribe-embedded-pi-session.does-not-emit-duplicate-block-replies-text.test.ts

@@ -113,7 +113,7 @@ describe("subscribeEmbeddedPiSession", () => {
     handler?.({ type: "message_end", message: assistantMessageWithNewline });
     handler?.({ type: "message_end", message: assistantMessageTrimmed });
 
-    expect(subscription.assistantTexts).toEqual(["Hello world\n"]);
+    expect(subscription.assistantTexts).toEqual(["Hello world"]);
   });
   it("does not duplicate assistantTexts when message_end repeats with reasoning blocks", () => {
     let handler: SessionEventHandler | undefined;

src/cli/update-cli.ts

@@ -17,6 +17,7 @@ import {
   runGatewayUpdate,
   type UpdateRunResult,
   type UpdateStepInfo,
+  type UpdateStepResult,
   type UpdateStepProgress,
   type UpdateStepResult,
 } from "../infra/update-runner.js";

src/slack/monitor/message-handler/prepare.ts

@@ -481,7 +481,6 @@ export async function prepareSlackMessage(params: {
     ParentSessionKey: threadKeys.parentSessionKey,
     ThreadStarterBody: threadStarterBody,
     ThreadLabel: threadLabel,
-    MessageThreadId: isThreadReply ? threadTs : undefined,
     Timestamp: message.ts ? Math.round(Number(message.ts) * 1000) : undefined,
     WasMentioned: isRoomish ? effectiveWasMentioned : undefined,
     MediaPath: media?.path,