fix: a bug where a reference to an missing variable in PowerShell would cause an exception (#2619)

baywet · ferrarimarco · web-flow · commit 0279fcf19c6e · 2026-02-13T17:32:35.000Z
* fix: a bug where a reference to an missing variable in PowerShell would cause an exception

Signed-off-by: Vincent Biret &lt;vincentbiret@hotmail.com&gt;

---------

Signed-off-by: Vincent Biret &lt;vincentbiret@hotmail.com&gt;
Co-authored-by: Marco Ferrari &lt;ferrari.marco@gmail.com&gt;
diff --git a/src/util/coerce-option.ts b/src/util/coerce-option.ts
@@ -17,7 +17,11 @@ import {readFileSync, statSync, Stats} from 'fs';
 // if an option looks like a file path, assume it's a
 // path to a key and load it.
 export function coerceOption(option: string): string {
-  if (option.match(/[\\/]/)) {
+  // Only check .match if option is a string, to avoid TypeError.
+  // This might happen if the user didn't pass any payload for
+  // the option, or if the payload is empty. Example:
+  // --token $EMPTY_VAR
+  if (typeof option === 'string' && option.match(/[\\/]/)) {
     try {
       const stat: Stats = statSync(option);
       if (stat.isDirectory()) return option;
diff --git a/test/util/coerce-option.ts b/test/util/coerce-option.ts
@@ -38,4 +38,11 @@ describe('coerceOption', () => {
     const coerced = coerceOption(resolve(fixturesPath, 'key.txt'));
     expect(coerced).to.equal('abc123');
   });
+  it('should not throw when a non-string value is passed', () => {
+    expect(() => coerceOption(undefined as unknown as string)).not.to.Throw();
+    expect(() => coerceOption(null as unknown as string)).not.to.Throw();
+    expect(() => coerceOption(123 as unknown as string)).not.to.Throw();
+    expect(() => coerceOption({} as unknown as string)).not.to.Throw();
+    expect(() => coerceOption([] as unknown as string)).not.to.Throw();
+  });
 });
