@pavp/storywright 1.13.1 → 1.13.3

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "storywright",
-  "version": "1.12.0",
+  "version": "1.13.3",
   "description": "PM skills for Claude Code — turn ambiguous inputs (prompts, screenshots, Figma links) into Jira-ready user stories.",
   "author": "pavp",
   "license": "MIT",

package/commands/story-from-figma.md

@@ -13,7 +13,7 @@ Follow the skill's procedure:
 2. Phase 0.5 — detect companion text / screenshots. Use text as canonical for `User Story / Scope / Business Goal`; Figma as canonical for `Components / States / Flows`. Surface conflicts as BLOCKING clarifications. Never silently pick a winner.
 3. Phase 1 — inventory pages and frames; group by prototype links into flows.
 4. Phase 2 — per flow: identify goal, entry point, states, components. Score per-inference confidence (HIGH/MEDIUM/LOW). Mark MEDIUM/LOW with `> ⚠️ Assumed:` blockquotes.
-5. Phase 3 — for any flow that fails INVEST (Small), hand off to `/story-split` before generating.
-6. Phase 4 — invoke `story-generate` per flow. Emit dual outputs + `flow-summary.md` mapping stories → frames for traceability.
+5. Phase 3 — for any flow that fails INVEST (Small) OR whose deterministic pre-split count ≥2, STOP that flow's draft, route to `/story-split`, and mark it `SPLIT RECOMMENDED` in `flow-summary.md`. Continue with the other flows.
+6. Phase 4 — invoke `story-generate` per drafted flow. Emit the full 3-file trio per story (`story-<N>.standard.md` + `story-<N>.jira-wiki.md` + `story-<N>.dev.md`) + `flow-summary.md` mapping stories → frames for traceability.
 
 Output in the input language (text language wins if mixed inputs).

package/commands/story-split.md

@@ -20,6 +20,6 @@ Follow the skill's procedure:
 4. Draft a split plan as a Markdown table with rationale, pattern, and proposed children.
 5. Run the strategic evaluation: does the split reveal low-value work? Are children equal-sized?
 6. STOP and ask the user to approve / adjust / cancel. Never auto-split.
-7. After approval: write `epic.md` + one stub per child + `split-plan.md` decision trail.
+7. After approval: write `epic.md` (single file — epic metadata, holds the decision trail; NO `split-plan.md`) + the full 3-file trio per child (`story-<N>.standard.md` + `story-<N>.jira-wiki.md` + `story-<N>.dev.md`).
 8. Coherence check + recursive re-split for children still >1 sprint.
