buddy-workbench 0.1.7 → 0.1.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "buddy-workbench",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "private": false,
   "type": "module",
   "bin": {

package/server/routes/launchers.js

@@ -1,6 +1,6 @@
 import { Router } from 'express';
 import { listLaunchers, saveLaunchers } from '../repositories/launchers.js';
-import { runScript, runningScripts, scriptErrorLogs, scriptLogs, stopScript } from '../services/process-manager.js';
+import { runScript, runningErrorCounts, runningScripts, scriptErrorLogs, scriptLogs, stopScript } from '../services/process-manager.js';
 import { readPackageScripts } from '../services/package-scripts.js';
 import { currentGitBranch } from '../services/git.js';
 
@@ -23,7 +23,7 @@ router.get('/', async (_req, res) => {
 router.post('/', (req, res) => { const config = validLauncher(req.body); if (!config) return invalid(res); const launcher = { id: crypto.randomUUID(), ...config }; const launchers = listLaunchers(); saveLaunchers([...launchers, launcher]); res.status(201).json(launcher); });
 router.put('/:id', (req, res) => { const config = validLauncher(req.body); if (!config) return invalid(res); const launchers = listLaunchers(); const index = launchers.findIndex((item) => item.id === req.params.id); if (index === -1) return res.status(404).json({ error: 'Configuration not found.' }); const launcher = { id: req.params.id, ...config }; launchers[index] = launcher; saveLaunchers(launchers); res.json(launcher); });
 router.delete('/:id', (req, res) => { const launchers = listLaunchers(); const next = launchers.filter((item) => item.id !== req.params.id); if (next.length === launchers.length) return res.status(404).json({ error: 'Configuration not found.' }); saveLaunchers(next); res.status(204).end(); });
-router.get('/running', (_req, res) => res.json(runningScripts()));
+router.get('/running', (_req, res) => res.json({ running: runningScripts(), errorCounts: runningErrorCounts() }));
 router.post('/:id/stop', (req, res) => { try { stopScript(req.params.id, req.body?.scriptId); res.json({ ok: true }); } catch (error) { res.status(404).json({ error: error.message }); } });
 router.post('/:id/start', (req, res) => {
   const launcher = listLaunchers().find((item) => item.id === req.params.id);

package/server/routes/pr-review.js

@@ -375,4 +375,59 @@ router.get('/review-prs', async (req, res) => {
   }
 });
 
+router.post('/comment', async (req, res) => {
+  const { projectKey, repositorySlug, pullRequestId, filePath, line, commentText } = req.body || {};
+  if (!projectKey || !repositorySlug || !pullRequestId || !commentText) {
+    return res.status(400).json({ error: 'Missing required parameters.' });
+  }
+
+  const host = getBitbucketHost();
+  const settings = readSettings();
+  const token = settings.bitbucketAccessToken;
+
+  if (!host || !token) {
+    return res.status(400).json({ error: 'Bitbucket Access Token or Host not configured.' });
+  }
+
+  const url = `https://${host}/rest/api/1.0/projects/${projectKey}/repos/${repositorySlug}/pull-requests/${pullRequestId}/comments`;
+  const headers = {
+    'Content-Type': 'application/json',
+    'Accept': 'application/json',
+    'Authorization': `Bearer ${token}`
+  };
+
+  const payload = {
+    text: commentText
+  };
+
+  if (filePath && line) {
+    payload.anchor = {
+      line: parseInt(line, 10),
+      lineType: 'ADDED',
+      fileType: 'TO',
+      path: filePath
+    };
+  }
+
+  try {
+    const response = await fetch(url, {
+      method: 'POST',
+      headers,
+      body: JSON.stringify(payload)
+    });
+
+    if (!response.ok) {
+      const errJson = await response.json().catch(() => ({}));
+      return res.status(response.status).json({
+        error: errJson.errors?.[0]?.message || errJson.message || `Bitbucket API error: ${response.statusText}`
+      });
+    }
+
+    const data = await response.json();
+    res.json({ success: true, data });
+  } catch (error) {
+    res.status(500).json({ error: error.message });
+  }
+});
+
 export default router;

package/server/services/process-manager.js

@@ -14,13 +14,69 @@ const savedPortHistory = new Map(listPortHistory().map((record) => [record.port,
 const supervisorPath = fileURLToPath(new URL('./script-supervisor.js', import.meta.url));
 const execFileAsync = promisify(execFile);
 const keyFor = (launcherId, scriptId) => `${launcherId}:${scriptId}`;
+function countErrorBlocks(text, isStderr = false) {
+  if (!text) return 0;
+  const lines = text.split(/\r?\n/);
+  let count = 0;
+  let inErrorBlock = false;
+
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (!trimmed) {
+      inErrorBlock = false;
+      continue;
+    }
+
+    const isNewHeader =
+      /^(?:\[?(?:error|fatal|fail|failed)\]?|error:|uncaught|fatal:)/i.test(trimmed) ||
+      /^(?:TypeError|ReferenceError|SyntaxError|RangeError|EvalError|URIError|Error|UnhandledPromiseRejectionWarning|UnhandledPromiseRejection):/i.test(trimmed) ||
+      /^npm ERR! (?:code|syscall|path|errno)/i.test(trimmed) ||
+      /^\[vite\] Internal server error/i.test(trimmed) ||
+      /\bTS\d{4,5}\b/i.test(trimmed) ||
+      /\berror TS\d+/i.test(trimmed) ||
+      /^(?:\[TypeScript\]|TS ERROR)/i.test(trimmed);
+
+    if (isNewHeader) {
+      inErrorBlock = false;
+    }
+
+    const isErrorLine =
+      isStderr ||
+      isNewHeader ||
+      /\b(error|failed|fatal|exception)\b/i.test(trimmed) ||
+      /^npm ERR!/i.test(trimmed) ||
+      /\bTS\d{4,5}\b/i.test(trimmed) ||
+      /^\s*at\s+[\w\d_$.<>]+\s+\(/i.test(line);
+
+    if (isErrorLine) {
+      if (!inErrorBlock) {
+        count++;
+        inErrorBlock = true;
+      }
+    } else {
+      inErrorBlock = false;
+    }
+  }
+
+  return count;
+}
+
 const appendLog = (key, type, chunk) => {
-  const current = logs.get(key) || { output: '', error: '' };
-  current[type] = `${current[type]}${chunk}`.split(/\r?\n/).slice(-1000).join('\n');
+  const current = logs.get(key) || { output: '', error: '', errorCount: 0 };
+  const str = String(chunk || '');
+  current[type] = `${current[type]}${str}`.split(/\r?\n/).slice(-1000).join('\n');
+  current.errorCount = countErrorBlocks(current.error, true) + countErrorBlocks(current.output, false);
   logs.set(key, current);
 };
 
 export function runningScripts() { return [...running.keys()]; }
+export function runningErrorCounts() {
+  const counts = {};
+  for (const [key, data] of logs.entries()) {
+    if (running.has(key) && data?.errorCount) counts[key] = data.errorCount;
+  }
+  return counts;
+}
 export function scriptLogs(launcherId, scriptId) { return logs.get(keyFor(launcherId, scriptId))?.output || ''; }
 export function scriptErrorLogs(launcherId, scriptId) { return logs.get(keyFor(launcherId, scriptId))?.error || ''; }
 
@@ -149,7 +205,7 @@ export function runScript(launcher, script) {
   const key = keyFor(launcher.id, script.id);
   if (running.has(key)) throw new Error('This script is already running.');
   if (!existsSync(launcher.folder)) throw new Error('The configured project folder does not exist.');
-  logs.set(key, { output: `$ ${script.command}\n`, error: '' });
+  logs.set(key, { output: `$ ${script.command}\n`, error: '', errorCount: 0 });
   const child = spawn(process.execPath, [supervisorPath], { cwd: launcher.folder, detached: process.platform !== 'win32', stdio: ['ignore', 'pipe', 'pipe'], env: { ...process.env, ...(launcher.executor === 'pnpm' || /^\s*pnpm\b/.test(script.command) ? { CI: 'true' } : {}), BUDDY_PARENT_PID: String(process.pid), BUDDY_SCRIPT_COMMAND: script.command } });
   child.stdout.on('data', (chunk) => appendLog(key, 'output', chunk.toString()));
   child.stderr.on('data', (chunk) => appendLog(key, 'error', chunk.toString()));
@@ -158,7 +214,13 @@ export function runScript(launcher, script) {
   launchHistory.push({ key, groupPid: child.pid, launcher: launcher.alias, script: script.name, startedAt: new Date().toISOString(), ports: new Map() });
   if (launchHistory.length > 200) launchHistory.splice(0, launchHistory.length - 200);
   setTimeout(() => { void observeLauncherPorts().catch(() => {}); }, 1000).unref();
-  child.on('exit', (code) => { appendLog(key, code === 0 ? 'output' : 'error', `\nProcess exited with code ${code}.\n`); running.delete(key); retireGroup(child.pid); });
+  child.on('exit', (code) => {
+    appendLog(key, code === 0 ? 'output' : 'error', `\nProcess exited with code ${code}.\n`);
+    const item = logs.get(key);
+    if (item) item.errorCount = 0;
+    running.delete(key);
+    retireGroup(child.pid);
+  });
 }
 
 export function stopScript(launcherId, scriptId) {
@@ -166,6 +228,8 @@ export function stopScript(launcherId, scriptId) {
   const child = running.get(key);
   if (!child) throw new Error('This script is not running.');
   appendLog(key, 'output', '\nStop requested.\n');
+  const item = logs.get(key);
+  if (item) item.errorCount = 0;
   stoppingGroups.add(child.pid);
   terminateGroup(child.pid, 'SIGTERM');
   setTimeout(() => { terminateGroup(child.pid, 'SIGKILL'); stoppingGroups.delete(child.pid); processGroups.delete(child.pid); }, 1500).unref();