Back to Reactive Resume

Independently review plan 15A picture fitting

docs/execution/briefs/review-plan-15a.md

5.3.01.9 KB
Original Source

Independently review plan 15A picture fitting

Review only. Read pinned approved plan 15, current AGENTS.md, RTK, applicable code-review skill, implementation report, and complete origin/main...d891afd667dc571dcee642d54f851f6bde45fad8 diff. Revalidate live issue/PR/base state.

Review Standards and Spec axes. Verify:

  • schema/default/sample/v4 import compatibility defaults missing/invalid fit to Cover without corrupting data;
  • Cover retains current crop dialog, cancel/error/locked/autosave/undo behavior; Contain uploads full selected file through existing validated storage path and never implies already-cropped pixels can be restored;
  • sidebar and every PDF template route through one shared fit contract; semantic CSS precedence and borders/shadows stay correct; Cover output remains backward compatible;
  • controls are named, accessible, localized through correct catalog workflow, and generated docs/skill schema match source;
  • tests assert behavior rather than implementation, raster tolerances are meaningful, E2E does not add brittle global state, hardcoded local assumptions, unsafe cleanup, production-only dependencies, or a hidden network requirement;
  • exact issue #2782 scope; no claims for #3168/#3088/#2794 and no unrelated generated artifacts.

Run fresh base/live checks, focused schema/web/PDF/import tests, affected typechecks, boundaries, narrow Biome/catalog/docs, diff/scope. Inspect E2E source and run dedicated raster E2E when disposable DB/ports are safely available; otherwise state exact gate. Do not accept implementation report as proof without independent commands/probes.

Write .orchestration/plan-15a-review.md with findings first and file/line evidence, exact head, commands/results, skips, risks, and publication verdict. Do not edit tracked files, push, open PR, merge, mutate issues, or spawn subagents. Final response at most ten lines.