Skip to content

chore: use tsx for hot reload#30

Merged
weaponsforge merged 1 commit into
devfrom
chore/PHR-29
Apr 18, 2026
Merged

chore: use tsx for hot reload#30
weaponsforge merged 1 commit into
devfrom
chore/PHR-29

Conversation

@weaponsforge
Copy link
Copy Markdown
Owner

@weaponsforge weaponsforge commented Apr 18, 2026

Summary

  • Use tsx for hot reload and uninstall nodemon

Related Issues

Type of Change

  • Refactor

Checklist

  • I have read the CONTRIBUTING.md
  • I have tested my changes locally
  • I have linked relevant issues

Summary by CodeRabbit

  • Chores
    • Streamlined development environment configuration by replacing nodemon with direct TypeScript execution, improving development server startup and file-watching capabilities.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0a43d84c-ef4a-486f-aa0f-6c61a15c54d0

📥 Commits

Reviewing files that changed from the base of the PR and between 007bd10 and a305c8e.

⛔ Files ignored due to path filters (1)
  • server/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (3)
  • server/nodemon.debug.json
  • server/nodemon.json
  • server/package.json
💤 Files with no reviewable changes (2)
  • server/nodemon.json
  • server/nodemon.debug.json

📝 Walkthrough

Walkthrough

This PR removes nodemon configuration files and migrates the development runtime from nodemon to tsx's built-in file-watching capabilities. Development scripts now use tsx watch for local development and tsx watch --inspect-brk for Docker debugging, with nodemon removed as a development dependency.

Changes

Cohort / File(s) Summary
Nodemon Configuration Removal
server/nodemon.debug.json, server/nodemon.json
Deleted nodemon debug and standard configuration files that previously managed TypeScript file watching and process restarting.
Development Scripts
server/package.json
Updated dev script to use tsx src/server.ts for file watching; updated docker:dev script to use tsx watch --inspect-brk=0.0.0.0:9229 src/server.ts for debugging; removed nodemon from devDependencies.

Possibly related issues

  • Issue #29: Directly addresses the migration from nodemon to tsx for hot-reload functionality by removing nodemon configurations and updating development scripts accordingly.

Poem

🐰 Farewell, nodemon's watchful eye,

TSX takes the wheel on high,

Faster reloads, scripts so spry,

Dev mode leaps into the sky! ✨


🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: use tsx for hot reload' accurately summarizes the main change: replacing nodemon with tsx for hot reload functionality.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/PHR-29

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@weaponsforge weaponsforge merged commit 8023b23 into dev Apr 18, 2026
2 checks passed
@weaponsforge weaponsforge deleted the chore/PHR-29 branch April 18, 2026 10:40
@weaponsforge weaponsforge mentioned this pull request Apr 22, 2026
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