@friggframework/core 2.0.0--canary.710.9045bbc.0 → 2.0.0--canary.711.f8b6445.0

package/CLAUDE.md

@@ -224,7 +224,7 @@ packages/core/
   (`remove-entities-from-integration.js`) refuses to leave a module with no
   entity (`LAST_ENTITY_OF_MODULE`, 409). Re-adding an attached entity or
   removing a detached one changes nothing; a detached entity is not deleted.
-  `integration-repository-entities-parity.db.test.js` runs every adapter against
+  `integration-repository-entities-parity.test.js` runs every adapter against
   real databases when `INTEGRATION_ENTITIES_PARITY_MONGO_URL` /
   `INTEGRATION_ENTITIES_PARITY_POSTGRES_URL` are set. The DocumentDB adapter
   writes only `Integration.entityIds` (as its `createIntegration` does), not
@@ -266,7 +266,7 @@ packages/core/
   mapping keys to leave out of the rows; never write such rows back. Path
   segments must match `^[A-Za-z_][A-Za-z0-9_]*$`. The PostgreSQL, MongoDB and
   DocumentDB adapters give the same pages and totals;
-  `integration-mapping-repository-query-parity.db.test.js` checks that against
+  `integration-mapping-repository-query-parity.test.js` checks that against
   real databases when `QUERY_MAPPINGS_PARITY_MONGO_URL` /
   `QUERY_MAPPINGS_PARITY_POSTGRES_URL` are set. Two orderings still differ:
   strings compare by the database collation on PostgreSQL and by code point

package/application/commands/README.md

