-
Notifications
You must be signed in to change notification settings - Fork 2.4k
fix(health): measure heap severity against the V8 heap limit #1224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
inix-x
wants to merge
19
commits into
rohitg00:main
Choose a base branch
from
inix-x:fix/1223-heap-severity-denominator
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
44e5372
fix(health): measure heap severity against the V8 limit
inix-x 357ba53
fix(viewer): draw the heap gauge against the V8 limit
inix-x 36c8624
fix(viewer): compute the heap percentage from raw bytes
inix-x e87a7b6
test(viewer): cover the heap gauge ceiling and precision
inix-x ebdf961
test(viewer): note why the gauge test asserts on source
inix-x 5843b18
fix(viewer): pick the gauge colour on the unrounded percentage
inix-x 26e0378
refactor(types): drop the redundant heapSizeLimit comment
inix-x 9fee40b
fix(health): act on the KV probe the monitor already collected
inix-x 8fd1c32
fix(cli): exit when the iii engine dies after startup
inix-x 922b768
build(railway): build the image from this repo instead of the registry
inix-x f5b2ba6
docs: record the hang-resilience investigation and its corrections
inix-x 6a27bca
fix(state): reclaim index generations the manifest can no longer name
inix-x 25045d8
fix(providers): bound provider concurrency and stop counting 429s as …
inix-x f5c822f
refactor(providers): read the concurrency limit through getEnvVar
inix-x b8656b0
fix(mcp): read graph stats from the snapshot instead of enumerating t…
inix-x 26ef3ed
docs(mcp): drop what-narrating comments from the graph stats read
inix-x 20b1a24
docs(state): drop the review marker from the gc ledger append
inix-x 76db640
feat(cli): forward the engine's own stdout and stderr, bounded and op…
inix-x 42bcdce
test(cli): pin the rate cap that keeps the death report reachable
inix-x File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| node_modules | ||
| .git | ||
| dist | ||
| eval | ||
| benchmark | ||
| test | ||
| .claude | ||
| .serena | ||
| docs | ||
| *.log |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Correct the watchdog removal date.
August 26, 2026 is tomorrow relative to August 25, 2026. This past-tense statement says that the removal already occurred. Use the actual removal date or future tense.
🤖 Prompt for AI Agents