refactor(init): scope numeric org ID fix to sentry init only

The previous fix to resolveOrgFromDsn was too broad — returning null
broke DSN org auto-detection for read commands like `sentry issue list`
that work fine with numeric org IDs.

Revert resolveOrgFromDsn to its original behavior. Instead, handle the
numeric ID resolution in resolveOrgSlug (local-ops.ts), which is only
used by sentry init's project creation path. When the prefetched org is
a raw numeric string, attempt to resolve it to a slug via the org regions
cache (getOrgByNumericId). On cache miss — empty cache or org from a
different Sentry account — fall through to listOrganizations() so the
user can select from their accessible orgs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: betegon

src/lib/init/local-ops.ts

@@ -675,7 +675,19 @@ async function resolveOrgSlug(
 ): Promise<string | LocalOpResult> {
   const resolved = await resolveOrgPrefetched(cwd);
   if (resolved) {
-    return resolved.org;
+    // If the detected org is a raw numeric ID (extracted from a DSN), try to
+    // resolve it to a real slug. Numeric IDs can fail for write operations like
+    // project/team creation, and may belong to a different Sentry account.
+    if (/^\d+$/.test(resolved.org)) {
+      const { getOrgByNumericId } = await import("../db/regions.js");
+      const match = await getOrgByNumericId(resolved.org);
+      if (match) {
+        return match.slug;
+      }
+      // Cache miss — fall through to listOrganizations() for proper selection
+    } else {
+      return resolved.org;
+    }
   }
 
   // Fallback: list user's organizations (SQLite-cached after login/first call)

src/lib/resolve-target.ts

@@ -231,18 +231,11 @@ export async function resolveOrgFromDsn(
     }
   }
 
-  // Check org regions cache for a slug mapped to this numeric ID.
-  // The cache is populated by `sentry auth login` and `org list`.
-  const { getOrgByNumericId } = await import("./db/regions.js");
-  const orgMatch = await getOrgByNumericId(dsn.orgId);
-  if (orgMatch) {
-    return { org: orgMatch.slug, detectedFrom };
-  }
-
-  // Can't resolve to slug (empty cache or org belongs to a different account).
-  // Return null so the caller falls through to listOrganizations() for proper
-  // interactive org selection instead of passing a raw numeric ID to the API.
-  return null;
+  // Fall back to numeric org ID (API accepts both slug and numeric ID)
+  return {
+    org: dsn.orgId,
+    detectedFrom,
+  };
 }
 
 /**

test/isolated/resolve-target.test.ts

@@ -53,7 +53,6 @@ const mockGetProject = mock(() =>
 );
 const mockFindProjectByDsnKey = mock(() => Promise.resolve(null));
 const mockFindProjectsByPattern = mock(() => Promise.resolve([]));
-const mockGetOrgByNumericId = mock(() => Promise.resolve(undefined));
 
 // Mock all dependency modules
 mock.module("../../src/lib/db/defaults.js", () => ({
@@ -92,10 +91,6 @@ mock.module("../../src/lib/api-client.js", () => ({
   findProjectsByPattern: mockFindProjectsByPattern,
 }));
 
-mock.module("../../src/lib/db/regions.js", () => ({
-  getOrgByNumericId: mockGetOrgByNumericId,
-}));
-
 import { ContextError } from "../../src/lib/errors.js";
 // Now import the module under test (after mocks are set up)
 import {
@@ -123,7 +118,6 @@ function resetAllMocks() {
   mockGetProject.mockReset();
   mockFindProjectByDsnKey.mockReset();
   mockFindProjectsByPattern.mockReset();
-  mockGetOrgByNumericId.mockReset();
 
   // Set sensible defaults
   mockGetDefaultOrganization.mockResolvedValue(null);
@@ -145,7 +139,6 @@ function resetAllMocks() {
   mockGetCachedProject.mockResolvedValue(null);
   mockGetCachedProjectByDsnKey.mockResolvedValue(null);
   mockGetCachedDsn.mockResolvedValue(null);
-  mockGetOrgByNumericId.mockResolvedValue(undefined);
   mockFindProjectsByPattern.mockResolvedValue([]);
 }
 
@@ -205,26 +198,7 @@ describe("resolveOrg", () => {
     expect(mockDetectDsn).toHaveBeenCalled();
   });
 
-  test("returns null when DSN detected but no project/org cache", async () => {
-    mockGetDefaultOrganization.mockResolvedValue(null);
-    mockDetectDsn.mockResolvedValue({
-      raw: "https://abc@o123.ingest.sentry.io/456",
-      protocol: "https",
-      publicKey: "abc",
-      host: "o123.ingest.sentry.io",
-      projectId: "456",
-      orgId: "123",
-      source: "env",
-    });
-    mockGetCachedProject.mockResolvedValue(null);
-    mockGetOrgByNumericId.mockResolvedValue(undefined);
-
-    const result = await resolveOrg({ cwd: "/test" });
-
-    expect(result).toBeNull();
-  });
-
-  test("resolves org slug from regions cache when project cache misses", async () => {
+  test("returns numeric orgId when DSN detected but no cache", async () => {
     mockGetDefaultOrganization.mockResolvedValue(null);
     mockDetectDsn.mockResolvedValue({
       raw: "https://abc@o123.ingest.sentry.io/456",
@@ -236,16 +210,11 @@ describe("resolveOrg", () => {
       source: "env",
     });
     mockGetCachedProject.mockResolvedValue(null);
-    mockGetOrgByNumericId.mockResolvedValue({
-      slug: "my-org",
-      regionUrl: "https://sentry.io",
-    });
 
     const result = await resolveOrg({ cwd: "/test" });
 
     expect(result).not.toBeNull();
-    expect(result?.org).toBe("my-org");
-    expect(mockGetOrgByNumericId).toHaveBeenCalledWith("123");
+    expect(result?.org).toBe("123");
   });
 
   test("returns null when no org found from any source", async () => {