jettypod 4.4.106 → 4.4.107

package/apps/dashboard/app/api/claude/sessions/route.ts

@@ -1,5 +1,5 @@
 import { NextResponse } from 'next/server';
-import { listSessions, createSession, linkSession, getSessionByWorkItem } from '@/lib/db';
+import { listSessions, createSession, linkSession, getSessionByWorkItem, closeSession } from '@/lib/db';
 
 export const dynamic = 'force-dynamic';
 
@@ -59,3 +59,25 @@ export async function PATCH(request: Request) {
     return NextResponse.json({ error: 'Failed to link session' }, { status: 500 });
   }
 }
+
+// DELETE /api/claude/sessions - Close a session
+export async function DELETE(request: Request) {
+  try {
+    const { searchParams } = new URL(request.url);
+    const sessionId = searchParams.get('sessionId');
+
+    if (!sessionId) {
+      return NextResponse.json({ error: 'sessionId required' }, { status: 400 });
+    }
+
+    const success = closeSession(parseInt(sessionId, 10));
+    if (!success) {
+      return NextResponse.json({ error: 'Session not found' }, { status: 404 });
+    }
+
+    return NextResponse.json({ success: true });
+  } catch (error) {
+    console.error('Failed to close session:', error);
+    return NextResponse.json({ error: 'Failed to close session' }, { status: 500 });
+  }
+}

package/apps/dashboard/components/ClaudePanel.tsx

@@ -40,6 +40,7 @@ interface ClaudePanelProps {
   standaloneSessions?: SessionItem[];
   onSelectSession?: (sessionId: string) => void;
   onNewSession?: () => void;
+  onCloseSession?: (sessionId: string) => void;
   showSessionList?: boolean;
 }
 
