feat: Track session upload origin with explicit values

Implement origin tracking for Mixpanel analytics without requiring
users to reinstall hooks:

Changes:
- Renamed hookTrigger → origin internally for clarity
- Explicit origin values: manual-upload, hook-sessionstart,
  hook-precompact, hook-sessionend
- No fallbacks - null values indicate bugs (visible in analytics)
- Removed --origin CLI flag (never used, only added confusion)
- Removed misleading [DEPRECATED] labels
- Hook commands unchanged - continue using --hook-trigger
- CLI maps --hook-trigger to origin format internally

Header sent to server:
- x-vibe-config-hook-trigger → x-vibe-config-origin

Backwards compatibility:
- Installed hooks continue using --hook-trigger flag
- No reinstallation required
- Mapping happens transparently in CLI

All 716 tests passing.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Author: dannyshmueli

src/commands/send.ts

@@ -99,6 +99,11 @@ export async function send(options: SendOptions): Promise<void> {
  * Execute send in interactive mode with full UI
  */
 async function executeInteractiveSend(options: SendOptions): Promise<void> {
+  // Explicitly set origin for manual uploads (no fallback)
+  if (!options.origin) {
+    options.origin = 'manual-upload';
+  }
+
   const orchestrator = new SendOrchestrator();
   const progressUI = new SendProgressUI(options.silent);
   const summaryUI = new SendSummaryUI(options.silent);

src/index.ts

@@ -105,12 +105,16 @@ program
   .option('-a, --all', 'Send sessions from all projects (default: current project only)')
   .option('--silent', 'Run in silent mode (for hook execution)')
   .option('--background', 'Run upload in background (for hooks)')
-  .option('--hook-trigger <type>', 'Hook that triggered this command (sessionstart, precompact, sessionend)')
+  .option('--hook-trigger <type>', 'Hook trigger type (used by installed hooks)')
   .option('--hook-version <version>', 'Hook version (for tracking hook updates)')
   .option('--test', 'Test mode for hook validation (exits without processing)')
   .option('--claude-project-dir <dir>', 'Claude project directory from $CLAUDE_PROJECT_DIR')
   .action(async (options) => {
     try {
+      // Map hook-trigger to origin format for internal use
+      if (options.hookTrigger) {
+        options.origin = `hook-${options.hookTrigger}`;
+      }
       await sendWithTimeout(options);
     } catch (error) {
       handleError(error);

src/lib/api-client.ts

@@ -61,7 +61,7 @@ export interface UploadResult {
 
 export interface CLIConfiguration {
   version: string;
-  hookTrigger?: string | null;
+  origin?: string | null;
   statusline: {
     personality: 'gordon' | 'vibe-log' | 'custom';
     customPersonality?: {
@@ -89,7 +89,7 @@ function generateRequestId(): string {
 }
 
 // Gather current CLI configuration for sending to server
-async function gatherCLIConfiguration(hookTrigger?: string): Promise<CLIConfiguration | null> {
+async function gatherCLIConfiguration(origin?: string): Promise<CLIConfiguration | null> {
   try {
     // Get statusline configuration
     const statuslineConfig = getStatusLinePersonality();
@@ -99,7 +99,7 @@ async function gatherCLIConfiguration(hookTrigger?: string): Promise<CLIConfigur
 
     return {
       version: require('../../package.json').version,
-      hookTrigger: hookTrigger || null,
+      origin: origin || null,  // No fallback - must be explicitly set
       statusline: {
         personality: statuslineConfig.personality,
         customPersonality: statuslineConfig.customPersonality
@@ -393,10 +393,10 @@ class SecureApiClient {
   async uploadSessions(
     sessions: Session[],
     onProgress?: (current: number, total: number, sizeKB?: number) => void,
-    hookTrigger?: string
+    origin?: string
   ): Promise<any> {
     // Gather CLI configuration to send with the upload
-    const cliConfig = await gatherCLIConfiguration(hookTrigger);
+    const cliConfig = await gatherCLIConfiguration(origin);
 
     // Validate and sanitize sessions
     const sanitizedSessions = sessions.map(session => this.sanitizeSession(session));
@@ -476,8 +476,8 @@ class SecureApiClient {
           const configHeaders: Record<string, string> = {};
           if (cliConfig) {
             configHeaders['x-vibe-config-version'] = cliConfig.version;
-            if (cliConfig.hookTrigger) {
-              configHeaders['x-vibe-config-hook-trigger'] = cliConfig.hookTrigger;
+            if (cliConfig.origin) {
+              configHeaders['x-vibe-config-origin'] = cliConfig.origin;
             }
             configHeaders['x-vibe-config-statusline'] = JSON.stringify(cliConfig.statusline);
             configHeaders['x-vibe-config-hooks'] = JSON.stringify(cliConfig.hooks);

src/lib/hooks/hooks-controller.ts

@@ -187,11 +187,14 @@ export function buildHookCommand(
   hookTrigger: 'sessionstart' | 'precompact' | 'sessionend',
   mode?: 'all' | 'selected'
 ): string {
+  // Keep using --hook-trigger for backwards compatibility with existing installed hooks
+  // The mapping to origin happens in index.ts action handler
+
   // For global mode (track all), use --all flag instead of --claude-project-dir
   if (mode === 'all') {
     return `${cliPath} send --silent --background --hook-trigger=${hookTrigger} --hook-version=${HOOKS_VERSION} --all`;
   }
-  
+
   // For selected mode or backward compatibility, use --claude-project-dir
   return `${cliPath} send --silent --background --hook-trigger=${hookTrigger} --hook-version=${HOOKS_VERSION} --claude-project-dir="$CLAUDE_PROJECT_DIR"`;
 }

src/lib/orchestrators/send-orchestrator.ts

@@ -24,7 +24,8 @@ export interface SendOptions {
   dry?: boolean;
   all?: boolean;
   silent?: boolean;
-  hookTrigger?: string;
+  origin?: string;  // Origin of upload: 'manual-upload', 'hook-sessionstart', 'hook-precompact', 'hook-sessionend'
+  hookTrigger?: string;  // Hook trigger (mapped to origin internally)
   hookVersion?: string;
   test?: boolean;
   background?: boolean;
@@ -358,7 +359,9 @@ export class SendOrchestrator {
         console.log('[DEBUG] SendOrchestrator.uploadSessions called with', apiSessions.length, 'sessions');
       }
       logger.debug(`Uploading ${apiSessions.length} sessions to API`);
-      const result = await apiClient.uploadSessions(apiSessions, onProgress, options.hookTrigger);
+      // Map hookTrigger to origin format (hookTrigger is used by installed hooks)
+      const origin = options.origin || (options.hookTrigger ? `hook-${options.hookTrigger}` : undefined);
+      const result = await apiClient.uploadSessions(apiSessions, onProgress, origin);
       if (process.env.VIBELOG_DEBUG === 'true') {
         console.log('[DEBUG] Upload completed successfully');
       }

tests/unit/commands/send-directory-filter.test.ts

@@ -178,7 +178,7 @@ describe('Send Command - Directory Filtering', () => {
           })
         ]),
         expect.any(Function), // Progress callback
-        undefined // hookTrigger
+        "manual-upload" // origin - explicitly set for manual uploads
       );
     });
 

tests/unit/lib/hooks/hooks-controller.test.ts

@@ -61,7 +61,7 @@ describe('Hooks Controller', () => {
       it('should maintain backward compatibility when mode is undefined', () => {
         const cliPath = 'npx vibe-log-cli';
         const command = buildHookCommand(cliPath, 'sessionstart');
-        
+
         // Should default to current behavior (with --claude-project-dir)
         expect(command).not.toContain('--all');
         expect(command).toContain('--claude-project-dir="$CLAUDE_PROJECT_DIR"');
@@ -70,10 +70,10 @@ describe('Hooks Controller', () => {
 
       it('should handle both sessionstart and precompact triggers with mode="all"', () => {
         const cliPath = 'node /path/to/vibe-log.js';
-        
+
         const sessionStartCommand = buildHookCommand(cliPath, 'sessionstart', 'all');
         expect(sessionStartCommand).toBe('node /path/to/vibe-log.js send --silent --background --hook-trigger=sessionstart --hook-version=1.0.0 --all');
-        
+
         const preCompactCommand = buildHookCommand(cliPath, 'precompact', 'all');
         expect(preCompactCommand).toBe('node /path/to/vibe-log.js send --silent --background --hook-trigger=precompact --hook-version=1.0.0 --all');
       });

tests/unit/lib/orchestrators/send-orchestrator.test.ts

@@ -401,6 +401,7 @@ describe('SendOrchestrator', () => {
       ).rejects.toThrow('Network failure');
 
       // Error should have been logged
+      // Third parameter is origin (should be undefined for error test)
       expect(mockApiClient.uploadSessions).toHaveBeenCalledWith(apiSessions, undefined, undefined);
     });