Use `dynamicCompileOptions` instead of `compilerOptions.runes` for `create`
Los mantenedores suelen responder en 4 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- nodejs, typescript
Línea de trabajo
Busca en las plantillas del scaffolder el bloque sveltekit({ compilerOptions: { runes: ... } }), que es el único código que menciona el payload. Reemplázalo por la forma dynamicCompileOptions del issue y comprueba si también conviene hacer el cambio opcional de node:path a sep. Estará listo cuando el proyecto generado siga forzando runes para el código de la aplicación, pero no para node_modules. Puedes confirmarlo generando un proyecto e inspeccionando su svelte.config.js.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
As I was experimenting with the project scaffolder, I noticed I could steal that compilerOptions.runes trick to make sure runes are enforced in Svelte Changelog.
But while copying it and reading the docs, I noticed they mention Vite users (SvelteKit) should prefer dynamicCompileOptions. dynamicCompileOptions docs also provide an (outdated but) explicit example for that kind of check.
As a result, the template should probably be changed:
sveltekit({
// Force runes mode for the project, except for libraries. Can be removed in Svelte 6.
- compilerOptions: {
- runes: ({ filename }) =>
- filename.split(/[/\\]/).includes('node_modules') ? undefined : true
+ dynamicCompileOptions: ({ filename, compileOptions }) => {
+ if (!filename.split(/[/\\]/).includes('node_modules') && !compileOptions.runes) {
+ return { runes: true };
+ }
},
adapter: adapter()
})
Additionally, Node (and Bun) environments can take advantage of "node:path".sep instead of a (non-precompiled) regex, improving performance:
+import { sep } from 'node:path';
...
- if (!filename.split(/[/\\]/).includes('node_modules') && !compileOptions.runes) {
+ if (!filename.split(sep).includes('node_modules') && !compileOptions.runes) {
...
- Lenguaje dominante
- TypeScript
- Estrellas
- 513
- Forks
- 110
- Merge medio
- 5 d 1 h
- PR fusionados (30 d)
- 24
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la 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 sveltejs/cli
-
[bug] Better Auth addon generates legacy D1 binding access with SvelteKit 3 and Cloudflare adapter 8Abiertobug pkg:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
sveltejs/cli#1450 · 2 comentarios ·
Los mantenedores suelen responder en 4 días
-
pkg:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
sveltejs/cli#1404 · 1 comentario ·
Los mantenedores suelen responder en 4 días
-
pkg:add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
sveltejs/cli#1399 · 2 comentarios ·
Los mantenedores suelen responder en 4 días
-
pkg:migrate
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
sveltejs/cli#1461 · 2 comentarios ·
Los mantenedores suelen responder en 4 días
-
clear `.svelte-kit` on migrationAbiertoenhancement pkg:migrate
Dificultad 3/5 1-2 días Aptitud para principiantes 53/100
sveltejs/cli#1440 · 3 comentarios · 1 reacción ·
Los mantenedores suelen responder en 4 días
Todos los issues de sveltejs/cli
Issues similares
-
area:docs documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 12 días
-
area:docs good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
thainanluiz/TSSuper#30 ·
-
[analytics-core] getQueryParams does not decode `+` as space and truncates values containing `=`Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
amplitude/Amplitude-TypeScript#2179 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
ci good first issue severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
TMHSDigital/GH-Arcade#111 ·