Add dependency injection and golden-file tests for all commands

- Rename telemetry → sentry across adapters, interfaces, and container
- Introduce cmddeps.Deps struct for injectable config, client, and stdout
- Add NewCmd*WithDeps constructors to all commands for testability
- Add --output json flag to analysis command
- Add golden-file-based tests for analysis, issues, metrics, vulnerabilities,
  repo status, repo analyzers, and auth status commands
- Add NewWithGraphQLClient and NewTestService test helpers
- Always show auth URL in login flow regardless of browser open result
- Update justfile with new test paths

Author: jai-deepsource

command/analysis/analysis.go

@@ -2,11 +2,15 @@ package analysis
 
 import (
 	"context"
+	"encoding/json"
 	"fmt"
+	"io"
+	"os"
 	"strings"
 	"time"
 
 	"github.com/MakeNowJust/heredoc"
+	"github.com/deepsourcelabs/cli/command/cmddeps"
 	"github.com/deepsourcelabs/cli/config"
 	"github.com/deepsourcelabs/cli/deepsource"
 	"github.com/deepsourcelabs/cli/deepsource/runs"
@@ -19,14 +23,28 @@ import (
 )
 
 type AnalysisOptions struct {
-	RepoArg   string
-	LimitArg  int
-	commitOid string
+	RepoArg      string
+	LimitArg     int
+	OutputFormat string
+	commitOid    string
+	deps         *cmddeps.Deps
+}
+
+func (opts *AnalysisOptions) stdout() io.Writer {
+	if opts.deps != nil && opts.deps.Stdout != nil {
+		return opts.deps.Stdout
+	}
+	return os.Stdout
 }
 
 func NewCmdAnalysis() *cobra.Command {
+	return NewCmdAnalysisWithDeps(nil)
+}
+
+func NewCmdAnalysisWithDeps(deps *cmddeps.Deps) *cobra.Command {
 	opts := AnalysisOptions{
 		LimitArg: 20,
+		deps:     deps,
 	}
 
 	doc := heredoc.Docf(`
@@ -63,17 +81,29 @@ func NewCmdAnalysis() *cobra.Command {
 	cmd.Flags().StringVarP(&opts.RepoArg, "repo", "r", "", "List history for the specified repository")
 	cmd.Flags().IntVarP(&opts.LimitArg, "limit", "l", 20, "Number of analysis runs to fetch")
 	cmd.Flags().StringVar(&opts.commitOid, "commit", "", "Show metadata and issues summary for a specific commit")
+	cmd.Flags().StringVarP(&opts.OutputFormat, "output", "o", "pretty", "Output format: pretty, json")
 
 	_ = cmd.RegisterFlagCompletionFunc("repo", func(cmd *cobra.Command, args []string, toComplete string) ([]string, cobra.ShellCompDirective) {
 		return completion.RepoCompletionCandidates(), cobra.ShellCompDirectiveNoFileComp
 	})
+	_ = cmd.RegisterFlagCompletionFunc("output", func(cmd *cobra.Command, args []string, toComplete string) ([]string, cobra.ShellCompDirective) {
+		return []string{
+			"pretty\tPretty-printed output",
+			"json\tJSON output",
+		}, cobra.ShellCompDirectiveNoFileComp
+	})
 
 	return cmd
 }
 
 // runList fetches and displays a table of recent analysis runs.
 func (opts *AnalysisOptions) runList() error {
-	cfgMgr := config.DefaultManager()
+	var cfgMgr *config.Manager
+	if opts.deps != nil && opts.deps.ConfigMgr != nil {
+		cfgMgr = opts.deps.ConfigMgr
+	} else {
+		cfgMgr = config.DefaultManager()
+	}
 	cfg, err := cfgMgr.Load()
 	if err != nil {
 		return clierrors.NewCLIError(clierrors.ErrInvalidConfig, "Error reading DeepSource CLI config", err)
@@ -87,13 +117,18 @@ func (opts *AnalysisOptions) runList() error {
 		return err
 	}
 
-	client, err := deepsource.New(deepsource.ClientOpts{
-		Token:            cfg.Token,
-		HostName:         cfg.Host,
-		OnTokenRefreshed: cfgMgr.TokenRefreshCallback(),
-	})
-	if err != nil {
-		return err
+	var client *deepsource.Client
+	if opts.deps != nil && opts.deps.Client != nil {
+		client = opts.deps.Client
+	} else {
+		client, err = deepsource.New(deepsource.ClientOpts{
+			Token:            cfg.Token,
+			HostName:         cfg.Host,
+			OnTokenRefreshed: cfgMgr.TokenRefreshCallback(),
+		})
+		if err != nil {
+			return err
+		}
 	}
 
 	ctx := context.Background()
@@ -107,13 +142,22 @@ func (opts *AnalysisOptions) runList() error {
 		return nil
 	}
 
+	if opts.OutputFormat == "json" {
+		return opts.outputRunsJSON(analysisRuns)
+	}
+
 	showRunsTable(analysisRuns)
 	return nil
 }
 
 // runDetail fetches and displays metadata + issues summary for a single commit.
 func (opts *AnalysisOptions) runDetail(ctx context.Context) error {
-	cfgMgr := config.DefaultManager()
+	var cfgMgr *config.Manager
+	if opts.deps != nil && opts.deps.ConfigMgr != nil {
+		cfgMgr = opts.deps.ConfigMgr
+	} else {
+		cfgMgr = config.DefaultManager()
+	}
 	cfg, err := cfgMgr.Load()
 	if err != nil {
 		return clierrors.NewCLIError(clierrors.ErrInvalidConfig, "Error reading DeepSource CLI config", err)
@@ -122,13 +166,18 @@ func (opts *AnalysisOptions) runDetail(ctx context.Context) error {
 		return err
 	}
 
-	client, err := deepsource.New(deepsource.ClientOpts{
-		Token:            cfg.Token,
-		HostName:         cfg.Host,
-		OnTokenRefreshed: cfgMgr.TokenRefreshCallback(),
-	})
-	if err != nil {
-		return err
+	var client *deepsource.Client
+	if opts.deps != nil && opts.deps.Client != nil {
+		client = opts.deps.Client
+	} else {
+		client, err = deepsource.New(deepsource.ClientOpts{
+			Token:            cfg.Token,
+			HostName:         cfg.Host,
+			OnTokenRefreshed: cfgMgr.TokenRefreshCallback(),
+		})
+		if err != nil {
+			return err
+		}
 	}
 
 	commitOid := opts.commitOid
@@ -137,6 +186,10 @@ func (opts *AnalysisOptions) runDetail(ctx context.Context) error {
 		return clierrors.NewCLIError(clierrors.ErrAPIError, "Failed to fetch run details", err)
 	}
 
+	if opts.OutputFormat == "json" {
+		return opts.outputRunDetailJSON(runWithIssues)
+	}
+
 	commitShort := commitOid
 	if len(commitShort) > 8 {
 		commitShort = commitShort[:8]
@@ -162,6 +215,79 @@ func (opts *AnalysisOptions) runDetail(ctx context.Context) error {
 	return nil
 }
 
+// --- JSON output ---
+
+type AnalysisRunJSON struct {
+	CommitOid              string     `json:"commit_oid"`
+	BranchName             string     `json:"branch_name"`
+	Status                 string     `json:"status"`
+	OccurrencesIntroduced  int        `json:"occurrences_introduced"`
+	OccurrencesResolved    int        `json:"occurrences_resolved"`
+	OccurrencesSuppressed  int        `json:"occurrences_suppressed"`
+	FinishedAt             *time.Time `json:"finished_at"`
+}
+
+type RunDetailJSON struct {
+	CommitOid  string         `json:"commit_oid"`
+	BranchName string         `json:"branch_name"`
+	Status     string         `json:"status"`
+	Issues     []RunIssueJSON `json:"issues"`
+}
+
+type RunIssueJSON struct {
+	Path     string `json:"path"`
+	Title    string `json:"title"`
+	Code     string `json:"code"`
+	Category string `json:"category"`
+	Severity string `json:"severity"`
+}
+
+func (opts *AnalysisOptions) outputRunsJSON(analysisRuns []runs.AnalysisRun) error {
+	result := make([]AnalysisRunJSON, 0, len(analysisRuns))
+	for _, run := range analysisRuns {
+		result = append(result, AnalysisRunJSON{
+			CommitOid:              run.CommitOid,
+			BranchName:             run.BranchName,
+			Status:                 run.Status,
+			OccurrencesIntroduced:  run.OccurrencesIntroduced,
+			OccurrencesResolved:    run.OccurrencesResolved,
+			OccurrencesSuppressed:  run.OccurrencesSuppressed,
+			FinishedAt:             run.FinishedAt,
+		})
+	}
+	data, err := json.MarshalIndent(result, "", "  ")
+	if err != nil {
+		return clierrors.NewCLIError(clierrors.ErrAPIError, "Failed to format JSON output", err)
+	}
+	fmt.Fprintln(opts.stdout(), string(data))
+	return nil
+}
+
+func (opts *AnalysisOptions) outputRunDetailJSON(runWithIssues *runs.RunWithIssues) error {
+	issuesJSON := make([]RunIssueJSON, 0, len(runWithIssues.Issues))
+	for _, issue := range runWithIssues.Issues {
+		issuesJSON = append(issuesJSON, RunIssueJSON{
+			Path:     issue.Path,
+			Title:    issue.Title,
+			Code:     issue.IssueCode,
+			Category: issue.Category,
+			Severity: issue.Severity,
+		})
+	}
+	result := RunDetailJSON{
+		CommitOid:  runWithIssues.CommitOid,
+		BranchName: runWithIssues.BranchName,
+		Status:     runWithIssues.Status,
+		Issues:     issuesJSON,
+	}
+	data, err := json.MarshalIndent(result, "", "  ")
+	if err != nil {
+		return clierrors.NewCLIError(clierrors.ErrAPIError, "Failed to format JSON output", err)
+	}
+	fmt.Fprintln(opts.stdout(), string(data))
+	return nil
+}
+
 // --- Display helpers ---
 
 func showRunsTable(analysisRuns []runs.AnalysisRun) {

command/analysis/tests/analysis_test.go

@@ -0,0 +1,77 @@
+package tests
+
+import (
+	"bytes"
+	"path/filepath"
+	"runtime"
+	"strings"
+	"testing"
+
+	"github.com/deepsourcelabs/cli/command/cmddeps"
+	analysisCmd "github.com/deepsourcelabs/cli/command/analysis"
+	"github.com/deepsourcelabs/cli/deepsource"
+	"github.com/deepsourcelabs/cli/internal/testutil"
+)
+
+func goldenPath(name string) string {
+	_, callerFile, _, _ := runtime.Caller(0)
+	return filepath.Join(filepath.Dir(callerFile), "golden_files", name)
+}
+
+func TestAnalysisListRuns(t *testing.T) {
+	cfgMgr := testutil.CreateTestConfigManager(t, "test-token", "deepsource.com", "test@example.com")
+	mock := testutil.MockQueryFunc(t, map[string]string{
+		"GetAnalysisRuns": goldenPath("runs_list_response.json"),
+	})
+	client := deepsource.NewWithGraphQLClient(mock)
+
+	var buf bytes.Buffer
+	deps := &cmddeps.Deps{
+		Client:    client,
+		ConfigMgr: cfgMgr,
+		Stdout:    &buf,
+	}
+
+	cmd := analysisCmd.NewCmdAnalysisWithDeps(deps)
+	cmd.SetArgs([]string{"--repo", "gh/testowner/testrepo", "--output", "json"})
+
+	if err := cmd.Execute(); err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+
+	expected := string(testutil.LoadGoldenFile(t, goldenPath("runs_list_output.json")))
+	got := buf.String()
+
+	if strings.TrimSpace(got) != strings.TrimSpace(expected) {
+		t.Errorf("output mismatch.\nExpected:\n%s\nGot:\n%s", expected, got)
+	}
+}
+
+func TestAnalysisRunDetail(t *testing.T) {
+	cfgMgr := testutil.CreateTestConfigManager(t, "test-token", "deepsource.com", "test@example.com")
+	mock := testutil.MockQueryFunc(t, map[string]string{
+		"GetRunIssues": goldenPath("run_detail_response.json"),
+	})
+	client := deepsource.NewWithGraphQLClient(mock)
+
+	var buf bytes.Buffer
+	deps := &cmddeps.Deps{
+		Client:    client,
+		ConfigMgr: cfgMgr,
+		Stdout:    &buf,
+	}
+
+	cmd := analysisCmd.NewCmdAnalysisWithDeps(deps)
+	cmd.SetArgs([]string{"--repo", "gh/testowner/testrepo", "--commit", "abc123f", "--output", "json"})
+
+	if err := cmd.Execute(); err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+
+	expected := string(testutil.LoadGoldenFile(t, goldenPath("run_detail_output.json")))
+	got := buf.String()
+
+	if strings.TrimSpace(got) != strings.TrimSpace(expected) {
+		t.Errorf("output mismatch.\nExpected:\n%s\nGot:\n%s", expected, got)
+	}
+}

command/analysis/tests/golden_files/run_detail_output.json

@@ -0,0 +1,21 @@
+{
+  "commit_oid": "abc123f",
+  "branch_name": "main",
+  "status": "SUCCESS",
+  "issues": [
+    {
+      "path": "cmd/server/main.go",
+      "title": "Error return value not checked",
+      "code": "GO-R1005",
+      "category": "BUG_RISK",
+      "severity": "MAJOR"
+    },
+    {
+      "path": "internal/handler/auth.go",
+      "title": "Potential SQL injection",
+      "code": "GO-S1001",
+      "category": "SECURITY",
+      "severity": "CRITICAL"
+    }
+  ]
+}

command/analysis/tests/golden_files/run_detail_response.json

@@ -0,0 +1,52 @@
+{
+  "run": {
+    "runUid": "run-uid-001",
+    "commitOid": "abc123f",
+    "branchName": "main",
+    "status": "SUCCESS",
+    "checks": {
+      "edges": [
+        {
+          "node": {
+            "analyzer": {
+              "name": "Go",
+              "shortcode": "go"
+            },
+            "issues": {
+              "edges": [
+                {
+                  "node": {
+                    "source": "static",
+                    "path": "cmd/server/main.go",
+                    "beginLine": 42,
+                    "beginColumn": 5,
+                    "endLine": 42,
+                    "endColumn": 30,
+                    "title": "Error return value not checked",
+                    "shortcode": "GO-R1005",
+                    "category": "BUG_RISK",
+                    "severity": "MAJOR"
+                  }
+                },
+                {
+                  "node": {
+                    "source": "ai",
+                    "path": "internal/handler/auth.go",
+                    "beginLine": 15,
+                    "beginColumn": 1,
+                    "endLine": 20,
+                    "endColumn": 2,
+                    "title": "Potential SQL injection",
+                    "shortcode": "GO-S1001",
+                    "category": "SECURITY",
+                    "severity": "CRITICAL"
+                  }
+                }
+              ]
+            }
+          }
+        }
+      ]
+    }
+  }
+}