質問一覧に関するvueファイル削除

[kushimegu]

Issue

• [Vue.js廃止] 質問一覧をvanilla JSに置き換える #8499

概要

質問一覧表示のvueを廃止した。このIssueに対してPR#8663でvueを利用しない仕組みを実装していたがvueファイルを削除していなかったため追加でPRを作成した。

変更確認方法

1. chore/delete-questions-vueをローカルに取り込む
2. foreman start -f Procfile.devでサーバーを立ち上げる
3. ユーザーhajimeでログインする
4. 質問一覧ページにアクセスし、質問一覧が表示されることを確認する
5. 未解決・解決済み・全ての質問をクリックするとそれぞれに対応して質問一覧が入れ替わることを確認する
6. OS X Mountain Lionをクリーンインストールするの質問一覧ページにアクセスし、全て・解決済み・未解決の質問に対応した質問一覧が表示されることを確認する
7. PC性能の見方を知るの質問一覧ページにアクセスし、全ての質問には「質問はありません」解決済みでは「解決済みのQ&Aはありません」未解決の欄では「未解決のQ&Aはありません」と表示されることを確認する
8. ユーザーごとの質問一覧ページに質問一覧が表示されることを確認する
9. ログアウトし、ユーザーhatsunoでログインする
10. ユーザーの質問一覧ページに「質問はありません」と表示されることを確認する

Screenshot

vue.js廃止のため画面の変更はない。

Summary by CodeRabbit

• Refactor
  • 質問一覧・質問カードの旧実装を撤去し、新しい実装へ切替
  • 起動時の旧動的コンポーネントマウント機構を廃止
• UI
  • 質問一覧の表示・ページネーションの挙動を更新
  • 読み込みプレースホルダーと空状態メッセージを改善
• Chores
  • 未使用資産を削除して読み込み負荷を軽減

[coderabbitai]

Walkthrough

Vue の動的マウント基盤（VueMounter）と関連する SFC（question.vue、questions.vue）を削除し、app/javascript/packs/application.js からそれらの import・登録・mount 呼び出しを除去した。ReactRailsUJS と Cocooned の初期化は維持される。

Changes

Cohort / File(s)	Summary of Changes
Vue マウント基盤の撤去 app/javascript/VueMounter.js, app/javascript/packs/application.js	VueMounter クラス（デフォルト輸出）の削除。application.js から VueMounter と関連コンポーネントの import／登録／mount 呼び出しを削除。
質問関連コンポーネント削除 app/javascript/components/question.vue, app/javascript/components/questions.vue	question.vue と questions.vue を丸ごと削除。クライアント側での質問一覧取得・ページング・レンダリングのロジックと Question/Questions コンポーネント定義が削除された。

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant B as Browser
  participant VM as VueMounter
  participant Qs as Questionsコンポーネント
  participant API as /api/questions.json

  Note over B,VM: 変更前（削除されたフロー）
  B->>VM: DOMContentLoaded (data-vue を検出)
  VM->>Qs: 登録済みコンポーネントをマウント
  Qs->>API: 質問一覧取得 (page/filters)
  API-->>Qs: JSON { questions, totalPages }
  Qs-->>B: 質問一覧をレンダリング（ページング含む）

Loading

sequenceDiagram
  autonumber
  participant B as Browser

  Note over B: 変更後
  B-->>B: Vue による自動マウント／質問一覧取得処理は存在しない

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• 質問一覧・詳細表示機能の非Vue化 #8663 — 同領域で Vue ベースの Questions/Question コンポーネント削除およびマウント機構撤去を行っているため、コードレベルで直接関連。
• ユーザーの提出物表示に関わる不要なvueファイルの削除 #8814 — 同じ VueMounter マウントフローから SFC を取り除く変更を含み、同様の import/registration 削除があるため関連性高い。

Suggested labels

雑事

Suggested reviewers

• kitarou888
• komagata

Poem

ぴょんと跳ねてケーブル断ち、Vueの森にさよならを。
古い枝は落とし、新しい道をつくるよ。
質問たちは別の風に乗り、ページは静かにめくられる。
うさぎは跳ねて差分を確認、OKの笛を吹くよ 🐇✨

Tip

🔌 Remote MCP (Model Context Protocol) integration is now available!

Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats.

---

📜 Recent review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 42df384 and 5ce8c0f.

📒 Files selected for processing (4)

• app/javascript/VueMounter.js (0 hunks)
• app/javascript/components/question.vue (0 hunks)
• app/javascript/components/questions.vue (0 hunks)
• app/javascript/packs/application.js (0 hunks)

💤 Files with no reviewable changes (4)

• app/javascript/components/question.vue
• app/javascript/packs/application.js
• app/javascript/VueMounter.js
• app/javascript/components/questions.vue

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build_and_test

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/delete-questions-vue

---

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

Status, Documentation and Community

• Visit our Status Page to check the current availability of CodeRabbit.
• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[kushimegu]

@kitarou888
概要に書いたのですが、以前作成したPRに不備があり追加でPRを作成しています。そちらのレビューをしていただいたkitarouさんに、よければこちらのレビューをお願いしたいのですがいかがでしょうか。

[kitarou888]

@kushimegu
わかりました。対応したいと思います。（私がレビューしたときはたしかにvueファイルは削除されていた気がするのですが…🤔）
コンフリクトを解消しましたらまたご連絡いただければと思います。

[kushimegu]

@kitarou888
すみません、コンフリクトの表示に気づかず…
解消できましたので改めてよろしくお願いします。

[kitarou888]

@kushimegu
Vue関連ファイルなしで正しく動作できていることを確認できましたのでApproveします。

[komagata]

Vue.jsがもう少しで削除できそうなので少し急いでいるので僕の方でレビューしちゃいますね。

確認させて頂きました。OKです〜🙆‍♂️

[kushimegu]

@kitarou888
レビューありがとうございました！

@komagata
遅くなってしまってすみません、ありがとうございます。