Skip to content

Conversation

@jonaslagoni
Copy link
Contributor

@jonaslagoni jonaslagoni commented Jan 22, 2026

Note

OpenAPI HTTP client generation

  • Add OpenAPI support to TypeScript channels (http_client) via new openapi.ts and wiring in channels/index.ts to generate per-operation fetch clients
  • Update HTTP fetch protocol: switch response parsing to unmarshal(...), emit common types once, and enhance integration with parameters (path building)
  • Extend OpenAPI parameters: add getChannelWithParameters and refine serializeUrl output

Tooling, tests, docs

  • Fix YAML generator indentation in MCP config tools
  • Normalize bundled docs keys and add AI Assistants Q&A; minor docs content updates
  • Add OpenAPI channels tests, snapshots, and runtime example including channels (http_client)

Written by Cursor Bugbot for commit ea148b7. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Jan 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
the-codegen-project Ready Ready Preview, Comment Jan 22, 2026 8:51pm
the-codegen-project-mcp Ready Ready Preview, Comment Jan 22, 2026 8:51pm

@netlify
Copy link

netlify bot commented Jan 22, 2026

Deploy Preview for the-codegen-project ready!

Name Link
🔨 Latest commit ea148b7
🔍 Latest deploy log https://app.netlify.com/projects/the-codegen-project/deploys/69728d8f19f9530008fe0554
😎 Deploy Preview https://deploy-preview-311--the-codegen-project.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

// Generate response parsing
const responseParseCode = replyMessageModule
? `const responseData = ${replyMessageModule}.unmarshalByStatusCode(rawData, response.status);`
? `const responseData = ${replyMessageModule}.unmarshal(rawData);`
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Invalid TypeScript generated for array response types

High Severity

When an OpenAPI operation returns an array type (like Pet[]), the generated code produces invalid TypeScript: Pet[].unmarshal(rawData). This is syntactically invalid because you cannot call methods on type literals. The snapshot at line 1330 confirms this generates non-compiling code. This affects any OpenAPI GET operation returning arrays.

Additional Locations (1)

Fix in Cursor Fix in Web

channelParameters: parameterModel?.model as
| ConstrainedObjectModel
| undefined
});
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OpenAPI servers not extracted, defaults to localhost

Medium Severity

The renderHttpFetchClient call doesn't pass the servers parameter, even though OpenAPI documents define server URLs (e.g., http://petstore.swagger.io/v2 in the test spec). The generated HTTP clients default to 'localhost:3000' instead of the actual API server from the spec. Users must manually override the server for every request, defeating the purpose of defining servers in the OpenAPI document.

Fix in Cursor Fix in Web

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.

2 participants