From 761c7f197174e0dfbe303d0a9a3ed9954d39da57 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 22 Mar 2026 04:25:00 +0000
Subject: [PATCH] Changes before error encountered

Co-authored-by: pdf114514 <57948770+pdf114514@users.noreply.github.com>
Agent-Logs-Url: https://github.com/pdf114514/CarReservation/sessions/a42d4e36-a3cf-4ff7-b1cb-f076e601b1b8
---
 backend/server.js                             |  2 +-
 frontend/src/components/CarManagement.jsx     |  4 +-
 .../src/components/CarManagement.module.css   | 24 ++++++++-
 frontend/src/components/ScheduleView.jsx      | 18 ++++---
 .../src/components/ScheduleView.module.css    | 51 ++++++++++++++++---
 frontend/src/hooks/useWebSocket.js            | 26 ++++++++--
 frontend/vite.config.js                       |  6 +--
 7 files changed, 107 insertions(+), 24 deletions(-)

diff --git a/backend/server.js b/backend/server.js
index 392e090..66801f5 100644
--- a/backend/server.js
+++ b/backend/server.js
@@ -75,7 +75,7 @@ if (carCount.cnt === 0) {
 }
 
 // --- WebSocket Server ---
-const wss = new WebSocketServer({ server, path: '/ws' });
+const wss = new WebSocketServer({ server, path: '/api/ws' });
 
 function broadcast(message) {
   const data = JSON.stringify(message);
diff --git a/frontend/src/components/CarManagement.jsx b/frontend/src/components/CarManagement.jsx
index 96db33c..5202b17 100644
--- a/frontend/src/components/CarManagement.jsx
+++ b/frontend/src/components/CarManagement.jsx
@@ -275,8 +275,8 @@ export default function CarManagement({ reloadKey = 0 }) {
                             : car.inspection_expiry
                           : '-'}
                       </td>
-                      <td>{car.has_etc ? '🛣️ あり' : 'なし'}</td>
-                      <td>{car.tire_type === 'スタットレス' ? '❄️ スタットレス' : 'ノーマル'}</td>
+                      <td>{car.has_etc ? <span className={styles.badgeEtc}>ETC</span> : 'なし'}</td>
+                      <td>{car.tire_type === 'スタットレス' ? <span className={styles.badgeStudless}>スタッドレス</span> : 'ノーマル'}</td>
                       <td className={styles.actions}>
                         <button className={styles.btnEdit} onClick={() => startEdit(car)}>
                           編集
diff --git a/frontend/src/components/CarManagement.module.css b/frontend/src/components/CarManagement.module.css
index 4db27c7..849113c 100644
--- a/frontend/src/components/CarManagement.module.css
+++ b/frontend/src/components/CarManagement.module.css
@@ -79,7 +79,7 @@
   background: white;
   border-radius: 10px;
   box-shadow: 0 1px 4px rgba(0,0,0,0.08);
-  overflow: hidden;
+  overflow-x: auto;
 }
 
 .table {
@@ -218,3 +218,25 @@
   font-weight: 600;
 }
 
+.badgeEtc {
+  display: inline-block;
+  background: #7c3aed;
+  color: #fff;
+  font-size: 11px;
+  font-weight: 700;
+  padding: 2px 7px;
+  border-radius: 4px;
+  letter-spacing: 0.04em;
+}
+
+.badgeStudless {
+  display: inline-block;
+  background: #0ea5e9;
+  color: #fff;
+  font-size: 11px;
+  font-weight: 700;
+  padding: 2px 7px;
+  border-radius: 4px;
+  letter-spacing: 0.03em;
+}
+
diff --git a/frontend/src/components/ScheduleView.jsx b/frontend/src/components/ScheduleView.jsx
index 92f23bb..89a53ec 100644
--- a/frontend/src/components/ScheduleView.jsx
+++ b/frontend/src/components/ScheduleView.jsx
@@ -7,7 +7,7 @@ import ReservationModal from './ReservationModal.jsx';
 import styles from './ScheduleView.module.css';
 
 const CELL_WIDTH = 52; // px per day column
-const ROW_HEIGHT = 52; // px per car row
+const ROW_HEIGHT = 64; // px per car row
 const LABEL_WIDTH = 140; // px for car name column
 const HEADER_HEIGHT = 72; // px for the date header row
 const DAYS_SHOWN = 21; // number of days to show
@@ -443,13 +443,15 @@ export default function ScheduleView({ reloadKey = 0 }) {
                   title={car.description || car.name}
                 >
                   <span className={styles.carDot} style={{ background: color.border }} />
-                  <span className={styles.carName}>{car.name}</span>
-                  <span className={styles.carIcons}>
-                    {car.has_etc ? <span title="ETC あり">🛣️</span> : null}
-                    {car.tire_type === 'スタットレス' ? <span title="スタットレスタイヤ">❄️</span> : null}
-                    {isInspectionExpirySoon(car.inspection_expiry) ? (
-                      <span title={`車検満了日: ${car.inspection_expiry}（まもなく期限切れ）`}>⚠️</span>
-                    ) : null}
+                  <span className={styles.carLabelContent}>
+                    <span className={styles.carName}>{car.name}</span>
+                    <span className={styles.carBadges}>
+                      {car.has_etc ? <span className={styles.badgeEtc}>ETC</span> : null}
+                      {car.tire_type === 'スタッドレス' ? <span className={styles.badgeStudless}>スタッドレス</span> : null}
+                      {isInspectionExpirySoon(car.inspection_expiry) ? (
+                        <span className={styles.badgeWarn} title={`車検満了日: ${car.inspection_expiry}（まもなく期限切れ）`}>車検</span>
+                      ) : null}
+                    </span>
                   </span>
                 </div>
 
diff --git a/frontend/src/components/ScheduleView.module.css b/frontend/src/components/ScheduleView.module.css
index c222856..100bee3 100644
--- a/frontend/src/components/ScheduleView.module.css
+++ b/frontend/src/components/ScheduleView.module.css
@@ -175,6 +175,14 @@
   flex-shrink: 0;
 }
 
