Merge branch 'main' into test/function-304

Author: pieh

packages/blobs/src/environment.ts

@@ -47,7 +47,7 @@ export const getEnvironment = (): EnvironmentVariables => {
 declare global {
   // Using `var` so that the declaration is hoisted in such a way that we can
   // reference it before it's initialized.
-  // eslint-disable-next-line no-var
+
   var netlifyBlobsContext: unknown
 }
 

packages/cache/src/fetchwithcache.ts

@@ -134,7 +134,11 @@ export const fetchWithCache: FetchWithCache = async (
   if (onCachePut) {
     await onCachePut(cachePut)
   } else {
-    const requestContext = (globalThis as GlobalScope).Netlify?.context
+    // NOTE: when `requestContext` is assigned via a single expression here, we
+    // hit some `@typescript-eslint/no-unsafe-assignment` bug. TODO(serhalp): try
+    // to reduce this down to a minimal repro and file an issue.
+    const netlifyGlobal: NetlifyGlobal | undefined = (globalThis as GlobalScope).Netlify
+    const requestContext = netlifyGlobal?.context
 
     if (requestContext) {
       requestContext.waitUntil(cachePut)

packages/edge-functions/src/main.ts

@@ -3,7 +3,7 @@ import type { NetlifyGlobal } from '@netlify/types'
 declare global {
   // Using `var` so that the declaration is hoisted in such a way that we can
   // reference it before it's initialized.
-  // eslint-disable-next-line no-var
+
   var Netlify: NetlifyGlobal
 }
 

packages/runtime/src/lib/globals.ts

@@ -7,7 +7,6 @@ declare global {
   // Using `var` so that the declaration is hoisted in such a way that we can
   // reference it before it's initialized.
 
-  // eslint-disable-next-line no-var
   var Netlify: NetlifyGlobal
 }