git:20260926.3185b84 to git:20260927.23e825b

1 added, 1 removed. Audit A to A.

---

name: database-review

- description: Review database, query, API/server-function, loader/action, and data-layer changes for correctness, performance, authorization, connection failures, and migration safety. Use whenever database queries/schema/data access change or cleanup touches backend data flow.

+ description: Review database, query, API/server-function, loader/action, and data-layer changes for correctness, performance, authorization, connection failures, and migration safety. MUST be used before creating or modifying schema or migrations, before any database write, and whenever queries, data access, or backend data flow change, including during cleanup.

---

# Database Review

## Safety Default

Database interaction is read-only by default and restricted to SELECT queries against clone/dev databases unless the user explicitly grants write access for the task. Never INSERT, UPDATE, DELETE, ALTER, DROP, TRUNCATE, create tables, apply migrations, or perform destructive operations without explicit authorization.

Do not create/modify/run migrations as a side effect. Schema/migration work requires an explicit request. Prefer small, reviewable, reversible migrations and prepare them in code without applying to live databases unless explicitly requested. Never touch production credentials/secrets.

## Query and Data-Layer Review

For changed queries, APIs, server functions, loaders, actions, or fetching:

- Avoid N+1 queries and duplicate requests.

- Avoid unnecessary columns/records.

- Use appropriate filtering, pagination, joins, indexes, caching, batching, or parallelization when justified.

- Reuse data already available in the request/component flow.

- Avoid unnecessary sequential blocking.

- Verify query correctness, ownership, and authorization constraints.

- Do not prematurely optimize unaffected code.

## Data Integrity

- Validate inputs.

- Ensure mutations cannot create duplicate/inconsistent records.

- Verify derived totals/counts/statuses use the correct authoritative records.

- Never replace failed data with fake/default business values that imply success.

## Failure Handling

For database, API, network, or external-service failures:

- Never expose raw database errors, SQL errors, stack traces, exception details, internal paths, or sensitive implementation details to end users.

- Show a safe, user-friendly fallback message such as `Something went wrong. Please try again.` when no more specific actionable message is appropriate.

- Use specific user-facing messages when they are safe and useful, such as `Unable to load records. Please try again.`

- Preserve the original technical error for server-side logging/debugging where appropriate.

- Handle rejected requests and unexpected responses without unnecessary application crashes.

- Provide retry behavior when the operation can safely be retried.

- Prevent destructive, stale, or misleading UI state after a failed operation.

- Preserve previously valid data when sensible instead of replacing it with fake/default business values.

- Never present a failed operation as successful.

## Documentation

Keep schema/query changes synchronized with relevant architecture, decisions, modules, patterns, bugs, or WIP notes when reusable knowledge changes.

## Related Skills

Keep query correctness, data integrity, schema behavior, query-level performance, migration safety, and read/write safety here. Use `backend-review` for handler and service concerns such as retries, idempotency, and resource cleanup; `performance-review` for cross-stack measurement and caching methodology; `dependency-review` for ORM or driver changes.