relay-flow 0.3.12-alpha → 0.3.13-alpha

package/README.md

@@ -28,13 +28,13 @@ same `relay-flow-plugin` package with host-specific entrypoints.
 **OpenCode**
 
 ```sh
-opencode plugin relay-flow-plugin@0.3.12-alpha
+opencode plugin relay-flow-plugin@0.3.13-alpha
 ```
 
 **Pi**
 
 ```sh
-pi install npm:relay-flow-plugin@0.3.12-alpha
+pi install npm:relay-flow-plugin@0.3.13-alpha
 ```
 
 Pi loads the package's `pi.ts` extension from its manifest. Do not add
@@ -164,7 +164,7 @@ OpenCode plugin configuration uses both entrypoints. The server entrypoint is li
 ```json
 {
   "$schema": "https://opencode.ai/config.json",
-  "plugin": ["relay-flow-plugin@0.3.12-alpha"]
+  "plugin": ["relay-flow-plugin@0.3.13-alpha"]
 }
 ```
 
@@ -173,7 +173,7 @@ The native HITL approval entrypoint is listed in `.opencode/tui.json`:
 ```json
 {
   "$schema": "https://opencode.ai/tui.json",
-  "plugin": ["relay-flow-plugin@0.3.12-alpha"]
+  "plugin": ["relay-flow-plugin@0.3.13-alpha"]
 }
 ```
 
@@ -189,7 +189,7 @@ Pi plugin: install the same published package manually in Pi's global package
 settings before starting a Pi harness session:
 
 ```sh
-pi install npm:relay-flow-plugin@0.3.12-alpha
+pi install npm:relay-flow-plugin@0.3.13-alpha
 ```
 
 Relay-flow does not install or configure the package automatically. Pi resolves

package/cmd/relay-flow/commands_test.go

@@ -308,6 +308,45 @@ func TestReportValidationErrorIsStructuredOnStderr(t *testing.T) {
 	}
 }
 
