Skip to content

line:memory-trim-diagnostics: restore fail-open runtime diagnostics - #132

Open
Skywind5487 wants to merge 1 commit into
fork/memory-trim-policyfrom
fork/memory-trim-diagnostics
Open

Skywind5487 wants to merge 1 commit into
fork/memory-trim-policyfrom
fork/memory-trim-diagnostics

Conversation

@Skywind5487

Copy link
Copy Markdown
Owner

Work line: \line:memory-trim-diagnostics\

Part of merge unit
epair:memory-runtime\ (#110).

Depends on: PR #131 (\line:memory-trim-policy).

Commits

  1. \ eat(mem-trim): restore fail-open runtime diagnostics\ — GC/trim timing split + fragmentation + VmSwap

Behavior

  • Separate \gc_ms\ and \ rim_ms\ timing attribution
  • Best-effort \malloc_info(3)\ fragmentation stats (process totals, no double-count)
  • \VmSwap\ evidence from /proc/self/status\
  • Diagnostic failures degrade gracefully, never block recovery

Coverage

6 diagnostics tests in \ ests/hermes_cli/test_mem_trim_diagnostics.py.

Composed into integration PR #120 (
epair:memory-runtime).

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

૮ >ﻌ< ა ci review

ran on 64445e2 — feat(mem-trim): restore fail-open runtime diagnostics

❌ Job failures

Python tests / Run tests slice 2/12 · View job

Job Python tests / Run tests slice 2/12 failed.


⚠️ Warnings

CI timings · View report · View job

Wall time 5m23s vs 3m25s (+57.6%). 12 job(s) slower, 9 faster, 3 unchanged.

  • OSV scan / Scan lockfiles / osv-scan: +26.0s
  • OS-specific tests / macOS-only tests: +21.0s
  • Python tests / Run tests slice 9/12: -15.0s
  • Python tests / Run tests slice 11/12: +12.0s
  • Python tests / Run tests slice 7/12: -10.0s

OSV vulnerability scan · View job

5 known vulnerabilities found in pinned dependencies.

How to fix:

Review the findings in the Security tab. Update the affected dependencies if a patched version is available.

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.

1 participant