[CAP-9160] skip auth and workspace prompts for --local commands (#321)

Skip the interactive login and workspace-selection prompts when `--local` is passed to workflow commands.

GitOrigin-RevId: bd51c011a345af5c8cb5301fa563323f2b6e2c49

Author: blue-keleher

cmd/root.go

@@ -59,13 +59,15 @@ var rootCmd = &cobra.Command{
 
 			var m tea.Model = stack
 
-			if cmd.Name() != deps.Commands.WorkspaceSetCmd().Name() {
+			local := isLocalCommand(cmd)
+
+			if !local && cmd.Name() != deps.Commands.WorkspaceSetCmd().Name() {
 				if !config.IsWorkspaceSet() {
 					m = tui.NewConfigWrapper(m, "Set Workspace", views.NewWorkspaceView(ctx, views.ListWorkspaceInput{}))
 				}
 			}
 
-			if cmd.Name() != loginCmd.Name() {
+			if !local && cmd.Name() != loginCmd.Name() {
 				m = tui.NewConfigWrapper(m, "Login", views.NewLoginView(ctx))
 			}
 
@@ -81,6 +83,15 @@ var rootCmd = &cobra.Command{
 	},
 }
 
+func isLocalCommand(cmd *cobra.Command) bool {
+	v, err := cmd.Flags().GetBool("local")
+	if err != nil {
+		// Flag doesn't exist or isn't a bool
+		return false
+	}
+	return v
+}
+
 func setupWorkflowCommands(deps *dependencies.Dependencies) {
 	deps.Commands.Workflow.TaskListCmd = NewTaskListCmd(deps)
 	deps.Commands.Workflow.TaskRunCmd = NewTaskRunStartCmd(deps)

cmd/task.go

@@ -134,7 +134,7 @@ func (d *localDeps) WorkflowLoader() *workflowviews.WorkflowLoader {
 }
 
 func (d *localDeps) LogLoader() *views.LogLoader {
-	return views.NewLogLoader(d.LogRepo(), nil, nil, nil, nil)
+	return views.NewLocalLogLoader(d.LogRepo())
 }
 
 func getLocalDeps(cmd *cobra.Command, deps flows.WorkflowDeps) (flows.WorkflowDeps, bool, error) {

pkg/tui/views/logloader.go

@@ -22,12 +22,17 @@ type LogLoader struct {
 	kvRepo       *keyvalue.Repo
 	postgresRepo *postgres.Repo
 	workflowRepo *workflow.Repo
+	local        bool
 }
 
 func NewLogLoader(logRepo *logs.LogRepo, serviceRepo *service.Repo, kvRepo *keyvalue.Repo, postgresRepo *postgres.Repo, workflowRepo *workflow.Repo) *LogLoader {
 	return &LogLoader{logRepo: logRepo, serviceRepo: serviceRepo, kvRepo: kvRepo, postgresRepo: postgresRepo, workflowRepo: workflowRepo}
 }
 
+func NewLocalLogLoader(logRepo *logs.LogRepo) *LogLoader {
+	return &LogLoader{logRepo: logRepo, local: true}
+}
+
 func (l *LogLoader) LoadLogData(ctx context.Context, in LogInput) (*tui.LogResult, error) {
 	params, err := l.ToParam(ctx, in)
 	if err != nil {
@@ -119,9 +124,13 @@ func (l *LogLoader) getResourceIDsFromIDOrNames(ctx context.Context, idOrNames [
 }
 
 func (l *LogLoader) ToParam(ctx context.Context, in LogInput) (*client.ListLogsParams, error) {
-	ownerID, err := config.WorkspaceID()
-	if err != nil {
-		return nil, fmt.Errorf("error getting workspace ID: %v", err)
+	var ownerID string
+	if !l.local {
+		var err error
+		ownerID, err = config.WorkspaceID()
+		if err != nil {
+			return nil, fmt.Errorf("error getting workspace ID: %v", err)
+		}
 	}
 
 	if in.Limit == 0 {

pkg/tui/views/logloader_test.go

@@ -0,0 +1,56 @@
+package views_test
+
+import (
+	"context"
+	"testing"
+
+	"github.com/render-oss/cli/pkg/config"
+	"github.com/render-oss/cli/pkg/tui/views"
+	"github.com/stretchr/testify/require"
+)
+
+func TestLogLoaderToParam(t *testing.T) {
+	clearAuthEnv := func(t *testing.T) {
+		t.Helper()
+		t.Setenv("RENDER_WORKSPACE", "")
+		t.Setenv("RENDER_API_KEY", "")
+		t.Setenv("RENDER_CLI_CONFIG_PATH", t.TempDir()+"/nonexistent.yaml")
+	}
+
+	resourceID := "srv-abcdef1234567890abcd"
+
+	t.Run("local does not require workspace", func(t *testing.T) {
+		clearAuthEnv(t)
+
+		loader := views.NewLocalLogLoader(nil)
+		params, err := loader.ToParam(context.Background(), views.LogInput{
+			ResourceIDs: []string{"wfl-local"},
+		})
+		require.NoError(t, err)
+		require.Equal(t, "", params.OwnerId)
+		require.Equal(t, []string{"wfl-local"}, params.Resource)
+	})
+
+	t.Run("production requires workspace", func(t *testing.T) {
+		clearAuthEnv(t)
+
+		loader := views.NewLogLoader(nil, nil, nil, nil, nil)
+		_, err := loader.ToParam(context.Background(), views.LogInput{
+			ResourceIDs: []string{resourceID},
+		})
+		require.ErrorContains(t, err, config.ErrNoWorkspace.Error())
+	})
+
+	t.Run("production includes workspace when set", func(t *testing.T) {
+		clearAuthEnv(t)
+		t.Setenv("RENDER_WORKSPACE", "wrk-test123")
+
+		loader := views.NewLogLoader(nil, nil, nil, nil, nil)
+		params, err := loader.ToParam(context.Background(), views.LogInput{
+			ResourceIDs: []string{resourceID},
+		})
+		require.NoError(t, err)
+		require.Equal(t, "wrk-test123", params.OwnerId)
+		require.Equal(t, []string{resourceID}, params.Resource)
+	})
+}