Fixes #1581. ## What was wrong `codegraph init` / `codegraph index` died with `Segmentation fault` — the whole CLI process, not a parse worker — on a C/C++ file with very deep brace nesting (llvm's `clang/test/Parser/parser_overflow.c`, 16,384 nested `{`). The reporter's diagnosis is exactly right: tree-sitter's parser is iterative, so the file parses fine, and then the native kernel's **recursive walker** (`visit_node` → `visit_for_calls_and_structure` → …, one frame per AST level) overflowed the thread's stack. A native overflow can't be caught the way a wasm abort can, and a parse worker is a thread of the `codegraph` process, so the SIGSEGV took the entire indexer down — no message, no per-file fallback, no partial index. Two things made "just give the worker a bigger stack" the wrong fix: - it only moves the cliff — reproduced here: the reporter's 16,384-deep file kills a default 4 MiB worker (rc=132 on macOS / 139 on Linux), and a 100k-deep file kills the 8 MiB **main** thread too; - the walkers are shared by every kernel-routed language (20 of them), and each has several recursion points with different frame sizes, so no single stack size is a provable bound. Meanwhile the wasm path already handles this shape gracefully: its JS walker catches its own `RangeError` per file and stores a partial result with a `parse_error`. The kernel just needed a way to get there instead of dying. ## What this does **The kernel guards its own recursion against the calling thread's real stack bounds and defers a too-deep file to wasm** — the same `defer:` routing signal it already uses for files with parse errors, which `src/extraction/kernel/index.ts` treats as "take the wasm path for this file", silently. - `codegraph-kernel/src/stack.rs`: per-thread stack bounds from the OS, computed once per thread and cached — glibc/musl `pthread_getattr_np` + `pthread_attr_getstack`, macOS `pthread_get_stackaddr_np` + `pthread_get_stacksize_np`, Win32 `GetCurrentThreadStackLimits` (a hand-declared `kernel32` extern; no `windows-sys`). `exhausted()` is one thread-local load and one compare: true once the stack pointer is within a 256 KiB red zone of the limit, and it latches a flag. Where the OS can't report bounds it falls back to a fixed 1 MiB descent budget measured from the entry stack pointer — safe on anything from Node's 4 MiB worker default up. So the guard is exact on the 4 MiB worker, the 8 MiB main thread, and any `resourceLimits.stackSizeMb` alike. - `stack_guard!()` (defined in `lib.rs`) is the first statement of every recursive walker function — all **150** self-recursive or on-cycle functions across the 15 walker modules, found by script (every cycle in the call graph, not just direct self-calls). It returns `Default::default()` (`()`, `false`, `None`, `""`) so an exhausted walk simply stops descending; a hook returning `false` sends its caller down the generic child walk, whose own guard returns at once. - `extract_file` runs the whole walk under `stack::run_guarded`: if the flag is set afterwards the (truncated) result is discarded and replaced by `defer: nesting too deep for the native walker — wasm recovery handles it`. - `parse-pool.ts`: a comment at `new Worker(scriptPath)` records why there is deliberately no `resourceLimits.stackSizeMb` bump. - No new crates beyond `libc` as a direct unix dependency (already in `Cargo.lock` transitively). No wire/ABI change. Net effect for the reporter's repo: `deep.c` goes to the wasm path, lands as `function foo` plus a recorded parse warning, and the other 31,607 files index normally. `CODEGRAPH_KERNEL=0` and the `exclude` workaround are no longer needed. ## Tests **Rust unit tests** (`cargo test`, 21 passed — 7 new in `stack.rs`): the walkers for C, C++, Rust, TypeScript and Python are driven on a **1 MiB** thread (a quarter of Node's worker default) with 30k-deep nesting and must return `defer:` instead of crashing; shallow files are untouched; the latch resets between runs; the OS bounds are sane on the main thread and describe a small thread's own stack. **`__tests__/kernel-deep-nesting.test.ts`** (new, 8 tests — skips without a staged `.node`, fails under `CODEGRAPH_KERNEL_EXPECT=1` if the kernel is missing, like the other kernel suites): - every default-routed language (all 20) survives a 60k-deep expression on the main thread — clean result or the wasm fallback's partial result, never a crash; - the reporter's exact 16,384-brace C file is indexed (partial) on the main thread; - 200-deep expressions in every language still take the kernel path clean (the guard never trips on normal code); - inside a **default-sized 4 MiB `worker_threads` Worker** through `dist/`: the reporter's `deep.c` and a 60k-deep expression in every language come back `deferred` with exit 0, and a normal file still extracts natively; - end-to-end through the built CLI: `codegraph init` on a repo holding `deep.c` + `ok.c` exits 0 and records both files, with both functions. **Existing kernel suites**: all 15 (`kernel-*-parity`, `kernel-scaffold`, `kernel-retry-materialize`, `kernel-grammar-parity`) pass unchanged, 147 tests — the guard never fires on the parity fixtures. **Reporter's probes** (`one.js` from the issue, default 4 MiB worker, this build): `deep.c` → `deferred`, exitCode=0 (was rc=132/139); `deep100k.c` → `deferred`, exitCode=0. Main thread: `deep.c` / `deep100k.c` → wasm partial with `Parse error: Maximum call stack size exceeded`; a 6,000-term binary expression and a 3,000-branch `else if` chain stay on the kernel path with clean results. **Perf** (same `dist/`, only the `.node` swapped via `CODEGRAPH_KERNEL_PATH`; interleaved main/new ×3, `codegraph init`, macOS arm64): | repo | main (median) | guarded (median) | nodes / edges | |---|---|---|---| | express (141 files) | 0.60 s (0.58–0.65) | 0.61 s (0.58–0.61) | 1,084 / identical | | redis (786 C/H files) | 4.44 s (4.39–4.66) | 4.49 s (4.41–4.70) | 19,942 / 76,446 identical | Within run-to-run noise, as expected for one TLS load + compare per recursion entry. **Linux (Docker, `node:22-bookworm`, kernel built in-container, `docker run --rm --init`)** — the reporter's platform and the glibc `pthread_getattr_np` bounds path: ``` === platform === Linux efe3cc86947b 6.12.54-linuxkit #1 SMP Tue Nov 4 21:21:47 UTC 2025 aarch64 GNU/Linux v22.22.3 -rwxr-xr-x 1 root root 35332288 Aug 22 18:02 codegraph-kernel/prebuilds/linux-arm64/codegraph-kernel.node === reporter repro (issue #1581): 16,384-brace deep.c, codegraph init === │ └ Done init exit code: 0 file: deep.c file: deep100k.c file: ok.c function: add function: bar function: foo === worker probe: kernel raw extract in a default 4 MiB worker === deep.c: deferred deep.c: worker exitCode=0 deep100k.c: deferred deep100k.c: worker exitCode=0 ok.c: kernel nodes=2 ok.c: worker exitCode=0 === cargo test stack:: (glibc pthread_getattr_np bounds path) === test stack::tests::os_bounds_are_sane_on_this_platform ... ok test stack::tests::small_stack_reports_its_own_bounds ... ok test stack::tests::normal_files_are_untouched_by_the_guard ... ok test stack::tests::deep_braces_c_defer_instead_of_crashing ... ok test stack::tests::latch_resets_between_runs ... ok test stack::tests::deep_parens_cpp_rust_ts_python_defer_instead_of_crashing ... ok test result: ok. 6 passed; 0 failed; 0 ignored; 0 measured; 15 filtered out; finished in 0.23s === vitest: kernel-deep-nesting + kernel-scaffold === ✓ __tests__/kernel-scaffold.test.ts (10 tests) 30ms ✓ __tests__/kernel-deep-nesting.test.ts (8 tests) 36989ms Test Files 2 passed (2) Tests 18 passed (18) ``` (The pre-fix crash was reproduced on macOS — rc=132 in a default worker, rc=139 on the main thread at 100k depth — not re-run inside this container; the reporter's Linux x86_64 trace is the SIGSEGV form of the same overflow.) **Windows (Parallels ARM64 VM, MSVC 14.44, `cargo 1.97`, kernel built on the VM, `GetCurrentThreadStackLimits` path)**: ``` head: cbf8485 fix(kernel): guard the native walkers against stack overflow and defer deep files to wasm (#1581) === cargo build --release (win32-arm64) === Finished `release` profile [optimized] target(s) in 2m 04s staged: 35086848 bytes === cargo test (stack guard unit tests) === test stack::tests::normal_files_are_untouched_by_the_guard ... ok test stack::tests::os_bounds_are_sane_on_this_platform ... ok test stack::tests::small_stack_reports_its_own_bounds ... ok test stack::tests::deep_braces_c_defer_instead_of_crashing ... ok test stack::tests::latch_resets_between_runs ... ok test stack::tests::deep_parens_cpp_rust_ts_python_defer_instead_of_crashing ... ok test result: ok. 6 passed; 0 failed; 0 ignored; 0 measured; 15 filtered out; finished in 0.49s === reporter repro: codegraph init on a 16,384-brace deep.c === └ Done init exit code: 0 === vitest: deep-nesting + scaffold (CODEGRAPH_KERNEL_EXPECT=1) === ✓ __tests__/kernel-scaffold.test.ts (10 tests) 55ms ✓ __tests__/kernel-deep-nesting.test.ts (8 tests) 67239ms ✓ every default-routed language survives a 60k-deep expression on the main thread 52801ms ✓ inside a default-sized (4 MiB) parse worker, through dist/ > defers a 60k-deep expression in every default-routed language 13050ms ✓ end-to-end: codegraph init on a repo holding the deep file > exits 0 and records deep.c alongside the normal files 936ms Test Files 2 passed (2) Tests 18 passed (18) ``` (The end-to-end test is what reads the Windows index back through `node:sqlite` — `files` = `deep.c`, `ok.c`; functions `add`, `foo`.) Full `npm test` on this branch (macOS arm64, kernel staged): **190 files passed, 3,185 tests passed, 10 skipped, 0 failed.** Clippy note: `cargo clippy` on the current toolchain (1.92) reports 18 pre-existing lints (`manual_contains`, `unnecessary_to_owned`, …) in walker code this PR only touched by inserting guard lines; none are in `stack.rs`/`lib.rs`. Left alone to keep the diff reviewable. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01LxZj6W6Y1SHXwvpT3uwJpK
274 lines
11 KiB
Rust
274 lines
11 KiB
Rust
//! Stack-budget guard for the recursive walkers (#1581).
|
|
//!
|
|
//! Every language walker recurses per AST level (`visit_node` →
|
|
//! `visit_for_calls_and_structure` → …). tree-sitter's own parser is
|
|
//! iterative, so a pathologically nested file — clang's
|
|
//! `parser_overflow.c` nests 16,384 `{`, fuzzer corpora go deeper — parses
|
|
//! fine and then overflows the WALKER's native stack. A native overflow is
|
|
//! uncatchable: the parse worker is a thread of the `codegraph` process, so
|
|
//! the SIGSEGV takes the whole indexer down with no message, no partial
|
|
//! index and no per-file fallback. Worker threads get Node's 4 MiB default
|
|
//! stack; the main thread's 8 MiB only moves the cliff (100k levels still
|
|
//! kill it).
|
|
//!
|
|
//! The guard turns "about to overflow" into the kernel's existing `defer:`
|
|
//! routing signal: `exhausted()` is checked at the top of every recursive
|
|
//! walker function (the `stack_guard!` macro in lib.rs), returns `true` once
|
|
//! the stack pointer is within `RED_ZONE` of the thread's stack limit, and
|
|
//! latches a per-thread flag. `run_guarded` wraps a whole extraction: when
|
|
//! the flag is set afterwards the result is discarded and replaced by a
|
|
//! `defer:` error, which the TS side (`src/extraction/kernel/index.ts`)
|
|
//! already treats as "this file takes the wasm path" — and the wasm walker
|
|
//! catches its own JS `RangeError` per file, so the file lands as a partial
|
|
//! result with a recorded parse error instead of a dead process.
|
|
//!
|
|
//! The per-thread stack bounds come from the OS (glibc/musl
|
|
//! `pthread_getattr_np`, macOS `pthread_get_stackaddr_np`, Win32
|
|
//! `GetCurrentThreadStackLimits`), computed once per thread and cached, so
|
|
//! the guard is exact on the 4 MiB worker, the 8 MiB main thread and any
|
|
//! `resourceLimits.stackSizeMb` alike. Where the bounds are unavailable the
|
|
//! guard falls back to a fixed descent budget measured from the entry stack
|
|
//! pointer. Hot path: one thread-local load and one compare.
|
|
|
|
use std::cell::Cell;
|
|
|
|
/// Headroom kept free below the deepest walker frame: the napi return path,
|
|
/// tree-sitter's node accessors and the error formatting all still need to
|
|
/// run after the guard trips, and the frames BETWEEN two guard checks (an
|
|
/// `extract_class` between two `visit_node`s) are never more than a few KiB.
|
|
const RED_ZONE: usize = 256 * 1024;
|
|
|
|
/// Descent budget when the OS can't report the thread's stack bounds — safe
|
|
/// on anything from Node's 4 MiB worker default upwards.
|
|
const FALLBACK_BUDGET: usize = 1024 * 1024;
|
|
|
|
thread_local! {
|
|
/// Lowest stack-pointer value the walker may reach before the guard
|
|
/// trips. `0` = not computed yet for this thread.
|
|
static THRESHOLD: Cell<usize> = const { Cell::new(0) };
|
|
/// `true` when the thread's threshold came from real OS bounds (fixed
|
|
/// for the thread's lifetime) rather than the per-call fallback budget.
|
|
static THRESHOLD_IS_OS: Cell<bool> = const { Cell::new(false) };
|
|
/// Latched by `exhausted()`; read by `run_guarded` after the walk.
|
|
static OVERFLOWED: Cell<bool> = const { Cell::new(false) };
|
|
}
|
|
|
|
/// Approximate current stack pointer: the address of a local. Stacks grow
|
|
/// downward on every target the kernel ships for (x86_64 / aarch64).
|
|
#[inline(always)]
|
|
fn current_sp() -> usize {
|
|
let marker = 0u8;
|
|
std::hint::black_box(&marker) as *const u8 as usize
|
|
}
|
|
|
|
/// Low (deepest) address of the calling thread's stack, from the OS.
|
|
#[cfg(target_os = "linux")]
|
|
fn os_stack_low() -> Option<usize> {
|
|
// SAFETY: plain pthread queries on the calling thread; `attr` is
|
|
// initialised by pthread_getattr_np and destroyed before returning.
|
|
unsafe {
|
|
let mut attr: libc::pthread_attr_t = std::mem::zeroed();
|
|
if libc::pthread_getattr_np(libc::pthread_self(), &mut attr) != 0 {
|
|
return None;
|
|
}
|
|
let mut addr: *mut libc::c_void = std::ptr::null_mut();
|
|
let mut size: libc::size_t = 0;
|
|
let rc = libc::pthread_attr_getstack(&attr, &mut addr, &mut size);
|
|
libc::pthread_attr_destroy(&mut attr);
|
|
if rc != 0 || addr.is_null() || size == 0 {
|
|
return None;
|
|
}
|
|
Some(addr as usize)
|
|
}
|
|
}
|
|
|
|
#[cfg(target_os = "macos")]
|
|
fn os_stack_low() -> Option<usize> {
|
|
// SAFETY: plain pthread queries on the calling thread.
|
|
unsafe {
|
|
let me = libc::pthread_self();
|
|
// pthread_get_stackaddr_np returns the HIGH end (the stack base).
|
|
let high = libc::pthread_get_stackaddr_np(me) as usize;
|
|
let size = libc::pthread_get_stacksize_np(me);
|
|
if high == 0 || size == 0 || size > high {
|
|
return None;
|
|
}
|
|
Some(high - size)
|
|
}
|
|
}
|
|
|
|
#[cfg(windows)]
|
|
fn os_stack_low() -> Option<usize> {
|
|
#[link(name = "kernel32")]
|
|
extern "system" {
|
|
// Win8+ (the bundled Node runtime needs Win10 anyway). Reports the
|
|
// full RESERVED range; Windows commits pages on demand down to it.
|
|
fn GetCurrentThreadStackLimits(low_limit: *mut usize, high_limit: *mut usize);
|
|
}
|
|
let mut low: usize = 0;
|
|
let mut high: usize = 0;
|
|
// SAFETY: both out-pointers are valid for the duration of the call.
|
|
unsafe { GetCurrentThreadStackLimits(&mut low, &mut high) };
|
|
if low == 0 || high <= low {
|
|
return None;
|
|
}
|
|
Some(low)
|
|
}
|
|
|
|
#[cfg(not(any(target_os = "linux", target_os = "macos", windows)))]
|
|
fn os_stack_low() -> Option<usize> {
|
|
None
|
|
}
|
|
|
|
/// Arm the guard for one extraction on the calling thread: clear the latch
|
|
/// and (re)compute the threshold. OS bounds are computed once per thread;
|
|
/// the fallback budget is re-anchored at every call's entry stack pointer.
|
|
pub fn begin() {
|
|
OVERFLOWED.with(|o| o.set(false));
|
|
let cached = THRESHOLD.with(|t| t.get());
|
|
if cached != 0 && THRESHOLD_IS_OS.with(|f| f.get()) {
|
|
return;
|
|
}
|
|
match os_stack_low() {
|
|
Some(low) => {
|
|
THRESHOLD.with(|t| t.set(low.saturating_add(RED_ZONE)));
|
|
THRESHOLD_IS_OS.with(|f| f.set(true));
|
|
}
|
|
None => {
|
|
THRESHOLD.with(|t| t.set(current_sp().saturating_sub(FALLBACK_BUDGET).max(1)));
|
|
THRESHOLD_IS_OS.with(|f| f.set(false));
|
|
}
|
|
}
|
|
}
|
|
|
|
/// `true` once the walker has descended to within `RED_ZONE` of the stack
|
|
/// limit. Latches `OVERFLOWED` so `run_guarded` can discard the result. A
|
|
/// thread that never called `begin()` (a direct unit-test call) arms itself
|
|
/// lazily from the current position.
|
|
#[inline(always)]
|
|
pub fn exhausted() -> bool {
|
|
let threshold = THRESHOLD.with(|t| t.get());
|
|
if threshold == 0 {
|
|
begin();
|
|
return exhausted();
|
|
}
|
|
if current_sp() < threshold {
|
|
OVERFLOWED.with(|o| o.set(true));
|
|
true
|
|
} else {
|
|
false
|
|
}
|
|
}
|
|
|
|
/// Whether the guard tripped since the last `begin()`.
|
|
pub fn overflowed() -> bool {
|
|
OVERFLOWED.with(|o| o.get())
|
|
}
|
|
|
|
/// Run one extraction under the guard. A walk that tripped the guard returns
|
|
/// a `defer:` error — the TS side's routine "take the wasm path" signal —
|
|
/// regardless of what the truncated walk produced.
|
|
pub fn run_guarded<T>(f: impl FnOnce() -> Result<T, String>) -> Result<T, String> {
|
|
begin();
|
|
let out = f();
|
|
if overflowed() {
|
|
return Err(
|
|
"defer: nesting too deep for the native walker — wasm recovery handles it".to_string(),
|
|
);
|
|
}
|
|
out
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod tests {
|
|
use super::*;
|
|
|
|
/// 1 MiB is a quarter of Node's worker default; a guard that holds here
|
|
/// holds on every real thread. Without the guard these walks SIGSEGV the
|
|
/// test process instead of failing an assertion.
|
|
const SMALL_STACK: usize = 1 << 20;
|
|
const DEPTH: usize = 30_000;
|
|
|
|
fn on_small_stack<T: Send + 'static>(f: impl FnOnce() -> T + Send + 'static) -> T {
|
|
std::thread::Builder::new()
|
|
.stack_size(SMALL_STACK)
|
|
.spawn(f)
|
|
.expect("spawn")
|
|
.join()
|
|
.expect("walker thread panicked")
|
|
}
|
|
|
|
fn nested_parens(prefix: &str, suffix: &str) -> String {
|
|
format!("{prefix}{}1{}{suffix}", "(".repeat(DEPTH), ")".repeat(DEPTH))
|
|
}
|
|
|
|
#[test]
|
|
fn os_bounds_are_sane_on_this_platform() {
|
|
// Every shipped target has an OS implementation; the fallback budget
|
|
// is only for platforms the kernel is not built for.
|
|
let low = os_stack_low().expect("OS stack bounds available");
|
|
let sp = current_sp();
|
|
assert!(low < sp, "stack low {low:#x} must be below the current sp {sp:#x}");
|
|
assert!(sp - low < 1 << 31, "implausible stack size {}", sp - low);
|
|
}
|
|
|
|
#[test]
|
|
fn small_stack_reports_its_own_bounds() {
|
|
on_small_stack(|| {
|
|
let low = os_stack_low().expect("OS stack bounds available");
|
|
let used = current_sp() - low;
|
|
// std/the OS round the requested size up a little (macOS reports
|
|
// 1,060,864 for a 1 MiB request); the point is that the bounds
|
|
// describe THIS thread's small stack, not the main thread's.
|
|
assert!(
|
|
used <= SMALL_STACK + 128 * 1024,
|
|
"used {used} is not within the {SMALL_STACK}-byte stack"
|
|
);
|
|
});
|
|
}
|
|
|
|
#[test]
|
|
fn deep_braces_c_defer_instead_of_crashing() {
|
|
let src = format!("void foo(void) {{\n{}{}\n}}\n", "{".repeat(DEPTH), "}".repeat(DEPTH));
|
|
let r = on_small_stack(move || run_guarded(|| crate::ccpp::extract("deep.c", &src, "c")));
|
|
let err = r.err().expect("deep nesting must defer");
|
|
assert!(err.starts_with("defer:"), "unexpected error: {err}");
|
|
}
|
|
|
|
type Extract = fn(&str) -> Result<crate::buffers::EmitOut, String>;
|
|
|
|
#[test]
|
|
fn deep_parens_cpp_rust_ts_python_defer_instead_of_crashing() {
|
|
let cases: [(&str, Extract, String); 4] = [
|
|
("deep.cpp", |s| crate::ccpp::extract("deep.cpp", s, "cpp"), nested_parens("int f() { return ", "; }\n")),
|
|
("deep.rs", |s| crate::rustlang::extract("deep.rs", s), nested_parens("fn f() -> i32 { ", " }\n")),
|
|
("deep.ts", |s| crate::tsjs::extract("deep.ts", s, "typescript"), nested_parens("function f() { return ", "; }\n")),
|
|
("deep.py", |s| crate::python::extract("deep.py", s), nested_parens("def f():\n return ", "\n")),
|
|
];
|
|
for (name, extract, src) in cases {
|
|
let r = on_small_stack(move || run_guarded(|| extract(&src)));
|
|
let err = r.err().unwrap_or_else(|| panic!("{name}: deep nesting must defer"));
|
|
assert!(err.starts_with("defer:"), "{name}: unexpected error: {err}");
|
|
}
|
|
}
|
|
|
|
#[test]
|
|
fn normal_files_are_untouched_by_the_guard() {
|
|
let src = "int add(int a, int b) { return a + b; }\nint main(void) { return add(1, 2); }\n";
|
|
let r = on_small_stack(move || run_guarded(|| crate::ccpp::extract("ok.c", src, "c")));
|
|
assert!(r.is_ok(), "a shallow file must not defer: {:?}", r.err());
|
|
assert!(!overflowed());
|
|
}
|
|
|
|
#[test]
|
|
fn latch_resets_between_runs() {
|
|
let deep = format!("void foo(void) {{\n{}{}\n}}\n", "{".repeat(DEPTH), "}".repeat(DEPTH));
|
|
on_small_stack(move || {
|
|
assert!(run_guarded(|| crate::ccpp::extract("deep.c", &deep, "c")).is_err());
|
|
// The latch from the deep file must not poison the next, shallow one.
|
|
let ok = run_guarded(|| crate::ccpp::extract("ok.c", "int x;\n", "c"));
|
|
assert!(ok.is_ok(), "latch leaked into the next run: {:?}", ok.err());
|
|
});
|
|
}
|
|
}
|