@aria-cli/tools 1.0.9 → 1.0.11

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (241) hide show
  1. package/package.json +9 -5
  2. package/src/__tests__/web-fetch-download.test.ts +0 -433
  3. package/src/__tests__/web-tools.test.ts +0 -619
  4. package/src/ask-user-interaction.ts +0 -33
  5. package/src/cache/web-cache.ts +0 -110
  6. package/src/definitions/arion.ts +0 -118
  7. package/src/definitions/browser/browser.ts +0 -502
  8. package/src/definitions/browser/index.ts +0 -5
  9. package/src/definitions/browser/pw-downloads.ts +0 -142
  10. package/src/definitions/browser/pw-interactions.ts +0 -282
  11. package/src/definitions/browser/pw-responses.ts +0 -98
  12. package/src/definitions/browser/pw-session.ts +0 -405
  13. package/src/definitions/browser/pw-shared.ts +0 -85
  14. package/src/definitions/browser/pw-snapshot.ts +0 -383
  15. package/src/definitions/browser/pw-state.ts +0 -101
  16. package/src/definitions/browser/types.ts +0 -203
  17. package/src/definitions/code-intelligence.ts +0 -526
  18. package/src/definitions/core.ts +0 -118
  19. package/src/definitions/delegation.ts +0 -567
  20. package/src/definitions/deploy.ts +0 -73
  21. package/src/definitions/filesystem.ts +0 -217
  22. package/src/definitions/frg.ts +0 -67
  23. package/src/definitions/index.ts +0 -28
  24. package/src/definitions/memory.ts +0 -150
  25. package/src/definitions/messaging.ts +0 -734
  26. package/src/definitions/meta.ts +0 -392
  27. package/src/definitions/network.ts +0 -179
  28. package/src/definitions/outlook.ts +0 -318
  29. package/src/definitions/patch/apply-patch.ts +0 -235
  30. package/src/definitions/patch/fuzzy-match.ts +0 -217
  31. package/src/definitions/patch/index.ts +0 -1
  32. package/src/definitions/patch/patch-parser.ts +0 -297
  33. package/src/definitions/patch/sandbox-paths.ts +0 -129
  34. package/src/definitions/process/index.ts +0 -5
  35. package/src/definitions/process/process-registry.ts +0 -303
  36. package/src/definitions/process/process.ts +0 -456
  37. package/src/definitions/process/pty-keys.ts +0 -298
  38. package/src/definitions/process/session-slug.ts +0 -147
  39. package/src/definitions/quip.ts +0 -225
  40. package/src/definitions/search.ts +0 -67
  41. package/src/definitions/session-history.ts +0 -79
  42. package/src/definitions/shell.ts +0 -202
  43. package/src/definitions/slack.ts +0 -211
  44. package/src/definitions/web.ts +0 -119
  45. package/src/executors/apply-patch.ts +0 -1035
  46. package/src/executors/arion.ts +0 -199
  47. package/src/executors/code-intelligence.ts +0 -1179
  48. package/src/executors/deploy.ts +0 -1066
  49. package/src/executors/filesystem.ts +0 -1428
  50. package/src/executors/frg-freshness.ts +0 -743
  51. package/src/executors/frg.ts +0 -394
  52. package/src/executors/index.ts +0 -280
  53. package/src/executors/learning-meta.ts +0 -1367
  54. package/src/executors/lsp-client.ts +0 -355
  55. package/src/executors/memory.ts +0 -978
  56. package/src/executors/meta.ts +0 -293
  57. package/src/executors/process-registry.ts +0 -570
  58. package/src/executors/pty-session-store.ts +0 -43
  59. package/src/executors/pty.ts +0 -342
  60. package/src/executors/restart.ts +0 -133
  61. package/src/executors/search-freshness.ts +0 -249
  62. package/src/executors/search-types.ts +0 -98
  63. package/src/executors/search.ts +0 -89
  64. package/src/executors/self-diagnose.ts +0 -552
  65. package/src/executors/session-history.ts +0 -435
  66. package/src/executors/shell-safety.ts +0 -519
  67. package/src/executors/shell.ts +0 -1243
  68. package/src/executors/utils.ts +0 -40
  69. package/src/executors/web.ts +0 -786
  70. package/src/extraction/content-extraction.ts +0 -281
  71. package/src/extraction/index.ts +0 -5
  72. package/src/headless-control-contract.ts +0 -1149
  73. package/src/index.ts +0 -788
  74. package/src/local-control-http-auth.ts +0 -2
  75. package/src/mcp/client.ts +0 -218
  76. package/src/mcp/connection.ts +0 -568
  77. package/src/mcp/index.ts +0 -11
  78. package/src/mcp/jsonrpc.ts +0 -195
  79. package/src/mcp/types.ts +0 -199
  80. package/src/network-control-adapter.ts +0 -88
  81. package/src/network-runtime/address-types.ts +0 -218
  82. package/src/network-runtime/db-owner-fencing.ts +0 -91
  83. package/src/network-runtime/delivery-receipts.ts +0 -372
  84. package/src/network-runtime/direct-endpoint-authority.ts +0 -35
  85. package/src/network-runtime/index.ts +0 -316
  86. package/src/network-runtime/local-control-contract.ts +0 -784
  87. package/src/network-runtime/node-store-contract.ts +0 -46
  88. package/src/network-runtime/pair-route-contract.ts +0 -97
  89. package/src/network-runtime/peer-capabilities.ts +0 -48
  90. package/src/network-runtime/peer-principal-ref.ts +0 -20
  91. package/src/network-runtime/peer-state-machine.ts +0 -160
  92. package/src/network-runtime/protocol-schemas.ts +0 -265
  93. package/src/network-runtime/runtime-bootstrap-contract.ts +0 -83
  94. package/src/outlook/desktop-session.ts +0 -409
  95. package/src/policy.ts +0 -171
  96. package/src/providers/brave.ts +0 -80
  97. package/src/providers/duckduckgo.ts +0 -199
  98. package/src/providers/exa.ts +0 -85
  99. package/src/providers/firecrawl.ts +0 -77
  100. package/src/providers/index.ts +0 -8
  101. package/src/providers/jina.ts +0 -70
  102. package/src/providers/router.ts +0 -121
  103. package/src/providers/search-provider.ts +0 -74
  104. package/src/providers/tavily.ts +0 -74
  105. package/src/quip/desktop-session.ts +0 -435
  106. package/src/registry/index.ts +0 -1
  107. package/src/registry/registry.ts +0 -905
  108. package/src/runtime-socket-local-control-client.ts +0 -632
  109. package/src/security/dns-normalization.ts +0 -34
  110. package/src/security/dns-pinning.ts +0 -138
  111. package/src/security/external-content.ts +0 -129
  112. package/src/security/ssrf.ts +0 -207
  113. package/src/slack/desktop-session.ts +0 -493
  114. package/src/tool-factory.ts +0 -91
  115. package/src/types.ts +0 -1341
  116. package/src/utils/retry.ts +0 -163
  117. package/src/utils/safe-parse-json.ts +0 -176
  118. package/src/utils/url.ts +0 -20
  119. package/tests/benchmarks/registry.bench.ts +0 -57
  120. package/tests/cache/web-cache.test.ts +0 -147
  121. package/tests/critical-integration.test.ts +0 -1465
  122. package/tests/definitions/apply-patch.test.ts +0 -586
  123. package/tests/definitions/browser.test.ts +0 -495
  124. package/tests/definitions/delegation-pause-resume.test.ts +0 -758
  125. package/tests/definitions/execution.test.ts +0 -671
  126. package/tests/definitions/messaging-inbox-scope.test.ts +0 -229
  127. package/tests/definitions/messaging.test.ts +0 -1468
  128. package/tests/definitions/outlook.test.ts +0 -30
  129. package/tests/definitions/process.test.ts +0 -469
  130. package/tests/definitions/slack.test.ts +0 -28
  131. package/tests/definitions/tool-inventory.test.ts +0 -218
  132. package/tests/e2e/delegation-quest-orchestration.e2e.test.ts +0 -433
  133. package/tests/e2e/memory-tool-discovery-contract.e2e.test.ts +0 -81
  134. package/tests/executors/apply-patch.test.ts +0 -538
  135. package/tests/executors/arion.test.ts +0 -309
  136. package/tests/executors/conversation-primitives.test.ts +0 -250
  137. package/tests/executors/deploy.test.ts +0 -746
  138. package/tests/executors/filesystem-tools.test.ts +0 -357
  139. package/tests/executors/filesystem.test.ts +0 -959
  140. package/tests/executors/frg-freshness.test.ts +0 -136
  141. package/tests/executors/frg-merge.test.ts +0 -70
  142. package/tests/executors/frg-session-content.test.ts +0 -40
  143. package/tests/executors/frg.test.ts +0 -56
  144. package/tests/executors/memory-bugfixes.test.ts +0 -257
  145. package/tests/executors/memory-real-memoria.integration.test.ts +0 -316
  146. package/tests/executors/memory.test.ts +0 -853
  147. package/tests/executors/meta-tools.test.ts +0 -411
  148. package/tests/executors/meta.test.ts +0 -683
  149. package/tests/executors/path-containment.test.ts +0 -51
  150. package/tests/executors/process-registry.test.ts +0 -505
  151. package/tests/executors/pty.test.ts +0 -664
  152. package/tests/executors/quest-security.test.ts +0 -249
  153. package/tests/executors/read-file-media.test.ts +0 -230
  154. package/tests/executors/recall-knowledge-schema.test.ts +0 -209
  155. package/tests/executors/recall-tags.test.ts +0 -278
  156. package/tests/executors/remember-null-safety.contract.test.ts +0 -41
  157. package/tests/executors/restart.test.ts +0 -67
  158. package/tests/executors/search-unified.test.ts +0 -381
  159. package/tests/executors/session-history.test.ts +0 -340
  160. package/tests/executors/session-transcript.test.ts +0 -561
  161. package/tests/executors/shell-abort.test.ts +0 -416
  162. package/tests/executors/shell-env-blocklist.test.ts +0 -648
  163. package/tests/executors/shell-env-process.test.ts +0 -245
  164. package/tests/executors/shell-process-registry.test.ts +0 -334
  165. package/tests/executors/shell-tools.test.ts +0 -393
  166. package/tests/executors/shell.test.ts +0 -690
  167. package/tests/executors/web-abort-vs-timeout.test.ts +0 -213
  168. package/tests/executors/web-integration.test.ts +0 -633
  169. package/tests/executors/web-symlink.test.ts +0 -18
  170. package/tests/executors/web.test.ts +0 -1400
  171. package/tests/executors/write-stdin.test.ts +0 -145
  172. package/tests/extraction/content-extraction.test.ts +0 -153
  173. package/tests/guards/tools-default-test-lane.integration.test.ts +0 -21
  174. package/tests/guards/tools-package-test-commands.e2e.test.ts +0 -43
  175. package/tests/guards/tools-test-lane-manifest.contract.test.ts +0 -76
  176. package/tests/guards/tools-vitest-workspace-alias.contract.test.ts +0 -63
  177. package/tests/helpers/async-waits.ts +0 -53
  178. package/tests/integration/headless-control-contract.integration.test.ts +0 -153
  179. package/tests/integration/memory-tool-schema-parity.integration.test.ts +0 -67
  180. package/tests/integration/meta-tools-round-trip.integration.test.ts +0 -506
  181. package/tests/integration/quest-round-trip.test.ts +0 -303
  182. package/tests/integration/registry-executor-flow.test.ts +0 -85
  183. package/tests/integration.test.ts +0 -177
  184. package/tests/loading-tier.test.ts +0 -126
  185. package/tests/mcp/client-reconnect.test.ts +0 -267
  186. package/tests/mcp/connection.test.ts +0 -846
  187. package/tests/mcp/injectable-logger.test.ts +0 -83
  188. package/tests/mcp/jsonrpc.test.ts +0 -109
  189. package/tests/mcp/lifecycle.test.ts +0 -879
  190. package/tests/network-runtime/address-types.contract.test.ts +0 -143
  191. package/tests/network-runtime/continuity-bind-schema.contract.test.ts +0 -203
  192. package/tests/network-runtime/local-control-contract.test.ts +0 -869
  193. package/tests/network-runtime/local-control-invite-token.contract.test.ts +0 -146
  194. package/tests/network-runtime/node-store-contract.test.ts +0 -11
  195. package/tests/network-runtime/pair-protocol-nodeid.contract.test.ts +0 -15
  196. package/tests/network-runtime/peer-state-machine.contract.test.ts +0 -148
  197. package/tests/network-runtime/protocol-schemas.contract.test.ts +0 -512
  198. package/tests/network-runtime/relay-pending-nodeid.contract.test.ts +0 -62
  199. package/tests/network-runtime/runtime-bootstrap-contract.test.ts +0 -227
  200. package/tests/network-runtime/runtime-socket-local-control-client.test.ts +0 -621
  201. package/tests/network-runtime/wait-for-message-script.test.ts +0 -288
  202. package/tests/parallel.test.ts +0 -71
  203. package/tests/policy.test.ts +0 -184
  204. package/tests/print-default-test-lane.ts +0 -14
  205. package/tests/print-test-lane-manifest.ts +0 -22
  206. package/tests/providers/brave.test.ts +0 -159
  207. package/tests/providers/duckduckgo.test.ts +0 -207
  208. package/tests/providers/exa.test.ts +0 -175
  209. package/tests/providers/firecrawl.test.ts +0 -168
  210. package/tests/providers/jina.test.ts +0 -144
  211. package/tests/providers/router.test.ts +0 -328
  212. package/tests/providers/tavily.test.ts +0 -165
  213. package/tests/registry/discovery.test.ts +0 -154
  214. package/tests/registry/injectable-logger.test.ts +0 -230
  215. package/tests/registry/input-validation.test.ts +0 -361
  216. package/tests/registry/interface-completeness.test.ts +0 -85
  217. package/tests/registry/mcp-integration.test.ts +0 -103
  218. package/tests/registry/mcp-read-only-hint.test.ts +0 -60
  219. package/tests/registry/memoria-discovery.test.ts +0 -390
  220. package/tests/registry/nested-validation.test.ts +0 -283
  221. package/tests/registry/pseudo-tool-filtering.test.ts +0 -258
  222. package/tests/registry/registration-lifecycle.test.ts +0 -133
  223. package/tests/registry-validation.test.ts +0 -424
  224. package/tests/registry.test.ts +0 -460
  225. package/tests/security/dns-pinning.test.ts +0 -162
  226. package/tests/security/external-content.test.ts +0 -144
  227. package/tests/security/ssrf.test.ts +0 -118
  228. package/tests/shell-safety-integration.test.ts +0 -32
  229. package/tests/shell-safety.test.ts +0 -365
  230. package/tests/slack/desktop-session.test.ts +0 -50
  231. package/tests/test-lane-manifest.ts +0 -440
  232. package/tests/test-utils.ts +0 -27
  233. package/tests/tool-factory.test.ts +0 -188
  234. package/tests/utils/retry.test.ts +0 -231
  235. package/tests/utils/url.test.ts +0 -63
  236. package/tsconfig.cjs.json +0 -24
  237. package/tsconfig.json +0 -12
  238. package/vitest.config.ts +0 -55
  239. package/vitest.e2e.config.ts +0 -24
  240. package/vitest.integration.config.ts +0 -24
  241. package/vitest.native.config.ts +0 -24
