fix: don't toggle cli cursor on non-TTY

[AriPerkkio]

Description

• Logger should not toggle CLI cursor when not in TTY

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

• It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
• Ideally, include a test that fails without this PR but passes with it.
• Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

• Run the tests with pnpm test:ci.

Documentation

• If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

• Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

[netlify]

✅ Deploy Preview for vitest-dev ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	f80a646
🔍 Latest deploy log	https://app.netlify.com/sites/vitest-dev/deploys/679351bcedc3310008181774
😎 Deploy Preview	https://deploy-preview-7336--vitest-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[netlify]

✅ Deploy Preview for vitest-dev ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	95810dd
🔍 Latest deploy log	https://app.netlify.com/sites/vitest-dev/deploys/6793520def699400083c543a
😎 Deploy Preview	https://deploy-preview-7336--vitest-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[hi-ogawa]

Are these removals also the part of the fix?

[AriPerkkio]

Yes, currently all onClose listeners were completely ignored for some reason.

vitest/packages/vitest/src/node/core.ts

Lines 120 to 124 in 926ca95

	constructor(
	public readonly mode: VitestRunMode,
	options: VitestOptions = {},
	) {
	this.logger = new Logger(this, options.stdout, options.stderr)

vitest/packages/vitest/src/node/logger.ts

Lines 320 to 325 in 926ca95

	this.ctx.onClose(() => {
	process.off('SIGINT', onExit)
	process.off('SIGTERM', onExit)
	process.off('exit', onExit)
	cleanup()
	})

vitest/packages/vitest/src/node/core.ts

Lines 545 to 546 in 926ca95

	async start(filters?: string[]): Promise<TestRunResult> {
	this._onClose = []

This prevented cli cursor clearing. The added test case was failing without this.

[hi-ogawa]

Thanks!