showreel v1.1.6 — shot.mjs entry guard + CLI parse test coverage

shot.mjs called main() unconditionally — alone among the scripts it had no
import.meta.url entry guard, so importing it (to reuse parse, or from a test)
ran the CLI against the importer's argv (printed usage, called process.exit).
Wrapped main() in the standard guard; the CLI entry is unchanged.

Exported parse from shot/compose and added argument-parsing tests for shot,
compose and tape — the CLI parsers that had no dedicated coverage. The shot
bug surfaced precisely because the new test imported it. 547 tests.

Author: HeyRenan

.claude-plugin/marketplace.json

@@ -5,14 +5,14 @@
   },
   "metadata": {
     "description": "Personal marketplace hosting the Showreel plugin",
-    "version": "1.1.5"
+    "version": "1.1.6"
   },
   "plugins": [
     {
       "name": "showreel",
       "source": "./showreel",
       "description": "Your UI, on its showreel \u2014 annotated screenshots, feature demos, cinematic flow recordings (gif/mp4), terminal captures and before/after composites in one command each. Self-contained Chromium motor, deterministic placement, self-validated output; no MCP, cheap on tokens.",
-      "version": "1.1.5"
+      "version": "1.1.6"
     }
   ]
 }

CHANGELOG.md

@@ -9,6 +9,19 @@ Showreel turns "show me" into a finished visual — annotated screenshots, isola
 demos, flow GIFs, terminal recordings and before/after composites — driven entirely
 by CSS selectors and JSON steps.
 
+## [1.1.6] — 2026-06-24
+
+### Fixed
+- **`shot.mjs` ran `main()` on import.** Unlike every other script it had no
+  entry-point guard, so importing it (to reuse `parse`, or from a test) executed
+  the CLI against the importer's argv — printing usage and calling `process.exit`.
+  Wrapped `main()` in the standard `import.meta.url` guard.
+
+### Tests
+- Exported `parse` from `shot`/`compose` and added argument-parsing tests for
+  `shot`, `compose` and `tape` — the CLI parsers that had no dedicated coverage.
+  547 tests.
+
 ## [1.1.5] — 2026-06-24
 
 ### Fixed
@@ -161,6 +174,7 @@ First stable release — the full capture motor.
   offline two implementations of the same time contract. 530 tests, no network or
   browser needed.
 
+[1.1.6]: https://github.com/HeyRenan/showreel/compare/v1.1.5...v1.1.6
 [1.1.5]: https://github.com/HeyRenan/showreel/compare/v1.1.4...v1.1.5
 [1.1.4]: https://github.com/HeyRenan/showreel/compare/v1.1.3...v1.1.4
 [1.1.3]: https://github.com/HeyRenan/showreel/compare/v1.1.2...v1.1.3

showreel/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "showreel",
-  "version": "1.1.5",
+  "version": "1.1.6",
   "description": "Your UI, on its showreel \u2014 annotated screenshots, feature demos, cinematic flow recordings (gif/mp4), terminal captures and before/after composites in one command each. Self-contained Chromium motor, deterministic placement, self-validated output; no MCP, cheap on tokens.",
   "author": {
     "name": "Renan"

showreel/scripts/__tests__/cli-parse.test.mjs

@@ -0,0 +1,53 @@
+// cli-parse.test.mjs — argument-parsing guards for the CLI tools that lacked a
+// dedicated parse test (shot, compose, tape). Locks in the behaviour intensive
+// testing exercised by hand: positional mapping, numeric/string flag validation,
+// unknown-flag rejection, and the surplus-positional guard.
+
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { parse as shotParse } from '../shot.mjs';
+import { parse as composeParse } from '../compose.mjs';
+import { parseArgs as tapeParse } from '../tape.mjs';
+
+test('shot: positionals map to url/selector/out; flags parse', () => {
+  const a = shotParse(['http://x/', '.card', 'out.png', '--width', '1440', '--pad', '8']);
+  assert.equal(a.url, 'http://x/');
+  assert.equal(a.selector, '.card');
+  assert.equal(a.out, 'out.png');
+  assert.equal(a.width, 1440);
+  assert.equal(a.pad, 8);
+});
+
+test('shot: bad numeric flag, unknown flag, and surplus positionals all throw', () => {
+  assert.throws(() => shotParse(['u', 's', 'o', '--width', 'abc']), /--width must be a number/);
+  assert.throws(() => shotParse(['u', 's', 'o', '--nope']), /shot: unknown arg --nope/);
+  assert.throws(() => shotParse(['u', 's', 'o', 'extra']), /too many positional/);
+});
+
+test('compose: positionals map to aPng/bPng/out; flags parse', () => {
+  const a = composeParse(['a.png', 'b.png', 'out.png', '--labels', 'Before,After', '--gap', '12']);
+  assert.equal(a.aPng, 'a.png');
+  assert.equal(a.bPng, 'b.png');
+  assert.equal(a.out, 'out.png');
+  assert.equal(a.labels, 'Before,After');
+  assert.equal(a.gap, 12);
+});
+
+test('compose: unknown flag, surplus positionals, and a flag-as-value all throw', () => {
+  assert.throws(() => composeParse(['a', 'b', 'o', '--frob']), /compose: unknown arg --frob/);
+  assert.throws(() => composeParse(['a', 'b', 'o', 'x']), /too many positional/);
+  assert.throws(() => composeParse(['a', 'b', 'o', '--labels', '--gap']), /--labels needs a value/);
+});
+
+test('tape: positionals collected; numeric + string flags parse', () => {
+  const a = tapeParse(['out.gif', '--width', '900', '--steps-json', '[]', '--theme', 'dark']);
+  assert.deepEqual(a.positional, ['out.gif']);
+  assert.equal(a.width, 900);
+  assert.equal(a.stepsJson, '[]');
+  assert.equal(a.theme, 'dark');
+});
+
+test('tape: unknown flag and a non-integer width throw', () => {
+  assert.throws(() => tapeParse(['o.gif', '--bogus']), /tape: unknown arg --bogus/);
+  assert.throws(() => tapeParse(['o.gif', '--width', '12.5']), /--width must be a whole number/);
+});

showreel/scripts/compose.mjs

@@ -109,7 +109,7 @@ async function composeGifs(a, labels) {
   console.log('OK ' + a.out);
 }
 
-function parse(argv) {
+export function parse(argv) {
   const a = { gap: 24, pad: 28, labelH: 44 };
   const pos = [];
   for (let i = 0; i < argv.length; i++) {

showreel/scripts/shot.mjs

@@ -7,7 +7,7 @@ import { writeFileSync } from 'node:fs';
 import { Browser } from '../lib/browser.mjs';
 import { num } from './cli-args.mjs';
 
-function parse(argv) {
+export function parse(argv) {
   const a = { width: 900, height: 1400, dpr: 1, pad: 16 };
   const pos = [];
   for (let i = 0; i < argv.length; i++) {
@@ -56,4 +56,6 @@ async function main() {
   }
 }
 
-main().catch((e) => { console.error(String(e.message || e)); process.exit(1); });
+if (import.meta.url === `file://${process.argv[1]}`) {
+  main().catch((e) => { console.error(String(e.message || e)); process.exit(1); });
+}