Skip to content

fix(lsp): remove stale switch expression in SymbolKindCode delegate - #118

Merged
akiojin merged 1 commit into
mainfrom
develop
Mar 11, 2026
Merged

fix(lsp): remove stale switch expression in SymbolKindCode delegate#118
akiojin merged 1 commit into
mainfrom
develop

Conversation

@akiojin

@akiojin akiojin commented Mar 11, 2026

Copy link
Copy Markdown
Owner

Summary

Release v0.5.0 のLSPビルド失敗を修正するホットフィックス。

Changes

  • lsp/Program.cs:1452 の不正な kind switch => 構文を除去(Server.Core への委譲リファクタ時の残骸)

Context

PR #116 マージ後のリリースワークフローで全LSPプラットフォームのビルドが失敗。
このマージ後にタグ v0.5.0 を再作成してリリースを再実行する。

Summary by CodeRabbit

リリースノート

このプルリクエストはコード内部の改善が含まれていますが、ユーザーに対する機能的な変更はありません。

  • Refactor
    • 内部のコード構造を改善し、コード保守性を向上させました。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@akiojin
akiojin enabled auto-merge March 11, 2026 06:33
@akiojin
akiojin merged commit a6da91a into main Mar 11, 2026
17 of 19 checks passed
@coderabbitai

coderabbitai Bot commented Mar 11, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 50b871dc-cd40-48c2-af9a-e95753906b08

📥 Commits

Reviewing files that changed from the base of the PR and between 32e9f5a and 50a6b00.

📒 Files selected for processing (1)
  • lsp/Program.cs

Walkthrough

lsp/Program.csSymbolKindCodeメソッドが、switch式を使った式形式のボディから、LspCodeIndexModels.SymbolKindCode(kind)へのデリゲートを実行する通常のメソッドボディに変更されました。機能的には同じ値を返します。

Changes

Cohort / File(s) Summary
メソッド実装の改善
lsp/Program.cs
SymbolKindCodeメソッドがswitch式から別のメソッドへのデリゲートに変更されました。公開インターフェースに影響なし。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 メソッドの衣を着替えて、
デリゲートの手へ優しく託す
同じ答えを返しながら、
コードはより清く、より正しく…
シンプルな改革、大きな調和 ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

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

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

@codecov-commenter

codecov-commenter commented Mar 11, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants