@mettlecast/domain-cdk-packer 0.2.96 → 0.2.98

package/dist/DomainStack.js

@@ -217,7 +217,18 @@ export class DomainStack extends cdk.Stack {
         const createPrimitiveHandlers = (entries, primitiveType) => {
             const lambdas = [];
             const byId = new Map();
-            for (const group of splitByOutboundAccess(entries)) {
+            // Cross-domain invocation resolves one deterministic internal-action
+            // Lambda per domain. Splitting actions by outbound access would create
+            // multiple groups with the same physical name and leave some actions
+            // unreachable through ctx.actions. Use internet-capable networking when
+            // any action needs it so all internal actions stay in that one group.
+            const groups = primitiveType === 'action'
+                ? [{
+                        outboundAccess: entries.some(entry => normaliseOutboundAccess(entry.outboundAccess) === 'internet') ? 'internet' : 'internal',
+                        entries,
+                    }]
+                : splitByOutboundAccess(entries);
+            for (const group of groups) {
                 if (primitiveType === 'action') {
                     // Split API-exposed actions from internal actions so they can be
                     // grouped into a single Lambda each (instead of one Lambda per action).

package/dist/__tests__/cross-domain-wiring.test.js

@@ -52,6 +52,17 @@ const registry = {
             inputSchema: { type: 'object', properties: {}, required: [] },
             outputSchema: { type: 'object', properties: {}, required: [] },
         },
+        {
+            id: 'internet-helper',
+            kind: 'action',
+            handlerFile: 'src/handlers/internet-helper.ts',
+            backendAccess: 'domain',
+            exposure: { type: 'internal' },
+            idempotent: false,
+            outboundAccess: 'internet',
+            inputSchema: { type: 'object', properties: {}, required: [] },
+            outputSchema: { type: 'object', properties: {}, required: [] },
+        },
     ],
     integrations: [],
     events: [],
@@ -61,6 +72,7 @@ const registry = {
 beforeAll(() => {
     fs.mkdirSync(path.join(DOMAIN_ROOT, 'src', 'handlers'), { recursive: true });
     fs.writeFileSync(path.join(DOMAIN_ROOT, 'src', 'handlers', 'internal-helper.ts'), `export const internalHelper = { id: "internal-helper", backendAccess: "domain", exposure: { type: "internal" }, input: { parse: (x) => x }, output: { parse: (x) => x }, idempotent: false, handler: async () => ({ ok: true }) };\n`);
+    fs.writeFileSync(path.join(DOMAIN_ROOT, 'src', 'handlers', 'internet-helper.ts'), `export const internetHelper = { id: "internet-helper", backendAccess: "domain", exposure: { type: "internal" }, input: { parse: (x) => x }, output: { parse: (x) => x }, idempotent: false, handler: async () => ({ ok: true }) };\n`);
     fs.writeFileSync(path.join(DOMAIN_ROOT, 'src', 'handlers', 'on-thing-created.ts'), `export const onThingCreated = { id: "on-thing-created", event: "thing.created", semverRange: "^1", handler: async () => {} };\n`);
     fs.mkdirSync(path.join(DIST_ROOT, 'action'), { recursive: true });
     fs.mkdirSync(path.join(DIST_ROOT, 'subscriber'), { recursive: true });
@@ -70,7 +82,7 @@ afterAll(() => {
     fs.rmSync(path.join(process.cwd(), 'dist', 'domains', 'test-domain'), { recursive: true, force: true });
 });
 describe('DomainStack cross-domain action wiring (#5226)', () => {
-    it('gives the internal-action grouped Lambda a deterministic FunctionName', () => {
+    it('keeps mixed network-access internal actions in one deterministic Lambda', () => {
         const app = new cdk.App();
         const stack = new DomainStack(app, 'TestCrossDomainStack', {
             registry,
@@ -81,7 +93,9 @@ describe('DomainStack cross-domain action wiring (#5226)', () => {
         });
         const template = Template.fromStack(stack);
         const lambdas = template.findResources('AWS::Lambda::Function');
-        const actionLambda = Object.values(lambdas).find(l => l.Properties.FunctionName === 'test-dev-test-domain-action');
+        const actionLambdas = Object.values(lambdas).filter(l => l.Properties.FunctionName === 'test-dev-test-domain-action');
+        expect(actionLambdas).toHaveLength(1);
+        const [actionLambda] = actionLambdas;
         expect(actionLambda).toBeDefined();
     });
     it('injects TIB_ACTION_LAMBDA_ARNS into subscriber and action Lambdas', () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mettlecast/domain-cdk-packer",
-  "version": "0.2.96",
+  "version": "0.2.98",
   "type": "module",
   "publishConfig": {
     "registry": "https://registry.npmjs.org",