訪問記録を付けた経由スポットを、訪問予定リストから消さず訪問済みにする

これまでは訪問を記録するとその経由スポットをリストから外していたが、
旅程は「その旅行で何を回ったか」の記録でもあるため、行が消えると後から
辿れなくなる。visit_plan_list_items に visited_at を足し、消す代わりに
訪問済みの印を付けるようにした(migrations/008)。

- 印は POST /api/visits が付ける。対象は本人の全リストで、既に印のある行は
  上書きしない(最初にそこへ行った時刻を残す)。かつてはクライアント側で、
  しかも地図でそのリストを経路表示しているときだけ動いていたため、
  リストを開かずに記録しても何も起きていなかった
- 訪問済みは経路から外す(地図の紫の矢印=buildPlanListPath と、リスト詳細の
  Google マップの経路検索)。済んだ場所を通り続ける線が残ると、次にどこへ
  行くかが読めなくなるため。リスト自体には残るので詳細では並んだままになる
- 印は手で付け外しできる(リスト詳細の各行のボタン →
  PATCH /api/visit-plan-lists/[id]/items/[spotId])。記録するほどでもない
  立ち寄りや、誤って付けた分を直せる
- PATCH /api/visit-plan-lists/[id] は経由スポットを丸ごと入れ替えるため、
  visited_at を控えてから入れ直す(戻さないと並び替えやタイトルの変更だけで
  訪問済みが消える)
- 経路の取り直しは onVisitRecorded を廃して onPlanListChange に一本化した。
  サーバーが印を付けるようになり、クライアントは読み直すだけでよくなったため。
  スポット一覧から記録したときも件数が更新されるようになった

1スポットごとの訪問予定(visit_plans)は従来どおり訪問記録で削除する
(ブックマークは行った時点で役目が終わるため)。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: claude

CLAUDE.md

@@ -0,0 +1,54 @@
+import { NextResponse } from "next/server";
+import { query } from "@/lib/db";
+import { getCurrentUserId } from "@/lib/auth/current-user";
+import type { VisitPlanList } from "@/lib/types";
+import { PLAN_LIST_COLUMNS } from "@/lib/visitPlanListSql";
+
+/**
+ * 訪問予定リストの経由スポット1件の「訪問済み」(visited_at)を付け外しする。
+ * 訪問記録を付けたときは POST /api/visits が自動で立てるが、記録するほどでもない
+ * 立ち寄りや、誤って付けた分をここで手直しできる。
+ *
+ * 更新後のリストをそのまま返すので、呼び出し側は取り直さずに画面へ反映できる。
+ */
+export async function PATCH(
+  request: Request,
+  { params }: { params: Promise<{ id: string; spotId: string }> }
+) {
+  const userId = await getCurrentUserId();
+  if (!userId) {
+    return NextResponse.json({ error: "unauthorized" }, { status: 401 });
+  }
+  const { id, spotId } = await params;
+
+  const body = await request.json();
+  if (typeof body?.visited !== "boolean") {
+    return NextResponse.json(
+      { error: "visited (boolean) は必須です。" },
+      { status: 400 }
+    );
+  }
+
+  // 本人のリストの経由スポットだけを更新する(他人のリストは404扱いで存在も伏せる)
+  const { rowCount } = await query(
+    `update visit_plan_list_items it
+        set visited_at = case when $3 then now() else null end
+       from visit_plan_lists l
+      where it.list_id = l.id
+        and l.id = $1 and l.user_id = $2 and it.spot_id = $4`,
+    [id, userId, body.visited, spotId]
+  );
+  if (!rowCount) {
+    return NextResponse.json({ error: "not found" }, { status: 404 });
+  }
+
+  const { rows } = await query<VisitPlanList>(
+    `select ${PLAN_LIST_COLUMNS}
+       from visit_plan_lists l
+       left join visit_plan_list_items i on i.list_id = l.id
+      where l.id = $1 and l.user_id = $2
+      group by l.id`,
+    [id, userId]
+  );
+  return NextResponse.json({ data: rows[0] });
+}

app/api/visit-plan-lists/[id]/items/[spotId]/route.ts

@@ -2,20 +2,13 @@ import { NextResponse } from "next/server";
 import { query } from "@/lib/db";
 import { getCurrentUserId } from "@/lib/auth/current-user";
 import type { VisitPlanList } from "@/lib/types";
