docs(AGENTS): reflect bump-next-js outcome (Next 16, ESLint v9, typescript devDep) #5

Merged

varutasu merged 1 commit from docs/bump-next-js-agents-update into main

2026-05-23 10:36:44 -04:00

varutasu commented

2026-05-23 03:36:31 -04:00

(Migrated from github.com)

Summary

Updates AGENTS.md to reflect the convoy bump-next-js outcome (PR #4 / commit e57ea17): Next.js 15 → 16, ESLint v8 → v9.39.4 + flat config, and a new typescript@^5.9.3 devDep.
Adds three new common gotchas so future agents don't trip on the non-obvious bits: why typescript is installed despite the project being JavaScript-only, why ESLint is pinned to v9 (not v10), and that Turbopack is now the default bundler.
Adds a § 7 Deployment line noting VERCEL_AUTOMATION_BYPASS_SECRET exists locally for the queued adopt-playwright-smoke convoy to use against protected preview deploys.

Convoy + Brief

Convoy: .convoys/bump-next-js.md (merged via PR #4 / commit e57ea17)
Brief: .convoys/bump-next-js/brief-1-bump-next-and-migrate-config.md
This PR: doc-writer pass per the convoy's Roles list — separate from the implementer PR, as the role spec requires.

Acceptance criteria

AGENTS.md § 1 Framework line reflects Next.js 16
Gotcha #9 added: typescript devDep is for eslint-config-next@16's lint chain only (no tsconfig.json, no .ts files)
Gotcha #10 added: ESLint pinned to v9.39.4 (maintenance), points at the queued bump-eslint-10 follow-up convoy
Gotcha #11 added: Turbopack is now the default bundler with --webpack fallback
§ 7 Deployment references VERCEL_AUTOMATION_BYPASS_SECRET (name only, no value)
No scope expansion: only AGENTS.md edited; no CHANGELOG.md, no README.md, no .cursor/rules/*.mdc, no docs/SCHEMA_MAP.md, no code, no .convoys/ files, no .github/CODEOWNERS

Test plan

Docs-only — no test surface. Verification performed:

git diff AGENTS.md reviewed (5 insertions, 1 deletion across §§ 1, 4, 7).
All claims cross-checked against .convoys/bump-next-js.md § Architecture and Decisions A–D.
Decision pointers (Decision C for the typescript note, Decision D + "Follow-up convoys queued" for the ESLint pin) are accurate and point at sections that exist in the convoy file.

Pipeline gates

[n/a] CI: lint, types, build, unit tests
[n/a] Visual diff
[n/a] A11y audit
[n/a] Design-system audit
Reviewer report (manual human review)
[n/a] Smoke on staging

Notes for reviewer

The CHANGELOG omission is deliberate. CHANGELOG.md and DEVELOPER_CHANGELOG.md don't exist yet in this repo; per the convoy's Roles section, changelog adoption is queued under launch-polish. Don't add changelog files in this PR.
Gotcha numbering: previous max was #8, new gotchas continue as #9, #10, #11 (no renumbering of existing entries).
The § 1 Framework line now cross-references Gotcha #9 so readers landing at the tech-stack section find the TypeScript-is-just-for-lint context.
VERCEL_AUTOMATION_BYPASS_SECRET is referenced by name only — no value is included. The env var lives in .env.local (already in .gitignore).

"While I was in here" observations (flagged, NOT fixed)

README.md:16 says **Frontend**: Next.js 15, React 18, TypeScript — wrong on two counts (version is now 16, language is JavaScript). Per dispatch, README staleness is out of scope for this docs pass. Worth a follow-up tightening in launch-polish (or whatever convoy next touches README copy).

Made with Cursor

## Summary - Updates `AGENTS.md` to reflect the convoy `bump-next-js` outcome (PR #4 / commit `e57ea17`): Next.js 15 → 16, ESLint v8 → v9.39.4 + flat config, and a new `typescript@^5.9.3` devDep. - Adds three new common gotchas so future agents don't trip on the non-obvious bits: why `typescript` is installed despite the project being JavaScript-only, why ESLint is pinned to v9 (not v10), and that Turbopack is now the default bundler. - Adds a § 7 Deployment line noting `VERCEL_AUTOMATION_BYPASS_SECRET` exists locally for the queued `adopt-playwright-smoke` convoy to use against protected preview deploys. ## Convoy + Brief - Convoy: `.convoys/bump-next-js.md` (merged via PR #4 / commit `e57ea17`) - Brief: `.convoys/bump-next-js/brief-1-bump-next-and-migrate-config.md` - This PR: doc-writer pass per the convoy's Roles list — separate from the implementer PR, as the role spec requires. ## Acceptance criteria - [x] `AGENTS.md` § 1 Framework line reflects Next.js 16 - [x] Gotcha #9 added: `typescript` devDep is for `eslint-config-next@16`'s lint chain only (no `tsconfig.json`, no `.ts` files) - [x] Gotcha #10 added: ESLint pinned to v9.39.4 (maintenance), points at the queued `bump-eslint-10` follow-up convoy - [x] Gotcha #11 added: Turbopack is now the default bundler with `--webpack` fallback - [x] § 7 Deployment references `VERCEL_AUTOMATION_BYPASS_SECRET` (name only, no value) - [x] No scope expansion: only `AGENTS.md` edited; no `CHANGELOG.md`, no `README.md`, no `.cursor/rules/*.mdc`, no `docs/SCHEMA_MAP.md`, no code, no `.convoys/` files, no `.github/CODEOWNERS` ## Test plan Docs-only — no test surface. Verification performed: - `git diff AGENTS.md` reviewed (5 insertions, 1 deletion across §§ 1, 4, 7). - All claims cross-checked against `.convoys/bump-next-js.md` § Architecture and Decisions A–D. - Decision pointers (Decision C for the `typescript` note, Decision D + "Follow-up convoys queued" for the ESLint pin) are accurate and point at sections that exist in the convoy file. ## Pipeline gates  - [n/a] CI: lint, types, build, unit tests - [n/a] Visual diff - [n/a] A11y audit - [n/a] Design-system audit - [ ] Reviewer report (manual human review) - [n/a] Smoke on staging ## Notes for reviewer - The CHANGELOG omission is deliberate. `CHANGELOG.md` and `DEVELOPER_CHANGELOG.md` don't exist yet in this repo; per the convoy's Roles section, changelog adoption is queued under `launch-polish`. Don't add changelog files in this PR. - Gotcha numbering: previous max was #8, new gotchas continue as #9, #10, #11 (no renumbering of existing entries). - The § 1 Framework line now cross-references Gotcha #9 so readers landing at the tech-stack section find the TypeScript-is-just-for-lint context. - `VERCEL_AUTOMATION_BYPASS_SECRET` is referenced by name only — no value is included. The env var lives in `.env.local` (already in `.gitignore`). ## "While I was in here" observations (flagged, NOT fixed) - `README.md:16` says `**Frontend**: Next.js 15, React 18, TypeScript` — wrong on two counts (version is now 16, language is JavaScript). Per dispatch, README staleness is out of scope for this docs pass. Worth a follow-up tightening in `launch-polish` (or whatever convoy next touches README copy). Made with [Cursor](https://cursor.com)

vercel[bot] commented

2026-05-23 03:36:36 -04:00

(Migrated from github.com)

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
tcg-vault	Ready	Preview, Comment	May 23, 2026 7:36am

[vc]: #IDMrPGmXNP3823LyZ2G8H4zpFwKOc4DrFwJOBXil8CM=:eyJpc01vbm9yZXBvIjp0cnVlLCJ0eXBlIjoiZ2l0aHViIiwicHJvamVjdHMiOlt7Im5hbWUiOiJ0Y2ctdmF1bHQiLCJwcm9qZWN0SWQiOiJwcmpfRjZXOEVvRkd3Y0g3aWVGcnRvRlNlOXdVVkFhNSIsImluc3BlY3RvclVybCI6Imh0dHBzOi8vdmVyY2VsLmNvbS9yYW5kYWxsLXN0aWxsd2VsbHMtcHJvamVjdHMvdGNnLXZhdWx0LzZRVkVDZFZaazZTUW1BR0J0Vk1yUUFVV3E4blkiLCJwcmV2aWV3VXJsIjoidGNnLXZhdWx0LWdpdC1kb2NzLWJ1bXAtbmV4dC00MGE0MTgtcmFuZGFsbC1zdGlsbHdlbGxzLXByb2plY3RzLnZlcmNlbC5hcHAiLCJuZXh0Q29tbWl0U3RhdHVzIjoiREVQTE9ZRUQiLCJsaXZlRmVlZGJhY2siOnsicmVzb2x2ZWQiOjAsInVucmVzb2x2ZWQiOjAsInRvdGFsIjowLCJsaW5rIjoidGNnLXZhdWx0LWdpdC1kb2NzLWJ1bXAtbmV4dC00MGE0MTgtcmFuZGFsbC1zdGlsbHdlbGxzLXByb2plY3RzLnZlcmNlbC5hcHAifSwicm9vdERpcmVjdG9yeSI6bnVsbH1dLCJyZXF1ZXN0UmV2aWV3VXJsIjoiaHR0cHM6Ly92ZXJjZWwuY29tL3ZlcmNlbC1hZ2VudC9yZXF1ZXN0LXJldmlldz9vd25lcj12YXJ1dGFzdSZyZXBvPXRjZy12YXVsdCZwcj01In0= The latest updates on your projects. Learn more about [Vercel for GitHub](https://vercel.link/github-learn-more). | Project | Deployment | Actions | Updated (UTC) | | :--- | :----- | :------ | :------ | | [tcg-vault](https://vercel.com/randall-stillwells-projects/tcg-vault) | ![Ready](https://vercel.com/static/status/ready.svg) [Ready](https://vercel.com/randall-stillwells-projects/tcg-vault/6QVECdVZk6SQmAGBtVMrQAUWq8nY) | [Preview](https://tcg-vault-git-docs-bump-next-40a418-randall-stillwells-projects.vercel.app), [Comment](https://vercel.live/open-feedback/tcg-vault-git-docs-bump-next-40a418-randall-stillwells-projects.vercel.app?via=pr-comment-feedback-link) | May 23, 2026 7:36am | <a href="https://vercel.com/vercel-agent/request-review?owner=varutasu&repo=tcg-vault&pr=5" rel="noreferrer"><picture><source media="(prefers-color-scheme: dark)" srcset="https://agents-vade-review.vercel.sh/request-review-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://agents-vade-review.vercel.sh/request-review-light.svg"><img src="https://agents-vade-review.vercel.sh/request-review-light.svg" alt="Request Review"></picture></a>

github-actions[bot] commented

2026-05-23 03:36:40 -04:00

(Migrated from github.com)

Pipeline Health

Build + CI gates

Gate	Status
Vercel build (Preview)	✅ pass
CI: Lint	✅ pass
CI: Schema map fresh	❌ skipped
Preview smoke	❌ failure
Visual diff	⏭ skipped or pending

Build runs on Vercel; this CI runs lint and schema-map drift only (no duplicate build).

Role reports

Role	Status
Reviewer report	⏳ pending
A11y audit	⏳ pending
Design system audit	⏳ pending

See individual comments above for details. This rollup updates automatically.

## Pipeline Health ### Build + CI gates | Gate | Status | | --- | --- | | Vercel build (Preview) | ✅ pass | | CI: Lint | ✅ pass | | CI: Schema map fresh | ❌ skipped | | Preview smoke | ❌ failure | | Visual diff | ⏭ skipped or pending | _Build runs on Vercel; this CI runs lint and schema-map drift only (no duplicate build)._ ### Role reports | Role | Status | | --- | --- | | Reviewer report | ⏳ pending | | A11y audit | ⏳ pending | | Design system audit | ⏳ pending | See individual comments above for details. This rollup updates automatically.