Fix the SFI-reading heuristic for Node v24+

[umanwizard]

This, along with #269 , contributes to fixing parca-dev/parca-agent#3180 .

It should be clear what is going on from the comments; if not, I can flesh them out.

I think we can close parca-dev/parca-agent#3180 once both of these lands since the behavior is the same as in v20, but since some frames are still symbolized as line "0" in Manoj's test program, I want to investigate those before adding a corresponding test.

[gnurizen]

Probably need to fix the failing coredump test, looks like maybe you correctly fixed the line number? But someone added this test w/o adding hello3.js to coredump/testsources/node we should bring that up upstream.

[umanwizard]

It looks like I'm the one who added that coredump test... unfortunately I didn't save the corresponding hello3.js.

[gnurizen]

Could try running strings on the coredump and fishing it out!

[umanwizard]

That's a great idea, and it worked.

Here's the source code:

function bar(a, s) {
        a(s)
}

function foo() {
        bar(a, "Hello world!")
}

a = console.log
while (1) {
        foo()
}

The symbolization of the function names is weird, probably some inlining or TCO going on. But indeed the work is going on on line 1, not line 0, so I think the change makes sense.