Skip to content

fix: normalize repository.url in npm placeholder package#423

Merged
nullvariant merged 1 commit into
mainfrom
fix/npm-placeholder-repository-url
Apr 1, 2026
Merged

fix: normalize repository.url in npm placeholder package#423
nullvariant merged 1 commit into
mainfrom
fix/npm-placeholder-repository-url

Conversation

@nullvariant
Copy link
Copy Markdown
Owner

Summary

  • Add git+ prefix to repository.url to match npm registry convention
  • Suppresses npm publish warning about URL normalization

Test plan

  • npm pkg fix no longer reports warnings for this field

Add git+ prefix to repository URL to match npm registry convention
and suppress npm publish warning.

🖥️ IDE: [Cursor](https://cursor.sh)
🔌 Extension: [Claude Code](https://claude.ai/download)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Model-Raw: claude-opus-4-6-20250415
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Normalize repository.url with git+ prefix in npm package

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Add git+ prefix to repository URL in npm package
• Matches npm registry convention for git repositories
• Suppresses npm publish warning about URL normalization
Diagram
flowchart LR
  A["package.json<br/>repository.url"] -- "add git+ prefix" --> B["git+https://github.com/..."]
Loading

Grey Divider

File Changes

1. packages/git-id-switcher-npm/package.json 🐞 Bug fix +1/-1

Add git+ prefix to repository URL

• Updated repository.url field to include git+ prefix
• Changed from https://github.com/... to git+https://github.com/...
• Aligns with npm registry conventions for git repositories
• Eliminates npm publish warnings about URL normalization

packages/git-id-switcher-npm/package.json


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented Apr 1, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@nullvariant-mimi
Copy link
Copy Markdown
Contributor

🐰 Mimi's Validation Report ✅

All checks are looking good! Great job! 🎉

⏳ Some checks are still running. I will keep watching!


バリデーターを通してくださいね

This report was carefully prepared by nullvariant-mimi[bot]

@nullvariant-luna
Copy link
Copy Markdown
Contributor

👧 Luna's Exploration Report 📦

No new dependencies added. Just version bumps! Nothing to explore here... 😴


Botに418返そうよ!

This report was curiously compiled by nullvariant-luna[bot]

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 4d76987.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@nullvariant-blaze
Copy link
Copy Markdown
Contributor

🐗 Blaze's Release Review 📦

No version bump. Just dependency changes... boring. Wake me up when it's deploy time! 😤


よっしゃ!デプロイしまくるぞ!

This review was ENTHUSIASTICALLY filed by nullvariant-blaze[bot]

@nullvariant-ciel
Copy link
Copy Markdown
Contributor

nullvariant-ciel Bot commented Apr 1, 2026

🕊️ Ciel's Mediation 🌤️

*~~ floating down from the clouds ~~ The zoo seems a bit noisy today...*

3 zoo members have reviewed this PR.

Zoo Member Status
🐰 Mimi Commented
👧 Luna Commented
🐗 Blaze Commented

⚖️ The zoo has mixed opinions. Some are concerned, some are fine with it. Please review each comment carefully and make the final call.


まあまあ、ほどほどに。

This mediation was peacefully delivered by nullvariant-ciel[bot]

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 1, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@nullvariant nullvariant merged commit 43e1645 into main Apr 1, 2026
34 of 37 checks passed
@nullvariant nullvariant deleted the fix/npm-placeholder-repository-url branch April 1, 2026 08:24
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.

1 participant