From 13c3d9ec789d0dcd6ebde232795cc2a84cb596a6 Mon Sep 17 00:00:00 2001
From: Ariel Caplan <amcaplan@users.noreply.github.com>
Date: Mon, 31 Oct 2022 18:35:51 +0200
Subject: [PATCH] Ignore unset environment variables (#536)

---
 src/parser/parse.ts       |  2 +-
 test/parser/parse.test.ts | 20 ++++++++++++++++++++
 2 files changed, 21 insertions(+), 1 deletion(-)

diff --git a/src/parser/parse.ts b/src/parser/parse.ts
index ffd8c7dce..58557880e 100644
--- a/src/parser/parse.ts
+++ b/src/parser/parse.ts
@@ -211,7 +211,7 @@ export class Parser<T extends ParserInput, TFlags extends OutputFlags<T['flags']
     for (const k of Object.keys(this.input.flags)) {
       const flag = this.input.flags[k]
       if (flags[k]) continue
-      if (flag.env) {
+      if (flag.env && Object.prototype.hasOwnProperty.call(process.env, flag.env)) {
         const input = process.env[flag.env]
         if (flag.type === 'option') {
           if (input) {
diff --git a/test/parser/parse.test.ts b/test/parser/parse.test.ts
index 1cd605877..54599390e 100644
--- a/test/parser/parse.test.ts
+++ b/test/parser/parse.test.ts
@@ -910,6 +910,26 @@ See more help with --help`)
           delete process.env.TEST_FOO
         })
       }
+
+      it('ignores unset environment variables', async () => {
+        delete process.env.TEST_FOO
+        const out = await parse([], {
+          flags: {
+            foo: flags.boolean({env: 'TEST_FOO'}),
+          },
+        })
+        expect(out.flags.foo).to.be.undefined
+      })
+
+      it('uses default when environment variable is unset', async () => {
+        delete process.env.TEST_FOO
+        const out = await parse([], {
+          flags: {
+            foo: flags.boolean({env: 'TEST_FOO', default: true}),
+          },
+        })
+        expect(out.flags.foo).to.be.true
+      })
     })
   })