relay-flow 0.3.0-alpha → 0.3.2-alpha

package/README.md

@@ -24,13 +24,13 @@ same `relay-flow-plugin` package with host-specific entrypoints.
 **OpenCode**
 
 ```sh
-opencode plugin relay-flow-plugin@0.3.0-alpha
+opencode plugin relay-flow-plugin@0.3.2-alpha
 ```
 
 **Pi**
 
 ```sh
-pi install npm:relay-flow-plugin@0.3.0-alpha
+pi install npm:relay-flow-plugin@0.3.2-alpha
 ```
 
 Pi loads the package's `pi.ts` extension from its manifest. Do not add
@@ -160,7 +160,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.0-alpha"]
+  "plugin": ["relay-flow-plugin@0.3.2-alpha"]
 }
 ```
 
@@ -169,7 +169,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.0-alpha"]
+  "plugin": ["relay-flow-plugin@0.3.2-alpha"]
 }
 ```
 
@@ -185,7 +185,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.0-alpha
+pi install npm:relay-flow-plugin@0.3.2-alpha
 ```
 
 Relay-flow does not install or configure the package automatically. Pi resolves

package/cmd/relay-flow/main.go

@@ -853,8 +853,8 @@ func cmdServe(p paths.Paths, args []string) int {
 }
 
 var (
-	backgroundServeStartupTimeout = 10 * time.Second
-	backgroundServePollInterval   = 50 * time.Millisecond
+	backgroundServeStartupTimeout = 5 * time.Minute
+	backgroundServePollInterval   = 250 * time.Millisecond
 )
 
 func startBackgroundServe(p paths.Paths, recover, debug bool) error {

package/internal/harness/opencode/opencode.go

@@ -88,7 +88,7 @@ func init() {
 type Harness struct {
 	templates Config
 	// listAgents is the test seam; nil → real `opencode agent list`.
-	listAgents func(ctx context.Context) ([]string, error)
+	listAgents func(ctx context.Context, repoPath string) ([]string, error)
 }
 
 // New returns the production Harness.
@@ -110,7 +110,7 @@ func (h *Harness) SetupRepo(_ context.Context, repoPath string) error {
 // ValidateAgent reports whether name is a known OpenCode agent for the
 // repo, per `opencode agent list` (agent names are the unindented first
 // tokens).
-func (h *Harness) ValidateAgent(ctx context.Context, _ string, agent string) error {
+func (h *Harness) ValidateAgent(ctx context.Context, repoPath, agent string) error {
 	if agent == "" {
 		return fmt.Errorf("opencode: agent name is empty")
 	}
@@ -118,7 +118,7 @@ func (h *Harness) ValidateAgent(ctx context.Context, _ string, agent string) err
 	if list == nil {
 		list = listAgents
 	}
-	names, err := list(ctx)
+	names, err := list(ctx, repoPath)
 	if err != nil {
 		return err
 	}
@@ -236,8 +236,10 @@ func relayFlowHome() (string, error) {
 
 // listAgents runs `opencode agent list` and returns the agent names
 // (the unindented lines' first tokens).
-func listAgents(ctx context.Context) ([]string, error) {
-	out, err := exec.CommandContext(ctx, "opencode", "agent", "list").Output()
+func listAgents(ctx context.Context, repoPath string) ([]string, error) {
+	cmd := exec.CommandContext(ctx, "opencode", "agent", "list")
+	cmd.Dir = repoPath
+	out, err := cmd.Output()
 	if err != nil {
 		return nil, fmt.Errorf("opencode agent list: %w", err)
 	}

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.0-alpha"
+const configuredPlugin = "relay-flow-plugin@0.3.2-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.0-alpha"
+const relayFlowPlugin = "relay-flow-plugin@0.3.2-alpha"
 
 type jsoncToken struct {
 	kind       byte

package/internal/harness/opencode/validation_test.go

@@ -0,0 +1,106 @@
+package opencode
+
+import (
+	"context"
+	"os"
+	"path/filepath"
+	"runtime"
+	"strings"
+	"testing"
+)
+
+func TestValidateAgentPassesRepositoryToListAgentsSeam(t *testing.T) {
+	repoPath := t.TempDir()
+	var gotRepoPath string
+	h := New()
+	h.listAgents = func(_ context.Context, path string) ([]string, error) {
+		gotRepoPath = path
+		return []string{"build"}, nil
+	}
+
+	if err := h.ValidateAgent(context.Background(), repoPath, "build"); err != nil {
+		t.Fatalf("ValidateAgent: %v", err)
+	}
+	if gotRepoPath != repoPath {
+		t.Fatalf("listAgents repoPath = %q, want %q", gotRepoPath, repoPath)
+	}
+}
+
+func TestValidateAgentUsesRepositoryWorkingDirectory(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("fake opencode uses a POSIX shell")
+	}
+
+	repoA := t.TempDir()
+	repoB := t.TempDir()
+	fakeDir := t.TempDir()
+	fake := filepath.Join(fakeDir, "opencode")
+	script := `#!/bin/sh
+set -eu
+[ "$#" -eq 2 ] && [ "$1" = "agent" ] && [ "$2" = "list" ] || exit 2
+actual=$(pwd)
+case "$actual" in
+  "$OPENCODE_REPO_A")
+    printf 'repo-a-agent details\n'
+    ;;
+  "$OPENCODE_REPO_B")
+    printf 'repo-b-agent details\n'
+    ;;
+  *)
+    printf 'unexpected working directory: %s\n' "$actual" >&2
+    exit 3
+    ;;
+esac
+printf '  ignored-indented\n'
+printf '[ignored-json]\n'
+printf '{"ignored":true}\n'
+`
+	if err := os.WriteFile(fake, []byte(script), 0o700); err != nil {
+		t.Fatal(err)
+	}
+	t.Setenv("PATH", fakeDir)
+	t.Setenv("OPENCODE_REPO_A", repoA)
+	t.Setenv("OPENCODE_REPO_B", repoB)
+
+	h := New()
+	if err := h.ValidateAgent(context.Background(), repoA, "repo-a-agent"); err != nil {
+		t.Fatalf("agent from repository A rejected: %v", err)
+	}
+	if err := h.ValidateAgent(context.Background(), repoB, "repo-b-agent"); err != nil {
+		t.Fatalf("agent from repository B rejected: %v", err)
+	}
+	if err := h.ValidateAgent(context.Background(), repoA, "repo-b-agent"); err == nil {
+		t.Fatal("repository B agent accepted while validating repository A")
+	}
+	if err := h.ValidateAgent(context.Background(), repoB, "repo-a-agent"); err == nil {
+		t.Fatal("repository A agent accepted while validating repository B")
+	}
+}
+
+func TestValidateAgentPropagatesListCommandFailure(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("fake opencode uses a POSIX shell")
+	}
+
+	fakeDir := t.TempDir()
+	fake := filepath.Join(fakeDir, "opencode")
+	script := `#!/bin/sh
+set -eu
+[ "$#" -eq 2 ] && [ "$1" = "agent" ] && [ "$2" = "list" ] || exit 2
+printf 'agent list failed\n' >&2
+exit 17
+`
+	if err := os.WriteFile(fake, []byte(script), 0o700); err != nil {
+		t.Fatal(err)
+	}
+	t.Setenv("PATH", fakeDir)
+
+	h := New()
+	err := h.ValidateAgent(context.Background(), t.TempDir(), "build")
+	if err == nil {
+		t.Fatal("ValidateAgent succeeded after opencode agent list failed")
+	}
+	if !strings.Contains(err.Error(), "opencode agent list") {
+		t.Fatalf("error = %q, want opencode agent list context", err)
+	}
+}

package/package.json

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