正規表現の不要なエスケープを一括除去するクリーンアップ

コードベース全体に散在していた正規表現の不要なエスケープ文字と、TypeScriptの型表記の不統一が修正されました。動作に影響を与えない純粋なクリーンアップですが、コードの可読性と正確性が向上しています。

背景

この変更は、oxfmt（フォーマッター）と oxlint（リンター）を用いたコードベースの静的解析から生まれました。PR作者がこれらのツールを試用したところ、複数のファイルで不要なエスケープが検出されました。正規表現内では、文字クラス [...] 内のほとんどの記号はエスケープ不要であり、誤ったエスケープはコードの意図を読み取りにくくする原因になります。

また、TypeScript の組み込み型において Symbol（コンストラクタ関数）と symbol（プリミティブ型）の混用も発見されました。型引数に Symbol を使うことは技術的には誤りであり、正しくは小文字の symbol を用いるべきです。

技術的な変更

変更は大きく3種類に分類できます。

1. 正規表現の文字クラス内の不要なエスケープ除去

バックスラッシュで保護する必要のない文字が複数箇所で誤ってエスケープされていました。/、.、?、# などは文字クラス [...] の内側では特殊な意味を持たないため、エスケープは不要です。

- let linkRegex = /<link rel="stylesheet" href="([a-zA-Z0-9\/_\.\?=%-]+)"/gi
+ let linkRegex = /<link rel="stylesheet" href="([a-zA-Z0-9/_.?=%-]+)"/gi

- const INLINE_STYLE_ID_RE = /[?&]index\=\d+\.css$/
+ const INLINE_STYLE_ID_RE = /[?&]index=\d+\.css$/

- if (/[\#\?].*\.svg$/.test(file)) {
+ if (/[#?].*\.svg$/.test(file)) {

2. 文字クラス外での不要なエスケープ除去

: は正規表現において特殊文字ではなく、文字クラス外でもエスケープ不要です。migrate-postcss.ts では \: が4箇所で : に修正されました。

- return /['"']?tailwindcss['"']?\: ?\{\}/.test(line)
+ return /['"']?tailwindcss['"']?: ?\{\}/.test(line)

3. RegExp コンストラクタ内でのエスケープ修正

infer-data-type.ts では、テンプレートリテラルで RegExp を構築する際に \s が単一のバックスラッシュで記述されており、正規表現エンジンに渡される文字列が s になっていました。

- const IS_FRACTION = new RegExp(`^${HAS_NUMBER.source}\s*/\s*${HAS_NUMBER.source}$`)
+ const IS_FRACTION = new RegExp(`^${HAS_NUMBER.source}\\s*/\\s*${HAS_NUMBER.source}$`)

これは唯一、動作に影響を与えうる修正です。修正前は空白文字のマッチに失敗していた可能性がありますが、テストスイートのパスにより意図した動作への修正であることが確認されています。

4. TypeScript の型表記の統一

canonicalize-candidates.ts では、型引数に使われていた Symbol（コンストラクタ関数の型）が、正しいプリミティブ型である symbol に統一されました。

- [UTILITY_SIGNATURE_KEY]: DefaultMap<SignatureOptions, DefaultMap<string, string | Symbol>>
+ [UTILITY_SIGNATURE_KEY]: DefaultMap<SignatureOptions, DefaultMap<string, string | symbol>>

- return new DefaultMap<string, string | Symbol>((utility) => {
+ return new DefaultMap<string, string | symbol>((utility) => {

- return new DefaultMap<string, string | Symbol>((variant) => {
+ return new DefaultMap<string, string | symbol>((variant) => {

さらに candidate.test.ts では、テンプレートリテラル文字列内の \_ というエスケープシーケンス（JavaScriptでは無効）を使ったテストケース2件が削除されました。これらは誤った記述であり、実際には _ として解釈されていたため、テスト対象の候補文字列が重複していました。

設計判断

ツール駆動のクリーンアップという手法が採用されています。oxlintのような静的解析ツールを既存コードベースに試用することで、人手によるレビューでは見逃されやすい細粒度の問題を効率的に発見しています。PRの説明文には「これらの依存関係を後でプロジェクトに追加するかもしれない」と記されており、今回の変更はツール導入の前段階として位置づけられています。

変更のほとんどは意味的に等価な置換ですが、IS_FRACTION の \\s 修正と candidate.test.ts の重複テスト削除は正確性の修正を含んでいます。これらを機能変更と切り分けず同一PRにまとめたことは、「全テストがパスする」という検証基準で安全性を担保しつつ、関連する整理を一度に行う判断として合理的です。

まとめ

この変更は、正規表現エスケープの誤りとTypeScript型の不正確な記述を一括で修正し、コードベースの品質を底上げするものです。特に IS_FRACTION のエスケープ修正のように、「不要なエスケープに見えて実は動作バグだった」ケースが含まれており、静的解析ツールの導入がもたらす副次的な価値をよく示しています。