+func TestReportOtherAPIErrorsPreserveCodeForRetryClassification(t *testing.T) {
+	const report = `{"runId":"run-1","node":"coding","reportId":"s:m","report":{"status":"success","nextStep":"end"}}`
+	for _, tc := range []struct {
+		name, code string
+		err        error
+	}{
+		{"invalid request", "invalid", fmt.Errorf("%w: malformed report", server.ErrInvalid)},
+		{"server failure", "internalError", errors.New("temporary backend failure")},
+	} {
+		t.Run(tc.name, func(t *testing.T) {
+			home := t.TempDir()
+			serveAck(t, home, runsvc.ReportAck{}, tc.err)
+			reader, writer, err := os.Pipe()
+			if err != nil {
+				t.Fatal(err)
+			}
+			original := os.Stderr
+			os.Stderr = writer
+			code := cli(t, home, report, "report")
+			os.Stderr = original
+			_ = writer.Close()
+			defer reader.Close()
+			output, err := io.ReadAll(reader)
+			if err != nil {
+				t.Fatal(err)
+			}
+			var response struct {
+				Error struct {
+					Code    string `json:"code"`
+					Message string `json:"message"`
+				} `json:"error"`
+			}
+			if err := json.Unmarshal(output, &response); err != nil || code != exitFail || response.Error.Code != tc.code || response.Error.Message != tc.err.Error() {
+				t.Fatalf("CLI response: exit=%d stderr=%q parsed=%+v err=%v", code, output, response, err)
+			}
+		})
+	}
+}
+
 func TestReportUnreachableServerExits1(t *testing.T) {
 	valid := `{"runId":"payments/basicFlow/PAY-101","node":"coding","reportId":"s:m","report":{"status":"success","nextStep":"end","summary":{"completed":"x","commits":"abc123","notCompleted":"None","issuesDiscovered":"None","verification":"x","notes":"None"},"feedback":{"reasonForNextStep":"None","requiredActions":"None","relevantContext":"None","expectedResult":"None"}}}`
 	if code := cli(t, t.TempDir(), valid, "report"); code != 1 {

package/cmd/relay-flow/main.go

@@ -1148,7 +1148,7 @@ func cmdReport(c *server.Client, stdin io.Reader) int {
 	defer cancel()
 	if _, err := c.SubmitReport(ctx, req); err != nil {
 		var apiErr *server.APIError
-		if errors.As(err, &apiErr) && apiErr.Code == "invalidReport" {
+		if errors.As(err, &apiErr) {
 			_ = json.NewEncoder(os.Stderr).Encode(map[string]any{
 				"error": map[string]string{"code": apiErr.Code, "message": apiErr.Message},
 			})

package/internal/execution/goworkflows/engine_test.go

@@ -316,7 +316,7 @@ func TestInvalidReportIsPermanentAndDoesNotSignal(t *testing.T) {
 	if !errors.Is(err, run.ErrInvalidReport) || ack.Accepted {
 		t.Fatalf("invalid report: ack=%+v err=%v", ack, err)
 	}
-	if want := `report selects "end": every feedback field must be "None" because end has no mailbox`; err.Error() != want {
+	if want := `NEXT STEP is end, so FEEDBACK must be exactly None.`; err.Error() != want {
 		t.Fatalf("validation message = %q, want %q", err, want)
 	}
 	after, err := engine.GetRun(context.Background(), rid)

package/internal/execution/temporal/operations_test.go

@@ -85,7 +85,7 @@ func TestTemporalSubmitReportRejectsInvalidReportBeforeSignal(t *testing.T) {
 	if !errors.Is(err, run.ErrInvalidReport) || ack.Accepted || client.signaled {
 		t.Fatalf("invalid Temporal report: ack=%+v err=%v signaled=%v", ack, err, client.signaled)
 	}
-	if want := `report selects "end": every feedback field must be "None" because end has no mailbox`; err.Error() != want {
+	if want := `NEXT STEP is end, so FEEDBACK must be exactly None.`; err.Error() != want {
 		t.Fatalf("validation message = %q, want %q", err, want)
 	}
 }

package/internal/harness/opencode/opencode_test.go

@@ -14,7 +14,7 @@ import (
 	"github.com/rajpopat27/relay-flow/internal/workflow"
 )
 
-const configuredPlugin = "relay-flow-plugin@0.3.12-alpha"
+const configuredPlugin = "relay-flow-plugin@0.3.13-alpha"
 
 func TestBuildCommandArgv(t *testing.T) {
 	t.Setenv("RELAY_FLOW_HOME", "/var/lib/relay-flow-test")

package/internal/harness/opencode/repo_setup.go

@@ -10,7 +10,7 @@ import (
 	"github.com/rajpopat27/relay-flow/internal/config"
 )
 
-const relayFlowPlugin = "relay-flow-plugin@0.3.12-alpha"
+const relayFlowPlugin = "relay-flow-plugin@0.3.13-alpha"
 
 type jsoncToken struct {
 	kind       byte

package/internal/workflow/report.go

@@ -50,9 +50,6 @@ func (w *Workflow) ValidateReport(node string, report Report) error {
 	if report.Status != OutcomeSuccess && report.Status != OutcomeFailure {
 		return fmt.Errorf("report status %q must be %q or %q", report.Status, OutcomeSuccess, OutcomeFailure)
 	}
-	if report.Status == OutcomeFailure && report.NextStep == EndNode {
-		return fmt.Errorf("failure reports cannot select %q", EndNode)
-	}
 	for _, field := range []struct{ name, value string }{
 		{"summary.completed", report.Summary.Completed},
 		{"summary.commits", report.Summary.Commits},
@@ -74,6 +71,9 @@ func (w *Workflow) ValidateReport(node string, report Report) error {
 	if err != nil {
 		return err
 	}
+	if report.Status == OutcomeFailure && report.NextStep == EndNode {
+		return fmt.Errorf("STATUS is failure, so NEXT STEP cannot be end; valid targets: %s", strings.Join(sortedTargets(routes), ", "))
+	}
 	legal := false
 	for _, r := range routes {
 		if r.Target == report.NextStep {
@@ -88,7 +88,7 @@ func (w *Workflow) ValidateReport(node string, report Report) error {
 	if report.NextStep == EndNode {
 		f := report.Feedback
 		if f.ReasonForNextStep != None || f.RequiredActions != None || f.RelevantContext != None || f.ExpectedResult != None {
-			return fmt.Errorf("report selects %q: every feedback field must be %q because end has no mailbox", EndNode, None)
+			return fmt.Errorf("NEXT STEP is end, so FEEDBACK must be exactly None.")
 		}
 	}
 	return nil

package/internal/workflow/report_test.go

@@ -208,8 +208,8 @@ func TestValidateReportNextStepMustMatchStatusRoute(t *testing.T) {
 			Summary:  fullSummary(),
 			Feedback: fullFeedback(),
 		}
-		if err := wf.ValidateReport("coding", r); err == nil {
-			t.Fatal("failure report selecting a success-only target accepted")
+		if err := wf.ValidateReport("coding", r); err == nil || !strings.Contains(err.Error(), "valid targets: coding") {
+			t.Fatalf("failure report selecting a success-only target: %v, want configured failure route", err)
 		}
 	})
 
@@ -265,8 +265,8 @@ func TestValidateReportEndRequiresNoneFeedback(t *testing.T) {
 			Summary:  fullSummary(),
 			Feedback: fullFeedback(),
 		}
-		if err := wf.ValidateReport("coding", r); err == nil {
-			t.Fatal("end report carrying feedback accepted; end has no mailbox")
+		if err := wf.ValidateReport("coding", r); err == nil || err.Error() != "NEXT STEP is end, so FEEDBACK must be exactly None." {
+			t.Fatalf("end report error = %v, want actionable four-field correction", err)
 		}
 	})
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "relay-flow",
-  "version": "0.3.12-alpha",
+  "version": "0.3.13-alpha",
   "description": "Graph-based agent workflow engine — tracker-agnostic, pluggable runners",
   "bin": {
     "relay-flow": "./bin/relay-flow.js",