refactor: unify dry-run and normal output in project create

Both paths now construct a ProjectCreatedResult and return { data }
through buildCommand's output wrapper. The single formatProjectCreated
formatter handles both modes — dry-run adds a dryRun flag that adjusts
the heading and team source note wording.

Removes DryRunData type, formatDryRun function, and writeOutput import.
Net -32 lines.

Author: BYK

src/commands/project/create.ts

@@ -34,14 +34,7 @@ import {
   formatProjectCreated,
   type ProjectCreatedResult,
 } from "../../lib/formatters/human.js";
-import {
-  escapeMarkdownInline,
-  isPlainOutput,
-  mdKvTable,
-  renderMarkdown,
-  safeCodeSpan,
-} from "../../lib/formatters/markdown.js";
-import { writeOutput } from "../../lib/formatters/output.js";
+import { isPlainOutput } from "../../lib/formatters/markdown.js";
 import { buildMarkdownTable, type Column } from "../../lib/formatters/table.js";
 import { renderTextTable } from "../../lib/formatters/text-table.js";
 import { logger } from "../../lib/logger.js";
@@ -65,50 +58,6 @@ const log = logger.withTag("project.create");
 /** Usage hint template — base command without positionals */
 const USAGE_HINT = "sentry project create <org>/<name> <platform>";
 
