fix: resolve tsconfig paths correctly when baseUrl is absent / baseUrl未指定時のtsconfig pathsの解決を修正

[kota-shidara]

TypeScript 7以降に対応するための提案になります。よろしくお願いします！

背景

• tsconfigにおいて、TypeScript 6.0 で baseUrl が非推奨化され、TypeScript 7.0 では完全に廃止されるため、paths は baseUrl なしで書く必要があります。
• resolveImportPath.js の現状の実装では、解決後のエイリアスに ./ が残ります。例: @/domains/foo → ./src/domains/foo
• .eslintrc の module 設定では ./ 無しで書くため、./ で解決されたものとマッチせず、該当エイリアスを使うimportに対してルールが機能しなくなっていました。

具体的な再現例:

// tsconfig.json
{
  "compilerOptions": {
    "paths": { "@/*": ["./src/*"] }
  }
}

// .eslintrc: src/domains/payment 配下からしかimportを許可しない設定
"strict-dependencies/strict-dependencies": ["error",
  [{ "module": "src/domains/payment", "allowReferenceFrom": ["src/domains/payment"] }]
]

// 本来は違反として検出されるべきimport
// src/pages/checkout.tsx
import { chargeCard } from '@/domains/payment/chargeCard'

paths の値に付いた ./ のせいで、このimportは ./src/domains/payment/chargeCard と解決されます。
module の isMatch は startsWith で判定しますが、"./src/..." は "src/domains/payment" で始まらないため false となり、違反として報告されませんでした。

対応

• baseUrl 未指定時のデフォルトを "." とし、値が必ず path.join を通るようにしました。

path.join('.', './src/*')  // → 'src/*'   ./ が正規化され、module 設定とマッチするようになる
path.join('.', 'src/*')    // → 'src/*'   ./ を持たない既存の値は変化しない

./ は path.join が落とすため、解決結果が module 設定と同じ形式に揃います。./ のない既存の設定は結果が変わりません。

テスト

baseUrl なしで paths の値に ./ が付くケースを追加しました。

相談

コメントとして相談を記載しております！ご確認よろしくお願いします。
#198 (comment)

[kota-shidara]

[相談]
このテストに関して、消すべきかを迷いました。存在しても害ではないのですが、TypeScriptの仕様としてありえないものとなっているため、削除したほうがテストケースとしてはよさそうに思います。

pathに関するドキュメントで、baseUrlが存在しない場合は、tsconfigを起点とする相対pathを指すと書かれています。
また、baseUrlが存在しない状態で、このテストのように相対pathではない形で書くと、error TS5090: Non-relative paths are not allowed when 'baseUrl' is not set. Did you forget a leading './'?のエラーが発生します。

よってこのテストの状態は現実的に起こり得ないと思っており、削除した方がよいのではないかと考えています。

このPRの中で行うのでもよいですし、別の方が良ければ（既存のテストが全て壊れていない安心感等もあったりすると思うので）、このままにしておこうと思います。