feat!: extract dev logic into new @netlify/functions-dev package (#475)

BREAKING CHANGE: The `/dev` export, introduced in version 3.1.0, has been removed. 

The local development functionality has been extracted into a new `@netlify/functions-dev` package. If your application relies on that logic, you should `npm install @netlify/functions-dev` and update your import:

```diff
-import { FunctionsHandler } from "@netlify/functions/dev"
+import { FunctionsHandler } from "@netlify/functions-dev"
```

No other changes are needed.

Author: eduardoboucas

.github/workflows/release-please.yaml

@@ -119,13 +119,24 @@ jobs:
         env:
           NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
       - if:
-          ${{ steps.release.outputs['packages/functions--release_created'] || github.event_name == 'workflow_dispatch'
-          }}
+          ${{ steps.release.outputs['packages/functions/prod--release_created'] || github.event_name ==
+          'workflow_dispatch' }}
+        run: |
+          if [[ "${{ github.event_name }}" == "workflow_dispatch" ]]; then
+            npm publish packages/functions/prod/ --provenance --access=public || true
+          else
+            npm publish packages/functions/prod/ --provenance --access=public
+          fi
+        env:
+          NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
+      - if:
+          ${{ steps.release.outputs['packages/functions/dev--release_created'] || github.event_name ==
+          'workflow_dispatch' }}
         run: |
           if [[ "${{ github.event_name }}" == "workflow_dispatch" ]]; then
-            npm publish packages/functions/ --provenance --access=public || true
+            npm publish packages/functions/dev/ --provenance --access=public || true
           else
-            npm publish packages/functions/ --provenance --access=public
+            npm publish packages/functions/dev/ --provenance --access=public
           fi
         env:
           NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}

.release-please-manifest.json

@@ -5,7 +5,8 @@
   "packages/dev": "4.6.0",
   "packages/dev-utils": "4.3.0",
   "packages/edge-functions": "2.19.0",
-  "packages/functions": "4.3.0",
+  "packages/functions/prod": "4.3.0",
+  "packages/functions/dev": "0.0.0",
   "packages/headers": "2.1.0",
   "packages/images": "1.3.0",
   "packages/nuxt-module": "0.2.0",

eslint_temporary_suppressions.js

