这是indexloc提供的服务，不要输入任何密码
Skip to content

Fix path parameter serialization with flat params structure #2985

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
+42,525 −68,428
Draft

Fix path parameter serialization with flat params structure #2985

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2ed35d3
Initial plan
Copilot Nov 15, 2025
f040aef
Fix path parameter serialization in flat SDK params structure
Copilot Nov 15, 2025
fcc2d7f
WIP: Resolve merge conflicts with main branch (blocked by pre-existin…
Copilot Nov 19, 2025
f59dfb0
Successfully resolve merge conflicts with main branch
Copilot Nov 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
5 changes: 5 additions & 0 deletions .changeset/large-rings-lick.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@hey-api/openapi-ts': patch
---

**@tanstack/query**: prettier infinite query options
44 changes: 0 additions & 44 deletions .github/workflows/codeql.yml
View file
Open in desktop

This file was deleted.

87 changes: 47 additions & 40 deletions dev/openapi-ts.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,14 +40,16 @@ export default defineConfig(() => {
// 'circular.yaml',
// 'dutchie.json',
// 'invalid',
// 'full.yaml',
'full.yaml',
// 'object-property-names.yaml',
// 'openai.yaml',
// 'opencode.yaml',
// 'pagination-ref.yaml',
// 'sdk-instance.yaml',
// 'string-with-format.yaml',
// 'transformers.json',
// 'type-format.yaml',
'validators.yaml',
// 'validators.yaml',
// 'validators-circular-ref.json',
// 'validators-circular-ref-2.yaml',
// 'zoom-video-sdk.json',
Expand Down Expand Up @@ -85,26 +87,26 @@ export default defineConfig(() => {
},
// name: 'foo',
output: [
// {
// // case: 'snake_case',
// clean: true,
// fileName: {
// // case: 'snake_case',
// // name: '{{name}}.renamed',
// suffix: '.meh',
// },
// // format: 'prettier',
// importFileExtension: '.ts',
// // indexFile: false,
// // lint: 'eslint',
// path: path.resolve(__dirname, '.gen'),
// tsConfigPath: path.resolve(
// __dirname,
// 'tsconfig',
// 'tsconfig.nodenext.json',
// ),
// },
'.gen',
{
// case: 'snake_case',
// clean: true,
// fileName: {
// // case: 'snake_case',
// // name: '{{name}}.renamed',
// suffix: '.meh',
// },
// format: 'prettier',
// importFileExtension: '.ts',
// indexFile: false,
// lint: 'eslint',
path: path.resolve(__dirname, '.gen'),
tsConfigPath: path.resolve(
__dirname,
'tsconfig',
'tsconfig.nodenext.json',
),
},
// '.gen',
],
parser: {
filters: {
Expand Down Expand Up @@ -224,17 +226,17 @@ export default defineConfig(() => {
{
// case: 'snake_case',
// definitions: '你_snake_{{name}}',
// enums: {
// // case: 'PascalCase',
// // constantsIgnoreNull: true,
// // enabled: false,
// mode: 'javascript',
// },
enums: {
// case: 'PascalCase',
// constantsIgnoreNull: true,
// enabled: false,
// mode: 'typescript',
},
// errors: {
// error: '他們_error_{{name}}',
// name: '你們_errors_{{name}}',
// },
// name: '@hey-api/typescript',
name: '@hey-api/typescript',
// requests: '我們_data_{{name}}',
// responses: {
// name: '我_responses_{{name}}',
Expand All @@ -248,7 +250,7 @@ export default defineConfig(() => {
// },
},
{
asClass: true,
// asClass: true,
// auth: false,
// classNameBuilder: '{{name}}',
// classNameBuilder: '{{name}}Service',
Expand All @@ -259,7 +261,7 @@ export default defineConfig(() => {
// fields.unwrap('path')
// },
// include...
instance: true,
// instance: true,
name: '@hey-api/sdk',
// operationId: false,
// paramsStructure: 'flat',
Expand All @@ -268,7 +270,7 @@ export default defineConfig(() => {
// signature: 'client',
// signature: 'object',
// transformer: '@hey-api/transformers',
transformer: true,
// transformer: true,
validator: 'valibot',
// validator: {
// request: 'zod',
Expand Down Expand Up @@ -306,16 +308,21 @@ export default defineConfig(() => {
// infiniteQueryKeys: {
// name: '{{name}}IQK',
// },
// infiniteQueryOptions: {
// name: '{{name}}IQO',
// },
infiniteQueryOptions: {
meta() {
return {
custom: 'value',
};
},
// name: '{{name}}IQO',
},
mutationOptions: {
// name: '{{name}}MO',
meta() {
return {
custom: 'value',
};
},
// name: '{{name}}MO',
},
name: '@tanstack/react-query',
queryKeys: {
Expand Down Expand Up @@ -351,9 +358,9 @@ export default defineConfig(() => {
},
{
name: 'arktype',
// types: {
// infer: true,
// },
types: {
infer: true,
},
},
{
// case: 'SCREAMING_SNAKE_CASE',
Expand Down Expand Up @@ -425,7 +432,7 @@ export default defineConfig(() => {
{
// case: 'snake_case',
// comments: false,
compatibilityVersion: 'mini',
compatibilityVersion: 4,
dates: {
// local: true,
// offset: true,
Expand Down
2 changes: 1 addition & 1 deletion dev/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"devDependencies": {
"@hey-api/codegen-core": "workspace:*",
"@hey-api/openapi-ts": "workspace:*",
"@pinia/colada": "0.17.7",
"@pinia/colada": "0.17.8",
"@tanstack/angular-query-experimental": "5.73.3",
"@tanstack/react-query": "5.73.3",
"@tanstack/solid-query": "5.73.3",
Expand Down
66 changes: 10 additions & 56 deletions examples/openapi-ts-angular-common/src/client/sdk.gen.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,12 +94,7 @@ export const addPet = <ThrowOnError extends boolean = true>(
'data'
>({
responseStyle: 'data',
security: [
{
scheme: 'bearer',
type: 'http',
},
],
security: [{ scheme: 'bearer', type: 'http' }],
url: '/pet',
...options,
headers: {
Expand All @@ -123,12 +118,7 @@ export const updatePet = <ThrowOnError extends boolean = true>(
'data'
>({
responseStyle: 'data',
security: [
{
scheme: 'bearer',
type: 'http',
},
],
security: [{ scheme: 'bearer', type: 'http' }],
url: '/pet',
...options,
headers: {
Expand All @@ -152,12 +142,7 @@ export const findPetsByStatus = <ThrowOnError extends boolean = true>(
'data'
>({
responseStyle: 'data',
security: [
{
scheme: 'bearer',
type: 'http',
},
],
security: [{ scheme: 'bearer', type: 'http' }],
url: '/pet/findByStatus',
...options,
});
Expand All @@ -177,12 +162,7 @@ export const findPetsByTags = <ThrowOnError extends boolean = true>(
'data'
>({
responseStyle: 'data',
security: [
{
scheme: 'bearer',
type: 'http',
},
],
security: [{ scheme: 'bearer', type: 'http' }],
url: '/pet/findByTags',
...options,
});
Expand All @@ -202,12 +182,7 @@ export const deletePet = <ThrowOnError extends boolean = true>(
'data'
>({
responseStyle: 'data',
security: [
{
scheme: 'bearer',
type: 'http',
},
],
security: [{ scheme: 'bearer', type: 'http' }],
url: '/pet/{petId}',
...options,
});
Expand All @@ -228,14 +203,8 @@ export const getPetById = <ThrowOnError extends boolean = true>(
>({
responseStyle: 'data',
security: [
{
name: 'api_key',
type: 'apiKey',
},
{
scheme: 'bearer',
type: 'http',
},
{ name: 'api_key', type: 'apiKey' },
{ scheme: 'bearer', type: 'http' },
],
url: '/pet/{petId}',
...options,
Expand All @@ -256,12 +225,7 @@ export const updatePetWithForm = <ThrowOnError extends boolean = true>(
'data'
>({
responseStyle: 'data',
security: [
{
scheme: 'bearer',
type: 'http',
},
],
security: [{ scheme: 'bearer', type: 'http' }],
url: '/pet/{petId}',
...options,
});
Expand All @@ -282,12 +246,7 @@ export const uploadFile = <ThrowOnError extends boolean = true>(
>({
bodySerializer: null,
responseStyle: 'data',
security: [
{
scheme: 'bearer',
type: 'http',
},
],
security: [{ scheme: 'bearer', type: 'http' }],
url: '/pet/{petId}/uploadImage',
...options,
headers: {
Expand All @@ -311,12 +270,7 @@ export const getInventory = <ThrowOnError extends boolean = true>(
'data'
>({
responseStyle: 'data',
security: [
{
name: 'api_key',
type: 'apiKey',
},
],
security: [{ name: 'api_key', type: 'apiKey' }],
url: '/store/inventory',
...options,
});
Expand Down
Loading