Skip to content

Commit cfa4095

Browse files
committed
fix: prevent double normalization warning in two-arg path
parseOrgProjectArg already warns internally about slug normalization. Don't propagate 'normalized' flag from targetArgToTraceTarget so callers' warnIfNormalized won't emit a duplicate warning.
1 parent 66c8298 commit cfa4095

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

src/lib/trace-target.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -213,6 +213,10 @@ export function parseSlashSeparatedTraceTarget(
213213
* Convert a target argument + validated trace ID into a trace target.
214214
* Delegates org/project parsing to {@link parseOrgProjectArg}.
215215
*
216+
* Note: `parseOrgProjectArg` already emits slug normalization warnings
217+
* internally, so `normalized` is NOT propagated here to avoid double
218+
* warnings when callers also check via {@link warnIfNormalized}.
219+
*
216220
* @internal Exported for testing
217221
*/
218222
export function targetArgToTraceTarget(
@@ -228,7 +232,6 @@ export function targetArgToTraceTarget(
228232
traceId,
229233
org: parsed.org,
230234
project: parsed.project,
231-
normalized: parsed.normalized,
232235
};
233236

234237
case "org-all":
@@ -237,15 +240,13 @@ export function targetArgToTraceTarget(
237240
type: "org-scoped",
238241
traceId,
239242
org: parsed.org,
240-
normalized: parsed.normalized,
241243
};
242244

243245
case "project-search":
244246
return {
245247
type: "project-search",
246248
traceId,
247249
projectSlug: parsed.projectSlug,
248-
normalized: parsed.normalized,
249250
};
250251

251252
case "auto-detect":

0 commit comments

Comments
 (0)