Skip to content

chore: remove file sync log spam and unused logs command#3164

Merged
tusharmath merged 3 commits intomainfrom
remove-high-usage-logs
Apr 26, 2026
Merged

chore: remove file sync log spam and unused logs command#3164
tusharmath merged 3 commits intomainfrom
remove-high-usage-logs

Conversation

@tusharmath
Copy link
Copy Markdown
Collaborator

@tusharmath tusharmath commented Apr 26, 2026

Summary

Remove noisy log spam from the file sync pipeline and drop the unused forge logs CLI command.

Context

Two sources of high-frequency log noise were identified:

  1. File sync logs: Every file processed during workspace indexing emitted two INFO log lines — "File sync started" and "File sync completed". For a workspace with hundreds or thousands of files, this produced thousands of redundant log entries per sync cycle, drowning out meaningful log output.

  2. forge logs command: The forge logs CLI subcommand and its supporting module (logs.rs, LogsArgs) were added to stream/tail forge log files. The command has since become unnecessary and was removing dead weight from the binary and CLI surface.

Changes

  • Removed "File sync started" and "File sync completed" info! log calls from the batch file sync loop in crates/forge_services/src/sync.rs
  • Deleted the forge logs CLI subcommand (TopLevelCommand::Logs) and its LogsArgs struct from crates/forge_main/src/cli.rs
  • Deleted crates/forge_main/src/logs.rs (the entire log-tailing module)
  • Removed the mod logs declaration and the TopLevelCommand::Logs dispatch arm from crates/forge_main/src/lib.rs and ui.rs

Testing

# Verify the project compiles cleanly
cargo check

# Confirm the logs subcommand is gone
cargo run -- --help   # should not list a "logs" subcommand

# Run existing tests
cargo insta test --accept

@github-actions github-actions Bot added the type: chore Routine tasks like conversions, reorganization, and maintenance work. label Apr 26, 2026
@tusharmath tusharmath changed the title chore(sync): remove file sync log spam chore: remove file sync log spam and unused logs command Apr 26, 2026
@tusharmath tusharmath enabled auto-merge (squash) April 26, 2026 04:54
@tusharmath tusharmath merged commit 8f7744d into main Apr 26, 2026
8 checks passed
@tusharmath tusharmath deleted the remove-high-usage-logs branch April 26, 2026 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: chore Routine tasks like conversions, reorganization, and maintenance work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant