docs(agents): require build + lint + test locally before pushing [guardrail-change]
Some checks failed
CI/CD / loc-budget (push) Successful in 22s
CI/CD / guardrail-integrity (push) Has been skipped
CI/CD / go-lint (push) Has been skipped
CI/CD / python-lint (push) Has been skipped
CI/CD / nodejs-lint (push) Has been skipped
CI/CD / test-go-ai-compliance (push) Successful in 43s
CI/CD / test-python-backend-compliance (push) Failing after 38s
CI/CD / test-python-document-crawler (push) Successful in 29s
CI/CD / test-python-dsms-gateway (push) Successful in 28s
CI/CD / sbom-scan (push) Has been skipped
CI/CD / validate-canonical-controls (push) Successful in 16s

CI was failing on admin-compliance build. Adds mandatory pre-push
checklist to AGENTS.typescript.md.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Sharang Parnerkar
2026-04-19 16:38:21 +02:00
parent 5e7d5d0a18
commit 71a4a3d7f3

View File

@@ -106,6 +106,26 @@ Default is Server Component. Add `"use client"` only when required:
- `prettier`.
- `next build` clean. No `// @ts-ignore`. `// @ts-expect-error` only with a comment explaining why.
## Before every push — MANDATORY
Run all three steps for every affected service (`admin-compliance/`, `developer-portal/`) before pushing. CI runs the same checks and will fail if you skip this.
```bash
cd admin-compliance # or developer-portal
# 1. Build — catches type errors and module resolution failures
npm run build
# 2. Lint
npx tsc --noEmit
npx eslint . --max-warnings 0
# 3. Tests
npm test
```
All three must exit 0. Do not push if any step fails.
## Performance
- Use `next/dynamic` for heavy client-only components.