fix: preserve markdown links in table plain output by escaping user data at source

Fixes plain-mode table escaping that was destroying intentional markdown links in SHORT ID column. Moved escaping of user-supplied data (TITLE, NAME, URL fields) to column definitions following the pattern used by log/trace tables. Removed blanket escapeMarkdownCell from buildMarkdownTable to preserve intentional markdown syntax like [text](url) links.

Author: cursoragent

src/commands/org/list.ts

@@ -10,6 +10,7 @@ import { buildCommand } from "../../lib/command.js";
 import { DEFAULT_SENTRY_HOST } from "../../lib/constants.js";
 import { getAllOrgRegions } from "../../lib/db/regions.js";
 import { writeFooter, writeJson } from "../../lib/formatters/index.js";
+import { escapeMarkdownCell } from "../../lib/formatters/markdown.js";
 import { type Column, writeTable } from "../../lib/formatters/table.js";
 import { buildListLimitFlag, LIST_JSON_FLAG } from "../../lib/list-command.js";
 
@@ -108,7 +109,7 @@ export const listCommand = buildCommand({
       ...(showRegion
         ? [{ header: "REGION", value: (r: OrgRow) => r.region ?? "" }]
         : []),
-      { header: "NAME", value: (r) => r.name },
+      { header: "NAME", value: (r) => escapeMarkdownCell(r.name) },
     ];
 
     writeTable(stdout, rows, columns);

src/commands/project/list.ts

@@ -32,6 +32,7 @@ import {
 } from "../../lib/db/pagination.js";
 import { AuthError, ContextError } from "../../lib/errors.js";
 import { writeFooter, writeJson } from "../../lib/formatters/index.js";
+import { escapeMarkdownCell } from "../../lib/formatters/markdown.js";
 import { type Column, writeTable } from "../../lib/formatters/table.js";
 import {
   buildListCommand,
@@ -229,7 +230,7 @@ async function resolveOrgsForAutoDetect(cwd: string): Promise<OrgResolution> {
 const PROJECT_COLUMNS: Column<ProjectWithOrg>[] = [
   { header: "ORG", value: (p) => p.orgSlug || "" },
   { header: "PROJECT", value: (p) => p.slug },
-  { header: "NAME", value: (p) => p.name },
+  { header: "NAME", value: (p) => escapeMarkdownCell(p.name) },
   { header: "PLATFORM", value: (p) => p.platform || "" },
 ];
 

src/commands/repo/list.ts

@@ -14,6 +14,7 @@ import {
   listRepositories,
   listRepositoriesPaginated,
 } from "../../lib/api-client.js";
+import { escapeMarkdownCell } from "../../lib/formatters/markdown.js";
 import { type Column, writeTable } from "../../lib/formatters/table.js";
 import {
   buildOrgListCommand,
@@ -31,10 +32,10 @@ type RepositoryWithOrg = SentryRepository & { orgSlug?: string };
 /** Column definitions for the repository table. */
 const REPO_COLUMNS: Column<RepositoryWithOrg>[] = [
   { header: "ORG", value: (r) => r.orgSlug || "" },
-  { header: "NAME", value: (r) => r.name },
+  { header: "NAME", value: (r) => escapeMarkdownCell(r.name) },
   { header: "PROVIDER", value: (r) => r.provider.name },
   { header: "STATUS", value: (r) => r.status },
-  { header: "URL", value: (r) => r.url || "" },
+  { header: "URL", value: (r) => escapeMarkdownCell(r.url || "") },
 ];
 
 /** Shared config that plugs into the org-list framework. */

src/commands/team/list.ts

@@ -15,6 +15,7 @@ import {
   listTeams,
   listTeamsPaginated,
 } from "../../lib/api-client.js";
+import { escapeMarkdownCell } from "../../lib/formatters/markdown.js";
 import { type Column, writeTable } from "../../lib/formatters/table.js";
 import {
   buildOrgListCommand,
@@ -33,7 +34,7 @@ type TeamWithOrg = SentryTeam & { orgSlug?: string };
 const TEAM_COLUMNS: Column<TeamWithOrg>[] = [
   { header: "ORG", value: (t) => t.orgSlug || "" },
   { header: "SLUG", value: (t) => t.slug },
-  { header: "NAME", value: (t) => t.name },
+  { header: "NAME", value: (t) => escapeMarkdownCell(t.name) },
   {
     header: "MEMBERS",
     value: (t) => String(t.memberCount ?? ""),

src/lib/formatters/human.ts

@@ -477,7 +477,7 @@ export function writeIssueTable(
     },
     {
       header: "TITLE",
-      value: ({ issue }) => issue.title,
+      value: ({ issue }) => escapeMarkdownCell(issue.title),
     }
   );
 

src/lib/formatters/table.ts

@@ -55,7 +55,7 @@ export function buildMarkdownTable<T>(
   const rows = items
     .map(
       (item) =>
-        `| ${columns.map((c) => escapeMarkdownCell(stripColorTags(c.value(item)))).join(" | ")} |`
+        `| ${columns.map((c) => stripColorTags(c.value(item))).join(" | ")} |`
     )
     .join("\n");
   return `${header}\n${separator}\n${rows}`;