feat(reference): extract_token_at_cursor を C# lexer 化（Phase 4-C / Refs #191）

[akiojin]

Summary

• SPEC feat(reference): unity-csharp-reference Phase 4 - 内部実装の改善 (umbrella) #191 (Phase 4 umbrella) のサブタスク Phase 4-C として、extract_token_at_cursor をミニ C# lexer 化する。
• Phase 3 (SPEC feat(reference): unity-csharp-reference Phase 3 - バージョン差分と LSP 連携実験 #188) で MVP land した regex ベースの token extractor は line comment (// ...) や string literal ("...") 内の identifier も拾ってしまう問題があった。resolve-symbol-at の偽陽性候補を抑制する。

Changes

• src/reference/mod.rs::is_cursor_in_comment_or_string を新設。カーソル位置までの 1 行を scan し、// line comment 内と "..." string literal 内かを判定する。\" エスケープは string を閉じない扱い。
• extract_token_at_cursor がカーソル位置の class を判定し、comment 内 / string 内なら None を返すように改良。
• 新規 RED テスト 4 件:
  • extract_token_at_cursor_skips_line_comment
  • extract_token_at_cursor_skips_string_literal
  • extract_token_at_cursor_returns_ident_after_closed_string
  • extract_token_at_cursor_handles_escaped_quote_in_string

Testing

• cargo fmt -- --check — clean
• cargo clippy --all-targets -- -D warnings — clean
• cargo test --bin unity-cli — 386 passed / 0 failed
• cargo llvm-cov --all-targets --summary-only -- --test-threads=1 — TOTAL line coverage 91.03%
• unity-cli skills lint --severity error — 15 skills / 0 violations

Closing Issues

• None (umbrella SPEC feat(reference): unity-csharp-reference Phase 4 - 内部実装の改善 (umbrella) #191 は完了せず、サブタスク追加でも生き続けるため Refs のみ)

Related Issues / Links

• Umbrella: feat(reference): unity-csharp-reference Phase 4 - 内部実装の改善 (umbrella) #191
• Phase 1-3: feat(reference): UnityCsReference のローカルキャッシュ参照機構（Phase 1） #185 / feat(reference): unity-csharp-reference Phase 2 - シンボル lookup インデックス #187 / feat(reference): unity-csharp-reference Phase 3 - バージョン差分と LSP 連携実験 #188 (MERGED)
• Phase 4-B: PR feat(reference): compute_line_diff を LCS / Myers 化（Phase 4-B / Refs #191） #193 (MERGED)

Checklist

• Tests added/updated (4 件)
• Lint/format passed (cargo clippy, cargo fmt, unity-cli skills lint)
• Documentation updated — none required (内部実装の改善で公開 API 変更なし)
• Migration/backfill plan included — Not applicable: 入出力の構造は不変、tokenName が null で返るケースが拡張されるだけ
• CHANGELOG impact considered — minor bump per Conventional Commits (feat)

Context

resolve-symbol-at は project file の cursor 位置から reference cache の候補を返す薄い wrapper。Phase 3 MVP では正規表現ベースの identifier 抽出だったため、LLM が cursor をコメントや文字列リテラル内に置いたときに偽の tokenName を返してしまい、candidates も虚しく埋まっていた。本 PR で line comment と string literal の 2 ケースを除外し、cursor が code 領域にあるときだけ token を抽出する。

Out of Scope

umbrella SPEC #191 で継続管理する範囲:

• block comment /* ... */ の multi-line 判定。現状は 1 行内のみ。
• char literal '...' や interpolated string $"..."、verbatim string @"..." の特例。
• preprocessor directive #if / #endif の中の cursor。
• Phase 4-A (member-level シンボル抽出) / Phase 4-D (reference fetch の zip fallback) / Phase 4-E (vector embedding) は umbrella の Tasks として未着手。

Notes

• block comment の multi-line 判定はファイル全体スキャンが必要で、本 PR では意図的にスコープアウト。LLM のカーソルが block comment 内に来るケースは line comment より稀との判断。

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3bfa7598-9e77-440e-a085-f7e3484c8292

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch work/20260511-0223

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}