-type DryRunData = {
-  organization: string;
-  team: string;
-  teamSource: string;
-  name: string;
-  slug: string;
-  platform: string;
-};
-
-/** Format dry-run preview as human-readable markdown */
-function formatDryRun(data: DryRunData): string {
-  const lines: string[] = [];
-
-  lines.push(
-    `## <muted>Dry run</muted> — project '${escapeMarkdownInline(data.name)}' in ${escapeMarkdownInline(data.organization)}`
-  );
-  lines.push("");
-
-  // Team source notes (same pattern as formatProjectCreated)
-  if (data.teamSource === "auto-created") {
-    lines.push(
-      `> **Note:** Would create team '${escapeMarkdownInline(data.team)}' (org has no teams).`
-    );
-    lines.push("");
-  } else if (data.teamSource === "auto-selected") {
-    lines.push(
-      `> **Note:** Would use team '${escapeMarkdownInline(data.team)}'. See all teams: \`sentry team list\``
-    );
-    lines.push("");
-  }
-
-  const kvRows: [string, string][] = [
-    ["Name", escapeMarkdownInline(data.name)],
-    ["Slug", safeCodeSpan(data.slug)],
-    ["Org", safeCodeSpan(data.organization)],
-    ["Team", safeCodeSpan(data.team)],
-    ["Platform", data.platform],
-  ];
-
-  lines.push(mdKvTable(kvRows));
-
-  return renderMarkdown(lines.join("\n"));
-}
-
 type CreateFlags = {
   readonly team?: string;
   readonly "dry-run": boolean;
@@ -440,23 +389,23 @@ export const createCommand = buildCommand({
       dryRun: flags["dry-run"],
     });
 
+    const expectedSlug = slugify(name);
+
     // Dry-run mode: show what would be created without creating it
     if (flags["dry-run"]) {
-      const dryRunData = {
-        organization: orgSlug,
-        team: team.slug,
+      const result: ProjectCreatedResult = {
+        project: { id: "", slug: expectedSlug, name, platform },
+        orgSlug,
+        teamSlug: team.slug,
         teamSource: team.source,
-        name,
-        slug: slugify(name),
-        platform,
+        requestedPlatform: platform,
+        dsn: null,
+        url: "",
+        slugDiverged: false,
+        expectedSlug,
+        dryRun: true,
       };
-
-      writeOutput(this.stdout, dryRunData, {
-        json: flags.json,
-        fields: flags.fields,
-        formatHuman: formatDryRun,
-      });
-      return;
+      return { data: result };
     }
 
     // Create the project
@@ -471,8 +420,6 @@ export const createCommand = buildCommand({
     // Fetch DSN (best-effort)
     const dsn = await tryGetPrimaryDsn(orgSlug, project.slug);
 
-    const expectedSlug = slugify(name);
-
     const result: ProjectCreatedResult = {
       project,
       orgSlug,

src/lib/formatters/human.ts

@@ -1631,6 +1631,8 @@ export type ProjectCreatedResult = {
   slugDiverged: boolean;
   /** The slug the user expected (derived from the project name) */
   expectedSlug: string;
+  /** When true, nothing was actually created — output uses tentative wording */
+  dryRun?: boolean;
 };
 
 /**
@@ -1644,35 +1646,45 @@ export type ProjectCreatedResult = {
  */
 export function formatProjectCreated(result: ProjectCreatedResult): string {
   const lines: string[] = [];
+  const dry = result.dryRun === true;
+  const nameEsc = escapeMarkdownInline(result.project.name);
+  const orgEsc = escapeMarkdownInline(result.orgSlug);
 
-  lines.push(
-    `## Created project '${escapeMarkdownInline(result.project.name)}' in ${escapeMarkdownInline(result.orgSlug)}`
-  );
+  // Heading
+  if (dry) {
+    lines.push(`## <muted>Dry run</muted> — project '${nameEsc}' in ${orgEsc}`);
+  } else {
+    lines.push(`## Created project '${nameEsc}' in ${orgEsc}`);
+  }
   lines.push("");
 
-  // Slug divergence note
+  // Slug divergence note (never applies in dry-run — we can't predict server renames)
   if (result.slugDiverged) {
     lines.push(
       `> **Note:** Slug \`${result.project.slug}\` was assigned because \`${result.expectedSlug}\` is already taken.`
     );
     lines.push("");
   }
 
-  // Team source notes
+  // Team source notes — tentative wording in dry-run
   if (result.teamSource === "auto-created") {
     lines.push(
-      `> **Note:** Created team '${escapeMarkdownInline(result.teamSlug)}' (org had no teams).`
+      dry
+        ? `> **Note:** Would create team '${escapeMarkdownInline(result.teamSlug)}' (org has no teams).`
+        : `> **Note:** Created team '${escapeMarkdownInline(result.teamSlug)}' (org had no teams).`
     );
     lines.push("");
   } else if (result.teamSource === "auto-selected") {
     lines.push(
-      `> **Note:** Using team '${escapeMarkdownInline(result.teamSlug)}'. See all teams: \`sentry team list\``
+      dry
+        ? `> **Note:** Would use team '${escapeMarkdownInline(result.teamSlug)}'. See all teams: \`sentry team list\``
+        : `> **Note:** Using team '${escapeMarkdownInline(result.teamSlug)}'. See all teams: \`sentry team list\``
     );
     lines.push("");
   }
 
   const kvRows: [string, string][] = [
-    ["Project", escapeMarkdownInline(result.project.name)],
+    ["Project", nameEsc],
     ["Slug", safeCodeSpan(result.project.slug)],
     ["Org", safeCodeSpan(result.orgSlug)],
     ["Team", safeCodeSpan(result.teamSlug)],
@@ -1681,13 +1693,19 @@ export function formatProjectCreated(result: ProjectCreatedResult): string {
   if (result.dsn) {
     kvRows.push(["DSN", safeCodeSpan(result.dsn)]);
   }
-  kvRows.push(["URL", result.url]);
+  if (result.url) {
+    kvRows.push(["URL", result.url]);
+  }
 
   lines.push(mdKvTable(kvRows));
-  lines.push("");
-  lines.push(
-    `*Tip: Use \`sentry project view ${result.orgSlug}/${result.project.slug}\` for details*`
-  );
+
+  // Tip footer — only when a real project exists to view
+  if (!dry) {
+    lines.push("");
+    lines.push(
+      `*Tip: Use \`sentry project view ${result.orgSlug}/${result.project.slug}\` for details*`
+    );
+  }
 
   return renderMarkdown(lines.join("\n"));
 }

test/commands/project/create.test.ts

@@ -691,11 +691,14 @@ describe("project create", () => {
 
     const output = stdoutWrite.mock.calls.map((c) => c[0]).join("");
     const parsed = JSON.parse(output);
-    expect(parsed.organization).toBe("acme-corp");
-    expect(parsed.team).toBe("engineering");
-    expect(parsed.name).toBe("my-app");
-    expect(parsed.slug).toBe("my-app");
-    expect(parsed.platform).toBe("node");
+    // Same ProjectCreatedResult shape as normal path
+    expect(parsed.orgSlug).toBe("acme-corp");
+    expect(parsed.teamSlug).toBe("engineering");
+    expect(parsed.project.name).toBe("my-app");
+    expect(parsed.project.slug).toBe("my-app");
+    expect(parsed.project.platform).toBe("node");
+    expect(parsed.dsn).toBeNull();
+    expect(parsed.dryRun).toBe(true);
 
     // Should NOT call createProject
     expect(createProjectSpy).not.toHaveBeenCalled();