Skip to content

fix: gate memory critical on real pressure#1006

Merged
wbugitlab1 merged 8 commits into
mainfrom
issue/491-memory-critical-pressure
Jun 19, 2026
Merged

fix: gate memory critical on real pressure#1006
wbugitlab1 merged 8 commits into
mainfrom
issue/491-memory-critical-pressure

Conversation

@wbugitlab1

Copy link
Copy Markdown
Owner

Summary

  • Gate memory_critical on real memory pressure: high heap ratio now also needs either critical process RSS or low host free memory.
  • Keep warn/degraded memory behavior on the existing RSS floor.
  • Collect host memory in health snapshots and document the new health threshold knobs.

Closes #491.

Verification

  • corepack pnpm exec vitest run test/health-thresholds.test.ts test/health-monitor.test.ts --exclude test/integration.test.ts
  • corepack pnpm run lint
  • corepack pnpm run build
  • semgrep scan --config p/default --error --metrics=off src/health/thresholds.ts src/health/monitor.ts src/types.ts test/health-thresholds.test.ts test/health-monitor.test.ts
  • gitleaks protect --staged --redact

Notes

  • OSV not run: no dependency, lockfile, package-manager, container, vendored, or third-party package surface changed.
  • The original upstream PR wording around heapUsed / heapTotal is stale in this fork because current code already uses the effective V8/Node heap ceiling, but the missing real-pressure gate was still valid.

@wbugitlab1 wbugitlab1 merged commit eacce17 into main Jun 19, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[upstream PR 735] f<!-- -->ix(health): gate memory_critical on real memory pressure

1 participant