Skip to content

fix: リファクタリング後のインポートパス修正(62ファイル)#85

Closed
fukukei23 wants to merge 1 commit into
mainfrom
fix/issue-84-import-paths
Closed

fix: リファクタリング後のインポートパス修正(62ファイル)#85
fukukei23 wants to merge 1 commit into
mainfrom
fix/issue-84-import-paths

Conversation

@fukukei23
Copy link
Copy Markdown
Owner

Summary

  • リファクタリング(cdb0792, 790877c)で移動されたモジュールのインポートパスを全テストファイル・ソースファイルで更新
  • 48件のcollection error → 0件に解消
  • 62ファイル、255行の置換(実質的な動作変更なし)

移動マッピング

旧パス 新パス
agents.context_analyzer analyzer.context_analyzer
agents.github_pr_creator utils.github_pr_creator
agents.guardian_auto_reviewer guard.guardian_auto_reviewer
agents.patch_applier services.patch_applier
agents.policy_interface config.policy_interface
agents.test_generator_prompt utils.test_generator_prompt
agents.test_strategy utils.test_strategy
gradio_app.* archive.gradio_app.*
modules.* archive.modules.*
ui.nexus_dashboard archive.ui.nexus_dashboard
ui.self_healing_dashboard archive.ui.self_healing_dashboard
ventures.vc_agent archive.ventures.ventures.vc_agent

Test plan

  • python -m pytest tests/ -q → 5441 passed, 0 failed
  • mock.patch()内のパスも含め全更新
  • ソースファイル内の自己参照も修正(archive/内のllm_helper参照等)

Closes #84

🤖 Generated with Claude Code

リファクタリング(1be87e1, 005726b)でモジュールが移動されたが、
テストファイルとソースファイル内のインポートパスが更新されていなかった。

主な修正:
- agents → analyzer/config/guard/services/utils への移動に対応
- gradio_app/modules/ui/ventures → archive/ への移動に対応
- mock.patch()内のパスも含め全て更新

テスト結果: 5441 passed, 0 failed(修正前: 48 collection errors)

Closes #84

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@fukukei23
Copy link
Copy Markdown
Owner Author

Closing: main has diverged significantly (92+ commits of refactoring). Merge conflicts in most files. Will re-implement needed tests on current main when NexusCore becomes active again.

@fukukei23 fukukei23 closed this May 21, 2026
@fukukei23 fukukei23 deleted the fix/issue-84-import-paths branch May 21, 2026 20:14
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.

fix: リファクタリング後の48件のインポートエラー修正

1 participant