
review
by bdsqqq
SKILL.md
name: review description: "epistemic standards for evaluation and analysis. load before reviewing code, debugging, reporting findings, or any task where claims must be defensible. enforces trace-or-delete, confidence labeling, falsification."
review
epistemic standards for producing defensible findings.
when to load
- evaluating code, docs, or designs
- debugging / root cause analysis
- codebase archaeology
- any task where you will report findings to others
principles
-
trace or delete — every claim traces to code, logs, or data. if you can't show the evidence, delete the claim or label it a hunch.
- prevents: pattern → fact leap — "this looks like X" silently becomes "this IS X"
-
facts, not assumptions — "the code shows X" not "this is probably X". be specific: line numbers, exact conditions, concrete paths.
- prevents: vague language hiding weak evidence — hedging lets you avoid committing to what you actually know
-
label confidence — VERIFIED (traced), HUNCH (pattern recognition, not traced), QUESTION (needs input). never present hunches as findings.
- prevents: false certainty — unlabeled claims inherit unearned authority
-
falsify, don't confirm — design tests that would DISPROVE your hypothesis. ask: "what would make this NOT a bug?"
- prevents: confirmation bias — you naturally notice supporting evidence and ignore contradictions; tunnel vision locks you into your first theory
quality criteria
when evaluating contributions, check:
- proven correctness — have you seen it work? not "does the code look right" — have you actually run it?
- types tell the truth — are you lying to the compiler? do abstractions match their names?
- naming is honest — would someone reading this in six months be confused?
- edges tested — what happens on the worst path, not just the happy path?
- self-consistent abstractions — can you explain it start to finish, and each part in isolation?
slop indicators:
- missing tests
- contradictions in abstractions
- names that lie about what they contain
applying to findings
before reporting an issue:
1. can i cite the exact code location? (line numbers, file paths)
2. did i trace the actual conditions, or pattern-match?
3. did i try to prove myself wrong?
4. what's my confidence: VERIFIED / HUNCH / QUESTION?
if any answer is weak, either investigate more or label appropriately.
report format
## finding: <title>
**confidence:** VERIFIED | HUNCH | QUESTION
**location:** file:line or range
**evidence:** what the code actually shows
**falsification attempted:** what would disprove this, did i check?
counter-example (slop)
## finding: possible race condition
**confidence:** VERIFIED
**location:** src/cache/invalidator.ts
**evidence:** the code looks like it might have a race condition because invalidate() calls multiple async functions
**falsification attempted:** none
problems: confidence says VERIFIED but evidence is "looks like" (pattern-match, not trace). no line numbers. no falsification. this is pattern→fact leap.
example (correct)
## finding: race condition in cache invalidation
**confidence:** VERIFIED
**location:** src/cache/invalidator.ts:47-52
**evidence:** `invalidate()` awaits `fetch()` but not `write()`. concurrent calls can read stale data between L47 fetch completing and L52 write persisting. reproduced by adding 50ms delay to write() and calling invalidate() twice in quick succession — second call returns stale value.
**falsification attempted:** checked if a mutex guards the block (none). checked if write() is synchronous (it's not — returns Promise). confirmed issue by adding delay and observing stale read.
Score
Total Score
Based on repository quality metrics
SKILL.mdファイルが含まれている
ライセンスが設定されている
100文字以上の説明がある
GitHub Stars 100以上
3ヶ月以内に更新がある
10回以上フォークされている
オープンIssueが50未満
プログラミング言語が設定されている
1つ以上のタグが設定されている
Reviews
Reviews coming soon