@shaykec/claude-teach 0.6.7 → 0.6.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shaykec/claude-teach",
-  "version": "0.6.7",
+  "version": "0.6.8",
   "description": "Socratic AI teaching platform — learn anything through guided dialogue, visual canvas, and gamification",
   "type": "module",
   "main": "src/cli.js",
@@ -15,7 +15,7 @@
     "commander": "^12.0.0",
     "js-yaml": "^4.1.0",
     "chalk": "^5.3.0",
-    "@shaykec/bridge": "0.4.8",
+    "@shaykec/bridge": "0.4.9",
     "@shaykec/shared": "0.1.3",
     "@shaykec/plugin": "0.2.12",
     "@shaykec/extension": "0.1.0"

package/src/cli.e2e.test.js

@@ -674,7 +674,7 @@ describe('CLI E2E: Setup command', () => {
     const { stdout, exitCode } = runCli(['start', '--help']);
     expect(exitCode).toBe(0);
     const clean = stripAnsi(stdout);
-    expect(clean).toContain('Launch Claude Code');
+    expect(clean).toContain('Launch ClaudeTeach');
     expect(clean).toContain('--no-server');
     expect(clean).toContain('--port');
   });
@@ -881,7 +881,7 @@ describe('CLI E2E: Subcommand help texts', () => {
     { args: ['level-up', '--help'], contains: 'belt roadmap' },
     { args: ['serve', '--help'], contains: 'bridge server' },
     { args: ['inbox', '--help'], contains: 'captured from the browser' },
-    { args: ['start', '--help'], contains: 'Launch Claude Code' },
+    { args: ['start', '--help'], contains: 'Launch ClaudeTeach' },
     { args: ['setup', '--help'], contains: 'Check environment' },
     { args: ['registry:build', '--help'], contains: 'Rebuild the module registry' },
     { args: ['install', '--help'], contains: 'Install a module pack' },
@@ -916,7 +916,7 @@ describe('CLI E2E: npx comprehensive', () => {
   it('npx start --help shows launch description', () => {
     const { stdout, exitCode } = runNpx(['start', '--help'], { timeout: 60000 });
     expect(exitCode).toBe(0);
-    expect(stripAnsi(stdout)).toContain('Launch Claude Code');
+    expect(stripAnsi(stdout)).toContain('Launch ClaudeTeach');
   });
 
   it('npx setup --help shows setup options', () => {

package/src/cli.js

@@ -435,7 +435,8 @@ program
 // --- start ---
 program
   .command('start')
-  .description('Launch Claude Code with ClaudeTeach — starts bridge server + loads teaching skills')
+  .description('Launch ClaudeTeach — browser chat by default, or --terminal for CLI mode')
+  .option('--terminal', 'Use terminal CLI mode instead of browser chat')
   .option('--no-server', 'Skip starting the bridge server')
   .option('--no-browser', 'Skip opening the canvas in the browser')
   .option('--port <port>', 'Bridge server port', '3456')
@@ -470,10 +471,10 @@ program
         server = await new Promise((resolve, reject) => {
           const srv = startServer({
             port,
+            pluginDir,
             progressProvider: { getProgress: () => loadProgress(ROOT) },
             onReady: () => resolve(srv),
           });
-          // Handle port-in-use and other startup errors
           srv.server.on('error', (err) => {
             if (err.code === 'EADDRINUSE') {
               reject(new Error(`Port ${port} is already in use. Kill the old process or use --port <other>`));
@@ -492,38 +493,51 @@ program
     // 4. Open canvas in browser (unless --no-browser or --no-server)
     if (server && opts.browser !== false) {
       const canvasUrl = `http://localhost:${opts.port}`;
-      const platform = process.platform;
-      if (platform === 'darwin') {
-        exec(`open "${canvasUrl}"`);
-      } else if (platform === 'linux') {
-        exec(`xdg-open "${canvasUrl}"`);
-      } else if (platform === 'win32') {
-        exec(`start "" "${canvasUrl}"`);
-      }
+      openUrl(canvasUrl);
       console.log(chalk.dim(`  Canvas: ${canvasUrl}`));
     }
 
-    // 5. Launch
-    console.log(chalk.green('Launching Claude Code with ClaudeTeach...'));
-    console.log(chalk.dim(`  Plugin: ${pluginDir}`));
+    if (opts.terminal) {
+      // --- Terminal mode: spawn Claude Code with stdio: 'inherit' (legacy) ---
+      console.log(chalk.green('Launching Claude Code in terminal mode...'));
+      console.log(chalk.dim(`  Plugin: ${pluginDir}`));
 
-    const claudeArgs = [
-      '--plugin-dir', pluginDir,
-      '--allowedTools', 'Bash(*),Read,Write,Edit,Glob,Grep',
-      ...cmd.args,
-    ];
-    const child = spawn('claude', claudeArgs, { stdio: 'inherit' });
+      const claudeArgs = [
+        '--plugin-dir', pluginDir,
+        '--allowedTools', 'Bash(*),Read,Write,Edit,Glob,Grep',
+        ...cmd.args,
+      ];
+      const child = spawn('claude', claudeArgs, { stdio: 'inherit' });
 
-    child.on('error', (err) => {
-      console.error(chalk.red(`Failed to start Claude Code: ${err.message}`));
-      if (server) server.close();
-      process.exit(1);
-    });
+      child.on('error', (err) => {
+        console.error(chalk.red(`Failed to start Claude Code: ${err.message}`));
+        if (server) server.close();
+        process.exit(1);
+      });
 
-    child.on('close', (code) => {
-      if (server) server.close();
-      process.exit(code ?? 0);
-    });
+      child.on('close', (code) => {
+        if (server) server.close();
+        process.exit(code ?? 0);
+      });
+    } else {
+      // --- Browser chat mode (default): Claude runs via SDK in the bridge ---
+      console.log(chalk.green('ClaudeTeach is ready — chat in your browser!'));
+      console.log(chalk.dim(`  Plugin: ${pluginDir}`));
+      console.log(chalk.dim(`  Claude Code runs via the Agent SDK (no terminal TTY)`));
+      console.log(chalk.dim(`  Use --terminal flag for classic CLI mode`));
+
+      // Keep the process alive — the bridge server handles everything.
+      // On SIGINT, clean up and exit.
+      process.on('SIGINT', () => {
+        console.log(chalk.dim('\nShutting down...'));
+        if (server) server.close();
+        process.exit(0);
+      });
+      process.on('SIGTERM', () => {
+        if (server) server.close();
+        process.exit(0);
+      });
+    }
   });
 
 // --- setup ---