@@ -1,648 +0,0 @@
1
- /**
2
- * @aria/tools - Shell env var blocklist tests (C1 + I1)
3
- *
4
- * Verifies that dangerous environment variables are stripped from both
5
- * inputEnv and ctx.env before being passed to child processes.
6
- */
7
-
8
- import { describe, it, expect, beforeEach, afterEach } from "vitest";
9
- import * as fs from "node:fs/promises";
10
- import * as path from "node:path";
11
- import * as os from "node:os";
12
- import type { ToolContext } from "../../src/types.js";
13
- import { executeBash } from "../../src/executors/shell.js";
14
-
15
- // Uses realpath to resolve symlinks (e.g., /tmp -> /private/tmp on macOS)
16
- const createTempDir = async (): Promise<string> => {
17
- const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), "aria-blocklist-test-"));
18
- return fs.realpath(tempDir);
19
- };
20
-
21
- const cleanupTempDir = async (dir: string): Promise<void> => {
22
- await fs.rm(dir, { recursive: true, force: true });
23
- };
24
-
25
- const createContext = (workingDir: string, env: Record<string, string> = {}): ToolContext => ({
26
- workingDir,
27
- env,
28
- confirm: async () => true,
29
- });
30
-
31
- /**
32
- * Complete list of all blocked environment variables.
33
- * Must stay in sync with BLOCKED_INPUT_ENV_VARS in shell.ts.
34
- */
35
- const ALL_BLOCKED_VARS = [
36
- // Original blocklist
37
- "LD_PRELOAD",
38
- "LD_LIBRARY_PATH",
39
- "DYLD_INSERT_LIBRARIES",
40
- "DYLD_FRAMEWORK_PATH",
41
- "DYLD_LIBRARY_PATH",
42
- "BASH_ENV",
43
- "ENV",
44
- "CDPATH",
45
- "GLOBIGNORE",
46
- "PROMPT_COMMAND",
47
- "SHELLOPTS",
48
- "BASHOPTS",
49
- // Git context hijacks
50
- "GIT_DIR",
51
- "GIT_WORK_TREE",
52
- "GIT_INDEX_FILE",
53
- "GIT_OBJECT_DIRECTORY",
54
- "GIT_ALTERNATE_OBJECT_DIRECTORIES",
55
- "GIT_COMMON_DIR",
56
- "GIT_PREFIX",
57
- "GIT_INTERNAL_SUPER_PREFIX",
58
- "GIT_CONFIG",
59
- "GIT_CONFIG_GLOBAL",
60
- "GIT_CONFIG_SYSTEM",
61
- "GIT_CONFIG_COUNT",
62
- "GIT_CEILING_DIRECTORIES",
63
- // Expanded blocklist (aria-mx5)
64
- "PATH",
65
- "NODE_OPTIONS",
66
- "NODE_PATH",
67
- "PYTHONPATH",
68
- "IFS",
69
- "HOME",
70
- "SHELL",
71
- "EDITOR",
72
- "VISUAL",
73
- // Java runtime hijacks (aria-skg)
74
- "JAVA_TOOL_OPTIONS",
75
- "_JAVA_OPTIONS",
76
- "CLASSPATH",
77
- // Ruby runtime hijacks (aria-skg)
78
- "RUBYOPT",
79
- "GEM_HOME",
80
- "GEM_PATH",
81
- // Perl runtime hijacks (aria-skg)
82
- "PERL5OPT",
83
- "PERL5LIB",
84
- // Python startup hijack (aria-skg)
85
- "PYTHONSTARTUP",
86
- ];
87
-
88
- describe("Shell env var blocklist", () => {
89
- let tempDir: string;
90
-
91
- beforeEach(async () => {
92
- tempDir = await createTempDir();
93
- });
94
-
95
- afterEach(async () => {
96
- await cleanupTempDir(tempDir);
97
- });
98
-
99
- describe("inputEnv filtering", () => {
100
- it("should strip LD_PRELOAD from inputEnv", async () => {
101
- const ctx = createContext(tempDir);
102
- const result = await executeBash(
103
- { command: "echo LD_PRELOAD=$LD_PRELOAD", env: { LD_PRELOAD: "/evil/lib.so" } },
104
- ctx,
105
- );
106
-
107
- expect(result.success).toBe(true);
108
- const data = result.data as { stdout: string };
109
- expect(data.stdout.trim()).toBe("LD_PRELOAD=");
110
- });
111
-
112
- it("should strip BASH_ENV from inputEnv", async () => {
113
- const ctx = createContext(tempDir);
114
- const result = await executeBash(
115
- { command: "echo BASH_ENV=$BASH_ENV", env: { BASH_ENV: "/evil/script.sh" } },
116
- ctx,
117
- );
118
-
119
- expect(result.success).toBe(true);
120
- const data = result.data as { stdout: string };
121
- expect(data.stdout.trim()).toBe("BASH_ENV=");
122
- });
123
-
124
- it("should strip indexed git config vars from inputEnv", async () => {
125
- const ctx = createContext(tempDir);
126
- const result = await executeBash(
127
- {
128
- command: "env | grep -Ec '^GIT_CONFIG_(KEY|VALUE)_0=' || true",
129
- env: {
130
- GIT_CONFIG_KEY_0: "core.bare",
131
- GIT_CONFIG_VALUE_0: "true",
132
- },
133
- },
134
- ctx,
135
- );
136
-
137
- expect(result.success).toBe(true);
138
- const data = result.data as { stdout: string };
139
- expect(data.stdout.trim()).toBe("0");
140
- });
141
-
142
- it("should strip DYLD_INSERT_LIBRARIES from inputEnv", async () => {
143
- const ctx = createContext(tempDir);
144
- const result = await executeBash(
145
- {
146
- command: "echo DYLD=$DYLD_INSERT_LIBRARIES",
147
- env: { DYLD_INSERT_LIBRARIES: "/evil/dylib.dylib" },
148
- },
149
- ctx,
150
- );
151
-
152
- expect(result.success).toBe(true);
153
- const data = result.data as { stdout: string };
154
- expect(data.stdout.trim()).toBe("DYLD=");
155
- });
156
-
157
- it("should strip GIT_DIR from inputEnv", async () => {
158
- const ctx = createContext(tempDir);
159
- const result = await executeBash(
160
- {
161
- command: "echo GIT_DIR=$GIT_DIR",
162
- env: { GIT_DIR: "/tmp/not-repo" },
163
- },
164
- ctx,
165
- );
166
-
167
- expect(result.success).toBe(true);
168
- const data = result.data as { stdout: string };
169
- expect(data.stdout.trim()).toBe("GIT_DIR=");
170
- });
171
-
172
- it("should strip GIT_CONFIG_KEY_* prefix vars from inputEnv", async () => {
173
- const ctx = createContext(tempDir);
174
- const result = await executeBash(
175
- {
176
- command: "env | grep -c '^GIT_CONFIG_KEY_0=' || true",
177
- env: {
178
- GIT_CONFIG_COUNT: "1",
179
- GIT_CONFIG_KEY_0: "user.name",
180
- GIT_CONFIG_VALUE_0: "poisoned",
181
- },
182
- },
183
- ctx,
184
- );
185
-
186
- expect(result.success).toBe(true);
187
- const data = result.data as { stdout: string };
188
- expect(data.stdout.trim()).toBe("0");
189
- });
190
-
191
- it("should strip NODE_OPTIONS from inputEnv", async () => {
192
- const ctx = createContext(tempDir);
193
- const result = await executeBash(
194
- {
195
- command: "echo NODE_OPTIONS=$NODE_OPTIONS",
196
- env: { NODE_OPTIONS: "--require /evil/payload.js" },
197
- },
198
- ctx,
199
- );
200
-
201
- expect(result.success).toBe(true);
202
- const data = result.data as { stdout: string };
203
- expect(data.stdout.trim()).toBe("NODE_OPTIONS=");
204
- });
205
-
206
- it("should strip NODE_PATH from inputEnv", async () => {
207
- const ctx = createContext(tempDir);
208
- // NODE_PATH may be set by vitest in process.env, so check that the evil value is not present
209
- const result = await executeBash(
210
- {
211
- command: "env | grep -c '^NODE_PATH=/evil/modules$' || true",
212
- env: { NODE_PATH: "/evil/modules" },
213
- },
214
- ctx,
215
- );
216
-
217
- expect(result.success).toBe(true);
218
- const data = result.data as { stdout: string };
219
- expect(data.stdout.trim()).toBe("0");
220
- });
221
-
222
- it("should strip PYTHONPATH from inputEnv", async () => {
223
- const ctx = createContext(tempDir);
224
- const result = await executeBash(
225
- { command: "echo PYTHONPATH=$PYTHONPATH", env: { PYTHONPATH: "/evil/python" } },
226
- ctx,
227
- );
228
-
229
- expect(result.success).toBe(true);
230
- const data = result.data as { stdout: string };
231
- expect(data.stdout.trim()).toBe("PYTHONPATH=");
232
- });
233
-
234
- it("should strip IFS from inputEnv", async () => {
235
- const ctx = createContext(tempDir);
236
- const result = await executeBash({ command: "echo IFS=$IFS", env: { IFS: "EVIL" } }, ctx);
237
-
238
- expect(result.success).toBe(true);
239
- const data = result.data as { stdout: string };
240
- // IFS is not set to "EVIL" — it should be whatever the shell default is
241
- expect(data.stdout.trim()).not.toContain("EVIL");
242
- });
243
-
244
- it("should strip EDITOR from inputEnv", async () => {
245
- const ctx = createContext(tempDir);
246
- const result = await executeBash(
247
- { command: "echo EDITOR=$EDITOR", env: { EDITOR: "/evil/editor" } },
248
- ctx,
249
- );
250
-
251
- expect(result.success).toBe(true);
252
- const data = result.data as { stdout: string };
253
- expect(data.stdout.trim()).not.toContain("/evil/editor");
254
- });
255
-
256
- it("should strip VISUAL from inputEnv", async () => {
257
- const ctx = createContext(tempDir);
258
- const result = await executeBash(
259
- { command: "echo VISUAL=$VISUAL", env: { VISUAL: "/evil/visual" } },
260
- ctx,
261
- );
262
-
263
- expect(result.success).toBe(true);
264
- const data = result.data as { stdout: string };
265
- expect(data.stdout.trim()).not.toContain("/evil/visual");
266
- });
267
-
268
- it("should handle case-insensitive blocking (lowercase)", async () => {
269
- const ctx = createContext(tempDir);
270
- const result = await executeBash(
271
- { command: "echo $ld_preload", env: { ld_preload: "/evil/lib.so" } },
272
- ctx,
273
- );
274
-
275
- expect(result.success).toBe(true);
276
- const data = result.data as { stdout: string };
277
- // Variable should be stripped; output should be empty
278
- expect(data.stdout.trim()).toBe("");
279
- });
280
-
281
- it("should handle case-insensitive blocking (mixed case)", async () => {
282
- const ctx = createContext(tempDir);
283
- const result = await executeBash(
284
- { command: "echo $Bash_Env", env: { Bash_Env: "/evil/script.sh" } },
285
- ctx,
286
- );
287
-
288
- expect(result.success).toBe(true);
289
- const data = result.data as { stdout: string };
290
- expect(data.stdout.trim()).toBe("");
291
- });
292
-
293
- it("should handle case-insensitive blocking for new vars (mixed case Node_Options)", async () => {
294
- const ctx = createContext(tempDir);
295
- const result = await executeBash(
296
- { command: "echo $Node_Options", env: { Node_Options: "--require /evil/payload.js" } },
297
- ctx,
298
- );
299
-
300
- expect(result.success).toBe(true);
301
- const data = result.data as { stdout: string };
302
- expect(data.stdout.trim()).toBe("");
303
- });
304
-
305
- it("should pass through non-blocked variables", async () => {
306
- const ctx = createContext(tempDir);
307
- const result = await executeBash(
308
- { command: "echo $SAFE_VAR", env: { SAFE_VAR: "safe_value" } },
309
- ctx,
310
- );
311
-
312
- expect(result.success).toBe(true);
313
- const data = result.data as { stdout: string };
314
- expect(data.stdout.trim()).toBe("safe_value");
315
- });
316
-
317
- it("should strip blocked vars while passing non-blocked vars simultaneously", async () => {
318
- const ctx = createContext(tempDir);
319
- const result = await executeBash(
320
- {
321
- command: 'echo "safe=$MY_SAFE ld=$LD_PRELOAD"',
322
- env: {
323
- MY_SAFE: "hello",
324
- LD_PRELOAD: "/evil/lib.so",
325
- },
326
- },
327
- ctx,
328
- );
329
-
330
- expect(result.success).toBe(true);
331
- const data = result.data as { stdout: string };
332
- expect(data.stdout.trim()).toBe("safe=hello ld=");
333
- });
334
-
335
- it("should strip all known blocked variables from inputEnv", async () => {
336
- const blockedVars: Record<string, string> = {};
337
- for (const varName of ALL_BLOCKED_VARS) {
338
- blockedVars[varName] = `evil_${varName.toLowerCase()}`;
339
- }
340
- // Add a canary that should survive
341
- blockedVars["CANARY"] = "canary_value";
342
-
343
- const ctx = createContext(tempDir);
344
- const result = await executeBash({ command: "echo $CANARY", env: blockedVars }, ctx);
345
-
346
- expect(result.success).toBe(true);
347
- const data = result.data as { stdout: string };
348
- // The non-blocked CANARY should still be available
349
- expect(data.stdout.trim()).toBe("canary_value");
350
- });
351
-
352
- it.each([
353
- ["GIT_DIR", "/evil/.git"],
354
- ["GIT_WORK_TREE", "/evil/worktree"],
355
- ["GIT_CONFIG_GLOBAL", "/evil/gitconfig"],
356
- ["PATH", "/evil/bin"],
357
- ["NODE_OPTIONS", "--require /evil/payload.js"],
358
- ["NODE_PATH", "/evil/node_modules"],
359
- ["PYTHONPATH", "/evil/python"],
360
- ["IFS", "EVIL"],
361
- ["HOME", "/evil/home"],
362
- ["SHELL", "/evil/shell"],
363
- ["EDITOR", "/evil/editor"],
364
- ["VISUAL", "/evil/visual"],
365
- // Java (aria-skg)
366
- ["JAVA_TOOL_OPTIONS", "-javaagent:/evil/agent.jar"],
367
- ["_JAVA_OPTIONS", "-Xbootclasspath:/evil/boot.jar"],
368
- ["CLASSPATH", "/evil/classes"],
369
- // Ruby (aria-skg)
370
- ["RUBYOPT", "-r/evil/payload"],
371
- ["GEM_HOME", "/evil/gems"],
372
- ["GEM_PATH", "/evil/gem_path"],
373
- // Perl (aria-skg)
374
- ["PERL5OPT", "-M/evil/module"],
375
- ["PERL5LIB", "/evil/perl_lib"],
376
- // Python (aria-skg)
377
- ["PYTHONSTARTUP", "/evil/startup.py"],
378
- ])("should strip %s from inputEnv (parametric)", async (varName, evilValue) => {
379
- const ctx = createContext(tempDir);
380
- // Use env command + grep to check if the variable is present in the child env
381
- const result = await executeBash(
382
- {
383
- command: `env | grep -c '^${varName}=${evilValue.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}' || true`,
384
- env: { [varName]: evilValue },
385
- },
386
- ctx,
387
- );
388
-
389
- expect(result.success).toBe(true);
390
- const data = result.data as { stdout: string };
391
- // grep -c should return 0 (no matches) — the evil value was not passed through
392
- expect(data.stdout.trim()).toBe("0");
393
- });
394
- });
395
-
396
- describe("ctx.env filtering (I1 fix)", () => {
397
- it("should strip LD_PRELOAD from ctx.env", async () => {
398
- const ctx = createContext(tempDir, { LD_PRELOAD: "/evil/lib.so" });
399
- const result = await executeBash({ command: "echo LD_PRELOAD=$LD_PRELOAD" }, ctx);
400
-
401
- expect(result.success).toBe(true);
402
- const data = result.data as { stdout: string };
403
- expect(data.stdout.trim()).toBe("LD_PRELOAD=");
404
- });
405
-
406
- it("should strip BASH_ENV from ctx.env", async () => {
407
- const ctx = createContext(tempDir, { BASH_ENV: "/evil/script.sh" });
408
- const result = await executeBash({ command: "echo BASH_ENV=$BASH_ENV" }, ctx);
409
-
410
- expect(result.success).toBe(true);
411
- const data = result.data as { stdout: string };
412
- expect(data.stdout.trim()).toBe("BASH_ENV=");
413
- });
414
-
415
- it("should strip GIT_WORK_TREE from ctx.env", async () => {
416
- const ctx = createContext(tempDir, { GIT_WORK_TREE: "/tmp/not-repo" });
417
- const result = await executeBash({ command: "echo GIT_WORK_TREE=$GIT_WORK_TREE" }, ctx);
418
-
419
- expect(result.success).toBe(true);
420
- const data = result.data as { stdout: string };
421
- expect(data.stdout.trim()).toBe("GIT_WORK_TREE=");
422
- });
423
-
424
- it("should strip indexed git config vars from ctx.env", async () => {
425
- const ctx = createContext(tempDir, {
426
- GIT_CONFIG_KEY_0: "core.bare",
427
- GIT_CONFIG_VALUE_0: "true",
428
- });
429
- const result = await executeBash(
430
- { command: "env | grep -Ec '^GIT_CONFIG_(KEY|VALUE)_0=' || true" },
431
- ctx,
432
- );
433
-
434
- expect(result.success).toBe(true);
435
- const data = result.data as { stdout: string };
436
- expect(data.stdout.trim()).toBe("0");
437
- });
438
-
439
- it("should strip DYLD_INSERT_LIBRARIES from ctx.env", async () => {
440
- const ctx = createContext(tempDir, { DYLD_INSERT_LIBRARIES: "/evil/dylib.dylib" });
441
- const result = await executeBash({ command: "echo DYLD=$DYLD_INSERT_LIBRARIES" }, ctx);
442
-
443
- expect(result.success).toBe(true);
444
- const data = result.data as { stdout: string };
445
- expect(data.stdout.trim()).toBe("DYLD=");
446
- });
447
-
448
- it("should strip NODE_OPTIONS from ctx.env", async () => {
449
- const ctx = createContext(tempDir, { NODE_OPTIONS: "--require /evil/payload.js" });
450
- const result = await executeBash({ command: "echo NODE_OPTIONS=$NODE_OPTIONS" }, ctx);
451
-
452
- expect(result.success).toBe(true);
453
- const data = result.data as { stdout: string };
454
- expect(data.stdout.trim()).toBe("NODE_OPTIONS=");
455
- });
456
-
457
- it("should strip NODE_PATH from ctx.env", async () => {
458
- const ctx = createContext(tempDir, { NODE_PATH: "/evil/modules" });
459
- // NODE_PATH may be set by vitest in process.env, so check that the evil value is not present
460
- const result = await executeBash(
461
- { command: "env | grep -c '^NODE_PATH=/evil/modules$' || true" },
462
- ctx,
463
- );
464
-
465
- expect(result.success).toBe(true);
466
- const data = result.data as { stdout: string };
467
- expect(data.stdout.trim()).toBe("0");
468
- });
469
-
470
- it("should strip PYTHONPATH from ctx.env", async () => {
471
- const ctx = createContext(tempDir, { PYTHONPATH: "/evil/python" });
472
- const result = await executeBash({ command: "echo PYTHONPATH=$PYTHONPATH" }, ctx);
473
-
474
- expect(result.success).toBe(true);
475
- const data = result.data as { stdout: string };
476
- expect(data.stdout.trim()).toBe("PYTHONPATH=");
477
- });
478
-
479
- it("should strip HOME from ctx.env", async () => {
480
- const ctx = createContext(tempDir, { HOME: "/evil/home" });
481
- const result = await executeBash(
482
- { command: "env | grep -c '^HOME=/evil/home' || true" },
483
- ctx,
484
- );
485
-
486
- expect(result.success).toBe(true);
487
- const data = result.data as { stdout: string };
488
- expect(data.stdout.trim()).toBe("0");
489
- });
490
-
491
- it("should strip SHELL from ctx.env", async () => {
492
- const ctx = createContext(tempDir, { SHELL: "/evil/shell" });
493
- const result = await executeBash(
494
- { command: "env | grep -c '^SHELL=/evil/shell' || true" },
495
- ctx,
496
- );
497
-
498
- expect(result.success).toBe(true);
499
- const data = result.data as { stdout: string };
500
- expect(data.stdout.trim()).toBe("0");
501
- });
502
-
503
- it("should handle case-insensitive blocking on ctx.env", async () => {
504
- const ctx = createContext(tempDir, { ld_preload: "/evil/lib.so" });
505
- const result = await executeBash({ command: "echo $ld_preload" }, ctx);
506
-
507
- expect(result.success).toBe(true);
508
- const data = result.data as { stdout: string };
509
- expect(data.stdout.trim()).toBe("");
510
- });
511
-
512
- it("should handle case-insensitive blocking on ctx.env for new vars", async () => {
513
- const ctx = createContext(tempDir, { node_options: "--require /evil/payload.js" });
514
- const result = await executeBash({ command: "echo $node_options" }, ctx);
515
-
516
- expect(result.success).toBe(true);
517
- const data = result.data as { stdout: string };
518
- expect(data.stdout.trim()).toBe("");
519
- });
520
-
521
- it("should pass through non-blocked ctx.env variables", async () => {
522
- const ctx = createContext(tempDir, {
523
- SAFE_CTX_VAR: "ctx_safe_value",
524
- LD_PRELOAD: "/evil/lib.so",
525
- });
526
- const result = await executeBash(
527
- { command: 'echo "safe=$SAFE_CTX_VAR ld=$LD_PRELOAD"' },
528
- ctx,
529
- );
530
-
531
- expect(result.success).toBe(true);
532
- const data = result.data as { stdout: string };
533
- expect(data.stdout.trim()).toBe("safe=ctx_safe_value ld=");
534
- });
535
-
536
- it.each([
537
- ["GIT_DIR", "/evil/.git"],
538
- ["GIT_WORK_TREE", "/evil/worktree"],
539
- ["GIT_CONFIG_GLOBAL", "/evil/gitconfig"],
540
- ["PATH", "/evil/bin"],
541
- ["NODE_OPTIONS", "--require /evil/payload.js"],
542
- ["NODE_PATH", "/evil/node_modules"],
543
- ["PYTHONPATH", "/evil/python"],
544
- ["IFS", "EVIL"],
545
- ["HOME", "/evil/home"],
546
- ["SHELL", "/evil/shell"],
547
- ["EDITOR", "/evil/editor"],
548
- ["VISUAL", "/evil/visual"],
549
- // Java (aria-skg)
550
- ["JAVA_TOOL_OPTIONS", "-javaagent:/evil/agent.jar"],
551
- ["_JAVA_OPTIONS", "-Xbootclasspath:/evil/boot.jar"],
552
- ["CLASSPATH", "/evil/classes"],
553
- // Ruby (aria-skg)
554
- ["RUBYOPT", "-r/evil/payload"],
555
- ["GEM_HOME", "/evil/gems"],
556
- ["GEM_PATH", "/evil/gem_path"],
557
- // Perl (aria-skg)
558
- ["PERL5OPT", "-M/evil/module"],
559
- ["PERL5LIB", "/evil/perl_lib"],
560
- // Python (aria-skg)
561
- ["PYTHONSTARTUP", "/evil/startup.py"],
562
- ])("should strip %s from ctx.env (parametric)", async (varName, evilValue) => {
563
- const ctx = createContext(tempDir, { [varName]: evilValue });
564
- const result = await executeBash(
565
- {
566
- command: `env | grep -c '^${varName}=${evilValue.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}' || true`,
567
- },
568
- ctx,
569
- );
570
-
571
- expect(result.success).toBe(true);
572
- const data = result.data as { stdout: string };
573
- expect(data.stdout.trim()).toBe("0");
574
- });
575
- });
576
-
577
- describe("end-to-end: blocked vars not visible in child process", () => {
578
- it("should not leak LD_PRELOAD to child process env via inputEnv", async () => {
579
- const ctx = createContext(tempDir);
580
- // Use env command to dump all environment, then grep for LD_PRELOAD
581
- const result = await executeBash(
582
- {
583
- command: "env | grep -c '^LD_PRELOAD=' || true",
584
- env: { LD_PRELOAD: "/evil/lib.so" },
585
- },
586
- ctx,
587
- );
588
-
589
- expect(result.success).toBe(true);
590
- const data = result.data as { stdout: string };
591
- // grep -c should return 0 (no matches)
592
- expect(data.stdout.trim()).toBe("0");
593
- });
594
-
595
- it("should not leak LD_PRELOAD to child process env via ctx.env", async () => {
596
- const ctx = createContext(tempDir, { LD_PRELOAD: "/evil/lib.so" });
597
- const result = await executeBash({ command: "env | grep -c '^LD_PRELOAD=' || true" }, ctx);
598
-
599
- expect(result.success).toBe(true);
600
- const data = result.data as { stdout: string };
601
- expect(data.stdout.trim()).toBe("0");
602
- });
603
-
604
- it("should not leak any expanded blocklist vars via inputEnv", async () => {
605
- const ctx = createContext(tempDir);
606
- const evilEnv: Record<string, string> = {
607
- NODE_OPTIONS: "--require /evil/payload.js",
608
- NODE_PATH: "/evil/modules",
609
- PYTHONPATH: "/evil/python",
610
- EDITOR: "/evil/editor",
611
- VISUAL: "/evil/visual",
612
- };
613
-
614
- const result = await executeBash(
615
- {
616
- command: ["env | grep -c '^NODE_OPTIONS=--require /evil' || true"].join("; "),
617
- env: evilEnv,
618
- },
619
- ctx,
620
- );
621
-
622
- expect(result.success).toBe(true);
623
- const data = result.data as { stdout: string };
624
- expect(data.stdout.trim()).toBe("0");
625
- });
626
-
627
- it("should not leak any expanded blocklist vars via ctx.env", async () => {
628
- const ctx = createContext(tempDir, {
629
- NODE_OPTIONS: "--require /evil/payload.js",
630
- NODE_PATH: "/evil/modules",
631
- PYTHONPATH: "/evil/python",
632
- EDITOR: "/evil/editor",
633
- VISUAL: "/evil/visual",
634
- });
635
-
636
- const result = await executeBash(
637
- {
638
- command: "env | grep -c '^NODE_OPTIONS=--require /evil' || true",
639
- },
640
- ctx,
641
- );
642
-
643
- expect(result.success).toBe(true);
644
- const data = result.data as { stdout: string };
645
- expect(data.stdout.trim()).toBe("0");
646
- });
647
- });
648
- });