@ryanfw/prompt-orchestration-pipeline 0.15.0 → 0.15.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryanfw/prompt-orchestration-pipeline",
-  "version": "0.15.0",
+  "version": "0.15.1",
   "description": "A Prompt-orchestration pipeline (POP) is a framework for building, running, and experimenting with complex chains of LLM tasks.",
   "type": "module",
   "main": "src/ui/server.js",

package/src/core/orchestrator.js

@@ -123,10 +123,8 @@ export async function startOrchestrator(opts) {
     } catch {}
 
     // Move seed to current/{jobId}/seed.json
-    logger.log("Moving file", { from: filePath, to: dest });
     try {
       await moveFile(filePath, dest);
-      logger.log("Successfully moved file", { destination: dest });
     } catch (error) {
       logger.error("Failed to move file", {
         from: filePath,
@@ -173,12 +171,6 @@ export async function startOrchestrator(opts) {
         // Apply artifact initialization to the status
         const updatedStatus = applyArtifacts(status);
         await fs.writeFile(statusPath, JSON.stringify(updatedStatus, null, 2));
-
-        logger.log("Initialized status from upload artifacts", {
-          jobId,
-          pipeline: seed?.pipeline,
-          artifactsCount: updatedStatus.files?.artifacts?.length || 0,
-        });
       } catch (artifactError) {
         // Don't fail job startup if artifact initialization fails, just log
         logger.warn("Failed to initialize status from artifacts", {
@@ -233,7 +225,6 @@ export async function startOrchestrator(opts) {
 
   // Watch pending directory for seeds
   const watchPattern = path.join(dirs.pending, "*.json");
-  logger.log("Watching pattern", { pattern: watchPattern });
   const watcher = watcherFactory(watchPattern, {
     ignoreInitial: false,
     awaitWriteFinish: false, // Disable awaitWriteFinish for faster detection
@@ -243,7 +234,6 @@ export async function startOrchestrator(opts) {
   // Wait for watcher to be ready before resolving
   await new Promise((resolve, reject) => {
     watcher.on("ready", () => {
-      logger.log("Watcher is ready");
       resolve();
     });
 
@@ -254,7 +244,6 @@ export async function startOrchestrator(opts) {
   });
 
   watcher.on("add", (file) => {
-    logger.log("Detected file add", { file });
     // Return promise so tests awaiting the add handler block until processing completes
     return handleSeedAdd(file);
   });
@@ -330,13 +319,6 @@ function spawnRunner(
     const availablePipelines = Object.keys(configSnapshot?.pipelines ?? {});
     const pipelineSlug = seed?.pipeline;
 
-    logger.log("spawnRunner invoked", {
-      jobId,
-      pipelineSlug: pipelineSlug ?? null,
-      availablePipelines,
-      seedKeys: seed ? Object.keys(seed) : null,
-    });
-
     if (!availablePipelines.length) {
       logger.warn(
         "No pipelines registered in config() when spawnRunner invoked"

package/src/core/pipeline-runner.js

@@ -143,10 +143,6 @@ try {
       startFromTask &&
       taskNames.indexOf(taskName) < taskNames.indexOf(startFromTask)
     ) {
-      logger.log("Skipping task before restart point", {
-        taskName,
-        startFromTask,
-      });
       continue;
     }
 
@@ -155,7 +151,6 @@ try {
         const outputPath = path.join(workDir, "tasks", taskName, "output.json");
         const output = JSON.parse(await fs.readFile(outputPath, "utf8"));
         pipelineArtifacts[taskName] = output;
-        logger.log("Task already completed", { taskName });
       } catch {
         logger.warn("Failed to read completed task output", { taskName });
       }
@@ -190,7 +185,6 @@ try {
       }
     }
 
-    logger.log("Starting task", { taskName });
     await updateStatus(taskName, {
       state: TaskState.RUNNING,
       startedAt: now(),
@@ -265,13 +259,6 @@ try {
           process.exitCode = 1;
           process.exit(1);
         }
-
-        logger.log("Task symlinks repaired successfully", {
-          taskName,
-          taskDir,
-          repairDuration: repairResult.duration,
-          relocatedEntry: repairResult.relocatedEntry,
-        });
       } else {
         logger.debug("Task symlinks validation passed", {
           taskName,
@@ -295,7 +282,6 @@ try {
         statusPath: tasksStatusPath,
       });
 
-      logger.log("Running task", { taskName, modulePath: absoluteModulePath });
       const result = await runPipeline(relocatedEntry, ctx);
 
       if (!result.ok) {
@@ -363,13 +349,6 @@ try {
         process.exit(1);
       }
 
-      logger.log("Task completed successfully", {
-        taskName,
-        executionTimeMs:
-          result.logs?.reduce((total, log) => total + (log.ms || 0), 0) || 0,
-        refinementAttempts: result.refinementAttempts || 0,
-      });
-
       // The file I/O system automatically handles writing outputs and updating tasks-status.json
       // No need to manually write output.json or enumerate artifacts
 
@@ -396,7 +375,6 @@ try {
 
       // Check if this is a single task run and we've completed the target task
       if (runSingleTask && taskName === startFromTask) {
-        logger.log("Stopping after single task execution", { taskName });
         break;
       }
     } catch (err) {
@@ -415,19 +393,6 @@ try {
     await fs.mkdir(COMPLETE_DIR, { recursive: true });
     const dest = path.join(COMPLETE_DIR, jobId);
 
-    logger.log("Pipeline completed", {
-      jobId,
-      totalExecutionTime: Object.values(status.tasks).reduce(
-        (total, t) => total + (t.executionTimeMs || 0),
-        0
-      ),
-      totalRefinementAttempts: Object.values(status.tasks).reduce(
-        (total, t) => total + (t.refinementAttempts || 0),
-        0
-      ),
-      finalArtifacts: Object.keys(pipelineArtifacts),
-    });
-
     await fs.rename(workDir, dest);
     await appendLine(
       path.join(COMPLETE_DIR, "runs.jsonl"),
@@ -449,8 +414,6 @@ try {
 
     // Clean up task symlinks to avoid dangling links in archives
     await cleanupTaskSymlinks(dest);
-  } else {
-    logger.log("Single task run completed, job remains in current", { jobId });
   }
 } catch (error) {
   throw error;

package/src/core/status-writer.js

@@ -164,13 +164,8 @@ export async function writeJobStatus(jobDir, updateFn) {
 
   const next = prev
     .then(async () => {
-      logger.group("Status Write Operation");
-      logger.log(`Updating status for job: ${jobId}`);
-      logger.log(`Status file path: ${statusPath}`);
-
       // Read existing status or create default
       const current = await readStatusFile(statusPath, jobId);
-      logger.log("Current status snapshot:", current);
 
       // Validate basic structure
       const validated = validateStatusSnapshot(current);
@@ -188,11 +183,9 @@ export async function writeJobStatus(jobDir, updateFn) {
       );
 
       snapshot.lastUpdated = new Date().toISOString();
-      logger.log("Status after update function:", snapshot);
 
       // Atomic write
       await atomicWrite(statusPath, snapshot);
-      logger.log("Status file written successfully");
 
       // Emit SSE event for tasks-status.json change using logger
       try {
@@ -202,7 +195,6 @@ export async function writeJobStatus(jobDir, updateFn) {
           jobId,
         };
         await logger.sse("state:change", eventData);
-        logger.log("SSE event broadcasted successfully");
       } catch (error) {
         // Don't fail the write if SSE emission fails
         logger.error("Failed to emit SSE event:", error);
@@ -218,10 +210,6 @@ export async function writeJobStatus(jobDir, updateFn) {
             reason: snapshot.lifecycleBlockReason,
           };
           await logger.sse("lifecycle_block", lifecycleEventData);
-          logger.log(
-            "lifecycle_block SSE event broadcasted successfully",
-            lifecycleEventData
-          );
         } catch (error) {
           // Don't fail the write if SSE emission fails
           logger.error("Failed to emit lifecycle_block SSE event:", error);
@@ -310,9 +298,6 @@ export async function updateTaskStatus(jobDir, taskId, taskUpdateFn) {
 
   const next = prev
     .then(async () => {
-      logger.group("Task Status Update Operation");
-      logger.log(`Updating task ${taskId} for job: ${jobId}`);
-
       const statusPath = path.join(jobDir, "tasks-status.json");
 
       // Read existing status or create default
@@ -336,7 +321,6 @@ export async function updateTaskStatus(jobDir, taskId, taskUpdateFn) {
 
       // Atomic write
       await atomicWrite(statusPath, validated);
-      logger.log("Task status file written successfully");
 
       // Emit task:updated SSE event after successful write
       try {
@@ -346,13 +330,11 @@ export async function updateTaskStatus(jobDir, taskId, taskUpdateFn) {
           task: validated.tasks[taskId],
         };
         await logger.sse("task:updated", eventData);
-        logger.log("task:updated SSE event broadcasted successfully");
       } catch (error) {
         // Don't fail the write if SSE emission fails
         logger.error("Failed to emit task:updated SSE event:", error);
       }
 
-      logger.groupEnd();
       resultSnapshot = validated;
     })
     .catch((e) => {

package/src/core/symlink-utils.js

@@ -196,12 +196,6 @@ export async function repairTaskSymlinks(taskDir, poRoot, taskModulePath) {
   const startTime = Date.now();
 
   try {
-    logger.log("Repairing task symlinks", {
-      taskDir,
-      poRoot,
-      taskModulePath,
-    });
-
     // Use existing ensureTaskSymlinkBridge for repairs
     const relocatedEntry = await ensureTaskSymlinkBridge({
       taskDir,
@@ -211,12 +205,6 @@ export async function repairTaskSymlinks(taskDir, poRoot, taskModulePath) {
 
     const duration = Date.now() - startTime;
 
-    logger.log("Task symlinks repaired successfully", {
-      taskDir,
-      duration,
-      relocatedEntry,
-    });
-
     return {
       success: true,
       relocatedEntry,

package/src/core/task-runner.js

@@ -484,10 +484,6 @@ export async function runPipeline(modulePath, initialContext = {}) {
 
     // Skip stages when skipIf predicate returns true
     if (stageConfig.skipIf && stageConfig.skipIf(context.flags)) {
-      logger.log("Skipping stage", {
-        stage: stageName,
-        reason: "skipIf predicate returned true",
-      });
       context.logs.push({
         stage: stageName,
         action: "skipped",
@@ -499,7 +495,6 @@ export async function runPipeline(modulePath, initialContext = {}) {
 
     // Skip if handler is not available (not implemented)
     if (typeof stageHandler !== "function") {
-      logger.log("Stage not available, skipping", { stage: stageName });
       logs.push({
         stage: stageName,
         skipped: true,
@@ -526,11 +521,6 @@ export async function runPipeline(modulePath, initialContext = {}) {
     // Set current stage before execution
     context.currentStage = stageName;
 
-    logger.log("Starting stage execution", {
-      stage: stageName,
-      taskName: context.meta.taskName,
-    });
-
     // Write stage start status using writeJobStatus
     if (context.meta.workDir && context.meta.taskName) {
       try {
@@ -719,12 +709,6 @@ export async function runPipeline(modulePath, initialContext = {}) {
       );
 
       const ms = +(performance.now() - start).toFixed(2);
-      logger.log("Stage completed successfully", {
-        stage: stageName,
-        executionTimeMs: ms,
-        outputType: typeof stageResult.output,
-        flagKeys: Object.keys(stageResult.flags),
-      });
       logs.push({
         stage: stageName,
         ok: true,
@@ -819,13 +803,6 @@ export async function runPipeline(modulePath, initialContext = {}) {
 
   llmEvents.off("llm:request:complete", onLLMComplete);
 
-  logger.log("Pipeline completed successfully", {
-    taskName: context.meta.taskName,
-    totalStages: PIPELINE_STAGES.length,
-    executedStages: logs.filter((l) => l.ok).length,
-    llmMetricsCount: llmMetrics.length,
-  });
-
   // Write final status with currentStage: null to indicate completion
   if (context.meta.workDir && context.meta.taskName) {
     try {