worclaude 2.9.2 → 2.10.0

package/CHANGELOG.md

@@ -4,6 +4,44 @@ All notable changes to worclaude are documented in this file. Format loosely fol
 
 ## [Unreleased]
 
+## [2.10.0] — 2026-04-29
+
+Drops support for Node 18, which reached LTS end-of-life on 2025-04-30 (12 months before this release). The drop unblocks two Dependabot PRs stuck on Node-20-only features (`inquirer 13`'s `util.styleText` and `ora 9`'s regex `v` flag) and ships those bumps in the same release. Also recovers from a Dependabot routing misconfiguration: `.github/dependabot.yml` now declares `target-branch: develop` for both ecosystems, fixing a config gap that caused 5 PRs in the v2.9.3 → v2.10.0 window to be opened against main instead of develop. Their content is preserved across both branches via a recovery sync.
+
+### Breaking
+
+- **Node 18 no longer supported** (PR #167) — `engines.node` is now `>=20.0.0`. Running `npm install -g worclaude` on Node 18 will print an `EBADENGINE` warning (npm doesn't block by default but the warning is visible). CI test matrix dropped from `[18, 20, 22]` to `[20, 22]`. Required-status-checks on the `develop-protection` and `main-protection` rulesets updated accordingly. Tech-stack mentions refreshed in CLAUDE.md, AGENTS.md, README.md, `docs/guide/getting-started.md`, and `templates/specs/spec-md-library.md`.
+
+### Changed
+
+- **`ora` 8.2.0 → 9.4.0** (PR #169) — major bump. ora 9 uses regex `v` flag (Node 20+); previously blocked by the v2.9.x Node 18 matrix.
+- **`inquirer` 12.11.1 → 13.4.2** (PR #169) — major bump. inquirer 13 uses `util.styleText` (Node 20.12+); previously blocked by the v2.9.x Node 18 matrix.
+- **Dependabot routing fixed** (PR #168) — added `target-branch: develop` to both `npm` and `github-actions` ecosystems in `.github/dependabot.yml`. Previously, Dependabot defaulted to the repo's default branch (main), causing PRs to misroute. Future Dependabot Monday runs will correctly target develop.
+
+### Internal
+
+- **Recovery sync develop ← main** (PR #168) — brings 5 misrouted Dependabot squash commits from main onto develop (prettier 3.8.3, claude-code-action 1.0.109, actions/cache 5, vitest 4, eslint 10). All updates were legitimate; merge made via `git merge origin/main --no-ff` with auto-resolution.
+
+Release group: 3 PRs (1 minor, 1 patch, 1 none). No missing Version bump declarations.
+
+## [2.9.3] — 2026-04-29
+
+Security tooling refresh shipped as a paired group: a CI-tooling migration from Snyk (whose free-tier scan limit had blocked the v2.9.2 release PR) to a GitHub-native open-source SCA stack (Dependabot + OSV-Scanner), and the cleanup of the inaugural CodeQL scan after enabling the default setup. CodeQL surfaced 5 findings — 2× High "Incomplete multi-character sanitization" on the project-scanner README detector's HTML-stripping helpers, and 3× Medium "Workflow does not contain permissions" on `ci.yml`'s three jobs — all closed in this release. The sanitization fix extracts a `stripUntilStable(text, regex)` helper for the do-while-until-stable pattern; the permissions fix adds a top-level `permissions: contents: read` block matching the rest of the repo's workflows. SECURITY.md's AI-detected typosquat section also refined with the actual chain context: the `claude` npm package is `bcherny/redirect-claude`, an intentional Boris-Cherny-maintained typosquat-warning redirect, not an abandoned package as previously documented.
+
+### Fixed
+
+- **CodeQL findings — incomplete multi-character sanitization** (PR #158) — `stripHtmlComments` and `stripHtmlTags` in `src/core/project-scanner/detectors/readme.js` rewritten to delegate to a private `stripUntilStable(text, regex)` helper that applies the regex repeatedly until stable. Closes the two High-severity `js/incomplete-multi-character-sanitization` alerts. Defense-in-depth: verified during plan-mode that no input distinguishes single-pass from looped output for these specific regex patterns; the fix satisfies the static analyzer without behavioral change.
+- **CodeQL findings — missing workflow permissions** (PR #158) — top-level `permissions: contents: read` added to `.github/workflows/ci.yml`. Closes the three Medium-severity `Workflow does not contain permissions` alerts (`test` matrix, `format-check`, `plugin-validate` jobs). Brings ci.yml in line with the rest of the repo's workflows, all of which already declared explicit permissions.
+
+### Changed
+
+- **CI scanner stack: Snyk → Dependabot + OSV-Scanner** (PR #157) — Snyk's free-tier monthly scan limit had blocked the v2.9.2 release PR. Replaced with two free, GitHub-native SCA tools: `.github/dependabot.yml` (npm + github-actions ecosystems, weekly Monday 03:00 UTC, minor/patch grouped, `open-pull-requests-limit: 5`) and `.github/workflows/osv-scanner.yml` invoking `google/osv-scanner-action@v2.3.5` as job-level reusable workflows. SARIF upload routes findings to the Security tab. `.snyk` deleted; `SECURITY.md` and `CONTRIBUTING.md` updated to vendor-neutral language.
+- **SECURITY.md typosquat-alert section** (PR #158) — refined to document Socket's chain inference (`worclaude` → `claude` → `@anthropic-ai/claude-code`) with [`bcherny/redirect-claude`](https://github.com/bcherny/redirect-claude) context, replacing the inaccurate "abandoned package" phrasing.
+
+### Docs
+
+- **CONTRIBUTING.md** (PR #157) — replaced "Snyk security score" with vendor-neutral "supply-chain trust signal that SCA tools (OSV-Scanner, Socket, Dependabot) and consumers rely on".
+
 ## [2.9.2] — 2026-04-28
 
 `upstream-check` workflow rebuild: fixes a 5-day silence and migrates to the official client library. Root cause of the silence: the daily workflow committed `.github/upstream-state.json` and pushed to `main`, but `main`'s branch protection (PR-required + 4 required status checks) rejected every push with `GH013`. State never advanced, items were re-evaluated daily, and the `Create issue` step was gated behind state-push success — silent forever. State persistence is now `actions/cache@v4` (key prefix `upstream-state-v3-`); the workflow no longer touches the git tree, `contents: write` permission dropped. Migration to [`@sefaertunc/anthropic-watch-client`](https://www.npmjs.com/package/@sefaertunc/anthropic-watch-client) replaces ~80 lines of hand-rolled fetch/dedup with composite-`uniqueKey` dedup (the `id`-only dedup at `scripts/upstream-precheck.mjs:95` was already silently dropping items where two sources shared an ID — `2.1.114` was the live example), version-gated fetch (`FeedVersionMismatchError`), and typed errors. Claude prompt + `upstream-watcher` agent + `docs/reference/upstream-automation.md` updated for the `community` source category (Reddit, HN, Twitter, GitHub commits — informational only per upstream's contract). Source counts no longer hardcoded — derived from `summary.sourcesChecked`.

package/CONTRIBUTING.md

@@ -78,7 +78,8 @@ npm provenance (SLSA attestations).
 4. Verify the "Provenance" badge on `https://www.npmjs.com/package/worclaude`.
 
 Do not run `npm publish` from a local machine — manual publishes omit provenance
-and weaken the Snyk security score.
+and weaken the supply-chain trust signal that SCA tools (OSV-Scanner, Socket,
+Dependabot) and consumers rely on.
 
 ## Reporting Bugs
 

package/README.md

@@ -7,7 +7,7 @@
   <a href="https://www.npmjs.com/package/worclaude"><img src="https://img.shields.io/npm/dm/worclaude" alt="downloads" /></a>
   <a href="https://github.com/sefaertunc/Worclaude/actions/workflows/ci.yml"><img src="https://img.shields.io/github/actions/workflow/status/sefaertunc/Worclaude/ci.yml?label=tests" alt="tests" /></a>
   <a href="LICENSE"><img src="https://img.shields.io/github/license/sefaertunc/Worclaude" alt="license" /></a>
-  <img src="https://img.shields.io/badge/node-%3E%3D18-brightgreen" alt="node >= 18" />
+  <img src="https://img.shields.io/badge/node-%3E%3D20-brightgreen" alt="node >= 20" />
   <img src="https://img.shields.io/badge/built%20for-Claude%20Code-cc785c" alt="Built for Claude Code" />
 </p>
 

package/SECURITY.md

@@ -22,8 +22,9 @@ If the vulnerability is accepted, a fix will be prioritized and released as a pa
 
 ## Supply Chain Scanner Findings
 
-Automated SCA tools (Socket, Snyk, GitHub Dependabot) sometimes surface
-alerts that are not real exposures for worclaude. The most common cases:
+Automated SCA tools (Socket, OSV-Scanner, GitHub Dependabot) sometimes
+surface alerts that are not real exposures for worclaude. The most common
+cases:
 
 ### Test fixture manifests are not real dependencies
 
@@ -42,9 +43,10 @@ These fixtures are:
   dependency lists; it never imports or runs the packages named inside.
 
 Worclaude's repo includes `socket.yml` to stop Socket from scanning this
-directory, and a `.snyk` policy file with an equivalent `exclude.global`
-entry for Snyk Open Source. Other SCA tools may need an equivalent
-`ignore` directive.
+directory. The OSV-Scanner workflow scopes itself to the root
+`package-lock.json` (via `--lockfile=`), which inherently skips fixture
+lockfiles without a separate config. Other SCA tools may need an
+equivalent `ignore` directive.
 
 ### Real runtime dependencies
 
@@ -111,11 +113,26 @@ triggers because the package name `worclaude` contains the substring
 `claude`. The package was published under this name from day one
 (2026-02), the npm namespace is owned by the original author
 (`sefaertunc`), and the package is the canonical home for the workflow
-described in this README. There is no upstream `claude` workflow
-scaffolder being typosquatted — `claude` on npm is an unrelated
-abandoned package. Renaming a published, indexed package would break
-every existing user's CLI alias and slash-command muscle memory; the
-alert is accepted as a permanent false positive.
+described in this README.
+
+Socket's chain inference (`worclaude` → `claude` → `@anthropic-ai/claude-code`)
+produces a false positive at every link:
+
+- **`worclaude`** is a workflow scaffolder for Claude Code, not a redirect
+  or rename of any other package.
+- **`claude`** on npm is itself an intentional typosquat-warning redirect
+  by Boris Cherny ([github.com/bcherny/redirect-claude](https://github.com/bcherny/redirect-claude)),
+  deprecated by design — its README points users to `@anthropic-ai/claude-code`.
+  Socket flags `claude` as a typosquat of `@anthropic-ai/claude-code`,
+  which is exactly the case the redirect was authored to handle.
+- **`@anthropic-ai/claude-code`** is Anthropic's canonical Claude Code
+  product — the destination both `worclaude` (as Boris-Cherny-tips-inspired
+  workflow tooling per the README's Acknowledgments) and `claude` (as a
+  redirect) point readers toward.
+
+Renaming a published, indexed package would break every existing user's
+CLI alias and slash-command muscle memory. The alert is accepted as a
+permanent false positive.
 
 ### URL-strings supply-chain alert (template content)
 
@@ -128,3 +145,19 @@ at runtime; the only HTTP code path is `src/utils/npm.js`, which
 queries the npm registry for the latest published version during
 `worclaude upgrade` and `worclaude status`. The flagged strings are
 content, not endpoints.
+
+### CI scanner stack
+
+Worclaude's CI runs two free, open-source SCA tools:
+
+- **OSV-Scanner** (`.github/workflows/osv-scanner.yml`) — scans
+  `package-lock.json` against the [OSV.dev](https://osv.dev) database
+  on every PR (fails on vulnerability) and sweeps `main` weekly
+  (warn-only). Findings upload as SARIF to the repo's Security tab.
+- **Dependabot** (`.github/dependabot.yml` + Settings → Code security →
+  Dependabot security updates) — auto-opens fix PRs when a CVE lands
+  affecting a tracked dep, plus weekly version-update PRs grouped by
+  minor/patch.
+
+Snyk was retired on 2026-04-28 (post-v2.9.2). Both replacements run
+unconditionally — no scan-quota limits.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "worclaude",
-  "version": "2.9.2",
+  "version": "2.10.0",
   "description": "The Workflow Layer for Claude Code — scaffold agents, commands, skills, hooks, and memory into any project",
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   "funding": "https://github.com/sponsors/sefaertunc",
   "author": "Sefa Ertunç",
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=20.0.0"
   },
   "scripts": {
     "test": "vitest run",
@@ -68,16 +68,16 @@
     "chalk": "^5.4.1",
     "commander": "^13.1.0",
     "fs-extra": "^11.3.0",
-    "inquirer": "^12.5.0",
-    "ora": "^8.2.0",
+    "inquirer": "^13.4.2",
+    "ora": "^9.4.0",
     "smol-toml": "^1.6.1",
     "yaml": "^2.8.3"
   },
   "devDependencies": {
-    "eslint": "^9.22.0",
+    "eslint": "^10.2.1",
     "prettier": "^3.5.3",
     "vitepress": "^1.6.4",
-    "vitest": "^3.0.9"
+    "vitest": "^4.1.5"
   },
   "overrides": {
     "brace-expansion": "^1.1.13",

package/src/core/project-scanner/detectors/readme.js

@@ -10,12 +10,21 @@ const MIN_DESCRIPTION_LENGTH = 20;
 const MAX_README_BYTES = 512 * 1024;
 const BADGE_PATTERNS = [/^\s*\[!\[.*?\]\(.*?\)\]\(.*?\)\s*$/, /^\s*!\[.*?\]\(.*?\)\s*$/];
 
+function stripUntilStable(text, regex) {
+  let prev;
+  do {
+    prev = text;
+    text = text.replace(regex, '');
+  } while (text !== prev);
+  return text;
+}
+
 function stripHtmlComments(text) {
-  return text.replace(/<!--[\s\S]*?-->/g, '');
+  return stripUntilStable(text, /<!--[\s\S]*?-->/g);
 }
 
 function stripHtmlTags(text) {
-  return text.replace(/<[^>]+>/g, '');
+  return stripUntilStable(text, /<[^>]+>/g);
 }
 
 function isSkippable(line) {

package/templates/specs/spec-md-library.md

@@ -52,7 +52,7 @@ catch (error) { /* [ErrorType]: [message] */ }
 ## Compatibility Matrix
 | Runtime/Version        | Supported | Notes                      |
 |------------------------|-----------|----------------------------|
-| [Node.js >= 18]       | Yes       | [ESM and CJS]             |
+| [Node.js >= 20]       | Yes       | [ESM and CJS]             |
 | [Node.js 16]          | No        | [Reason]                   |
 | [Python >= 3.10]      | Yes       | [Type hints required]      |
 | [Browser (ESM)]       | Yes       | [Bundle size: ~N kB gzip]  |