Skip to content

feat(charm): add lipgloss styling with Slack brand colors under charm experiment#365

Draft
srtaalej wants to merge 1 commit intomainfrom
charm-lipgloss-migration
Draft

feat(charm): add lipgloss styling with Slack brand colors under charm experiment#365
srtaalej wants to merge 1 commit intomainfrom
charm-lipgloss-migration

Conversation

@srtaalej
Copy link
Contributor

@srtaalej srtaalej commented Mar 6, 2026

Changelog

Adds lipgloss-based output styling under the charm experiment flag, using a centralized Slack brand color palette shared across all CLI components, and migrates all direct Styler() usage to named style functions. 🦋

Summary

This PR introduces a centralized Slack brand color palette (colors.go) as the single source of truth for all styling across lipgloss, huh themes, and future bubbletea components

  • Adds lipgloss-based styling to all CLI output functions (behind the charm experiment flag), alongside existing aurora styling
  • Migrates all external Styler() call sites to use new style functions (Green(), Red(), Yellow(), Gray())
  • Consolidates all legacy aurora code into one clearly marked block for easy removal when the charm experiment is permanently enabled

Requirements

@srtaalej srtaalej added this to the Next Release milestone Mar 6, 2026
@srtaalej srtaalej self-assigned this Mar 6, 2026
@srtaalej srtaalej added experiment Experimental feature accessed behind the --experiment flag or toggle semver:patch Use on pull requests to describe the release version increment labels Mar 6, 2026
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 28.30189% with 76 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.92%. Comparing base (c449532) to head (0c3d9d6).

Files with missing lines Patch % Lines
internal/style/style.go 24.13% 66 Missing ⚠️
internal/pkg/platform/activity.go 33.33% 4 Missing ⚠️
cmd/upgrade/upgrade.go 0.00% 2 Missing ⚠️
internal/pkg/apps/install.go 0.00% 2 Missing ⚠️
cmd/manifest/validate.go 50.00% 1 Missing ⚠️
internal/iostreams/printer.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #365      +/-   ##
==========================================
- Coverage   65.06%   64.92%   -0.15%     
==========================================
  Files         215      215              
  Lines       18179    18248      +69     
==========================================
+ Hits        11829    11847      +18     
- Misses       5254     5306      +52     
+ Partials     1096     1095       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

experiment Experimental feature accessed behind the --experiment flag or toggle semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant