@pixelbyte-software/pixcode 1.51.8 → 1.52.0

package/server/routes/mcp-utils.js

@@ -1,13 +1,13 @@
-/**
- * MCP UTILITIES API ROUTES
- * ========================
- * 
- * API endpoints for MCP server detection and configuration utilities.
- * These endpoints expose centralized MCP detection functionality.
- */
-
-import express from 'express';
-
-const router = express.Router();
-
-export default router;
+/**
+ * MCP UTILITIES API ROUTES
+ * ========================
+ * 
+ * API endpoints for MCP server detection and configuration utilities.
+ * These endpoints expose centralized MCP detection functionality.
+ */
+
+import express from 'express';
+
+const router = express.Router();
+
+export default router;

package/server/routes/messages.js

@@ -1,14 +1,14 @@
-/**
- * Unified messages endpoint.
- *
- * GET /api/sessions/:sessionId/messages?provider=claude&projectName=foo&limit=50&offset=0
- *
- * Replaces the four provider-specific session message endpoints with a single route
- * that delegates to the appropriate adapter via the provider registry.
- *
- * @module routes/messages
- */
-
+/**
+ * Unified messages endpoint.
+ *
+ * GET /api/sessions/:sessionId/messages?provider=claude&projectName=foo&limit=50&offset=0
+ *
+ * Replaces the four provider-specific session message endpoints with a single route
+ * that delegates to the appropriate adapter via the provider registry.
+ *
+ * @module routes/messages
+ */
+
 import path from 'node:path';
 
 import express from 'express';
@@ -16,7 +16,7 @@ import express from 'express';
 import { sessionsService } from '../modules/providers/services/sessions.service.js';
 import { userHasProjectAccess } from '../services/platformization.js';
 import { appConfigDb } from '../database/db.js';
-
+
 const router = express.Router();
 const SESSION_OWNERSHIP_KEY = 'session_ownership';
 
@@ -33,32 +33,32 @@ function canUserReadSession(user, provider, sessionId) {
   const owner = readSessionOwnership()[`${provider || 'claude'}:${sessionId}`];
   return Boolean(owner?.userId && Number(owner.userId) === Number(user?.id ?? user?.userId));
 }
-
-/**
- * GET /api/sessions/:sessionId/messages
- *
- * Auth: authenticateToken applied at mount level in index.js
- *
- * Query params:
- *   provider    - 'claude' | 'cursor' | 'codex' | 'gemini' (default: 'claude')
- *   projectName - required for claude provider
- *   projectPath - required for cursor provider (absolute path used for cwdId hash)
- *   limit       - page size (omit or null for all)
- *   offset      - pagination offset (default: 0)
- */
-router.get('/:sessionId/messages', async (req, res) => {
-  try {
-    const { sessionId } = req.params;
-    const provider = String(req.query.provider || 'claude').trim().toLowerCase();
+
+/**
+ * GET /api/sessions/:sessionId/messages
+ *
+ * Auth: authenticateToken applied at mount level in index.js
+ *
+ * Query params:
+ *   provider    - 'claude' | 'cursor' | 'codex' | 'gemini' (default: 'claude')
+ *   projectName - required for claude provider
+ *   projectPath - required for cursor provider (absolute path used for cwdId hash)
+ *   limit       - page size (omit or null for all)
+ *   offset      - pagination offset (default: 0)
+ */
+router.get('/:sessionId/messages', async (req, res) => {
+  try {
+    const { sessionId } = req.params;
+    const provider = String(req.query.provider || 'claude').trim().toLowerCase();
     const projectName = req.query.projectName || '';
     const projectPath = req.query.projectPath || '';
-    const limitParam = req.query.limit;
-    const limit = limitParam !== undefined && limitParam !== null && limitParam !== ''
-      ? parseInt(limitParam, 10)
-      : null;
-    const offset = parseInt(req.query.offset || '0', 10);
-
-    const availableProviders = sessionsService.listProviderIds();
+    const limitParam = req.query.limit;
+    const limit = limitParam !== undefined && limitParam !== null && limitParam !== ''
+      ? parseInt(limitParam, 10)
+      : null;
+    const offset = parseInt(req.query.offset || '0', 10);
+
+    const availableProviders = sessionsService.listProviderIds();
     if (!availableProviders.includes(provider)) {
       const available = availableProviders.join(', ');
       return res.status(400).json({ error: `Unknown provider: ${provider}. Available: ${available}` });
@@ -81,17 +81,17 @@ router.get('/:sessionId/messages', async (req, res) => {
     }
 
     const result = await sessionsService.fetchHistory(provider, sessionId, {
-      projectName,
-      projectPath,
-      limit,
-      offset,
-    });
-
-    return res.json(result);
-  } catch (error) {
-    console.error('Error fetching unified messages:', error);
-    return res.status(500).json({ error: 'Failed to fetch messages' });
-  }
-});
-
-export default router;
+      projectName,
+      projectPath,
+      limit,
+      offset,
+    });
+
+    return res.json(result);
+  } catch (error) {
+    console.error('Error fetching unified messages:', error);
+    return res.status(500).json({ error: 'Failed to fetch messages' });
+  }
+});
+
+export default router;

