Skip to content
This repository was archived by the owner on Dec 12, 2025. It is now read-only.

Improve modelfetch CLI with cleaner logging and native hot reload#99

Merged
phuctm97 merged 2 commits intomainfrom
improve-cli-logging
Aug 11, 2025
Merged

Improve modelfetch CLI with cleaner logging and native hot reload#99
phuctm97 merged 2 commits intomainfrom
improve-cli-logging

Conversation

@phuctm97
Copy link
Copy Markdown
Owner

Summary

  • Removed file-based logging system to reduce clutter and simplify the CLI
  • Refactored runtime detection to use shared constants for better maintainability
  • Added native hot reload support for Bun runtime using --hot flag, similar to Deno's implementation

Test plan

  • Test modelfetch serve command with Node.js runtime
  • Test modelfetch serve command with Deno runtime (with hot reload)
  • Test modelfetch serve command with Bun runtime (with hot reload)
  • Test modelfetch dev command with all runtimes
  • Verify no logging files are created in ~/.modelfetch/logs

🤖 Generated with Claude Code

Removed file-based logging system and refactored runtime detection to use shared constants. Added native hot reload support for Bun runtime using --hot flag, similar to Deno's --watch-hmr implementation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 11, 2025

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

Name Status Preview Comments Updated (UTC)
modelfetch-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 11, 2025 9:33am

@phuctm97 phuctm97 merged commit f0a9c3a into main Aug 11, 2025
1 of 2 checks passed
@phuctm97 phuctm97 deleted the improve-cli-logging branch August 11, 2025 09:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant