Use `dynamicCompileOptions` instead of `compilerOptions.runes` for `create`
メンテナーはふだん 4 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 72/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- nodejs, typescript
調査の方向性
scaffolder のテンプレートから sveltekit({ compilerOptions: { runes: ... } }) ブロックを検索してください。これは payload が挙げている唯一のコードです。issue にある dynamicCompileOptions の形式に置き換え、任意の node:path から sep への変更も行うべきか確認してください。生成されたプロジェクトがアプリのコードには runes を強制し、node_modules には強制しない状態になれば完了です。プロジェクトを生成して svelte.config.js を確認することで検証できます。
索引モデルが issue の本文から書いたものです。
説明
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) {
...
- 主要言語
- TypeScript
- スター
- 513
- フォーク
- 110
- 平均マージ
- 5日 1時間
- マージ済み PR（30日）
- 24
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
sveltejs/cli のほかの issue
-
[bug] Better Auth addon generates legacy D1 binding access with SvelteKit 3 and Cloudflare adapter 8オープンbug pkg:add
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
sveltejs/cli#1450 · コメント 2 件 ·
メンテナーはふだん 4 日以内に返信
-
pkg:add
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
sveltejs/cli#1404 · コメント 1 件 ·
メンテナーはふだん 4 日以内に返信
-
pkg:add
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
sveltejs/cli#1399 · コメント 2 件 ·
メンテナーはふだん 4 日以内に返信
-
pkg:migrate
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
sveltejs/cli#1461 · コメント 2 件 ·
メンテナーはふだん 4 日以内に返信
-
enhancement pkg:migrate
難易度 3/5 1〜2日 初心者へのやさしさ 53/100
sveltejs/cli#1440 · コメント 3 件 · リアクション 1 件 ·
メンテナーはふだん 4 日以内に返信
似ている issue
-
area:docs documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 12 日以内に返信
-
area:docs good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
thainanluiz/TSSuper#30 ·
-
[analytics-core] getQueryParams does not decode `+` as space and truncates values containing `=`オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
amplitude/Amplitude-TypeScript#2179 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
ci good first issue severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
TMHSDigital/GH-Arcade#111 ·