Skip to content

fix: add /api prefix to Dune service endpoint routes#457

Merged
brendanjryan merged 2 commits intotempoxyz:mainfrom
va3093:fix/dune-api-prefix
Mar 25, 2026
Merged

fix: add /api prefix to Dune service endpoint routes#457
brendanjryan merged 2 commits intotempoxyz:mainfrom
va3093:fix/dune-api-prefix

Conversation

@va3093
Copy link
Contributor

@va3093 va3093 commented Mar 25, 2026

Summary

  • Adds /api prefix to all three Dune service endpoint routes in services.ts
  • Regenerated discovery.json and llms.txt

Updated routes

Before After
POST /v1/sql/execute POST /api/v1/sql/execute
GET /v1/execution/:execution_id/csv GET /api/v1/execution/:execution_id/csv
GET /v1/execution/:execution_id/results GET /api/v1/execution/:execution_id/results

Note: discovery.json drift

The regenerated discovery.json includes a few unrelated changes to the QuickNode service (categories, intent, description fields). This is because discovery.json on main has drifted from what generate-discovery.ts produces from services.ts. The QuickNode diff is not from this PR — it is pre-existing drift that surfaces when the generator runs.

@vercel
Copy link

vercel bot commented Mar 25, 2026

@va3093 is attempting to deploy a commit to the Tempo Team on Vercel.

A member of the Team first needs to authorize it.

Update all three Dune endpoint routes to include the /api prefix:
- POST /api/v1/sql/execute
- GET /api/v1/execution/:execution_id/csv
- GET /api/v1/execution/:execution_id/results

Regenerated discovery.json to reflect the changes.
@va3093 va3093 force-pushed the fix/dune-api-prefix branch from ecb8eb2 to bc9fb58 Compare March 25, 2026 14:01
@brendanjryan brendanjryan merged commit 183e76a into tempoxyz:main Mar 25, 2026
1 check failed
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