Skip to content

feat(ui): Analysis UI#1017

Open
chrisgalanis wants to merge 12 commits intomainfrom
chris/analysis-ui
Open

feat(ui): Analysis UI#1017
chrisgalanis wants to merge 12 commits intomainfrom
chris/analysis-ui

Conversation

@chrisgalanis
Copy link
Contributor

@chrisgalanis chrisgalanis commented Feb 24, 2026

This pull request introduces a new mechanism for selecting operator default colors based on the current release type (such as "Analysis" or "Prod") for both Pearl and Diamond orb types. It achieves this by reading the OS release type at runtime and updating the logic throughout the codebase to use the appropriate color constants. Additionally, new color constants specific to the "Analysis" release type are added.

Release-type aware color selection:

  • Added a global static RELEASE_TYPE (using LazyLock) in ui/src/main.rs that reads the current OrbRelease from /etc/os-release, defaulting to Prod if unavailable.
  • Introduced helper functions pearl_operator_default() and diamond_operator_default() in ui/src/engine/mod.rs to select the correct color constant based on RELEASE_TYPE.
  • Updated all usages of PEARL_OPERATOR_DEFAULT and DIAMOND_OPERATOR_DEFAULT in operator logic (idle, pulse, signup_phase, and runner modules) to use the new helper functions, ensuring color selection is dynamic based on release type. [1] [2] [3] [4] [5] [6]

New color constants:

  • Added PEARL_OPERATOR_ANALYSIS and DIAMOND_OPERATOR_ANALYSIS color constants to the Argb implementation for use in the "Analysis" release type. [1] [2]

Dependency and import updates:

  • Added once_cell and updated orb-info dependencies in ui/Cargo.toml to support the new static initialization.
  • Updated imports in ui/src/engine/mod.rs and ui/src/main.rs to include orb_info::orb_os_release::{OrbOsRelease, OrbRelease} and related types. [1] [2]

@chrisgalanis chrisgalanis requested a review from a team as a code owner February 24, 2026 14:42
@AlexKaravaev
Copy link
Contributor

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ecfc4d1284

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chrisgalanis chrisgalanis changed the title feat: Analysis UI feat(ui): Analysis UI Feb 24, 2026
@chrisgalanis
Copy link
Contributor Author

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: dd2789ad57

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chrisgalanis
Copy link
Contributor Author

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Breezy!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chrisgalanis chrisgalanis requested review from AlexKaravaev, karelispanagiotis and vmenge and removed request for AlexKaravaev February 24, 2026 16:09
@chrisgalanis
Copy link
Contributor Author

@cursor review

@chrisgalanis
Copy link
Contributor Author

Thanks for the Review @AlexKaravaev ! I will merge after I test the binary:

  • On Analysis Pearl/Diamond
  • On Dev Pearl/Diamond

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