Skip to content

Conversation

@Nihal-a
Copy link

@Nihal-a Nihal-a commented Feb 10, 2026

Type: Micro-fix

This PR adjusts a debug logging statement to use the appropriate logging level.

Changes

  • Changed logger.info() to logger.debug() in core/framework/graph/executor.py:371
  • Removed redundant "🔍 Debug:" prefix from log message
  • Total lines changed: 1

Why This Qualifies as a Micro-fix

✅ < 20 lines changed (only 1 line)
✅ No logic, API, or database changes
✅ Linting/cleanup category (logging best practices)
✅ Zero risk of breaking changes

Per CONTRIBUTING.md, micro-fixes can be submitted without prior issue assignment.

Problem

Debug-level information was appearing in production logs at INFO level, adding noise to log output.

Solution

Use the appropriate debug logging level so the message only appears when debug logging is explicitly enabled.

Testing

  • ruff check passes (linting) - all files clean
  • ruff format --check passes (formatting) - 239 files already formatted
  • pytest tests/ passes - 683 tests passed, 0 failed

Checklist

  • Follow conventional commit format
  • Changes limited to < 20 lines
  • No functional changes
  • Lint and tests pass
  • Marked as micro-fix for fast-track review

[micro-fix,code-quality,logging,chore]

…) to logger.debug() for session resume state logging in executor.py line 371. Removes explicit Debug: prefix as it's redundant when using the proper logging level. This prevents debug-level information from appearing in production logs at INFO level, reducing log noise.
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