From lisa
Code quality review checklist. Correctness, coding philosophy compliance, test coverage, documentation quality. Findings ranked by severity in plain English.
npx claudepluginhub codyswanngt/lisa --plugin lisaThis skill uses the workspace's default tool permissions.
Review code quality for changed files. Explain all findings in plain English as if speaking to someone with no programming background.
Guides Next.js Cache Components and Partial Prerendering (PPR) with cacheComponents enabled. Implements 'use cache', cacheLife(), cacheTag(), revalidateTag(), static/dynamic optimization, and cache debugging.
Guides building MCP servers enabling LLMs to interact with external services via tools. Covers best practices, TypeScript/Node (MCP SDK), Python (FastMCP).
Generates original PNG/PDF visual art via design philosophy manifestos for posters, graphics, and static designs on user request.
Review code quality for changed files. Explain all findings in plain English as if speaking to someone with no programming background.
For each changed file, evaluate:
let, no mutations, functional transformations)? Correct function structure (variables, side effects, return)?Rank findings by severity:
Broken logic or violates hard project rules.
Could cause problems later or reduce maintainability.
Minor improvements, not blocking.
For each finding:
What: The function changes the original list instead of creating a new one. Why: Other code using that list could see unexpected changes, causing hard-to-track bugs. Where:
src/utils/transform.ts:42Fix: Use[...items].sort()instead ofitems.sort()to create a copy first.
bun run test to confirm tests pass