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.