lvin
lvin16mo ago

not sure why this code is erroring

for (const number of genNumbers()) {
const result = await fetchPages(number);
if (result instanceof Error) break;
const urls = parse(result);
await Promise.all(
urls.map((url) =>
fetchPage(`https://test.nl${url}`).then(({ name, videos }) =>
Deno.writeTextFile(name, JSON.stringify(videos))
)
)
);
}
for (const number of genNumbers()) {
const result = await fetchPages(number);
if (result instanceof Error) break;
const urls = parse(result);
await Promise.all(
urls.map((url) =>
fetchPage(`https://test.nl${url}`).then(({ name, videos }) =>
Deno.writeTextFile(name, JSON.stringify(videos))
)
)
);
}
Task generate deno run -A ./scripts/generate.ts
error: Uncaught TypeError: Expected string at position 1
Deno.writeTextFile(name, JSON.stringify(videos))
^
at writeFile (ext:deno_fs/30_fs.js:807:18)
at Object.writeTextFile (ext:deno_fs/30_fs.js:850:12)
at file:///home/al/Code/crookedy/scripts/generate.ts:76:14
at eventLoopTick (ext:core/01_core.js:183:11)
at async Promise.all (index 10)
at async file:///home/al/Code/crookedy/scripts/generate.ts:73:3
Task generate deno run -A ./scripts/generate.ts
error: Uncaught TypeError: Expected string at position 1
Deno.writeTextFile(name, JSON.stringify(videos))
^
at writeFile (ext:deno_fs/30_fs.js:807:18)
at Object.writeTextFile (ext:deno_fs/30_fs.js:850:12)
at file:///home/al/Code/crookedy/scripts/generate.ts:76:14
at eventLoopTick (ext:core/01_core.js:183:11)
at async Promise.all (index 10)
at async file:///home/al/Code/crookedy/scripts/generate.ts:73:3
9 Replies
NDH
NDH16mo ago
may have to await the parse(result)?
lvin
lvin16mo ago
no that function is synchrous, and using writeTextFileSync doesn't help either
Esente
Esente16mo ago
Can you check if you actually get a string for name? The error says that the first argument for writeTextFile is not a string.
lvin
lvin16mo ago
horrible error message
Mr.Possumz
Mr.Possumz16mo ago
If Promise.all() is expecting an iterable of promises, technically what you're returning is a composition of a promise and a chained statement. I don't often use Promise.all but I'm betting that fetchPage is generating the promise and passing the promise to the chained .then() statement, which fails to destructure the name and videos variables. You probably just need to declare the map callback as async to generate a promise.
lvin
lvin16mo ago
nah, functions only need to be tagged w/ async if im using await inside them. the map converts all the urls Array<string> to Array<Promise<{name:string,videos:Array<Video>}>> hence why i can use promise.all with it the name parameter for Deno.writefile was undefined instead of a string so thats why shit borked
Kyiro
Kyiro16mo ago
console log videos
NeTT
NeTT16mo ago
That was a poor error message indeed. Might need to open an issue for enhancement.
ybabts
ybabts16mo ago
It would be helpful to share the functions you're using inside your snippet. Whatever object you're destructuring in the then statement, the name property isn't always a string, hence the error. You should log the output of the object you're destructuring to confirm this. The solution would be to type check name and videos to ensure that they are the proper types before attempting to write them to a file. Also I am interested in what this is here
const result = await fetchPages(number);
if (result instanceof Error) break;
const result = await fetchPages(number);
if (result instanceof Error) break;