Skip to content

fix: remove health check remove#371

Merged
shadowfax92 merged 1 commit intomainfrom
fix/server-clean-up-issue
Feb 27, 2026
Merged

fix: remove health check remove#371
shadowfax92 merged 1 commit intomainfrom
fix/server-clean-up-issue

Conversation

@shadowfax92
Copy link
Contributor

No description provided.

@shadowfax92 shadowfax92 merged commit 96797ed into main Feb 27, 2026
4 of 6 checks passed
@github-actions github-actions bot added the fix label Feb 27, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 27, 2026

Greptile Summary

Removed obsolete watchdog timer from health check endpoint. The watchdog previously terminated the process if no health checks were received within 5 minutes. This functionality is now handled more robustly by the CDP backend's keepalive mechanism (introduced in #365), which actively monitors the connection and handles reconnection gracefully.

  • Removed HEALTH_CHECK_TIMEOUT constant, watchdogTimer variable, and resetWatchdog() function
  • Cleaned up unused imports (EXIT_CODES, logger)
  • Health endpoint now simply returns status and CDP connection state
  • Version bumped to 0.0.61

This cleanup aligns with the codebase guideline to remove unused/dead code rather than leaving it in place.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • This is a straightforward cleanup that removes obsolete code. The watchdog functionality has been superseded by a more robust CDP keepalive mechanism. The changes are minimal, well-understood, and reduce code complexity without affecting functionality.
  • No files require special attention

Important Files Changed

Filename Overview
apps/server/src/api/routes/health.ts Removed obsolete watchdog timer logic, now handled by CDP backend keepalive

Last reviewed commit: c57d1eb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant