cronli5 0.3.1 → 0.3.4

package/CHANGELOG.md

@@ -6,6 +6,50 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+## [0.3.4]
+
+### Fixed
+
+- **zh:** a bounded *parity* month step dropped its bound — `0 0 1 2-10/2`
+  (months 2,4,6,8,10) rendered "每个偶数月" (every even-numbered month), which
+  wrongly includes December. zh now enumerates a bounded parity step
+  ("2、4、6、8、10月") like en/es/de/fi; the open `*/2`/`2/2` step keeps
+  "单数月/偶数月". The exact analogue of the 0.3.3 day-step fix; found by auditing
+  the bug class that fix implied (the bounded case was thin in the corpora).
+
+## [0.3.3]
+
+### Fixed
+
+- **zh:** a bounded day-of-month step dropped its bounds — `0 0 9-17/2` rendered
+  "每2天" (every 2 days) instead of the days 9, 11, 13, 15, 17. It now enumerates
+  the days like en/es/de/fi ("每月9、11、13、15、17日"); the open `*/N` step keeps
+  "每N天". Surfaced by the new coverage tests (the zh corpus had only open-step
+  rows, so the bounded branch was untested).
+
+### Changed
+
+- Test coverage closed to its true floor after the Vitest migration: 74 new
+  verified corpus rows (core and the English renderer now ~100%), thresholds
+  raised to lines 98.5 / branches 97 / functions 99.2 / statements 98.5. The
+  remaining uncovered branches are genuinely-unreachable defensive guards.
+
+## [0.3.2]
+
+Tooling and docs only — the published library is functionally unchanged.
+
+### Changed
+
+- **Test runner migrated to Vitest** (from mocha + c8-over-tsx). V8 coverage is
+  source-accurate, so the esbuild/tsx phantom-branch artifact is gone and
+  thresholds are gated at the true numbers. (The old c8 figures were wrong in
+  both directions — inflated as well as deflated; real coverage is ~98 lines /
+  96 branches / 99 functions, and the genuine untested-source gaps it revealed
+  are now tracked in the backlog.)
+- Pruned the shipped rendering items (0.2.0–0.3.1) from the backlog, and ran a
+  comment/reference cleanup over the scripts and tooling (stale `IR`→`Schedule`,
+  util-split, and mocha/c8 references; process-label scrub; dead code).
+
 ## [0.3.1]
 
 ### Changed

package/README.md

@@ -330,13 +330,13 @@ required.
 
 ## Development
 
