test(cli,packages): cover requirements CLI branches and subdir error paths

Closes the patch-coverage gap: no-configuration and unlabeled
require_one_of rendering plus inspect-error propagation in
af show-requirements, and the root-resolution / manifest-missing error
branches of ResolvePackageSubdir.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: AbirAbbas

control-plane/internal/cli/show_requirements_test.go

@@ -84,3 +84,36 @@ func TestShowRequirements_RejectsUnknownOutputFormat(t *testing.T) {
 	cmd.SilenceErrors = true
 	require.Error(t, cmd.Execute())
 }
+
+func TestShowRequirements_TextNoConfigurationNeeded(t *testing.T) {
+	cmd := NewShowRequirementsCommand()
+	cmd.SetArgs([]string{"testdata/show-requirements-bare"})
+
+	out := captureOutput(t, func() {
+		require.NoError(t, cmd.Execute())
+	})
+
+	require.Contains(t, out, "This node needs no user configuration.")
+	require.Contains(t, out, "Install: af install testdata/show-requirements-bare")
+}
+
+func TestShowRequirements_TextUnlabeledRequireOneOfGroup(t *testing.T) {
+	cmd := NewShowRequirementsCommand()
+	cmd.SetArgs([]string{"testdata/show-requirements-unlabeled"})
+
+	out := captureOutput(t, func() {
+		require.NoError(t, cmd.Execute())
+	})
+
+	require.Contains(t, out, "At least one of")
+	require.Contains(t, out, "one of these")
+}
+
+func TestShowRequirements_InspectErrorPropagates(t *testing.T) {
+	cmd := NewShowRequirementsCommand()
+	cmd.SetArgs([]string{"testdata/does-not-exist"})
+	cmd.SilenceUsage = true
+	cmd.SilenceErrors = true
+
+	require.Error(t, cmd.Execute())
+}

control-plane/internal/cli/testdata/show-requirements-bare/agentfield-package.yaml

@@ -231,3 +231,23 @@ func TestInstallPathGoSubdirBuildsRelativeToSubdir(t *testing.T) {
 		t.Fatalf("build must be rooted at the subdir, not the repo root")
 	}
 }
+
+func TestResolvePackageSubdir_ErrorBranches(t *testing.T) {
+	t.Run("nonexistent root fails resolution", func(t *testing.T) {
+		_, err := ResolvePackageSubdir(filepath.Join(t.TempDir(), "gone"), "sub")
+		if err == nil || !strings.Contains(err.Error(), "failed to resolve package root") {
+			t.Fatalf("expected root resolution error, got %v", err)
+		}
+	})
+
+	t.Run("existing subdir without manifest names the expected path", func(t *testing.T) {
+		root := t.TempDir()
+		if err := os.MkdirAll(filepath.Join(root, "empty"), 0o755); err != nil {
+			t.Fatal(err)
+		}
+		_, err := ResolvePackageSubdir(root, "empty")
+		if err == nil || !strings.Contains(err.Error(), "agentfield-package.yaml") {
+			t.Fatalf("expected missing-manifest error, got %v", err)
+		}
+	})
+}