relay-flow 0.2.5-alpha → 0.2.7-alpha

package/internal/task/beads/beads_test.go

@@ -194,17 +194,86 @@ func TestExtractWorkflowClaimsKeepsOnlyWorkflowLabels(t *testing.T) {
 	}
 }
 
+func TestRepositoryLabelIsStableAndValidated(t *testing.T) {
+	got, err := RepositoryLabel("Payments_API")
+	if err != nil {
+		t.Fatal(err)
+	}
+	if got != "repo:payments_api" {
+		t.Fatalf("RepositoryLabel = %q, want repo:payments_api", got)
+	}
+	for _, name := range []string{"", " payments", "payments/api", "payments api"} {
+		if _, err := RepositoryLabel(name); err == nil {
+			t.Fatalf("RepositoryLabel accepted invalid name %q", name)
+		}
+	}
+}
+
+func TestRepositoryLabelEnforcesMaximumLength(t *testing.T) {
+	validName := strings.Repeat("a", 250)
+	label, err := RepositoryLabel(validName)
+	if err != nil {
+		t.Fatalf("maximum-length repository name rejected: %v", err)
+	}
+	if len(label) != maxRepositoryLabelLen {
+		t.Fatalf("maximum-length label has length %d, want %d", len(label), maxRepositoryLabelLen)
+	}
+
+	tooLong := strings.Repeat("a", 251)
+	if _, err := RepositoryLabel(tooLong); err == nil || !strings.Contains(err.Error(), "maximum is 255") {
+		t.Fatalf("overlong repository name error = %v, want the 255-character validation", err)
+	}
+}
+
+func TestRegistrationKeyAndNewSystemRejectOverlongRepositoryName(t *testing.T) {
+	name := strings.Repeat("a", 251)
+	workspace := t.TempDir()
+	if _, err := task.RegistrationKey("beads", task.RepoRegistrationSpec{
+		Name:       name,
+		RepoConfig: config.RawValues{"beadsDir": workspace},
+	}); err == nil {
+		t.Fatal("registration identity accepted an overlong repository name")
+	}
+
+	_, err := newSystem(context.Background(), task.RepoSpec{
+		Name:       name,
+		Path:       t.TempDir(),
+		RepoConfig: config.RawValues{"beadsDir": filepath.Join(workspace, "missing")},
+	})
+	if err == nil || !strings.Contains(err.Error(), "maximum is 255") {
+		t.Fatalf("newSystem overlong-name error = %v, want the label validation before workspace probing", err)
+	}
+}
+
+func TestPollFiltersMissingOtherAndAmbiguousRepositoryLabels(t *testing.T) {
+	fake := &pollClient{ready: []bdcli.Issue{
+		{ID: "payments-1", Title: "payments", Labels: []string{"repo:payments", "wf:implementation"}},
+		{ID: "unowned-1", Title: "unowned", Labels: []string{"wf:implementation"}},
+		{ID: "platform-1", Title: "platform", Labels: []string{"repo:platform", "wf:implementation"}},
+		{ID: "ambiguous-1", Title: "ambiguous", Labels: []string{"repo:payments", "repo:platform", "wf:implementation"}},
+		{ID: "duplicate-1", Title: "duplicate", Labels: []string{"repo:payments", "repo:payments"}},
+	}}
+	got, err := (&system{cli: fake, repositoryLabel: "repo:payments"}).Poll(context.Background())
+	if err != nil {
+		t.Fatal(err)
+	}
+	if len(got) != 1 || got[0].ID != "payments-1" {
+		t.Fatalf("Poll = %+v, want only the payments-labelled parent", got)
+	}
+	if got[0].WorkflowClaims[0] != "wf:implementation" {
+		t.Fatalf("workflow labels were not retained after repository filtering: %+v", got[0])
+	}
+}
+
 func TestPollDeduplicatesReadyAndClaimedParentsByIssueID(t *testing.T) {
-	ready := bdcli.Issue{ID: "demo-a1b2", Title: "ready copy", Status: "open", IssueType: "epic"}
-	claimedDuplicate := bdcli.Issue{ID: "demo-a1b2", Title: "claimed copy", Status: "in_progress", IssueType: "epic", Labels: []string{"wf:implementation"}}
-	claimedOnly := bdcli.Issue{ID: "demo-c3d4", Title: "claimed only", Status: "blocked", IssueType: "task", Labels: []string{"wf:review"}}
+	ready := bdcli.Issue{ID: "demo-a1b2", Title: "ready copy", Status: "open", IssueType: "epic", Labels: []string{"repo:payments"}}
+	claimedDuplicate := bdcli.Issue{ID: "demo-a1b2", Title: "claimed copy", Status: "in_progress", IssueType: "epic", Labels: []string{"repo:payments", "wf:implementation"}}
+	claimedOnly := bdcli.Issue{ID: "demo-c3d4", Title: "claimed only", Status: "blocked", IssueType: "task", Labels: []string{"repo:payments", "wf:review"}}
 	fake := &pollClient{
 		ready:   []bdcli.Issue{ready},
 		claimed: []bdcli.Issue{claimedDuplicate, claimedOnly},
 	}
-	sys := &system{cli: fake}
-
-	got, err := sys.Poll(context.Background())
+	got, err := (&system{cli: fake, repositoryLabel: "repo:payments"}).Poll(context.Background())
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -230,10 +299,10 @@ func TestPollDeduplicatesReadyAndClaimedParentsByIssueID(t *testing.T) {
 }
 
 func TestNormalizedChildIsExcludedFromPolling(t *testing.T) {
-	parent := bdcli.Issue{ID: "demo-parent", Title: "parent", Status: "open", IssueType: "epic"}
-	child := bdcli.Issue{ID: "demo-parent.1", Title: "demo-parent:implement", Status: "open", IssueType: "task", Parent: "demo-parent"}
+	parent := bdcli.Issue{ID: "demo-parent", Title: "parent", Status: "open", IssueType: "epic", Labels: []string{"repo:payments"}}
+	child := bdcli.Issue{ID: "demo-parent.1", Title: "demo-parent:implement", Status: "open", IssueType: "task", Parent: "demo-parent", Labels: []string{"repo:payments"}}
 	fake := &pollClient{ready: []bdcli.Issue{child, parent}, claimed: []bdcli.Issue{child}}
-	got, err := (&system{cli: fake}).Poll(context.Background())
+	got, err := (&system{cli: fake, repositoryLabel: "repo:payments"}).Poll(context.Background())
 	if err != nil {
 		t.Fatal(err)
 	}

package/internal/task/beads/repo_composition_test.go

@@ -99,7 +99,7 @@ func TestBeadsRepoRegistrationUsesIndependentSystemsAndPollers(t *testing.T) {
 	assertBDInvocationReachedRepoAndWorkspace(t, logPath, platformPath, platformBeads)
 }
 
-func TestBeadsRepoRegistrationRejectsSharedCanonicalWorkspace(t *testing.T) {
+func TestBeadsRepoRegistrationAllowsSharedCanonicalWorkspaceWithDistinctLabels(t *testing.T) {
 	root := t.TempDir()
 	installRepoCompositionFakeBD(t)
 	configPath := filepath.Join(root, "config.yaml")
@@ -135,11 +135,55 @@ func TestBeadsRepoRegistrationRejectsSharedCanonicalWorkspace(t *testing.T) {
 	if _, err := svc.Register(context.Background(), repo.RegisterInput{
 		Name: "platform", Path: secondPath,
 		TaskConfig: config.RawValues{"beadsDir": filepath.Join(workspace, ".")},
+	}); err != nil {
+		t.Fatalf("second repo sharing canonical beadsDir with a distinct label was rejected: %v", err)
+	}
+	if len(svc.Registry().List()) != 2 {
+		t.Fatalf("registry after shared workspace registration = %d, want 2", len(svc.Registry().List()))
+	}
+}
+
+func TestBeadsRepoRegistrationRejectsRepositoryLabelCollision(t *testing.T) {
+	root := t.TempDir()
+	installRepoCompositionFakeBD(t)
+	configPath := filepath.Join(root, "config.yaml")
+	if err := config.SaveMachine(configPath, &config.Machine{
+		TaskPlugin: "beads", RunnerPlugin: "orca", HarnessPlugin: "opencode",
+		Repos: map[string]config.Repo{},
+	}); err != nil {
+		t.Fatal(err)
+	}
+	firstPath := filepath.Join(root, "code", "payments")
+	secondPath := filepath.Join(root, "code", "PAYMENTS")
+	workspace := filepath.Join(root, "beads", "shared", ".beads")
+	for _, path := range []string{firstPath, secondPath, workspace} {
+		if err := os.MkdirAll(path, 0o755); err != nil {
+			t.Fatal(err)
+		}
+	}
+
+	svc := repo.NewService(repo.ServiceConfig{
+		ConfigPath: configPath,
+		TaskPlugin: "beads",
+		Runner:     &compositionRunner{},
+		Harness:    &compositionHarness{},
+		Active:     &compositionActive{},
+		Workflows:  &compositionWorkflowRefs{},
+	})
+	if _, err := svc.Register(context.Background(), repo.RegisterInput{
+		Name: "payments", Path: firstPath,
+		TaskConfig: config.RawValues{"beadsDir": workspace},
+	}); err != nil {
+		t.Fatal(err)
+	}
+	if _, err := svc.Register(context.Background(), repo.RegisterInput{
+		Name: "PAYMENTS", Path: secondPath,
+		TaskConfig: config.RawValues{"beadsDir": workspace},
 	}); err == nil {
-		t.Fatal("second repo sharing canonical beadsDir was accepted")
+		t.Fatal("repository label collision was accepted")
 	}
 	if len(svc.Registry().List()) != 1 {
-		t.Fatalf("registry after duplicate scope rejection = %d, want 1", len(svc.Registry().List()))
+		t.Fatalf("registry after label collision = %d, want 1", len(svc.Registry().List()))
 	}
 }
 

package/internal/task/factory.go

@@ -20,13 +20,25 @@ type RepoSpec struct {
 	RepoConfig config.RawValues
 }
 
+// RepoRegistrationSpec carries the values a task plugin may need to derive a
+// registration identity. Unlike RepoSpec, it does not include the code path
+// or construct a task System.
+type RepoRegistrationSpec struct {
+	Name       string
+	RootConfig config.RawValues
+	RepoConfig config.RawValues
+}
+
 // Factory constructs a task System. RequiredRepoKeys returns the explicit
 // repo YAML keys needed at registration. TaskScopeKey derives an opaque
 // canonical physical task scope (such as Jira site/project/component) used
-// to reject duplicate scope registration.
+// to reject duplicate scope registration. RegistrationKey optionally derives
+// the complete registration identity when a plugin permits sharing a
+// physical scope with distinct logical repositories.
 type Factory struct {
 	RequiredRepoKeys   func() []string
 	TaskScopeKey       func(rootConfig, repoConfig config.RawValues) (string, error)
+	RegistrationKey    func(RepoRegistrationSpec) (string, error)
 	Auth               func(context.Context, []string, io.Reader) error
 	DefaultConfig      func() config.RawValues
 	ValidateTextConfig func(config.RawValues) error
@@ -122,7 +134,10 @@ func RequiredRepoKeys(name string) ([]string, error) {
 	return f.RequiredRepoKeys(), nil
 }
 
-// TaskScopeKey derives the canonical task scope for the named plugin.
+// TaskScopeKey derives the canonical physical task scope for the named
+// plugin. It is retained separately from RegistrationKey so callers that
+// need provider workspace identity do not have to know about logical repo
+// ownership.
 func TaskScopeKey(name string, rootConfig, repoConfig config.RawValues) (string, error) {
 	f, err := lookup(name)
 	if err != nil {
@@ -131,6 +146,20 @@ func TaskScopeKey(name string, rootConfig, repoConfig config.RawValues) (string,
 	return f.TaskScopeKey(rootConfig, repoConfig)
 }
 
+// RegistrationKey derives the identity used to reject conflicting repo
+// registrations. Plugins that do not provide a custom key fall back to their
+// physical TaskScopeKey, preserving the original duplicate-scope behavior.
+func RegistrationKey(name string, spec RepoRegistrationSpec) (string, error) {
+	f, err := lookup(name)
+	if err != nil {
+		return "", err
+	}
+	if f.RegistrationKey != nil {
+		return f.RegistrationKey(spec)
+	}
+	return f.TaskScopeKey(spec.RootConfig, spec.RepoConfig)
+}
+
 // ValidateName returns an error listing registered names when name is not
 // a registered plugin. Used by `relay-flow init` to reject unknown plugin
 // selections without constructing a System.

package/internal/task/task.go

@@ -99,6 +99,16 @@ type System interface {
 	ResetForRecovery(ctx context.Context, parent TicketRef, mailboxes []Mailbox, taskConfig config.RawValues) error
 }
 
+// AgentEnvironment is an optional adapter capability. It returns the
+// task-system workspace environment an agent process must receive so agent
+// task commands address the same workspace as relay-flow. Values are
+// workspace selectors only, never credentials; an empty value explicitly
+// neutralizes an ambient selector. Core forwards the map through the harness
+// launch spec without learning adapter vocabulary.
+type AgentEnvironment interface {
+	AgentEnv() map[string]string
+}
+
 // RestartPreparer is an optional adapter capability used only by explicit
 // run restarts. It reopens relay-owned mailbox state while preserving all
 // comments, labels, and descriptions. A human-owned incompatible state must

package/package.json

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