Feature request: Expose original source text for TSqlFragment (Debugger & API usability)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza con la API TSqlFragment y sus miembros existentes FirstTokenIndex, LastTokenIndex, ScriptTokenStream, StartOffset y FragmentLength. Revisa cómo representan los fragmentos su intervalo de origen y determina el comportamiento esperado del texto expuesto, incluida su utilidad para la depuración y para los consumidores. Se considerará terminado cuando un fragmento pueda proporcionar su texto de origen original sin que cada consumidor tenga que reconstruirlo manualmente, con la cobertura correspondiente allí donde el repositorio admita pruebas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
Working with Microsoft.SqlServer.TransactSql.ScriptDom is unnecessarily hard in practice,
especially during debugging.
TSqlFragment objects do not expose the original source text they represent.
Instead, consumers must manually reconstruct the SQL text via:
FirstTokenIndex / LastTokenIndexScriptTokenStream- or
StartOffset / FragmentLength
This makes debugging extremely cumbersome:
in Visual Studio, inspecting a fragment does not show the SQL code,
only a large object graph with token indices.
This significantly affects developer productivity and discoverability of the AST.
Current workaround (non-trivial)
Every consumer needs to implement helper code such as:
extension(TSqlFragment node)
{
public string SourcePart => string.Join("",
node
.ScriptTokenStream
.Skip(node.FirstTokenIndex)
.Take(node.LastTokenIndex - node.FirstTokenIndex + 1)
.Select(i => i.Text)
);
}
- Lenguaje dominante
- GAP
- Estrellas
- 277
- Forks
- 43
- Merge medio
- 6 d 17 h
- PR fusionados (30 d)
- 3
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoft/SqlScriptDOM
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
microsoft/SqlScriptDOM#228 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
microsoft/SqlScriptDOM#183 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
microsoft/SqlScriptDOM#226 · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
microsoft/SqlScriptDOM#225 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
microsoft/SqlScriptDOM#224 ·
Todos los issues de microsoft/SqlScriptDOM
Issues similares
-
[Bug]: Discord Activity titles with emoji are rejected as over 80 characters when they are not Abiertoclawsweeper:linked-pr-open clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:message-loss issue-rating: 🦞 diamond lobster maturity:stable P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
growthbook/growthbook#7100 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
punkpeye/mcp-remote#369 ·
-
Client customer-reported needs-team-attention question Service Attention WebPubSub
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Azure/azure-sdk-for-net#63292 · 3 comentarios · 1 reacción ·
-
bug p:3-mid pydanty:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
pydantic/pydantic-ai#8643 · 1 comentario ·