Filter undefined values from env format output (#235)

* Initial plan

* Fix undefined values in env format output

Co-authored-by: philmillman <3722211+philmillman@users.noreply.github.com>

* Remove undefined items from `dmno resolve --format env` output

* upgrade turbo cache action

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: philmillman <3722211+philmillman@users.noreply.github.com>
Co-authored-by: philmillman <phil@dmno.dev>

Author: Copilot

.changeset/tasty-paws-cover.md

@@ -0,0 +1,5 @@
+---
+"dmno": patch
+---
+
+Remove undefined items from `dmno resolve --format env` output

.github/workflows/release-preview.yaml

@@ -29,7 +29,7 @@ jobs:
       - name: Install node deps
         run: pnpm i
       - name: Enable turborepo build cache
-        uses: rharkor/caching-for-turbo@v1.5
+        uses: rharkor/caching-for-turbo@v2.3.2
       - name: Build publishable DMNO modules 
         run: pnpm build:libs
 

.github/workflows/release.yaml

@@ -25,7 +25,7 @@ jobs:
       - name: Install node deps
         run: pnpm i
       - name: Enable turborepo build cache
-        uses: rharkor/caching-for-turbo@v1.5
+        uses: rharkor/caching-for-turbo@v2.3.2
       - name: Create Release Pull Request or Publish to npm
         id: changesets
         uses: changesets/action@v1

packages/core/src/cli/lib/env-file-helpers.test.ts

@@ -18,4 +18,8 @@ describe('stringifyObjectAsEnvFile', () => {
     const result = stringifyObjectAsEnvFile({ foo: 'bar"baz' });
     expect(result).toEqual('foo="bar\\"baz"');
   });
+  test('excludes undefined values', () => {
+    const result = stringifyObjectAsEnvFile({ foo: 'bar', baz: undefined, qux: 'test' });
+    expect(result).toEqual('foo="bar"\nqux="test"');
+  });
 });

packages/core/src/cli/lib/env-file-helpers.ts

@@ -1,11 +1,13 @@
 export function stringifyObjectAsEnvFile(obj: Record<string, any>) {
-  return Object.entries(obj).map(([key, value]) => {
-    // Handle newlines and quotes by wrapping in double quotes and escaping
-    const formattedValue = String(value)
-      .replace(/\\/g, '\\\\') // escape backslashes first
-      .replace(/\n/g, '\\n') // escape newlines
-      .replace(/"/g, '\\"'); // escape double quotes
+  return Object.entries(obj)
+    .filter(([_key, value]) => value !== undefined) // Exclude undefined values
+    .map(([key, value]) => {
+      // Handle newlines and quotes by wrapping in double quotes and escaping
+      const formattedValue = String(value)
+        .replace(/\\/g, '\\\\') // escape backslashes first
+        .replace(/\n/g, '\\n') // escape newlines
+        .replace(/"/g, '\\"'); // escape double quotes
 
-    return `${key}="${formattedValue}"`;
-  }).join('\n');
+      return `${key}="${formattedValue}"`;
+    }).join('\n');
 }