Skip to content

feat: add --verbose and --quiet flags to toolkit CLI#859

Merged
ozgb merged 11 commits into
mainfrom
ozgb-toolkit-log-levels
Mar 6, 2026
Merged

feat: add --verbose and --quiet flags to toolkit CLI#859
ozgb merged 11 commits into
mainfrom
ozgb-toolkit-log-levels

Conversation

@ozgb

@ozgb ozgb commented Mar 4, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Overview

Add global -v/--verbose (debug level) and -q/--quiet (warn level) flags,
with info as the default. Demote noisy per-batch fetch/compute/verify
log messages from info to debug, keeping high-level progress and
summaries at info. Restore debug-level transaction logging removed in
PR #802

Ticket: https://shielded.atlassian.net/browse/PM-22220

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

@ozgb
feat: add --verbose and --quiet flags to toolkit CLI
0b0e290 
Add global -v/--verbose (debug level) and -q/--quiet (warn level) flags,
with info as the default. Demote noisy per-batch fetch/compute/verify
log messages from info to debug, keeping high-level progress and
summaries at info. Restore debug-level transaction logging removed in
PR #802.
@ozgb ozgb requested a review from a team as a code owner March 4, 2026 16:53
@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

kics-logo

KICS version: v2.1.19

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 47
LOW LOW 3
INFO INFO 59
TRACE TRACE 0
TOTAL TOTAL 109
Metric Values
Files scanned placeholder 26
Files parsed placeholder 26
Files failed to scan placeholder 0
Total executed queries placeholder 73
Queries failed to execute placeholder 0
Execution time placeholder 11

ozgb added 6 commits March 4, 2026 16:57
@ozgb
fix: update println -> info to respect --quiet
4cd95b2
@ozgb
refactor: convert println status/dry-run messages to log macros
3498b8c 
Convert status messages to log::info!, dry-run messages to log::info!,
error output to eprintln!, and debug-level diagnostics to log::debug!.
This ensures --quiet suppresses all status output while command results
(stdout) remain unaffected.
@ozgb
chore: add ticket link to change file
bd58cbf
@ozgb
chore: cargo fmt
a6350a9
@ozgb
feat: default to text logger; add --log-json option
1d03e4b
@ozgb
docs: add change file
1dae425
LGLO
LGLO reviewed Mar 5, 2026
View reviewed changes
Comment thread util/toolkit/src/log_writer.rs
LGLO
LGLO reviewed Mar 5, 2026
View reviewed changes
Comment thread util/toolkit/src/cli.rs Outdated
jsidorenko
jsidorenko reviewed Mar 5, 2026
View reviewed changes
Comment thread util/toolkit/src/cli.rs
@ozgb ozgb added this pull request to the merge queue Mar 6, 2026
Merged via the queue into main with commit a191737 Mar 6, 2026
34 checks passed
@ozgb ozgb deleted the ozgb-toolkit-log-levels branch March 6, 2026 10:49
@gilescope gilescope added this to the node-1.0.0 milestone Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LGLO LGLO LGLO left review comments

@jsidorenko jsidorenko jsidorenko approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

node-1.0.0

Development

Successfully merging this pull request may close these issues.

5 participants

@ozgb @LGLO @jsidorenko @gilescope @TheNewAutonomy