Skip to content

feat(app): sessions list improvements#30941

Merged
Hona merged 2 commits into
anomalyco:devfrom
arvsrn:session-list-improvements
Jun 5, 2026
Merged

feat(app): sessions list improvements#30941
Hona merged 2 commits into
anomalyco:devfrom
arvsrn:session-list-improvements

Conversation

@arvsrn
Copy link
Copy Markdown
Contributor

@arvsrn arvsrn commented Jun 5, 2026

Issue for this PR

N/A

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

  • Added bg-layer-04 color token
  • Replaced TabStateIndicator with project avatar
  • Added indicator for sessions with open tabs

How did you verify your code works?

  • Ran the app locally and verified the updated UI manually
  • Ran type checks locally (bun turbo typecheck)

Screenshots / recordings

image

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

"v2-background-bg-layer-01": "var(--v2-grey-200)",
"v2-background-bg-layer-02": "var(--v2-grey-300)",
"v2-background-bg-layer-03": "var(--v2-grey-400)",
"v2-background-bg-layer-04": "var(--v2-grey-400)",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This override is applied after the semantic mapping in resolveThemeVariantV2(), so OC-2 still resolves bg-layer-04 to the pre-tuning colors (grey-400 here and grey-700 in dark mode). Please regenerate oc-2.json with packages/ui/script/build-oc2-v2-overrides.ts after updating theme.css so the open-tab indicator receives the tuned values.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

grey-400 and gray-700 is correct as per Figma, I only changed the tokens for generated themes to bring their contrast closer to OC-2

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, the other OC-2 tokens were outdated; the design system must've been updated... I made a quick PR to fix: #31071

@Hona Hona enabled auto-merge (squash) June 5, 2026 21:59
@Hona Hona merged commit 1fd9c77 into anomalyco:dev Jun 5, 2026
14 of 15 checks passed
ShamirSecret pushed a commit to ShamirSecret/auto-code-machine that referenced this pull request Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants