worclaude 2.10.2 → 2.10.4

package/CHANGELOG.md

@@ -4,6 +4,42 @@ All notable changes to worclaude are documented in this file. Format loosely fol
 
 ## [Unreleased]
 
+## [2.10.4] — 2026-05-14
+
+Patch release fixing the silent failure mode of the daily `upstream-check` workflow. The Claude session writes precheck output to `RUNNER_TEMP` (an absolute path outside the workspace), but `.claude/settings.json` only granted `Read(*.json)` for workspace-relative paths. Claude was burning turns hitting permission walls — 6 denials per run in failed-run logs — and either timed out at `max_turns` (runs #31, #32) or emitted a parse-error envelope that auto-opened a tracking issue (#184). This release adds an explicit `Read(/home/runner/work/_temp/**)` grant, bumps `anthropics/claude-code-action` 11 patch versions to current, refreshes the README banner with new background art, and rolls up two weeks of routine Dependabot bumps.
+
+### Fixed
+
+- **upstream-check: grant Claude Read on RUNNER_TEMP + bump claude-code-action to v1.0.122** (PR #187) — adds `Read(/home/runner/work/_temp/**)` to `.claude/settings.json` so the Claude action can read `new-items.json` / `feed-report.json` / `next-state.json` from `scripts/upstream-precheck.mjs:192-194`. Bumps the SHA pin from v1.0.111 → v1.0.122 to pick up upstream's own bug fixes. Together with the `--max-turns 40` cap from #181, this eliminates the failure mode that produced runs #31, #32, and parse-error issue #184. Issue #184 closed with a back-link to the fix.
+
+### Changed
+
+- **Banner refresh + archive previous variant** (PR #188) — second refresh of the README banner; new right-side background art (sun-burst → blurred floral focal point) better matches the existing logo motif. Logo, wordmark, tagline, and 2048×682 dimensions unchanged → no README layout impact. Lossless-compressed with `oxipng -o 6 --strip safe` (974 KB raw → 818 KB final, also 34 KB smaller than the previous banner). Previous banner archived to `assets/old/banner2.png` per the convention established in #182.
+- **gitignore `.mcp.json` and `docs/handoffs/`, ship `.mcp.example.json`** (PR #183) — MCP server config may carry tokens; tracking is now opt-in via the example file. Forward-looking session handoffs in `docs/handoffs/` are kept private to the working copy.
+- **Banner refresh and add logo variants** (PR #182) — earlier asset refresh that established `assets/old/` as a tracked archive directory and added logo variants alongside the banner.
+- **upstream-check: bump claude max-turns 25 → 40** (PR #181) — preemptive headroom; this was the partial mitigation that allowed run #33 to succeed before #187 fixed the actual root cause.
+- ⚠ **No `Version bump:` declaration — under-documented:** dependabot PRs **#186** (`bump the minor-and-patch group with 2 updates`), **#185** (`bump the actions-minor-and-patch group with 3 updates` — included the v1.0.111 → v1.0.119 claude-code-action bump that #187 subsumed via the v1.0.122 resolution), **#180** (`bump the minor-and-patch group with 2 updates`), and **#179** (`bump anthropics/claude-code-action from 1.0.109 to 1.0.111`) were merged without `Version bump:` lines and treated as `none`.
+
+Release group: 9 PRs (1 patch, 4 none, 4 missing declarations). v2.10.3 → v2.10.4.
+
+## [2.10.3] — 2026-04-30
+
+Fixes a user-reported breakage where every arrow-selectable prompt across `worclaude init` / `upgrade` / `restore` / `delete` accepted typed text instead of arrow-key navigation. Root cause: PR #169 (v2.10.0) bumped `inquirer` 12 → 13 with no code changes, but inquirer 13 renamed its legacy `list` prompt type to `select` and ships a runner that silently falls back to `type: 'input'` (free-text) for unknown types — so every `type: 'list'` site degraded silently. The "Everything look right?" prompt looped because typed `y`/`yes` did not strictly equal the choice value `'yes'`. Renames all 20 sites, migrates 4 index-based defaults to value-based (inquirer 13's `select` matches by value, not index), aligns the CLAUDE.md tech-stack metadata that had drifted behind `package.json` for three deps (Inquirer / Ora / Commander), and closes the test-suite blind spot — the prior `vi.mock('inquirer')` setup never inspected `spec.type`.
+
+### Fixed
+
+- **Arrow-selectable prompts work again across init / upgrade / restore / delete** (PR #177) — renamed all 20 `type: 'list'` sites to `type: 'select'` across `src/commands/{init,upgrade,restore,delete}.js` and `src/prompts/{claude-md-merge,conflict-resolution,project-type,tech-stack}.js`. Migrated 4 index-based `default:` values to value-based: `init.js:93,111`, `restore.js:51`, `delete.js:113,156`. Without this, "Generate plugin.json?", "Scaffold memory files?", "Install GitHub Action?", "Everything look right?", "Which step do you want to adjust?", restore and delete confirmations, and the standalone hook-conflict / project-type / claude-md-merge prompts all rendered as silent free-text inputs.
+
+### Tests
+
+- **Structural regression coverage for prompt types** (PR #177) — new `tests/utils/prompt-types.js` exposes `VALID_INQUIRER_TYPES` (the 10 v13 built-in prompt names) plus `expectAllValidPromptTypes(inquirer, label)` which inspects every captured `inquirer.prompt.mock.calls` spec and fails when any `type` is not a v13 built-in. New `tests/prompts/prompt-types.test.js` exercises the four standalone prompt modules + agent-selection. Single-line assertions added to `tests/commands/{init,upgrade,restore,delete}.test.js` so the ~30 sites in command files are also covered. Three stale `expect(spec.type).toBe('list')` assertions in `init.test.js` updated to `'select'`. Test counts: 992 → 999.
+
+### Docs
+
+- **CLAUDE.md tech-stack metadata aligned with package.json** (PR #177) — Inquirer ^12.5.0 → ^13.4.2, Ora ^8.2.0 → ^9.4.0, Commander ^13.1.0 → ^14.0.3 (all three were drifted; CLAUDE.md's own "watch for breaking changes between majors" warning had not been acted on at the time of PR #169's bump). Two new Gotchas added: (1) the silent `type` → `input` fallback in inquirer 13's legacy runner (`node_modules/inquirer/dist/ui/prompt.js`), and (2) inquirer 13 `select` defaults take the choice value, not its index — `default: 1` (second choice) becomes `default: <that-choice's-value>`.
+
+Release group: 1 PR (patch). v2.10.2 → v2.10.3. No missing declarations.
+
 ## [2.10.2] — 2026-04-30
 
 Bundles a dogfood catch-up of this repo's own `.claude/` (skipping from v2.8.0 to v2.10.1 in one upgrade run) with a real scaffold bug fix surfaced during that upgrade — `.claude/workflow-ref/` (transient upgrade-time conflict references) was missing from the scaffolder's `updateGitignore` entries, so every user running `worclaude upgrade` had those files pollute their git status. Also closes the BACKLOG "claude --worktree command visibility" item via a docs-only fix: investigation confirmed Claude Code does not create a "minimal `.claude/`" (the worktree's `.claude/` is a normal git checkout from `origin/HEAD`); the perceived missing-commands symptom was caused by main lagging develop, and the existing `subagent-usage` skill already documented the mechanism. The gap was direct `claude --worktree` users with no agent-mediated freshness preamble — filled with a one-paragraph copy-paste reset block.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "worclaude",
-  "version": "2.10.2",
+  "version": "2.10.4",
   "description": "The Workflow Layer for Claude Code — scaffold agents, commands, skills, hooks, and memory into any project",
   "type": "module",
   "bin": {

package/src/commands/delete.js

@@ -32,7 +32,7 @@ export async function deleteCommand() {
   // Step 1: Mode selection
   const { mode } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'mode',
       message: 'What would you like to do?',
       choices: [
@@ -82,7 +82,7 @@ export async function deleteCommand() {
     display.newline();
     const { modifiedAction } = await inquirer.prompt([
       {
-        type: 'list',
+        type: 'select',
         name: 'modifiedAction',
         message: `${classification.modified.length} file(s) have been customized. What should we do?`,
         choices: [
@@ -110,7 +110,7 @@ export async function deleteCommand() {
 
     const { rootAction } = await inquirer.prompt([
       {
-        type: 'list',
+        type: 'select',
         name: 'rootAction',
         message: 'What would you like to do with these files?',
         choices: [
@@ -118,7 +118,7 @@ export async function deleteCommand() {
           { name: 'Let me choose which to remove', value: 'choose' },
           { name: 'Remove all', value: 'remove' },
         ],
-        default: 0,
+        default: 'keep',
       },
     ]);
 
@@ -153,14 +153,14 @@ export async function deleteCommand() {
 
   const { confirm } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'confirm',
       message: 'Confirm deletion?',
       choices: [
         { name: 'Yes, delete', value: true },
         { name: 'No, cancel', value: false },
       ],
-      default: 1,
+      default: false,
     },
   ]);
 

package/src/commands/init.js

@@ -83,14 +83,14 @@ async function runAgents(selections) {
 async function runOptionalExtras(selections) {
   const previouslySelected = new Set(selections.optionalFeatures || []);
   const questions = OPTIONAL_FEATURES.map((feature) => ({
-    type: 'list',
+    type: 'select',
     name: feature.id,
     message: feature.label,
     choices: [
       { name: 'Yes', value: true },
       { name: 'No', value: false },
     ],
-    default: previouslySelected.has(feature.id) ? 0 : 1,
+    default: previouslySelected.has(feature.id),
   }));
   const answers = await inquirer.prompt(questions);
   const optionalFeatures = OPTIONAL_FEATURES.filter((f) => answers[f.id]).map((f) => f.id);
@@ -100,7 +100,7 @@ async function runOptionalExtras(selections) {
   // at it. Phase 7 T7.3.
   const { installGithubAction } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'installGithubAction',
       message:
         'Install Claude Code\'s GitHub Action for the @claude "compounding engineering" workflow?',
@@ -108,7 +108,7 @@ async function runOptionalExtras(selections) {
         { name: 'Yes — show me the install instructions now', value: true },
         { name: "No — I'll do it later", value: false },
       ],
-      default: 1,
+      default: false,
     },
   ]);
 
@@ -179,7 +179,7 @@ async function showConfirmation(selections) {
 
   const { confirmation } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'confirmation',
       message: 'Everything look right?',
       choices: [
@@ -241,7 +241,7 @@ async function runInteractivePrompts(projectRoot) {
     } else if (confirmation === 'adjust') {
       const { step } = await inquirer.prompt([
         {
-          type: 'list',
+          type: 'select',
           name: 'step',
           message: 'Which step do you want to adjust?',
           choices: CONFIRMATION_STEPS,
@@ -772,7 +772,7 @@ export async function initCommand() {
 
     const { proceed } = await inquirer.prompt([
       {
-        type: 'list',
+        type: 'select',
         name: 'proceed',
         message: 'Proceed with workflow installation?',
         choices: [

package/src/commands/restore.js

@@ -27,7 +27,7 @@ export async function restoreCommand() {
 
   const { selected } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'selected',
       message: 'Select backup to restore:',
       choices,
@@ -48,14 +48,14 @@ export async function restoreCommand() {
 
   const { confirm } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'confirm',
       message: 'Confirm restore?',
       choices: [
         { name: 'Yes', value: true },
         { name: 'No', value: false },
       ],
-      default: 1,
+      default: false,
     },
   ]);
 

package/src/commands/upgrade.js

@@ -233,7 +233,7 @@ async function applyRepairPass(projectRoot, plan, variables) {
 async function promptProceed(message) {
   const { proceed } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'proceed',
       message,
       choices: [
@@ -357,7 +357,7 @@ async function promptAndScaffoldOptionalFeatures(projectRoot, meta, { yes, dryRu
   for (const feature of available) {
     const { accept } = await inquirer.prompt([
       {
-        type: 'list',
+        type: 'select',
         name: 'accept',
         message: feature.label,
         choices: [
@@ -397,7 +397,7 @@ export async function upgradeCommand(options = {}) {
     );
     const { doUpdate } = await inquirer.prompt([
       {
-        type: 'list',
+        type: 'select',
         name: 'doUpdate',
         message: 'Update worclaude CLI?',
         choices: [
@@ -533,7 +533,7 @@ export async function upgradeCommand(options = {}) {
       agentReport = await patchAgentDescriptions(projectRoot, meta, async (agentName) => {
         const { patch } = await inquirer.prompt([
           {
-            type: 'list',
+            type: 'select',
             name: 'patch',
             message: `Agent "${agentName}" has been customized. Add missing description field?`,
             choices: [

package/src/prompts/claude-md-merge.js

@@ -107,7 +107,7 @@ export async function promptClaudeMdMerge(existingContent, missingSections) {
 
   const { choice } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'choice',
       message: 'How would you like to handle CLAUDE.md?',
       choices: [
@@ -139,7 +139,7 @@ export async function interactiveSectionMerge(existingContent, renderedTemplate,
 
     const { addSection } = await inquirer.prompt([
       {
-        type: 'list',
+        type: 'select',
         name: 'addSection',
         message: `Add "${sectionName}" to your CLAUDE.md?`,
         choices: [

package/src/prompts/conflict-resolution.js

@@ -16,7 +16,7 @@ export async function promptHookConflict(hookCategory, existingHook, workflowHoo
 
   const { resolution } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'resolution',
       message: 'How would you like to resolve this?',
       choices: [

package/src/prompts/project-type.js

@@ -37,7 +37,7 @@ export async function promptProjectType() {
       display.dim('You may not need both.');
       const { continueAnyway } = await inquirer.prompt([
         {
-          type: 'list',
+          type: 'select',
           name: 'continueAnyway',
           message: 'Continue anyway?',
           choices: [

package/src/prompts/tech-stack.js

@@ -20,7 +20,7 @@ export async function promptTechStack(_projectTypes) {
 
   const { useDocker } = await inquirer.prompt([
     {
-      type: 'list',
+      type: 'select',
       name: 'useDocker',
       message: 'Do you use Docker currently?',
       choices: [