fix(alerts): seed logicType/conditions when issue alert edit adds first action filter

packages/cli/src/commands/alert/issues/edit.ts

@@ -123,7 +123,18 @@ function applyIssueEdits(
     const actionFilters = Array.isArray(body.actionFilters)
       ? (body.actionFilters as Record<string, unknown>[])
       : [];
-    const filter = (actionFilters[0] as Record<string, unknown>) ?? {};
+    const existingFilter = actionFilters[0] as
+      | Record<string, unknown>
+      | undefined;
+    // When the rule has no action filter yet, a partial edit (e.g. only
+    // --action) must still emit a valid DataConditionGroup. Seed the same
+    // defaults the create path uses so the group carries a logicType and a
+    // conditions array; the workflows API rejects a group without a logic
+    // type. An existing filter keeps its own logicType/conditions.
+    const filter: Record<string, unknown> = existingFilter ?? {
+      logicType: matchToLogicType(undefined),
+      conditions: [],
+    };
     if (actions !== undefined) {
       filter.actions = actions;
     }

packages/cli/test/commands/alert/issues/edit.test.ts

@@ -138,6 +138,48 @@ describe("alert issues edit", () => {
     });
   });
 
+  test("seeds logicType and conditions when the rule has no action filter yet", async () => {
+    const context = createContext();
+    resolveSpy.mockResolvedValue({ targets: [sampleTarget] });
+    getRuleSpy.mockResolvedValue(sampleRule);
+    getDocSpy.mockResolvedValue({
+      id: "42",
+      name: "Rule Alpha",
+      enabled: true,
+      config: { frequency: 30 },
+      triggers: {
+        logicType: "any-short",
+        conditions: [{ id: "old-condition" }],
+      },
+    });
+    putSpy.mockResolvedValue({ id: "42", name: "Rule Alpha", enabled: true });
+    const func = (await editCommand.loader()) as unknown as (
+      this: unknown,
+      flags: EditFlags,
+      arg: string
+    ) => Promise<void>;
+
+    await func.call(
+      context,
+      { action: ['{"id":"new-action"}'], json: true },
+      "test-org/test-project/42"
+    );
+
+    expect(putSpy).toHaveBeenCalledWith("test-org", "42", {
+      id: "42",
+      name: "Rule Alpha",
+      enabled: true,
+      config: { frequency: 30 },
+      triggers: {
+        logicType: "any-short",
+        conditions: [{ id: "old-condition" }],
+      },
+      actionFilters: [
+        { logicType: "all", conditions: [], actions: [{ id: "new-action" }] },
+      ],
+    });
+  });
+
   test("maps the workflow enabled field to a status label in output", async () => {
     const context = createContext();
     resolveSpy.mockResolvedValue({ targets: [sampleTarget] });