biome

Author: cramforce

PROJECT.md

@@ -205,7 +205,6 @@ Woohoo
 
 ## Implementation phase 13
 
-- Block `return await` with biome
 - Do we handle stop of execution in multi-line script for non-zero return values?
 
 ## Implementation phase 14
@@ -214,3 +213,7 @@ Woohoo
   - Search the web for official grammars and use an existing npm package for parsing the grammer and the input
 - Further extend composition tests
 - Start by writing `real-parser.md` and then confirm further work.
+
+## Implementation phase 15
+
+- Implement set -e

src/BashEnv.ts

@@ -2,24 +2,24 @@ import { createLazyCommands } from "./commands/registry.js";
 import { type IFileSystem, VirtualFs } from "./fs.js";
 import type { InitialFiles } from "./fs-interface.js";
 import {
-  executeIfStatement,
+  type BuiltinContext,
+  evaluateTopLevelTest,
+  executeCaseStatement,
   executeForLoop,
-  executeWhileLoop,
+  executeIfStatement,
   executeUntilLoop,
-  executeCaseStatement,
-  evaluateTopLevelTest,
+  executeWhileLoop,
   executeWithHereDoc,
   expandVariablesAsync,
+  type HereDocContext,
   handleCd,
+  handleExit,
   handleExport,
-  handleUnset,
   handleLocal,
-  handleExit,
-  handleVariableAssignment,
   handleTestExpression,
+  handleUnset,
+  handleVariableAssignment,
   type InterpreterContext,
-  type HereDocContext,
-  type BuiltinContext,
 } from "./interpreter/index.js";
 import {
   GlobExpander,
@@ -743,7 +743,10 @@ export class BashEnv {
 
     // Handle [[ ]] test expressions
     if (expandedCommand === "[[") {
-      return await handleTestExpression(expandedArgs, this.getInterpreterContext());
+      return await handleTestExpression(
+        expandedArgs,
+        this.getInterpreterContext(),
+      );
     }
 
     // Handle variable assignment: VAR=value (no args, command contains =)

src/agent-examples/codebase-exploration.test.ts

@@ -185,21 +185,45 @@ describe("Agent Scenario: Disk Usage Analysis with ls -h and du -h", () => {
       files: {
         // Small config files
         "/project/package.json": { content: '{"name": "app"}', mode: 0o644 },
-        "/project/tsconfig.json": { content: '{"target": "es2020"}', mode: 0o644 },
+        "/project/tsconfig.json": {
+          content: '{"target": "es2020"}',
+          mode: 0o644,
+        },
         // Source files of varying sizes
         "/project/src/index.ts": { content: "a".repeat(500), mode: 0o644 },
         "/project/src/utils.ts": { content: "b".repeat(2 * 1024), mode: 0o644 }, // 2K
         "/project/src/api.ts": { content: "c".repeat(5 * 1024), mode: 0o644 }, // 5K
         // Large build artifacts
-        "/project/dist/bundle.js": { content: "d".repeat(150 * 1024), mode: 0o644 }, // 150K
-        "/project/dist/bundle.js.map": { content: "e".repeat(300 * 1024), mode: 0o644 }, // 300K
+        "/project/dist/bundle.js": {
+          content: "d".repeat(150 * 1024),
+          mode: 0o644,
+        }, // 150K
+        "/project/dist/bundle.js.map": {
+          content: "e".repeat(300 * 1024),
+          mode: 0o644,
+        }, // 300K
         // Node modules simulation
-        "/project/node_modules/lodash/index.js": { content: "f".repeat(500 * 1024), mode: 0o644 }, // 500K
-        "/project/node_modules/lodash/package.json": { content: '{"name": "lodash"}', mode: 0o644 },
-        "/project/node_modules/express/index.js": { content: "g".repeat(200 * 1024), mode: 0o644 }, // 200K
+        "/project/node_modules/lodash/index.js": {
+          content: "f".repeat(500 * 1024),
+          mode: 0o644,
+        }, // 500K
+        "/project/node_modules/lodash/package.json": {
+          content: '{"name": "lodash"}',
+          mode: 0o644,
+        },
+        "/project/node_modules/express/index.js": {
+          content: "g".repeat(200 * 1024),
+          mode: 0o644,
+        }, // 200K
         // Log files
-        "/project/logs/app.log": { content: "h".repeat(1024 * 1024), mode: 0o644 }, // 1M
-        "/project/logs/error.log": { content: "i".repeat(50 * 1024), mode: 0o644 }, // 50K
+        "/project/logs/app.log": {
+          content: "h".repeat(1024 * 1024),
+          mode: 0o644,
+        }, // 1M
+        "/project/logs/error.log": {
+          content: "i".repeat(50 * 1024),
+          mode: 0o644,
+        }, // 50K
       },
       cwd: "/project",
     });

src/agent-examples/refactoring-workflow.test.ts

@@ -388,9 +388,7 @@ fifth line`,
     it("should convert single imports to barrel export", async () => {
       const env = createAdvancedEnv();
       // Transform import to export (simpler pattern)
-      const result = await env.exec(
-        "sed 's/import/export/' /code/imports.ts",
-      );
+      const result = await env.exec("sed 's/import/export/' /code/imports.ts");
       expect(result.stdout).toContain("export { foo }");
       expect(result.stdout).toContain("export { bar }");
       expect(result.exitCode).toBe(0);
@@ -399,9 +397,7 @@ fifth line`,
     it("should find functions that need documentation", async () => {
       const env = createAdvancedEnv();
       // Find function declarations that could use JSDoc
-      const result = await env.exec(
-        "grep -n '^function' /code/deprecated.ts",
-      );
+      const result = await env.exec("grep -n '^function' /code/deprecated.ts");
       expect(result.stdout).toContain("function oldFunc");
       expect(result.stdout).toContain("function activeFunc");
       expect(result.exitCode).toBe(0);

src/agent-examples/security-audit.test.ts

@@ -274,12 +274,27 @@ describe("Agent Scenario: File Permission Audit with find -perm", () => {
   const createPermEnv = () =>
     new BashEnv({
       files: {
-        "/server/bin/start.sh": { content: "#!/bin/bash\nnode app.js", mode: 0o755 },
-        "/server/bin/deploy.sh": { content: "#!/bin/bash\n# deploy script", mode: 0o755 },
-        "/server/bin/backup.sh": { content: "#!/bin/bash\n# backup", mode: 0o700 },
+        "/server/bin/start.sh": {
+          content: "#!/bin/bash\nnode app.js",
+          mode: 0o755,
+        },
+        "/server/bin/deploy.sh": {
+          content: "#!/bin/bash\n# deploy script",
+          mode: 0o755,
+        },
+        "/server/bin/backup.sh": {
+          content: "#!/bin/bash\n# backup",
+          mode: 0o700,
+        },
         "/server/config/app.json": { content: '{"port": 3000}', mode: 0o644 },
-        "/server/config/secrets.json": { content: '{"key": "secret"}', mode: 0o600 },
-        "/server/config/db.json": { content: '{"host": "localhost"}', mode: 0o644 },
+        "/server/config/secrets.json": {
+          content: '{"key": "secret"}',
+          mode: 0o600,
+        },
+        "/server/config/db.json": {
+          content: '{"host": "localhost"}',
+          mode: 0o644,
+        },
         "/server/logs/app.log": { content: "log data", mode: 0o644 },
         "/server/logs/error.log": { content: "errors", mode: 0o644 },
         "/server/data/users.db": { content: "user data", mode: 0o600 },

src/agent-examples/text-processing-workflows.test.ts

@@ -698,7 +698,7 @@ END_RECORD`,
         "awk '/ERROR BLOCK START/,/ERROR BLOCK END/ { count++ } END { print count }' /logs/server.log",
       );
       // Two blocks, 5 lines each (START + error + stack + 2 traces + END)
-      expect(parseInt(result.stdout.trim())).toBeGreaterThan(8);
+      expect(parseInt(result.stdout.trim(), 10)).toBeGreaterThan(8);
       expect(result.exitCode).toBe(0);
     });
   });
@@ -718,9 +718,7 @@ END_RECORD`,
 
     it("should extract cache settings", async () => {
       const env = createLogEnv();
-      const result = await env.exec(
-        "grep -A3 '\\[cache\\]' /logs/config.ini",
-      );
+      const result = await env.exec("grep -A3 '\\[cache\\]' /logs/config.ini");
       expect(result.stdout).toContain("[cache]");
       expect(result.stdout).toContain("enabled=true");
       expect(result.stdout).toContain("ttl=3600");
@@ -732,9 +730,11 @@ END_RECORD`,
     it("should combine record fields using getline", async () => {
       const env = createLogEnv();
       const result = await env.exec(
-        "awk '/^RECORD:/ { id=$2; getline; split($0,a,\": \"); name=a[2]; getline; split($0,b,\": \"); email=b[2]; print id, name, email }' /data/records.txt",
+        'awk \'/^RECORD:/ { id=$2; getline; split($0,a,": "); name=a[2]; getline; split($0,b,": "); email=b[2]; print id, name, email }\' /data/records.txt',
+      );
+      expect(result.stdout).toContain(
+        "user001 Alice Johnson alice@example.com",
       );
-      expect(result.stdout).toContain("user001 Alice Johnson alice@example.com");
       expect(result.stdout).toContain("user002 Bob Smith bob@example.com");
       expect(result.exitCode).toBe(0);
     });
@@ -895,7 +895,7 @@ Stack trace follows...`,
     it("should sanitize for SQL safety (remove quotes)", async () => {
       const env = createSanitizeEnv();
       const result = await env.exec(
-        "echo \"user'; DROP TABLE users;--\" | tr -d \"';\"",
+        'echo "user\'; DROP TABLE users;--" | tr -d "\';"',
       );
       expect(result.stdout).not.toContain("'");
       expect(result.stdout).not.toContain(";");

src/commands/awk/awk.math.test.ts

@@ -76,7 +76,9 @@ describe("awk math functions", () => {
 
     it("srand() can be called with a seed", async () => {
       const env = new BashEnv();
-      const result = await env.exec("echo '' | awk '{ srand(42); print rand() }'");
+      const result = await env.exec(
+        "echo '' | awk '{ srand(42); print rand() }'",
+      );
       const value = parseFloat(result.stdout.trim());
       expect(value).toBeGreaterThanOrEqual(0);
       expect(value).toBeLessThan(1);

src/commands/awk/executor.ts

@@ -576,7 +576,9 @@ function handleGetline(stmt: string, ctx: AwkContext): string {
   if (stmt === "getline") {
     // Read into $0
     ctx.line = nextLine;
-    ctx.fields = ctx.fieldSep ? nextLine.split(ctx.fieldSep) : nextLine.split(/\s+/);
+    ctx.fields = ctx.fieldSep
+      ? nextLine.split(ctx.fieldSep)
+      : nextLine.split(/\s+/);
     ctx.NF = ctx.fields.length;
     ctx.NR++;
     ctx.lineIndex = nextLineIndex;

src/commands/awk/expressions.ts

@@ -70,9 +70,12 @@ export function evaluateExpression(
       case "srand": {
         // In real awk, srand() seeds the random number generator
         // We'll just return a value and the random will work with Math.random()
-        const seed = args.length > 0 ? Number(evaluateExpression(args[0], ctx)) : Date.now();
+        const seed =
+          args.length > 0
+            ? Number(evaluateExpression(args[0], ctx))
+            : Date.now();
         // Store seed for reference (doesn't actually change behavior)
-        ctx.vars["_srand_seed"] = seed;
+        ctx.vars._srand_seed = seed;
         return seed;
       }
     }

src/commands/awk/parser.ts

@@ -48,7 +48,9 @@ export function parseAwkProgram(program: string): ParsedProgram {
       }
     } else {
       // Pattern range { action }: /start/,/end/ { action }
-      const rangeAction = remaining.match(/^\/([^/]*)\/\s*,\s*\/([^/]*)\/\s*\{/);
+      const rangeAction = remaining.match(
+        /^\/([^/]*)\/\s*,\s*\/([^/]*)\/\s*\{/,
+      );
       if (rangeAction) {
         const actionStart = remaining.indexOf("{");
         const endBrace = findMatchingBrace(remaining, actionStart);
@@ -97,7 +99,10 @@ export function parseAwkProgram(program: string): ParsedProgram {
               }
             } else {
               // Condition only (no action) or just a print expression
-              if (remaining.startsWith("print") || remaining.startsWith("printf")) {
+              if (
+                remaining.startsWith("print") ||
+                remaining.startsWith("printf")
+              ) {
                 result.main.push({ pattern: null, action: remaining });
               } else {
                 // It's a condition without action - default to print