refactor: use Promise.withResolvers in node-only flows and tests

[trivikr]

🔗 Linked issue

N/A

🧭 Context

This refactor replaces manually constructed deferred promises in Node-only code paths and related tests with Promise.withResolvers.

The goal is to make these flows easier to read, avoid external resolve variable capture, and align the codebase with the Node 24 runtime already required by the project.

📚 Description

This change updates the connector CLI and interactive npm client to use Promise.withResolvers for child-process and PTY lifecycle coordination, rather than wrapping those flows in new Promise(...) with externally assigned resolvers.

It also updates the affected Nuxt component tests to use the same pattern for mocked loading states. The behavior is unchanged; this is a readability and consistency refactor focused on code paths that only run in environments where Promise.withResolvers is available.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Apr 4, 2026 7:02pm

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Apr 4, 2026 7:02pm
npmx-lunaria	Ignored		Apr 4, 2026 7:02pm

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 916eb1bb-48bb-4466-b3a7-d0253fdbc746

📥 Commits

Reviewing files that changed from the base of the PR and between 23dc158 and ab078a1.

📒 Files selected for processing (4)

• cli/src/cli.ts
• cli/src/npm-client.ts
• test/nuxt/components/Package/Versions.spec.ts
• test/nuxt/components/VersionSelector.spec.ts

---

📝 Walkthrough

Walkthrough

This pull request refactors promise construction across multiple files to use Promise.withResolvers<T>() instead of manually creating promises with new Promise(...). The changes affect two production files in the CLI module (runNpmLogin() in cli/src/cli.ts and execNpmInteractive() in cli/src/npm-client.ts) and two test files that handle promise resolution patterns. The refactoring maintains existing functionality whilst changing how resolve functions are captured and wired, shifting code structure without altering observable runtime behaviour.

Suggested reviewers

• alexdln
• ghostdevv

🚥 Pre-merge checks | ✅ 1

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description check	✅ Passed	The pull request description clearly relates to the changeset, explaining the refactor from manual promise construction to Promise.withResolvers across CLI, npm client, and test files.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ghostdevv]

ty!!