Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

deps/inspection/commands/test/bugs.ts (1)

61-81: ⚡ Quick win

Add tests for remaining repository URL normalization branches.

Coverage currently validates git+https and trailing-slash trimming, but not git:// and git@github.com: paths handled by normalizeRepositoryUrl. Add both to prevent regressions in the advertised fallback behavior.

Proposed test additions

 test('bugs: normalizes git+https repository URL with .git suffix', async () => {
   mockOpen.mockClear()
   const dir = tempDir()
   fs.writeFileSync(path.join(dir, 'package.json'), JSON.stringify({
     name: 'test-pkg',
     repository: { url: 'git+https://github.com/test/pkg.git' },
   }))
   await bugs.handler({ ...BASE_OPTIONS, dir }, [])
   expect(mockOpen).toHaveBeenCalledWith('https://github.com/test/pkg/issues')
 })
 
+test('bugs: normalizes git:// repository URL', async () => {
+  mockOpen.mockClear()
+  const dir = tempDir()
+  fs.writeFileSync(path.join(dir, 'package.json'), JSON.stringify({
+    name: 'test-pkg',
+    repository: 'git://github.com/test/pkg.git',
+  }))
+  await bugs.handler({ ...BASE_OPTIONS, dir }, [])
+  expect(mockOpen).toHaveBeenCalledWith('https://github.com/test/pkg/issues')
+})
+
+test('bugs: normalizes git@github.com: repository URL', async () => {
+  mockOpen.mockClear()
+  const dir = tempDir()
+  fs.writeFileSync(path.join(dir, 'package.json'), JSON.stringify({
+    name: 'test-pkg',
+    repository: 'git@github.com:test/pkg.git',
+  }))
+  await bugs.handler({ ...BASE_OPTIONS, dir }, [])
+  expect(mockOpen).toHaveBeenCalledWith('https://github.com/test/pkg/issues')
+})
+
 test('bugs: trims trailing slash from repository URL before appending /issues', async () => {
   mockOpen.mockClear()
   const dir = tempDir()
   fs.writeFileSync(path.join(dir, 'package.json'), JSON.stringify({
     name: 'test-pkg',

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@deps/inspection/commands/test/bugs.ts` around lines 61 - 81, Add two tests to
deps/inspection/commands/test/bugs.ts that exercise the remaining branches of
normalizeRepositoryUrl: one where package.json repository is
"git://github.com/test/pkg.git" and one where it's "git@github.com:test/pkg.git"
(or similar SSH form). Follow the existing test pattern that clears mockOpen,
writes package.json into tempDir, calls await bugs.handler({ ...BASE_OPTIONS,
dir }, []), and asserts mockOpen was called with
'https://github.com/test/pkg/issues'; reference the existing tests using
mockOpen, tempDir, fs.writeFileSync, and bugs.handler to locate where to insert
these cases.

.changeset/pnpm-bugs-command.md (1)

6-6: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Use the issue URL (not pull URL) for #11279.

The text references linked issue #11279, but the hyperlink points to a PR path. Please switch to the issue URL for accurate release-note traceability.

Suggested patch

-Added the `pnpm bugs` command that opens a package's bug tracker URL in the browser. With no arguments, it reads the current project's `package.json`; with one or more package names, it fetches each package's metadata from the registry and opens its bug tracker. Falls back to `<repository>/issues` when the `bugs` field is missing [`#11279`](https://github.com/pnpm/pnpm/pull/11279).
+Added the `pnpm bugs` command that opens a package's bug tracker URL in the browser. With no arguments, it reads the current project's `package.json`; with one or more package names, it fetches each package's metadata from the registry and opens its bug tracker. Falls back to `<repository>/issues` when the `bugs` field is missing [`#11279`](https://github.com/pnpm/pnpm/issues/11279).

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.changeset/pnpm-bugs-command.md at line 6, Update the hyperlink for the
referenced ticket in the changelog entry: in .changeset/pnpm-bugs-command.md
replace the PR URL "https://github.com/pnpm/pnpm/pull/11279" with the issue URL
"https://github.com/pnpm/pnpm/issues/11279" so the reference to `#11279` points
to the issue rather than the pull request.

deps/inspection/commands/test/bugs.ts (1)

111-118: ⚡ Quick win

Tighten the registry lookup assertion to verify bug-tracker semantics.

startsWith('http') is too broad; this can pass on non-bug URLs. Assert an issues-like target (or an exact expected URL for the fixture package) to better lock behavior.

Suggested patch

   await bugs.handler(BASE_OPTIONS, ['is-negative'])
   expect(mockOpen).toHaveBeenCalledTimes(1)
   const calledUrl = mockOpen.mock.calls[0][0]
   expect(typeof calledUrl).toBe('string')
-  expect((calledUrl as string).startsWith('http')).toBe(true)
+  expect(calledUrl).toMatch(/^https?:\/\/.+\/issues\/?$/)

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@deps/inspection/commands/test/bugs.ts` around lines 111 - 118, The test
"bugs: looks up package on registry by name" currently asserts only that the
opened URL starts with "http", which is too loose; update the assertion on
calledUrl (from mockOpen.mock.calls[0][0]) to verify bug-tracker semantics
instead—either assert an exact expected URL for the fixture package
"is-negative" (preferred) or at minimum assert the URL contains an issues-like
path segment such as "/issues" or a known issue host (e.g., "github.com") so the
test guarantees the command opens a bug/issue page rather than any arbitrary
HTTP URL.

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes