fix(dashboard): rename x/y flags to col/row so Stricli can parse them (Sentry CLI-RN)

Stricli FLAG_NAME_PATTERN requires 2+ characters after "--", so flags
named "x" and "y" (producing --x and --y) were silently treated as
positional arguments. Users passing "--x 0 --y 0" got "too many
positional arguments" errors. Rename to --col/--row (matching grid
terminology) and add -x/-y as short aliases which Stricli handles via
its alias mechanism.

Co-authored-by: Miguel Betegón <miguelbetegongarcia@gmail.com>

Author: cursoragent

src/commands/dashboard/widget/add.ts

@@ -96,7 +96,7 @@ export const addCommand = buildCommand({
       "Sort shorthand (--sort flag):\n" +
       "  count          → count()         (ascending)\n" +
       "  -count         → -count()        (descending)\n\n" +
-      "Layout flags (--x, --y, --width, --height) control widget position\n" +
+      "Layout flags (--col/-x, --row/-y, --width, --height) control widget position\n" +
       "and size in the 6-column dashboard grid. Omitted values use auto-layout.",
   },
   output: {
@@ -156,13 +156,13 @@ export const addCommand = buildCommand({
         brief: "Result limit",
         optional: true,
       },
-      x: {
+      col: {
         kind: "parsed",
         parse: numberParser,
         brief: "Grid column position (0-based, 0–5)",
         optional: true,
       },
-      y: {
+      row: {
         kind: "parsed",
         parse: numberParser,
         brief: "Grid row position (0-based)",
@@ -195,6 +195,8 @@ export const addCommand = buildCommand({
       s: "sort",
       n: "limit",
       l: "layout",
+      x: "col",
+      y: "row",
     },
   },
   async *func(this: SentryContext, flags: AddFlags, ...args: string[]) {
@@ -256,8 +258,8 @@ export const addCommand = buildCommand({
     newWidget = assignDefaultLayout(newWidget, updateBody.widgets, layoutMode);
 
     const hasExplicitLayout =
-      flags.x !== undefined ||
-      flags.y !== undefined ||
+      flags.col !== undefined ||
+      flags.row !== undefined ||
       flags.width !== undefined ||
       flags.height !== undefined;
 
@@ -267,8 +269,8 @@ export const addCommand = buildCommand({
         ...newWidget,
         layout: {
           ...baseLayout,
-          ...(flags.x !== undefined && { x: flags.x }),
-          ...(flags.y !== undefined && { y: flags.y }),
+          ...(flags.col !== undefined && { x: flags.col }),
+          ...(flags.row !== undefined && { y: flags.row }),
           ...(flags.width !== undefined && { w: flags.width }),
           ...(flags.height !== undefined && { h: flags.height }),
         },
@@ -277,7 +279,7 @@ export const addCommand = buildCommand({
       // (e.g., --x 5 on a table widget with auto-width 6 → 5+6=11 > 6)
       const finalLayout = newWidget.layout ?? baseLayout;
       validateWidgetLayout(
-        { x: finalLayout.x, width: finalLayout.w },
+        { col: finalLayout.x, width: finalLayout.w },
         finalLayout
       );
     }

src/commands/dashboard/widget/edit.ts

@@ -85,8 +85,8 @@ function mergeLayout(
   existing: DashboardWidget
 ): DashboardWidget["layout"] {
   const hasChange =
-    flags.x !== undefined ||
-    flags.y !== undefined ||
+    flags.col !== undefined ||
+    flags.row !== undefined ||
     flags.width !== undefined ||
     flags.height !== undefined;
 
@@ -96,8 +96,8 @@ function mergeLayout(
 
   return {
     ...(existing.layout ?? FALLBACK_LAYOUT),
-    ...(flags.x !== undefined && { x: flags.x }),
-    ...(flags.y !== undefined && { y: flags.y }),
+    ...(flags.col !== undefined && { x: flags.col }),
+    ...(flags.row !== undefined && { y: flags.row }),
     ...(flags.width !== undefined && { w: flags.width }),
     ...(flags.height !== undefined && { h: flags.height }),
   };
@@ -198,7 +198,7 @@ export const editCommand = buildCommand({
       "The dashboard can be specified by numeric ID or title.\n" +
       "Identify the widget by --index (0-based) or --title.\n" +
       "Only provided flags are changed — omitted values are preserved.\n\n" +
-      "Layout flags (--x, --y, --width, --height) control widget position\n" +
+      "Layout flags (--col/-x, --row/-y, --width, --height) control widget position\n" +
       "and size in the 6-column dashboard grid.\n\n" +
       "Examples:\n" +
       "  sentry dashboard widget edit 12345 --title 'Error Rate' --display bar\n" +
@@ -282,13 +282,13 @@ export const editCommand = buildCommand({
         brief: "Result limit",
         optional: true,
       },
-      x: {
+      col: {
         kind: "parsed",
         parse: numberParser,
         brief: "Grid column position (0-based, 0–5)",
         optional: true,
       },
-      y: {
+      row: {
         kind: "parsed",
         parse: numberParser,
         brief: "Grid row position (0-based)",
@@ -316,6 +316,8 @@ export const editCommand = buildCommand({
       g: "group-by",
       s: "sort",
       n: "limit",
+      x: "col",
+      y: "row",
     },
   },
   async *func(this: SentryContext, flags: EditFlags, ...args: string[]) {

src/types/dashboard.ts

@@ -710,8 +710,8 @@ export function assignDefaultLayout(
 
 /** Shared layout flags accepted by widget add and edit commands */
 export type WidgetLayoutFlags = {
-  readonly x?: number;
-  readonly y?: number;
+  readonly col?: number;
+  readonly row?: number;
   readonly width?: number;
   readonly height?: number;
 };
@@ -751,11 +751,11 @@ export function validateWidgetLayout(
   flags: WidgetLayoutFlags,
   existing?: DashboardWidgetLayout
 ): void {
-  if (flags.x !== undefined) {
-    assertLayoutInt(flags.x, "x", 0, GRID_COLUMNS - 1);
+  if (flags.col !== undefined) {
+    assertLayoutInt(flags.col, "col", 0, GRID_COLUMNS - 1);
   }
-  if (flags.y !== undefined) {
-    assertLayoutInt(flags.y, "y", 0);
+  if (flags.row !== undefined) {
+    assertLayoutInt(flags.row, "row", 0);
   }
   if (flags.width !== undefined) {
     assertLayoutInt(flags.width, "width", 1, GRID_COLUMNS);
@@ -764,17 +764,17 @@ export function validateWidgetLayout(
     assertLayoutInt(flags.height, "height", 1);
   }
 
-  // Cross-validate x + width doesn't overflow the grid
-  const effectiveX = flags.x ?? existing?.x;
+  // Cross-validate col + width doesn't overflow the grid
+  const effectiveX = flags.col ?? existing?.x;
   const effectiveW = flags.width ?? existing?.w;
   if (
     effectiveX !== undefined &&
     effectiveW !== undefined &&
     effectiveX + effectiveW > GRID_COLUMNS
   ) {
     throw new ValidationError(
-      `Widget overflows the grid: x(${effectiveX}) + width(${effectiveW}) = ${effectiveX + effectiveW}, but the grid is ${GRID_COLUMNS} columns wide.`,
-      "x"
+      `Widget overflows the grid: col(${effectiveX}) + width(${effectiveW}) = ${effectiveX + effectiveW}, but the grid is ${GRID_COLUMNS} columns wide.`,
+      "col"
     );
   }
 }

test/commands/dashboard/widget/add.test.ts

@@ -269,7 +269,7 @@ describe("dashboard widget add", () => {
   // Layout flag tests
   // -------------------------------------------------------------------------
 
-  test("uses explicit layout when --x --y --width --height provided", async () => {
+  test("uses explicit layout when --col --row --width --height provided", async () => {
     const { context } = createMockContext();
     const func = await addCommand.loader();
     await func.call(
@@ -278,8 +278,8 @@ describe("dashboard widget add", () => {
         json: false,
         display: "line",
         query: ["count"],
-        x: 0,
-        y: 5,
+        col: 0,
+        row: 5,
         width: 6,
         height: 3,
       },
@@ -304,7 +304,7 @@ describe("dashboard widget add", () => {
         json: false,
         display: "big_number",
         query: ["count"],
-        x: 4,
+        col: 4,
       },
       "123",
       "Positioned Counter"
@@ -333,14 +333,14 @@ describe("dashboard widget add", () => {
     expect(err.message).toContain("--width");
   });
 
-  test("throws ValidationError when --x overflows with auto-layout default width", async () => {
-    // table display defaults to w=6, so --x 1 would produce x=1 + w=6 = 7 > 6
+  test("throws ValidationError when --col overflows with auto-layout default width", async () => {
+    // table display defaults to w=6, so --col 1 would produce x=1 + w=6 = 7 > 6
     const { context } = createMockContext();
     const func = await addCommand.loader();
     const err = await func
       .call(
         context,
-        { json: false, display: "table", query: ["count"], x: 1 },
+        { json: false, display: "table", query: ["count"], col: 1 },
         "123",
         "Wide Table"
       )
@@ -349,19 +349,19 @@ describe("dashboard widget add", () => {
     expect(err.message).toContain("overflows the grid");
   });
 
-  test("throws ValidationError for negative y", async () => {
+  test("throws ValidationError for negative row", async () => {
     const { context } = createMockContext();
     const func = await addCommand.loader();
     const err = await func
       .call(
         context,
-        { json: false, display: "line", query: ["count"], y: -1 },
+        { json: false, display: "line", query: ["count"], row: -1 },
         "123",
         "Bad Y"
       )
       .catch((e: Error) => e);
     expect(err).toBeInstanceOf(ValidationError);
-    expect(err.message).toContain("--y");
+    expect(err.message).toContain("--row");
   });
 
   test("auto-defaults orderby when group-by + limit provided", async () => {

test/commands/dashboard/widget/edit.test.ts

@@ -291,10 +291,10 @@ describe("dashboard widget edit", () => {
   // Layout flag tests
   // -------------------------------------------------------------------------
 
-  test("applies --x and --y layout flags to existing widget", async () => {
+  test("applies --col and --row layout flags to existing widget", async () => {
     const { context } = createMockContext();
     const func = await editCommand.loader();
-    await func.call(context, { json: false, index: 0, x: 4, y: 3 }, "123");
+    await func.call(context, { json: false, index: 0, col: 4, row: 3 }, "123");
 
     const body = updateDashboardSpy.mock.calls[0]?.[2];
     const edited = body.widgets[0];
@@ -310,7 +310,7 @@ describe("dashboard widget edit", () => {
     const func = await editCommand.loader();
     await func.call(
       context,
-      { json: false, index: 1, x: 0, width: 6, height: 4 },
+      { json: false, index: 1, col: 0, width: 6, height: 4 },
       "123"
     );
 
@@ -332,14 +332,14 @@ describe("dashboard widget edit", () => {
     expect(edited.layout).toEqual({ x: 0, y: 0, w: 2, h: 1 });
   });
 
-  test("throws ValidationError for x out of range", async () => {
+  test("throws ValidationError for col out of range", async () => {
     const { context } = createMockContext();
     const func = await editCommand.loader();
     const err = await func
-      .call(context, { json: false, index: 0, x: 6 }, "123")
+      .call(context, { json: false, index: 0, col: 6 }, "123")
       .catch((e: Error) => e);
     expect(err).toBeInstanceOf(ValidationError);
-    expect(err.message).toContain("--x");
+    expect(err.message).toContain("--col");
   });
 
   test("throws ValidationError for negative width", async () => {
@@ -352,8 +352,8 @@ describe("dashboard widget edit", () => {
     expect(err.message).toContain("--width");
   });
 
-  test("throws ValidationError when --x overflows with fallback width on layoutless widget", async () => {
-    // Widget without layout uses FALLBACK_LAYOUT (w=3), so --x 4 → 4+3=7 > 6
+  test("throws ValidationError when --col overflows with fallback width on layoutless widget", async () => {
+    // Widget without layout uses FALLBACK_LAYOUT (w=3), so --col 4 → 4+3=7 > 6
     getDashboardSpy.mockResolvedValueOnce({
       ...sampleDashboard,
       widgets: [
@@ -377,17 +377,17 @@ describe("dashboard widget edit", () => {
     const { context } = createMockContext();
     const func = await editCommand.loader();
     const err = await func
-      .call(context, { json: false, index: 0, x: 4 }, "123")
+      .call(context, { json: false, index: 0, col: 4 }, "123")
       .catch((e: Error) => e);
     expect(err).toBeInstanceOf(ValidationError);
     expect(err.message).toContain("overflows the grid");
   });
 
-  test("throws ValidationError when x + width overflows grid", async () => {
+  test("throws ValidationError when col + width overflows grid", async () => {
     const { context } = createMockContext();
     const func = await editCommand.loader();
     const err = await func
-      .call(context, { json: false, index: 0, x: 4, width: 4 }, "123")
+      .call(context, { json: false, index: 0, col: 4, width: 4 }, "123")
       .catch((e: Error) => e);
     expect(err).toBeInstanceOf(ValidationError);
     expect(err.message).toContain("overflows the grid");