code-review.mdc

---
description: "PR checklist for @contentstack/delivery-sdk — API, types, core bump, tests"
alwaysApply: true
---

# Code review — `@contentstack/delivery-sdk`

## Public API

- **Exported** `stack`, **Stack**, query/entry/asset types match **README** and **`.d.ts`** output in **`dist/modern/`**.
- **JSDoc** on **`stack()`** and key public methods when behavior or options change.

## Compatibility

- Avoid breaking **StackConfig** or method chains without semver strategy; document migration for breaking changes.

## Core / deps

- **`@contentstack/core`** version changes: verify interceptors, errors, and **httpClient** options in **`contentstack.ts`**.

## Tests

- **Unit** coverage for new logic; **API** updates when CDA request/response behavior changes; **browser** if bundling or globals affected.

## Security

- No hardcoded tokens; no logging secrets in new code.