@@ -363,13 +363,13 @@ export default [
     },
   },
   {
-    files: ['packages/functions/dev/events.ts'],
+    files: ['packages/functions/dev/src/events.ts'],
     rules: {
       '@typescript-eslint/no-empty-object-type': 'off',
     },
   },
   {
-    files: ['packages/functions/dev/function.ts'],
+    files: ['packages/functions/dev/src/function.ts'],
     rules: {
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-unsafe-assignment': 'off',
@@ -382,14 +382,14 @@ export default [
     },
   },
   {
-    files: ['packages/functions/dev/main.test.ts'],
+    files: ['packages/functions/dev/src/main.test.ts'],
     rules: {
       '@typescript-eslint/no-non-null-assertion': 'off',
       '@typescript-eslint/no-unnecessary-condition': 'off',
     },
   },
   {
-    files: ['packages/functions/dev/main.ts'],
+    files: ['packages/functions/dev/src/main.ts'],
     rules: {
       '@typescript-eslint/prefer-nullish-coalescing': 'off',
       '@typescript-eslint/no-misused-spread': 'off',
@@ -398,7 +398,7 @@ export default [
     },
   },
   {
-    files: ['packages/functions/dev/registry.ts'],
+    files: ['packages/functions/dev/src/registry.ts'],
     rules: {
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-unsafe-assignment': 'off',
@@ -412,14 +412,14 @@ export default [
     },
   },
   {
-    files: ['packages/functions/dev/runtimes/index.ts'],
+    files: ['packages/functions/dev/src/runtimes/index.ts'],
     rules: {
       '@typescript-eslint/no-unused-vars': 'off',
       '@typescript-eslint/no-explicit-any': 'off',
     },
   },
   {
-    files: ['packages/functions/dev/runtimes/nodejs/builder.ts'],
+    files: ['packages/functions/dev/src/runtimes/nodejs/builder.ts'],
     rules: {
       '@typescript-eslint/no-unused-vars': 'off',
       'n/no-missing-import': 'off',
@@ -431,13 +431,13 @@ export default [
     },
   },
   {
-    files: ['packages/functions/dev/runtimes/nodejs/config.ts'],
+    files: ['packages/functions/dev/src/runtimes/nodejs/config.ts'],
     rules: {
       '@typescript-eslint/prefer-reduce-type-parameter': 'off',
     },
   },
   {
-    files: ['packages/functions/dev/runtimes/nodejs/index.ts'],
+    files: ['packages/functions/dev/src/runtimes/nodejs/index.ts'],
     rules: {
       '@typescript-eslint/no-unsafe-member-access': 'off',
       '@typescript-eslint/no-unsafe-assignment': 'off',
@@ -447,40 +447,40 @@ export default [
     },
   },
   {
-    files: ['packages/functions/dev/runtimes/nodejs/lambda.ts'],
+    files: ['packages/functions/dev/src/runtimes/nodejs/lambda.ts'],
     rules: {
       '@typescript-eslint/no-unnecessary-condition': 'off',
       '@typescript-eslint/require-await': 'off',
     },
   },
   {
-    files: ['packages/functions/internal.d.ts'],
+    files: ['packages/functions/prod/internal.d.ts'],
     rules: {
       'n/no-missing-import': 'off',
     },
   },
   {
-    files: ['packages/functions/src/function/handler.ts'],
+    files: ['packages/functions/prod/src/function/handler.ts'],
     rules: {
       '@typescript-eslint/prefer-function-type': 'off',
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-invalid-void-type': 'off',
     },
   },
   {
-    files: ['packages/functions/src/function/handler_context.ts'],
+    files: ['packages/functions/prod/src/function/handler_context.ts'],
     rules: {
       '@typescript-eslint/no-explicit-any': 'off',
     },
   },
   {
-    files: ['packages/functions/src/function/handler_response.ts'],
+    files: ['packages/functions/prod/src/function/handler_response.ts'],
     rules: {
       '@typescript-eslint/no-explicit-any': 'off',
     },
   },
   {
-    files: ['packages/functions/src/lib/builder.test.ts'],
+    files: ['packages/functions/prod/src/lib/builder.test.ts'],
     rules: {
       'n/no-missing-import': 'off',
       '@typescript-eslint/no-unsafe-assignment': 'off',
@@ -490,51 +490,51 @@ export default [
     },
   },
   {
-    files: ['packages/functions/src/lib/builder.ts'],
+    files: ['packages/functions/prod/src/lib/builder.ts'],
     rules: {
       '@typescript-eslint/no-unnecessary-condition': 'off',
       '@typescript-eslint/prefer-nullish-coalescing': 'off',
       '@typescript-eslint/no-confusing-void-expression': 'off',
     },
   },
   {
-    files: ['packages/functions/src/lib/purge_cache.test.ts'],
+    files: ['packages/functions/prod/src/lib/purge_cache.test.ts'],
     rules: {
       'n/no-missing-import': 'off',
       '@typescript-eslint/no-unsafe-assignment': 'off',
       '@typescript-eslint/no-unsafe-member-access': 'off',
     },
   },
   {
-    files: ['packages/functions/src/lib/purge_cache.ts'],
+    files: ['packages/functions/prod/src/lib/purge_cache.ts'],
     rules: {
       '@typescript-eslint/no-unnecessary-condition': 'off',
       '@typescript-eslint/prefer-nullish-coalescing': 'off',
       '@typescript-eslint/restrict-template-expressions': 'off',
     },
   },
   {
-    files: ['packages/functions/src/lib/schedule.ts'],
+    files: ['packages/functions/prod/src/lib/schedule.ts'],
     rules: {
       '@typescript-eslint/no-unused-vars': 'off',
     },
   },
   {
-    files: ['packages/functions/src/lib/system_logger.test.ts'],
+    files: ['packages/functions/prod/src/lib/system_logger.test.ts'],
     rules: {
       '@typescript-eslint/no-unsafe-assignment': 'off',
       '@typescript-eslint/no-unsafe-member-access': 'off',
       '@typescript-eslint/no-unsafe-call': 'off',
     },
   },
   {
-    files: ['packages/functions/src/lib/system_logger.ts'],
+    files: ['packages/functions/prod/src/lib/system_logger.ts'],
     rules: {
       '@typescript-eslint/no-unnecessary-condition': 'off',
     },
   },
   {
-    files: ['packages/functions/test/types/Handler.test-d.ts'],
+    files: ['packages/functions/prod/test/types/Handler.test-d.ts'],
     rules: {
       '@typescript-eslint/no-unused-vars': 'off',
     },

package-lock.json

@@ -9,7 +9,8 @@
     "packages/blobs",
     "packages/cache",
     "packages/edge-functions",
-    "packages/functions",
+    "packages/functions/prod",
+    "packages/functions/dev",
     "packages/headers",
     "packages/images",
     "packages/redirects",

package.json

@@ -56,7 +56,7 @@
     "@netlify/config": "^23.2.0",
     "@netlify/dev-utils": "4.3.0",
     "@netlify/edge-functions": "2.19.0",
-    "@netlify/functions": "4.3.0",
+    "@netlify/functions-dev": "0.0.0",
     "@netlify/headers": "2.1.0",
     "@netlify/images": "1.3.0",
     "@netlify/redirects": "3.1.0",

packages/dev/package.json

@@ -14,7 +14,7 @@ import {
   HTTPServer,
 } from '@netlify/dev-utils'
 import { EdgeFunctionsHandler } from '@netlify/edge-functions/dev'
-import { FunctionsHandler } from '@netlify/functions/dev'
+import { FunctionsHandler } from '@netlify/functions-dev'
 import { HeadersHandler, type HeadersCollector } from '@netlify/headers'
 import { ImageHandler } from '@netlify/images'
 import { RedirectsHandler } from '@netlify/redirects'

packages/dev/src/main.ts

@@ -1,3 +1,2 @@
-dist
-dist-dev
-node_modules
+dist/
+node_modules/