fix: remove misleading 'Since Last Shutdown' from active sessions title (#1072) by microsasa · Pull Request #1074 · microsasa/cli-tools

microsasa · 2026-04-24T15:02:01Z

Summary

_render_active_section_from in report.py hardcoded the table title "🟢 Active Sessions (Since Last Shutdown)" for all active sessions — including pure-active sessions that have never had a shutdown event (has_shutdown_metrics=False).

This changes the title to "🟢 Active Sessions" unconditionally (Option A from the issue). The columns already indicate what data is being shown; the parenthetical qualifier only added confusion for pure-active sessions.

Changes

src/copilot_usage/report.py: Change table title from "🟢 Active Sessions (Since Last Shutdown)" to "🟢 Active Sessions"
tests/copilot_usage/test_report.py: Update existing resumed-session test to assert absence of qualifier; add new test_pure_active_section_title_no_since_last_shutdown test
tests/e2e/test_e2e.py: Update _ACTIVE_MARKER constant to match new title

Closes #1072

Warning

⚠️ Firewall blocked 3 domains

The following domains were blocked by the firewall during workflow execution:

astral.sh
index.crates.io
pypi.org

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

network:
  allowed:
    - defaults
    - "astral.sh"
    - "index.crates.io"
    - "pypi.org"

See Network Configuration for more information.

Generated by Issue Implementer · ● 10M · ◷

The active-sessions table title in _render_active_section_from hardcoded 'Since Last Shutdown', which is inaccurate for pure-active sessions (has_shutdown_metrics=False) that have never had a shutdown event. Change the title to '🟢 Active Sessions' unconditionally (Option A from issue #1072). The columns already convey what data is being shown. - Update existing resumed-session test to assert absence of qualifier - Add new test for pure-active sessions verifying no 'Since Last Shutdown' - Update e2e _ACTIVE_MARKER constant to match new title Closes #1072 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Removes the misleading “Since Last Shutdown” qualifier from the Active Sessions table title so it is accurate for both resumed sessions and pure-active sessions.

Changes:

Update the Rich table title in the interactive summary active section to “🟢 Active Sessions”.
Adjust unit tests to assert the qualifier is absent, and add a regression test for pure-active sessions.
Update the e2e active-section marker to match the new title text.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
`src/copilot_usage/report.py`	Updates the active sessions table title to remove the “Since Last Shutdown” qualifier.
`tests/copilot_usage/test_report.py`	Updates existing assertions and adds a new unit test ensuring the qualifier never appears for pure-active sessions.
`tests/e2e/test_e2e.py`	Updates the string marker used to split historical vs active output in interactive e2e tests.

The Table() constructor call was left in multi-line format after shortening the title string. With the shorter title, the call fits on a single line (66 chars < 88 line limit), so ruff format requires it to be collapsed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

github-actions · 2026-04-24T15:08:49Z

Commit pushed: 8b57f52

Generated by CI Fixer

microsasa · 2026-04-24T15:08:50Z

CI Fix Applied 🔧

The check job failed due to a ruff format violation in src/copilot_usage/report.py.

Root cause: The Table() constructor was left in multi-line format after shortening the title from "🟢 Active Sessions (Since Last Shutdown)" to "🟢 Active Sessions". The shorter string makes the call fit on a single line (66 chars < 88 line limit), so ruff format requires it to be collapsed.

Fix: Collapsed the Table() call to a single line.

Warning

⚠️ Firewall blocked 1 domain

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

pypi.org

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

network:
  allowed:
    - defaults
    - "pypi.org"

See Network Configuration for more information.

Generated by CI Fixer · ● 5.4M · ◷

microsasa · 2026-04-24T15:12:06Z

CI fix already attempted once — stopping to prevent loops. Manual intervention needed.

Generated by CI Fixer · ● 483.5K · ◷

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Copilot · 2026-04-24T15:12:35Z

+    table = Table(title="🟢 Active Sessions", border_style="green")
    table.add_column("Name", style="bold", max_width=40)
    table.add_column("Model")
    table.add_column("Model Calls", justify="right")


The render_full_summary docstring still says “Section 2: Active sessions since last shutdown.” (see report.py around lines 593–596), but the section title is now unqualified and the active stats can represent full-lifetime data for pure-active sessions. Please update the docstring to match the new behavior to avoid misleading future maintainers.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

github-actions · 2026-04-24T15:18:34Z

Commit pushed: 70fb682

Generated by Review Responder

microsasa · 2026-04-24T15:44:32Z

❌ Pipeline orchestrator: review-response loop reached 5 rounds. Marking as stuck for human review.

Copilot AI review requested due to automatic review settings April 24, 2026 15:02

microsasa added the aw Created by agentic workflow label Apr 24, 2026

microsasa enabled auto-merge April 24, 2026 15:02

Copilot started reviewing on behalf of microsasa April 24, 2026 15:02 View session

Copilot AI reviewed Apr 24, 2026

View reviewed changes

microsasa added the aw-ci-fix-attempted CI fixer has attempted to fix CI label Apr 24, 2026

microsasa requested a review from Copilot April 24, 2026 15:09

Copilot started reviewing on behalf of microsasa April 24, 2026 15:10 View session

Copilot AI reviewed Apr 24, 2026

View reviewed changes

microsasa added the aw-review-response-1 Review response round 1 label Apr 24, 2026

fix: address review comments

8f20f4c

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

microsasa added aw-review-response-2 Review response round 2 aw-review-response-3 Review response round 3 aw-review-response-4 Responder dispatch round 4 (orchestrator-managed) aw-review-response-5 Responder dispatch round 5 (orchestrator-managed) aw-pr-stuck:review labels Apr 24, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: remove misleading 'Since Last Shutdown' from active sessions title (#1072)#1074

fix: remove misleading 'Since Last Shutdown' from active sessions title (#1072)#1074
microsasa wants to merge 3 commits intomainfrom
fix/1072-active-section-title-4592161bd5805d96

microsasa commented Apr 24, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

microsasa commented Apr 24, 2026

Uh oh!

microsasa commented Apr 24, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 24, 2026

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

microsasa commented Apr 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

microsasa commented Apr 24, 2026

Summary

Changes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

microsasa commented Apr 24, 2026

Uh oh!

microsasa commented Apr 24, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Apr 24, 2026

Uh oh!

microsasa commented Apr 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants