fix(telemetry): centralize sentry.org/project tags in resolution functions

Move setOrgProjectContext() from individual command files into the shared
resolution functions in resolve-target.ts, trace-target.ts, and
dashboard/resolve.ts. This ensures every command that resolves an org or
project automatically gets sentry.org and sentry.project telemetry tags,
eliminating the class of bugs where a command forgets to call setContext().

Previously ~15 commands manually called this.setContext() after resolution,
while ~15 others (all dashboard/*, event/view, project/*, trial/*, org/view)
forgot — causing missing telemetry tags on error events.

Changes:
- Add setOrgProjectContext calls to 6 functions in resolve-target.ts
- Add setOrgProjectContext calls to 2 functions in trace-target.ts
- Add setOrgProjectContext call to resolveOrgFromTarget in dashboard/resolve.ts
- Remove setContext from SentryContext interface and buildContext
- Remove all manual setContext calls from 12 command files
- Update 35 test files to remove setContext from mock contexts
- Delete 7 test blocks that asserted setContext was called

Author: BYK

AGENTS.md

@@ -9,6 +9,7 @@ import { listDashboards } from "../../lib/api-client.js";
 import type { parseOrgProjectArg } from "../../lib/arg-parsing.js";
 import { ContextError, ValidationError } from "../../lib/errors.js";
 import { resolveOrg } from "../../lib/resolve-target.js";
+import { setOrgProjectContext } from "../../lib/telemetry.js";
 import { isAllDigits } from "../../lib/utils.js";
 import {
   type DashboardWidget,
@@ -52,9 +53,11 @@ export async function resolveOrgFromTarget(
   switch (parsed.type) {
     case "explicit":
     case "org-all":
+      setOrgProjectContext([parsed.org], []);
       return parsed.org;
     case "project-search":
     case "auto-detect": {
+      // resolveOrg already sets telemetry context
       const resolved = await resolveOrg({ cwd });
       if (!resolved) {
         throw new ContextError("Organization", usageHint);

src/commands/dashboard/resolve.ts

@@ -74,7 +74,7 @@ export const explainCommand = buildCommand({
   },
   async *func(this: SentryContext, flags: ExplainFlags, issueArg: string) {
     applyFreshFlag(flags);
-    const { cwd, setContext } = this;
+    const { cwd } = this;
 
     // Declare org outside try block so it's accessible in catch for error messages
     let resolvedOrg: string | undefined;
@@ -88,9 +88,6 @@ export const explainCommand = buildCommand({
       });
       resolvedOrg = org;
 
-      // Set telemetry context so SeerError events carry the org tag
-      setContext([org], []);
-
       // Ensure root cause analysis exists (triggers if needed)
       const state = await ensureRootCauseAnalysis({
         org,

src/commands/issue/explain.ts

@@ -74,6 +74,7 @@ import {
   toNumericId,
 } from "../../lib/resolve-target.js";
 import { getApiBaseUrl } from "../../lib/sentry-client.js";
+import { setOrgProjectContext } from "../../lib/telemetry.js";
 import type {
   ProjectAliasEntry,
   SentryIssue,
@@ -349,6 +350,7 @@ async function resolveTargetsFromParsedArg(
 
     case "explicit": {
       // Single explicit target — fetch project ID for API query param
+      setOrgProjectContext([parsed.org], [parsed.project]);
       const projectId = await fetchProjectId(parsed.org, parsed.project);
       return {
         targets: [
@@ -365,6 +367,7 @@ async function resolveTargetsFromParsedArg(
 
     case "org-all": {
       // List all projects in the specified org
+      setOrgProjectContext([parsed.org], []);
       const projects = await listProjects(parsed.org);
       const targets: ResolvedTarget[] = projects.map((p) => ({
         org: parsed.org,
@@ -460,6 +463,10 @@ async function resolveTargetsFromParsedArg(
         projectDisplay: m.name,
       }));
 
+      const uniqueOrgs = [...new Set(targets.map((t) => t.org))];
+      const uniqueProjects = [...new Set(targets.map((t) => t.project))];
+      setOrgProjectContext(uniqueOrgs, uniqueProjects);
+
       return {
         targets,
         footer:
@@ -819,7 +826,6 @@ async function fetchOrgAllIssues(
 type OrgAllIssuesOptions = {
   org: string;
   flags: ListFlags;
-  setContext: (orgs: string[], projects: string[]) => void;
 };
 
 /**
@@ -832,7 +838,7 @@ type OrgAllIssuesOptions = {
 async function handleOrgAllIssues(
   options: OrgAllIssuesOptions
 ): Promise<IssueListResult> {
-  const { org, flags, setContext } = options;
+  const { org, flags } = options;
   // Encode sort + query in context key so cursors from different searches don't collide.
   const contextKey = buildPaginationContextKey("org", org, {
     sort: flags.sort,
@@ -841,8 +847,6 @@ async function handleOrgAllIssues(
   });
   const cursor = resolveOrgCursor(flags.cursor, PAGINATION_KEY, contextKey);
 
-  setContext([org], []);
-
   let issuesResult: IssuesPage;
   try {
     issuesResult = await withProgress(
@@ -915,7 +919,6 @@ type ResolvedTargetsOptions = {
   parsed: ReturnType<typeof parseOrgProjectArg>;
   flags: ListFlags;
   cwd: string;
-  setContext: (orgs: string[], projects: string[]) => void;
 };
 
 /** Default --period value (used to detect user-implicit vs explicit). */
@@ -1049,15 +1052,11 @@ function build403Detail(originalDetail: string | undefined): string {
 async function handleResolvedTargets(
   options: ResolvedTargetsOptions
 ): Promise<IssueListResult> {
-  const { parsed, flags, cwd, setContext } = options;
+  const { parsed, flags, cwd } = options;
 
   const { targets, footer, skippedSelfHosted, detectedDsns } =
     await resolveTargetsFromParsedArg(parsed, cwd);
 
-  const orgs = [...new Set(targets.map((t) => t.org))];
-  const projects = [...new Set(targets.map((t) => t.project))];
-  setContext(orgs, projects);
-
   if (targets.length === 0) {
     if (skippedSelfHosted) {
       throw new ContextError("Organization and project", USAGE_HINT, [
@@ -1479,7 +1478,7 @@ export const listCommand = buildListCommand("issue", {
     },
   },
   async *func(this: SentryContext, flags: ListFlags, target?: string) {
-    const { cwd, setContext } = this;
+    const { cwd } = this;
 
     const parsed = parseOrgProjectArg(target);
 
@@ -1502,7 +1501,6 @@ export const listCommand = buildListCommand("issue", {
       handleResolvedTargets({
         ...ctx,
         flags,
-        setContext,
       });
 
     const result = (await dispatchOrgScopedList({
@@ -1521,7 +1519,6 @@ export const listCommand = buildListCommand("issue", {
           handleOrgAllIssues({
             org: ctx.parsed.org,
             flags,
-            setContext,
           }),
       },
     })) as IssueListResult;

src/commands/issue/list.ts

@@ -193,7 +193,7 @@ export const planCommand = buildCommand({
   },
   async *func(this: SentryContext, flags: PlanFlags, issueArg: string) {
     applyFreshFlag(flags);
-    const { cwd, setContext } = this;
+    const { cwd } = this;
 
     // Declare org outside try block so it's accessible in catch for error messages
     let resolvedOrg: string | undefined;
@@ -207,9 +207,6 @@ export const planCommand = buildCommand({
       });
       resolvedOrg = org;
 
-      // Set telemetry context so SeerError events carry the org tag
-      setContext([org], []);
-
       // Ensure root cause analysis exists (runs explain if needed)
       const state = await ensureRootCauseAnalysis({
         org,

src/commands/issue/plan.ts

@@ -40,6 +40,7 @@ import {
 } from "../../lib/resolve-target.js";
 import { parseSentryUrl } from "../../lib/sentry-url-parser.js";
 import { buildIssueUrl } from "../../lib/sentry-urls.js";
+import { setOrgProjectContext } from "../../lib/telemetry.js";
 import { isAllDigits } from "../../lib/utils.js";
 import type { SentryIssue } from "../../types/index.js";
 import { type AutofixState, isTerminalStatus } from "../../types/seer.js";
@@ -530,24 +531,28 @@ export async function resolveIssue(
   const parsed = parseIssueArg(issueArg);
   const commandHint = buildCommandHint(command, issueArg);
 
+  let result: ResolvedIssueResult;
+
   switch (parsed.type) {
     case "numeric":
-      return resolveNumericIssue(parsed.id, cwd, command);
+      result = await resolveNumericIssue(parsed.id, cwd, command);
+      break;
 
     case "explicit": {
       // Full context: org + project + suffix
       const org = await resolveEffectiveOrg(parsed.org);
       const fullShortId = expandToFullShortId(parsed.suffix, parsed.project);
       const issue = await getIssueByShortId(org, fullShortId);
-      return { org, issue };
+      result = { org, issue };
+      break;
     }
 
     case "explicit-org-numeric": {
       // Org + numeric ID — use org-scoped endpoint for proper region routing.
       const org = await resolveEffectiveOrg(parsed.org);
       try {
         const issue = await getIssueInOrg(org, parsed.numericId);
-        return { org, issue };
+        result = { org, issue };
       } catch (err) {
         if (err instanceof ApiError && err.status === 404) {
           throw new ResolutionError(
@@ -562,30 +567,40 @@ export async function resolveIssue(
         }
         throw err;
       }
+      break;
     }
 
     case "explicit-org-suffix": {
       // Org + suffix only - ambiguous without project, always errors
       const org = await resolveEffectiveOrg(parsed.org);
-      return resolveExplicitOrgSuffix(org, parsed.suffix, commandHint);
+      result = await resolveExplicitOrgSuffix(org, parsed.suffix, commandHint);
+      break;
     }
 
     case "project-search":
       // Project slug + suffix - search across orgs
-      return resolveProjectSearch(
+      result = await resolveProjectSearch(
         parsed.projectSlug,
         parsed.suffix,
         cwd,
         commandHint
       );
+      break;
 
     case "suffix-only":
       // Just suffix - need DSN for org and project
-      return resolveSuffixOnly(parsed.suffix, cwd, commandHint);
+      result = await resolveSuffixOnly(parsed.suffix, cwd, commandHint);
+      break;
 
     case "selector":
       // Magic @ selector - fetch top issue by sort criteria
-      return resolveSelector(parsed.selector, parsed.org, cwd, commandHint);
+      result = await resolveSelector(
+        parsed.selector,
+        parsed.org,
+        cwd,
+        commandHint
+      );
+      break;
 
     default: {
       // Exhaustive check - this should never be reached
@@ -595,6 +610,16 @@ export async function resolveIssue(
       );
     }
   }
+
+  // Set telemetry context from the resolved result
+  if (result.org) {
+    setOrgProjectContext(
+      [result.org],
+      result.issue.project?.slug ? [result.issue.project.slug] : []
+    );
+  }
+
+  return result;
 }
 
 /**

src/commands/issue/utils.ts

@@ -146,7 +146,7 @@ export const viewCommand = buildCommand({
   },
   async *func(this: SentryContext, flags: ViewFlags, issueArg: string) {
     applyFreshFlag(flags);
-    const { cwd, setContext } = this;
+    const { cwd } = this;
 
     // Resolve issue using shared resolution logic
     const { org: orgSlug, issue } = await resolveIssue({
@@ -155,12 +155,6 @@ export const viewCommand = buildCommand({
       command: "view",
     });
 
-    // Set telemetry context
-    setContext(
-      orgSlug ? [orgSlug] : [],
-      issue.project?.slug ? [issue.project.slug] : []
-    );
-
     if (flags.web) {
       await openInBrowser(issue.permalink, "issue");
       return;

src/commands/issue/view.ts

@@ -645,7 +645,7 @@ export const listCommand = buildListCommand(
       },
     },
     async *func(this: SentryContext, flags: ListFlags, ...args: string[]) {
-      const { cwd, setContext } = this;
+      const { cwd } = this;
 
       const parsed = parseLogListArgs(args);
 
@@ -657,8 +657,6 @@ export const listCommand = buildListCommand(
           cwd,
           TRACE_USAGE_HINT
         );
-        setContext([org], []);
-
         if (flags.follow) {
           // Banner (suppressed in JSON mode)
           writeFollowBanner(
@@ -725,8 +723,6 @@ export const listCommand = buildListCommand(
           cwd,
           COMMAND_NAME
         );
-        setContext([org], [project]);
-
         if (flags.follow) {
           writeFollowBanner(
             flags.follow ?? DEFAULT_POLL_INTERVAL,

src/commands/log/list.ts

@@ -349,7 +349,7 @@ export const viewCommand = buildCommand({
   },
   async *func(this: SentryContext, flags: ViewFlags, ...args: string[]) {
     applyFreshFlag(flags);
-    const { cwd, setContext } = this;
+    const { cwd } = this;
     const cmdLog = logger.withTag("log.view");
 
     // Parse positional args
@@ -365,9 +365,6 @@ export const viewCommand = buildCommand({
       throw new ContextError("Organization and project", USAGE_HINT);
     }
 
-    // Set telemetry context
-    setContext([target.org], [target.project]);
-
     if (flags.web) {
       await handleWebOpen(target.org, logIds);
       return;

src/commands/log/view.ts

@@ -239,7 +239,6 @@ function jsonTransformSpanList(data: SpanListData, fields?: string[]): unknown {
 type ModeContext = {
   cwd: string;
   flags: ListFlags;
-  setContext: (orgs: string[], projects: string[]) => void;
 };
 
 /**
@@ -259,8 +258,6 @@ async function handleTraceMode(
     cwd,
     TRACE_USAGE_HINT
   );
-  ctx.setContext([org], [project]);
-
   const queryParts = [`trace:${traceId}`];
   if (flags.query) {
     queryParts.push(translateSpanQuery(flags.query));
@@ -324,8 +321,6 @@ async function handleProjectMode(
     cwd,
     COMMAND_NAME
   );
-  ctx.setContext([org], [project]);
-
   const apiQuery = flags.query ? translateSpanQuery(flags.query) : undefined;
 
   const contextKey = buildPaginationContextKey(
@@ -447,9 +442,9 @@ export const listCommand = buildListCommand("span", {
     },
   },
   async *func(this: SentryContext, flags: ListFlags, ...args: string[]) {
-    const { cwd, setContext } = this;
+    const { cwd } = this;
     const parsed = parseSpanListArgs(args);
-    const modeCtx: ModeContext = { cwd, flags, setContext };
+    const modeCtx: ModeContext = { cwd, flags };
 
     const { output, hint } =
       parsed.mode === "trace"