lint

Author: cramforce

src/BashEnv.ts

@@ -27,12 +27,7 @@ import {
   type SecureFetch,
 } from "./network/index.js";
 import { type ParseException, parse } from "./parser/parser.js";
-import type {
-  BashExecResult,
-  Command,
-  CommandRegistry,
-  ExecResult,
-} from "./types.js";
+import type { BashExecResult, Command, CommandRegistry } from "./types.js";
 
 // Default protection limits
 const DEFAULT_MAX_CALL_DEPTH = 100;

src/commands/alias/alias.test.ts

@@ -76,7 +76,9 @@ describe("unalias command", () => {
 
   it("should remove all aliases with -a", async () => {
     const env = new BashEnv();
-    const result = await env.exec("alias ll='ls -la' la='ls -a'; unalias -a; alias");
+    const result = await env.exec(
+      "alias ll='ls -la' la='ls -a'; unalias -a; alias",
+    );
     expect(result.stdout).toBe("");
     expect(result.exitCode).toBe(0);
   });

src/comparison-tests/export.comparison.test.ts

@@ -25,11 +25,7 @@ describe("export command - Real Bash Comparison", () => {
 
     it("should set multiple variables", async () => {
       const env = await setupFiles(testDir, {});
-      await compareOutputs(
-        env,
-        testDir,
-        "export A=1 B=2 C=3; echo $A $B $C",
-      );
+      await compareOutputs(env, testDir, "export A=1 B=2 C=3; echo $A $B $C");
     });
 
     it("should handle value with equals sign", async () => {
@@ -43,7 +39,7 @@ describe("export command - Real Bash Comparison", () => {
 
     it("should handle empty value", async () => {
       const env = await setupFiles(testDir, {});
-      await compareOutputs(env, testDir, "export EMPTY=; echo \"[$EMPTY]\"");
+      await compareOutputs(env, testDir, 'export EMPTY=; echo "[$EMPTY]"');
     });
   });
 
@@ -76,19 +72,15 @@ describe("export command - Real Bash Comparison", () => {
       await compareOutputs(
         env,
         testDir,
-        "export NAME=world; echo \"hello $NAME\"",
+        'export NAME=world; echo "hello $NAME"',
       );
     });
   });
 
   describe("inline export", () => {
     it("should allow setting and using in same line", async () => {
       const env = await setupFiles(testDir, {});
-      await compareOutputs(
-        env,
-        testDir,
-        "export X=42 && echo $X",
-      );
+      await compareOutputs(env, testDir, "export X=42 && echo $X");
     });
   });
 });

src/interpreter/builtins/export.test.ts

@@ -25,7 +25,9 @@ describe("export builtin", () => {
 
     it("should create empty variable when NAME has no value", async () => {
       const env = new BashEnv();
-      const result = await env.exec('export EMPTY; test -z "$EMPTY" && echo empty');
+      const result = await env.exec(
+        'export EMPTY; test -z "$EMPTY" && echo empty',
+      );
       expect(result.stdout).toBe("empty\n");
     });
 
@@ -60,7 +62,7 @@ describe("export builtin", () => {
 
     it("should list newly exported variables within same exec", async () => {
       const env = new BashEnv();
-      const result = await env.exec("export MSG=\"it's working\"; export");
+      const result = await env.exec('export MSG="it\'s working"; export');
       expect(result.stdout).toContain("it'\\''s working");
     });
 
@@ -75,7 +77,9 @@ describe("export builtin", () => {
   describe("un-exporting with -n", () => {
     it("should remove variable with -n (within same exec)", async () => {
       const env = new BashEnv({ env: { FOO: "bar" } });
-      const result = await env.exec('export -n FOO; test -z "$FOO" && echo removed');
+      const result = await env.exec(
+        'export -n FOO; test -z "$FOO" && echo removed',
+      );
       expect(result.stdout).toBe("removed\n");
     });
 
@@ -91,7 +95,9 @@ describe("export builtin", () => {
   describe("variable usage", () => {
     it("exported variable should be available in same exec", async () => {
       const env = new BashEnv();
-      const result = await env.exec("export GREETING=hello; echo $GREETING world");
+      const result = await env.exec(
+        "export GREETING=hello; echo $GREETING world",
+      );
       expect(result.stdout).toBe("hello world\n");
     });
 
@@ -103,7 +109,9 @@ describe("export builtin", () => {
 
     it("should work with conditional", async () => {
       const env = new BashEnv();
-      const result = await env.exec('export DEBUG=1; [ "$DEBUG" = "1" ] && echo debug_on');
+      const result = await env.exec(
+        'export DEBUG=1; [ "$DEBUG" = "1" ] && echo debug_on',
+      );
       expect(result.stdout).toBe("debug_on\n");
     });
 

src/interpreter/builtins/export.ts

@@ -24,11 +24,7 @@ export function handleExport(
     if (arg === "-n") {
       unexport = true;
     } else if (arg === "-p") {
-      // -p is same as no args, just list
-      continue;
     } else if (arg === "--") {
-      // End of options
-      continue;
     } else {
       processedArgs.push(arg);
     }