nodejs/cli-extension: resolving sdk paths and error log should be improved
- Lenguaje dominante
- Java
- Estrellas
- 10.5k
- Forks
- 1.5k
- Merge medio
- 1 d 11 h
- PR fusionados (30 d)
- 128
Descripción
1. Doing like this will result error: `Error: Cannot find package '@github/copilot-sdk' imported from ...`
```ts
import { joinSession } from "@github/copilot-sdk/extension.js";
```
instead of..
```ts
import { joinSession } from "@github/copilot-sdk/extension";
```
(`.js` difference)
2. Error handling should be more improved, at CLI `v1.0.12`, it shows only single line not stacktrace etc, so hard to debug. e.g:
```
[INFO] Launching extension: /tmp/copilot/test/.github/extensions/test/extension.mjs
[INFO] [extension:/tmp/copilot/test/.github/extensions/test/extension.mjs] [extension-bootstrap] Failed to load extension: SyntaxError: Unexpected token 'export'
[INFO] Extension exited: /tmp/copilot/test/.github/extensions/test/extension.mjs (code=1, signal=null)
[ERROR] Extension failed during startup: /tmp/copilot/test/.github/extensions/test/extension.mjs (code=1, signal=null)
```
```
[INFO] Found 1 extension(s), launching 1 (0 disabled)
[INFO] Launching extension: /tmp/copilot/test/.github/extensions/test/extension.mjs
[INFO] [extension:/tmp/copilot/test/.github/extensions/test/extension.mjs] [extension-bootstrap] Failed to load extension: Error: Invalid package config /tmp/copilot/test/.github/extensions/test/package.json.
[INFO] Extension exited: /tmp/copilot/test/.github/extensions/test/extension.mjs (code=1, signal=null)
[ERROR] Extension failed during startup: /tmp/copilot/test/.github/extensions/test/extension.mjs (code=1, signal=null)
```
Guía de contribución
Línea de trabajo
Reproduce el fallo de importación del SDK y los errores de inicio de la extensión usando los ejemplos del issue. Rastrea la resolución de rutas y el registro de errores de inicio de la extensión de la CLI; se considera terminado cuando la importación de la extensión funciona sin el sufijo .js y los fallos de inicio muestran un stack trace útil en lugar de solo un resumen de una línea.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- node.js, typescript
- Área
- cli, developer-experience
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100