fix(blueprint): adding delete (#111)

## Description

<!-- Provide a brief description of your changes -->

**Note:** PR titles should follow [Conventional
Commits](https://www.conventionalcommits.org/) format (e.g.,
`feat(devbox): add support for custom env vars` or `fix(snapshot):
resolve pagination issue`) as they are used for automatic release notes
generation.

## Type of Change

<!-- Mark the relevant option with an 'x' -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] Documentation update
- [ ] Code refactoring
- [ ] Performance improvement
- [ ] Test updates

## Related Issues

<!-- Link to related issues using #issue-number -->
Closes #

## Changes Made

<!-- Describe the changes in detail -->

## Testing

<!-- Describe how you tested your changes -->

- [ ] I have tested locally
- [ ] I have added/updated tests
- [ ] All existing tests pass

## Checklist

- [ ] My code follows the code style of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published

## Screenshots (if applicable)

<!-- Add screenshots to help explain your changes -->

## Additional Notes

<!-- Any additional information that reviewers should know -->

Author: dines-rl

README.md

@@ -119,6 +119,7 @@ rli blueprint list                       # List all blueprints
 rli blueprint create                     # Create a new blueprint
 rli blueprint get <name-or-id>           # Get blueprint details by name or ID (...
 rli blueprint logs <name-or-id>          # Get blueprint build logs by name or I...