package/server/routes/network.js

@@ -1,7 +1,7 @@
-import os from 'os';
-
-import express from 'express';
-
+import os from 'os';
+
+import express from 'express';
+
 import {
   getTunnelState,
   getUpnpState,
@@ -9,118 +9,118 @@ import {
   stopTunnel,
 } from '../services/external-access.js';
 import { requireAdmin } from '../middleware/auth.js';
-
-const router = express.Router();
-
-// Accept any env key the rest of the server honors. Keep this aligned with server/cli.js.
-const resolveServerPort = () => {
-  const raw = process.env.SERVER_PORT || process.env.PORT || '3001';
-  const port = Number(raw);
-  return Number.isFinite(port) && port > 0 ? port : 3001;
-};
-
-// Filter to usable LAN addresses. We skip:
-//  - internal (loopback) addresses, since QR-ing "127.0.0.1" is useless for a phone
-//  - link-local IPv6, since pasting "fe80::…%iface" into a phone browser won't resolve
-//  - IPv6 addresses in general for the first pass (most users want the v4 QR)
-const listLanEndpoints = () => {
-  const interfaces = os.networkInterfaces();
-  const endpoints = [];
-
-  for (const [ifaceName, addrs] of Object.entries(interfaces)) {
-    if (!addrs) continue;
-    for (const addr of addrs) {
-      if (addr.internal) continue;
-      if (addr.family !== 'IPv4' && addr.family !== 4) continue;
-      endpoints.push({
-        host: addr.address,
-        label: ifaceName,
-        family: 'IPv4',
-      });
-    }
-  }
-
-  return endpoints;
-};
-
-router.get('/endpoints', (req, res) => {
-  try {
-    const port = resolveServerPort();
-    const endpoints = listLanEndpoints().map((entry) => ({
-      ...entry,
-      url: `http://${entry.host}:${port}`,
-    }));
-
-    res.json({
-      port,
-      hostname: os.hostname(),
-      endpoints,
-    });
-  } catch (error) {
-    console.error('Error listing network endpoints:', error);
-    res.status(500).json({ error: 'Failed to list network endpoints' });
-  }
-});
-
-// ============================================================================
-// External access (UPnP + tunnel)
-// ============================================================================
-
-router.get('/external', (req, res) => {
-  try {
-    res.json({
-      upnp: getUpnpState(),
-      tunnel: getTunnelState(),
-    });
-  } catch (error) {
-    console.error('Error reading external-access state:', error);
-    res.status(500).json({ error: 'Failed to read external access state' });
-  }
-});
-
-// UPnP endpoints removed in v1.32 (see external-access.js for rationale).
-// Clients hitting /upnp get a 410 so the UI can show a clear "moved to
-// tunnel" hint without mistaking the absence for a transient 404.
-router.post('/upnp', (_req, res) => {
-  res.status(410).json({
-    error: 'UPnP removed in v1.32 — use cloudflared or ngrok tunnels instead',
-    upnp: getUpnpState(),
-  });
-});
-router.delete('/upnp', (_req, res) => {
-  res.status(410).json({
-    error: 'UPnP removed in v1.32 — use cloudflared or ngrok tunnels instead',
-    upnp: getUpnpState(),
-  });
-});
-
+
+const router = express.Router();
+
+// Accept any env key the rest of the server honors. Keep this aligned with server/cli.js.
+const resolveServerPort = () => {
+  const raw = process.env.SERVER_PORT || process.env.PORT || '3001';
+  const port = Number(raw);
+  return Number.isFinite(port) && port > 0 ? port : 3001;
+};
+
+// Filter to usable LAN addresses. We skip:
+//  - internal (loopback) addresses, since QR-ing "127.0.0.1" is useless for a phone
+//  - link-local IPv6, since pasting "fe80::…%iface" into a phone browser won't resolve
+//  - IPv6 addresses in general for the first pass (most users want the v4 QR)
+const listLanEndpoints = () => {
+  const interfaces = os.networkInterfaces();
+  const endpoints = [];
+
+  for (const [ifaceName, addrs] of Object.entries(interfaces)) {
+    if (!addrs) continue;
+    for (const addr of addrs) {
+      if (addr.internal) continue;
+      if (addr.family !== 'IPv4' && addr.family !== 4) continue;
+      endpoints.push({
+        host: addr.address,
+        label: ifaceName,
+        family: 'IPv4',
+      });
+    }
+  }
+
+  return endpoints;
+};
+
+router.get('/endpoints', (req, res) => {
+  try {
+    const port = resolveServerPort();
+    const endpoints = listLanEndpoints().map((entry) => ({
+      ...entry,
+      url: `http://${entry.host}:${port}`,
+    }));
+
+    res.json({
+      port,
+      hostname: os.hostname(),
+      endpoints,
+    });
+  } catch (error) {
+    console.error('Error listing network endpoints:', error);
+    res.status(500).json({ error: 'Failed to list network endpoints' });
+  }
+});
+
+// ============================================================================
+// External access (UPnP + tunnel)
+// ============================================================================
+
+router.get('/external', (req, res) => {
+  try {
+    res.json({
+      upnp: getUpnpState(),
+      tunnel: getTunnelState(),
+    });
+  } catch (error) {
+    console.error('Error reading external-access state:', error);
+    res.status(500).json({ error: 'Failed to read external access state' });
+  }
+});
+
+// UPnP endpoints removed in v1.32 (see external-access.js for rationale).
+// Clients hitting /upnp get a 410 so the UI can show a clear "moved to
+// tunnel" hint without mistaking the absence for a transient 404.
+router.post('/upnp', (_req, res) => {
+  res.status(410).json({
+    error: 'UPnP removed in v1.32 — use cloudflared or ngrok tunnels instead',
+    upnp: getUpnpState(),
+  });
+});
+router.delete('/upnp', (_req, res) => {
+  res.status(410).json({
+    error: 'UPnP removed in v1.32 — use cloudflared or ngrok tunnels instead',
+    upnp: getUpnpState(),
+  });
+});
+
 router.post('/tunnel', requireAdmin, async (req, res) => {
-  const port = resolveServerPort();
-  try {
-    const state = await startTunnel({ port, persistPreference: true });
-    res.json({ success: true, tunnel: state });
-  } catch (error) {
-    console.error('Tunnel start failed:', error);
-    // 424 Failed Dependency is the best match for "we can't start because a
-    // required external binary is missing" — it tells the UI to show the
-    // "install cloudflared/ngrok" hint rather than a generic server error.
-    const status = error?.code === 'ENOENT_TUNNEL' ? 424 : 502;
-    res.status(status).json({
-      error: error?.message || 'Tunnel start failed',
-      installHint: error?.installHint,
-      tunnel: getTunnelState(),
-    });
-  }
-});
-
+  const port = resolveServerPort();
+  try {
+    const state = await startTunnel({ port, persistPreference: true });
+    res.json({ success: true, tunnel: state });
+  } catch (error) {
+    console.error('Tunnel start failed:', error);
+    // 424 Failed Dependency is the best match for "we can't start because a
+    // required external binary is missing" — it tells the UI to show the
+    // "install cloudflared/ngrok" hint rather than a generic server error.
+    const status = error?.code === 'ENOENT_TUNNEL' ? 424 : 502;
+    res.status(status).json({
+      error: error?.message || 'Tunnel start failed',
+      installHint: error?.installHint,
+      tunnel: getTunnelState(),
+    });
+  }
+});
+
 router.delete('/tunnel', requireAdmin, async (req, res) => {
-  try {
-    const state = await stopTunnel({ persistPreference: true });
-    res.json({ success: true, tunnel: state });
-  } catch (error) {
-    console.error('Tunnel stop failed:', error);
-    res.status(502).json({ error: error?.message || 'Tunnel stop failed', tunnel: getTunnelState() });
-  }
-});
-
-export default router;
+  try {
+    const state = await stopTunnel({ persistPreference: true });
+    res.json({ success: true, tunnel: state });
+  } catch (error) {
+    console.error('Tunnel stop failed:', error);
+    res.status(502).json({ error: error?.message || 'Tunnel stop failed', tunnel: getTunnelState() });
+  }
+});
+
+export default router;