+import { PLAN_LIST_COLUMNS } from "@/lib/visitPlanListSql";
 
 const DATE_RE = /^\d{4}-\d{2}-\d{2}$/;
 
 /** 指定リスト(本人)のspot_ids付き1件を返すSELECT(GET/PATCHの返却で共用) */
 const LIST_SELECT = `
-  select l.id, l.spot_type_id, l.title, l.description,
-         to_char(l.start_date, 'YYYY-MM-DD') as start_date,
-         to_char(l.end_date, 'YYYY-MM-DD') as end_date,
-         l.created_at, l.updated_at,
-         coalesce(
-           array_agg(i.spot_id order by i.seq)
-             filter (where i.spot_id is not null),
-           '{}'
-         ) as spot_ids
+  select ${PLAN_LIST_COLUMNS}
     from visit_plan_lists l
     left join visit_plan_list_items i on i.list_id = l.id
    where l.id = $1 and l.user_id = $2
@@ -98,7 +91,13 @@ export async function PATCH(
   );
 
   // 経由スポットは丸ごと置き換える(重複除去+存在するスポットに限定)。
-  // 地図で別スポット種別を重ねて追加できるため種別は問わない(itemsテーブルも種別非依存)
+  // 地図で別スポット種別を重ねて追加できるため種別は問わない(itemsテーブルも種別非依存)。
+  // 置き換えで行が作り直されるため、訪問済み(visited_at)は先に控えて後で戻す
+  // —— 戻さないと、リストの並び替えやタイトルの編集をしただけで訪問済みが消える
+  const before = await query<{ spot_id: string; visited_at: string | null }>(
+    "select spot_id, visited_at from visit_plan_list_items where list_id = $1",
+    [id]
+  );
   await query("delete from visit_plan_list_items where list_id = $1", [id]);
   const ordered = spotIds.filter((s, i) => spotIds.indexOf(s) === i);
   if (ordered.length > 0) {
@@ -110,6 +109,16 @@ export async function PATCH(
        on conflict (list_id, spot_id) do nothing`,
       [id, ordered]
     );
+    const visited = before.rows.filter((r) => r.visited_at !== null);
+    if (visited.length > 0) {
+      await query(
+        `update visit_plan_list_items it
+            set visited_at = v.visited_at
+           from unnest($2::uuid[], $3::timestamptz[]) as v(spot_id, visited_at)
+          where it.list_id = $1 and it.spot_id = v.spot_id`,
+        [id, visited.map((r) => r.spot_id), visited.map((r) => r.visited_at)]
+      );
+    }
   }
 
   const { rows } = await query<VisitPlanList>(LIST_SELECT, [id, userId]);

app/api/visit-plan-lists/[id]/route.ts

@@ -2,13 +2,15 @@ import { NextResponse } from "next/server";
 import { query } from "@/lib/db";
 import { getCurrentUserId } from "@/lib/auth/current-user";
 import type { VisitPlanList } from "@/lib/types";
