Skip to content

fix(koshien): プロジェクトロード時の Ruby バージョン強制切替(v1)と警告を廃止する (#1037)#1039

Merged
takaokouji merged 1 commit into
developfrom
topic/autopilot-1037
Jul 12, 2026
Merged

fix(koshien): プロジェクトロード時の Ruby バージョン強制切替(v1)と警告を廃止する (#1037)#1039
takaokouji merged 1 commit into
developfrom
topic/autopilot-1037

Conversation

@smalruby3-editor-bot

Copy link
Copy Markdown

Summary

甲子園(koshien)拡張を含むプロジェクトをロードすると、Ruby バージョンが 2 のとき alert が表示され強制的に v1 に切り替わっていた。v1/v2 とも動作し v2 が正しいため、この警告と強制切り替えは不要。これを削除する。

Changes Made

  • src/lib/project-loader-utils.js: loadProjectWithChecks の koshien 検出分岐(vm.hasKoshienProject → alert + onSetRubyVersion('1'))を削除。未使用になった currentRubyVersion / onSetRubyVersion 引数をシグネチャ・JSDoc から除去。Mesh V1 自動移行と失敗時復元(fix(url-loader): 特定 Scratch プロジェクト URL が読み込めず初期画面に戻る #972)はそのまま維持
  • 全呼び出し元の配線を更新(rubyVersion / onSetRubyVersion の props / mapStateToProps / mapDispatchToProps / propTypes / 未使用 import を除去):
    • src/lib/url-loader-hoc.jsx
    • src/lib/sb-file-uploader-hoc.jsx
    • src/containers/google-drive-loader-hoc.jsx
    • src/containers/use-student-assignment.js(+ 呼び出し側 src/containers/classroom-modal.jsxdispatch 受け渡し)
  • 未使用 locale メッセージを削除: src/lib/shared-messages.tschangedRubyVersionByKoshiensrc/locales/{ja,ja-Hira,en}.js の該当エントリ(ja / ja-Hira は対で削除)。
  • VM 側 vm.hasKoshienProject / serialization/smalruby-migration.jsdetectKoshien は他用途があるため据え置き(本 issue は GUI 側の強制切替・警告の除去が目的)。

Test Coverage

  • test/unit/lib/project-loader-utils.test.js: 「koshien プロジェクト + rubyVersion=2 で alert せず・バージョンを変えない(hasKoshienProject を呼ばない)」を pin。旧「forces ruby version 1」テストを置換。
  • Mesh V1 自動移行・失敗時復元のテストは維持し全て pass。
  • cd packages/scratch-gui && npm run lint--max-warnings 0 + prettier)通過。
  • 関連 unit test(project-loader-utils / url-loader-hoc / sb-file-uploader-hoc / google-drive-state-management)16 件 pass。

Related Issues

Closes #1037

koshien プロジェクトのロード時に rubyVersion=2 だと alert を出して v1 に強制切替して

いたが、v1/v2 とも動作し v2 が正しいため不要になった。loadProjectWithChecks の

koshien 分岐を削除し、未使用になった currentRubyVersion/onSetRubyVersion 引数と、

各呼び出し元 (url-loader / sb-file-uploader / google-drive / use-student-assignment)

の配線、未使用 locale メッセージ changedRubyVersionByKoshien を除去した。

Mesh V1 自動移行と失敗時復元 (#972) は維持している。

Closes #1037

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@smalruby3-editor-bot smalruby3-editor-bot Bot added the 🤖 autopilot autopilot が処理対象にした Issue/PR(daemon が付与) label Jul 12, 2026
@smalruby3-editor-bot

smalruby3-editor-bot Bot commented Jul 12, 2026

Copy link
Copy Markdown
Author

🤖 autopilot status

field value
Status Review
AI Status
HITL Yes
Size middle

Linked issue #1037. Maintained by autopilot (single writer); do not edit.

@github-actions

Copy link
Copy Markdown

🚀 Preview deployed: https://smalruby.jp/smalruby3-editor/topic/autopilot-1037/

@smalruby3-editor-bot

Copy link
Copy Markdown
Author

[FYI] koshien の強制切替削除により、VM 側の vm.hasKoshienProject()packages/scratch-vm/src/virtual-machine.js)と detectKoshienserialization/smalruby-migration.js)は GUI からの呼び出し元が完全に無くなり実質デッドコードになりました(repo 全体 grep で残る参照は本 PR のテスト mock と定義自身のみ)。

本 issue #1037 の指示どおり「他用途があれば残す/無理に消さない」に沿って据え置きが正解です。将来 cross-package の整理を行う際の候補として記録するのみで、本 PR での対応は不要(コードは変更しません)。

@smalruby3-editor-bot

Copy link
Copy Markdown
Author

🤖 敵対的レビュー完了。指摘 1 件(Must 0 / Question 0 / FYI 1)。Must なし。実装は issue #1037 の要求(全ロード経路での koshien 強制切替・alert 廃止、未使用引数/props/import/locale の整理、ja/ja-Hira 対削除、Mesh V1 自動移行と #972 復元の維持)を過不足なく満たし、lint(--max-warnings 0 + prettier)と関連 unit test(project-loader-utils / url-loader-hoc / google-drive-state-management 計 14 件)が pass。全 4 呼び出し元(url-loader-hoc / sb-file-uploader-hoc / google-drive-loader-hoc / use-student-assignment)が更新済み、project-state.js はコメント参照のみで変更不要も正しく判断されています。人間レビューへ。

@smalruby3-editor-bot smalruby3-editor-bot Bot added the 🙋 HITL 人間の対応待ち(Project HITL=Yes の投影) label Jul 12, 2026
@smalruby3-editor-bot smalruby3-editor-bot Bot marked this pull request as ready for review July 12, 2026 18:47
@smalruby3-editor-bot smalruby3-editor-bot Bot removed the 🙋 HITL 人間の対応待ち(Project HITL=Yes の投影) label Jul 12, 2026
@smalruby3-editor-bot smalruby3-editor-bot Bot marked this pull request as draft July 12, 2026 18:47
@smalruby3-editor-bot

Copy link
Copy Markdown
Author

🤖 差し戻し(🙋 解除)を受けてレビューを再確認しました。

指摘 1 件(対応 0 / 返信 0 / 対応不要 1):

人間からの変更要求・質問コメントは無く、未対応の Must / Question もありません。コードに変更はありません。人間のマージ判断待ちです。

@smalruby3-editor-bot smalruby3-editor-bot Bot added the 🙋 HITL 人間の対応待ち(Project HITL=Yes の投影) label Jul 12, 2026
@smalruby3-editor-bot smalruby3-editor-bot Bot marked this pull request as ready for review July 12, 2026 18:54
@takaokouji takaokouji merged commit c6c1273 into develop Jul 12, 2026
9 checks passed
@takaokouji takaokouji deleted the topic/autopilot-1037 branch July 12, 2026 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖 autopilot autopilot が処理対象にした Issue/PR(daemon が付与) 🙋 HITL 人間の対応待ち(Project HITL=Yes の投影)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(koshien): プロジェクトロード時の Ruby バージョン強制切替(v1)と警告を廃止する

1 participant