Skip to content

feat(request): add watch option to request command#50

Merged
yusukebe merged 2 commits intomainfrom
feat/add-watch-option-to-request
Nov 26, 2025
Merged

feat(request): add watch option to request command#50
yusukebe merged 2 commits intomainfrom
feat/add-watch-option-to-request

Conversation

@usualoma
Copy link
Member

What is this?

Adds the --watch option to hono request.

When making changes while verifying detailed behavior using $ hono request, it would be convenient to incrementally build and review results with each save.

CleanShot.2025-11-24.at.14.25.53.mp4

Explanation of Changes

src/utils/build.ts

On the main branch, “.js” files were imported directly, but such cases are rare (most users likely use TypeScript). Since it's easier to let esbuild handle everything, including file monitoring, I've set it up to bundle and execute all files using esbuild.

@usualoma
Copy link
Member Author

Hi @yusukebe, how about something like this?

@yusukebe
Copy link
Member

Hi @usualoma

I like this feature! I think we can merge this immediately.

Hey @ysknsid25 ! Before merging, can you review the implementation of this PR? We don't have to look at the small details.

@ysknsid25
Copy link
Contributor

Okay, Dokey 🫡

@usualoma
Copy link
Member Author

8cf858c
The descriptions of the options lacked consistency, so I adjusted them.

@ysknsid25
Copy link
Contributor

@usualoma @yusukebe
I read this changes. it looks very good to me 😊

Copy link
Member

@yusukebe yusukebe left a comment

Choose a reason for hiding this comment

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

LGTM!

@yusukebe
Copy link
Member

@usualoma @ysknsid25 Thank you!!

Let's go with this. I'll merge and release a new version.

Regarding the next step of Hono CLI, I want to discuss this issue: #51

@yusukebe yusukebe merged commit e3464e7 into main Nov 26, 2025
3 checks passed
@yusukebe yusukebe deleted the feat/add-watch-option-to-request branch November 26, 2025 10:05
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.

3 participants