Commit @c39948a9cf21cf4de2cfbf64237f577edb7e9235

임동욱 08-20

merge: console.error undefined 인자 제거

@c39948a9cf21cf4de2cfbf64237f577edb7e9235

1099710

c39948a

lib/http/backend-fetch.ts

--- lib/http/backend-fetch.ts

+++ lib/http/backend-fetch.ts


   timeoutMs?: number;
 };
 
-function communicationError(reason: string, detail: unknown): BackendResult<never> {
-  // 실패 상세는 서버 콘솔에만 남기고, 호출부·화면에는 일반화된 메시지만 전달한다.
-  console.error(`[backend-fetch] ${reason}`, detail);
+/**
+ * 실패 상세는 서버 콘솔에만 남기고, 호출부·화면에는 일반화된 메시지만 전달한다.
+ *
+ * 이슈: `detail`이 `undefined`일 때 `console.error(msg, undefined)`로 부르면 안 된다. 콘솔을
+ *   계측하는 개발 도구(예: VS Code 확장 Console Ninja)가 인자를 훑다가
+ *   `Cannot read properties of undefined (reading 'stack')`으로 터지고, 그러면 이 함수가
+ *   값을 돌려주지 못해 `backendFetch`가 결과 대신 예외를 던진다. 인자 자체를 넘기지 않는다.
+ */
+function communicationError(
+  reason: string,
+  detail?: unknown
+): BackendResult<never> {
+  if (detail === undefined) {
+    console.error(`[backend-fetch] ${reason}`);
+  } else {
+    console.error(`[backend-fetch] ${reason}`, detail);
+  }
   return {
     ok: false,
     code: COMMUNICATION_ERROR_CODE,

     }
 
     return communicationError(
-      `파일 응답의 예상치 못한 HTTP 상태: ${response.status}`,
-      undefined
+      `파일 응답의 예상치 못한 HTTP 상태: ${response.status}`
     );
   }
 

       };
     }
 
-    return communicationError(`예상치 못한 HTTP 상태: ${response.status}`, undefined);
+    return communicationError(`예상치 못한 HTTP 상태: ${response.status}`);
   }
 
   // 본문 없는 2xx를 허용한 호출은 먼저 글자를 읽어 비었는지 본다 — 비어 있으면 `response.json()`이

       };
     }
     return communicationError(
-      `업로드 응답의 예상치 못한 HTTP 상태: ${response.status}`,
-      undefined
+      `업로드 응답의 예상치 못한 HTTP 상태: ${response.status}`
     );
   }
 

Add a comment

Open 0
Closed 0

List

...	...	@@ -113,9 +113,23 @@
113	113	timeoutMs?: number;
114	114	};
115	115
116		-function communicationError(reason: string, detail: unknown): BackendResult<never> {
117		- // 실패 상세는 서버 콘솔에만 남기고, 호출부·화면에는 일반화된 메시지만 전달한다.
118		- console.error(`[backend-fetch] ${reason}`, detail);
	116	+/**
	117	+ * 실패 상세는 서버 콘솔에만 남기고, 호출부·화면에는 일반화된 메시지만 전달한다.
	118	+ *
	119	+ * 이슈: `detail`이 `undefined`일 때 `console.error(msg, undefined)`로 부르면 안 된다. 콘솔을
	120	+ * 계측하는 개발 도구(예: VS Code 확장 Console Ninja)가 인자를 훑다가
	121	+ * `Cannot read properties of undefined (reading 'stack')`으로 터지고, 그러면 이 함수가
	122	+ * 값을 돌려주지 못해 `backendFetch`가 결과 대신 예외를 던진다. 인자 자체를 넘기지 않는다.
	123	+ */
	124	+function communicationError(
	125	+ reason: string,
	126	+ detail?: unknown
	127	+): BackendResult<never> {
	128	+ if (detail === undefined) {
	129	+ console.error(`[backend-fetch] ${reason}`);
	130	+ } else {
	131	+ console.error(`[backend-fetch] ${reason}`, detail);
	132	+ }
119	133	return {
120	134	ok: false,
121	135	code: COMMUNICATION_ERROR_CODE,
...	...	@@ -201,8 +215,7 @@
201	215	}
202	216
203	217	return communicationError(
204		- `파일 응답의 예상치 못한 HTTP 상태: ${response.status}`,
205		- undefined
	218	+ `파일 응답의 예상치 못한 HTTP 상태: ${response.status}`
206	219	);
207	220	}
208	221
...	...	@@ -285,7 +298,7 @@
285	298	};
286	299	}
287	300
288		- return communicationError(`예상치 못한 HTTP 상태: ${response.status}`, undefined);
	301	+ return communicationError(`예상치 못한 HTTP 상태: ${response.status}`);
289	302	}
290	303
291	304	// 본문 없는 2xx를 허용한 호출은 먼저 글자를 읽어 비었는지 본다 — 비어 있으면 `response.json()`이
...	...	@@ -380,8 +393,7 @@
380	393	};
381	394	}
382	395	return communicationError(
383		- `업로드 응답의 예상치 못한 HTTP 상태: ${response.status}`,
384		- undefined
	396	+ `업로드 응답의 예상치 못한 HTTP 상태: ${response.status}`
385	397	);
386	398	}
387	399

Delete comment