CI docs build memory: what to review
| Field | Value |
|---|---|
| Branch | feat/admin-and-merchant-portals, draft PR #976: feat(admin): portal navigation, financials and monitoring, plus the agent-process work behind them, not merged |
| Project | README.md |
| Issue | None filed. Dispatched directly from the 2026-08-24 agenda as a same-day CI fix |
| Servers you need | None. There is nothing to click. One env: block in one CI step |
| Local origins | Not applicable |
Is there anything to look at?
One command, and one CI run.
grep -rn "max-old-space-size" .github/workflows/ci.yml- It should show
NODE_OPTIONS: --max-old-space-size=3584on the Build documentation step inside the Build (Docs) job, and nowhere else. - The real check is the CI run on this branch's PR. Build (Docs) is the job that was dying; it should pass.
What changed here?
The step had no Node heap ceiling AT ALL, so it ran on Node's unbounded default and died near 2005MB.
- It was never a case of a 4096MB ceiling being exceeded. There was no ceiling. No
NODE_OPTIONSand no--max-old-space-sizewere set anywhere on that step before this fix.
The fix is scoped to that one step, so no other job's memory behaviour changes.
What is deliberately NOT done?
The value shipped is 3584, not the 4096 the originating agenda block asked for, and the difference is open.
- 3584MB lands at roughly half of the runner's 7GB standard allocation.
- Whether that gives enough headroom, or should be reconciled to 4096, is not decided by this record. If Build (Docs) OOMs again on this branch's PR, that is the answer.
This project folder was a loose, backwards-stated file before it became a folder.
- The single commit on this branch is that correction, not the fix. The fix itself landed as
67bf3882earlier on the same branch.
What open findings touch this project?
- Nothing from
../merchant-and-venue-dashboards/branch-review-0828.md. No workflow memory finding was raised.
What issue does each piece close?
| Piece | Issue | State |
|---|---|---|
| The heap ceiling | None. Dispatched from the agenda as a same-day fix | Landed as 67bf3882 |
| The PR whose CI run surfaced the OOM | #949: fix(admin): onboard admins with a portal password, and census every admin-role decision site (#941) | Merged |