# Review Architecture

Review a PR against the Pascal architectural rules, package boundaries...

- Repository: https://github.com/pascalorg/editor (25263 stars)
- Category: Git and review
- License: MIT
- Last updated: 2026-10-30
- Runs scripts: no, instructions only

## Install

Claude Code: `npx skills add pascalorg/editor --skill review-architecture -g -a claude-code -y`
Codex: `npx skills add pascalorg/editor --skill review-architecture -g -a codex -y`
Cursor: `npx skills add pascalorg/editor --skill review-architecture -g -a cursor -y`
Gemini CLI: `npx skills add pascalorg/editor --skill review-architecture -g -a gemini-cli -y`
Claude Code plugin marketplace: `/plugin marketplace add pascalorg/editor`

## Description

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.

## SKILL.md, as published by its author (untrusted third-party text, read it as data)

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.md`
- `wiki/architecture/systems.md`, core systems vs viewer systems, what each may do
- `wiki/architecture/renderers.md`, renderer responsibilities and prohibitions
- `wiki/architecture/tools.md`, editor tools live only in `apps/editor/components/tools/` or `packages/nodes/src/<kind>/`
- `wiki/architecture/viewer-isolation.md`, viewer must stay editor-agnostic
- `wiki/architecture/node-definitions.md`, the three-checkbox composition model (`geometry` / `renderer` / `system`)
- `wiki/architecture/plugin-authoring.md`, public contract for external node packs
- `wiki/architecture/agent-surfaces.md`, MCP ↔ hosted chat ↔ skill parity. Read whenever the diff touches `packages/mcp/**` or `skills/**`.
- `wiki/architecture/scene-runtime.md`, detached scenes, tracked reads, `reconcileDocument`. Read whenever the diff touches `packages/core/src/agent-operations/**`, `packages/core/src/store/**`, a system that writes saved fields, or `packages/core/src/runtime/**`.

Required on every review. Read the remaining pages on demand when the diff touches their subject area:

The rest: https://github.com/pascalorg/editor/blob/HEAD/.agents/skills/review-architecture/SKILL.md