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

@@ -819,7 +819,6 @@ async function fetchOrgAllIssues(
 type OrgAllIssuesOptions = {
   org: string;
   flags: ListFlags;
-  setContext: (orgs: string[], projects: string[]) => void;
 };
 
 /**
@@ -832,7 +831,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 +840,6 @@ async function handleOrgAllIssues(
   });
   const cursor = resolveOrgCursor(flags.cursor, PAGINATION_KEY, contextKey);
 
-  setContext([org], []);
-
   let issuesResult: IssuesPage;
   try {
     issuesResult = await withProgress(
@@ -915,7 +912,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 +1045,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 +1471,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 +1494,6 @@ export const listCommand = buildListCommand("issue", {
       handleResolvedTargets({
         ...ctx,
         flags,
-        setContext,
       });
 
     const result = (await dispatchOrgScopedList({
@@ -1521,7 +1512,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

@@ -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"

src/commands/span/list.ts

@@ -265,20 +265,18 @@ export const viewCommand = buildCommand({
   },
   async *func(this: SentryContext, flags: ViewFlags, ...args: string[]) {
     applyFreshFlag(flags);
-    const { cwd, setContext } = this;
+    const { cwd } = this;
 
     // Parse positional args: first is trace target, rest are span IDs
     const { traceTarget, spanIds } = parsePositionalArgs(args);
     warnIfNormalized(traceTarget, "span.view");
 
     // Resolve org/project
-    const { traceId, org, project } = await resolveTraceOrgProject(
+    const { traceId, org } = await resolveTraceOrgProject(
       traceTarget,
       cwd,
       USAGE_HINT
     );
-    setContext([org], [project]);
-
     // Fetch trace data (single fetch for all span lookups)
     const timestamp = Math.floor(Date.now() / 1000);
     const spans = await getDetailedTrace(org, traceId, timestamp);