feat: make operationParameter optional within endpoint parameters (#1888)

* Bump @api3/ois to v2.2.0 and zod to v3.22.2

* Bump oisFormat in configs and test files

* feat: make operationParameter optional within endpoint parameters

Author: dcroote

.changeset/hungry-clouds-grow.md

@@ -0,0 +1,9 @@
+---
+'@api3/airnode-validator': minor
+'@api3/airnode-deployer': minor
+'@api3/airnode-examples': minor
+'@api3/airnode-adapter': minor
+'@api3/airnode-node': minor
+---
+
+Bump OIS to v2.2.0 and make operationParameter optional within endpoint parameters

packages/airnode-adapter/package.json

@@ -19,7 +19,7 @@
     "test:watch": "yarn test:ts --watch"
   },
   "dependencies": {
-    "@api3/ois": "2.1.0",
+    "@api3/ois": "2.2.0",
     "@api3/promise-utils": "^0.4.0",
     "axios": "^1.5.0",
     "bignumber.js": "^9.1.2",

packages/airnode-adapter/src/request-building/parameters.test.ts

@@ -81,6 +81,33 @@ describe('fixed parameters', () => {
       headers: {},
     });
   });
+
+  it('ignores parameters without operationParameter', () => {
+    const ois = fixtures.buildOIS();
+    ois.apiSpecifications.paths['/convert'].get!.parameters.push({ in: 'query', name: 'noOperationParameter' });
+    const options = fixtures.buildCacheRequestOptions({
+      ois,
+      parameters: { f: 'ETH', amount: '1', no_op: 'myValue' },
+    });
+    options.endpoint.parameters.push({
+      name: 'no_op',
+      // operationParameter is absent
+    });
+    options.operation.parameters.push({ name: 'noOperationParameter', in: 'query' });
+    const res = parameters.buildParameters(options);
+    expect(res).toEqual({
+      paths: {},
+      query: {
+        // Expectedly absent:
+        // noOperationParameter: 'myValue',
+        access_key: 'super-secret-key',
+        amount: '1',
+        from: 'ETH',
+        to: 'USD',
+      },
+      headers: {},
+    });
+  });
 });
 
 describe('user parameters', () => {

packages/airnode-adapter/src/request-building/parameters.ts

@@ -3,7 +3,7 @@ import * as authentication from './authentication';
 import * as cookies from './cookies';
 import { BuilderParameters, CachedBuildRequestOptions, RequestParameters } from '../types';
 
-function initalParameters(): BuilderParameters {
+function initialParameters(): BuilderParameters {
   return {
     paths: {},
     query: {},
@@ -49,7 +49,7 @@ function buildFixedParameters(options: CachedBuildRequestOptions): BuilderParame
     }
 
     return appendParameter(acc, target, name, parameter.value);
-  }, initalParameters());
+  }, initialParameters());
 }
 
 function buildUserParameters(options: CachedBuildRequestOptions): BuilderParameters {
@@ -66,16 +66,19 @@ function buildUserParameters(options: CachedBuildRequestOptions): BuilderParamet
 
     // Double check that the parameter exists in the API specification
     const apiParameter = operation.parameters.find(
-      (p) => p.name === parameter.operationParameter.name && p.in === parameter.operationParameter.in
+      (p) =>
+        parameter.operationParameter &&
+        p.name === parameter.operationParameter.name &&
+        p.in === parameter.operationParameter.in
     );
-    if (!apiParameter) {
+    if (!apiParameter || !parameter.operationParameter) {
       return acc;
     }
 
     const { name, in: target } = parameter.operationParameter;
 
     return appendParameter(acc, target, name, parameters[key]);
-  }, initalParameters());
+  }, initialParameters());
 }
 
 export function buildParameters(options: CachedBuildRequestOptions): RequestParameters {

packages/airnode-adapter/test/fixtures/ois.ts

@@ -2,7 +2,7 @@ import { OIS } from '@api3/ois';
 
 export function buildOIS(overrides?: Partial<OIS>): OIS {
   return {
-    oisFormat: '2.1.0',
+    oisFormat: '2.2.0',
     version: '1.2.3',
     title: 'Currency Converter API',
     apiSpecifications: {

packages/airnode-deployer/config/config.example.json

@@ -94,7 +94,7 @@
   "templates": [],
   "ois": [
     {
-      "oisFormat": "2.1.0",
+      "oisFormat": "2.2.0",
       "title": "CoinGecko basic request",
       "version": "1.0.0",
       "apiSpecifications": {

packages/airnode-deployer/package.json

@@ -43,7 +43,7 @@
     "lodash": "^4.17.21",
     "ora": "^5.4.1",
     "yargs": "^17.7.2",
-    "zod": "^3.21.4"
+    "zod": "^3.22.2"
   },
   "devDependencies": {
     "@aws-sdk/util-stream-node": "^3.370.0",

packages/airnode-deployer/test/fixtures/config.aws.valid.json

@@ -95,7 +95,7 @@
   "templates": [],
   "ois": [
     {
-      "oisFormat": "2.1.0",
+      "oisFormat": "2.2.0",
       "title": "CoinGecko basic request",
       "version": "1.0.0",
       "apiSpecifications": {

packages/airnode-deployer/test/fixtures/config.gcp.valid.json

@@ -93,7 +93,7 @@
   "templates": [],
   "ois": [
     {
-      "oisFormat": "2.1.0",
+      "oisFormat": "2.2.0",
       "title": "CoinGecko basic request",
       "version": "1.0.0",
       "apiSpecifications": {

packages/airnode-examples/integrations/authenticated-coinmarketcap/config.example.json

@@ -83,7 +83,7 @@
   "templates": [],
   "ois": [
     {
-      "oisFormat": "2.1.0",
+      "oisFormat": "2.2.0",
       "title": "CoinMarketCap Basic Authenticated Request",
       "version": "1.0.0",
       "apiSpecifications": {