+import { PLAN_LIST_COLUMNS } from "@/lib/visitPlanListSql";
 
 const DATE_RE = /^\d{4}-\d{2}-\d{2}$/;
 
 /**
  * 現在のユーザーの、指定スポット種別の訪問予定リスト一覧。各リストの経由スポットは
  * seq順の spot_ids(UUID配列)として返す(スポットの詳細は呼び出し側が保持済みの
- * 一覧から解決する)。
+ * 一覧から解決する)。訪問済みの経由スポットは spot_ids に残したまま
+ * visited_spot_ids にも入る。
  */
 export async function GET(request: Request) {
   const userId = await getCurrentUserId();
@@ -22,15 +24,7 @@ export async function GET(request: Request) {
   }
 
   const { rows } = await query<VisitPlanList>(
-    `select l.id, l.spot_type_id, l.title, l.description,
-            to_char(l.start_date, 'YYYY-MM-DD') as start_date,
-            to_char(l.end_date, 'YYYY-MM-DD') as end_date,
-            l.created_at, l.updated_at,
-            coalesce(
-              array_agg(i.spot_id order by i.seq)
-                filter (where i.spot_id is not null),
-              '{}'
-            ) as spot_ids
+    `select ${PLAN_LIST_COLUMNS}
        from visit_plan_lists l
        left join visit_plan_list_items i on i.list_id = l.id
       where l.user_id = $1
@@ -119,15 +113,7 @@ export async function POST(request: Request) {
   }
 
   const created = await query<VisitPlanList>(
-    `select l.id, l.spot_type_id, l.title, l.description,
-            to_char(l.start_date, 'YYYY-MM-DD') as start_date,
-            to_char(l.end_date, 'YYYY-MM-DD') as end_date,
-            l.created_at, l.updated_at,
-            coalesce(
-              array_agg(i.spot_id order by i.seq)
-                filter (where i.spot_id is not null),
-              '{}'
-            ) as spot_ids
+    `select ${PLAN_LIST_COLUMNS}
        from visit_plan_lists l
        left join visit_plan_list_items i on i.list_id = l.id
       where l.id = $1

app/api/visit-plan-lists/route.ts

@@ -80,15 +80,30 @@ export async function POST(request: Request) {
     throw e;
   }
 
-  // 訪問を記録したら、その場所は訪問予定リストから自動的に外す。ただし
-  // 日時なしの未訪問記録(=まだ行っていない下調べのメモ)は行きたい場所の
-  // ままなので外さない(日時ありの未訪問記録は「訪れたが改めて来たい」
-  // 記録のため、通常の訪問と同じく外す)
+  // 訪問を記録したら、その場所は訪問予定(行きたい場所のブックマーク)から
+  // 自動的に外す。ただし日時なしの未訪問記録(=まだ行っていない下調べのメモ)は
+  // 行きたい場所のままなので外さない(日時ありの未訪問記録は「訪れたが改めて
+  // 来たい」記録のため、通常の訪問と同じく外す)
   if (!(unvisited && !body.visited_on)) {
     await query("delete from visit_plans where user_id = $1 and spot_id = $2", [
       userId,
       body.spot_id,
     ]);
+    // 訪問予定リスト(旅程)側は行を消さず、訪問済みの印を付けるだけにする。
+    // 消してしまうと「その旅程で何を回ったか」が後から辿れなくなるため。
+    // 経路(地図の紫の矢印・Google マップの経路検索)からはこの印で外れる。
+    // 本人の全リストが対象で、既に印が付いている行は日時を上書きしない
+    // (最初にそこへ行った時刻を残す)
+    await query(
+      `update visit_plan_list_items it
+          set visited_at = now()
+         from visit_plan_lists l
+        where it.list_id = l.id
+          and l.user_id = $1
+          and it.spot_id = $2
+          and it.visited_at is null`,
+      [userId, body.spot_id]
+    );
   }
 
   return NextResponse.json({ data: rows[0] });

app/api/visits/route.ts

@@ -663,6 +663,9 @@ type RouteFeatureProps = {
 /**
  * 選んだ訪問予定リスト(旅程)の経路。そのリストのスポットをリスト順に並べる
  * (見えないスポット=未ダウンロード等は除いて残りを繋ぐ)。
+ * **訪問済みの経由スポットは経路に載せない** —— 済んだ場所を通り続ける線が
+ * 残ると「次にどこへ行くか」が読めなくなるため。リスト自体からは消えないので、
+ * 訪問予定リストの詳細では訪問済みとして並んだままになる。
  */
 function buildPlanListPath(
   planLists: VisitPlanList[],
@@ -672,7 +675,9 @@ function buildPlanListPath(
   if (!filters.planListId) return [];
   const list = planLists.find((l) => l.id === filters.planListId);
   if (!list) return [];
+  const visited = new Set(list.visited_spot_ids);
   return list.spot_ids
+    .filter((id) => !visited.has(id))
     .map((id) => spotById.get(id))
     .filter((s): s is Spot => s !== undefined);
 }
@@ -1381,31 +1386,6 @@ export default function MapView({
     [filters, setFilters, planLists, pathSpotById, fitMapToSpots]
   );
 
-  // 地図で訪問予定リストを経路表示中に、そのリスト内のスポットへ新しく訪問記録したら、
-  // 自動でそのスポットをリストから外す(訪問済みが経路に残り続けないように)。
-  // 表示中のリスト(filters.planListId)にそのスポットが含まれるときだけ動く
-  const handleVisitRecorded = useCallback(
-    async (spotId: string) => {
-      const listId = filters.planListId;
-      if (!listId) return;
-      const list = planLists.find((l) => l.id === listId);
-      if (!list || !list.spot_ids.includes(spotId)) return;
-      const nextSpotIds = list.spot_ids.filter((id) => id !== spotId);
-      const { data } = await api.visitPlanLists.update(listId, {
-        title: list.title,
-        description: list.description,
-        start_date: list.start_date,
-        end_date: list.end_date,
-        spot_ids: nextSpotIds,
-      });
-      setPlanLists((prev) =>
-        prev.map((l) =>
-          l.id === listId ? (data ?? { ...l, spot_ids: nextSpotIds }) : l
-        )
-      );
-    },
-    [filters.planListId, planLists]
-  );
   // マウント時と、マウント中に種別が切り替わった場合に、その種別の保存済み条件を読む
   useEffect(() => {
     setFiltersState(loadSavedFilters(spotTypeKey));
@@ -3793,7 +3773,6 @@ export default function MapView({
           allowPlanList
           onClose={() => setOverlayDetailSpotId(null)}
           onVisitChange={loadVisits}
-          onVisitRecorded={handleVisitRecorded}
           // 重ね表示スポットを現在の種別のリストへ追加したら、経路表示中のリストの
           // 線にも反映されるようリスト一覧を取り直す
           onPlanListChange={loadPlanLists}
@@ -3868,7 +3847,6 @@ export default function MapView({
           spots={spots}
           onClose={() => setDetailSpotId(null)}
           onVisitChange={loadVisits}
-          onVisitRecorded={handleVisitRecorded}
           // 既存の訪問予定リストへの追加をリスト一覧へ反映する(経路表示中の
           // リストに追加した場合、地図の紫の経路も引き直される)
           onPlanListChange={loadPlanLists}

components/MapView.tsx

@@ -89,7 +89,6 @@ export default function SpotDetailModal({
   allowPlanList = false,
   onClose,
   onVisitChange,
-  onVisitRecorded,
   onSpotChange,
   onSpotDeleted,
   onVisitPlanChange,
@@ -126,7 +125,6 @@ export default function SpotDetailModal({
   /** 新しい訪問記録が追加されたときだけ、そのスポットIDとともに呼ばれる
    * (地図で経路表示中の訪問予定リストから、訪問済みスポットを自動で外すのに使う。
    *  訪問記録の編集・削除では呼ばれない) */
-  onVisitRecorded?: (spotId: string) => void;
   /** スポット自体の編集・承認/却下で内容が変わったときに、変更後の内容とともに呼ばれる
    * (呼び出し元の一覧の再取得・公開スポットキャッシュの更新用) */
   onSpotChange?: (spot: Spot) => void;
@@ -858,11 +856,14 @@ export default function SpotDetailModal({
             // 訪問記録時、サーバー側で訪問予定からも自動的に外れる
             onVisitPlanChange?.();
             onReviewChange?.();
-            // 地図で経路表示中の訪問予定リストから、訪問済みスポットを自動で外す。
-            // 日時なしの未訪問記録(下調べ)はサーバー側でも訪問予定が残る
-            // (まだ行っていない)ため、リストからも外さない
+            // 訪問記録を付けると、サーバー側でその場所を含む訪問予定リストの
+            // 経由スポットに訪問済みの印が付き、地図の経路から外れる。印の付いた
+            // 状態を出すためにリストを取り直す(自分の「訪問予定」欄と、リストを
+            // 持っている呼び出し元の両方)。日時なしの未訪問記録(下調べ)は
+            // サーバー側でも印が付かない(まだ行っていない)ため取り直さない
             if (saved && !(saved.unvisited && !saved.visited_on)) {
-              onVisitRecorded?.(spot.id);
+              loadPlanLists();
+              onPlanListChange?.();
             }
           }}
         />
@@ -987,6 +988,12 @@ export default function SpotDetailModal({
             // 表示対象の差し替えを頼む(onOpenSpot未指定の呼び出し元では何もしない)
             if (id !== spotId) onOpenSpot?.(id);
           }}
+          // 訪問済みの付け外しは経路(地図の紫の矢印)の見え方を変えるため、
+          // リストを持っている呼び出し元にも取り直させる
+          onChanged={() => {
+            loadPlanLists();
+            onPlanListChange?.();
+          }}
         />
       )}
       {editingPlanList && typeKey && (

components/SpotDetailModal.tsx

@@ -619,6 +619,8 @@ export default function SpotsView({
                             {formatPlanDateRange(list.start_date, list.end_date)}
                             {" ・ "}
                             {list.spot_ids.length}スポット
+                            {list.visited_spot_ids.length > 0 &&
+                              `(訪問済み ${list.visited_spot_ids.length})`}
                           </p>
                         </div>
                         <span className="shrink-0 text-gray-400">›</span>
@@ -1020,6 +1022,7 @@ export default function SpotsView({
               setDetailListId(null);
               loadPlanLists();
             }}
+            onChanged={loadPlanLists}
             onOpenSpot={(id) => {
               setDetailListId(null);
               setDetailSpotId(id);