2XX status code is always ignored

[maddrag0n]

Current Behavior

'2XX' status is always ignored

Expected Behavior

in absence of a more specific 200 range status code, the 2XX code should be used to type the response (see https://swagger.io/specification/v3/#responses-object)

Example

this leads to generated ApiResponse type to be typed to any

"responses": {
  "2XX": {
    "description": "…",
    "content": {
      "application/json": {
        "schema": {
          "$ref": "#/components/schemas/MyComponent"
        }
      }
    }
  }
}

[maddrag0n]

closing because unclear whether that was solved with 2.0.0 (which I cannot run because cli is not exported as export in package.json)

[markerikson]

@maddrag0n can you clarify the packaging comment?

[maddrag0n]

@markerikson we are calling the cli programmatically like this

const cliPath = require.resolve('@rtk-query/codegen-openapi/lib/bin/cli');

const runReduxCodegen = async (specUrl: string | undefined, configPath: string, outputPath: string): Promise<void> => (
  new Promise((resolve, reject) => {
    exec(
      `tsx ${cliPath} ${configPath}`,
      {
        env: {
          ...process.env,
          SPEC_URL: specUrl,
          STORE_ROOT_PATH: outputPath,
        },
      },
      (error, stdout, stderr) => {
        if (error) {
          console.error(`exec error: ${error}`);
          reject(new Error(`Execution failed with error: ${stderr || 'Unknown error'}`));
        } else {
          console.log(stdout);

          if (stderr) {
            reject(new Error(`Execution produced errors: ${stderr}`));
          } else {
            resolve();
          }
        }
      },
    );
  })
);

when upgrading from 1.2.0 to 2.0.0, the error message is that /lib/bin/cli is not in exports of the package

nodeLinker is pnp

[markerikson]

Yeah, we don't have that exposed here:

redux-toolkit/packages/rtk-query-codegen-openapi/package.json

Lines 10 to 22 in cb089e6

	"exports": {
	"./package.json": "./package.json",
	".": {
	"import": {
	"types": "./lib/index.d.mts",
	"default": "./lib/index.mjs"
	},
	"default": {
	"types": "./lib/index.d.ts",
	"default": "./lib/index.js"
	}
	}
	},

We didn't anticipate anyone trying to do what you're doing :)

Ought to be doable, though - just need to add it as an entry point in the exports section (ie "./cli").

Opened #4976 to track that, and I'll keep this issue open as well.

Not immediately sure when we'll have time to work on the codegen specifically, but it ought to be a fairly simple change if you'd like to file a PR!

[maddrag0n]

I was able to confirm 2.0.0 does not solve the 2XX incompatibility