-The library has no runtime dependencies. The toolchain (ESLint, Mocha, Chai,
-c8, esbuild) lives in `devDependencies`.
+The library has no runtime dependencies. The toolchain (ESLint, Vitest, Chai,
+esbuild) lives in `devDependencies`.
 
 ```bash
 npm install       # install dev dependencies (also wires the git hooks)
-npm test          # run the Mocha test suite (runs against src/, no build needed)
-npm run coverage  # run tests with c8 coverage and enforce thresholds
+npm test          # run the Vitest suite (runs against src/, no build needed)
+npm run coverage  # run tests with Vitest V8 coverage and enforce thresholds
 npm run lint      # lint source and tests with ESLint
 npm run build     # emit dist/ (ESM + CJS) and the minified browser global
 npm run verify    # the full CI gate: lint, types, tests, coverage, docs, build

package/dist/lang/zh.cjs

@@ -674,13 +674,20 @@ function render(schedule, plan, opts) {
     opts
   );
 }
+function boundedParityMonth(schedule) {
+  if (schedule.shapes.month !== "step" || isOpenStep(schedule.pattern.month)) {
+    return false;
+  }
+  const segs = segmentsOf(schedule, "month");
+  return segs.length === 1 && segs[0].kind === "step" && segs[0].interval === 2;
+}
 function monthPhrase(schedule) {
   if (schedule.pattern.month === "*") {
     return "";
   }
   const segs = segmentsOf(schedule, "month");
   const first = segs[0];
-  if (segs.length === 1 && first.kind === "step" && first.interval === 2) {
+  if (segs.length === 1 && first.kind === "step" && first.interval === 2 && isOpenStep(schedule.pattern.month)) {
     return "\u6BCF\u4E2A" + (first.fires[0] % 2 ? "\u5947" : "\u5076") + "\u6570\u6708";
   }
   if (segs.length === 1 && first.kind === "range") {
@@ -698,14 +705,14 @@ function monthPhrase(schedule) {
 }
 function dayList(schedule) {
   const segs = segmentsOf(schedule, "date");
-  if (segs.every((seg) => seg.kind === "single")) {
-    return segs.map((seg) => seg.value).join("\u3001") + "\u65E5";
+  if (segs.every((seg) => seg.kind === "single" || seg.kind === "step")) {
+    return fireValues(segs).join("\u3001") + "\u65E5";
   }
   return segs.map(function day(seg) {
     if (seg.kind === "range") {
       return seg.bounds[0] + "\u65E5\u81F3" + seg.bounds[1] + "\u65E5";
     }
-    return seg.value + "\u65E5";
+    return seg.kind === "step" ? seg.fires.join("\u3001") + "\u65E5" : seg.value + "\u65E5";
   }).join("\u3001");
 }
 function quartzDate(token, monthPrefix) {
@@ -743,20 +750,20 @@ function datePhrase(schedule) {
   if (schedule.shapes.date === "quartz") {
     return quartzDate(date, month || "\u672C\u6708");
   }
-  if (schedule.shapes.date === "step") {
+  if (schedule.shapes.date === "step" && isOpenStep(date)) {
     return month + cadence(stepSegment(schedule, "date").interval, "\u5929");
   }
   if (!month) {
     return "\u6BCF\u6708" + dayList(schedule);
   }
-  const monthMulti = schedule.shapes.month === "range" || schedule.shapes.month === "list";
+  const monthMulti = schedule.shapes.month === "range" || schedule.shapes.month === "list" || boundedParityMonth(schedule);
   return month + (monthMulti ? "\uFF0C" : "") + dayList(schedule);
 }
 function dateCore(schedule, quartzPrefix) {
   if (schedule.shapes.date === "quartz") {
     return quartzDate(schedule.pattern.date, quartzPrefix);
   }
-  if (schedule.shapes.date === "step") {
+  if (schedule.shapes.date === "step" && isOpenStep(schedule.pattern.date)) {
     return cadence(stepSegment(schedule, "date").interval, "\u5929");
   }
   return dayList(schedule);
@@ -837,7 +844,8 @@ function composePoint(schedule, core) {
   }
   const dateSet = isSet(schedule.pattern.date);
   const weekdaySet = isSet(schedule.pattern.weekday);
-  const comma = dateSet && weekdaySet || schedule.shapes.date === "step";
+  const dateCadence = schedule.shapes.date === "step" && isOpenStep(schedule.pattern.date);
+  const comma = dateSet && weekdaySet || dateCadence;
   return qual + (comma ? "\uFF0C" : "") + core;
 }
 function composeCadence(schedule, core) {

package/dist/lang/zh.js

@@ -648,13 +648,20 @@ function render(schedule, plan, opts) {
     opts
   );
 }
+function boundedParityMonth(schedule) {
+  if (schedule.shapes.month !== "step" || isOpenStep(schedule.pattern.month)) {
+    return false;
+  }
+  const segs = segmentsOf(schedule, "month");
+  return segs.length === 1 && segs[0].kind === "step" && segs[0].interval === 2;
+}
 function monthPhrase(schedule) {
   if (schedule.pattern.month === "*") {
     return "";
   }
   const segs = segmentsOf(schedule, "month");
   const first = segs[0];
-  if (segs.length === 1 && first.kind === "step" && first.interval === 2) {
+  if (segs.length === 1 && first.kind === "step" && first.interval === 2 && isOpenStep(schedule.pattern.month)) {
     return "\u6BCF\u4E2A" + (first.fires[0] % 2 ? "\u5947" : "\u5076") + "\u6570\u6708";
   }
   if (segs.length === 1 && first.kind === "range") {
@@ -672,14 +679,14 @@ function monthPhrase(schedule) {
 }
 function dayList(schedule) {
   const segs = segmentsOf(schedule, "date");
-  if (segs.every((seg) => seg.kind === "single")) {
-    return segs.map((seg) => seg.value).join("\u3001") + "\u65E5";
+  if (segs.every((seg) => seg.kind === "single" || seg.kind === "step")) {
+    return fireValues(segs).join("\u3001") + "\u65E5";
   }
   return segs.map(function day(seg) {
     if (seg.kind === "range") {
       return seg.bounds[0] + "\u65E5\u81F3" + seg.bounds[1] + "\u65E5";
     }
-    return seg.value + "\u65E5";
+    return seg.kind === "step" ? seg.fires.join("\u3001") + "\u65E5" : seg.value + "\u65E5";
   }).join("\u3001");
 }
 function quartzDate(token, monthPrefix) {
@@ -717,20 +724,20 @@ function datePhrase(schedule) {
   if (schedule.shapes.date === "quartz") {
     return quartzDate(date, month || "\u672C\u6708");
   }
-  if (schedule.shapes.date === "step") {
+  if (schedule.shapes.date === "step" && isOpenStep(date)) {
     return month + cadence(stepSegment(schedule, "date").interval, "\u5929");
   }
   if (!month) {
     return "\u6BCF\u6708" + dayList(schedule);
   }
-  const monthMulti = schedule.shapes.month === "range" || schedule.shapes.month === "list";
+  const monthMulti = schedule.shapes.month === "range" || schedule.shapes.month === "list" || boundedParityMonth(schedule);
   return month + (monthMulti ? "\uFF0C" : "") + dayList(schedule);
 }
 function dateCore(schedule, quartzPrefix) {
   if (schedule.shapes.date === "quartz") {
     return quartzDate(schedule.pattern.date, quartzPrefix);
   }
-  if (schedule.shapes.date === "step") {
+  if (schedule.shapes.date === "step" && isOpenStep(schedule.pattern.date)) {
     return cadence(stepSegment(schedule, "date").interval, "\u5929");
   }
   return dayList(schedule);
@@ -811,7 +818,8 @@ function composePoint(schedule, core) {
   }
   const dateSet = isSet(schedule.pattern.date);
   const weekdaySet = isSet(schedule.pattern.weekday);
-  const comma = dateSet && weekdaySet || schedule.shapes.date === "step";
+  const dateCadence = schedule.shapes.date === "step" && isOpenStep(schedule.pattern.date);
+  const comma = dateSet && weekdaySet || dateCadence;
   return qual + (comma ? "\uFF0C" : "") + core;
 }
 function composeCadence(schedule, core) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cronli5",
-  "version": "0.3.1",
+  "version": "0.3.4",
   "description": "Cron Like I'm Five: A Cron to English Utility",
   "repository": {
     "type": "git",
@@ -65,11 +65,11 @@
     "lint": "eslint src test cli.js eslint.config.js scripts tooling/scripts",
     "metamorphic": "node --import tsx tooling/scripts/metamorphic.mjs",
     "lint:fix": "eslint src test cli.js eslint.config.js scripts tooling/scripts --fix",
-    "test": "mocha",
+    "test": "vitest run",
     "types": "tsc -p tsconfig.types.json",
     "test:types": "npm run types && tsd",
     "typecheck": "tsc -p tsconfig.json",
-    "coverage": "c8 mocha",
+    "coverage": "vitest run --coverage",
     "verify": "npm run lint && npm run typecheck && npm run test:types && npm test && npm run coverage && npm run conciseness && npm run docs -- --check && npm run build",
     "prepare": "node scripts/install-hooks.mjs",
     "prepublishOnly": "npm run lint && npm run typecheck && npm run test:types && npm run build && npm test"
@@ -90,18 +90,18 @@
   "devDependencies": {
     "@eslint/eslintrc": "^3.2.0",
     "@eslint/js": "^9.15.0",
-    "c8": "^10.1.3",
+    "@vitest/coverage-v8": "^4.1.9",
     "chai": "^4.5.0",
     "cronstrue": "^3.14.0",
     "esbuild": "^0.28.1",
     "eslint": "^9.15.0",
     "fast-check": "^3.23.1",
     "globals": "^15.12.0",
-    "mocha": "^11.0.1",
     "tsd": "^0.31.2",
     "tsx": "^4.22.4",
     "typescript": "^6.0.3",
-    "typescript-eslint": "^8.61.0"
+    "typescript-eslint": "^8.61.0",
+    "vitest": "^4.1.9"
   },
   "tsd": {
     "directory": "test-d"

package/src/lang/zh/index.ts

@@ -1041,8 +1041,26 @@ function render(schedule: Schedule, plan: PlanNode, opts: Opts): string {
 
 // --- Day-level qualifier (date / month / weekday / year). ---
 
-// The month phrase: "" (wildcard), "每个奇数月"/"每个偶数月" (step ×2),
+// Whether the month is a BOUNDED parity step ("2-10/2") — an interval-2 step
+// that does NOT span the open parity set. It enumerates as a list of singles
+// ("2、4、6、8、10月"), so it takes the multi-month comma like an explicit list,
+// unlike a single month or a non-parity bounded step ("3-11/3", glued).
+function boundedParityMonth(schedule: Schedule): boolean {
+  if (schedule.shapes.month !== 'step' ||
+      isOpenStep(schedule.pattern.month)) {
+    return false;
+  }
+
+  const segs = segmentsOf(schedule, 'month');
+
+  return segs.length === 1 && segs[0].kind === 'step' && segs[0].interval === 2;
+}
+
+// The month phrase: "" (wildcard), "每个奇数月"/"每个偶数月" (OPEN step ×2),
 // "1月至3月" (range), else the enumerated numbers sharing one 月 ("1、4、7、10月").
+// A BOUNDED parity step ("2-10/2" = months 2,4,6,8,10) fires a finite set, so
+// it enumerates through the number path below rather than the open parity class
+// — the "每个偶数月" wording asserts December too, which the bound excludes.
 function monthPhrase(schedule: Schedule): string {
   if (schedule.pattern.month === '*') {
     return '';
@@ -1051,7 +1069,8 @@ function monthPhrase(schedule: Schedule): string {
   const segs = segmentsOf(schedule, 'month');
   const first = segs[0];
 
-  if (segs.length === 1 && first.kind === 'step' && first.interval === 2) {
+  if (segs.length === 1 && first.kind === 'step' && first.interval === 2 &&
+      isOpenStep(schedule.pattern.month)) {
     return '每个' + (first.fires[0] % 2 ? '奇' : '偶') + '数月';
   }
 
@@ -1074,13 +1093,14 @@ function monthPhrase(schedule: Schedule): string {
   return nums.join('、') + '月';
 }
 
-// The day-of-month list. A pure list of singles shares one trailing 日
-// ("1、3、8日"); any range gives each segment its own 日 ("1至5日、10日").
+// The day-of-month list. A list of singles — or a bounded step enumerated to
+// its fires (9-17/2 = 9,11,13,15,17) — shares one trailing 日 ("1、3、8日",
+// "9、11、13、15、17日"); any range gives each segment its own 日 ("1至5日、10日").
 function dayList(schedule: Schedule): string {
   const segs = segmentsOf(schedule, 'date');
 
-  if (segs.every((seg) => seg.kind === 'single')) {
-    return segs.map((seg) => (seg as {value: string}).value).join('、') + '日';
+  if (segs.every((seg) => seg.kind === 'single' || seg.kind === 'step')) {
+    return fireValues(segs).join('、') + '日';
   }
 
   return segs.map(function day(seg) {
@@ -1088,7 +1108,9 @@ function dayList(schedule: Schedule): string {
       return seg.bounds[0] + '日至' + seg.bounds[1] + '日';
     }
 
-    return (seg as {value: string}).value + '日';
+    return seg.kind === 'step' ?
+      seg.fires.join('、') + '日' :
+      (seg as {value: string}).value + '日';
   }).join('、');
 }
 
@@ -1152,7 +1174,11 @@ function datePhrase(schedule: Schedule): string {
     return quartzDate(date, month || '本月');
   }
 
-  if (schedule.shapes.date === 'step') {
+  // An OPEN day step ("*/N") is a frequency — the bare "每N天" cadence. A
+  // BOUNDED step ("a-b/N") fires a finite set of days, so it enumerates them
+  // through the day-list path below, never the cadence (which would drop the
+  // bounds).
+  if (schedule.shapes.date === 'step' && isOpenStep(date)) {
     return month + cadence(stepSegment(schedule, 'date').interval, '天');
   }
 
@@ -1160,12 +1186,13 @@ function datePhrase(schedule: Schedule): string {
     return '每月' + dayList(schedule);
   }
 
-  // A multi-month scope (range/list) ends in 月 and would run straight into the
+  // A multi-month scope (range/list, or a bounded parity step that enumerates
+  // like a list — "2、4、6、8、10月") ends in 月 and would run straight into the
   // day — "6月至8月1日" reads "8月1日" as August 1st. The comma keeps the month
   // scope distinct from the day ("6月至8月，1日"). A single month stays glued
   // ("6月1日"), which is unambiguous.
   const monthMulti = schedule.shapes.month === 'range' ||
-    schedule.shapes.month === 'list';
+    schedule.shapes.month === 'list' || boundedParityMonth(schedule);
 
   return month + (monthMulti ? '，' : '') + dayList(schedule);
 }
@@ -1178,7 +1205,9 @@ function dateCore(schedule: Schedule, quartzPrefix: string): string {
     return quartzDate(schedule.pattern.date, quartzPrefix);
   }
 
-  if (schedule.shapes.date === 'step') {
+  // An open day step is the bare "每N天" cadence; a bounded step enumerates its
+  // days through dayList (see datePhrase), so the bounds are not dropped.
+  if (schedule.shapes.date === 'step' && isOpenStep(schedule.pattern.date)) {
     return cadence(stepSegment(schedule, 'date').interval, '天');
   }
 
@@ -1318,7 +1347,12 @@ function composePoint(schedule: Schedule, core: string): string {
 
   const dateSet = isSet(schedule.pattern.date);
   const weekdaySet = isSet(schedule.pattern.weekday);
-  const comma = dateSet && weekdaySet || schedule.shapes.date === 'step';
+  // The comma separates an OR union or the open "每N天" cadence from the core. A
+  // bounded date step renders as a glued day list ("每月9、11…日"), not a
+  // cadence, so it takes no comma — only an open step does.
+  const dateCadence = schedule.shapes.date === 'step' &&
+    isOpenStep(schedule.pattern.date);
+  const comma = dateSet && weekdaySet || dateCadence;
 
   return qual + (comma ? '，' : '') + core;
 }