Cree una matriz, inserte cada url en ella en el ciclo, luego mueva su llamada writeFile hasta el final.

 const puppeteer = require("puppeteer"); const fs = require('fs').promises; let browser; (async () => { browser = await puppeteer.launch({ headless: true, args: ['--no-sandbox'] }); const [page] = await browser.pages(); await page.goto('https://old.reddit.com/', { "waitUntil": "networkidle0" }); const aElems = await page.$$('.thumbnail'); const urls = []; for (let i = 0; i < aElems.length && i < 3; i++) { const href = await aElems[i].evaluate(e => e.href); const newPage = await browser.newPage(); await newPage.goto(href, {waitUntil: "networkidle0"}); const url = await newPage.evaluate(() => document.location.href); console.log(url); urls.push(url); } await fs.writeFile('export.json', JSON.stringify(urls)); })() .catch(err => console.error(err)) .finally(() => browser?.close()) ;

Puntas:

• Ya está en código asíncrono, por lo que writeFileSync parece subóptimo aquí en relación con la versión asíncrona.
• Use let en lugar de var para que no lo i rompiendo el alcance y apareciendo con un valor obsoleto fuera (o dentro) del bloque de bucle.
• Considere newPage.close(); al final del bucle. Ahora solo estás haciendo 3 páginas, pero si esto es temporal y vas a hacer 800, entonces es una gran idea.
• "waitUntil": "networkidle0" es muy lento. Dado que todo lo que está haciendo es acceder a document.location.href , probablemente pueda acelerar las cosas con waitUntil: "domcontentloaded" .
• JS usa camelCase , no snake_case .
• Si tiene un ElementHandle, puede simplemente elementHandle.evaluate(...) en lugar de page.evaluate(..., elementHandle) .
• Detecte errores con catch y limpie el recurso del browser con finally .
• let browser; no tenía sentido en su código original.