From fa7001aa8f255a0a2d6c8a611b0742f1034f1483 Mon Sep 17 00:00:00 2001
From: edithatogo <15080672+edithatogo@users.noreply.github.com>
Date: Sun, 6 Sep 2026 21:20:12 +0000
Subject: [PATCH] chore: replace console.log with application logger in
 browser-workflow.ts

---
 src/publish/browser-workflow.ts | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/src/publish/browser-workflow.ts b/src/publish/browser-workflow.ts
index b371f85..7ba1d10 100644
--- a/src/publish/browser-workflow.ts
+++ b/src/publish/browser-workflow.ts
@@ -1,3 +1,4 @@
+import { rootLogger } from "../util/logger.js";
 import { mkdir, writeFile } from "node:fs/promises";
 import { dirname } from "node:path";
 import { createStoredSession, loadSession, saveSession } from "../auth/session-store.js";
@@ -26,6 +27,8 @@ import {
   type TransportResolution,
 } from "./transport.js";
 
+const log = rootLogger.child({ component: "browser-workflow" });
+
 export interface WorkflowStep {
   name: string;
   status: "ok" | "error";
@@ -127,13 +130,13 @@ export async function runBrowserWorkflow(
       );
       await maybeWriteTrace(result, options.traceOut);
       await maybeWriteWorkflowRunLog(options, requirePublicationUrl(config), prepared, result);
-      console.log(JSON.stringify(result, null, 2));
+      log.info(result);
     } catch (error) {
       if (error instanceof LocalWorkflowError) {
         const result = buildFailedWorkflowResult(error, prepared, transport);
         await maybeWriteTrace(result, options.traceOut);
         await maybeWriteWorkflowRunLog(options, requirePublicationUrl(config), prepared, result);
-        console.error(JSON.stringify(result, null, 2));
+        log.error(result);
       }
       throw error;
     }
@@ -163,7 +166,7 @@ export async function runBrowserWorkflow(
       const result = await createDraftInBrowser(session, prepared, options, transport);
       await maybeWriteTrace(result, options.traceOut);
       await maybeWriteWorkflowRunLog(options, session.publicationUrl, prepared, result);
-      console.log(JSON.stringify(result, null, 2));
+      log.info(result);
     } catch (error) {
       if (error instanceof BrowserWorkflowError) {
         const result = {
@@ -176,7 +179,7 @@ export async function runBrowserWorkflow(
         };
         await maybeWriteTrace(result, options.traceOut);
         await maybeWriteWorkflowRunLog(options, session.publicationUrl, prepared, result);
-        console.error(JSON.stringify(result, null, 2));
+        log.error(result);
       }
 
       throw error;
@@ -291,7 +294,7 @@ async function createDraftInBrowser(
   );
 
   if (options.experimentalInjectState) {
-    console.warn(
+    log.warn(
       "Experimental editor-state injection is planned but not yet implemented. Falling back to the paste-based default path.",
     );
   }
@@ -447,7 +450,7 @@ async function createDraftInBrowser(
     const url = await session.page.url();
     const isReviewUrl = /\/publish\//.test(url) || /\/post\//.test(url);
     if (!isReviewUrl) {
-      console.warn(
+      log.warn(
         `Warning: Current URL "${url}" does not look like a publish review screen. Expected URL containing "/publish/" or "/post/".`,
       );
     }