【当日ページ】開催中、adminユーザーによるアイコン削除 #266

[YasunariIguchi]

不必要なアイコンをpostmanで削除できたことを受けて、イベントのユーザーグループのadminユーザーであればアイコンを削除できる機能を追加しました。

但し、実装したのはclaude codeです。
動作は問題なさそうです。

[atomisu0312]

いくつか気になった点をまとめておきます。

[atomisu0312]

0. CLAUDE.mdおよびドキュメントの整備

0 -> 1でコード生成するなら、ある程度規約を設定してあげたほうが良かったのかなと思いました。

[atomisu0312]

1 adminチェックの経路について

isAdminのチェックの際に、DBに問い合わせをしているのは分かりましたが、わざわざAPIを噛ませる必要ありますかね？
類似する処理だと、src/actionsを経由していたはずです。

[atomisu0312]

2. NonDraggableIconについて (fetch)

外部システムとのやりとりは基本的にserver componentsで定義していたはずです。
NonDraggableIconをclient componentに変えた上で、さらにfetchさせる必要性はあったのでしょうか。

せめてserver Actionで実行する処理をラップする方式にするとかしてもいい気がします。

[atomisu0312]

3. partyKitの修正について

プリフライトリクエストに対する応答はいいとして、
全部のレスポンスにcorsHeaderが追加されていますが、どうしても必要なんでしたっけ？

[atomisu0312]

4. front/src/lib/apiに色々追加しすぎ問題

front/src/lib/apiはapiを投げるときに使う、所謂お助け処理を用意していたはずです(front/src/lib/api/gyazo.tsなど)。
そこに、外部システムへの連携を書かれてもって気がします。

（本来はfront/src/lib/apiに外部システム連携を書くのが、実は正しいのかもしれませんが、統一感がなくなるので。。。）

[atomisu0312]

5. isAdminの判定タイミングについて

roomに紐づくeventがisAdminかどうかは、ページを開くタイミング（1回）で問題ないはずです。
今の実装だと、動かないアイコンの数だけDBに問い合わせるので、冗長にみえます。