@lovelybunch/api 1.0.59 → 1.0.61

package/dist/lib/jobs/job-runner.js

@@ -18,7 +18,7 @@ function resolveAgent(model) {
         return 'codex';
     return 'claude';
 }
-function buildCommand(agent, instruction) {
+function buildCommand(agent, instruction, options) {
     const quotedInstruction = shellQuote(instruction);
     switch (agent) {
         case 'gemini': {
@@ -26,12 +26,17 @@ function buildCommand(agent, instruction) {
             return { command: 'gemini', shellCommand: cmd };
         }
         case 'codex': {
-            const cmd = `codex ${quotedInstruction} --dangerously-bypass-approvals-and-sandbox`;
-            return { command: 'codex', shellCommand: cmd };
+            const baseCmd = `codex ${quotedInstruction} --dangerously-bypass-approvals-and-sandbox`;
+            const needsPseudoTty = options.runningAsRoot && process.platform !== 'win32';
+            const wrappedCmd = needsPseudoTty
+                ? `script -q -e -c ${shellQuote(baseCmd)} /dev/null`
+                : baseCmd;
+            return { command: 'codex', shellCommand: wrappedCmd };
         }
         case 'claude':
         default: {
-            const cmd = `claude ${quotedInstruction} --dangerously-skip-permissions`;
+            const prefix = options.runningAsRoot ? 'IS_SANDBOX=1 ' : '';
+            const cmd = `${prefix}claude ${quotedInstruction} --dangerously-skip-permissions`;
             return { command: 'claude', shellCommand: cmd };
         }
     }
@@ -85,7 +90,8 @@ export class JobRunner {
     async run(job, runId) {
         const agent = resolveAgent(job.model);
         const instruction = this.buildInstruction(job, CLI_AGENT_LABEL[agent] || agent);
-        const { shellCommand } = buildCommand(agent, instruction);
+        const runningAsRoot = typeof process.getuid === 'function' && process.getuid() === 0;
+        const { shellCommand } = buildCommand(agent, instruction, { runningAsRoot });
         const projectRoot = await this.projectRootPromise;
         const logPath = await this.ensureLogPath(job.id, runId);
         const logStream = createWriteStream(logPath, { flags: 'a' });

package/dist/routes/api/v1/agents/[id]/route.js

@@ -3,6 +3,24 @@ import { promises as fs } from 'fs';
 import path from 'path';
 import matter from 'gray-matter';
 const app = new Hono();
+function normalizeTools(value) {
+    if (!value)
+        return undefined;
+    if (Array.isArray(value)) {
+        const cleaned = value
+            .map((item) => (typeof item === 'string' ? item.trim() : ''))
+            .filter((item) => Boolean(item));
+        return cleaned.length > 0 ? Array.from(new Set(cleaned)) : undefined;
+    }
+    if (typeof value === 'string') {
+        const cleaned = value
+            .split(',')
+            .map((item) => item.trim())
+            .filter(Boolean);
+        return cleaned.length > 0 ? Array.from(new Set(cleaned)) : undefined;
+    }
+    return undefined;
+}
 function getAgentsPath() {
     let basePath;
     if (process.env.NODE_ENV === 'development' && process.env.GAIT_DEV_ROOT) {
@@ -47,11 +65,11 @@ app.get('/', async (c) => {
         const document = {
             filename,
             metadata: {
+                ...data,
                 name: data.name || title,
                 description: data.description || '',
                 color: data.color,
-                tools: data.tools,
-                ...data
+                tools: normalizeTools(data.tools)
             },
             content,
             title
@@ -98,6 +116,13 @@ app.put('/', async (c) => {
             name: body.name !== undefined ? body.name : currentData.name,
             description: body.description !== undefined ? body.description : currentData.description
         };
+        const normalizedTools = normalizeTools(body.metadata?.tools ?? updatedMetadata.tools);
+        if (normalizedTools) {
+            updatedMetadata.tools = normalizedTools;
+        }
+        else {
+            delete updatedMetadata.tools;
+        }
         // Handle name change - might need to rename file
         let newFilename = filename;
         let newFilePath = filePath;

package/dist/routes/api/v1/agents/route.js

@@ -3,6 +3,24 @@ import { promises as fs } from 'fs';
 import path from 'path';
 import matter from 'gray-matter';
 const app = new Hono();
+function normalizeTools(value) {
+    if (!value)
+        return undefined;
+    if (Array.isArray(value)) {
+        const cleaned = value
+            .map((item) => (typeof item === 'string' ? item.trim() : ''))
+            .filter((item) => Boolean(item));
+        return cleaned.length > 0 ? Array.from(new Set(cleaned)) : undefined;
+    }
+    if (typeof value === 'string') {
+        const cleaned = value
+            .split(',')
+            .map((item) => item.trim())
+            .filter(Boolean);
+        return cleaned.length > 0 ? Array.from(new Set(cleaned)) : undefined;
+    }
+    return undefined;
+}
 function getAgentsPath() {
     let basePath;
     if (process.env.NODE_ENV === 'development' && process.env.GAIT_DEV_ROOT) {
@@ -52,11 +70,11 @@ app.get('/', async (c) => {
                 return {
                     filename: file,
                     metadata: {
+                        ...data,
                         name: data.name || title,
                         description: data.description || '',
                         color: data.color,
-                        tools: data.tools,
-                        ...data
+                        tools: normalizeTools(data.tools)
                     },
                     content,
                     title
@@ -115,13 +133,22 @@ app.post('/', async (c) => {
             // File doesn't exist, which is what we want
         }
         // Prepare frontmatter - filter out undefined values
-        const frontmatter = Object.fromEntries(Object.entries({
+        const normalizedTools = normalizeTools(body.metadata?.tools);
+        const frontmatterSource = {
+            ...body.metadata,
             name: body.name,
-            description: body.description,
-            color: body.metadata?.color || 'blue',
-            tools: body.metadata?.tools,
-            ...body.metadata
-        }).filter(([_, value]) => value !== undefined));
+            description: body.description
+        };
+        frontmatterSource.color = typeof body.metadata?.color === 'string' && body.metadata.color
+            ? body.metadata.color
+            : 'blue';
+        if (normalizedTools) {
+            frontmatterSource.tools = normalizedTools;
+        }
+        else {
+            delete frontmatterSource.tools;
+        }
+        const frontmatter = Object.fromEntries(Object.entries(frontmatterSource).filter(([, value]) => value !== undefined));
         // Create the markdown content with frontmatter
         const fileContent = matter.stringify(body.content || '', frontmatter);
         await fs.writeFile(filePath, fileContent, 'utf-8');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lovelybunch/api",
-  "version": "1.0.59",
+  "version": "1.0.61",
   "type": "module",
   "main": "dist/server-with-static.js",
   "exports": {
@@ -32,9 +32,9 @@
   "dependencies": {
     "@hono/node-server": "^1.13.7",
     "@hono/node-ws": "^1.0.6",
-    "@lovelybunch/core": "^1.0.59",
-    "@lovelybunch/mcp": "^1.0.59",
-    "@lovelybunch/types": "^1.0.59",
+    "@lovelybunch/core": "^1.0.61",
+    "@lovelybunch/mcp": "^1.0.61",
+    "@lovelybunch/types": "^1.0.61",
     "arctic": "^1.9.2",
     "bcrypt": "^5.1.1",
     "cookie": "^0.6.0",