Skip to content

fix(issue): resolve shared links with the current API contract - #1628

Merged
betegon merged 1 commit into
mainfrom
bt/fix-shared-issue-resolution
Sep 23, 2026
Merged

betegon merged 1 commit into
mainfrom
bt/fix-shared-issue-resolution

Conversation

@betegon

@betegon betegon commented Sep 23, 2026

Copy link
Copy Markdown
Member

Summary

Shared issue links use an org-scoped public endpoint that returns id. The CLI was calling the retired unscoped endpoint and expecting groupID, so valid links could not resolve.

Resolve the organization from the URL or existing context, validate the shared issue ID, and fetch full details through the authenticated org-scoped API. Support canonical /organizations/<org>/share/issue/<shareId>/ URLs alongside SaaS subdomain links. Public share requests remain unauthenticated, with existing host-scoped custom headers preserved. Legacy links without organization context now produce a context error.

Supersedes #1401.

Test plan

  • 380 tests passed across the shared API, issue resolver, URL/argument parsers, and host/header security suites.
  • All six shared-link resolver regression cases fail against the original main implementation and pass with this change.
  • Full tsc --noEmit, pnpm run lint, pnpm run check:errors, and git diff --check passed.
  • Generated schema/docs/SDK prerequisites completed without tracked generated changes.
  • No live shared-link request, full unit suite, E2E suite, or packaging build was run.

@vercel

vercel Bot commented Sep 23, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
cli Ready Ready Preview Sep 23, 2026 3:36pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Sep 23, 2026 3:36pm UTC

Request Review

@betegon
betegon marked this pull request as ready for review September 23, 2026 17:46
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Sep 23, 2026
Comment thread packages/cli/src/lib/api/issues.ts
@betegon
betegon merged commit 127e6a4 into main Sep 23, 2026
35 checks passed
@betegon
betegon deleted the bt/fix-shared-issue-resolution branch September 23, 2026 18:25

This branch was successfully deployed

1 active and 1 inactive deployments
Preview – cli 9e667403 Deployed Sep 23, 2026 by vercel[bot]
Preview – sentry-local 9e667403 Deployed Sep 23, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant