← ClaudeAtlas

internal-reviewlisted

Reviews the local branch diff before QA and before any push — catches layering, coverage, style, and hygiene defects that QA cannot see. Posts nothing outside the workspace.
vsdudakov/troika · ★ 1 · Code & Development · score 62
Install: claude install-skill vsdudakov/troika
# Internal review (pre-PR) Pre-push local diff review. **Kind** procedure · **Used by** [reviewer](../../agents/reviewer.md) · **When** dev roles report done, before the tests run and before any push (develop-flow step 4) · **Ends with** a verdict file and a loop back to the dev role, nothing posted outside the workspace Read-only: no edit, push, or test. Set `TROIKA_WORKSPACE`. ## 1. Requirements Read plan and dev work log; they define scope. <a id="diff-new-files"></a> ## 2. Diff From the dev role's worktree. `<BASE>` is the profile's base ref, resolved once ([worktree › The base ref](../worktree/SKILL.md#base-ref)) — never hardcoded: ```bash BASE=<remote>/<default-branch> # PROFILE.md › Branches git fetch "${BASE%%/*}" git status --short # untracked files show as ?? git add -N -- . # intent-to-add, so new files enter the diff git --no-pager diff "$BASE"...HEAD # committed work git --no-pager diff # uncommitted, now including new files git --no-pager diff --stat "$BASE"...HEAD ``` Account for every `??` — an untracked, unreviewed file ships unseen. `git add -N` records intent-to-add so new files enter the diff; their content stays unstaged and release commits normally afterwards. Review the whole diff and needed context. <a id="runner"></a> ### Running this pass on the other family Preferred for independent review. **The tool and the exact c