scriveno 2.7.1 → 2.7.2

package/README.md

@@ -2,7 +2,7 @@
 
 [![CI](https://github.com/aihxp/scriveno/actions/workflows/ci.yml/badge.svg)](https://github.com/aihxp/scriveno/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
-[![Version](https://img.shields.io/badge/version-2.7.1-blue)](CHANGELOG.md)
+[![Version](https://img.shields.io/badge/version-2.7.2-blue)](CHANGELOG.md)
 [![npm](https://img.shields.io/npm/v/scriveno.svg)](https://www.npmjs.com/package/scriveno)
 [![Downloads](https://img.shields.io/npm/dm/scriveno.svg)](https://www.npmjs.com/package/scriveno)
 [![Status CLI](https://img.shields.io/badge/status%20CLI-scriveno%20status-blue)](docs/runtime-support.md#shared-auto-invoke-engine)
@@ -246,11 +246,11 @@ Scriveno currently ships installer targets for these AI tooling environments:
 
 ## Status
 
-**Version:** 2.7.1
+**Version:** 2.7.2
 
 Scriveno's core command surface is stable across 113 commands, 50 work types, and 11 installer targets. The current repo baseline includes shipped planning milestones through `v2.0 Publishing Cover Packaging`, plus the creative-context, record-store, branching-next, runtime-sync, adaptive concierge, human-first writing-safeguard, authenticity-diagnostic, domain-grilling, installer-marker cleanup, cross-runtime agent metadata, visible automation status, the shared `scriveno status --project .` auto-invoke engine, route-intelligence lanes, safe apply reporting, runtime smoke checks, agent availability checks, route graph audits, the full audit repair pass through `2.0.11`, the first-run proof surface in `2.5.0`, and the executable `/scr:first-run` path. See [Quick Proof](docs/quick-proof.md) for the fastest proof path, [Shipped Assets](docs/shipped-assets.md) for the canonical asset inventory, and [Runtime Support](docs/runtime-support.md) for the runtime compatibility matrix.
 
-Version `2.7.1` publishes Scriveno under the package name `scriveno`, so the current install command is `npx scriveno@latest`. The older `scriveno-cli` package name is historical and was unpublished during the rename, so npm cannot attach a deprecation notice to it while it has no active registry record. The older `scriven-cli` package remains on npm only as a deprecated legacy name that points users to `scriveno`. Do not treat either legacy package name as active unless a deliberate compatibility shim is republished. See [CHANGELOG](CHANGELOG.md) for the full list and [docs/release-notes.md](docs/release-notes.md) for the public-facing summary.
+Version `2.7.2` publishes Scriveno under the package name `scriveno`, so the current install command is `npx scriveno@latest`. The older `scriveno-cli` package name is historical and was unpublished during the rename, so npm cannot attach a deprecation notice to it while it has no active registry record. The older `scriven-cli` package remains on npm only as a deprecated legacy name that points users to `scriveno`. Do not treat either legacy package name as active unless a deliberate compatibility shim is republished. See [CHANGELOG](CHANGELOG.md) for the full list and [docs/release-notes.md](docs/release-notes.md) for the public-facing summary.
 
 Package history is tracked in [CHANGELOG.md](CHANGELOG.md), and the public-facing summary for this release is in [docs/release-notes.md](docs/release-notes.md).
 

package/commands/scr/new-work.md

@@ -69,7 +69,7 @@ Always create `RECORD.md` from `templates/RECORD.md` without renaming it. It is
 Write `.manuscript/config.json` by starting from `templates/config.json` and filling the project-specific values. The generated config must include the shared settings blocks that later commands read:
 ```json
 {
-  "scriveno_version": "2.7.1",
+  "scriveno_version": "2.7.2",
   "work_type": "<chosen>",
   "group": "<group>",
   "command_unit": "<unit>",

package/commands/scr/scan.md

@@ -38,8 +38,8 @@ Run every check. Do not stop on first finding. Bundle them all into one report.
 Read `STATE.md` for `Units drafted`, `Units planned`, `Units reviewed`, and `Total words`. Then count what is actually on disk:
 
 - **Units drafted** = files matching `.manuscript/drafts/body/*-DRAFT.md`
-- **Units planned** = files matching `.manuscript/*-*-PLAN.md` (numeric-prefixed plan files)
-- **Units reviewed** = files matching `.manuscript/*-EDITOR-NOTES.md`
+- **Units planned** = files matching `.manuscript/plans/*-PLAN.md`; legacy root-level `.manuscript/*-PLAN.md` also counts
+- **Units reviewed** = files matching `.manuscript/reviews/*-REVIEW.md`; legacy root-level `.manuscript/*-EDITOR-NOTES.md` also counts
 - **Total words** = `wc -w` summed across all `drafts/body/*-DRAFT.md` files
 
 For each metric where STATE.md value != filesystem value, emit:
@@ -67,7 +67,7 @@ For each orphan, emit one finding with the unit identifier and the file path.
 
 ### CHECK 3: Plan vs. draft alignment
 
-For each `*-PLAN.md` file in `.manuscript/`, check whether the corresponding `*-DRAFT.md` exists in `drafts/body/`. If a plan exists but no draft:
+For each `*-PLAN.md` file in `.manuscript/plans/` (and any legacy root-level `*-PLAN.md` file), check whether the corresponding `*-DRAFT.md` exists in `drafts/body/`. If a plan exists but no draft:
 
 ```
 INFO   Unit 8 has a plan ({N}-{A}-PLAN.md) but no draft yet.

package/data/CONSTRAINTS.json

@@ -1,6 +1,6 @@
 {
   "$schema": "./constraints.schema.json",
-  "version": "2.7.1",
+  "version": "2.7.2",
   "description": "Scriveno constraint system: work types, command availability, exports, and dependencies. Every command checks this file at runtime.",
   "_notes": {
     "sacred_keys": "Sacred subcommands live at commands/scr/sacred/<name>.md and run as /scr:sacred:<name>. Their CONSTRAINTS keys use the sacred:<name> form so /scr:help can render the runnable slash-command path directly. The sacred-numbering-format entry is a separate flat command (commands/scr/sacred-numbering-format.md) that surfaces the active tradition's numbering format. It used to be named sacred-verse-numbering, which collided with sacred:verse-numbering at install time -- both flattened to scr-sacred-verse-numbering. Renamed in v1.6.x; the installer now refuses to install when two sources share a flat skill name."

package/docs/configuration.md

@@ -57,7 +57,7 @@ When a writer runs `/scr:new-work`, Scriveno creates `.manuscript/config.json`.
 
 ```json
 {
-  "scriveno_version": "2.7.1",
+  "scriveno_version": "2.7.2",
   "work_type": "<chosen>",
   "group": "<group>",
   "command_unit": "<unit>",

package/docs/release-notes.md

@@ -2,11 +2,23 @@
 
 This document is the public-facing summary of what changed between package releases. For package history, see the root [CHANGELOG](../CHANGELOG.md).
 
+## 2.7.2 - 2026-05-30
+
+### What changed
+
+- Progress ledger correctness: reviewed units with open editor notes now stay in progress instead of being counted as done. Submitted units still count as done, and clean reviews count as done when the workflow stops at review.
+- `/scr:scan` now points its plan and review counts at the canonical `.manuscript/plans/` and `.manuscript/reviews/` directories, with legacy root-level fallbacks.
+- The 2.7.1 release text now matches the actual `scriveno status` output.
+
+### Why it matters
+
+The progress bar now reflects the writer's real state of work: review notes still waiting on revision no longer look finished.
+
 ## 2.7.1 - 2026-05-30
 
 ### What changed
 
-- `scriveno status` now prints the progress ledger directly (a `Progress:` line with bar, done / in progress / untouched, and pipeline position), so the deliverable view is visible from the bundled CLI, not only to runtimes that load the engine module.
+- `scriveno status` now prints the progress ledger directly (a `Progress:` line with the bar and done / in progress / untouched counts), so the deliverable view is visible from the bundled CLI, not only to runtimes that load the engine module.
 - Documentation-integrity pass: corrected the context-integrity layer description (now five files), added `PROGRESS.md` to the scan trust-file lists and the architecture template tree, and refreshed stale version and count references across the docs.
 
 ### Why it matters

package/docs/route-graph.md

@@ -13,7 +13,7 @@ The text report summarizes command count, graph edges, agent-capable routes, loc
 
 ## Current Shape
 
-As of `2.7.1`, the route graph contains:
+As of `2.7.2`, the route graph contains:
 
 - 113 commands
 - intent-order edges from `command_intents`

package/lib/auto-invoke-engine.js

@@ -91,6 +91,7 @@ const REVIEW_KEYWORDS = [
   'VOICE DRIFT',
   'CONTINUITY',
 ];
+const OPEN_REVIEW_KEYWORDS = REVIEW_KEYWORDS.filter((keyword) => keyword !== 'CONTINUITY');
 
 const CORE_PROJECT_FILES = [
   'WORK.md',
@@ -1404,6 +1405,30 @@ function ledgerUnitNumbers(dir, suffixRegex) {
   return found;
 }
 
+function ledgerUnitFileMap(dir, suffixRegex) {
+  const found = new Map();
+  let entries;
+  try {
+    entries = fs.readdirSync(dir);
+  } catch (err) {
+    return found;
+  }
+  for (const name of entries) {
+    const match = name.match(suffixRegex);
+    if (match) {
+      found.set(parseInt(match[1], 10), path.join(dir, name));
+    }
+  }
+  return found;
+}
+
+function mergeUnitFileMaps(target, source) {
+  for (const [unit, file] of source.entries()) {
+    if (!target.has(unit)) target.set(unit, file);
+  }
+  return target;
+}
+
 function ledgerOutlineUnitCount(manuscriptDir) {
   let text;
   try {
@@ -1426,6 +1451,39 @@ function ledgerBar(done, total, width) {
   return '█'.repeat(clamped) + '░'.repeat(cells - clamped);
 }
 
+function ledgerReferencedUnits(text) {
+  const units = new Set();
+  const pattern = /\b(?:unit|chapter|scene|section|act|part|surah|procedure|poem)\s*#?\s*(\d+)\b/gi;
+  let match;
+  while ((match = pattern.exec(text)) !== null) {
+    const unit = parseInt(match[1], 10);
+    if (unit > 0) units.add(unit);
+  }
+  return units;
+}
+
+function stateOpenRevisionUnits(manuscriptDir) {
+  const text = readText(path.join(manuscriptDir, 'STATE.md'));
+  const open = new Set();
+  for (const line of text.split(/\r?\n/)) {
+    if (!/(open revisions|unresolved review|editor notes.*awaiting revision|awaiting revision|revisions pending)/i.test(line)) {
+      continue;
+    }
+    for (const unit of ledgerReferencedUnits(line)) open.add(unit);
+  }
+  return open;
+}
+
+function stateSubmittedUnits(manuscriptDir) {
+  const text = readText(path.join(manuscriptDir, 'STATE.md'));
+  const submitted = new Set();
+  for (const line of text.split(/\r?\n/)) {
+    if (!/\b(submit|submitted)\b/i.test(line)) continue;
+    for (const unit of ledgerReferencedUnits(line)) submitted.add(unit);
+  }
+  return submitted;
+}
+
 // Returns deliverable progress for a project's .manuscript directory: total
 // units, per-stage counts, the done / in-progress / untouched buckets, a
 // percent, a rendered bar, and the unit-number sets. Derived purely from disk
@@ -1439,17 +1497,30 @@ function computeProgressLedger(manuscriptDir) {
   for (const unit of ledgerUnitNumbers(manuscriptDir, /^(\d+)\D.*PLAN\.md$/i)) {
     planned.add(unit);
   }
-  const reviewed = ledgerUnitNumbers(path.join(manuscriptDir, 'reviews'), /^(\d+)\D.*REVIEW\.md$/i);
-  for (const unit of ledgerUnitNumbers(manuscriptDir, /^(\d+)\D.*EDITOR-NOTES\.md$/i)) {
-    reviewed.add(unit);
+  const reviewFiles = ledgerUnitFileMap(path.join(manuscriptDir, 'reviews'), /^(\d+)\D.*REVIEW\.md$/i);
+  mergeUnitFileMaps(reviewFiles, ledgerUnitFileMap(manuscriptDir, /^(\d+)\D.*EDITOR-NOTES\.md$/i));
+  const reviewed = new Set(reviewFiles.keys());
+  const submitted = stateSubmittedUnits(manuscriptDir);
+  const openReviews = stateOpenRevisionUnits(manuscriptDir);
+  for (const [unit, file] of reviewFiles.entries()) {
+    if (containsAny(readText(file), OPEN_REVIEW_KEYWORDS)) {
+      openReviews.add(unit);
+    }
+  }
+  for (const unit of submitted) {
+    openReviews.delete(unit);
+  }
+  const doneUnits = new Set(submitted);
+  for (const unit of reviewed) {
+    if (!openReviews.has(unit)) doneUnits.add(unit);
   }
 
-  const worked = new Set([...planned, ...drafted, ...reviewed]);
+  const worked = new Set([...planned, ...drafted, ...reviewed, ...submitted]);
   const maxWorked = worked.size ? Math.max(...worked) : 0;
   const total = Math.max(ledgerOutlineUnitCount(manuscriptDir), worked.size, maxWorked);
 
-  const done = reviewed.size;
-  const inProgress = [...worked].filter((unit) => !reviewed.has(unit)).length;
+  const done = doneUnits.size;
+  const inProgress = [...worked].filter((unit) => !doneUnits.has(unit)).length;
   const untouched = Math.max(0, total - worked.size);
   const percent = total > 0 ? Math.round((done / total) * 100) : 0;
 
@@ -1458,6 +1529,7 @@ function computeProgressLedger(manuscriptDir) {
     drafted: drafted.size,
     planned: planned.size,
     reviewed: reviewed.size,
+    submitted: submitted.size,
     done,
     inProgress,
     untouched,
@@ -1467,6 +1539,9 @@ function computeProgressLedger(manuscriptDir) {
       drafted: [...drafted].sort((a, b) => a - b),
       planned: [...planned].sort((a, b) => a - b),
       reviewed: [...reviewed].sort((a, b) => a - b),
+      submitted: [...submitted].sort((a, b) => a - b),
+      openReviews: [...openReviews].sort((a, b) => a - b),
+      done: [...doneUnits].sort((a, b) => a - b),
     },
   };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scriveno",
-  "version": "2.7.1",
+  "version": "2.7.2",
   "description": "Spec-driven creative writing, publishing, and translation pipeline for AI coding agents. From blank page to published book.",
   "bin": {
     "scriveno": "bin/install.js"

package/templates/config.json

@@ -1,5 +1,5 @@
 {
-  "scriveno_version": "2.7.1",
+  "scriveno_version": "2.7.2",
   "work_type": "",
   "group": "",
   "command_unit": "",