Heap Profiler not building the same Flamegraph as in the Docs example

[alfonsograziano]

I was looking at the Heap profiler. I created a Memory Leak on a Next.js application with this code:

import styles from "./page.module.css";
import { headers } from "next/headers";

const array = []

const createMemory = () => {
  for(let i = 0; i < 1000; i++){
    array.push(i)
  }
}

const createMoreMemory = () => {
  for(let i = 0; i < 5000; i++){
    array.push(i)
  }
}

export default async function Home() {
  const headersList = headers();
  createMemory()
  createMoreMemory()
  return (
    <main className={styles.main}>{JSON.stringify(headersList, null, 2)}</main>
  );
}

I disabled optimizations and minification as well in the Next.js config like this:

module.exports = {
    productionBrowserSourceMaps: true,
    webpack: (config, { buildId, dev, isServer, defaultLoaders, webpack }) => {
      // Disable minification in production
        config.optimization.minimize = false;
        return config;
    },
  };
  

This is the output of the heapprofiler:

I was expecting to see the memory footprint of the createMemory and createMoreMemory functions. I can only see the Home function called in the flame graph.

This made me think that maybe, for some reason, the heap profiler was not working as intended on Next.js.
Before starting an investigation, I decided to test with the example provided in the Flame First Analysis documentation.

I just cloned the repo and I run the script as shown.
In this case the expected output is the one shown in the documentation:

The actual result on my machine is this one:

The "soManyAllocation" reference is missing.

This is my config:

• heapprofiler v5.0.0
• node v20.7.0

In both cases, since the functions that are using a lot of memory are missing, the heap profiler is not really useful. I'll try to play with it a little bit and reproduce exactly the example.

I also noticed that in the screenshot, the image reports the script called 1-server-with-leaks, but in the example repo, this script is not included. I'm not sure if is a different script or has just been renamed

[RafaelGSS]

That's due to recent Node.js versions.

[simoneb]

Following up to a conversation we were having with Alfonso, I'm wondering if there's an opportunity here to revise the approach used in Clinic which relies considerably on Node internals, and rather switch to use an approach that's more robust across Node.js versions, perhaps relying on published APIs. I'm not even sure if this is feasible, but I wonder for example what other profilers do, e.g. those built into Chrome.