Review Architecture
Review a PR against the Pascal architectural rules, package boundaries...
Install
npx skills add pascalorg/editor --skill review-architecture -g -a claude-code -y- Repository
- pascalorg/editor25.3k
- Category
- Git and review
- Updated
- 5d ago
- License
- MIT
- Context cost
- Mediumabout 3,000 tokens
- Runs scripts
- No, instructions only
What it does
Review a PR against the Pascal architectural rules, package boundaries (core/viewer/editor/nodes), the registry-driven composition model (def.geometry / def.renderer / def.system), legacy-dispatch regressions, the slots + world-scale-UV convention for new nodes/geometry, hook hygiene (useEditor/useScene/useViewer), and selector performance. Use when the user asks to review a PR, audit a branch, or check that changes respect the codebase's architecture.
Architectural review for Pascal PRs. The user will provide a PR URL, branch name, or ask to review the current branch.
1. Load the rules (required, do not skip)
Read these before reviewing any diff. They are the source of truth, not your training data:
wiki/architecture/layers.mdwiki/architecture/systems.md, core systems vs viewer systems, what each may dowiki/architecture/renderers.md, renderer responsibilities and prohibitionswiki/architecture/tools.md, editor tools live only inapps/editor/components/tools/orpackages/nodes/src/<kind>/wiki/architecture/viewer-isolation.md, viewer must stay editor-agnosticwiki/architecture/node-definitions.md, the three-checkbox composition model (geometry/renderer/system)wiki/architecture/plugin-authoring.md, public contract for external node packswiki/architecture/agent-surfaces.md, MCP ↔ hosted chat ↔ skill parity. Read whenever the diff touchespackages/mcp/**orskills/**.wiki/architecture/scene-runtime.md, detached scenes, tracked reads,reconcileDocument. Read whenever the diff touchespackages/core/src/agent-operations/**,packages/core/src/store/**, a system that writes saved fields, orpackages/core/src/runtime/**.
Required on every review. Read the remaining pages on demand when the diff touches their subject area:
