feat: Upgrade to latest openapi-fetch

[openint-bot]

Important

Upgrade openapi-fetch and openapi-typescript dependencies, update code for compatibility, and add integration tests for sdk-openint.

• Dependencies:
  • Upgrade openapi-fetch to ^0.13.3 in examples/package.json.
  • Upgrade openapi-typescript to ^7.4.4 in packages/cli/package.json and multiple SDKs' package.json files.
• Code Changes:
  • Use astToString in generateTypes() in utils.ts.
  • Modify createClient() in createClient.ts to use wrapAsPathBasedClient and handle HTTP methods with Uppercase<HttpMethod>.
• Tests:
  • Add tests for openapi-react-query and swr-openapi integration in index.spec.ts for sdk-openint.

^{This description was created by for 99edc1c. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to a185e73 in 2 minutes and 19 seconds

More details

• Looked at 8272 lines of code in 53 files
• Skipped 3 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. sdks/sdk-openint/src/index.spec.ts:50

• Draft comment:
  Consider adding assertions to verify the expected behavior of the component, such as checking the rendered output or the state of the component.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test case for 'swr-openapi integration' is missing an assertion to verify the expected behavior of the component. It currently only checks if the component is truthy, which doesn't confirm the integration works as intended.

Workflow ID: wflow_n266Gp1hUJrEnIri

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

👍 Looks good to me! Incremental review on 104d476 in 1 minute and 3 seconds

More details

• Looked at 46 lines of code in 3 files
• Skipped 1 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. examples/test.cjs:1

• Draft comment:
  The update from veniceSdkDef to openintSdkDef and venice to openint is consistent with the PR description and seems correct.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The code changes in test.cjs and test.mjs are consistent with the PR description, which mentions updating the SDK from 'venice' to 'openint'. The changes are correct and align with the intent of the PR.

2. examples/test.mjs:1

• Draft comment:
  The update from veniceSdkDef to openintSdkDef and venice to openint is consistent with the PR description and seems correct.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The code changes in test.cjs and test.mjs are consistent with the PR description, which mentions updating the SDK from 'venice' to 'openint'. The changes are correct and align with the intent of the PR.

Workflow ID: wflow_rJ5I9MGSr7cKLEfi

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

👍 Looks good to me! Incremental review on 43f79e8 in 51 seconds

More details

• Looked at 13 lines of code in 1 files
• Skipped 1 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. examples/package.json:12

• Draft comment:
  The upgrade to ^0.0.20 for @opensdks/runtime is appropriate for allowing non-breaking updates. Ensure that this version is compatible with the rest of the codebase.
• Reason this comment was not posted:
  Confidence changes required: 20%
  The upgrade of @opensdks/runtime to ^0.0.20 is consistent with the pattern of using caret (^) for versioning, which allows for non-breaking updates. This is a good practice for maintaining up-to-date dependencies without introducing breaking changes.

Workflow ID: wflow_is0mRyhFBK5Tpczi

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

👍 Looks good to me! Incremental review on 99edc1c in 56 seconds

More details

• Looked at 56 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. sdks/sdk-qbo/src/index.ts:83

• Draft comment:
  The type assertion as typeof client & typeof extension is unnecessary here since TypeScript can infer the type correctly after Object.assign. Consider removing it for cleaner code.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The use of Object.assign is correct, but the type assertion is unnecessary since the type is already inferred.

2. sdks/sdk-salesforce/src/index.ts:46

• Draft comment:
  The type assertion as typeof client & typeof extension is unnecessary here since TypeScript can infer the type correctly after Object.assign. Consider removing it for cleaner code.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The use of Object.assign is correct, but the type assertion is unnecessary since the type is already inferred.

Workflow ID: wflow_9RfmPmwlYGoV6Tf2

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.