feat(request): Improve hono request output

[ysknsid25]

resolves: #51

• Pretty Format
• -o <path> and -O
• -J and --json
• -i and -I

Sample

app.ts

import { Hono } from 'hono'
import { readFile } from 'node:fs/promises'
import { join } from 'node:path'

const app = new Hono()

app.get('/', (c) => {
  return c.json({
    message: 'Hello',
  })
})

app.get('/text', (c) => {
  return c.text('This is a plain text response.')
})

app.get('/image.png', async (c) => {
  try {
    const imagePath = join(process.cwd(), 'src', 'image.png')
    const imageData = await readFile(imagePath)
    return c.body(imageData.buffer, 200, {
      'Content-Type': 'image/png',
    })
  } catch (error) {
    console.error('Error serving image:', error)
    return c.text('Image not found', 404)
  }
})

export default app

use this png image

Default(Pretty Format)

node ./dist/cli.js request ./src/app.ts

Binary Response & not save

node ./dist/cli.js request ./src/app.ts -P /image.png

JSON Response & -o <path>

node ./dist/cli.js request ./src/app.ts -o index.json

JSON Response & -O

node ./dist/cli.js request ./src/app.ts -P /text -O

Binary Response & -o <path>

node ./dist/cli.js request ./src/app.ts -P /image.png -o test.png

Binary Response & -O

node ./dist/cli.js request ./src/app.ts -P /image.png -O

-J and --json

node ./dist/cli.js request ./src/app.ts -J

-I

node ./dist/cli.js request ./src/app.ts -I

-i

node ./dist/cli.js request ./src/app.ts -J -i

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@hono/cli@58

commit: 253b360

[yusukebe]

Hey @ysknsid25

If it's ready to review, please ping me!

[ysknsid25]

@yusukebe
A Happy New Year @yusukebe 🎍
Okey, dokey 🫡

[ysknsid25]

@yusukebe
Sorry for the wait 🙇
Please review 🙏🙏🙏

[yusukebe]

With the current implementation, if the path is /, the file name will be blank, and an error will occur.

Error saving file: ENOENT: no such file or directory, open ''

[ysknsid25]

I tried setting a default filename. When request to / and direct --remote-name, / is named index.
What do you think?👀

[yusukebe]

Ideally, adding an extension that depends on the content type is best. For example, if the content is application/json, the name will be index.json.

Plus, it's not just about this issue; should we also check whether the same file name exists in the current directory and avoid overwriting?

[ysknsid25]

For json, html, xml, and txt, which are likely to be responses for /, we have tried to add extensions based on the contentType 👀

We have also implemented overwrite prevention.

[yusukebe]

@ysknsid25 Thank you! I left some comments.

Hey @usualoma Can you also review this, if you have time!

[usualoma]

Thank you for the great work!
I've added one comment.

[usualoma]

Thank you!

[yusukebe]

LGTM!

[yusukebe]

@ysknsid25

Looks good! Let's land it. Thank you for your great work!