fix: pass workflow inputs to success and failure actions

.changeset/swift-otters-wander.md

@@ -0,0 +1,7 @@
+---
+"@redocly/openapi-core": minor
+"@redocly/respect-core": minor
+"@redocly/cli": minor
+---
+
+Added the `spec-parameters-in-by-context` Arazzo rule, which validates that a parameter's `in` field is specified when the parent step, success action, or failure action does not reference a `workflowId`. Extended success and failure action objects to accept a `parameters` property that maps to workflow inputs.

packages/core/src/config/__tests__/__snapshots__/config-resolvers.test.ts.snap

@@ -20,6 +20,7 @@ exports[`resolveConfig > should ignore minimal from the root and read local file
     "sourceDescription-name-unique": "error",
     "sourceDescription-type": "error",
     "sourceDescriptions-not-empty": "error",
+    "spec-parameters-in-by-context": "warn",
     "step-onFailure-unique": "warn",
     "step-onSuccess-unique": "warn",
     "stepId-unique": "error",
@@ -407,6 +408,7 @@ exports[`resolveConfig > should resolve extends with local file config which con
     "sourceDescription-name-unique": "error",
     "sourceDescription-type": "error",
     "sourceDescriptions-not-empty": "error",
+    "spec-parameters-in-by-context": "warn",
     "step-onFailure-unique": "warn",
     "step-onSuccess-unique": "warn",
     "stepId-unique": "error",

packages/core/src/config/__tests__/load.test.ts

@@ -142,6 +142,7 @@ describe('loadConfig', () => {
               "sourceDescription-name-unique": "off",
               "sourceDescription-type": "off",
               "sourceDescriptions-not-empty": "off",
+              "spec-parameters-in-by-context": "off",
               "step-onFailure-unique": "off",
               "step-onSuccess-unique": "off",
               "stepId-unique": "error",
@@ -466,6 +467,7 @@ describe('loadConfig', () => {
               "sourceDescription-name-unique": "error",
               "sourceDescription-type": "error",
               "sourceDescriptions-not-empty": "error",
+              "spec-parameters-in-by-context": "warn",
               "step-onFailure-unique": "warn",
               "step-onSuccess-unique": "warn",
               "stepId-unique": "error",
@@ -795,6 +797,7 @@ describe('loadConfig', () => {
               "sourceDescription-name-unique": "off",
               "sourceDescription-type": "off",
               "sourceDescriptions-not-empty": "off",
+              "spec-parameters-in-by-context": "off",
               "step-onFailure-unique": "off",
               "step-onSuccess-unique": "off",
               "stepId-unique": "error",
@@ -1208,6 +1211,7 @@ describe('loadConfig', () => {
               "sourceDescription-name-unique": "off",
               "sourceDescription-type": "off",
               "sourceDescriptions-not-empty": "off",
+              "spec-parameters-in-by-context": "off",
               "step-onFailure-unique": "off",
               "step-onSuccess-unique": "off",
               "stepId-unique": "error",
@@ -1532,6 +1536,7 @@ describe('loadConfig', () => {
               "sourceDescription-name-unique": "error",
               "sourceDescription-type": "error",
               "sourceDescriptions-not-empty": "error",
+              "spec-parameters-in-by-context": "warn",
               "step-onFailure-unique": "warn",
               "step-onSuccess-unique": "warn",
               "stepId-unique": "error",
@@ -1861,6 +1866,7 @@ describe('loadConfig', () => {
               "sourceDescription-name-unique": "off",
               "sourceDescription-type": "off",
               "sourceDescriptions-not-empty": "off",
+              "spec-parameters-in-by-context": "off",
               "step-onFailure-unique": "off",
               "step-onSuccess-unique": "off",
               "stepId-unique": "error",

packages/core/src/config/all.ts

@@ -294,6 +294,7 @@ const all: RawGovernanceConfig<'built-in'> = {
     'sourceDescription-name-unique': 'error',
     'sourceDescription-type': 'error',
     'sourceDescriptions-not-empty': 'error',
+    'spec-parameters-in-by-context': 'error',
     'step-onFailure-unique': 'error',
     'step-onSuccess-unique': 'error',
     'stepId-unique': 'error',

packages/core/src/config/minimal.ts

@@ -273,6 +273,7 @@ const minimal: RawGovernanceConfig<'built-in'> = {
     'sourceDescription-name-unique': 'off',
     'sourceDescription-type': 'off',
     'sourceDescriptions-not-empty': 'off',
+    'spec-parameters-in-by-context': 'off',
     'step-onFailure-unique': 'off',
     'step-onSuccess-unique': 'off',
     'stepId-unique': 'error',

packages/core/src/config/recommended-strict.ts

@@ -273,6 +273,7 @@ const recommendedStrict: RawGovernanceConfig<'built-in'> = {
     'sourceDescription-name-unique': 'error',
     'sourceDescription-type': 'error',
     'sourceDescriptions-not-empty': 'error',
+    'spec-parameters-in-by-context': 'error',
     'step-onFailure-unique': 'error',
     'step-onSuccess-unique': 'error',
     'stepId-unique': 'error',

packages/core/src/config/recommended.ts

@@ -273,6 +273,7 @@ const recommended: RawGovernanceConfig<'built-in'> = {
     'sourceDescription-name-unique': 'error',
     'sourceDescription-type': 'error',
     'sourceDescriptions-not-empty': 'error',
+    'spec-parameters-in-by-context': 'warn',
     'step-onFailure-unique': 'warn',
     'step-onSuccess-unique': 'warn',
     'stepId-unique': 'error',

packages/core/src/config/spec.ts

@@ -273,6 +273,7 @@ const spec: RawGovernanceConfig<'built-in'> = {
     'sourceDescription-name-unique': 'error',
     'sourceDescription-type': 'error',
     'sourceDescriptions-not-empty': 'error',
+    'spec-parameters-in-by-context': 'error',
     'step-onFailure-unique': 'error',
     'step-onSuccess-unique': 'error',
     'stepId-unique': 'error',