Six axes make the score. Twenty-four diagnostics point at fixes.
감점 비중은 인지 복잡도 37 · 함수 길이 23 · 중복 9 · 파일 크기 8 · 순차 I/O 5 · O(n²) 5(캡 기준, 합 87). 나머지 24종은 등급을 건드리지 않고 파일·줄만 짚는다.
Caps by axis: complexity 37, function length 23, duplication 9, file size 8, sequential I/O 5, O(n²) 5 — 87 total. The other twenty-four never touch the grade; they name the file and line.
쓰기만 하는 컬렉션Write-only collection진단diagnostic머지 15건15 merged
채우기만 하고 아무도 읽지 않는 Set·Map·배열. 소비하던 코드가 리팩터로 사라진 흔적이라 지워도 동작이 같고, 매번 채우는 비용만 사라진다. knip 은 .add 도 "사용"으로 봐서 못 잡는다.A Set, Map, or array that is filled but never read — what a refactor left behind. Removing it changes nothing but the cost of filling it. knip misses it because .add counts as a use.
고치기 전Before
const seen = new Set();
-for (const n of nodes) {
- seen.add(n.id); // 아무도 안 읽음
- render(n);
-}
행마다 다른 배열을 .find·.some·.includes 로 처음부터 훑는 자리. 한 번 Map·Set 으로 색인해두면 조회가 O(1)이 된다 — 데이터가 커질수록 차이가 제곱으로 벌어진다.Each row rescans another array with .find/.some/.includes. Index it once into a Map/Set and each lookup is O(1) — the gap grows with the square of the data.
테스트·프론트 구역, 정적 외곽, n 이 잘린 자리는 세지 않고 3곳 미만은 면제. 10k줄당 유예 1.0, 캡 5 — 판정 난 초기 PR 6건 중 4건이 닫혀서, 오탐이 등급을 뒤집지 않게 낮게 잡았다.Test/frontend zones, static outers, and capped-n sites are excluded; under three sites is free. Free below 1.0 per 10k lines, capped at 5 — four of the first six decided PRs were closed, so a false positive must not flip a grade.
고치기 전Before
for (const row of rows) {
- const user = users.find((u) => u.id === row.userId);
-}
고친 후After
const byId = new Map(users.map((u) => [u.id, u]));
-for (const row of rows) {
- const user = byId.get(row.userId);
-}
루프 한 바퀴마다 DB·네트워크를 부르는 N+1, 그리고 서로의 결과를 쓰지 않는 await 를 줄 세우는 것. IN (...) 한 번이나 Promise.all 로 묶으면 왕복이 셈으로 줄어든다.A DB or network call per loop pass (N+1), and awaits that never use each other's result queued one by one. One IN (...) or a Promise.all cuts the round trips — provable by counting.
같은 결함이라 한 축. 37곳 재측정에서 기존 점수와 rho=−0.18 로 독립이고 11곳(30%)에서 발동한다. 3곳 미만은 안 센다(재시도·커서 페이지네이션은 순차가 맞다). 유예 3.0/1000파일, 캡 5.One defect, one axis. Across 37 re-measured repos it is independent of the score (rho=−0.18) and fires in 11 (30%). Under three sites is ignored — retries and cursor pagination are meant to be serial. Free below 3.0 per 1,000 files, capped at 5.
고치기 전Before
const user = await getUser(id);
-const posts = await getPosts(id);
버린 반환값Discarded pure call진단diagnostic머지 3건3 merged
s.replace(…)·arr.concat(…) 를 부르고 결과를 대입하지 않는다. 문자열은 불변이라 그 줄은 아무것도 안 한다 — 지우려던 줄이 남고, 붙이려던 말이 빠진다.Calls s.replace(…) or arr.concat(…) and drops the result. Strings are immutable, so the line does nothing — what it meant to remove stays, what it meant to append is lost.
고치기 전Before
slug.replace(/\s+/g, '-');
-if (!ok) new Error('bad slug');
고친 후After
slug = slug.replace(/\s+/g, '-');
-if (!ok) throw new Error('bad slug');
async 함수를 await 없이 부르고 결과를 버린다. 실패해도 아무도 모르고, 호출한 쪽은 끝나기 전에 다음 단계로 넘어간다.An async call with no await and nothing holding the result. Failures vanish, and the caller moves on before it finishes.
고치기 전Before
async function save(data) {
- writeCache(data);
- log('saved');
-}
고친 후After
async function save(data) {
- await writeCache(data);
- log('saved');
-}
전역 정규식 상태Stateful /g regex진단diagnostic머지 1건1 merged
/g 정규식을 공유한 채 루프에서 .test() 하면 lastIndex 가 다음 호출로 새어, 같은 입력에 참·거짓이 번갈아 나온다. 성능이 아니라 조용히 틀린 답이다.A shared /g regex used with .test() in a loop leaks lastIndex into the next call, so the same input flips between true and false. Not slow — silently wrong.
고치기 전Before
const TAG = /^v\d+/g;
-tags.filter((t) => TAG.test(t));
고친 후After
const TAG = /^v\d+/;
-tags.filter((t) => TAG.test(t));
한 번에 읽어야 하는 양. 중첩 함수·주석을 뺀 자기 코드 줄 기준, 40줄 초과(JSX 60줄) 비율. 파일을 쪼개도 안 변한다 — 그래서 조작이 안 된다.How much you must read at once — a function's own code lines, nested functions and comments removed. Splitting files doesn't move it, so it can't be gamed.
인지 복잡도Cognitive complexity채점 02scored 02
함수를 머리로 따라가는 부담. SonarSource S3776 스펙과 정본값까지 일치 검증.How hard a function is to follow. Matches SonarSource S3776, verified to canonical values.
중복Duplication채점 03scored 03
토큰 단위 복사·붙여넣기 밀도. 74개 실측에서 점수와 상관 −0.11이라 비중을 16→9로 줄였다 — 대부분에겐 0점, 소수에게만 큰 항목이다.Token-level copy-paste density. Correlates −0.11 with score across 74 repos, so its cap was cut 16 → 9: zero for most, heavy for a few.
파일 크기File size채점 04scored 04
평균 줄 수 + 대형 파일 비중. 보조 항으로 강등(27→8) — 같은 코드를 6파일로 쪼개기만 해도 옛 공식은 +27점을 줬다. 함수 길이와 상관 +0.16이라 버리진 않았다.Average lines + oversized-file share, demoted to a minor term (27 → 8): splitting identical code into six files used to gain +27. Kept, because it correlates only +0.16 with function length.
쓰기만 하는 컬렉션Write-only collection진단diagnostic머지 15건15 merged
채우기만 하고 아무도 읽지 않는 Set·Map·배열. 소비하던 코드가 리팩터로 사라진 흔적이라 지워도 동작이 같고, 매번 채우는 비용만 사라진다. knip 은 .add 도 "사용"으로 봐서 못 잡는다.A Set, Map, or array that is filled but never read — what a refactor left behind. Removing it changes nothing but the cost of filling it. knip misses it because .add counts as a use.
고치기 전Before
const seen = newSet();
+for (const n of nodes) {
+ seen.add(n.id); // 아무도 안 읽음
+ render(n);
+}
행마다 다른 배열을 .find·.some·.includes 로 처음부터 훑는 자리. 한 번 Map·Set 으로 색인해두면 조회가 O(1)이 된다 — 데이터가 커질수록 차이가 제곱으로 벌어진다.Each row rescans another array with .find/.some/.includes. Index it once into a Map/Set and each lookup is O(1) — the gap grows with the square of the data.
테스트·프론트 구역, 정적 외곽, n 이 잘린 자리는 세지 않고 3곳 미만은 면제. 10k줄당 유예 1.0, 캡 5 — 판정 난 초기 PR 6건 중 4건이 닫혀서, 오탐이 등급을 뒤집지 않게 낮게 잡았다.Test/frontend zones, static outers, and capped-n sites are excluded; under three sites is free. Free below 1.0 per 10k lines, capped at 5 — four of the first six decided PRs were closed, so a false positive must not flip a grade.
고치기 전Before
for (const row of rows) {
+ const user = users.find((u) => u.id === row.userId);
+}
고친 후After
const byId = newMap(users.map((u) => [u.id, u]));
+for (const row of rows) {
+ const user = byId.get(row.userId);
+}
루프 한 바퀴마다 DB·네트워크를 부르는 N+1, 그리고 서로의 결과를 쓰지 않는 await 를 줄 세우는 것. IN (...) 한 번이나 Promise.all 로 묶으면 왕복이 셈으로 줄어든다.A DB or network call per loop pass (N+1), and awaits that never use each other's result queued one by one. One IN (...) or a Promise.all cuts the round trips — provable by counting.
같은 결함이라 한 축. 37곳 재측정에서 기존 점수와 rho=−0.18 로 독립이고 11곳(30%)에서 발동한다. 3곳 미만은 안 센다(재시도·커서 페이지네이션은 순차가 맞다). 유예 3.0/1000파일, 캡 5.One defect, one axis. Across 37 re-measured repos it is independent of the score (rho=−0.18) and fires in 11 (30%). Under three sites is ignored — retries and cursor pagination are meant to be serial. Free below 3.0 per 1,000 files, capped at 5.
고치기 전Before
const user = awaitgetUser(id);
+const posts = awaitgetPosts(id);
버린 반환값Discarded pure call진단diagnostic머지 3건3 merged
s.replace(…)·arr.concat(…) 를 부르고 결과를 대입하지 않는다. 문자열은 불변이라 그 줄은 아무것도 안 한다 — 지우려던 줄이 남고, 붙이려던 말이 빠진다.Calls s.replace(…) or arr.concat(…) and drops the result. Strings are immutable, so the line does nothing — what it meant to remove stays, what it meant to append is lost.
async 함수를 await 없이 부르고 결과를 버린다. 실패해도 아무도 모르고, 호출한 쪽은 끝나기 전에 다음 단계로 넘어간다.An async call with no await and nothing holding the result. Failures vanish, and the caller moves on before it finishes.
전역 정규식 상태Stateful /g regex진단diagnostic머지 1건1 merged
/g 정규식을 공유한 채 루프에서 .test() 하면 lastIndex 가 다음 호출로 새어, 같은 입력에 참·거짓이 번갈아 나온다. 성능이 아니라 조용히 틀린 답이다.A shared /g regex used with .test() in a loop leaks lastIndex into the next call, so the same input flips between true and false. Not slow — silently wrong.
고치기 전Before
const TAG = /^v\d+/g;
+tags.filter((t) => TAG.test(t));
고친 후After
const TAG = /^v\d+/;
+tags.filter((t) => TAG.test(t));
한 번에 읽어야 하는 양. 중첩 함수·주석을 뺀 자기 코드 줄 기준, 40줄 초과(JSX 60줄) 비율. 파일을 쪼개도 안 변한다 — 그래서 조작이 안 된다.How much you must read at once — a function's own code lines, nested functions and comments removed. Splitting files doesn't move it, so it can't be gamed.
인지 복잡도Cognitive complexity채점 02scored 02
함수를 머리로 따라가는 부담. SonarSource S3776 스펙과 정본값까지 일치 검증.How hard a function is to follow. Matches SonarSource S3776, verified to canonical values.
중복Duplication채점 03scored 03
토큰 단위 복사·붙여넣기 밀도. 74개 실측에서 점수와 상관 −0.11이라 비중을 16→9로 줄였다 — 대부분에겐 0점, 소수에게만 큰 항목이다.Token-level copy-paste density. Correlates −0.11 with score across 74 repos, so its cap was cut 16 → 9: zero for most, heavy for a few.
파일 크기File size채점 04scored 04
평균 줄 수 + 대형 파일 비중. 보조 항으로 강등(27→8) — 같은 코드를 6파일로 쪼개기만 해도 옛 공식은 +27점을 줬다. 함수 길이와 상관 +0.16이라 버리진 않았다.Average lines + oversized-file share, demoted to a minor term (27 → 8): splitting identical code into six files used to gain +27. Kept, because it correlates only +0.16 with function length.
성능 7performance · 7정확성 버그 9correctness · 9
diff --git a/tools/update-impact-cards.py b/tools/update-impact-cards.py
index 6a2c144..c04a6ff 100644
--- a/tools/update-impact-cards.py
+++ b/tools/update-impact-cards.py
@@ -659,6 +659,26 @@ def card(f, key, rank=None):
}
+# 예제 코드 색 — 빌드 때 span 으로 칠한다(런타임 하이라이터 없음). 예제가 짧아 토큰 몇 종이면 충분하다.
+_JS_KW = {"const", "let", "var", "for", "of", "if", "return", "async", "await", "function", "new", "throw"}
+_JS_TOK = re.compile(r"(?P//[^\n]*)|(?P'[^'\n]*')|(?P(?<=[(=,]\s)/(?:\\.|[^/\n])+/[gimsuy]*|(?<=[(=,])/(?:\\.|[^/\n])+/[gimsuy]*)"
+ r"|(?P[A-Za-z_$][\w$]*)(?P(?=\())?|(?P\b\d+\b)")
+
+
+def _hl(code):
+ out, i = [], 0
+ for m in _JS_TOK.finditer(code):
+ out.append(esc(code[i:m.start()]))
+ kind = m.lastgroup if m.lastgroup != "call" else "w"
+ t = m.group(0)
+ if kind == "w":
+ kind = "k" if t in _JS_KW else ("f" if m.group("call") is not None else "")
+ out.append(f'{esc(t)}' if kind else esc(t))
+ i = m.end()
+ out.append(esc(code[i:]))
+ return "".join(out)
+
+
def _star_num(v):
if not v:
return 0
@@ -701,8 +721,8 @@ def _star_num(v):
chips = f'
