@tanagram/cli 0.1.31 → 0.1.33

package/commands/run.go

@@ -75,7 +75,7 @@ func Run() error {
 		// Get API key once upfront before parallel processing
 		apiKey, err := config.GetAPIKey()
 		if err != nil {
-			return fmt.Errorf("failed to get API key: %w", err)
+			return err
 		}
 
 		fmt.Printf("\nSyncing policies with LLM (processing %d changed file(s) in parallel)...\n", len(filesToSync))
@@ -214,7 +214,7 @@ func Run() error {
 	// Get API key once upfront before checking
 	apiKey, err := config.GetAPIKey()
 	if err != nil {
-		return fmt.Errorf("failed to get API key: %w", err)
+		return err
 	}
 
 	// Check changes against policies (both regex and LLM-based)

package/commands/sync.go

@@ -235,7 +235,7 @@ func FindInstructionFiles(gitRoot string) ([]string, error) {
 func getAPIKey() (string, error) {
 	apiKey, err := config.GetAPIKey()
 	if err != nil {
-		return "", fmt.Errorf("failed to get API key: %w", err)
+		return "", err
 	}
 	return apiKey, nil
 }

package/config/config.go

@@ -114,7 +114,7 @@ func GetAPIKey() (string, error) {
 	fmt.Print("Validating API key... ")
 	if err := ValidateAPIKey(apiKey); err != nil {
 		fmt.Println("✗")
-		return "", fmt.Errorf("invalid API key: %w", err)
+		return "", err
 	}
 	fmt.Println("✓")
 

package/install.js

@@ -79,15 +79,8 @@ function buildBinary(goCommand) {
 
   console.log('🔧 Building Tanagram CLI...');
 
-  // Build ldflags to inject PostHog key at build time
-  let ldflags = '';
-  if (process.env.POSTHOG_WRITE_KEY) {
-    ldflags = `-ldflags="-X 'github.com/tanagram/cli/metrics.posthogWriteKey=${process.env.POSTHOG_WRITE_KEY}'"`;
-    console.log('  📊 Injecting PostHog metrics key...');
-  }
-
   try {
-    execSync(`"${goCommand}" build ${ldflags} -o "${binaryPath}" .`, {
+    execSync(`"${goCommand}" build -o "${binaryPath}" .`, {
       cwd: __dirname,
       stdio: 'inherit',
       env: {

package/main.go

@@ -10,9 +10,18 @@ import (
 )
 
 func main() {
-	// Initialize metrics (similar to PosthogService initialization)
 	metrics.Init()
-	defer metrics.Close()
+
+	exitCode := 0
+	defer func() {
+		metrics.Track("cli.exit", map[string]interface{}{
+			"exit_code": exitCode,
+		})
+		metrics.Close()
+		// os.Exit immediately exits without calling other `defer`s, so we need to group these two statements
+		// and call them in the right order.
+		os.Exit(exitCode)
+	}()
 
 	// Get subcommand (default to "run" if none provided)
 	subcommand := "run"
@@ -20,20 +29,24 @@ func main() {
 		subcommand = os.Args[1]
 	}
 
+	metrics.Track("cli.start", map[string]interface{}{
+		"subcommand": subcommand,
+	})
+
 	var err error
 	switch subcommand {
 	case "run":
-		metrics.Track("command.execute", map[string]interface{}{
+		metrics.Track("cli.command.execute", map[string]interface{}{
 			"command": "run",
 		})
 		err = commands.Run()
 	case "sync":
-		metrics.Track("command.execute", map[string]interface{}{
+		metrics.Track("cli.command.execute", map[string]interface{}{
 			"command": "sync",
 		})
 		err = commands.Sync()
 	case "list":
-		metrics.Track("command.execute", map[string]interface{}{
+		metrics.Track("cli.command.execute", map[string]interface{}{
 			"command": "list",
 		})
 		err = commands.List()
@@ -44,32 +57,35 @@ func main() {
 			fmt.Fprintf(os.Stderr, "\nAvailable subcommands:\n")
 			fmt.Fprintf(os.Stderr, "  claude    Setup Claude Code hook\n")
 			fmt.Fprintf(os.Stderr, "  list      Show hook installation status\n")
-			os.Exit(1)
+			exitCode = 1
+			return
 		}
 		subCmd := os.Args[2]
 		switch subCmd {
 		case "claude":
-			metrics.Track("command.execute", map[string]interface{}{
+			metrics.Track("cli.command.execute", map[string]interface{}{
 				"command": "config.claude",
 			})
 			err = commands.ConfigClaude()
 		case "list":
-			metrics.Track("command.execute", map[string]interface{}{
+			metrics.Track("cli.command.execute", map[string]interface{}{
 				"command": "config.list",
 			})
 			err = commands.ConfigList()
 		default:
 			fmt.Fprintf(os.Stderr, "Unknown config subcommand: %s\n", subCmd)
-			os.Exit(1)
+			exitCode = 1
+			return
 		}
 	case "welcome":
-		metrics.Track("command.execute", map[string]interface{}{
+		metrics.Track("cli.command.execute", map[string]interface{}{
 			"command": "welcome",
 		})
 		choice, err := tui.RunWelcomeScreen()
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "Error: %v\n", err)
-			os.Exit(1)
+			exitCode = 1
+			return
 		}
 		switch choice {
 		case tui.ChoiceImportPolicies:
@@ -83,13 +99,14 @@ func main() {
 		}
 		return
 	case "puzzle":
-		metrics.Track("command.execute", map[string]interface{}{
+		metrics.Track("cli.command.execute", map[string]interface{}{
 			"command": "puzzle",
 		})
 		err := tui.RunPuzzleEditor()
 		if err != nil {
 			fmt.Fprintf(os.Stderr, "Error: %v\n", err)
-			os.Exit(1)
+			exitCode = 1
+			return
 		}
 		return
 	case "help", "-h", "--help":
@@ -98,16 +115,18 @@ func main() {
 	default:
 		fmt.Fprintf(os.Stderr, "Unknown command: %s\n\n", subcommand)
 		printHelp()
-		os.Exit(1)
+		exitCode = 1
+		return
 	}
 
 	if err != nil {
-		metrics.Track("command.error", map[string]interface{}{
+		metrics.Track("cli.command.error", map[string]interface{}{
 			"command": subcommand,
 			"error":   err.Error(),
 		})
 		fmt.Fprintf(os.Stderr, "Error: %v\n", err)
-		os.Exit(1)
+		exitCode = 1
+		return
 	}
 }
 

package/metrics/metrics.go

@@ -12,7 +12,7 @@ var (
 	client posthog.Client
 
 	// Injected at build time via -ldflags
-	posthogWriteKey = ""
+	posthogWriteKey = "phc_sMsUvf0nK50rZdztSlX9rDJqIreLcXj4dyGS0tORQpQ"
 	posthogHost     = "https://us.i.posthog.com"
 )
 
@@ -76,10 +76,16 @@ func Track(event string, properties map[string]interface{}) {
 	properties["arch"] = runtime.GOARCH
 	properties["cli_version"] = getVersion()
 
+	// Build properties - include all custom properties from the caller
+	props := posthog.NewProperties()
+	for key, value := range properties {
+		props.Set(key, value)
+	}
+
 	err := client.Enqueue(posthog.Capture{
 		DistinctId: getDistinctId(),
 		Event:      event,
-		Properties: properties,
+		Properties: props,
 		Timestamp:  time.Now(),
 	})
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tanagram/cli",
-  "version": "0.1.31",
+  "version": "0.1.33",
   "description": "Tanagram - Catch sloppy code before it ships",
   "main": "index.js",
   "bin": {