Run any Skill in Manus with one click

refactor-preflight

Stars7

Forks0

UpdatedJune 13, 2026 at 05:53

Run the refactor guardrails (typecheck + knip dead-code + jscpd duplication) and report before opening a refactor PR. Use after extracting a use-case or before pushing any strangler-fig refactor branch, to confirm no dead code lingers and no copy-paste was introduced.

Installation

Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.

Run Skill in Manus

Source

IIIvan37

IIIvan37/pixsaur

View GitHub Repository View Creator Repositories

Download

Run Skill in Manus

Related occupationsSOC

Based on SOC occupation classification

Software DevelopersComputer and Mathematical Occupations·SOC 15-1252

SKILL.md

readonly

name	refactor-preflight
description	Run the refactor guardrails (typecheck + knip dead-code + jscpd duplication) and report before opening a refactor PR. Use after extracting a use-case or before pushing any strangler-fig refactor branch, to confirm no dead code lingers and no copy-paste was introduced.

Refactor preflight

Quick, report-only health check for a refactor branch. Detectors are currently report-only (ratchet mode) — they inform, they don't gate. Read the output and act; don't dismiss new findings just because the build is green.

Run

export PATH="$HOME/.nvm/versions/node/v24.14.1/bin:$PATH"   # WSL: node not on PATH
corepack pnpm refactor:preflight

This runs tsc --noEmit, then knip --no-exit-code (dead code / orphan exports), then jscpd src (copy-paste). To run pieces individually: corepack pnpm check:dead / corepack pnpm check:dup.

How to read it

knip — focus on what THIS branch changed. A use-case extraction that left the old helper as an unused export is the failure mode we care about: delete it. Distinguish your new orphans from pre-existing legacy ones (the baseline is noisy — that's expected in ratchet mode).
jscpd — any new clone that spans the branches you just touched (standard/EGX/Mode-R, etc.) means shared logic should be one helper. Factor it. Pre-existing clones in untouched files are not this PR's job.

Before declaring done

Also confirm the standard gate is green:

corepack pnpm test
corepack pnpm check        # known pre-existing failure: @radix-ui import in settings-panel.tsx

Report a short summary: new dead code (deleted? y/n), new duplication (factored? y/n), tests/typecheck status. Don't pad — name only what changed.

When the step is finished (not just checked), close it with the /session-report skill so docs/refactor/STATUS.md and the session log stay the resumable source of truth.

More from this repository

same repository

new-feature-hexa

IIIvan37/pixsaur

Build a NEW vertical slice (use-case + ports + pure core) OUTSIDE-IN — start from the consumer's need (a use-case / acceptance test), let it pull the domain into existence, then wire the adapter. Use when adding a genuinely new feature capability. For carving an EXISTING component/atom into a use-case, use extract-use-case instead.

2026-07-057

quality-gate

IIIvan37/pixsaur

Run the pixsaur quality guardrails (typecheck + comprehensive typecheck + biome & custom guards + tests with coverage + knip dead-code + jscpd duplication + Stryker on the pure core) and report. Use before declaring any change done, before a commit, or before opening a PR. Unlike a greenfield gate, pixsaur is a mature codebase mid-refactor — some detectors are ratcheted (report-only baseline), not zero-tolerance. Know which is which before you act.

2026-07-057

react-doctor

IIIvan37/pixsaur

Use when finishing a feature, fixing a bug, before committing React code, or when the user types `/doctor`, asks to scan, triage, or clean up React diagnostics. Covers lint, accessibility, bundle size, architecture. Includes a regression check and a full local-triage workflow that fetches the canonical playbook.

2026-07-057

react-testing-patterns

IIIvan37/pixsaur

React component and hook testing idiom for pixsaur (Testing Library + Vitest + happy-dom). Use when writing or reviewing any *.spec.tsx — component rendering, user interaction, async assertions, renderHook. Complements tdd-cycle (which owns red-green-refactor); this skill owns HOW the React tests are written in this repo.

2026-07-057

session-report

IIIvan37/pixsaur

Close a refactor step with a resumable session report. Use at the end of every step/PR of the clean-archi refactor so a fresh session can pick up exactly where this one left off. Updates the canonical docs/refactor/STATUS.md and appends a dated report under docs/refactor/sessions/.

2026-07-057

tdd-cycle

IIIvan37/pixsaur

Drive a change through strict red-green-refactor TDD. Use for ANY change to the pure core (src/libs/** and src/domain/**) or any pure logic — write the failing test first, the minimal code to pass, then refactor under green. Enforces one-assertion-per-test and triangulation.

2026-07-057

name	refactor-preflight
description	Run the refactor guardrails (typecheck + knip dead-code + jscpd duplication) and report before opening a refactor PR. Use after extracting a use-case or before pushing any strangler-fig refactor branch, to confirm no dead code lingers and no copy-paste was introduced.

Refactor preflight

Run

export PATH="$HOME/.nvm/versions/node/v24.14.1/bin:$PATH"   # WSL: node not on PATH
corepack pnpm refactor:preflight

This runs tsc --noEmit, then knip --no-exit-code (dead code / orphan exports), then jscpd src (copy-paste). To run pieces individually: corepack pnpm check:dead / corepack pnpm check:dup.

How to read it

knip — focus on what THIS branch changed. A use-case extraction that left the old helper as an unused export is the failure mode we care about: delete it. Distinguish your new orphans from pre-existing legacy ones (the baseline is noisy — that's expected in ratchet mode).
jscpd — any new clone that spans the branches you just touched (standard/EGX/Mode-R, etc.) means shared logic should be one helper. Factor it. Pre-existing clones in untouched files are not this PR's job.

Before declaring done

Also confirm the standard gate is green:

corepack pnpm test
corepack pnpm check        # known pre-existing failure: @radix-ui import in settings-panel.tsx

Report a short summary: new dead code (deleted? y/n), new duplication (factored? y/n), tests/typecheck status. Don't pad — name only what changed.

When the step is finished (not just checked), close it with the /session-report skill so docs/refactor/STATUS.md and the session log stay the resumable source of truth.