@usearazzo/runner 1.0.1-alpha.25 → 1.0.1-alpha.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@usearazzo/runner",
-  "version": "1.0.1-alpha.25",
+  "version": "1.0.1-alpha.26",
   "description": "Runner for Arazzo Workflows.",
   "keywords": [
     "arazzo",
@@ -73,8 +73,8 @@
     "@swaggerexpert/json-pointer": "^3.0.2",
     "@swaggerexpert/jsonpath": "^4.0.5",
     "@swaggerexpert/media-types": "^1.0.2",
-    "@usearazzo/parser": "1.0.1-alpha.25",
-    "@usearazzo/resolver": "1.0.1-alpha.25",
+    "@usearazzo/parser": "1.0.1-alpha.26",
+    "@usearazzo/resolver": "1.0.1-alpha.26",
     "@xmldom/xmldom": "^0.9.12",
     "fontoxpath": "^3.34.0",
     "ramda": "^0.32.0",
@@ -94,5 +94,5 @@
   "devDependencies": {
     "swagger-client": "^3.38.4"
   },
-  "gitHead": "37de6e45a52dabf8a924f09fed3ae4d41396be7d"
+  "gitHead": "8bacf6bf0e5d1e4c44a81583c5f1719bbbefaf97"
 }

package/src/document/ArazzoDocument.cjs

@@ -56,8 +56,9 @@ class ArazzoDocument extends _APIDocument.default {
 
   /**
    * The workflows of this document that can be addressed by `workflowId`, in
-   * document order. Reads this document only: source descriptions are neither
-   * loaded nor listed.
+   * document order. A duplicated `workflowId` is listed once, for its first
+   * workflow. Reads this document only: source descriptions are neither loaded
+   * nor listed.
    */
   listWorkflows() {
     return [...this.workflowIndex].map(([workflowId, jsonPointer]) => {

package/src/document/ArazzoDocument.mjs

@@ -48,8 +48,9 @@ class ArazzoDocument extends APIDocument {
 
   /**
    * The workflows of this document that can be addressed by `workflowId`, in
-   * document order. Reads this document only: source descriptions are neither
-   * loaded nor listed.
+   * document order. A duplicated `workflowId` is listed once, for its first
+   * workflow. Reads this document only: source descriptions are neither loaded
+   * nor listed.
    */
   listWorkflows() {
     return [...this.workflowIndex].map(([workflowId, jsonPointer]) => {

package/src/registry/providers/ArazzoDocumentRegistryProvider.cjs

@@ -99,7 +99,8 @@ class ArazzoDocumentRegistryProvider extends _DocumentRegistryProvider.default {
    * A source description's canonical URI is resolved here too — against the
    * document's base URI, into the exact form the document registry keys on —
    * so every consumer works with a ready-to-use, non-relative URI. The first
-   * element of a duplicated source description name wins.
+   * element of a duplicated source description name wins, and so does the
+   * first workflow of a duplicated workflowId.
    */
   #buildIndexes(uri, parseResult) {
     const workflowIndex = new _ArazzoWorkflowIndex.default();
@@ -111,6 +112,7 @@ class ArazzoDocumentRegistryProvider extends _DocumentRegistryProvider.default {
         const workflowId = (0, _apidomCore.toValue)(workflow.workflowId);
         if (typeof workflowId !== 'string') return path.skip();
         if (workflowId === '') return path.skip();
+        if (workflowIndex.has(workflowId)) return path.skip();
         const workflowPointer = path.formatPath('jsonpointer');
         workflowIndex.set(workflowId, workflowPointer);
 

package/src/registry/providers/ArazzoDocumentRegistryProvider.mjs

@@ -91,7 +91,8 @@ class ArazzoDocumentRegistryProvider extends DocumentRegistryProvider {
    * A source description's canonical URI is resolved here too — against the
    * document's base URI, into the exact form the document registry keys on —
    * so every consumer works with a ready-to-use, non-relative URI. The first
-   * element of a duplicated source description name wins.
+   * element of a duplicated source description name wins, and so does the
+   * first workflow of a duplicated workflowId.
    */
   #buildIndexes(uri, parseResult) {
     const workflowIndex = new ArazzoWorkflowIndex();
@@ -103,6 +104,7 @@ class ArazzoDocumentRegistryProvider extends DocumentRegistryProvider {
         const workflowId = toValue(workflow.workflowId);
         if (typeof workflowId !== 'string') return path.skip();
         if (workflowId === '') return path.skip();
+        if (workflowIndex.has(workflowId)) return path.skip();
         const workflowPointer = path.formatPath('jsonpointer');
         workflowIndex.set(workflowId, workflowPointer);
 

package/src/registry/providers/OpenAPIDocumentRegistryProvider.cjs

@@ -44,6 +44,7 @@ const providerOptionsOverride = exports.providerOptionsOverride = {
  * Provides OpenAPIDocument instances for the DocumentRegistry.
  *
  * Parses, dereferences, and builds an OperationIndex for OpenAPI documents.
+ * The first operation of a duplicated operationId wins.
  * @public
  */
 class OpenAPIDocumentRegistryProvider extends _DocumentRegistryProvider.default {
@@ -95,6 +96,7 @@ class OpenAPIDocumentRegistryProvider extends _DocumentRegistryProvider.default
         const operationId = (0, _apidomCore.toValue)(operation.operationId);
         if (typeof operationId !== 'string') return path.skip();
         if (operationId === '') return path.skip();
+        if (index.has(operationId)) return path.skip();
         index.set(operationId, path.formatPath('jsonpointer'));
         path.skip();
       }

package/src/registry/providers/OpenAPIDocumentRegistryProvider.mjs

@@ -36,6 +36,7 @@ export const providerOptionsOverride = {
  * Provides OpenAPIDocument instances for the DocumentRegistry.
  *
  * Parses, dereferences, and builds an OperationIndex for OpenAPI documents.
+ * The first operation of a duplicated operationId wins.
  * @public
  */
 class OpenAPIDocumentRegistryProvider extends DocumentRegistryProvider {
@@ -87,6 +88,7 @@ class OpenAPIDocumentRegistryProvider extends DocumentRegistryProvider {
         const operationId = toValue(operation.operationId);
         if (typeof operationId !== 'string') return path.skip();
         if (operationId === '') return path.skip();
+        if (index.has(operationId)) return path.skip();
         index.set(operationId, path.formatPath('jsonpointer'));
         path.skip();
       }

package/types/runner.d.ts

@@ -125,8 +125,9 @@ export declare class ArazzoDocument extends APIDocument {
     static is(doc: APIDocument): doc is ArazzoDocument;
     /**
      * The workflows of this document that can be addressed by `workflowId`, in
-     * document order. Reads this document only: source descriptions are neither
-     * loaded nor listed.
+     * document order. A duplicated `workflowId` is listed once, for its first
+     * workflow. Reads this document only: source descriptions are neither loaded
+     * nor listed.
      */
     listWorkflows(): WorkflowDescriptor[];
     /**
@@ -1085,6 +1086,7 @@ export declare interface OpenAPIDocumentAssemblerOptions {
  * Provides OpenAPIDocument instances for the DocumentRegistry.
  *
  * Parses, dereferences, and builds an OperationIndex for OpenAPI documents.
+ * The first operation of a duplicated operationId wins.
  * @public
  */
 export declare class OpenAPIDocumentRegistryProvider extends DocumentRegistryProvider {