Fix AI intro video system and safety check

- Increase signal file TTL from 30s to 60s (allows time for menu navigation)
- Fix video path resolution using process.cwd() instead of __dirname
- Add LIKU_SAFE_LAUNCH env var to bypass safety check in spawned windows
- Tighten safety check to only trigger on TERM_PROGRAM=vscode
- Update documentation with correct launch commands

Author: TayDa64

.github/copilot-instructions.md

@@ -11,7 +11,11 @@
 > 2. **THEN READ:** Game-specific docs in `docs/ai/` (e.g., `LIKU-CHESS.md`)
 > 
 > The game MUST run in a **separate terminal window**, not in VS Code's integrated terminal.
-> See LIKU-CORE.md for the correct `Start-Process` command.
+> 
+> **Correct command:**
+> ```powershell
+> Start-Process pwsh -ArgumentList "-NoExit", "-Command", "cd C:\dev\Liku-AI; `$env:LIKU_SAFE_LAUNCH='1'; node dist/index.js"
+> ```
 
 ---
 

LIKU-CORE.md

@@ -8,7 +8,7 @@
 
 ```powershell
 # 1. Spawn game in separate window (REQUIRED)
-Start-Process pwsh -ArgumentList "-NoExit", "-Command", "cd C:\dev\Liku-AI; node dist/index.js"
+Start-Process pwsh -ArgumentList "-NoExit", "-Command", "cd C:\dev\Liku-AI; `$env:LIKU_SAFE_LAUNCH='1'; node dist/index.js"
 
 # 2. Read state file to see current screen
 Get-Content .\likubuddy-state.txt
@@ -53,7 +53,7 @@ node send-command.js --key enter
 ### 🪟 Windows
 
 ```powershell
-Start-Process pwsh -ArgumentList "-NoExit", "-Command", "cd C:\dev\Liku-AI; node dist/index.js"
+Start-Process pwsh -ArgumentList "-NoExit", "-Command", "cd C:\dev\Liku-AI; `$env:LIKU_SAFE_LAUNCH='1'; node dist/index.js"
 ```
 
 This spawns a window titled "LikuBuddy Game Window". All `send-keys.ps1` commands auto-target this window.
@@ -174,22 +174,22 @@ Register your identity before playing Chess to trigger your intro video:
 
 **Windows:**
 ```powershell
-# Signal file format: Line 1 = agent ID, Line 2 = timestamp (expires after 30s)
+# Signal file format: Line 1 = agent ID, Line 2 = timestamp (expires after 60s)
 New-Item -ItemType Directory -Force -Path "$env:USERPROFILE\.liku-ai" | Out-Null
 $timestamp = [DateTimeOffset]::UtcNow.ToUnixTimeMilliseconds()
 Set-Content -Path "$env:USERPROFILE\.liku-ai\current-agent.txt" -Value "claude`n$timestamp"
 ```
 
 **Linux/macOS:**
 ```bash
-# Signal file format: Line 1 = agent ID, Line 2 = timestamp (expires after 30s)
+# Signal file format: Line 1 = agent ID, Line 2 = timestamp (expires after 60s)
 mkdir -p ~/.liku-ai
 echo -e "claude\n$(date +%s%3N)" > ~/.liku-ai/current-agent.txt
 ```
 
 **Supported IDs:** `claude`, `gemini`, `chatgpt`, `grok` (and aliases like `anthropic`, `openai`, `google`, etc.)
 
-> **Note:** Signal files expire after 30 seconds to prevent stale agent detection from previous sessions.
+> **Note:** Signal files expire after 60 seconds to prevent stale agent detection from previous sessions.
 
 ### ⏳ Intro Video Handling (CRITICAL)
 

src/index.tsx

@@ -123,28 +123,33 @@ const checkAISafetyPreFlight = (): boolean => {
   // Skip check if --ai flag is set (agent knows what it's doing)
   if (cli.flags.ai) return true;
 
-  // Skip check if explicitly disabled
+  // Skip check if explicitly disabled (used by Start-Process command)
   if (process.env.LIKU_SKIP_SAFETY_CHECK === '1') return true;
 
+  // Skip if LIKU_SAFE_LAUNCH is set (inherited doesn't matter)
+  if (process.env.LIKU_SAFE_LAUNCH === '1') return true;
+  
   // Check for VS Code integrated terminal indicators
-  const isVSCodeTerminal = !!(
+  // Note: Start-Process inherits env vars, so we need additional checks
+  const hasVSCodeEnv = !!(
     process.env.VSCODE_INJECTION ||
     process.env.TERM_PROGRAM === 'vscode' ||
     process.env.VSCODE_GIT_IPC_HANDLE ||
     process.env.VSCODE_PID
   );
 
-  // Check for GitHub Copilot / AI tool indicators
-  const isLikelyAISession = !!(
-    process.env.GITHUB_COPILOT_AGENT ||
-    process.env.CURSOR_SESSION ||
-    process.env.AIDER_SESSION ||
-    // Parent process check for AI tools running terminal commands
-    process.ppid && process.env.VSCODE_PID
+  // Check if this is truly an inline terminal (not spawned via Start-Process)
+  // Start-Process creates a new console window, which has different characteristics
+  const isInteractiveVSCodeTerminal = hasVSCodeEnv && (
+    // Check if we're in the VS Code pseudo-terminal (PTY)
+    process.env.TERM_PROGRAM === 'vscode' ||
+    // Check for VS Code injection (extension host)
+    process.env.VSCODE_INJECTION === '1'
   );
 
-  // If in VS Code terminal AND looks like AI session, warn and exit
-  if (isVSCodeTerminal) {
+  // If in VS Code's interactive terminal, warn and exit
+  if (isInteractiveVSCodeTerminal) {
+    const cwd = process.cwd().replace(/\\/g, '/');
     console.log(`
 ╔══════════════════════════════════════════════════════════════════════╗
 ║                    ⚠️  LIKU-AI SAFETY CHECK  ⚠️                        ║
