Thanks.

• communication
  • I think we can communicate via the HotChannel (e.g. transport.send({ type: 'clientError' })) instead of a fetch request. This way we can re-use the same code when we send error information from other environments.
• error collection
  • I think we should listen the unhandledrejection event as well to catch the unhandled promise rejections.
• codeframe generation
  • generateCodeFrame function can be used for the code frame generation.
  • fetch(info.filename) can be probably replaced with moduleGraph.getModuleByUrl(id).transformResult
  • [optional] It seems colno and lineno contains the position for the transformed script. I think we need to map that back to the original code. That should be possible by a code similar to

    vite/packages/vite/src/node/ssr/ssrStacktrace.ts

    Lines 36 to 49 in f20addc

    const mod = moduleGraph.getModuleById(id)

    const rawSourceMap = mod?.transformResult?.map

    if (!rawSourceMap) {

    return input

    }

    const traced = new TraceMap(rawSourceMap as any)

    const pos = originalPositionFor(traced, {

    line: Number(line) - offset,

    // stacktrace's column is 1-indexed, but sourcemap's one is 0-indexed

    column: Number(column) - 1,

    })
    .
  • I think we can remove the code highlight feature for the following reasons:
    • Since this feature is mainly for LLMs, it doesn't help much.
    • We don't have that in other places.
    • @shijikijs/cli is huge and will made Vite's install size to be 1.5x.