Add a recursion limit to improve stack overflow errors

Abierto
#55,687 6 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
25/100
Tipo de issue
Nueva funcionalidad
Claridad
Necesita aclaración
Estado de actividad
Estancado
Stack tecnológico
javascript, typescript
Área
compilers

Línea de trabajo

No se nombran archivos de implementación, pruebas ni puntos de entrada. Empieza revisando los comentarios vinculados de Visual Studio y el manejo actual de stack-overflow del compilador de TypeScript, y después determina el alcance y el diseño de un límite de recursión. El trabajo terminado debe incluir un diagnóstico mejor, con contexto recursivo útil, sin cambiar el comportamiento existente de JavaScript.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Awaiting More Feedback Suggestion
🔍 Search Terms

"recursion", "recursion limit", "stack overflow"

✅ Viability Checklist
  • This wouldn't be a breaking change in existing TypeScript/JavaScript code
  • This wouldn't change the runtime behavior of existing JavaScript code
  • This could be implemented without emitting different JS based on the types of the expressions
  • This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
  • This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals
⭐ Suggestion

It would be nice to have a recursion limit that allows a better error message in stack overflow errors like these, and that could potentially include more context about the recursive function.

📃 Motivating Example

This VS feedback item that shows how the error surfaced to the user is quite overwhelming.

💻 Use Cases
  1. What do you want to use this for? Debugging/fixing stack overflow errors.
  2. What shortcomings exist with current approaches? The error lists/output panes in editors aren't great at displaying huge stacks like these.
Lenguaje dominante
Go
Estrellas
111k
Forks
14.4k
Merge medio
1 d 19 h
PR fusionados (30 d)
117

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de microsoft/TypeScript

Todos los issues de microsoft/TypeScript

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.