Skip to content

fix: remove /preview/ prefix from frontend API calls#3873

Closed
jangjohn wants to merge 3 commits intoAgenta-AI:mainfrom
jangjohn:fix/remove-preview-prefix-v2
Closed

fix: remove /preview/ prefix from frontend API calls#3873
jangjohn wants to merge 3 commits intoAgenta-AI:mainfrom
jangjohn:fix/remove-preview-prefix-v2

Conversation

@jangjohn
Copy link

@jangjohn jangjohn commented Mar 2, 2026

Summary

Closes #3831

Supersedes #3871. Cleaned up to remove line ending changes.

Removes the /preview/ prefix from all frontend API calls across the web codebase, replacing them with canonical routes.

Changes

  • oss/src/ - OSS frontend service calls
  • ee/src/ - EE frontend service calls
  • packages/agenta-entities/src/ - entity API calls
  • packages/agenta-playground/src/ - playground API calls

Testing

  • Verified all changed routes against the backend route matrix
  • /preview/environments/ route was intentionally left unchanged as it has no canonical equivalent
<!-- devin-review-badge-begin -->

---

<a href="https://app.devin.ai/review/agenta-ai/agenta/pull/3873" target="_blank">
  <picture>
    <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
    <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin">
  </picture>
</a>
<!-- devin-review-badge-end -->

Resolves Agenta-AI#3831

All API calls in oss/src/, ee/src/, and packages/ updated to use
canonical routes without the /preview/ prefix.
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 2, 2026
@vercel
Copy link

vercel bot commented Mar 2, 2026

@jangjohn is attempting to deploy a commit to the agenta projects Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added bug Something isn't working Frontend labels Mar 2, 2026
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

View 5 additional findings in Devin Review.

Open in Devin Review

jangjohn added 2 commits March 2, 2026 22:54
Resolves Agenta-AI#3831

All API calls in oss/src/, ee/src/, packages/agenta-entities/src/,
and packages/agenta-playground/src/ updated to use canonical routes
without the /preview/ prefix.
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Mar 2, 2026
@jangjohn
Copy link
Author

jangjohn commented Mar 2, 2026

Hi @mmabrouk 👋

I’ve addressed the backend route gaps flagged by Devin by adding canonical mounts for:

  • /tools
  • /environments
  • /simple/environments

(while keeping the existing /preview/* mounts).

Could you please:

  1. Approve the pending GitHub Actions workflows, and
  2. Authorize the Vercel deployment check?

Happy to make any further adjustments if needed. Thanks!

Copy link
Member

@mmabrouk mmabrouk left a comment

Choose a reason for hiding this comment

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

Hey @jangjohn the PR is again not clean. The diff should include only the parts that are modified. Something might be wrong in your config.

@jangjohn
Copy link
Author

jangjohn commented Mar 5, 2026

Superseded by #3924

@jangjohn jangjohn closed this Mar 5, 2026
@jangjohn jangjohn deleted the fix/remove-preview-prefix-v2 branch March 5, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Frontend size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Frontend: migrate endpoints to remove preview/ prefix

2 participants