@@ -159,11 +164,9 @@ const checkAISafetyPreFlight = (): boolean => {
 ║  CORRECT LAUNCH METHOD (Windows PowerShell):                        ║
 ║                                                                      ║
 ║    Start-Process pwsh -ArgumentList "-NoExit", "-Command",          ║
-║      "cd ${process.cwd().replace(/\\/g, '/')}; node dist/index.js"                         ║
+║      "cd ${cwd}; \`$env:LIKU_SAFE_LAUNCH='1'; node dist/index.js"   ║
 ║                                                                      ║
-║  To bypass this check (humans only):                                ║
-║    - Set LIKU_SKIP_SAFETY_CHECK=1 environment variable              ║
-║    - Or run with: node dist/index.js --ai                           ║
+║  To bypass (experienced users): node dist/index.js --ai             ║
 ║                                                                      ║
 ╚══════════════════════════════════════════════════════════════════════╝
 `);

src/intro/IntroPlayer.ts

@@ -32,8 +32,8 @@ import { existsSync, readFileSync, writeFileSync, unlinkSync, mkdirSync, statSyn
 import path from 'path';
 import { fileURLToPath } from 'url';
 
-// Signal file staleness threshold (30 seconds)
-const SIGNAL_FILE_TTL_MS = 30 * 1000;
+// Signal file staleness threshold (60 seconds - allows time for menu navigation)
+const SIGNAL_FILE_TTL_MS = 60 * 1000;
 
 // ES module __dirname equivalent
 const __filename = fileURLToPath(import.meta.url);
@@ -334,6 +334,7 @@ export function detectAgent(): AIAgent | null {
   // Signal files older than SIGNAL_FILE_TTL_MS are considered stale
   const homeDir = process.env.HOME || process.env.USERPROFILE || '';
   const signalFile = path.join(homeDir, '.liku-ai', 'current-agent.txt');
+  
   if (existsSync(signalFile)) {
     try {
       const fileContent = readFileSync(signalFile, 'utf-8').trim();
@@ -375,7 +376,9 @@ export function detectAgent(): AIAgent | null {
  * Get the absolute path to an intro video
  */
 function getIntroVideoPath(agent: AIAgent): string | null {
-  const projectRoot = path.resolve(__dirname, '..', '..');
+  // __dirname is dist/intro/ when compiled, so go up to dist/, then to project root
+  // Or use process.cwd() which is more reliable for project root
+  const projectRoot = process.cwd();
 
   // Try MP4 first, then GIF
   const mp4Path = path.join(projectRoot, agent.introVideo);
@@ -389,6 +392,14 @@ function getIntroVideoPath(agent: AIAgent): string | null {
     return gifPath;
   }
 
+  // Fallback: Try relative to __dirname (for backward compatibility)
+  const fallbackRoot = path.resolve(__dirname, '..', '..');
+  const fallbackPath = path.join(fallbackRoot, agent.introVideo);
+  if (existsSync(fallbackPath)) {
+    return fallbackPath;
+  }
+  
+  console.log(`[Intro] Video not found: ${mp4Path} or ${fallbackPath}`);
   return null;
 }
 
@@ -397,21 +408,38 @@ function getIntroVideoPath(agent: AIAgent): string | null {
  * Returns the process info for state file notification
  */
 export function playIntroVideo(agent: AIAgent): { pid: number | null; duration: number } {
+  // Debug log helper
+  const debugLog = (msg: string) => {
+    try {
+      const line = `[${new Date().toISOString()}] [playIntroVideo] ${msg}\n`;
+      const debugPath = path.join(process.cwd(), 'intro-debug.txt');
+      const existing = existsSync(debugPath) ? readFileSync(debugPath, 'utf-8') : '';
+      writeFileSync(debugPath, existing + line);
+    } catch { /* ignore */ }
+  };
+  
+  debugLog(`Called for agent: ${agent.id}`);
+  
   const videoPath = getIntroVideoPath(agent);
 
+  debugLog(`Video path resolved: ${videoPath || 'null'}`);
+  
   if (!videoPath) {
+    debugLog(`No video file found for ${agent.name}`);
     console.log(`[Intro] No video file found for ${agent.name}`);
     return { pid: null, duration: 0 };
   }
 
   console.log(`[Intro] Playing: ${videoPath}`);
+  debugLog(`Playing video: ${videoPath}`);
 
   const plat = platform();
   const duration = agent.introDuration;
   let pid: number | null = null;
 
   try {
     if (plat === 'win32') {
+      debugLog(`Platform: win32, using Start-Process`);
       // Windows: Use Start-Process (proven to work) and spawn separate closer
       // Step 1: Open video immediately with Start-Process
       exec(`powershell -Command "Start-Process '${videoPath.replace(/'/g, "''")}'"`);
@@ -425,6 +453,7 @@ export function playIntroVideo(agent: AIAgent): { pid: number | null; duration:
       }).unref();
 
       pid = process.pid;
+      debugLog(`Video launched with pid: ${pid}`);
 
     } else if (plat === 'darwin') {
       // macOS: Use 'open' command and close after duration