fix: make Netlify.env working by injecting it in the entry file (netlify/zip-it-and-ship-it#1618)

* fix: make Netlify.env working by injecting it in the entry file

Fixes https://github.com/netlify/pod-dev-foundations/issues/602

* chore: fix make it available in global context

* chore: add testcase for environment variables

Author: Lukas Holzer

packages/zip-it-and-ship-it/src/runtimes/node/utils/entry_file.ts

@@ -37,6 +37,7 @@ const getEntryFileContents = (
       // See https://esbuild.github.io/content-types/#default-interop.
       'const funcModule = typeof func.default === "function" ? func : func.default',
 
+      `global.Netlify = bootstrap.getNetlifyGlobal()`,
       `export const handler = bootstrap.getLambdaHandler(funcModule)`,
     ].join(';')
   }

packages/zip-it-and-ship-it/tests/fixtures-esm/v2-api-environment-variables/function.mjs

@@ -0,0 +1,14 @@
+export default async () => {
+  Netlify.env.set('bar', 'bar!')
+
+  const text = `Foo ${Netlify.env.get('bar')}`
+
+  Netlify.env.delete('bar')
+
+  return Response.json({
+    text,
+    foo: Netlify.env.has('foo'),
+    bar: Netlify.env.get('bar'),
+    env: Netlify.env.toObject(),
+  })
+}

packages/zip-it-and-ship-it/tests/v2api.test.ts

@@ -467,4 +467,37 @@ describe.runIf(semver.gte(nodeVersion, '18.13.0'))('V2 functions API', () => {
       expect(customErrorInfo.location.runtime).toBe('js')
     }
   })
+
+  testMany(
+    'Retrieves the process environment through the Netlify.env helper',
+    ['bundler_default', 'bundler_esbuild', 'bundler_esbuild_zisi', 'bundler_default_nft', 'bundler_nft'],
+    async (options) => {
+      const { files, tmpDir } = await zipFixture('v2-api-environment-variables', {
+        fixtureDir: FIXTURES_ESM_DIR,
+        opts: merge(options, {
+          archiveFormat: ARCHIVE_FORMAT.NONE,
+        }),
+      })
+
+      vi.stubEnv('foo', 'foo!')
+
+      const [{ name: archive, entryFilename }] = files
+      const func = await importFunctionFile(`${tmpDir}/${archive}/${entryFilename}`)
+      const { body: bodyStream, statusCode } = await invokeLambda(func)
+      const body = await readAsBuffer(bodyStream)
+
+      const parsed = JSON.parse(body)
+
+      expect(parsed).toMatchObject({
+        text: 'Foo bar!',
+        foo: true,
+        env: expect.objectContaining({
+          foo: 'foo!',
+        }),
+      })
+      // bar got set and deleted again
+      expect(parsed).not.toHaveProperty('bar')
+      expect(statusCode).toBe(200)
+    },
+  )
 })