@prismatic-io/spectral 9.0.5 → 9.0.7

package/dist/generators/componentManifest/cli.js

@@ -17,7 +17,6 @@ exports.runMain = void 0;
 /* eslint-disable @typescript-eslint/no-unsafe-member-access */
 const path_1 = __importDefault(require("path"));
 const fs_extra_1 = require("fs-extra");
-const lodash_1 = require("lodash");
 const index_1 = require("./index");
 const createFlagHelpText_1 = require("../utils/createFlagHelpText");
 const getFlagStringValue_1 = require("../utils/getFlagStringValue");
@@ -113,7 +112,7 @@ const runMain = (process) => __awaiter(void 0, void 0, void 0, function* () {
         component,
         dryRun: flags.dry_run.value,
         skipSignatureVerify: flags.skip_signature_verify.value,
-        packageName: (_a = flags.name.value) !== null && _a !== void 0 ? _a : `@component-manifests/${(0, lodash_1.kebabCase)(component.key)}`,
+        packageName: (_a = flags.name.value) !== null && _a !== void 0 ? _a : `@component-manifests/${component.key}`,
         dependencies: {
             spectral: packageJson.version,
             dependencies: packageJson.dependencies,

package/dist/generators/componentManifest/createActions.js

@@ -18,15 +18,18 @@ const getInputs_1 = require("./getInputs");
 const getImports_1 = require("./getImports");
 const helpers_1 = require("./helpers");
 const createTemplate_1 = require("../utils/createTemplate");
+const createTypeInterface_1 = require("../utils/createTypeInterface");
+const createImport_1 = require("../utils/createImport");
 const createActions = (_a) => __awaiter(void 0, [_a], void 0, function* ({ component, dryRun, verbose, sourceDir, destinationDir, }) {
     var _b, _c;
     if (verbose) {
         console.info("Creating actions...");
     }
     const actionIndex = yield renderActionsIndex({
-        actions: Object.entries((_b = component.actions) !== null && _b !== void 0 ? _b : {}).map(([actionKey, action]) => {
+        imports: Object.entries((_b = component.actions) !== null && _b !== void 0 ? _b : {}).map(([actionKey, action]) => {
+            var _a;
             return {
-                key: action.key || actionKey,
+                import: (0, createImport_1.createImport)((_a = action.key) !== null && _a !== void 0 ? _a : actionKey),
             };
         }),
         dryRun,
@@ -35,12 +38,15 @@ const createActions = (_a) => __awaiter(void 0, [_a], void 0, function* ({ compo
         destinationDir,
     });
     const actions = yield Promise.all(Object.entries((_c = component.actions) !== null && _c !== void 0 ? _c : {}).map((_a) => __awaiter(void 0, [_a], void 0, function* ([actionKey, action]) {
+        var _b, _c;
         const inputs = (0, getInputs_1.getInputs)({
             inputs: action.inputs,
         });
         const imports = (0, getImports_1.getImports)({ inputs });
         return yield renderAction({
             action: {
+                typeInterface: (0, createTypeInterface_1.createTypeInterface)((_b = action.key) !== null && _b !== void 0 ? _b : actionKey),
+                import: (0, createImport_1.createImport)((_c = action.key) !== null && _c !== void 0 ? _c : actionKey),
                 key: action.key || actionKey,
                 label: action.display.description,
                 description: action.display.description,
@@ -62,12 +68,12 @@ const createActions = (_a) => __awaiter(void 0, [_a], void 0, function* ({ compo
     });
 });
 exports.createActions = createActions;
-const renderActionsIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ actions, dryRun, verbose, sourceDir, destinationDir, }) {
+const renderActionsIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ imports, dryRun, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "actions", "index.ts.ejs"),
         destination: path_1.default.join(destinationDir, "actions", "index.ts"),
         data: {
-            actions,
+            imports,
         },
         dryRun,
         verbose,
@@ -76,7 +82,7 @@ const renderActionsIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({
 const renderAction = (_a) => __awaiter(void 0, [_a], void 0, function* ({ action, dryRun, imports, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "actions", "action.ts.ejs"),
-        destination: path_1.default.join(destinationDir, "actions", `${action.key}.ts`),
+        destination: path_1.default.join(destinationDir, "actions", `${action.import}.ts`),
         data: {
             action,
             helpers: helpers_1.helpers,

package/dist/generators/componentManifest/createConnections.js

@@ -18,15 +18,17 @@ const getInputs_1 = require("./getInputs");
 const getImports_1 = require("./getImports");
 const helpers_1 = require("./helpers");
 const createTemplate_1 = require("../utils/createTemplate");
+const createTypeInterface_1 = require("../utils/createTypeInterface");
+const createImport_1 = require("../utils/createImport");
 const createConnections = (_a) => __awaiter(void 0, [_a], void 0, function* ({ component, dryRun, verbose, sourceDir, destinationDir, }) {
     var _b, _c;
     if (verbose) {
         console.info("Creating connections...");
     }
     const connectionIndex = yield renderConnectionsIndex({
-        connections: ((_b = component.connections) !== null && _b !== void 0 ? _b : []).map((connection) => {
+        imports: ((_b = component.connections) !== null && _b !== void 0 ? _b : []).map((connection) => {
             return {
-                key: connection.key,
+                import: (0, createImport_1.createImport)(connection.key),
             };
         }),
         dryRun,
@@ -42,6 +44,8 @@ const createConnections = (_a) => __awaiter(void 0, [_a], void 0, function* ({ c
         const onPremAvailable = connection.inputs.some((input) => input.onPremControlled || input.onPremiseControlled);
         return yield renderConnection({
             connection: {
+                typeInterface: (0, createTypeInterface_1.createTypeInterface)(connection.key),
+                import: (0, createImport_1.createImport)(connection.key),
                 key: connection.key,
                 label: connection.label,
                 comments: connection.comments,
@@ -64,12 +68,12 @@ const createConnections = (_a) => __awaiter(void 0, [_a], void 0, function* ({ c
     });
 });
 exports.createConnections = createConnections;
-const renderConnectionsIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ connections, dryRun, verbose, sourceDir, destinationDir, }) {
+const renderConnectionsIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ imports, dryRun, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "connections", "index.ts.ejs"),
         destination: path_1.default.join(destinationDir, "connections", "index.ts"),
         data: {
-            connections,
+            imports,
         },
         dryRun,
         verbose,
@@ -78,7 +82,7 @@ const renderConnectionsIndex = (_a) => __awaiter(void 0, [_a], void 0, function*
 const renderConnection = (_a) => __awaiter(void 0, [_a], void 0, function* ({ connection, dryRun, imports, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "connections", "connection.ts.ejs"),
-        destination: path_1.default.join(destinationDir, "connections", `${connection.key}.ts`),
+        destination: path_1.default.join(destinationDir, "connections", `${connection.import}.ts`),
         data: {
             connection,
             helpers: helpers_1.helpers,

package/dist/generators/componentManifest/createDataSources.js

@@ -18,15 +18,17 @@ const getInputs_1 = require("./getInputs");
 const getImports_1 = require("./getImports");
 const helpers_1 = require("./helpers");
 const createTemplate_1 = require("../utils/createTemplate");
+const createTypeInterface_1 = require("../utils/createTypeInterface");
+const createImport_1 = require("../utils/createImport");
 const createDataSources = (_a) => __awaiter(void 0, [_a], void 0, function* ({ component, dryRun, verbose, sourceDir, destinationDir, }) {
     var _b, _c;
     if (verbose) {
         console.info("Creating data sources...");
     }
     const dataSourceIndex = yield renderDataSourcesIndex({
-        dataSources: Object.entries((_b = component.dataSources) !== null && _b !== void 0 ? _b : {}).map(([dataSourceKey, dataSource]) => {
+        imports: Object.entries((_b = component.dataSources) !== null && _b !== void 0 ? _b : {}).map(([dataSourceKey, dataSource]) => {
             return {
-                key: dataSource.key || dataSourceKey,
+                import: (0, createImport_1.createImport)(dataSource.key || dataSourceKey),
             };
         }),
         dryRun,
@@ -41,6 +43,8 @@ const createDataSources = (_a) => __awaiter(void 0, [_a], void 0, function* ({ c
         const imports = (0, getImports_1.getImports)({ inputs });
         return yield renderDataSource({
             dataSource: {
+                type: (0, createTypeInterface_1.createTypeInterface)(dataSource.key || dataSourceKey),
+                import: (0, createImport_1.createImport)(dataSource.key || dataSourceKey),
                 key: dataSource.key || dataSourceKey,
                 label: dataSource.display.label,
                 description: dataSource.display.description,
@@ -63,12 +67,12 @@ const createDataSources = (_a) => __awaiter(void 0, [_a], void 0, function* ({ c
     });
 });
 exports.createDataSources = createDataSources;
-const renderDataSourcesIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ dataSources, dryRun, verbose, sourceDir, destinationDir, }) {
+const renderDataSourcesIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ imports, dryRun, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "dataSources", "index.ts.ejs"),
         destination: path_1.default.join(destinationDir, "dataSources", "index.ts"),
         data: {
-            dataSources,
+            imports,
         },
         dryRun,
         verbose,
@@ -77,7 +81,7 @@ const renderDataSourcesIndex = (_a) => __awaiter(void 0, [_a], void 0, function*
 const renderDataSource = (_a) => __awaiter(void 0, [_a], void 0, function* ({ dataSource, dryRun, imports, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "dataSources", "dataSource.ts.ejs"),
-        destination: path_1.default.join(destinationDir, "dataSources", `${dataSource.key}.ts`),
+        destination: path_1.default.join(destinationDir, "dataSources", `${dataSource.import}.ts`),
         data: {
             dataSource,
             helpers: helpers_1.helpers,

package/dist/generators/componentManifest/createTriggers.js

@@ -18,15 +18,17 @@ const getInputs_1 = require("./getInputs");
 const getImports_1 = require("./getImports");
 const helpers_1 = require("./helpers");
 const createTemplate_1 = require("../utils/createTemplate");
+const createTypeInterface_1 = require("../utils/createTypeInterface");
+const createImport_1 = require("../utils/createImport");
 const createTriggers = (_a) => __awaiter(void 0, [_a], void 0, function* ({ component, dryRun, verbose, sourceDir, destinationDir, }) {
     var _b, _c;
     if (verbose) {
         console.info("Creating triggers...");
     }
     const triggersIndex = yield renderTriggersIndex({
-        triggers: Object.entries((_b = component.triggers) !== null && _b !== void 0 ? _b : {}).map(([triggerKey, trigger]) => {
+        imports: Object.entries((_b = component.triggers) !== null && _b !== void 0 ? _b : {}).map(([triggerKey, trigger]) => {
             return {
-                key: trigger.key || triggerKey,
+                import: (0, createImport_1.createImport)(trigger.key || triggerKey),
             };
         }),
         dryRun,
@@ -41,6 +43,8 @@ const createTriggers = (_a) => __awaiter(void 0, [_a], void 0, function* ({ comp
         const imports = (0, getImports_1.getImports)({ inputs });
         return yield renderTrigger({
             trigger: {
+                typeInterface: (0, createTypeInterface_1.createTypeInterface)(trigger.key || triggerKey),
+                import: (0, createImport_1.createImport)(trigger.key || triggerKey),
                 key: trigger.key || triggerKey,
                 label: trigger.display.description,
                 description: trigger.display.description,
@@ -62,12 +66,12 @@ const createTriggers = (_a) => __awaiter(void 0, [_a], void 0, function* ({ comp
     });
 });
 exports.createTriggers = createTriggers;
-const renderTriggersIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ triggers, dryRun, verbose, sourceDir, destinationDir, }) {
+const renderTriggersIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({ imports, dryRun, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "triggers", "index.ts.ejs"),
         destination: path_1.default.join(destinationDir, "triggers", "index.ts"),
         data: {
-            triggers,
+            imports,
         },
         dryRun,
         verbose,
@@ -76,7 +80,7 @@ const renderTriggersIndex = (_a) => __awaiter(void 0, [_a], void 0, function* ({
 const renderTrigger = (_a) => __awaiter(void 0, [_a], void 0, function* ({ dryRun, imports, trigger, verbose, sourceDir, destinationDir, }) {
     return yield (0, createTemplate_1.createTemplate)({
         source: path_1.default.join(sourceDir, "triggers", "trigger.ts.ejs"),
-        destination: path_1.default.join(destinationDir, "triggers", `${trigger.key}.ts`),
+        destination: path_1.default.join(destinationDir, "triggers", `${trigger.import}.ts`),
         data: {
             helpers: helpers_1.helpers,
             imports,

package/dist/generators/componentManifest/helpers.d.ts

@@ -2,4 +2,5 @@ export declare const helpers: {
     createDependencyImports: ({ imports }: import("../utils/createDependencyImports").CreateDependencyImportsProps) => string[];
     capitalizeFirstLetter: (value: string) => string;
     generatePackageJsonVersion: () => string;
+    formatType: (key: string) => string;
 };

package/dist/generators/componentManifest/helpers.js

@@ -3,9 +3,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.helpers = void 0;
 const capitalizeFirstLetter_1 = require("../utils/capitalizeFirstLetter");
 const createDependencyImports_1 = require("../utils/createDependencyImports");
+const formatType_1 = require("../utils/formatType");
 const generatePackageJsonVersion_1 = require("../utils/generatePackageJsonVersion");
 exports.helpers = {
     createDependencyImports: createDependencyImports_1.createDependencyImports,
     capitalizeFirstLetter: capitalizeFirstLetter_1.capitalizeFirstLetter,
     generatePackageJsonVersion: generatePackageJsonVersion_1.generatePackageJsonVersion,
+    formatType: formatType_1.formatType,
 };

package/dist/generators/componentManifest/templates/actions/action.ts.ejs

@@ -1,7 +1,7 @@
 <%- include('../partials/imports.ejs', { imports, helpers }) %>
 
-export interface <%= helpers.capitalizeFirstLetter(action.key) %>Values {
-<%- include('../partials/performArgs.ejs', { inputs: action.inputs }) -%>
+export interface <%= action.typeInterface %>Values {
+<%- include('../partials/performArgs.ejs', { inputs: action.inputs, helpers }) -%>
 }
 
 /**
@@ -9,11 +9,12 @@ export interface <%= helpers.capitalizeFirstLetter(action.key) %>Values {
  *
  * @description <%- action.description %>
  */
-export const <%= action.key %> = {
+export const <%= action.import %> = {
+  key: "<%= action.key %>",
   perform: <TReturn>(
-    _values: <%= helpers.capitalizeFirstLetter(action.key) %>Values
+    _values: <%= action.typeInterface %>Values
   ): Promise<TReturn> => Promise.resolve<TReturn>({} as TReturn),
   inputs: {
-    <%- include('../partials/inputs.ejs', { inputs: action.inputs }) -%>
+    <%- include('../partials/inputs.ejs', { inputs: action.inputs, helpers }) -%>
   }
 } as const;

package/dist/generators/componentManifest/templates/actions/index.ts.ejs

@@ -1 +1 @@
-<%- include('../partials/importBarrel.ejs', { imports: actions }) -%>
+<%- include('../partials/importBarrel.ejs', { imports }) -%>

package/dist/generators/componentManifest/templates/connections/connection.ts.ejs

@@ -1,7 +1,7 @@
 <%- include('../partials/imports.ejs', { imports, helpers }) %>
 
-export interface <%= helpers.capitalizeFirstLetter(connection.key) %>Values {
-<%- include('../partials/performArgs.ejs', { inputs: connection.inputs }) -%>
+export interface <%= connection.typeInterface %>Values {
+<%- include('../partials/performArgs.ejs', { inputs: connection.inputs, helpers }) -%>
 }
 
 /**
@@ -9,14 +9,15 @@ export interface <%= helpers.capitalizeFirstLetter(connection.key) %>Values {
  *
  * @comments <%- connection.comments %>
  */
-export const <%= connection.key %> = {
+export const <%= connection.import %> = {
+  key: "<%= connection.key %>",
   perform: (
-    _values: <%= helpers.capitalizeFirstLetter(connection.key) %>Values
+    _values: <%= connection.typeInterface %>Values
   ): Promise<void> => Promise.resolve(),
   <%_ if (connection.onPremAvailable) { -%>
   onPremAvailable: true,
   <%_ } -%>
   inputs: {
-    <%- include('../partials/inputs.ejs', { inputs: connection.inputs }) -%>
+    <%- include('../partials/inputs.ejs', { inputs: connection.inputs, helpers }) -%>
   }
 } as const;

package/dist/generators/componentManifest/templates/connections/index.ts.ejs

@@ -1 +1 @@
-<%- include('../partials/importBarrel.ejs', { imports: connections }) -%>
+<%- include('../partials/importBarrel.ejs', { imports }) -%>

package/dist/generators/componentManifest/templates/dataSources/dataSource.ts.ejs

@@ -1,7 +1,7 @@
 <%- include('../partials/imports.ejs', { imports, helpers }) %>
 
-export interface <%= helpers.capitalizeFirstLetter(dataSource.key) %>Values {
-<%- include('../partials/performArgs.ejs', { inputs: dataSource.inputs }) -%>
+export interface <%= dataSource.typeInterface %>Values {
+<%- include('../partials/performArgs.ejs', { inputs: dataSource.inputs, helpers }) -%>
 }
 
 /**
@@ -9,12 +9,13 @@ export interface <%= helpers.capitalizeFirstLetter(dataSource.key) %>Values {
  *
  * @description <%- dataSource.description %>
  */
-export const <%= dataSource.key %> = {
+export const <%= dataSource.import %> = {
+  key: "<%= dataSource.key %>",
   perform: (
-    _values: <%= helpers.capitalizeFirstLetter(dataSource.key) %>Values
+    _values: <%= dataSource.typeInterface %>Values
   ): Promise<void> => Promise.resolve(),
   dataSourceType: "<%= dataSource.dataSourceType %>",
   inputs: {
-    <%- include('../partials/inputs.ejs', { inputs: dataSource.inputs }) -%>
+    <%- include('../partials/inputs.ejs', { inputs: dataSource.inputs, helpers }) -%>
   }
 } as const;

package/dist/generators/componentManifest/templates/dataSources/index.ts.ejs

@@ -1 +1 @@
-<%- include('../partials/importBarrel.ejs', { imports: dataSources }) -%>
+<%- include('../partials/importBarrel.ejs', { imports }) -%>

package/dist/generators/componentManifest/templates/partials/importBarrel.ejs

@@ -1,9 +1,9 @@
 <% imports.forEach((item) => { -%>
-import { <%= item.key %> } from "./<%= item.key %>";
+import { <%= item.import %> } from "./<%= item.import %>";
 <% }); -%>
 
 export default {
   <%_ imports.forEach((item) => { -%>
-  <%= item.key %>,
+  <%= item.import %>,
   <%_ }); -%>
 }

package/dist/generators/componentManifest/templates/partials/inputs.ejs

@@ -1,5 +1,5 @@
 <% inputs.forEach((input) => { -%>
-  <%= input.key %>: {
+  <%- helpers.formatType(input.key) %>: {
     inputType: "<%= input.inputType %>",
     <%_ if (input.collection) { -%>
     collection: "<%= input.collection %>",

package/dist/generators/componentManifest/templates/partials/performArgs.ejs

@@ -1,4 +1,4 @@
 <% inputs.forEach((input) => { -%>
   <%- input.docBlock -%>
-  <%= input.key %><%= input.required ? "" : "?" %>: <%- input.valueType.type ? input.valueType.type : input.valueType %>;
+  <%- helpers.formatType(input.key) %><%= input.required ? "" : "?" %>: <%- input.valueType.type ? input.valueType.type : input.valueType %>;
 <% }); -%>

package/dist/generators/componentManifest/templates/triggers/index.ts.ejs

@@ -1 +1 @@
-<%- include('../partials/importBarrel.ejs', { imports: triggers }) -%>
+<%- include('../partials/importBarrel.ejs', { imports }) -%>

package/dist/generators/componentManifest/templates/triggers/trigger.ts.ejs

@@ -1,7 +1,7 @@
 <%- include('../partials/imports.ejs', { imports, helpers }) %>
 
-export interface <%= helpers.capitalizeFirstLetter(trigger.key) %>Values {
-<%- include('../partials/performArgs.ejs', { inputs: trigger.inputs }) -%>
+export interface <%= trigger.typeInterface %>Values {
+<%- include('../partials/performArgs.ejs', { inputs: trigger.inputs, helpers }) -%>
 }
 
 /**
@@ -9,11 +9,12 @@ export interface <%= helpers.capitalizeFirstLetter(trigger.key) %>Values {
  *
  * @description <%- trigger.description %>
  */
-export const <%= trigger.key %> = {
+export const <%= trigger.import %> = {
+  key: "<%= trigger.key %>",
   perform: <TReturn>(
-    _values: <%= helpers.capitalizeFirstLetter(trigger.key) %>Values
+    _values: <%= trigger.typeInterface %>Values
   ): Promise<TReturn> => Promise.resolve<TReturn>({} as TReturn),
   inputs: {
-    <%- include('../partials/inputs.ejs', { inputs: trigger.inputs }) -%>
+    <%- include('../partials/inputs.ejs', { inputs: trigger.inputs, helpers }) -%>
   }
 } as const;

package/dist/generators/utils/createImport.d.ts

@@ -0,0 +1 @@
+export declare const createImport: (key: string) => string;

package/dist/generators/utils/createImport.js

@@ -0,0 +1,6 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.createImport = void 0;
+const lodash_1 = require("lodash");
+const createImport = (key) => (0, lodash_1.camelCase)(key);
+exports.createImport = createImport;

package/dist/generators/utils/createTypeInterface.d.ts

@@ -0,0 +1 @@
+export declare const createTypeInterface: (key: string) => string;

package/dist/generators/utils/createTypeInterface.js

@@ -0,0 +1,7 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.createTypeInterface = void 0;
+const lodash_1 = require("lodash");
+const capitalizeFirstLetter_1 = require("./capitalizeFirstLetter");
+const createTypeInterface = (key) => (0, capitalizeFirstLetter_1.capitalizeFirstLetter)((0, lodash_1.camelCase)(key));
+exports.createTypeInterface = createTypeInterface;

package/dist/generators/utils/formatType.d.ts

@@ -0,0 +1 @@
+export declare const formatType: (key: string) => string;

package/dist/generators/utils/formatType.js

@@ -0,0 +1,5 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.formatType = void 0;
+const formatType = (key) => ((key === null || key === void 0 ? void 0 : key.match(/[^a-zA-Z0-9]/)) ? `"${key}"` : key);
+exports.formatType = formatType;

package/dist/serverTypes/convertIntegration.js

@@ -156,21 +156,25 @@ const convertConfigVarPermissionAndVisibility = ({ permissionAndVisibilityType,
     };
 };
 const convertComponentReference = (componentReference, componentRegistry, referenceType) => {
-    var _a, _b;
+    var _a, _b, _c;
     const manifest = componentRegistry[componentReference.component];
     if (!manifest) {
         throw new Error(`Component with key "${componentReference.component}" not found in component registry.`);
     }
+    const manifestEntry = manifest[referenceType][componentReference.key];
+    if (!manifestEntry) {
+        throw new Error(`Component with key "${componentReference.component}" does not have an entry with key "${componentReference.key}" in the component registry.`);
+    }
     const ref = {
         component: {
             key: manifest.key,
             signature: (_a = manifest.signature) !== null && _a !== void 0 ? _a : "",
             isPublic: manifest.public,
         },
-        key: componentReference.key,
+        // older versions of the manifest did not contain a key so we fall back to the componentReference key
+        key: (_b = manifestEntry.key) !== null && _b !== void 0 ? _b : componentReference.key,
     };
-    const manifestEntry = manifest[referenceType][componentReference.key];
-    const inputs = Object.entries((_b = componentReference.values) !== null && _b !== void 0 ? _b : {}).reduce((result, [key, value]) => {
+    const inputs = Object.entries((_c = componentReference.values) !== null && _c !== void 0 ? _c : {}).reduce((result, [key, value]) => {
         const manifestEntryInput = manifestEntry.inputs[key];
         const type = manifestEntryInput.collection
             ? "complex"
@@ -397,9 +401,10 @@ const convertOnExecution = (onExecution, componentRegistry) => (context, params)
     const invoke = _components.invoke;
     // Construct the component methods from the component registry
     const componentMethods = Object.entries(componentRegistry).reduce((accumulator, [registryComponentKey, { key: componentKey, actions, public: isPublic, signature }]) => {
-        const componentActions = Object.entries(actions).reduce((actionsAccumulator, [actionKey, action]) => {
+        const componentActions = Object.entries(actions).reduce((actionsAccumulator, [registryActionKey, action]) => {
             // Define the method to be called for the action
             const invokeAction = (values) => __awaiter(void 0, void 0, void 0, function* () {
+                var _a;
                 // Transform the input values based on the action's inputs
                 const transformedValues = Object.entries(values).reduce((transformedAccumulator, [inputKey, inputValue]) => {
                     const { collection } = action.inputs[inputKey];
@@ -412,10 +417,11 @@ const convertOnExecution = (onExecution, componentRegistry) => (context, params)
                         signature: signature !== null && signature !== void 0 ? signature : "",
                         isPublic,
                     },
-                    key: actionKey,
+                    // older versions of manifests did not contain action.key so we fall back to the registry key
+                    key: (_a = action.key) !== null && _a !== void 0 ? _a : registryActionKey,
                 }, context, transformedValues);
             });
-            return Object.assign(Object.assign({}, actionsAccumulator), { [actionKey]: invokeAction });
+            return Object.assign(Object.assign({}, actionsAccumulator), { [registryActionKey]: invokeAction });
         }, {});
         return Object.assign(Object.assign({}, accumulator), { [registryComponentKey]: componentActions });
     }, {});

package/dist/types/ComponentManifest.d.ts

@@ -15,19 +15,23 @@ interface BaseInput {
     default?: unknown;
 }
 export interface ComponentManifestAction {
+    key?: string;
     perform: (values: any) => Promise<unknown>;
     inputs: Record<string, BaseInput>;
 }
 export interface ComponentManifestTrigger {
+    key?: string;
     perform: (values: any) => Promise<unknown>;
     inputs: Record<string, BaseInput>;
 }
 export interface ComponentManifestDataSource {
+    key?: string;
     perform: (values: any) => Promise<unknown>;
     dataSourceType: DataSourceType;
     inputs: Record<string, BaseInput>;
 }
 export interface ComponentManifestConnection {
+    key?: string;
     perform: (values: any) => Promise<unknown>;
     onPremAvailable?: boolean;
     inputs: Record<string, BaseInput & {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prismatic-io/spectral",
-  "version": "9.0.5",
+  "version": "9.0.7",
   "description": "Utility library for building Prismatic components",
   "keywords": ["prismatic"],
   "main": "dist/index.js",