@@ -67,10 +67,9 @@ class MyIntegration extends IntegrationBase {
     async hydrateFromExternalUser(externalUserId) {
         // Find integration context by external entity ID
         const result =
-            await this.commands.findIntegrationContextByExternalEntityId({
-                externalId: externalUserId,
-                type: 'my-integration', // integration config.type to match
-            });
+            await this.commands.findIntegrationContextByExternalEntityId(
+                externalUserId
+            );
 
         if (result.error) {
             return { error: result.error };
@@ -232,10 +231,9 @@ Manage integration records and load full integration contexts.
 ```javascript
 // Find integration context by external entity ID
 // Returns { context, error } where context includes record + hydrated modules
-const result = await commands.findIntegrationContextByExternalEntityId({
-    externalId: 'external-user-or-workspace-id',
-    type: 'my-integration', // integration config.type to match
-});
+const result = await commands.findIntegrationContextByExternalEntityId(
+    'external-user-or-workspace-id'
+);
 
 if (!result.error) {
     integration.setIntegrationRecord(result.context);
@@ -301,10 +299,7 @@ try {
 For integration context operations, errors are returned in the result:
 
 ```javascript
-const result = await commands.findIntegrationContextByExternalEntityId({
-    externalId: userId,
-    type: 'my-integration',
-});
+const result = await commands.findIntegrationContextByExternalEntityId(userId);
 
 if (result.error) {
     return { error: result.error };
@@ -354,10 +349,9 @@ const result =
 const { createFriggCommands } = require('@friggframework/core');
 
 this.commands = createFriggCommands({ integrationClass: MyIntegration });
-const result = await this.commands.findIntegrationContextByExternalEntityId({
-    externalId: userId,
-    type: 'my-integration',
-});
+const result = await this.commands.findIntegrationContextByExternalEntityId(
+    userId
+);
 ```
 
 ## Best Practices
@@ -439,7 +433,7 @@ if (!user) {
 |                 | `findEntitiesByIds(ids)`                                | Find multiple by IDs                      |
 |                 | `updateEntity(id, updates)`                             | Update entity properties                  |
 |                 | `deleteEntity(id)`                                      | Delete entity                             |
-| **Integration** | `findIntegrationContextByExternalEntityId({ externalId, type })` | Load integration + modules by external ID |
+| **Integration** | `findIntegrationContextByExternalEntityId(externalId)`  | Load integration + modules by external ID |
 |                 | `loadIntegrationContextById(integrationId)`             | Load integration + modules by ID          |
 
 ## Support

package/database/encryption/README.md

@@ -508,9 +508,9 @@ return entities.map((e) => ({
 
 **Verified:**
 
--   ✅ `postgres-relation-decryption.db.test.js` - Proves the bug exists
--   ✅ `postgres-decryption-fix-verification.db.test.js` - Verifies separate queries work
--   ✅ `mongo-decryption-fix-verification.db.test.js` - Verifies fix for MongoDB
+-   ✅ `postgres-relation-decryption.test.js` - Proves the bug exists
+-   ✅ `postgres-decryption-fix-verification.test.js` - Verifies separate queries work
+-   ✅ `mongo-decryption-fix-verification.test.js` - Verifies fix for MongoDB
 
 **Implementation Examples:**
 

package/database/encryption/encryption-schema-registry.js

@@ -174,46 +174,26 @@ function loadModuleEncryptionSchemas(integrations) {
         return;
     }
 
-    const { getModulesDefinitionFromIntegrationClasses } = require('../../integrations/utils/map-integration-dto');
+    const { getModulesDefinitionFromIntegrationClasses } = require('../integrations/utils/map-integration-dto');
 
-    // Integrations without modules contribute no credential fields.
-    const integrationsWithModules = integrations.filter(
-        (integration) => integration?.Definition?.modules
-    );
-    const moduleDefinitions = getModulesDefinitionFromIntegrationClasses(
-        integrationsWithModules
-    );
-
-    // Fields the core schema already encrypts (e.g. data.api_key) are
-    // covered; registering them again would fail validation and drop every
-    // other module field with them.
-    const coreCredentialFields = CORE_ENCRYPTION_SCHEMA.Credential.fields;
-    const credentialFields = extractCredentialFieldsFromModules(
-        moduleDefinitions
-    ).filter((field) => !coreCredentialFields.includes(field));
+    const moduleDefinitions = getModulesDefinitionFromIntegrationClasses(integrations);
+    const credentialFields = extractCredentialFieldsFromModules(moduleDefinitions);
 
     if (credentialFields.length === 0) {
         return;
     }
 
-    // Merge with whatever custom schema the app already registered rather
-    // than replacing it (registerCustomSchema overwrites).
-    const existingCredentialFields = customSchema.Credential?.fields || [];
-    const mergedSchema = {
-        ...customSchema,
+    const moduleSchema = {
         Credential: {
-            ...customSchema.Credential,
-            fields: [
-                ...new Set([...existingCredentialFields, ...credentialFields]),
-            ],
-        },
+            fields: credentialFields
+        }
     };
 
     logger.info(
         `Registering module-level encryption for ${credentialFields.length} credential fields`
     );
 
-    registerCustomSchema(mergedSchema);
+    registerCustomSchema(moduleSchema);
 }
 
 /**

package/jest-setup.js

@@ -1,13 +1,2 @@
-// Core's unit suites must not need a database, so this global setup only
-// sets the test environment. It deliberately does not start an in-memory
-// MongoDB (as @friggframework/test's globalSetup does): nothing in the default
-// suite talks to a live server, and a binary that failed to start used to stop
-// every core suite from running. Suites that need a real database are
-// `*.db.test.js` files, run by `npm run test:db` (see jest.db.config.js).
-const { overrideEnvironment } = require('@friggframework/test');
-
-module.exports = async function () {
-    if (!process.env.STAGE) {
-        overrideEnvironment({ STAGE: 'dev' });
-    }
-};
+const {globalSetup} = require('@friggframework/test');
+module.exports = globalSetup;

package/jest-teardown.js

@@ -1,5 +1,2 @@
-const { restoreEnvironment } = require('@friggframework/test');
-
-module.exports = async function () {
-    restoreEnvironment();
-};
+const {globalTeardown} = require('@friggframework/test');
+module.exports = globalTeardown;

package/jest.config.js

@@ -20,8 +20,4 @@ module.exports = {
     globalTeardown: './jest-teardown.js',
 
     setupFilesAfterEnv: ['./logs/jest-logger-setup.js'],
-
-    // `*.db.test.js` suites talk to a real MongoDB replica set / PostgreSQL
-    // and run separately via `npm run test:db` (jest.db.config.js).
-    testPathIgnorePatterns: ['/node_modules/', '\\.db\\.test\\.js$'],
 };

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.710.9045bbc.0",
+    "version": "2.0.0--canary.711.f8b6445.0",
     "dependencies": {
         "@aws-sdk/client-apigatewaymanagementapi": "^3.588.0",
         "@aws-sdk/client-kms": "^3.588.0",
@@ -48,9 +48,9 @@
         }
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0--canary.710.9045bbc.0",
-        "@friggframework/prettier-config": "2.0.0--canary.710.9045bbc.0",
-        "@friggframework/test": "2.0.0--canary.710.9045bbc.0",
+        "@friggframework/eslint-config": "2.0.0--canary.711.f8b6445.0",
+        "@friggframework/prettier-config": "2.0.0--canary.711.f8b6445.0",
+        "@friggframework/test": "2.0.0--canary.711.f8b6445.0",
         "@prisma/client": "^6.19.3",
         "@types/lodash": "4.17.15",
         "@typescript-eslint/eslint-plugin": "^8.0.0",
@@ -68,7 +68,6 @@
     "scripts": {
         "lint:fix": "prettier --write --loglevel error . && eslint . --fix",
         "test": "jest --passWithNoTests # TODO",
-        "test:db": "jest --config jest.db.config.js --runInBand",
         "prisma:generate:mongo": "npx prisma generate --schema ./prisma-mongodb/schema.prisma",
         "prisma:generate:postgres": "npx prisma generate --schema ./prisma-postgresql/schema.prisma",
         "prisma:generate": "npm run prisma:generate:mongo && npm run prisma:generate:postgres",
@@ -91,5 +90,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "9045bbc0a84c7c07271e8599d1600d58790537a6"
+    "gitHead": "f8b64453af51d5a060c85bacb037b6cc9fc8943f"
 }

package/queues/queuer-util.js

@@ -97,17 +97,16 @@ const QueuerUtil = {
                 Id: uuid(),
                 MessageBody: JSON.stringify(entry),
             });
-            // Sends 10, then starts a fresh buffer. Each command gets its own
-            // array: clearing the shared buffer in place would also empty the
-            // Entries of a command that was already sent.
+            // Sends 10, then purges the buffer
             if (buffer.length === batchSize) {
-                const batch = buffer.splice(0, buffer.length);
                 const command = new SendMessageBatchCommand({
-                    Entries: batch,
+                    Entries: buffer,
                     QueueUrl: queueUrl,
                 });
                 const result = await sqs.send(command);
-                inspectBatchResult(result, queueUrl, batch);
+                inspectBatchResult(result, queueUrl, buffer);
+                // Purge the buffer
+                buffer.splice(0, buffer.length);
             }
         }
 

package/jest.db.config.js

@@ -1,23 +0,0 @@
-/*
- * Database integration suites (`*.db.test.js`).
- *
- * These talk to real databases instead of mocking Prisma, so they are kept
- * out of the default `npm test` run and executed with `npm run test:db`.
- * They need generated Prisma clients (`npm run prisma:generate`) and:
- *   - a MongoDB replica set (Prisma requires one for MongoDB writes):
- *     FRIGG_TEST_MONGODB_URL, default
- *     mongodb://localhost:27017/frigg?replicaSet=rs0
- *   - PostgreSQL with the prisma-postgresql migrations applied:
- *     FRIGG_TEST_POSTGRES_URL, default
- *     postgresql://postgres:postgres@localhost:5432/frigg?schema=public
- * The repository parity suites skip themselves unless their own URLs are
- * set (INTEGRATION_ENTITIES_PARITY_* / QUERY_MAPPINGS_PARITY_*).
- */
-const base = require('./jest.config');
-
-module.exports = {
-    ...base,
-    coverageThreshold: undefined,
-    testMatch: ['**/*.db.test.js'],
-    testPathIgnorePatterns: ['/node_modules/'],
-};