loader: make registerPath concurrency-safe and add registerPaths (port of dpp.vim PR #39)

[Copilot]

Ports dpp.vim PR #39 to ddu.vim: makes registerPath safe under concurrent calls and adds a batch registerPaths API.

loader.ts

• Removed private #register helper and Mod type
• registerPath restructured:
  • Fast-path: returns immediately if path is already in #checkPaths
  • Deno.stat + importPlugin run outside the lock — concurrent calls proceed in parallel
  • State update (#exts, #checkPaths) happens inside the lock with a re-check to handle the race where multiple callers pass the fast-path simultaneously
  • Deno cache warning (console.warn) preserved on import failure
• registerPaths(type, paths) added: registers all paths concurrently via Promise.allSettled, logs per-path errors without rethrowing

// concurrent registrations of the same path are now safe
await Promise.all([
  loader.registerPath("source", path),
  loader.registerPath("source", path), // no-op after first wins the lock
]);

// batch registration — partial failures don't abort the rest
await loader.registerPaths("source", [pathA, pathB, badPath]);

Tests (denops/ddu/tests/)

New loader_concurrent_test.ts covering:

• Single-path registration succeeds
• Concurrent same-path registration is idempotent (8 parallel calls)
• Concurrent different-path registration all succeed
• Non-existent path throws
• registerPaths succeeds with all valid paths
• registerPaths swallows errors from invalid paths without throwing
• Concurrent registerPaths calls with overlapping paths are idempotent
• Re-registration after success is a no-op

Fixture modules (testdata/source_alpha.ts, source_beta.ts, source_gamma.ts) export a minimal Source class matching ddu.vim's type system ("source" instead of dpp.vim's "ext").

Original prompt

以下の変更を dpp.vim の PR #39 (Shougo/dpp.vim#39) と同様に各リポジトリへ適用してください。

対象リポジトリ: Shougo/ddc.vim, Shougo/ddu.vim, Shougo/ddx.vim, Shougo/ddt.vim

変更内容（実装・テスト両方）:

1. denops/*/loader.ts の改修

   • 既存の private helper (#register 相当) を置き換え、registerPath を以下の振る舞いにする:
     • 早期パス（fast-path）: path が既に登録済みなら即座に return する。
     • 実際の I/O と import はロックの外で行う（同時実行される別スレッドと並列に処理させる）。
     • import 成功後に内部状態（#exts と #checkPaths 等）を更新する処理のみをロック下で行い、同時に他の呼び出しが同じ path を登録していないか再チェックする。
     • import 時に Deno のモジュールキャッシュ問題が検出された場合は dpp.vim の実装と同じ警告（console.warn によるメッセージ）を出す。
   • registerPaths(paths: string[]) を追加し、複数パスを並列で registerPath に渡して登録する。失敗したパスは console.error に出力しつつ他の登録は続行する。
   • 既存の公開 API（registerExtension, getExtension 等）の外部挙動は変えない。

2. テスト追加

   • denops/*/tests ディレクトリに dpp.vim と同様のテストファイル（loader_concurrent_test.ts 相当）を追加し、以下を検証するテストを含める:
     • registerPath: 単一パスの登録が成功すること
     • registerPath: 同一パスの同時登録が冪等（いずれも成功しエラーにならない）であること
     • registerPath: 異なる複数パスの同時登録が成功すること
     • registerPath: 存在しないパスを登録すると例外が投げられること
     • registerPaths: 複数パス登録が成功すること、かつ一部無効なパスがあっても全体で例外を投げないこと
     • registerPaths の同時呼び出しが冪等であること
     • 再登録（成功後の再呼び出し）は no-op であること
     • テスト用フィクスチャ（testdata/*.ts）が読み込み可能であること
   • dpp.vim と同様に簡易の fixture モジュール（Ext/Protocol の minimal クラス）を denops/*/tests/testdata に追加する。

3. 既存のコードスタイル・型・パスに合わせて微調整

   • 各リポジトリの loader.ts の実際の名前空間・型定義（Ext/Protocol のクラス名や DppExtType 相当の命名）は各プロジェクトに合わせて調整してください。
   • 既存の importPlugin / isDenoCacheIssueError / EXT_ENTRY_POINT_FILE / Lock 等のユーティリティが各リポジトリに存在する想定です。見つからない場合は同等の処理を持つファイルを探して適切に参照すること。

4. PR 作成に関するメタデータ

   • PR タイトル: "loader: make registerPath concurrency-safe and add registerPaths + tests (from dpp.vim PR Refactor expand item with async #39)"
   • PR 説明: dpp.vim の PR Refactor expand item with async #39 への参照を含め、上記の変更点を要約して記載してください。
   • ベースブランチ: main

重要な注意点:

• dpp.vim の変更をそのままコピペするのではなく、各リポジトリの既存 API と型に合わせて差し戻しがないように実装してください。
• 単体テストは Deno の環境で実行できることを想定し、package/import の相対パスや URL を正しく解決してください。

もし実装上の差異や追加で確認が必要な点が見つかった場合は、PR の説明にその差分と理由を明記し、該当箇所でコメントしてください。

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: Shougo/dpp.vim#39

これと同様の変更を ddc.vim, ddu.vim, ddx.vim, ddt.vim に適用してください。

Assistant: 了解しました — これらのリポジトリに dpp.vim の PR #39 と同様の変更を適用して PR を作成できますが、いくつか確認させてください。

確認事項（短く答えてください）:

1. リポジトリのオーナーは「Shougo」でよいですか？（対象は Shougo/ddc.vim, Shougo/ddu.vim, Shougo/ddx.vim, Shougo/ddt.vim で良いか）
2. 各リポジトリで PR を直接作成してよいですか？（はい → こちらでブランチ作成・コミット・PR を作ります）
3. PR のベースブランチは main（または master）でよいですか？（指定がなければ main を想定します）
4. PR タイトル／説明は自動で「Apply loader concurrency and registerPaths changes (from dpp.vim PR Refactor expand item with async #39)」のように作成して良いですか。別文面が良ければ教えてください。

上の確認をいただければ、承諾後に各リポジトリへ同様の変更を適用して PR を作成します。

User: はい

Assistant: 先ほどのツール呼び出しのダイアログで承認をお願いします — 承認を確認次第、各リポジトリへの変更適用とPR作成を開始します。

User: @copilot Accepted Confirmation: Confirm agent session

Assistant: 開始しました — ご依頼の変更適用作業を開始し、進捗は下のタスクで追跡できます。今から各リポジトリ（Shougo/ddc.vim, Shougo/ddu.vim, Shougo/ddx.vim, Shougo/ddt.vim）に対して以下を順に行います：

• denops/*/loader.ts を dpp.vim の PR Refactor expand item with async #39 と同等の設計に合わせて書き換え（並列 import → ロックで状態更新、fast-path、registerPaths の追加、エラーメッセージは同様に保持）
• denops/*/tests に並列登録を検証するテストと testdata の fixture を追加
• ブランチを作成→コミット→PR を作成（タイトル/説明はご指定のものを使用）
• それぞれの PR リンクを作業完了ごとに報告

進めて問題があればここでお知らせください。追跡用タスクは以下です：

data:
    - title: '[WIP] Update denops loader.ts for improved path registration'
      repository: Shougo/ddc.vim
      url: https://github.com/copilot/tasks/pull/PR_kwDOFTvKk87P7dUW?session_id=bc7390b2-9739-4965-a027-447fefc6c1f2
      state: queued

次に、ddc.vim → ddu.vim → ddx.vim → ddt.vim の順で変更を適用して PR を作成します。PR 作成後に各 PR のリンクをここに貼ります。変更の内容や PR の本文で特に追記してほしい点があれば教えてください。

This pull request was created from Copilot chat.