feat: support dev orgs with runtime, in addition to entp#134
Conversation
There was a problem hiding this comment.
🤖 PR Reviewer
The diff introduces a well-structured getOrganizationFeatures method that bypasses the swagger client to call the Console BFF directly. The implementation is clean, error handling is consistent with the rest of the SDK, and test coverage is thorough. One minor issue: the res variable from fetchWithRetry is referenced after the if (!res.ok) block to build the return object, but res is a closure variable so it's fine — however there is a subtle issue where res.headers is accessed on the resolved response after the if block but the Headers object's iterability depends on the fetch implementation. This is a very minor concern and the code is overall high quality.
✅ LGTM! This PR looks good to merge.
💡 How to re-trigger
Comment /review or /pr-reviewer on this PR
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Description
Related:
Related Issue
Motivation and Context
How Has This Been Tested?
Locally linked everything and tested with a dev and entp org
aio login aio console org list aio console org select aio app initScreenshots (if appropriate):
Types of changes
Checklist: