Awaiting stderr results in script hanging, sometimes

[HarryPeach]

Hi, I'm writing a library for a terminal spinner, and sometimes the script will hang after writing.
I think I've narrowed it down to the Deno.write calls, as there seems to be some async issues going on.

The solution to a similar issue in #5515 was to await all write calls, which is what I've done but the error still persists.

The following code:

const kia = new Kia("test");
await kia.start();
await sleep(3000);
await kia.stop();

console.log("All done");

Results in this output and hang:

All the calls to Deno.write look like this:

export async function writeLine(
	rid: number,
	encoder: TextEncoder,
	text: string,
	indent?: number
) {
	await Deno.write(
		rid,
		encoder.encode(`\r${indent ? ESC + indent + "C" : ""}${text}`)
	);
}

And all calls are awaited

await writeLine(
			this.options.resource,
			this.textEncoder,
			`${flair} ${text}`,
			this.options.indent
		);

(the full code is at my Kia repo if you need more context)

[cknight]

Can you post a link to your repo? There's not enough context here to help you effectively.

[HarryPeach]

https://github.com/HarryPeach/kia/tree/dev

[bartlomieju]

Most likely same problem as in #4789

[HarryPeach]

I've tested and this issue still appears on deno version prior to 0.41.0, so it may not be the same issue

[bartlomieju]

@HarryPeach have you taken into account that Deno.write() might write less bytes than the buffer you give? Ie. you might need to call Deno.write() multiple times on the same buffer but with different offset. Can you try replacing Deno.write() calls with Deno.writeAll() to see if that helps?

[HarryPeach]

Replacing all instances of Deno.write() with Deno.writeAll() doesn't seem to fix it

[bnoordhuis]

@HarryPeach Do you have a small standalone test case I can try locally? I want to track this down but I need something minimal.

[crowlKats]

I have run the provided example and I cannot reproduce this, and since there has not been a reply from author since nearly 3 years to provide further information, i will close this as fixed.