Skip to content

feat(shell): Add shell run summary policy and mute flag#14

Merged
EspenAlbert merged 4 commits into
mainfrom
silent-logging
May 15, 2026
Merged

feat(shell): Add shell run summary policy and mute flag#14
EspenAlbert merged 4 commits into
mainfrom
silent-logging

Conversation

@EspenAlbert
Copy link
Copy Markdown
Owner

@EspenAlbert EspenAlbert commented May 15, 2026

Summary

  • Adds ShellRunSummary (all / errors_only) on AskShellSettings, env ASK_SHELL_SHELL_RUN_SUMMARY.
  • Skips per-command completion logs in rich_run_state.remove_run when summary is errors_only and the run exited zero.
  • Adds ShellConfig.mute_shell_summary to skip completion logs for a single run (probes).
  • log_task_done logs expected non-zero exits (allow_non_zero_exit) at warning instead of error.
  • run and run_and_wait accept mute_shell_summary.
  • Tests cover summary policy and mute; conftest mocked ShellConfig includes new fields.

Notes

  • Default remains all for backward compatibility. tfdo sets errors_only via env after its CLI callback.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.56%. Comparing base (dfbbfc3) to head (22f984d).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   85.43%   85.56%   +0.13%     
==========================================
  Files          20       20              
  Lines        1860     1878      +18     
==========================================
+ Hits         1589     1607      +18     
  Misses        271      271              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@EspenAlbert EspenAlbert changed the title todo update me with the PR title for branch silent-logging feat(shell): Add shell run summary policy and mute flag May 15, 2026
@EspenAlbert EspenAlbert marked this pull request as ready for review May 15, 2026 13:27
@EspenAlbert EspenAlbert enabled auto-merge (squash) May 15, 2026 13:29
@EspenAlbert EspenAlbert merged commit 425cf84 into main May 15, 2026
16 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