Skip to content

feat: Basic OpenAPI 3.2 support - #6857

Draft
davidkna-sap wants to merge 10 commits into
mainfrom
feat/openapi-3.2-base
Draft

feat: Basic OpenAPI 3.2 support#6857
davidkna-sap wants to merge 10 commits into
mainfrom
feat/openapi-3.2-base

Conversation

@davidkna-sap

Copy link
Copy Markdown
Member

Closes SAP/cloud-sdk-backlog#ISSUENUMBER..

davidkna-sap and others added 9 commits July 31, 2026 16:58
…to 3.1.1

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
…codeAllParameters

The sentinel key __raw_query_string__ was being passed through
encodeAllParameters like any other param, percent-encoding both the key
and the raw query string value (a=1&b=2 → a%3D1%26b%3D2).

- Extract the sentinel as RAW_QUERY_STRING_KEY exported constant to
  eliminate the magic-string duplication between http-client and openapi.
- In encodeAllParameters, pass the sentinel entry through verbatim; all
  other entries are still encoded with encodeURIComponent as before.
- Replace the isolated-serializer-only test with three tests: the
  original serializer unit test, a new E2E test that spies on
  executeWithMiddleware and asserts the raw value survives encoding, and
  a regression guard confirming normal params still get encoded.

Co-Authored-By: Claude <noreply@anthropic.com>
@davidkna-sap
davidkna-sap marked this pull request as draft August 5, 2026 14:47
@davidkna-sap
davidkna-sap force-pushed the feat/openapi-3.2-base branch from 5a3cc8c to af4e7c5 Compare August 6, 2026 09:21
@davidkna-sap
davidkna-sap force-pushed the feat/openapi-3.2-base branch from af4e7c5 to 21552e0 Compare August 20, 2026 11:24
@davidkna-sap
davidkna-sap force-pushed the feat/openapi-3.2-base branch from 48b8734 to decd001 Compare August 20, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant