Skip to content

Do not print an empty table when every addon is up to date - #55

Merged
refaim merged 1 commit into
masterfrom
fix-empty-status-table
Aug 30, 2026
Merged

Do not print an empty table when every addon is up to date#55
refaim merged 1 commit into
masterfrom
fix-empty-status-table

Conversation

@refaim

@refaim refaim commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Without -v the status table only receives addons that need attention, so when everything is up to date the command printed a table frame containing nothing but headers, followed by the summary line.

Before:

+---------+----------+---------------+----------------+
| addon   | status   | released_at   | installed_at   |
|---------+----------+---------------+----------------|
+---------+----------+---------------+----------------+
3 addons are up to date

After:

3 addons are up to date

The empty frame carries no information and reads as "nothing is tracked". That is not hypothetical — the reporter in #52 hit exactly this after his addons updated cleanly and filed it as DBM having disappeared from status.

Output is unchanged whenever the table has rows, and the summary keeps its other qualifier:

+-----------------+-----------+---------------+----------------+
| addon           | status    | released_at   | installed_at   |
|-----------------+-----------+---------------+----------------|
| SomeRandomAddon | untracked |               |                |
+-----------------+-----------+---------------+----------------+
3 other addons are up to date

status -v still lists every addon, up-to-date ones included.

The No addons found guard for an empty addon directory is untouched — that is a different case from "everything is up to date".

🤖 Generated with Claude Code

Without -v the status table only receives addons that need attention, so
when everything was up to date the command printed a table frame with
nothing but headers above the summary line. The empty frame carries no
information and reads as "nothing is tracked" - a user filed exactly that
misreading as part of #52.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@codecov

codecov Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (5b409e7) to head (f9c58ea).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #55   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines          583       584    +1     
  Branches        97        98    +1     
=========================================
+ Hits           583       584    +1     

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

@refaim
refaim merged commit f40cd5e into master Aug 30, 2026
7 checks passed
@refaim
refaim deleted the fix-empty-status-table branch August 30, 2026 16:10
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.

1 participant