@friggframework/core 2.0.0--canary.596.97a5c6b.0 → 2.0.0--canary.596.fc8739c.0

package/core/create-handler.js

@@ -80,10 +80,7 @@ const createHandler = (optionByName = {}) => {
             // If enabled (i.e. if SECRET_ARN is set in process.env) Fetch secrets from AWS Secrets Manager, and set them as environment variables.
             await secretsToEnv();
 
-            // Open the database connection up front when the handler needs it.
-            // Lazy-required so DB-free handlers (e.g. extension webhook
-            // receivers with useDatabase:false) never load the Prisma client.
-            // $connect is idempotent, so this safely reuses a warm connection.
+            // Lazy-required so DB-free handlers never load the Prisma client.
             if (shouldUseDatabase) {
                 const { connectPrisma } = require('../database/prisma');
                 await connectPrisma();

package/handlers/routers/health.js

@@ -511,10 +511,8 @@ router.get('/health/ready', async (_req, res) => {
     });
 });
 
-// shouldUseDatabase: false — health must NOT eagerly open a DB connection.
-// /health/live is a pure liveness probe (no DB), and /health/ready probes the
-// DB itself and degrades to 503 gracefully. Eager-connect at handler entry
-// would turn a DB outage into a 500 for both (killing healthy containers).
+// DB-free: /health/ready probes the DB itself and degrades to 503. Eager-connect
+// here would turn a DB outage into a 500, killing otherwise-healthy containers.
 const handler = createAppHandler('HTTP Event: Health', router, false);
 
 module.exports = { handler, router };

package/handlers/routers/integration-defined-routers.js

@@ -56,11 +56,8 @@ for (const IntegrationClass of integrationClasses) {
         }
     }
 
-    // Tier 3 Integration Extension routes — each binding gets a dedicated
-    // handler, namespaced under /{bindingName}, so multiple modules' extensions
-    // can declare the same relative path (e.g. two /webhooks) without colliding.
-    // Each per-binding handler carries its own shouldUseDatabase (resolved from
-    // the extension/binding `useDatabase`, default false → DB-free receiver).
+    // Each extension binding gets its own namespaced handler (/{bindingName}),
+    // so two modules' extensions can share a path like /webhooks without colliding.
     const bindingGroups = new Map();
     for (const extRoute of getExtensionRoutes(IntegrationClass)) {
         const namespacedPath = `/${extRoute.bindingName}${extRoute.path}`;
@@ -94,16 +91,12 @@ for (const IntegrationClass of integrationClasses) {
     };
 
     for (const [bindingName, group] of bindingGroups) {
-        // The function key is the wire contract with the devtools serverless
-        // generator (integration-builder.js builds the identical key). Keep
-        // the derivation here and there IN SYNC.
+        // Wire contract: integration-builder.js (devtools) derives the identical
+        // function key for the serverless config. Keep both in sync.
         const fnKey = `${IntegrationClass.Definition.name}__${sanitizeBindingKey(
             bindingName
         )}`;
-        // Two binding keys that sanitize to the same value (e.g. "hub-spot"
-        // and "hubspot") would silently overwrite each other's handler. The
-        // namespaced-path claim() above can't catch it (the paths differ), so
-        // fail loud here.
+        // Distinct binding keys can sanitize to the same fnKey — fail loud rather than overwrite.
         if (Object.prototype.hasOwnProperty.call(handlers, fnKey)) {
             throw new Error(
                 `Integration "${IntegrationClass.Definition.name}" extension handler conflict: ` +

package/integrations/extension.js

@@ -199,8 +199,6 @@ function getExtensionRoutes(IntegrationClass) {
             integrationName,
             binding
         );
-        // useDatabase is resolved at the binding level, falling back to the
-        // extension's declared value, then to false (DB-free by default).
         const useDatabase =
             binding.useDatabase ?? binding.extension.useDatabase ?? false;
         const routes = binding.extension.routes || [];

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.596.97a5c6b.0",
+    "version": "2.0.0--canary.596.fc8739c.0",
     "dependencies": {
         "@aws-sdk/client-apigatewaymanagementapi": "^3.588.0",
         "@aws-sdk/client-kms": "^3.588.0",
@@ -38,9 +38,9 @@
         }
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0--canary.596.97a5c6b.0",
-        "@friggframework/prettier-config": "2.0.0--canary.596.97a5c6b.0",
-        "@friggframework/test": "2.0.0--canary.596.97a5c6b.0",
+        "@friggframework/eslint-config": "2.0.0--canary.596.fc8739c.0",
+        "@friggframework/prettier-config": "2.0.0--canary.596.fc8739c.0",
+        "@friggframework/test": "2.0.0--canary.596.fc8739c.0",
         "@prisma/client": "^6.17.0",
         "@types/lodash": "4.17.15",
         "@typescript-eslint/eslint-plugin": "^8.0.0",
@@ -80,5 +80,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "97a5c6bc4217a0fcaaa6e6fa7baeb4b179f06d41"
+    "gitHead": "fc8739c1c49a9731b8465e0d5e864c71db20c581"
 }