Skip to content

feat: add tool surface compression to summary#144

Open
Jer1605 wants to merge 3 commits into
mainfrom
edgee-1479-compression-rate-data-in-the-cli
Open

feat: add tool surface compression to summary#144
Jer1605 wants to merge 3 commits into
mainfrom
edgee-1479-compression-rate-data-in-the-cli

Conversation

@Jer1605

@Jer1605 Jer1605 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • I have read the Contributor Guide
  • I have read and agree to the Code of Conduct
  • I have added a description of my changes and why I'd like them included in the section below

Description of Changes

image

Related Issues

List related issues here

@Jer1605
Jer1605 requested a review from a team as a code owner July 16, 2026 12:30
@SachaMorard

Copy link
Copy Markdown
Member

@Jer1605 it seems great!
Is it possible to add Brevity as well?

@Jer1605

Jer1605 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

@SachaMorard done :)

style(pad_left(&fmt_tokens(ts.after), 9)).cyan(),
fmt_bar(pct, 10),
style(pct).green(),
" {} {} {} Savings",

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 looks missaligned wrt the tools

@CLEMENTINATOR CLEMENTINATOR left a comment

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.

content looks good, but style is missaligned

" Tools {} -> {} {} {}% saved",
style(fmt_tokens(stats.total_uncompressed_tools_tokens)).dim(),
style(fmt_tokens(stats.total_compressed_tools_tokens)).cyan(),
fmt_bar(pct, 20),

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.

bars are not aligned

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.

3 participants