@@ -62,6 +63,7 @@ export function ClaudePanel({
   standaloneSessions = [],
   onSelectSession,
   onNewSession,
+  onCloseSession,
   showSessionList: showSessionListProp,
 }: ClaudePanelProps) {
   // Show session list by default when no active session, or when explicitly requested
@@ -218,6 +220,7 @@ export function ClaudePanel({
               sessions={standaloneSessions}
               onSelectSession={onSelectSession || (() => {})}
               onNewSession={onNewSession || (() => {})}
+              onCloseSession={onCloseSession}
             />
           ) : (
             <>

package/apps/dashboard/components/RealTimeKanbanWrapper.tsx

@@ -533,6 +533,32 @@ function RealTimeKanbanContent({ initialData, initialDecisions }: RealTimeKanban
     setShowSessionList(false);
   }, [sessions, standaloneSessions]);
 
+  const handleCloseSession = useCallback(async (sessionId: string) => {
+    // Call API to mark as completed in DB
+    try {
+      await fetch(`/api/claude/sessions?sessionId=${sessionId}`, {
+        method: 'DELETE',
+      });
+    } catch (error) {
+      console.error('Failed to close session:', error);
+    }
+
+    // Remove from standalone sessions list
+    setStandaloneSessions(prev => prev.filter(s => s.id !== sessionId));
+
+    // Remove from in-memory sessions
+    setSessions(prev => {
+      const updated = new Map(prev);
+      updated.delete(sessionId);
+      return updated;
+    });
+
+    // If this was the active session, clear it
+    if (activeSessionId === sessionId) {
+      setActiveSessionId(null);
+    }
+  }, [activeSessionId]);
+
   const handleOpenSessionList = useCallback(() => {
     setShowSessionList(true);
     setClaudePanelOpen(true);
@@ -639,6 +665,7 @@ function RealTimeKanbanContent({ initialData, initialDecisions }: RealTimeKanban
         standaloneSessions={standaloneSessions}
         onSelectSession={handleSelectStandaloneSession}
         onNewSession={handleNewSession}
+        onCloseSession={handleCloseSession}
         showSessionList={showSessionList}
       />
     </div>

package/apps/dashboard/components/SessionList.tsx

@@ -14,12 +14,14 @@ interface SessionListProps {
   sessions: SessionItem[];
   onSelectSession: (sessionId: string) => void;
   onNewSession: () => void;
+  onCloseSession?: (sessionId: string) => void;
 }
 
 export function SessionList({
   sessions,
   onSelectSession,
   onNewSession,
+  onCloseSession,
 }: SessionListProps) {
   return (
     <div className="flex-1 flex flex-col" data-testid="session-list">
@@ -46,31 +48,48 @@ export function SessionList({
         ) : (
           <div className="divide-y divide-zinc-800">
             {sessions.map((session) => (
-              <motion.button
+              <div
                 key={session.id}
-                onClick={() => onSelectSession(session.id)}
-                className="w-full px-4 py-3 text-left hover:bg-zinc-800/50 transition-colors"
-                whileHover={{ x: 4 }}
+                className="flex items-center hover:bg-zinc-800/50 transition-colors"
                 data-testid={`session-item-${session.id}`}
               >
-                <div className="flex items-center gap-2">
-                  <SessionIcon hasFeature={!!session.featureId} />
-                  <div className="flex-1 min-w-0">
-                    <p className="text-sm font-medium text-white truncate">
-                      {session.featureId ? session.featureTitle : session.title}
-                    </p>
-                    {session.featureId && (
-                      <span className="inline-flex items-center px-1.5 py-0.5 mt-1 text-xs font-medium bg-blue-900/50 text-blue-300 rounded">
-                        #{session.featureId}
-                      </span>
-                    )}
-                    {!session.featureId && (
-                      <p className="text-xs text-zinc-500 mt-0.5">Unlinked session</p>
-                    )}
+                <motion.button
+                  onClick={() => onSelectSession(session.id)}
+                  className="flex-1 px-4 py-3 text-left"
+                  whileHover={{ x: 4 }}
+                >
+                  <div className="flex items-center gap-2">
+                    <SessionIcon hasFeature={!!session.featureId} />
+                    <div className="flex-1 min-w-0">
+                      <p className="text-sm font-medium text-white truncate">
+                        {session.featureId ? session.featureTitle : session.title}
+                      </p>
+                      {session.featureId && (
+                        <span className="inline-flex items-center px-1.5 py-0.5 mt-1 text-xs font-medium bg-blue-900/50 text-blue-300 rounded">
+                          #{session.featureId}
+                        </span>
+                      )}
+                      {!session.featureId && (
+                        <p className="text-xs text-zinc-500 mt-0.5">Unlinked session</p>
+                      )}
+                    </div>
+                    <ChevronIcon />
                   </div>
-                  <ChevronIcon />
-                </div>
-              </motion.button>
+                </motion.button>
+                {onCloseSession && (
+                  <button
+                    onClick={(e) => {
+                      e.stopPropagation();
+                      onCloseSession(session.id);
+                    }}
+                    className="p-2 mr-2 rounded hover:bg-zinc-700 text-zinc-500 hover:text-zinc-300 transition-colors"
+                    aria-label="Close session"
+                    data-testid={`close-session-${session.id}`}
+                  >
+                    <CloseIcon />
+                  </button>
+                )}
+              </div>
             ))}
           </div>
         )}
@@ -118,3 +137,21 @@ function ChevronIcon() {
     </svg>
   );
 }
+
+function CloseIcon() {
+  return (
+    <svg
+      className="w-4 h-4"
+      fill="none"
+      stroke="currentColor"
+      viewBox="0 0 24 24"
+    >
+      <path
+        strokeLinecap="round"
+        strokeLinejoin="round"
+        strokeWidth={2}
+        d="M6 18L18 6M6 6l12 12"
+      />
+    </svg>
+  );
+}

package/apps/dashboard/hooks/useClaudeSessions.ts

@@ -23,6 +23,7 @@ interface UseClaudeSessionsReturn {
   switchSession: (workItemId: string) => void;
   stopSession: (workItemId: string) => void;
   retrySession: (workItemId: string) => void;
+  closeSession: (workItemId: string, dbSessionId?: number) => Promise<void>;
   closeAllSessions: () => void;
   isSessionRunning: (workItemId: string) => boolean;
 }
@@ -234,6 +235,38 @@ export function useClaudeSessions(): UseClaudeSessionsReturn {
     }
   }, [sessions, startSession]);
 
+  const closeSession = useCallback(async (workItemId: string, dbSessionId?: number) => {
+    const session = sessions.get(workItemId);
+
+    // Abort if running
+    if (session?.abortController) {
+      session.abortController.abort();
+    }
+
+    // Call API to mark as completed in DB (if we have a DB session ID)
+    if (dbSessionId) {
+      try {
+        await fetch(`/api/claude/sessions?sessionId=${dbSessionId}`, {
+          method: 'DELETE',
+        });
+      } catch (error) {
+        console.error('Failed to close session in DB:', error);
+      }
+    }
+
+    // Remove from local state
+    setSessions((prev) => {
+      const newMap = new Map(prev);
+      newMap.delete(workItemId);
+      return newMap;
+    });
+
+    // If this was the active session, clear it
+    if (activeSessionId === workItemId) {
+      setActiveSessionId(null);
+    }
+  }, [sessions, activeSessionId]);
+
   const closeAllSessions = useCallback(() => {
     sessions.forEach((session) => {
       if (session.abortController) {
@@ -259,6 +292,7 @@ export function useClaudeSessions(): UseClaudeSessionsReturn {
     switchSession,
     stopSession,
     retrySession,
+    closeSession,
     closeAllSessions,
     isSessionRunning,
   };

package/apps/dashboard/lib/db.ts

@@ -649,3 +649,18 @@ export function getSession(sessionId: number): ClaudeSession | null {
     db.close();
   }
 }
+
+// Close a session by ID (mark as completed)
+export function closeSession(sessionId: number): boolean {
+  const db = getWriteDb();
+  try {
+    const result = db.prepare(`
+      UPDATE claude_sessions
+      SET status = 'completed', completed_at = datetime('now')
+      WHERE id = ?
+    `).run(sessionId);
+    return result.changes > 0;
+  } finally {
+    db.close();
+  }
+}

package/jettypod.js

@@ -302,7 +302,9 @@ JettyPod: Structured workflow system with skills that guide complex workflows.
 
 ## ⚠️ CRITICAL: All Work Starts with request-routing
 
-**WHY:** request-routing creates a safe workspace (worktree) where your changes can actually be committed. Without it, you'll edit files on main, then discover the pre-commit hook blocks you—leaving uncommitted changes that can't be saved.
+**WHY:** The workflow initiated by request-routing eventually creates a safe workspace (worktree) where your changes can actually be committed. Without starting this workflow, you'll edit files on main, then discover the pre-commit hook blocks you—leaving uncommitted changes that can't be saved.
+
+**WHEN WORKTREE IS CREATED:** The worktree is created when \`jettypod work start\` runs—this happens in mode skills (speed-mode, stable-mode, chore-mode, bug-mode) or simple-improvement, NOT immediately in request-routing. Planning skills (epic-planning, feature-planning, chore-planning, bug-planning) are READ-ONLY investigation phases—do not edit files during planning.
 
 **FIRST RESPONSE RULE:** If user describes ANY code change, your FIRST action must be invoking request-routing. Not after reading files. Not after understanding the problem. FIRST.
 
@@ -313,10 +315,10 @@ JettyPod: Structured workflow system with skills that guide complex workflows.
 - "refactor", "migrate", "upgrade" (technical work)
 - "I noticed...", "I'm thinking..." (when followed by desired change)
 
-**⚠️ ANTI-PATTERN: Do NOT edit files before invoking request-routing.**
+**⚠️ ANTI-PATTERN: Do NOT edit files before the worktree exists.**
 Wrong: See problem → edit files → realize you're on main → ask about workflow
 Wrong: Read files → understand problem → try to fix → get blocked → create work item
-Right: User describes work → invoke request-routing → skill creates worktree → then edit
+Right: User describes work → invoke request-routing → complete workflow until \`work start\` creates worktree → then edit
 
 ## ⚠️ CRITICAL: Skills are MANDATORY for workflows
 Skills auto-activate and MUST complete their full workflow:

package/lib/git-hooks/post-merge

@@ -48,8 +48,9 @@ function backupDatabase() {
 
     fs.copyFileSync(sourcePath, backupPath);
 
-    // Stage the backup file
-    execSync(`git add "${backupPath}"`, {
+    // Stage the backup file (force-add because .jettypod-backup is gitignored
+    // to prevent worktree symlink corruption, but backups must be committed)
+    execSync(`git add -f "${backupPath}"`, {
       stdio: ['pipe', 'pipe', 'pipe']
     });
 

package/lib/git-hooks/pre-commit

@@ -43,7 +43,12 @@ function checkBranchRestriction() {
     console.error('  1. Undo your edits:     git checkout .');
     console.error('  2. Start the workflow:  Invoke request-routing skill');
     console.error('');
-    console.error('The skill creates a worktree where commits are allowed.');
+    console.error('HOW IT WORKS:');
+    console.error('  request-routing → planning skill → mode skill → work start');
+    console.error('');
+    console.error('  The worktree is created when `jettypod work start` runs.');
+    console.error('  Planning skills are READ-ONLY investigation phases.');
+    console.error('  Only edit files AFTER the worktree exists.');
     console.error('');
     return false;
   } catch (err) {

package/lib/work-tracking/index.js

@@ -558,6 +558,17 @@ function updateStatus(id, status) {
 
               // CRITICAL: Merge to main BEFORE cleanup
               if (isGitRepo()) {
+                // Check if branch exists before attempting merge
+                const branchExists = execSync(`git branch --list ${worktree.branch_name}`, {
+                  cwd: gitRoot,
+                  encoding: 'utf8',
+                  stdio: 'pipe'
+                }).trim();
+
+                if (!branchExists) {
+                  // Branch is gone (likely already merged/deleted) - skip merge, just cleanup stale record
+                  console.warn(`⚠️  Branch "${worktree.branch_name}" no longer exists - cleaning up stale record`);
+                } else {
                 try {
                   // Test merge to detect conflicts (without actually merging)
                   const mergeResult = execSync(`git merge --no-commit --no-ff ${worktree.branch_name}`, {
@@ -624,6 +635,7 @@ function updateStatus(id, status) {
                   console.warn(`   Worktree preserved for manual resolution`);
                   throw new Error('Merge failed - worktree preserved');
                 }
+                } // end else (branch exists)
               }
 
               // Only cleanup after successful merge

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jettypod",
-  "version": "4.4.106",
+  "version": "4.4.107",
   "description": "AI-powered development workflow manager with TDD, BDD, and automatic test generation",
   "main": "jettypod.js",
   "bin": {

package/skills-templates/bug-planning/SKILL.md

@@ -17,6 +17,25 @@ description: Guide structured bug investigation with symptom capture, hypothesis
 
 Guides Claude through systematic bug investigation. Produces a bug work item with clear breadcrumbs for implementation.
 
+## ⚠️ READ-ONLY PHASE
+
+**This skill is an investigation phase. No worktree exists yet.**
+
+🚫 **FORBIDDEN during this skill:**
+- Writing or editing any code files
+- Creating new files
+- Making implementation changes
+- Adding temporary debugging code
+
+✅ **ALLOWED during this skill:**
+- Reading files to understand the codebase
+- Running `jettypod` commands to create work items
+- Running diagnostic commands (git log, grep, etc.)
+- Asking the user questions
+- Analyzing symptoms and forming hypotheses
+
+**The worktree is created in Phase 6** when `jettypod work start` runs before invoking bug-mode.
+
 ## Instructions
 
 When this skill is activated, you are investigating a bug to identify root cause and plan the fix.

package/skills-templates/chore-planning/SKILL.md

@@ -7,6 +7,23 @@ description: Guide standalone chore planning with automatic type classification
 
 Guides Claude through standalone chore planning including automatic type classification, loading type-specific guidance from the taxonomy, building enriched context, and routing to chore-mode for execution. For chores under **technical epics**, detects this ancestry and passes context to skip mode progression.
 
+## ⚠️ READ-ONLY PHASE
+
+**This skill is a planning/investigation phase. No worktree exists yet.**
+
+🚫 **FORBIDDEN during this skill:**
+- Writing or editing any code files
+- Creating new files
+- Making implementation changes
+
+✅ **ALLOWED during this skill:**
+- Reading files to understand the codebase
+- Running `jettypod` commands to create work items
+- Asking the user questions
+- Analyzing and planning
+
+**The worktree is created later** when chore-mode runs `jettypod work start`.
+
 ## Instructions
 
 When this skill is activated, you are helping plan a standalone chore (one without a parent feature). Follow this structured approach:

package/skills-templates/epic-planning/SKILL.md

@@ -7,6 +7,23 @@ description: Guide epic planning with feature brainstorming and optional archite
 
 Guides Claude through comprehensive epic planning including feature identification and architectural decisions. For **technical epics**, skips feature brainstorming and creates chores directly.
 
+## ⚠️ READ-ONLY PHASE
+
+**This skill is a planning/investigation phase. No worktree exists yet.**
+
+🚫 **FORBIDDEN during this skill:**
+- Writing or editing any code files
+- Creating new files
+- Making implementation changes
+
+✅ **ALLOWED during this skill:**
+- Reading files to understand the codebase
+- Running `jettypod` commands to create work items
+- Asking the user questions
+- Analyzing and planning
+
+**The worktree is created later** when a mode skill (speed-mode, chore-mode, etc.) runs `jettypod work start`.
+
 ## Instructions
 
 When this skill is activated, you are helping plan an epic. Follow this structured approach:

package/skills-templates/feature-planning/SKILL.md

@@ -7,6 +7,27 @@ description: Guide feature planning with UX approach exploration and BDD scenari
 
 Guides Claude through feature planning including UX approach exploration, optional prototyping, and BDD scenario generation.
 
+## ⚠️ READ-ONLY UNTIL WORKTREE EXISTS
+
+**This skill starts as a planning/investigation phase. No worktree exists initially.**
+
+🚫 **FORBIDDEN until a worktree is created:**
+- Writing or editing any code files
+- Creating new files in the main repository
+
+✅ **ALLOWED during planning phases:**
+- Reading files to understand the codebase
+- Running `jettypod` commands
+- Asking the user questions
+- Analyzing and planning
+
+**Worktrees are created at specific steps:**
+- `work prototype start` (Step 4) - for UX prototyping
+- `work tests start` (Step 7) - for BDD test authoring
+- `work start` (Step 13) - for chore implementation (invokes speed-mode)
+
+**Only write files AFTER the relevant worktree command succeeds.**
+
 ## Instructions
 
 When this skill is activated, you are helping discover the best approach for a feature. Follow this structured approach: