feat: add templates command

[felipefreitag]

Add full templates CRUD support with list, get, create, update, delete, publish, and duplicate subcommands. Create requires --html or --html-file (prompts interactively if missing), matching the SDK's requirement.

[cubic-dev-ai]

5 issues found across 10 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/commands/templates/utils.ts">

<violation number="1" location="src/commands/templates/utils.ts:33">
P2: Reject empty numeric fallbacks instead of coercing `count:number:` to `fallbackValue: 0`.</violation>
</file>

<file name="src/commands/templates/delete.ts">

<violation number="1" location="src/commands/templates/delete.ts:9">
P2: Deleting by alias produces incorrect `--json` output because the helper echoes the input string into the `id` field instead of the API response.</violation>
</file>

<file name="src/commands/templates/create.ts">

<violation number="1" location="src/commands/templates/create.ts:81">
P2: Reject `--html` with `--html-file` instead of silently preferring the file body.</violation>
</file>

<file name="src/commands/templates/update.ts">

<violation number="1" location="src/commands/templates/update.ts:25">
P2: The update command is missing `variables` support even though `templates.update` now accepts it, so the help text documents a limitation that is no longer true.</violation>

<violation number="2" location="src/commands/templates/update.ts:79">
P1: Truthy checks here prevent supported empty-string updates from ever reaching `templates.update`.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P2: Deleting by alias produces incorrect --json output because the helper echoes the input string into the id field instead of the API response.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At src/commands/templates/delete.ts, line 9:

<comment>Deleting by alias produces incorrect `--json` output because the helper echoes the input string into the `id` field instead of the API response.</comment>

<file context>
@@ -0,0 +1,48 @@
+export const deleteTemplateCommand = new Command('delete')
+  .alias('rm')
+  .description('Delete a template')
+  .argument('<id>', 'Template ID or alias')
+  .option(
+    '--yes',
</file context>

[cubic-dev-ai]

3 issues found across 4 files (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/commands/templates/utils.ts">

<violation number="1" location="src/commands/templates/utils.ts:31">
P2: This now rejects valid empty-string fallbacks for `string` template variables. `--var NAME:string:` should still map to `fallbackValue: ''`; only empty numeric fallbacks need to be rejected.</violation>
</file>

<file name="src/commands/templates/update.ts">

<violation number="1" location="src/commands/templates/update.ts:71">
P2: Use the same null-check semantics in the `--html`/`--html-file` exclusivity guard; `--html ""` currently bypasses the conflict error.</violation>
</file>

<file name="src/commands/templates/create.ts">

<violation number="1" location="src/commands/templates/create.ts:79">
P2: Check option presence here rather than truthiness; `--html "" --html-file ...` currently bypasses the mutual-exclusion error.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}