Error: process.stdout.cursorTo is not a function
echo "OPENAI_API_KEY=api key..." > .env && humanify openai main.js export OPENAI_API_KEY="api key..." && humanify openai main.js OPENAI_TOKEN=api key... humanify openai main.js humanify --apiKey="api key..." main.js file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:67 process.stdout.cursorTo(0); ^
TypeError: process.stdout.cursorTo is not a function
at showPercentage (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:67:20)
at visitAllIdentifiers (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:56606:35)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:56720:12
at async unminify (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:196:27)
at async Command.
Node.js v20.17.0 file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:67 process.stdout.cursorTo(0); ^
TypeError: process.stdout.cursorTo is not a function
at showPercentage (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:67:20)
at visitAllIdentifiers (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:56606:35)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:56720:12
at async unminify (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:196:27)
at async Command.
Node.js v20.17.0 file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:67 process.stdout.cursorTo(0); ^
TypeError: process.stdout.cursorTo is not a function
at showPercentage (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:67:20)
at visitAllIdentifiers (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:56606:35)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:56720:12
at async unminify (file:///C:/Users/Rahim/AppData/Roaming/npm/node_modules/humanifyjs/dist/index.mjs:196:27)
at async Command.
Node.js v20.17.0 error: unknown option '--apiKey=api key...'
Usage: humanify [options] [command]
Unminify code using OpenAI's API or a local LLM
Which environment are you running humanify on? It seems that your terminal/stdout/something does not support clearing the row.
You should be able to fix this for now by using --verbose flag, which disables the clearing. I'll fix the underlying issue in coming versions.
And thank you for reporting the issue! 🙇
Thank you very much for your support
It worked fine when I used it before and now it doesn't work
Another thing I know is that I did console.log in different places but it doesn't display in the command lines but they are executing as far as I understand.
I checked into this more. Is it possible by any chance that you're trying to run humanify using an old node.js? It seems that stdout.cursorTo is introduced in node.js version 17, so this implies that your node.js version would have been under that. I confused that previously with stdout.clearLine, which would be absent in weird terminal environments
You can check via node --version
Closing due inactivity
process.stdout.cursorTo is only defined when the shell you're running in isn't capturing the output. You need to check process.stdout.isTTY before trying to access it.
Source: https://nodejs.org/api/process.html#a-note-on-process-io https://nodejs.org/api/tty.html#writestreamcursortox-y-callback
process.stdout.cursorTois only defined when the shell you're running in isn't capturing the output. You need to checkprocess.stdout.isTTYbefore trying to access it.
- https://nodejs.org/api/process.html#a-note-on-process-io
-
To check if a stream is connected to a TTY context, check the
isTTYproperty.
-
- https://nodejs.org/api/tty.html
-
When Node.js detects that it is being run with a text terminal ("TTY") attached,
process.stdinwill, by default, be initialized as an instance oftty.ReadStreamand bothprocess.stdoutandprocess.stderrwill, by default, be instances oftty.WriteStream. The preferred method of determining whether Node.js is being run within a TTY context is to check that the value of theprocess.stdout.isTTYproperty istrue
-
Is it possible by any chance that you're trying to run humanify using an old node.js? It seems that
stdout.cursorTois introduced in node.js version 17, so this implies that your node.js version would have been under that.
Based on the above, we can see this behaviour in node v22, so it wasn't just an issue with running an older version of node:
⇒ node --version
v22.16.0
⇒ node -p -e "Boolean(process.stdout.isTTY)"
true
⇒ node -p -e "Boolean(process.stdout.isTTY)" | cat
false
⇒ node -p -e "process.stdout.cursorTo(0)"
true
⇒ node -p -e "process.stdout.cursorTo(0)" | cat
[eval]:1
process.stdout.cursorTo(0)
^
TypeError: process.stdout.cursorTo is not a function
at [eval]:1:16
at runScriptInThisContext (node:internal/vm:209:10)
at node:internal/process/execution:449:12
at [eval]-wrapper:6:24
at runScriptInContext (node:internal/process/execution:447:60)
at evalFunction (node:internal/process/execution:87:30)
at evalScript (node:internal/process/execution:99:3)
at node:internal/main/eval_string:74:3
Node.js v22.16.0
I'll fix the underlying issue in coming versions.
@jehna So this is still a valid issue that should be fixed, and this issue should be re-opened to track that.
Usage:
https://github.com/jehna/humanify/blob/7beba2d32433e58bb77d0e1b0eda01c470fec3e2/src/progress.ts#L25-L31
Example of a potential fix:
export function showPercentage(percentage: number) {
const percentageStr = Math.round(percentage * 100);
const canUseCursorOps = process.stdout.isTTY
&& typeof process.stdout.clearLine === "function"
&& typeof process.stdout.cursorTo === "function";
if (!verbose.enabled && canUseCursorOps) {
process.stdout.clearLine(0);
process.stdout.cursorTo(0);
process.stdout.write(`Processing: ${percentageStr}%`);
} else {
verbose.log(`Processing: ${percentageStr}%`);
}
if (percentage === 1 && canUseCursorOps) {
process.stdout.write("\n");
}
}
Ideally a similar more explicit fix would be added to clearLine in showProgress as well, instead of just relying on the optional chaining syntax.
You can see that this would also be an issue with process.stdout.clearLine:
⇒ node -p -e "process.stdout.clearLine(0)" | cat
[eval]:1
process.stdout.clearLine(0)
^
TypeError: process.stdout.clearLine is not a function
at [eval]:1:16
at runScriptInThisContext (node:internal/vm:209:10)
at node:internal/process/execution:449:12
at [eval]-wrapper:6:24
at runScriptInContext (node:internal/process/execution:447:60)
at evalFunction (node:internal/process/execution:87:30)
at evalScript (node:internal/process/execution:99:3)
at node:internal/main/eval_string:74:3
Node.js v22.16.0
Except that in the code, you're calling process.stdout.clearLine with the optional chaining syntax like this:
https://github.com/jehna/humanify/blob/7beba2d32433e58bb77d0e1b0eda01c470fec3e2/src/progress.ts#L10
https://github.com/jehna/humanify/blob/7beba2d32433e58bb77d0e1b0eda01c470fec3e2/src/progress.ts#L28
Whereas for process.stdout.cursorTo you're not using the optional chaining syntax:
https://github.com/jehna/humanify/blob/7beba2d32433e58bb77d0e1b0eda01c470fec3e2/src/progress.ts#L29
Looking at the git blame / history, the progress feature seemed to be first added in 70e31966e405c60790073d032042e2b39fe448ce on Aug 9, 2024, and neither used the optional chaining syntax:
- https://github.com/jehna/humanify/commit/70e31966e405c60790073d032042e2b39fe448ce#diff-ae7dc0c4f05cb30c61f24ee7a16b994a4901803bdbcaf6c7dbedd40d6fd11386R24-R32
But then the optional chaining was added to clearLine in 05b6a8e261ebc6c9c4221e0ae1c34fe38c4531aa on Aug 9, 2024 as it broke the CI:
- https://github.com/jehna/humanify/commit/05b6a8e261ebc6c9c4221e0ae1c34fe38c4531aa
Since this issue is still valid, and I don't have the access to re-open it based on @ItsHarper 's findings/contribution; I created a new issue to track the ongoing bug, and how to fix it:
- https://github.com/jehna/humanify/issues/546