diff --git a/ci-review-prompt.md b/ci-review-prompt.md index 38e769b04..b3befafa8 100644 --- a/ci-review-prompt.md +++ b/ci-review-prompt.md @@ -114,6 +114,14 @@ deployment, and operation paths instead of judging the changed hunk in isolation; flag contradictions between PR intent, code, docs, tests, schemas, generated files, UI rendering, and consumers. +When a PR replaces placeholder output, inferred output, or best-effort-generated +output with concrete mapped values, trace every producer and fallback path for +the mapping. Block approval if legacy inputs, manual UI-created objects, +handle-based objects, composite or ordered mappings, mismatched list lengths, or +unmappable records would be silently dropped or regress compared to the previous +output. Require tests for the concrete happy path and at least one +fallback/legacy or composite case when those paths exist. + Review object naming and reserved-word safety for changed database tables, columns, primary keys, foreign keys, indexes, constraints, API fields, events, configuration keys, routes, classes, functions, methods, generated models, and diff --git a/code-reviewer-prompt.md b/code-reviewer-prompt.md index 94bcb2961..25dc48143 100644 --- a/code-reviewer-prompt.md +++ b/code-reviewer-prompt.md @@ -128,6 +128,12 @@ between PR intent, code, docs, tests, schemas, generated files, UI rendering, and consumers. For changed scrolling, animation, transition, or motion behavior, verify that `prefers-reduced-motion: reduce` users are not forced through smooth scrolling or animated motion. +When a PR replaces placeholder output, inferred output, or best-effort-generated +output with concrete mapped values, trace each producer and fallback path for +that mapping. Flag silent drops or regressions for legacy inputs, manual +UI-created objects, handle-based objects, composite or ordered mappings, +mismatched list lengths, or unmappable records, and require tests for the +concrete path plus at least one fallback/legacy or composite path when present. For modal, dialog, drawer, popover, and toast overlays, verify viewport anchoring, inset coverage, scroll behavior, and mobile clipping; overlays must not be positioned relative to an inner app panel when the user needs a