Fix generate_urls endpoint body handling #411

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

Jump to bottom

Open

seveibar wants to merge 1 commit into main from codex/fix-error-on-generate-urls

Contributor

seveibar commented Oct 12, 2025

Summary

reuse the request context body inside the generate_urls handler to avoid double reads
add validation and fallbacks when building the generated URL response
cover the regression with a Bun test that posts to /generate_urls

Testing

bun test tests/generate-urls.test.ts
bunx tsc --noEmit

https://chatgpt.com/codex/tasks/task_b_68ec206b3be8832ebec9b0f94e4e9450


          Fix generate_urls handler to reuse parsed request body

f2c7e32

vercel bot commented Oct 12, 2025 •

edited

Loading

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

Project	Deployment	Preview	Comments	Updated (UTC)
svg-tscircuit-com	Ready	Preview	Comment	Oct 12, 2025 9:59pm

vercel bot deployed to Preview

October 12, 2025 21:59

View deployment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet