fix(profile): address PR review comments

- Fix mismatched default periods: both list and view now use 24h default
- Handle duplicate segments with numeric suffixes (issues, issues2, etc.)
- Add --web/-w flag to profile list command
- Fix schema repair for transaction_aliases to use custom DDL with
  composite primary key

Author: BYK

src/commands/profile/list.ts

@@ -9,6 +9,7 @@ import { buildCommand, numberParser } from "@stricli/core";
 import type { SentryContext } from "../../context.js";
 import { getProject, listProfiledTransactions } from "../../lib/api-client.js";
 import { parseOrgProjectArg } from "../../lib/arg-parsing.js";
+import { openInBrowser } from "../../lib/browser.js";
 import {
   buildTransactionFingerprint,
   setTransactionAliases,
@@ -23,13 +24,15 @@ import {
   writeJson,
 } from "../../lib/formatters/index.js";
 import { resolveOrgAndProject } from "../../lib/resolve-target.js";
+import { buildProfilingSummaryUrl } from "../../lib/sentry-urls.js";
 import { buildTransactionAliases } from "../../lib/transaction-alias.js";
 import type { TransactionAliasEntry, Writer } from "../../types/index.js";
 
 type ListFlags = {
   readonly period: string;
   readonly limit: number;
   readonly json: boolean;
+  readonly web: boolean;
 };
 
 /** Valid period values */
@@ -101,8 +104,13 @@ export const listCommand = buildCommand({
         brief: "Output as JSON",
         default: false,
       },
+      web: {
+        kind: "boolean",
+        brief: "Open in browser",
+        default: false,
+      },
     },
-    aliases: { n: "limit" },
+    aliases: { n: "limit", w: "web" },
   },
   async func(
     this: SentryContext,
@@ -147,6 +155,16 @@ export const listCommand = buildCommand({
     // Set telemetry context
     setContext([resolvedTarget.org], [resolvedTarget.project]);
 
+    // Open in browser if requested
+    if (flags.web) {
+      await openInBrowser(
+        stdout,
+        buildProfilingSummaryUrl(resolvedTarget.org, resolvedTarget.project),
+        "profiling"
+      );
+      return;
+    }
+
     // Get project to retrieve numeric ID (required for profile API)
     const project = await getProject(
       resolvedTarget.org,

src/commands/profile/view.ts

@@ -92,7 +92,7 @@ export const viewCommand = buildCommand({
         kind: "parsed",
         parse: parsePeriod,
         brief: "Stats period: 1h, 24h, 7d, 14d, 30d",
-        default: "7d",
+        default: "24h",
       },
       limit: {
         kind: "parsed",

src/lib/db/schema.ts

@@ -372,8 +372,33 @@ export type RepairResult = {
   failed: string[];
 };
 
+/** Tables that require custom DDL (not auto-generated from TABLE_SCHEMAS) */
+const CUSTOM_DDL_TABLES = new Set(["transaction_aliases"]);
+
+function repairTransactionAliasesTable(
+  db: Database,
+  result: RepairResult
+): void {
+  if (tableExists(db, "transaction_aliases")) {
+    return;
+  }
+  try {
+    db.exec(TRANSACTION_ALIASES_DDL);
+    db.exec(TRANSACTION_ALIASES_INDEX);
+    result.fixed.push("Created table transaction_aliases");
+  } catch (e) {
+    const msg = e instanceof Error ? e.message : String(e);
+    result.failed.push(`Failed to create table transaction_aliases: ${msg}`);
+  }
+}
+
 function repairMissingTables(db: Database, result: RepairResult): void {
   for (const [tableName, ddl] of Object.entries(EXPECTED_TABLES)) {
+    // Skip tables that need custom DDL
+    if (CUSTOM_DDL_TABLES.has(tableName)) {
+      continue;
+    }
+
     if (tableExists(db, tableName)) {
       continue;
     }
@@ -385,6 +410,9 @@ function repairMissingTables(db: Database, result: RepairResult): void {
       result.failed.push(`Failed to create table ${tableName}: ${msg}`);
     }
   }
+
+  // Handle tables with custom DDL
+  repairTransactionAliasesTable(db, result);
 }
 
 function repairMissingColumns(db: Database, result: RepairResult): void {

src/lib/transaction-alias.ts

@@ -78,9 +78,36 @@ type TransactionInput = {
   projectSlug: string;
 };
 
+/**
+ * Disambiguate duplicate segments by appending numeric suffixes.
+ * e.g., ["issues", "events", "issues"] → ["issues", "events", "issues2"]
+ *
+ * @param segments - Array of extracted segments (may contain duplicates)
+ * @returns Array of unique segments with numeric suffixes for duplicates
+ */
+function disambiguateSegments(segments: string[]): string[] {
+  const seen = new Map<string, number>();
+  const result: string[] = [];
+
+  for (const segment of segments) {
+    const count = seen.get(segment) ?? 0;
+    seen.set(segment, count + 1);
+
+    if (count === 0) {
+      result.push(segment);
+    } else {
+      // Append numeric suffix for duplicates (issues2, issues3, etc.)
+      result.push(`${segment}${count + 1}`);
+    }
+  }
+
+  return result;
+}
+
 /**
  * Build aliases for a list of transactions.
  * Uses shortest unique prefix algorithm on extracted segments.
+ * Handles duplicate segments by appending numeric suffixes.
  *
  * @param transactions - Array of transaction inputs with org/project context
  * @returns Array of TransactionAliasEntry with idx, alias, and transaction
@@ -94,6 +121,17 @@ type TransactionInput = {
  * //   { idx: 1, alias: "i", transaction: "/api/0/organizations/{org}/issues/", ... },
  * //   { idx: 2, alias: "e", transaction: "/api/0/projects/{org}/{proj}/events/", ... },
  * // ]
+ *
+ * @example
+ * // Duplicate segments get numeric suffixes
+ * buildTransactionAliases([
+ *   { transaction: "/api/v1/issues/", ... },
+ *   { transaction: "/api/v2/issues/", ... },
+ * ])
+ * // => [
+ * //   { idx: 1, alias: "i", ... },   // from "issues"
+ * //   { idx: 2, alias: "is", ... },  // from "issues2" (disambiguated)
+ * // ]
  */
 export function buildTransactionAliases(
   transactions: TransactionInput[]
@@ -103,11 +141,14 @@ export function buildTransactionAliases(
   }
 
   // Extract segments from each transaction
-  const segments = transactions.map((t) =>
+  const rawSegments = transactions.map((t) =>
     extractTransactionSegment(t.transaction)
   );
 
-  // Find shortest unique prefixes for the segments
+  // Disambiguate duplicate segments with numeric suffixes
+  const segments = disambiguateSegments(rawSegments);
+
+  // Find shortest unique prefixes for the disambiguated segments
   const prefixMap = findShortestUniquePrefixes(segments);
 
   // Build result with 1-based indices