Skip to content

fix(clean): add Docker prune hint with 30-day filter#689

Merged
tw93 merged 2 commits intotw93:mainfrom
sebastianbreguel:fix/docker-prune-hint
Apr 7, 2026
Merged

fix(clean): add Docker prune hint with 30-day filter#689
tw93 merged 2 commits intotw93:mainfrom
sebastianbreguel:fix/docker-prune-hint

Conversation

@sebastianbreguel
Copy link
Copy Markdown
Contributor

Summary

  • Add a suggested docker system prune --filter until=720h hint below the existing "Review: docker system df" line
  • Users see guidance on how to reclaim Docker space safely (30-day threshold)
  • No automated execution — hint only

Safety Review

  • Does this change affect cleanup, uninstall, optimize, installer, remove, analyze delete, update, or install behavior?
    • No — this is a display-only change. No Docker daemon commands are executed.
  • Does this change affect path validation, protected directories, symlink handling, sudo boundaries, or release/install integrity?
    • No.
  • New boundary or risk change:
    • None. Adds one echo line with a suggested manual command.

Tests

  • No new tests needed — display-only change with no behavioral impact.

Users see "Docker unused data · skipped by default" but have no
guidance on how to reclaim space safely. Add a prune suggestion
with --filter until=720h (30 days) so users can act without
guessing the right command.
@sebastianbreguel sebastianbreguel requested a review from tw93 as a code owner April 6, 2026 15:28
The prune hint is display-only, not an executed command. Update
assertions to verify the hint appears instead of asserting absence.
Copy link
Copy Markdown
Owner

@tw93 tw93 left a comment

Choose a reason for hiding this comment

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

Reviewed against current cleanup behavior. This stays display-only, the prune command remains manual, the cleanup boundary does not expand, and the updated tests plus green CI are sufficient for merge.

@tw93 tw93 merged commit 2eab14e into tw93:main Apr 7, 2026
9 checks passed
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.

2 participants