fix(channels): remove unusable post-final progress override

vincentkoc · vincentkoc · commit a594d2ce7325 · 2026-06-22T12:06:42.000+08:00
diff --git a/src/channels/progress-draft-compositor.test.ts b/src/channels/progress-draft-compositor.test.ts
@@ -148,28 +148,6 @@ describe("createChannelProgressDraftCompositor", () => {
     expect(update).not.toHaveBeenCalled();
   });
 
-  it("allows explicit post-final progress notices", async () => {
-    const update = vi.fn();
-    const progress = createChannelProgressDraftCompositor({
-      entry: { streaming: { mode: "progress", progress: { label: "Shelling" } } },
-      mode: "progress",
-      active: true,
-      seed: "test",
-      update,
-    });
-
-    progress.markFinalReplyDelivered();
-
-    expect(await progress.pushToolProgress("🛠️ Hidden", { startImmediately: true })).toBe(false);
-    expect(
-      await progress.pushToolProgress("💨Fast: auto-on", {
-        startImmediately: true,
-        allowAfterFinal: true,
-      }),
-    ).toBe(true);
-    expect(update).toHaveBeenCalledWith("Shelling\n\n💨Fast: auto-on", { flush: true });
-  });
-
   it("composes reasoning deltas with tool progress", async () => {
     const update = vi.fn();
     const progress = createChannelProgressDraftCompositor({
diff --git a/src/channels/progress-draft-compositor.ts b/src/channels/progress-draft-compositor.ts
@@ -119,12 +119,9 @@ export function createChannelProgressDraftCompositor(params: {
 
   const noteProgress = async (
     line?: ChannelProgressDraftCompositorLine,
-    options?: { toolName?: string; startImmediately?: boolean; allowAfterFinal?: boolean },
+    options?: { toolName?: string; startImmediately?: boolean },
   ) => {
-    if (
-      !params.active ||
-      ((finalReplyStarted || finalReplyDelivered) && !options?.allowAfterFinal)
-    ) {
+    if (!params.active || finalReplyStarted || finalReplyDelivered) {
       return false;
     }
     if (options?.toolName !== undefined && !isChannelProgressDraftWorkToolName(options.toolName)) {