+.carLabelContent {
+  display: flex;
+  flex-direction: column;
+  gap: 3px;
+  min-width: 0;
+  flex: 1;
+}
+
 .carName {
   font-size: 13px;
   font-weight: 600;
@@ -182,15 +190,46 @@
   white-space: nowrap;
   overflow: hidden;
   text-overflow: ellipsis;
-  flex: 1;
-  min-width: 0;
 }
 
-.carIcons {
+.carBadges {
   display: flex;
-  gap: 2px;
-  font-size: 14px;
-  flex-shrink: 0;
+  gap: 4px;
+  flex-wrap: nowrap;
+}
+
+.badgeEtc {
+  background: #7c3aed;
+  color: #fff;
+  font-size: 10px;
+  font-weight: 700;
+  padding: 1px 5px;
+  border-radius: 4px;
+  white-space: nowrap;
+  letter-spacing: 0.03em;
+}
+
+.badgeStudless {
+  background: #0ea5e9;
+  color: #fff;
+  font-size: 10px;
+  font-weight: 700;
+  padding: 1px 5px;
+  border-radius: 4px;
+  white-space: nowrap;
+  letter-spacing: 0.03em;
+}
+
+.badgeWarn {
+  background: #d97706;
+  color: #fff;
+  font-size: 10px;
+  font-weight: 700;
+  padding: 1px 5px;
+  border-radius: 4px;
+  white-space: nowrap;
+  letter-spacing: 0.03em;
+  cursor: default;
 }
 
 /* Cell area */
diff --git a/frontend/src/hooks/useWebSocket.js b/frontend/src/hooks/useWebSocket.js
index caf6559..5420ac4 100644
--- a/frontend/src/hooks/useWebSocket.js
+++ b/frontend/src/hooks/useWebSocket.js
@@ -5,8 +5,9 @@ const WS_BASE = (() => {
   const loc = window.location;
   const proto = loc.protocol === 'https:' ? 'wss:' : 'ws:';
   // Use the same origin as the page; in development the Vite dev server
-  // proxies /ws to the backend, and in production nginx does the same.
-  return `${proto}//${loc.host}/ws`;
+  // proxies /api/ws to the backend, and in production nginx does the same
+  // via the existing location /api/ block (no extra nginx config needed).
+  return `${proto}//${loc.host}/api/ws`;
 })();
 
 const RECONNECT_DELAYS = [1000, 2000, 4000, 8000, 15000, 30000]; // ms
@@ -71,11 +72,30 @@ export default function useWebSocket(onMessage) {
   }, []);
 
   useEffect(() => {
+    // Reset on each mount so that React StrictMode's simulated unmount/remount
+    // does not leave unmountedRef permanently true and block reconnection.
+    unmountedRef.current = false;
     connect();
     return () => {
       unmountedRef.current = true;
       clearTimeout(retryTimerRef.current);
-      wsRef.current?.close();
+      const ws = wsRef.current;
+      if (ws) {
+        // Null out all handlers first to prevent any reconnect attempts.
+        ws.onopen = null;
+        ws.onclose = null;
+        ws.onerror = null;
+        ws.onmessage = null;
+        if (ws.readyState === WebSocket.CONNECTING) {
+          // Closing a CONNECTING socket triggers the browser warning
+          // "WebSocket is closed before the connection is established".
+          // Instead, schedule a close as soon as it opens so the server
+          // isn't left with a permanently idle connection.
+          ws.onopen = () => ws.close();
+        } else if (ws.readyState === WebSocket.OPEN) {
+          ws.close();
+        }
+      }
     };
   }, [connect]);
 
diff --git a/frontend/vite.config.js b/frontend/vite.config.js
index 096d712..80a2db0 100644
--- a/frontend/vite.config.js
+++ b/frontend/vite.config.js
@@ -5,10 +5,10 @@ import net from 'net';
 const backendOrigin = process.env.BACKEND_URL || 'http://localhost:3001';
 
 /**
- * Custom Vite plugin that tunnels WebSocket upgrade requests at /ws to the
+ * Custom Vite plugin that tunnels WebSocket upgrade requests at /api/ws to the
  * backend via raw TCP.  Vite's built-in proxy `ws: true` can silently drop
  * upgrade events that Vite's own HMR handler intercepts first.  This plugin
- * hooks directly onto `httpServer.upgrade` and handles the /ws path before
+ * hooks directly onto `httpServer.upgrade` and handles the /api/ws path before
  * Vite gets a chance to claim it.
  */
 function wsProxyPlugin() {
@@ -16,7 +16,7 @@ function wsProxyPlugin() {
     name: 'ws-proxy',
     configureServer(server) {
       server.httpServer?.on('upgrade', (req, socket, head) => {
-        if (req.url !== '/ws') return;
+        if (req.url !== '/api/ws') return;
 
         const { hostname, port: rawPort } = new URL(backendOrigin);
         const port = parseInt(rawPort) || 3001;