From e59e3415db9d554804b55abdcfe81a19722f00c4 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 23 Feb 2026 14:10:00 +0000
Subject: [PATCH 1/2] Initial plan


From ff95b7c2adeadc484058b5f61616e5f3a1a425d3 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 23 Feb 2026 14:13:13 +0000
Subject: [PATCH 2/2] docs: fix PipelineStep signature, add slog import, and
 handle LoadPlugin error in PLUGIN_DEVELOPMENT.md

Co-authored-by: intel352 <77607+intel352@users.noreply.github.com>
---
 docs/PLUGIN_DEVELOPMENT.md | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/docs/PLUGIN_DEVELOPMENT.md b/docs/PLUGIN_DEVELOPMENT.md
index d764faaa..88263ca2 100644
--- a/docs/PLUGIN_DEVELOPMENT.md
+++ b/docs/PLUGIN_DEVELOPMENT.md
@@ -152,7 +152,7 @@ The returned value must implement `module.PipelineStep`:
 ```go
 type PipelineStep interface {
     Name() string
-    Execute(ctx context.Context, pc *PipelineContext) error
+    Execute(ctx context.Context, pc *PipelineContext) (*StepResult, error)
 }
 ```
 
@@ -349,6 +349,7 @@ This enables workflows to fail fast with actionable errors rather than cryptic r
 package myplugin
 
 import (
+    "log/slog"
     "testing"
 
     "github.com/CrisisTextLine/modular"
@@ -368,7 +369,9 @@ func TestPluginLoads(t *testing.T) {
 func TestModuleCreation(t *testing.T) {
     app := modular.NewStdApplication(modular.NewStdConfigProvider(nil), nil)
     engine := workflow.NewStdEngine(app, slog.Default())
-    engine.LoadPlugin(New())
+    if err := engine.LoadPlugin(New()); err != nil {
+        t.Fatalf("LoadPlugin failed: %v", err)
+    }
 
     cfg := &config.WorkflowConfig{
         Modules: []config.ModuleConfig{