+rli blueprint delete <id>                # Delete a blueprint by ID
 rli blueprint prune <name>               # Delete old blueprint builds, keeping ...
 rli blueprint from-dockerfile            # Create a blueprint from a Dockerfile ...
 ```

src/commands/blueprint/delete.ts

@@ -0,0 +1,30 @@
+/**
+ * Delete blueprint command
+ */
+
+import { getClient } from "../../utils/client.js";
+import { output, outputError } from "../../utils/output.js";
+
+interface DeleteOptions {
+  output?: string;
+}
+
+export async function deleteBlueprint(id: string, options: DeleteOptions = {}) {
+  try {
+    const client = getClient();
+
+    await client.blueprints.delete(id);
+
+    // Default: just output the ID for easy scripting
+    if (!options.output || options.output === "text") {
+      console.log(id);
+    } else {
+      output(
+        { id, status: "deleted" },
+        { format: options.output, defaultFormat: "json" },
+      );
+    }
+  } catch (error) {
+    outputError("Failed to delete blueprint", error);
+  }
+}

src/commands/blueprint/prune.ts

@@ -75,6 +75,7 @@ function categorizeBlueprints(blueprints: Blueprint[], keepCount: number) {
   successful.sort((a, b) => (b.create_time_ms || 0) - (a.create_time_ms || 0));
 
   // Determine what to keep and delete
+  // keepCount of 0 means delete all (including successful builds)
   const toKeep = successful.slice(0, keepCount);
   const toDelete = [...successful.slice(keepCount), ...failed];
 
@@ -136,7 +137,11 @@ function displaySummary(
   // Show what will be kept
   console.log(`\nKeeping (${result.toKeep.length} most recent successful):`);
   if (result.toKeep.length === 0) {
-    console.log("  (none - no successful builds found)");
+    if (result.successful.length === 0) {
+      console.log("  (none - no successful builds found)");
+    } else {
+      console.log("  (none)");
+    }
   } else {
     for (const blueprint of result.toKeep) {
       console.log(
@@ -241,8 +246,8 @@ export async function pruneBlueprints(
     const autoConfirm = options.yes || false;
     const keepCount = parseInt(options.keep || "1", 10);
 
-    if (isNaN(keepCount) || keepCount < 1) {
-      outputError("--keep must be a positive integer");
+    if (isNaN(keepCount) || keepCount < 0) {
+      outputError("--keep must be a non-negative integer");
     }
 
     // Fetch all blueprints with the given name

src/utils/commands.ts

@@ -468,6 +468,20 @@ export function createProgram(): Command {
       await getBlueprintLogs({ id, ...options });
     });
 
+  blueprint
+    .command("delete <id>")
+    .description("Delete a blueprint by ID")
+    .alias("rm")
+    .option(
+      "-o, --output [format]",
+      "Output format: text|json|yaml (default: text)",
+    )
+    .action(async (id, options) => {
+      const { deleteBlueprint } =
+        await import("../commands/blueprint/delete.js");
+      await deleteBlueprint(id, options);
+    });
+
   blueprint
     .command("prune <name>")
     .description(

tests/__tests__/commands/blueprint/delete.test.ts

@@ -0,0 +1,131 @@
+/**
+ * Tests for blueprint delete command
+ */
+
+import { jest, describe, it, expect, beforeEach } from "@jest/globals";
+
+// Mock dependencies using the path alias
+const mockDelete = jest.fn();
+jest.unstable_mockModule("@/utils/client.js", () => ({
+  getClient: () => ({
+    blueprints: {
+      delete: mockDelete,
+    },
+  }),
+}));
+
+const mockOutput = jest.fn();
+const mockOutputError = jest.fn();
+jest.unstable_mockModule("@/utils/output.js", () => ({
+  output: mockOutput,
+  outputError: mockOutputError,
+}));
+
+describe("deleteBlueprint", () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+    (console.log as jest.Mock).mockClear();
+    mockDelete.mockReset();
+    mockOutput.mockReset();
+    mockOutputError.mockReset();
+  });
+
+  it("should delete a blueprint by ID", async () => {
+    mockDelete.mockResolvedValue(undefined);
+
+    const { deleteBlueprint } = await import(
+      "@/commands/blueprint/delete.js"
+    );
+    await deleteBlueprint("bpt_abc123", {});
+
+    expect(mockDelete).toHaveBeenCalledWith("bpt_abc123");
+    expect(console.log).toHaveBeenCalledWith("bpt_abc123");
+  });
+
+  it("should output JSON format when requested", async () => {
+    mockDelete.mockResolvedValue(undefined);
+
+    const { deleteBlueprint } = await import(
+      "@/commands/blueprint/delete.js"
+    );
+    await deleteBlueprint("bpt_json123", { output: "json" });
+
+    expect(mockDelete).toHaveBeenCalledWith("bpt_json123");
+    expect(mockOutput).toHaveBeenCalledWith(
+      { id: "bpt_json123", status: "deleted" },
+      { format: "json", defaultFormat: "json" },
+    );
+    expect(console.log).not.toHaveBeenCalledWith("bpt_json123");
+  });
+
+  it("should output YAML format when requested", async () => {
+    mockDelete.mockResolvedValue(undefined);
+
+    const { deleteBlueprint } = await import(
+      "@/commands/blueprint/delete.js"
+    );
+    await deleteBlueprint("bpt_yaml456", { output: "yaml" });
+
+    expect(mockDelete).toHaveBeenCalledWith("bpt_yaml456");
+    expect(mockOutput).toHaveBeenCalledWith(
+      { id: "bpt_yaml456", status: "deleted" },
+      { format: "yaml", defaultFormat: "json" },
+    );
+  });
+
+  it("should output just the ID in text format (default)", async () => {
+    mockDelete.mockResolvedValue(undefined);
+
+    const { deleteBlueprint } = await import(
+      "@/commands/blueprint/delete.js"
+    );
+    await deleteBlueprint("bpt_text789", { output: "text" });
+
+    expect(console.log).toHaveBeenCalledWith("bpt_text789");
+    expect(mockOutput).not.toHaveBeenCalled();
+  });
+
+  it("should output just the ID when no output option is provided", async () => {
+    mockDelete.mockResolvedValue(undefined);
+
+    const { deleteBlueprint } = await import(
+      "@/commands/blueprint/delete.js"
+    );
+    await deleteBlueprint("bpt_default", {});
+
+    expect(console.log).toHaveBeenCalledWith("bpt_default");
+    expect(mockOutput).not.toHaveBeenCalled();
+  });
+
+  it("should handle API errors gracefully", async () => {
+    const apiError = new Error("API Error: Forbidden");
+    mockDelete.mockRejectedValue(apiError);
+
+    const { deleteBlueprint } = await import(
+      "@/commands/blueprint/delete.js"
+    );
+    await deleteBlueprint("bpt_error", {});
+
+    expect(mockOutputError).toHaveBeenCalledWith(
+      "Failed to delete blueprint",
+      apiError,
+    );
+  });
+
+  it("should handle dependent snapshot errors gracefully", async () => {
+    const apiError = new Error(
+      "Blueprint has dependent snapshots and cannot be deleted",
+    );
+    mockDelete.mockRejectedValue(apiError);
+
+    const { deleteBlueprint } = await import(
+      "@/commands/blueprint/delete.js"
+    );
+    await deleteBlueprint("bpt_has_snapshots", {});
+
+    expect(mockOutputError).toHaveBeenCalledWith(
+      "Failed to delete blueprint",
+      apiError,
+    );
+  });
+});