Skip to content

[BUILD-839] fix(ide): open session URL after project export#2002

Open
jlaneve wants to merge 2 commits intomainfrom
agent/build-839
Open

[BUILD-839] fix(ide): open session URL after project export#2002
jlaneve wants to merge 2 commits intomainfrom
agent/build-839

Conversation

@jlaneve
Copy link
Contributor

@jlaneve jlaneve commented Jan 29, 2026

Summary

  • Updates openProjectInBrowser to accept a session ID and construct the session URL
  • After astro ide project export, the CLI now opens the session URL (with session ID) instead of the project URL

This fixes the issue where the Astro IDE page appears blank after project import because the frontend now requires session IDs in the URL path.

Related PRs

  • astronomer/astro PR (frontend backwards compatibility)

Test plan

  • Run astro ide project export and verify the browser opens to the session URL
  • Verify the URL format is {base}/{workspaceId}/astro-ide/{projectId}/sessions/{sessionId}
  • Confirm the IDE page loads with files visible

🤖 Generated with Claude Code

When exporting a project to Astro IDE via `astro ide project export`,
the CLI now opens the session URL instead of the project URL. This
ensures the user lands directly on their imported project's session
rather than a blank page.

The issue occurred because the frontend now requires session IDs in the
URL path for IDE pages. Previously, the CLI was opening the project URL
without a session ID, causing the page to appear blank.

[BUILD-839]

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coveralls-official
Copy link

coveralls-official bot commented Jan 29, 2026

Pull Request Test Coverage Report for Build 18fe4422-4c81-4faf-b71b-1e37c8ac1d40

Details

  • 0 of 7 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.002%) to 33.151%

Changes Missing Coverage Covered Lines Changed/Added Lines %
cloud/ide/project.go 0 7 0.0%
Totals Coverage Status
Change from base Build 055b4403-e142-4b85-a4c3-69b561e6393f: -0.002%
Covered Lines: 20868
Relevant Lines: 62948

💛 - Coveralls

@jlaneve jlaneve marked this pull request as ready for review February 7, 2026 18:20
@jlaneve jlaneve requested a review from a team as a code owner February 7, 2026 18:20
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.

1 participant