dsh-code-review
Reviewing a DeepSeek-Harness PR
This skill is guidance, not a complete checklist. Verify and fetch the PR's live base and exact head, then run pnpm --silent run change-scope --base <verified-base-ref> --head <verified-head-ref> before reading the diff and enough surrounding code to understand the design. The report identifies paths and dirty layers but does not replace semantic review. Re-establish the base and rerun it after a retarget or merge. Prioritize correctness, lifecycle, security, and broken required behavior over style; a short review with one substantiated blocker is better than a list of nits.
Read the requirements before judging the diff. Map the changed paths to the artifacts below, open each owning page from Sources of truth, and only then read the diff against those requirements. Judging a diff against memory of the rules is how a violation passes review.