@aion0/forge 0.4.9 → 0.4.11

package/components/WebTerminal.tsx

@@ -15,8 +15,6 @@ export interface WebTerminalHandle {
 export interface WebTerminalProps {
   onActiveSession?: (sessionName: string | null) => void;
   onCodeOpenChange?: (open: boolean) => void;
-  browserOpen?: boolean;
-  onBrowserToggle?: () => void;
 }
 
 // ─── Types ───────────────────────────────────────────────────
@@ -166,7 +164,7 @@ let globalDragging = false;
 
 // ─── Main component ─────────────────────────────────────────
 
-const WebTerminal = forwardRef<WebTerminalHandle, WebTerminalProps>(function WebTerminal({ onActiveSession, onCodeOpenChange, browserOpen, onBrowserToggle }, ref) {
+const WebTerminal = forwardRef<WebTerminalHandle, WebTerminalProps>(function WebTerminal({ onActiveSession, onCodeOpenChange }, ref) {
   const [tabs, setTabs] = useState<TabState[]>(() => {
     const tree = makeTerminal();
     return [{ id: nextId++, label: 'Terminal 1', tree, ratios: {}, activeId: firstTerminalId(tree) }];
@@ -478,6 +476,15 @@ const WebTerminal = forwardRef<WebTerminalHandle, WebTerminalProps>(function Web
     });
   }, [activeTab, updateActiveTab]);
 
+  const closePaneById = useCallback((id: number) => {
+    updateActiveTab(t => {
+      if (countTerminals(t.tree) <= 1) return t;
+      const newTree = removeNodeById(t.tree, id) || t.tree;
+      const newActiveId = t.activeId === id ? firstTerminalId(newTree) : t.activeId;
+      return { ...t, tree: newTree, activeId: newActiveId };
+    });
+  }, [updateActiveTab]);
+
   const setActiveId = useCallback((id: number) => {
     updateActiveTab(t => ({ ...t, activeId: id }));
   }, [updateActiveTab]);
@@ -636,20 +643,6 @@ const WebTerminal = forwardRef<WebTerminalHandle, WebTerminalProps>(function Web
               Code
             </button>
           )}
-          {onBrowserToggle && (
-            <button
-              onClick={onBrowserToggle}
-              className={`text-[11px] px-3 py-1 rounded font-bold ${browserOpen ? 'text-white bg-blue-500 hover:bg-blue-400' : 'text-blue-400 border border-blue-500 hover:bg-blue-500 hover:text-white'}`}
-              title={browserOpen ? 'Close browser' : 'Open browser'}
-            >
-              Browser
-            </button>
-          )}
-          {activeTab && countTerminals(activeTab.tree) > 1 && (
-            <button onClick={onClosePane} className="text-[10px] px-2 py-0.5 text-[var(--accent)] hover:text-red-400 hover:bg-[var(--term-border)] rounded font-medium">
-              Close Pane
-            </button>
-          )}
         </div>
       </div>
 
@@ -876,6 +869,8 @@ const WebTerminal = forwardRef<WebTerminalHandle, WebTerminalProps>(function Web
             onSessionConnected={onSessionConnected}
             refreshKeys={refreshKeys}
             skipPermissions={skipPermissions}
+            canClose={countTerminals(tab.tree) > 1}
+            onClosePane={tab.id === activeTabId ? closePaneById : undefined}
           />
         </div>
       ))}
@@ -888,7 +883,7 @@ export default WebTerminal;
 // ─── Pane renderer ───────────────────────────────────────────
 
 function PaneRenderer({
-  node, activeId, onFocus, ratios, setRatios, onSessionConnected, refreshKeys, skipPermissions,
+  node, activeId, onFocus, ratios, setRatios, onSessionConnected, refreshKeys, skipPermissions, canClose, onClosePane,
 }: {
   node: SplitNode;
   activeId: number;
@@ -898,11 +893,20 @@ function PaneRenderer({
   onSessionConnected: (paneId: number, sessionName: string) => void;
   refreshKeys: Record<number, number>;
   skipPermissions?: boolean;
+  canClose?: boolean;
+  onClosePane?: (id: number) => void;
 }) {
   if (node.type === 'terminal') {
     return (
-      <div className={`h-full w-full ${activeId === node.id ? 'ring-1 ring-[#7c5bf0]/50 ring-inset' : ''}`} onMouseDown={() => onFocus(node.id)}>
+      <div className={`h-full w-full relative group/pane ${activeId === node.id ? 'ring-1 ring-[#7c5bf0]/50 ring-inset' : ''}`} onMouseDown={() => onFocus(node.id)}>
         <MemoTerminalPane key={`${node.id}-${refreshKeys[node.id] || 0}`} id={node.id} sessionName={node.sessionName} projectPath={node.projectPath} skipPermissions={skipPermissions} onSessionConnected={onSessionConnected} />
+        {canClose && onClosePane && (
+          <button
+            onClick={(e) => { e.stopPropagation(); if (confirm('Close this pane?')) onClosePane(node.id); }}
+            className="absolute top-1.5 right-1.5 z-10 w-6 h-6 flex items-center justify-center rounded bg-red-500/80 text-white hover:bg-red-500 opacity-0 group-hover/pane:opacity-100 transition-opacity text-xs font-bold shadow"
+            title="Close this pane"
+          >✕</button>
+        )}
       </div>
     );
   }
@@ -911,8 +915,8 @@ function PaneRenderer({
 
   return (
     <DraggableSplit splitId={node.id} direction={node.direction} ratio={ratio} setRatios={setRatios}>
-      <PaneRenderer node={node.first} activeId={activeId} onFocus={onFocus} ratios={ratios} setRatios={setRatios} onSessionConnected={onSessionConnected} refreshKeys={refreshKeys} skipPermissions={skipPermissions} />
-      <PaneRenderer node={node.second} activeId={activeId} onFocus={onFocus} ratios={ratios} setRatios={setRatios} onSessionConnected={onSessionConnected} refreshKeys={refreshKeys} skipPermissions={skipPermissions} />
+      <PaneRenderer node={node.first} activeId={activeId} onFocus={onFocus} ratios={ratios} setRatios={setRatios} onSessionConnected={onSessionConnected} refreshKeys={refreshKeys} skipPermissions={skipPermissions} canClose={canClose} onClosePane={onClosePane} />
+      <PaneRenderer node={node.second} activeId={activeId} onFocus={onFocus} ratios={ratios} setRatios={setRatios} onSessionConnected={onSessionConnected} refreshKeys={refreshKeys} skipPermissions={skipPermissions} canClose={canClose} onClosePane={onClosePane} />
     </DraggableSplit>
   );
 }
@@ -1198,12 +1202,29 @@ const MemoTerminalPane = memo(function TerminalPane({
             // Auto-run claude for project tabs (only if no pendingCommand already set)
             if (isNewlyCreated && projectPathRef.current && !pendingCommands.has(id)) {
               isNewlyCreated = false;
-              setTimeout(() => {
-                if (!disposed && ws?.readyState === WebSocket.OPEN) {
+              // Check if project has existing claude sessions to decide -c flag
+              const pp = projectPathRef.current;
+              const pName = pp.replace(/\/+$/, '').split('/').pop() || '';
+              fetch(`/api/claude-sessions/${encodeURIComponent(pName)}`)
+                .then(r => r.json())
+                .then(sData => {
+                  const hasSession = Array.isArray(sData) ? sData.length > 0 : false;
+                  const resumeFlag = hasSession ? ' -c' : '';
                   const skipFlag = skipPermRef.current ? ' --dangerously-skip-permissions' : '';
-                  ws.send(JSON.stringify({ type: 'input', data: `cd "${projectPathRef.current}" && claude${skipFlag}\n` }));
-                }
-              }, 300);
+                  setTimeout(() => {
+                    if (!disposed && ws?.readyState === WebSocket.OPEN) {
+                      ws.send(JSON.stringify({ type: 'input', data: `cd "${pp}" && claude${resumeFlag}${skipFlag}\n` }));
+                    }
+                  }, 300);
+                })
+                .catch(() => {
+                  const skipFlag = skipPermRef.current ? ' --dangerously-skip-permissions' : '';
+                  setTimeout(() => {
+                    if (!disposed && ws?.readyState === WebSocket.OPEN) {
+                      ws.send(JSON.stringify({ type: 'input', data: `cd "${pp}" && claude${skipFlag}\n` }));
+                    }
+                  }, 300);
+                });
             }
             isNewlyCreated = false;
             // Force tmux to redraw by toggling size, then send reset

package/lib/claude-sessions.ts

@@ -257,8 +257,8 @@ export function tailSessionFile(
 
   watcher.on('error', (err) => onError?.(err));
 
-  // Poll every 5 seconds as fallback
-  const pollTimer = setInterval(readNewBytes, 5000);
+  // Poll every 1 second as fallback (fs.watch is unreliable on macOS)
+  const pollTimer = setInterval(readNewBytes, 1000);
 
   return () => {
     watcher.close();

package/lib/init.ts

@@ -82,11 +82,28 @@ export function ensureInitialized() {
   // Auto-detect claude path if not configured
   autoDetectClaude();
 
+  // Sync help docs + CLAUDE.md to data dir on startup
+  try {
+    const { existsSync, readdirSync, readFileSync, writeFileSync, mkdirSync } = require('node:fs');
+    const { join: joinPath } = require('node:path');
+    const { getConfigDir, getDataDir } = require('./dirs');
+    const helpDir = joinPath(getConfigDir(), 'help');
+    const sourceDir = joinPath(process.cwd(), 'lib', 'help-docs');
+    if (existsSync(sourceDir)) {
+      if (!existsSync(helpDir)) mkdirSync(helpDir, { recursive: true });
+      for (const f of readdirSync(sourceDir)) {
+        if (f.endsWith('.md')) writeFileSync(joinPath(helpDir, f), readFileSync(joinPath(sourceDir, f)));
+      }
+      const claudeMd = joinPath(helpDir, 'CLAUDE.md');
+      if (existsSync(claudeMd)) writeFileSync(joinPath(getDataDir(), 'CLAUDE.md'), readFileSync(claudeMd));
+    }
+  } catch {}
+
   // Sync skills registry (async, non-blocking) — on startup + every 30 min
   try {
     const { syncSkills } = require('./skills');
     syncSkills().catch(() => {});
-    setInterval(() => { syncSkills().catch(() => {}); }, 30 * 60 * 1000);
+    setInterval(() => { syncSkills().catch(() => {}); }, 60 * 60 * 1000);
   } catch {}
 
   // Task runner is safe in every worker (DB-level coordination)

package/lib/pipeline-scheduler.ts

@@ -150,9 +150,11 @@ export function deleteRun(id: string): void {
 
 function isDuplicate(projectPath: string, workflowName: string, dedupKey: string): boolean {
   const row = db().prepare(
-    'SELECT 1 FROM pipeline_runs WHERE project_path = ? AND workflow_name = ? AND dedup_key = ?'
-  ).get(projectPath, workflowName, dedupKey);
-  return !!row;
+    'SELECT status FROM pipeline_runs WHERE project_path = ? AND workflow_name = ? AND dedup_key = ? ORDER BY created_at DESC LIMIT 1'
+  ).get(projectPath, workflowName, dedupKey) as { status: string } | undefined;
+  if (!row) return false;
+  // Failed runs are not duplicates — allow retry
+  return row.status !== 'failed';
 }
 
 export function resetDedup(projectPath: string, workflowName: string, dedupKey: string): void {
@@ -222,7 +224,7 @@ function fetchOpenIssues(projectPath: string, labels: string[]): { number: numbe
   if (!repo) return [{ number: -1, title: '', error: 'Could not detect GitHub repo. Run: gh auth login' }];
   try {
     const labelFilter = labels.length > 0 ? ` --label "${labels.join(',')}"` : '';
-    const out = execSync(`gh issue list --state open --json number,title${labelFilter} -R ${repo}`, {
+    const out = execSync(`gh issue list --state open --limit 30 --json number,title${labelFilter} -R ${repo}`, {
       cwd: projectPath, encoding: 'utf-8', timeout: 15000, stdio: ['pipe', 'pipe', 'pipe'],
     });
     return JSON.parse(out) || [];
@@ -246,11 +248,18 @@ export function scanAndTriggerIssues(binding: ProjectPipelineBinding): { trigger
   const recentRuns = getRuns(binding.projectPath, binding.workflowName, 5);
   const hasRunning = recentRuns.some(r => r.status === 'running');
 
+  // Batch dedup check — one query instead of N
+  const processedKeys = new Set(
+    (db().prepare(
+      'SELECT dedup_key FROM pipeline_runs WHERE project_path = ? AND workflow_name = ? AND dedup_key IS NOT NULL AND status != ?'
+    ).all(binding.projectPath, binding.workflowName, 'failed') as { dedup_key: string }[])
+    .map(r => r.dedup_key)
+  );
+
   const newIssues: { number: number; title: string }[] = [];
   for (const issue of issues) {
     if (issue.number < 0) continue;
-    const dedupKey = `issue:${issue.number}`;
-    if (!isDuplicate(binding.projectPath, binding.workflowName, dedupKey)) {
+    if (!processedKeys.has(`issue:${issue.number}`)) {
       newIssues.push(issue);
     }
   }
@@ -269,6 +278,9 @@ export function scanAndTriggerIssues(binding: ProjectPipelineBinding): { trigger
 
   const issue = newIssues[0];
   const dedupKey = `issue:${issue.number}`;
+  // Remove old failed run so new dedup_key insert won't conflict
+  db().prepare('DELETE FROM pipeline_runs WHERE project_path = ? AND workflow_name = ? AND dedup_key = ? AND status = ?')
+    .run(binding.projectPath, binding.workflowName, dedupKey, 'failed');
   try {
     triggerPipeline(
       binding.projectPath, binding.projectName, binding.workflowName,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aion0/forge",
-  "version": "0.4.9",
+  "version": "0.4.11",
   "description": "Unified AI workflow platform — multi-model task orchestration, persistent sessions, web terminal, remote access",
   "type": "module",
   "scripts": {

package/components/PreviewPanel.tsx

@@ -1,167 +0,0 @@
-'use client';
-
-import { useState, useEffect, useCallback } from 'react';
-
-interface PreviewEntry {
-  port: number;
-  url: string | null;
-  status: string;
-  label?: string;
-}
-
-export default function PreviewPanel() {
-  const [previews, setPreviews] = useState<PreviewEntry[]>([]);
-  const [inputPort, setInputPort] = useState('');
-  const [inputLabel, setInputLabel] = useState('');
-  const [starting, setStarting] = useState(false);
-  const [error, setError] = useState('');
-  const [activePreview, setActivePreview] = useState<number | null>(null);
-  const [isRemote, setIsRemote] = useState(false);
-
-  useEffect(() => {
-    setIsRemote(!['localhost', '127.0.0.1'].includes(window.location.hostname));
-  }, []);
-
-  const fetchPreviews = useCallback(async () => {
-    try {
-      const res = await fetch('/api/preview');
-      const data = await res.json();
-      if (Array.isArray(data)) setPreviews(data);
-    } catch {}
-  }, []);
-
-  useEffect(() => {
-    fetchPreviews();
-    const timer = setInterval(fetchPreviews, 5000);
-    return () => clearInterval(timer);
-  }, [fetchPreviews]);
-
-  const handleStart = async () => {
-    const p = parseInt(inputPort);
-    if (!p || p < 1 || p > 65535) { setError('Invalid port'); return; }
-    setError('');
-    setStarting(true);
-    try {
-      const res = await fetch('/api/preview', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ action: 'start', port: p, label: inputLabel || undefined }),
-      });
-      const data = await res.json();
-      if (data.error) setError(data.error);
-      else {
-        setInputPort('');
-        setInputLabel('');
-        setActivePreview(p);
-      }
-      fetchPreviews();
-    } catch { setError('Failed'); }
-    setStarting(false);
-  };
-
-  const handleStop = async (port: number) => {
-    await fetch('/api/preview', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ action: 'stop', port }),
-    });
-    if (activePreview === port) setActivePreview(null);
-    fetchPreviews();
-  };
-
-  const active = previews.find(p => p.port === activePreview);
-  const previewSrc = active
-    ? (isRemote ? active.url : `http://localhost:${active.port}`)
-    : null;
-
-  return (
-    <div className="flex-1 flex flex-col min-h-0">
-      {/* Top bar */}
-      <div className="px-4 py-2 border-b border-[var(--border)] shrink-0 space-y-2">
-        {/* Preview list */}
-        <div className="flex items-center gap-2 flex-wrap">
-          <span className="text-[11px] font-semibold text-[var(--text-primary)]">Demo Preview</span>
-          {previews.map(p => (
-            <div key={p.port} className="flex items-center gap-1">
-              <button
-                onClick={() => setActivePreview(p.port)}
-                className={`text-[10px] px-2 py-0.5 rounded ${activePreview === p.port ? 'bg-[var(--accent)] text-white' : 'bg-[var(--bg-tertiary)] text-[var(--text-secondary)] hover:text-[var(--text-primary)]'}`}
-              >
-                <span className={`mr-1 ${p.status === 'running' ? 'text-green-400' : 'text-gray-500'}`}>●</span>
-                {p.label || `:${p.port}`}
-              </button>
-              {p.url && (
-                <button
-                  onClick={() => navigator.clipboard.writeText(p.url!)}
-                  className="text-[8px] text-green-400 hover:text-green-300 truncate max-w-[150px]"
-                  title={`Copy: ${p.url}`}
-                >
-                  {p.url.replace('https://', '').slice(0, 20)}...
-                </button>
-              )}
-              <button
-                onClick={() => handleStop(p.port)}
-                className="text-[9px] text-red-400 hover:text-red-300"
-              >
-                x
-              </button>
-            </div>
-          ))}
-        </div>
-
-        {/* Add new */}
-        <div className="flex items-center gap-2">
-          <input
-            type="number"
-            value={inputPort}
-            onChange={e => setInputPort(e.target.value)}
-            onKeyDown={e => e.key === 'Enter' && handleStart()}
-            placeholder="Port"
-            className="w-20 text-xs bg-[var(--bg-tertiary)] border border-[var(--border)] rounded px-2 py-1 text-[var(--text-primary)] focus:outline-none focus:border-[var(--accent)] font-mono"
-          />
-          <input
-            value={inputLabel}
-            onChange={e => setInputLabel(e.target.value)}
-            onKeyDown={e => e.key === 'Enter' && handleStart()}
-            placeholder="Label (optional)"
-            className="w-32 text-xs bg-[var(--bg-tertiary)] border border-[var(--border)] rounded px-2 py-1 text-[var(--text-primary)] focus:outline-none focus:border-[var(--accent)]"
-          />
-          <button
-            onClick={handleStart}
-            disabled={!inputPort || starting}
-            className="text-[10px] px-3 py-1 bg-[var(--accent)] text-white rounded hover:opacity-90 disabled:opacity-50"
-          >
-            {starting ? 'Starting...' : '+ Add'}
-          </button>
-          {active && (
-            <a
-              href={previewSrc || '#'}
-              target="_blank"
-              rel="noopener"
-              className="text-[10px] text-[var(--accent)] hover:underline ml-auto"
-            >
-              Open ↗
-            </a>
-          )}
-          {error && <span className="text-[10px] text-red-400">{error}</span>}
-        </div>
-      </div>
-
-      {/* Preview iframe */}
-      {previewSrc && active?.status === 'running' ? (
-        <iframe
-          src={previewSrc}
-          className="flex-1 w-full border-0 bg-white"
-          title="Preview"
-        />
-      ) : (
-        <div className="flex-1 flex items-center justify-center text-[var(--text-secondary)]">
-          <div className="text-center space-y-3 max-w-md">
-            <p className="text-sm">{previews.length > 0 ? 'Select a preview to display' : 'Preview local dev servers'}</p>
-            <p className="text-xs">Enter a port, add a label, and click Add. Each preview gets its own Cloudflare Tunnel URL.</p>
-          </div>
-        </div>
-      )}
-    </div>
-  );
-}