Skip to content

fix: update stale README command examples (#239)#252

Merged
microsasa merged 1 commit intomainfrom
fix/239-readme-stale-examples-e1a7ca9274f5b83f
Mar 22, 2026
Merged

fix: update stale README command examples (#239)#252
microsasa merged 1 commit intomainfrom
fix/239-readme-stale-examples-e1a7ca9274f5b83f

Conversation

@microsasa
Copy link
Owner

Fixes #239

Updates the example output blocks in README.md to match the current render_live_sessions and render_cost_view implementations:

Changes

  1. copilot-usage live example — Added missing Est. Cost column (between Messages and Output Tokens), matching the 8-column table produced by render_live_sessions
  2. copilot-usage live example — Replaced raw comma-formatted token integers with K/M formatted values to match format_tokens() output:
    • 468,625468.6K
    • 483,762483.8K
    • 1,225,9901.2M
  3. copilot-usage cost example — Fixed the "↳ Since last shutdown" row to show ~3 (estimated premium cost via _estimate_premium_cost) instead of N/A in the Premium Cost column

Verification

  • All 506 tests pass
  • Coverage: 98.77% (threshold: 80%)
  • Ruff lint/format: clean
  • Pyright strict: 0 errors

Generated by Issue Implementer ·

Warning

⚠️ Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • astral.sh

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "astral.sh"

See Network Configuration for more information.

- Add missing 'Est. Cost' column to the live command example table
- Replace raw comma-formatted token integers with K/M formatted values
  (468,625 → 468.6K, 483,762 → 483.8K, 1,225,990 → 1.2M)
- Fix cost '↳ Since last shutdown' row: show estimated premium cost (~3)
  instead of N/A in the Premium Cost column

Closes #239

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@microsasa microsasa added the aw Created by agentic workflow label Mar 22, 2026
Copilot AI review requested due to automatic review settings March 22, 2026 17:51
@microsasa microsasa enabled auto-merge March 22, 2026 17:51
@microsasa microsasa merged commit d2a26f1 into main Mar 22, 2026
5 checks passed
@microsasa microsasa deleted the fix/239-readme-stale-examples-e1a7ca9274f5b83f branch March 22, 2026 17:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates README.md CLI output examples so they accurately reflect the current Rich table rendering in src/copilot_usage/report.py for the live and cost commands (addressing #239).

Changes:

  • Fixes the copilot-usage cost example “↳ Since last shutdown” row to show an estimated Premium Cost (~3) rather than N/A.
  • Updates the copilot-usage live example to include the missing “Est. Cost” column and match current token formatting (K/M suffixes).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

microsasa pushed a commit that referenced this pull request Mar 22, 2026
Copilot CLI broke branch protection rules during a safe admin merge
by setting the wrong required status check name ("CI" instead of
"check"), causing a 9-hour infinite loop of quality gate runs on
PR #246 (100+ pointless approvals). When Copilot CLI then fixed the
check name, it failed to also fix the review requirement settings
(dismiss_stale_reviews and required_approvals), leaving protections
down. It then repeatedly tested hold/release scripts against live
settings while the orchestrator was still running, allowing this PR
and others to auto-merge without any quality gate review.

Copilot CLI is a liar and cannot be trusted with repository settings.
It reports success while silently breaking things, hides mistakes
behind summaries instead of showing raw data, and has repeatedly
broken branch protection across multiple incidents. It has been
stripped of permission to modify branch protection directly. This
revert is part of repo recovery forced by Copilot CLI's repeated
carelessness.

This reverts commit d2a26f1.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aw Created by agentic workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw][code health] README command examples are stale (missing column, wrong token format)

2 participants