Skip to content

Comments

Add Astro IDE commands to Astro CLI#1839

Merged
pritt20 merged 18 commits intomainfrom
add_polaris_cmd
Sep 3, 2025
Merged

Add Astro IDE commands to Astro CLI#1839
pritt20 merged 18 commits intomainfrom
add_polaris_cmd

Conversation

@pritt20
Copy link
Contributor

@pritt20 pritt20 commented Apr 17, 2025 •
edited
Loading

Description

This PR adds basic Astro cli commands for Polaris (Astro-IDE). The list of commands are as follows:

  • astro ide project list
  • astro ide project import
  • astro ide project export

🎟 Issue(s)

Related https://github.com/astronomer/astro/issues/29889

🧪 Functional Testing

Tested locally:

astro ide project list

astro ide project list
 NAME             ID                             
 cli-ide-test     cm9lnri9s004b01izy4x8c8st     
 polaris-test     cm9lqfo5100mk01nxhu0rf6pc     
 test-ide         cma3nhqgs00nm01nfzqxdaemb     
 test-project     cm9lsdrpr00uc01ky156ekgt3

astro ide project import

astro ide project import

Please select the project from the list below:
 #     PROJECTNAME      ID                            
 1                      cma3n11l2007001jhpeh68q9c     
 2     cli-ide-test     cm9lnri9s004b01izy4x8c8st     
 3     polaris-test     cm9lqfo5100mk01nxhu0rf6pc     
 4     test-ide         cma3nhqgs00nm01nfzqxdaemb     
 5     test-project     cm9lsdrpr00uc01ky156ekgt3    

> 4
Successfully exported project from cma3nhqgs00nm01nfzqxdaemb

astro ide project export

astro ide project export
Do you want to create a new project? (y/n)

> y
Enter project name:

> test-ide-project  
Successfully created project 'test-ide-project' in workspace 'clm7hapoz009a01ok5jn619kt'
Successfully exported project to cma3nkuj100oj01nfqmpiufzb

astro ide project export
Do you want to create a new project? (y/n)

> n

Please select the project from the list below:
 #     PROJECTNAME      ID                            
 1                      cma3n11l2007001jhpeh68q9c     
 2     cli-ide-test     cm9lnri9s004b01izy4x8c8st     
 3     polaris-test     cm9lqfo5100mk01nxhu0rf6pc     
 4     test-ide         cma3nhqgs00nm01nfzqxdaemb     
 5     test-project     cm9lsdrpr00uc01ky156ekgt3   

> 4
Successfully exported project to cma3nhqgs00nm01nfzqxdaemb

Add screenshots to illustrate the validity of these changes.

📋 Checklist

  • Rebased from the main (or release if patching) branch (before testing)
  • Ran make test before taking out of draft
  • Ran make lint before taking out of draft
  • Added/updated applicable tests
  • Tested against Astro-API (if necessary).
  • Tested against Houston-API and Astronomer (if necessary).
  • Communicated to/tagged owners of respective clients potentially impacted by these changes.
  • Updated any related documentation

@schnie
Copy link
Member

schnie commented Apr 22, 2025

@pritt20 we can open this for approvals and/or just merge it to polaris-main can't we?

@pritt20
Copy link
Contributor Author

pritt20 commented Apr 22, 2025

@pritt20 we can open this for approvals and/or just merge it to polaris-main can't we?

Yeah, nearly done, just working on to push couple of more changes to use specific endpoints.

@pritt20 pritt20 marked this pull request as ready for review April 30, 2025 08:41
@coveralls-official
Copy link

coveralls-official bot commented Aug 22, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build ad778067-0fe4-4579-bb07-448bee6b982e

Details

  • 54 of 4224 (1.28%) changed or added relevant lines in 5 files are covered.
  • 68 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-1.3%) to 38.12%
Changes Missing Coverage Covered Lines Changed/Added Lines %
cmd/cloud/ide.go 49 91 53.85%
cloud/ide/project.go 0 400 0.0%
astro-client-core/mocks/client.go 0 406 0.0%
astro-client-core/api.gen.go 4 3326 0.12%
Files with Coverage Reduction New Missed Lines %
astro-client-core/api.gen.go 68 0.16%
Totals Coverage Status
Change from base Build 358a99dd-d88c-4a0b-b401-1b5df21cf912: -1.3%
Covered Lines: 23734
Relevant Lines: 62261
💛 - Coveralls

@jeremybeard jeremybeard changed the title Add polaris commands to Astro CLI Add Astro IDE commands to Astro CLI Aug 28, 2025
jeremybeard
jeremybeard reviewed Aug 28, 2025
View reviewed changes
Copy link
Contributor

@jeremybeard jeremybeard left a comment

Choose a reason for hiding this comment

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

Can we also rebase the PR on main? If there's anything unmerged in "polaris-main" let's get those merged in too.

@pritt20 pritt20 changed the base branch from polaris-main to main September 2, 2025 16:09
jeremybeard
jeremybeard reviewed Sep 3, 2025
View reviewed changes
jeremybeard
jeremybeard approved these changes Sep 3, 2025
View reviewed changes
Copy link
Contributor

@jeremybeard jeremybeard left a comment

Choose a reason for hiding this comment

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

Approving ahead of final comments

@pritt20 pritt20 merged commit 26e6c9d into main Sep 3, 2025
6 checks passed
@pritt20 pritt20 deleted the add_polaris_cmd branch September 3, 2025 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jeremybeard jeremybeard jeremybeard approved these changes

@neel-astro neel-astro Awaiting requested review from neel-astro

@kushalmalani kushalmalani Awaiting requested review from kushalmalani

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pritt20 @schnie @jeremybeard