El "bloque interno" es una devolución de llamada a la instrucción fs.readFile . Lo que sucede es

1. fs.readFile se ejecuta e inicia una lectura asíncrona del archivo.
2. La siguiente instrucción se ejecuta, generando un "2".
3. Los pasos 1 y 2 se repiten para cada archivo en el directorio, de modo que se generan dos "2" en total.
4. Más tarde, cuando se completa la lectura de un archivo, se ejecuta el "bloque interno", generando un "1".
5. Dado que se leen dos archivos en total, el paso 4 se ejecuta dos veces y se emiten dos "1". Pero el primer "1" que se emite no pertenece necesariamente al primer archivo que se lee, porque un archivo más corto puede haber terminado de leerse antes que uno más largo, incluso si comenzó más tarde.

Si desea evitar tal asincronía, puede usar fs.readFileSync (y también fs.readdirSync ). Pero esto ralentizará la ejecución general, porque los archivos se leen uno tras otro, no en paralelo. Si el orden de las entradas en su portfolio.js no importa, es preferible la asincronía.

Tenga en cuenta, sin embargo, que agregar varios stock_details a un portfolio.js no hace que Javascript sea válido en general:

 {"Stock 1":100}{"Stock 2":200}

Suponiendo que cada archivo contiene líneas como

 A: 1 B: 2

(sin una nueva línea final), puede leerlos y procesarlos de forma asíncrona con el " sabor de promesas " del paquete fs :

 const fs = require("fs/promises"); fs.readdir("./my_stocks") .then(function(files) { var promises = []; for(each in files){ var file=files[each]; if(file!='portfolio.js'){ var fn="./my_stocks/"+file; promises.push(fs.readFile(fn) .then(data => data.toString().split('\n'))); } } return Promise.all(promises); }) .then(portfolio => fs.writeFile("./my_stocks/portfolio.js", JSON.stringify(portfolio)));