Skip to content

feat: add debug mode to npmx-connector#912

Merged
danielroe merged 4 commits intonpmx-dev:mainfrom
mikouaji:feat/npmx-connector-add-debug-and-new-otp-case
Feb 5, 2026
Merged

feat: add debug mode to npmx-connector#912
danielroe merged 4 commits intonpmx-dev:mainfrom
mikouaji:feat/npmx-connector-add-debug-and-new-otp-case

Conversation

@mikouaji
Copy link
Contributor

@mikouaji mikouaji commented Feb 4, 2026

Adds:

  • new pnpm dev:debug command that supplies extra logging
  • one more entry in the otpPatterns to detect differnt OTP request

@vercel
Copy link

vercel bot commented Feb 4, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 4, 2026 9:24pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 4, 2026 9:24pm
npmx-lunaria Ignored Ignored Feb 4, 2026 9:24pm

Request Review

@mikouaji mikouaji changed the title feat(npmx-connector): add debug mode feat: add debug mode to npmx-connector Feb 4, 2026
@codecov
Copy link

codecov bot commented Feb 4, 2026

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
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

A new npm script dev:debug was added to cli/package.json to run the CLI with DEBUG and NPMX_CLI_DEV environment variables. The cli/src/npm-client.ts file was modified to add debug logging across the npm execution path: before execution, on success, on failure, and within OTP and authentication-failure detection. The OTP detection patterns were extended to include an additional phrase.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description accurately reflects the changeset, documenting both the new debug command and additional OTP pattern detection.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

@danielroe danielroe added this pull request to the merge queue Feb 5, 2026
Merged via the queue into npmx-dev:main with commit 1f03429 Feb 5, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants