V2 config: ignore {file:...} in JSONC comments
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 70/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
Línea de trabajo
El problema está en la lógica de sustitución de tokens del analizador de configuración V2. Busque el código que maneja los tokens {file:...}, probablemente en un archivo como src/config/parser.ts o similar. La corrección debe omitir los tokens dentro de los comentarios JSONC (// y /* */), no solo las líneas que comienzan con //. Comience ejecutando los pasos de reproducción proporcionados para ver el fallo, luego examine la solicitud de extracción vinculada #50899 para comprender el contexto de la corrección prevista.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
In V2, a {file:...} token inside an inline // or /* ... */ comment in opencode.jsonc is treated as a real file reference. If that file does not exist, an otherwise valid config fails to load. The current substitution code skips only lines that begin with //.
Expected: comments are left alone; only active config values are substituted. Missing files in active values should still be errors.
Related #14478 concerned an active token shadowed by a commented duplicate in the older config path; here V2 dereferences tokens that occur only in comments.
Implementation: #50899 (targets v2).
Plugins
None
OpenCode version
V2 9c8a63e852722a9bced4a0de1179de58a85dfa20
Steps to reproduce
-
Create
opencode.jsoncwith:{ "username": "demo" // {file:./not-present.txt} } -
Load the configuration. It fails trying to read
not-present.txteven though the token is in a comment. A block comment containing the token has the same problem.
Screenshot and/or share link
Not applicable. A Core config regression test reproduces the failure with NotFound for missing-inline.txt before the fix.
Operating System
Linux x86_64
Terminal
Not applicable (Core test)
- Lenguaje dominante
- TypeScript
- Estrellas
- 209k
- Forks
- 27.5k
- Merge medio
- 9 h 53 min
- PR fusionados (30 d)
- 389
Preparar el entorno
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 anomalyco/opencode
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
anomalyco/opencode#51159 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
Todos los issues de anomalyco/opencode
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
diegosouzapw/OmniRoute#14869 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 94/100
Los mantenedores suelen responder en 1 día
-
status: waiting triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
freeCodeCamp/freeCodeCamp#70412 ·
Los mantenedores suelen responder en 1 día
-
Mend: dependency security vulnerability untriaged
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
opensearch-project/OpenSearch-Dashboards#12816 ·
Los mantenedores suelen responder en 1 día