Skip to content

Conversation

@codewithkenzo
Copy link
Owner

@codewithkenzo codewithkenzo commented Feb 3, 2026

Removed 6 debug console statements from 4 files:

  • ast-grep/downloader.ts
  • system-notify/utils.ts
  • comment-checker/downloader.ts
  • cli/run/runner.ts

Scout-detected cleanup.

Greptile Overview

Greptile Summary

Cleanup PR removing 6 debug console statements from 4 files across CLI, hooks, and tools directories.

Changes:

  • src/cli/run/runner.ts:55 - Removed session ID log (session info still available via other outputs)
  • src/hooks/comment-checker/downloader.ts:130,168 - Removed download progress messages (debugLog still active via env var)
  • src/tools/ast-grep/downloader.ts:82,108 - Removed download progress messages (error logs remain)
  • src/tools/system-notify/utils.ts:18 - Removed debug message for missing /proc/version (expected on non-WSL Linux)

All removed logs were either redundant, debug-level, or superseded by existing error handling. No behavioral changes or edge case impacts.

Confidence Score: 5/5

  • Safe to merge - purely cosmetic removal of debug statements with no logic changes
  • All removed console statements were non-critical debug/progress messages. Error logging remains intact in all downloaders. No behavioral changes, edge cases, or security impacts.
  • No files require special attention

Important Files Changed

Filename Overview
src/cli/run/runner.ts Removed session ID console.log, still visible via other output
src/hooks/comment-checker/downloader.ts Removed 2 user-facing download progress messages, debugLog remains
src/tools/ast-grep/downloader.ts Removed 2 download progress console.logs, still has error logging
src/tools/system-notify/utils.ts Removed console.debug for missing /proc/version, silent fallback now

Sequence Diagram

sequenceDiagram
    participant User
    participant CLI as CLI Runner
    participant Downloader as Binary Downloaders
    participant Utils as System Utils
    
    Note over CLI,Utils: Console.log statements removed from all paths
    
    User->>CLI: bunx oh-my-opencode run
    CLI->>CLI: createOpencode()
    CLI->>CLI: session.create()
    Note over CLI: Removed: console.log(Session ID)
    CLI->>User: Continue execution silently
    
    alt Binary Download Needed
        User->>Downloader: ensureAstGrepBinary()
        Downloader->>Downloader: downloadAstGrep()
        Note over Downloader: Removed: "Downloading..." & "ready" logs
        Downloader-->>User: Return binary path (silent)
        
        User->>Downloader: ensureCommentCheckerBinary()
        Downloader->>Downloader: downloadCommentChecker()
        Note over Downloader: Removed: "Downloading..." & "ready" logs
        Downloader-->>User: Return binary path (silent)
    end
    
    alt Platform Detection
        User->>Utils: detectPlatform()
        Utils->>Utils: readFileSync("/proc/version")
        Note over Utils: Removed: console.debug on error
        Utils-->>User: Return platform type (silent)
    end
Loading

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Thank you for your contribution! Before we can merge this PR, we need you to sign our Contributor License Agreement (CLA).

To sign the CLA, please comment on this PR with:

I have read the CLA Document and I hereby sign the CLA

This is a one-time requirement. Once signed, all your future contributions will be automatically accepted.


I have read the CLA Document and I hereby sign the CLA


Kenzo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@codewithkenzo
Copy link
Owner Author

I have read the CLA Document and I hereby sign the CLA

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