Skip to content

refactor(version-management): 移除 LLM 安装向导按钮及相关状态#27

Merged
newbe36524 merged 1 commit intoHagiCode-org:mainfrom
traceless929:main
Feb 27, 2026
Merged

refactor(version-management): 移除 LLM 安装向导按钮及相关状态#27
newbe36524 merged 1 commit intoHagiCode-org:mainfrom
traceless929:main

Conversation

@traceless929
Copy link
Contributor

@traceless929 traceless929 commented Feb 27, 2026

概要

  • 移除版本管理页面中的「Claude 安装」按钮
  • 移除 InstallationWizard 组件渲染及未使用的 llmWizardOpen 状态
  • 清理未使用的 InstallationWizard 导入

动机

LLM 安装向导按钮使用了硬编码的占位路径(/path/to/manifest.json),导致运行时出现 ENOENT 文件不存在错误。同时 llm:detect-config IPC 通道缺少对应的 handler 注册,控制台持续输出 No handler registered 错误。该功能尚未就绪,移除入口可保持界面整洁并消除不必要的运行时错误。

变更文件

  • src/renderer/components/VersionManagementPage.tsx
    • 移除 Claude 安装按钮
    • 移除 InstallationWizard 对话框渲染
    • 移除 llmWizardOpen 状态及 InstallationWizard 导入

测试

  • 版本管理页面正常渲染,无 Claude 安装按钮
  • 版本管理功能(安装、切换、卸载)无回归

Summary by CodeRabbit

  • Refactor
    • Removed the Claude installation UI button and installation wizard dialog from the version management page, streamlining the user interface while preserving underlying LLM functionality.

…ted state

Eliminate the LLM installation wizard and its associated state management from the Version Management Page for a cleaner interface. This change simplifies the component by removing unused code and improving maintainability.
@coderabbitai
Copy link

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c96a9c7 and b103a16.

📒 Files selected for processing (1)
  • src/renderer/components/VersionManagementPage.tsx

📝 Walkthrough

Walkthrough

This change removes the Claude installation UI button and LLM installation wizard dialog from VersionManagementPage, while retaining all underlying LLM-related state and logic. The modification simplifies the UI without affecting exported interfaces.

Changes

Cohort / File(s) Summary
UI Component Removal
src/renderer/components/VersionManagementPage.tsx
Removes Claude installation button and LLM installation wizard dialog. Eliminates llmWizardOpen state variable and InstallationWizard import while preserving all LLM-related logic (llmCallStatus, llmError, llmOutput, selectedRegion, llmVersionId).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A wizard once lived in the view,
But simple is better—we bid it adieu!
The logic remains, so sturdy and sound,
While UI grows cleaner, less cluttered around.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing the LLM installation wizard button and related state from the version management component.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@newbe36524 newbe36524 merged commit d7f000a into HagiCode-org:main Feb 27, 2026
4 checks passed
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