@aion0/forge 0.6.3 → 0.8.1

package/lib/help-docs/20-mantis-bug-fix.md

@@ -0,0 +1,115 @@
+# Mantis → Bug Fix → MR pipeline
+
+End-to-end: a Mantis bug surfaces (via a Forge **Job** polling mantis.get_bug
+/ mantis.search_bugs) → triggers a Pipeline → Pipeline checks out the right
+base branch in a worktree → headless Claude implements the fix → pipeline
+opens a GitLab MR via `glab` → pipeline pings the assignee + reporter on
+Teams with the MR URL.
+
+Mirrors the `gitlab-issue-fix-and-review` builtin but driven by Mantis
+content (description / priority / category / assignee / reporter), MR
+opens against an explicit `base_branch` you pass in from the Job (because
+Mantis doesn't carry milestones the way GitLab issues do).
+
+## Builtin name
+
+`mantis-bug-fix-and-mr` — registered in Forge's built-in workflow set,
+visible in the Pipelines view's workflow dropdown and pickable from the
+extension Jobs tab when you wire a Pipeline-dispatch Job.
+
+## Inputs (set by the Job's `input_template`)
+
+| Key | Required | Source |
+|---|---|---|
+| `bug_id` | yes | `{{item.id}}` |
+| `project` | yes | injected by `triggerPipeline` from the Job's project setting |
+| `base_branch` | **yes** | `{{item.product_version}}` mapped to a branch, OR a literal like `"release/25.4"` you write into the template |
+| `summary` | yes | `{{item.summary}}` |
+| `description` | yes | `{{item.description}}` |
+| `priority` | opt | `{{item.priority}}` |
+| `category` | opt | `{{item.category}}` |
+| `assignee` | opt | `{{item.assignee}}` — used as Teams chat name |
+| `reporter` | opt | `{{item.reporter}}` — used as Teams chat name |
+| `extra_context` | opt | literal hint text for Claude |
+| `mr_title_template` | opt | default `Fix Mantis #{bug_id}: {summary}` |
+| `mr_body_template` | opt | default closes-ref + Claude summary; vars `{bug_id} {summary} {description} {claude_summary}` |
+| `teams_message_template` | opt | default `🤖 Mantis #{bug_id} fixed — please review MR: {mr_url}\nBug: {summary}`; vars `{bug_id} {summary} {role} {mr_url}` |
+
+## Nodes
+
+```
+resolve              parse git remote → HOST + PROJECT_PATH; check glab; require base_branch
+worktree-setup       git worktree add -b fix/mantis-<id> .forge/worktrees/mantis-<id> origin/<base>
+fix-code             headless Claude — reads bug context, edits in worktree, commits
+push-and-mr          if any commits: push fix/mantis-<id>, glab mr create → MR_URL
+notify-teams         curl /api/connector-tool teams.send_message twice (assignee + reporter)
+```
+
+`fix-code` runs as a normal Claude task (not shell) so it can think, browse,
+edit, and commit. `notify-teams` short-circuits if `push-and-mr` couldn't
+create an MR (e.g. Claude made no changes).
+
+## Talking to connectors from a pipeline
+
+`notify-teams` calls `POST http://127.0.0.1:8403/api/connector-tool` —
+a loopback-only endpoint that wraps `lib/chat/tool-dispatcher`. Body:
+
+```json
+{ "plugin_id": "teams", "tool": "send_message",
+  "input": { "name": "Alice", "text": "MR: https://..." } }
+```
+
+Returns the standard `{ content, is_error }` tool-result shape. Works for
+`browser` / `http` / `shell` protocol connectors. Browser-protocol calls
+need the extension bridge connected at pipeline runtime (Chrome open +
+extension signed in).
+
+## Wiring it up from a Job
+
+1. Confirm `glab` is installed + authed for the target host:
+   `glab auth login --hostname <your-gitlab.example.com>`
+2. Make sure the project has a git remote (`git remote get-url origin`) the
+   `glab mr create` call can reach.
+3. Forge extension → Jobs → + New job. Pick:
+   - Connector / tool: `mantis` / `get_bug` (single bug) or `search_bugs` (a query)
+   - dispatch: Pipeline → workflow `mantis-bug-fix-and-mr` → your project
+   - `input_template` (auto-prefilled when you pick the workflow; map item
+     keys to inputs, hardcode `base_branch` if Mantis doesn't carry it):
+     ```json
+     {
+       "bug_id":      "{{item.id}}",
+       "summary":     "{{item.summary}}",
+       "description": "{{item.description}}",
+       "priority":    "{{item.priority}}",
+       "category":    "{{item.category}}",
+       "assignee":    "{{item.assignee}}",
+       "reporter":    "{{item.reporter}}",
+       "base_branch": "release/25.4"
+     }
+     ```
+4. Save → Run now (first tick is a backfill-no-op; click Reset dedup +
+   Run now to force one actual dispatch for testing).
+
+## Customising
+
+- Branch derivation rules (e.g. `target_version` → `release/<major.minor>`):
+  edit the `resolve` node in your local copy at
+  `~/.forge/data/flows/mantis-bug-fix-and-mr.yaml` (a copy will be created
+  when you click Edit in the Pipelines view; once a local file exists it
+  overrides the builtin).
+- MR title / body templates: set via the Job's `input_template` — no
+  pipeline edit needed.
+- Teams routing: today it uses the Mantis username verbatim as the Teams
+  chat name (substring match). For better mapping, post-process in
+  `notify-teams` to convert username → real name via a lookup table or
+  a second connector call.
+
+## Troubleshooting
+
+| Symptom | Cause + fix |
+|---|---|
+| `ERROR: base_branch is required` | The Job's `input_template` didn't supply it — Mantis bugs don't always carry one. Hardcode it in the template or map from `product_version`. |
+| `NO_CHANGES — Claude did not commit` | Bug description was too thin or Claude couldn't find the affected code. Add hints via `extra_context` or open the worktree manually and iterate. |
+| `glab mr create` returns nothing | Token expired / target branch protected / source branch already has an open MR. The pipeline falls back to `glab mr view` to surface the existing URL — check Pipelines log. |
+| Teams send returns `No extension connected to the bridge` | The pipeline ran when Chrome / extension weren't online. Notification fails but the MR still landed; re-fire the `notify-teams` node manually or message the people yourself. |
+| Mantis username doesn't match any Teams chat | The fuzzy substring match in `teams.send_message` falls back to whatever the LLM-less DOM script can match. Add a lookup step before `notify-teams` to translate names. |

package/lib/help-docs/CLAUDE.md

@@ -45,6 +45,11 @@ The token is valid for 24 hours. Store it in a variable and reuse for all API ca
 | `12-usage.md` | Token usage analytics — charts, heatmap, cost estimation, by model/project/source |
 | `13-ide-plugins.md` | VSCode extension + IntelliJ plugin — install, tabs, multi-connection, agent terminal launching |
 | `15-crafts.md` | Crafts — project-scoped mini-app tabs with SDK; AI-generated via "+ Craft" button |
+| `16-gitlab-autofix.md` | GitLab issue auto-fix — worktree + base-branch rule + Premium epic context + image attachments |
+| `17-connectors.md` | Browser extension connectors — plugin schema, HTTP API, settings sync, Mantis + GitLab built-ins |
+| `18-chrome-mcp.md` | Connect Forge Claude Code sessions to a real Chrome via chrome-devtools-mcp — dev-time browser access for connector authoring |
+| `19-jobs.md` | Jobs — scheduled connector polls that dedup and fan out to Pipeline / Chat |
+| `20-mantis-bug-fix.md` | Mantis → Bug Fix → MR builtin pipeline (mantis-bug-fix-and-mr) |
 
 ## Matching questions to docs
 
@@ -69,3 +74,8 @@ The token is valid for 24 hours. Store it in a variable and reuse for all API ca
 - VSCode/IntelliJ/IDE plugin/extension/marketplace → `13-ide-plugins.md`
 - vsce/vsix/JetBrains marketplace publish → `13-ide-plugins.md`
 - Craft/custom tab/mini-app/extend project/AI-generated tab/builder → `15-crafts.md`
+- GitLab/glab/MR/merge request/issue auto-fix/epic → `16-gitlab-autofix.md`
+- Connector/browser extension/plugin schema/Mantis/category=connector → `17-connectors.md`
+- Chrome MCP / chrome-devtools-mcp / dev-time browser / CDP / remote debugging → `18-chrome-mcp.md`
+- Job / scheduled job / connector poll / dedup / periodic fetch / Teams poll / Mantis bug poll → `19-jobs.md`
+- Mantis bug fix pipeline / mantis-bug-fix-and-mr / open MR for Mantis bug / notify Teams from pipeline / connector-tool endpoint → `20-mantis-bug-fix.md`

package/lib/init.ts

@@ -71,47 +71,60 @@ export function ensureInitialized() {
   if (gInit[initKey]) return;
   gInit[initKey] = true;
 
-  // Add timestamps to all console output
-  try { const { initLogger } = require('./logger'); initLogger(); } catch {}
-
-  // Migrate old data layout (~/.forge/* → ~/.forge/data/*) on first run
-  try { const { migrateDataDir } = require('./dirs'); migrateDataDir(); } catch {}
-
-  // Migrate plaintext secrets on startup
-  migrateSecrets();
-
-  // Cleanup old notifications
-  try {
-    const { cleanupNotifications } = require('./notifications');
-    cleanupNotifications();
-  } catch {}
-
-  // Auto-detect claude path if not configured
-  autoDetectAgents();
-
-  // Install/update forge skills to ~/.claude/skills/ on every startup
-  try {
-    const { installForgeSkills } = require('./workspace/skill-installer');
-    installForgeSkills('', '', '', Number(process.env.PORT) || 8403);
-    console.log('[init] Forge skills installed/updated');
-  } catch {}
+  // Per-step timing — earlier startup hangs were invisible because all
+  // these steps log heterogeneously (or not at all). \`time(label, fn)\`
+  // prints \`[init] <label> took 1234ms\` for anything taking >250ms so
+  // we can spot the slow one without instrumenting elsewhere.
+  const time = <T>(label: string, fn: () => T): T => {
+    const t = Date.now();
+    try { return fn(); }
+    finally {
+      const ms = Date.now() - t;
+      if (ms >= 250) console.log(`[init] ${label} took ${ms}ms`);
+    }
+  };
+
+  time('logger', () => { try { const { initLogger } = require('./logger'); initLogger(); } catch {} });
+  time('migrateDataDir', () => { try { const { migrateDataDir } = require('./dirs'); migrateDataDir(); } catch {} });
+  time('migrateSecrets', migrateSecrets);
+  time('migratePluginSecrets', () => {
+    try {
+      const { migratePluginSecrets } = require('./plugins/registry');
+      migratePluginSecrets();
+    } catch (e) { console.warn('[init] migratePluginSecrets failed:', (e as Error).message); }
+  });
+  time('cleanupNotifications', () => { try { const { cleanupNotifications } = require('./notifications'); cleanupNotifications(); } catch {} });
+  time('autoDetectAgents', autoDetectAgents);
+  time('logToolStatus', () => {
+    try { const { logToolStatus } = require('./health'); logToolStatus(); }
+    catch (e) { console.warn('[tools] health check failed:', (e as Error).message); }
+  });
+  time('installForgeSkills', () => {
+    try {
+      const { installForgeSkills } = require('./workspace/skill-installer');
+      installForgeSkills('', '', '', Number(process.env.PORT) || 8403);
+      console.log('[init] Forge skills installed/updated');
+    } catch {}
+  });
 
   // 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)));
+  time('syncHelpDocs', () => {
+    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));
       }
-      const claudeMd = joinPath(helpDir, 'CLAUDE.md');
-      if (existsSync(claudeMd)) writeFileSync(joinPath(getDataDir(), 'CLAUDE.md'), readFileSync(claudeMd));
-    }
-  } catch {}
+    } catch {}
+  });
 
   // Sync skills registry (async, non-blocking) — on startup + every 30 min
   try {
@@ -120,24 +133,44 @@ export function ensureInitialized() {
     setInterval(() => { syncSkills().catch(() => {}); }, 60 * 60 * 1000);
   } catch {}
 
-  // Usage scanner — scan JSONL files for token usage on startup + every hour
-  try {
-    const { scanUsage } = require('./usage-scanner');
-    scanUsage();
-    setInterval(() => { try { scanUsage(); } catch {} }, 60 * 60 * 1000);
-  } catch {}
+  // Usage scanner — defer to next tick so it doesn't block ensureInitialized().
+  // On a host with hundreds of project dirs in ~/.claude/projects/, the
+  // synchronous readdirSync + statSync loop can take 5-10s; running it on
+  // the critical path of the first API request made startup feel hung.
+  // setImmediate keeps it in the same process but yields the event loop first.
+  setImmediate(() => {
+    time('scanUsage (initial)', () => {
+      try { const { scanUsage } = require('./usage-scanner'); scanUsage(); } catch {}
+    });
+  });
+  time('require usage-scanner + hourly', () => {
+    try {
+      const { scanUsage } = require('./usage-scanner');
+      setInterval(() => { try { scanUsage(); } catch {} }, 60 * 60 * 1000);
+    } catch {}
+  });
 
   // Task runner is safe in every worker (DB-level coordination)
-  ensureRunnerStarted();
+  time('ensureRunnerStarted', ensureRunnerStarted);
 
   // Session watcher is safe (file-based, idempotent)
-  startWatcherLoop();
+  time('startWatcherLoop', startWatcherLoop);
 
   // Pipeline scheduler — periodic execution + issue scanning for project-bound workflows
-  try {
-    const { startScheduler } = require('./pipeline-scheduler');
-    startScheduler();
-  } catch {}
+  time('pipeline-scheduler', () => {
+    try {
+      const { startScheduler } = require('./pipeline-scheduler');
+      startScheduler();
+    } catch {}
+  });
+
+  // Jobs scheduler — periodic connector polls that fan out to pipelines / chat
+  time('jobs-scheduler', () => {
+    try {
+      const { startScheduler: startJobsScheduler } = require('./jobs/scheduler');
+      startJobsScheduler();
+    } catch (e) { console.error('[jobs-scheduler] start failed', e); }
+  });
 
   // If services are managed externally (forge-server), skip
   if (process.env.FORGE_EXTERNAL_SERVICES === '1') {
@@ -163,6 +196,8 @@ export function ensureInitialized() {
   startTerminalProcess();
   startTelegramProcess(); // spawns telegram-standalone
   startWorkspaceProcess(); // spawns workspace-standalone
+  startBrowserBridgeProcess(); // spawns browser-bridge-standalone
+  startChatProcess(); // spawns chat-standalone
 
   const settings = loadSettings();
   if (settings.tunnelAutoStart) {
@@ -264,3 +299,55 @@ function launchWorkspaceDaemon() {
   workspaceChild.on('exit', () => { workspaceChild = null; });
   console.log('[workspace] Started daemon (pid:', workspaceChild.pid, ')');
 }
+
+let chatChild: ReturnType<typeof spawn> | null = null;
+
+function startChatProcess() {
+  if (chatChild) return;
+
+  const chatPort = Number(process.env.CHAT_PORT) || 8408;
+
+  const net = require('node:net');
+  const tester = net.createServer();
+  tester.once('error', () => {
+    console.log(`[chat] Port ${chatPort} already in use, reusing existing`);
+  });
+  tester.once('listening', () => {
+    tester.close();
+    const script = join(process.cwd(), 'lib', 'chat-standalone.ts');
+    chatChild = spawn('npx', ['tsx', script], {
+      stdio: ['ignore', 'inherit', 'inherit'],
+      env: { ...process.env },
+      detached: false,
+    });
+    chatChild.on('exit', () => { chatChild = null; });
+    console.log('[chat] Started standalone (pid:', chatChild.pid, ')');
+  });
+  tester.listen(chatPort);
+}
+
+let bridgeChild: ReturnType<typeof spawn> | null = null;
+
+function startBrowserBridgeProcess() {
+  if (bridgeChild) return;
+
+  const bridgePort = Number(process.env.BRIDGE_PORT) || 8407;
+
+  const net = require('node:net');
+  const tester = net.createServer();
+  tester.once('error', () => {
+    console.log(`[bridge] Port ${bridgePort} already in use, reusing existing`);
+  });
+  tester.once('listening', () => {
+    tester.close();
+    const script = join(process.cwd(), 'lib', 'browser-bridge-standalone.ts');
+    bridgeChild = spawn('npx', ['tsx', script], {
+      stdio: ['ignore', 'inherit', 'inherit'],
+      env: { ...process.env },
+      detached: false,
+    });
+    bridgeChild.on('exit', () => { bridgeChild = null; });
+    console.log('[bridge] Started standalone (pid:', bridgeChild.pid, ')');
+  });
+  tester.listen(bridgePort);
+}

package/lib/iso-time.ts

@@ -0,0 +1,30 @@
+/**
+ * SQLite-friendly timestamp normalization.
+ *
+ * SQLite's `datetime('now')` returns "2026-05-19 18:30:00" — UTC, but no
+ * timezone marker. When that string lands in JavaScript:
+ *   new Date('2026-05-19 18:30:00')   // parsed as LOCAL time (wrong)
+ *
+ * Browsers shift the displayed value by the user's offset — a UTC 18:30
+ * looks like 18:30 in their local clock if treated as local, hours off
+ * from the real moment. Frontend Date.toLocaleString() then renders the
+ * wrong number.
+ *
+ * Append a 'Z' so JS treats it as UTC, and Date.toLocaleString() will
+ * automatically convert to the user's local timezone.
+ *
+ * Idempotent: if the string already has a 'Z' or a numeric offset, or
+ * is already in ISO 8601 (T separator), it's returned unchanged.
+ */
+export function toIsoUTC(s: string | null | undefined): string | null {
+  if (!s) return null;
+  // Already ISO with T separator → just ensure it has a TZ marker.
+  if (/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}/.test(s)) {
+    return /Z$|[+-]\d{2}:?\d{2}$/.test(s) ? s : s + 'Z';
+  }
+  // SQLite datetime('now') shape: "YYYY-MM-DD HH:MM:SS"
+  if (/^\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}(\.\d+)?$/.test(s)) {
+    return s.replace(' ', 'T') + 'Z';
+  }
+  return s;
+}