-9. Save artifacts under `./stories/<epic-slug>/`.
+9. Save artifacts under the canonical output folder `docs/storywright/YYYY-MM-DD-HHmm-<epic-slug>/`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pavp/storywright",
-  "version": "1.13.1",
+  "version": "1.13.3",
   "description": "PM Skills pack for Claude Code — turn ambiguous inputs (prompts, screenshots, Figma links) into Jira-ready user stories.",
   "keywords": [
     "claude",
@@ -53,6 +53,7 @@
     "@commitlint/cli": "^19.5.0",
     "@commitlint/config-conventional": "^19.5.0",
     "@semantic-release/changelog": "^6.0.3",
+    "@semantic-release/exec": "^7.1.0",
     "@semantic-release/git": "^10.0.1",
     "@semantic-release/github": "^11.0.0",
     "@semantic-release/npm": "^12.0.1",

package/scripts/install-skills.mjs

@@ -2,7 +2,7 @@
 import { homedir } from "node:os";
 import { cp, mkdir, readdir, readFile, rm, unlink, writeFile } from "node:fs/promises";
 import { join } from "node:path";
-import { execSync } from "node:child_process";
+import { execSync, execFileSync } from "node:child_process";
 import { REPO_ROOT, SKILLS_DIR, pathExists } from "./lib/skills.mjs";
 
 const skillsTarget = join(homedir(), ".claude", "skills", "storywright");
@@ -51,7 +51,9 @@ async function ensureGlobalGitignore() {
     globalIgnorePath = execSync("git config --global core.excludesFile", { encoding: "utf8" }).trim();
   } catch {
     globalIgnorePath = join(homedir(), ".gitignore_global");
-    execSync(`git config --global core.excludesFile "${globalIgnorePath}"`);
+    // Array form (no shell) so a homedir with spaces or shell metacharacters
+    // can't break or inject into the command.
+    execFileSync("git", ["config", "--global", "core.excludesFile", globalIgnorePath]);
   }
   if (globalIgnorePath === "~") {
     globalIgnorePath = homedir();

package/scripts/sync-plugin-version.mjs

@@ -0,0 +1,51 @@
+#!/usr/bin/env node
+// Write a version into the Claude marketplace manifest (.claude-plugin/plugin.json).
+//
+// semantic-release owns the version number in package.json (@semantic-release/npm
+// bumps it in-place). plugin.json is a separate file no release plugin touches, so
+// without this step the manifest drifts behind every release. Wired as the
+// @semantic-release/exec prepareCmd, it runs after the version is computed and
+// before @semantic-release/git commits the bump, keeping both files in lockstep.
+//
+// Usage: node scripts/sync-plugin-version.mjs <version>
+//   <version> defaults to package.json's version when omitted, so the script is
+//   also usable manually to repair drift outside a release.
+
+import { readFile, writeFile } from "node:fs/promises";
+import { fileURLToPath } from "node:url";
+import { dirname, join } from "node:path";
+
+const root = join(dirname(fileURLToPath(import.meta.url)), "..");
+const manifestPath = join(root, ".claude-plugin", "plugin.json");
+
+async function readJson(path) {
+  return JSON.parse(await readFile(path, "utf8"));
+}
+
+const version = process.argv[2] ?? (await readJson(join(root, "package.json"))).version;
+
+if (!/^\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?(?:\+[0-9A-Za-z.-]+)?$/.test(version)) {
+  console.error(`✗ Refusing to write an invalid semver into plugin.json: "${version}"`);
+  process.exit(1);
+}
+
+const raw = await readFile(manifestPath, "utf8");
+
+const versionLine = /("version"\s*:\s*")([^"]*)(")/;
+const match = raw.match(versionLine);
+if (!match) {
+  console.error(`✗ Could not find a "version" field in ${manifestPath}`);
+  process.exit(1);
+}
+
+const previous = match[2];
+if (previous === version) {
+  console.log(`✓ plugin.json already at ${version} — no change`);
+  process.exit(0);
+}
+
+// Surgical replace of only the version value — preserves the file's existing
+// formatting (indentation, array style) so releases don't churn unrelated lines.
+const updated = raw.replace(versionLine, `$1${version}$3`);
+await writeFile(manifestPath, updated);
+console.log(`✓ plugin.json version ${previous} → ${version}`);

package/skills/story-generate/SKILL.md

@@ -56,7 +56,7 @@ If the base **deterministic pre-split test** returns count ≥2:
 - Show candidate children + dep notes + V audit.
 - Ask via `AskUserQuestion`: "This story has [N] independent flows. Run split now?" with options:
   - "Yes, split" → execute `[[story-split]]` inline (epic + children + .storywright-context.json).
-  - "Continue without split" → proceed with single-story path (fill canonical block → INVEST → render dual outputs).
+  - "Continue without split" → proceed with single-story path (fill canonical block → INVEST → render the 3-file trio).
   - "No, keep as-is" → stop. No output written.
 - Do NOT silently auto-split without the AskUserQuestion confirmation.
 

package/skills/story-refine/SKILL.md

@@ -48,7 +48,7 @@ If the base **deterministic pre-split test** returns count ≥2:
 - Show candidate children + per-pair dep notes (base rule 10) + V audit (base rule 11).
 - Ask via `AskUserQuestion`: "This story has [N] independent flows. Run split now?" with options:
   - "Yes, split" → execute `[[story-split]]` inline (epic + children + .storywright-context.json).
-  - "Continue without split" → proceed with single-story path (fill canonical block → INVEST → render dual outputs).
+  - "Continue without split" → proceed with single-story path (fill canonical block → INVEST → render the 3-file trio).
   - "No, keep as-is" → stop. No output written.
 - Do NOT silently auto-split without the AskUserQuestion confirmation.