Skip to content

fix(socket logs): remove debug sockets logs#532

Merged
icecrasher321 merged 2 commits into
mainfrom
fix/realtime
Jun 24, 2025
Merged

fix(socket logs): remove debug sockets logs#532
icecrasher321 merged 2 commits into
mainfrom
fix/realtime

Conversation

@icecrasher321
Copy link
Copy Markdown
Collaborator

Description

Remove sockets debug logs.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Code refactoring (no functional changes)

How Has This Been Tested?

No change to functionality

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • All tests pass locally and in CI (bun run test)
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have updated version numbers as needed (if needed)
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

  • My changes do not introduce any new security vulnerabilities
  • I have considered the security implications of my changes

Additional Information:

Any additional information, configuration or data that might be necessary to reproduce the issue or use the feature.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sim ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 2:32am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Skipped (Inspect) Jun 24, 2025 2:32am

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Improves code maintainability by removing debug-level socket logs from apps/sim/socket-server/index.ts, cleaning up verbose development logging while preserving essential error reporting.

  • Removed verbose socket authentication success logs
  • Cleaned up workflow operation tracking logs
  • Removed redundant subflow update logs
  • Eliminated unnecessary parallel block update logs

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

@icecrasher321 icecrasher321 merged commit ca134eb into main Jun 24, 2025
5 checks passed
@waleedlatif1 waleedlatif1 deleted the fix/realtime branch June 28, 2025 04:47
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.

1 participant