@kalyx/core 1.0.0-rc.10 → 1.0.0-rc.12

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # @kalyx/core
 
+## 1.0.0-rc.12
+
+### Patch Changes
+
+- 0556886: fix(core): validate inputs to `to12Hour` and `to24Hour`
+
+  `to12Hour(hours24)` and `to24Hour(hours12, period)` are public exports from `@kalyx/core` but had no input validation. The previous silent arithmetic mapped invalid inputs onto plausible-looking but wrong outputs and hid caller bugs:
+  - `to12Hour(24)` returned `{ hours12: 12, period: 'PM' }` (because `24 % 12 = 0` → mapped to 12)
+  - `to12Hour(-1)` returned `{ hours12: -1, period: 'AM' }`
+  - `to24Hour(13, 'PM')` returned `25`
+  - `to24Hour(0, 'AM')` returned `0` (but `0` is not a valid 12-hour clock value — midnight is `12 AM`)
+
+  Both functions now throw `RangeError` with a clear message when the input is outside its valid integer range (`[0, 23]` for `to12Hour`, `[1, 12]` for `to24Hour`). `Number.isInteger` guards non-integers and `NaN`. No `@kalyx/react` callers ever passed invalid values, so the internal contracts are unchanged; only direct `@kalyx/core` users who relied on the silent-wrong behaviour see the new exception.
+
+## 1.0.0-rc.11
+
+### Minor Changes
+
+- c8a6609: fix(rangepicker): announce next selection target and final range to screen readers
+
+  `<RangePicker.Calendar>` now announces context-aware messages through its existing `role="status"` live region:
+  - After the first click (start), it announces `<formatted-date>. Now select end date.` so screen-reader users know the next click commits the other endpoint.
+  - After the second click (end), it announces `Range selected: <start> – <end>` instead of just the bare date — matching the swap-if-before behaviour so the announcement always reflects what was committed.
+  - Week-mode commits now share the same `Range selected: ...` prefix for consistency.
+
+  The two new strings are wired through `RangePickerLabels.selectingEnd` and `RangePickerLabels.rangeSelected` with English defaults, and they are fully overridable via the existing `labels` prop for i18n. `@kalyx/core` gets a `minor` bump because `RangePickerLabels` gained required fields (with defaults supplied by `DEFAULT_RANGEPICKER_LABELS`); any consumer constructing a literal `RangePickerLabels` from scratch will need to add the two keys.
+
+### Patch Changes
+
+- 19ac1c0: fix(core): allow `generateMinutes` step values up to 60
+
+  `generateMinutes(step)` rejected any step above 30, which prevented legitimate cases like `step=45` (quarter-and-three-quarters past the hour) and `step=60` (on-the-hour only). The slot-generation loop already works for any 1–60 integer, so the upper bound is now 60 with the same error message format. Steps `0`, `61+`, and negative values still throw. No callers in `@kalyx/react` relied on the previous narrower bound.
+
 ## 1.0.0-rc.10
 
 ### Minor Changes

package/dist/index.cjs

@@ -477,12 +477,18 @@ function formatTimeString(time, withSeconds = false) {
   return `${hh}:${mm}`;
 }
 function to12Hour(hours24) {
+  if (!Number.isInteger(hours24) || hours24 < 0 || hours24 > 23) {
+    throw new RangeError(`[to12Hour] hours24 must be an integer in [0, 23], got ${hours24}`);
+  }
   const period = hours24 >= 12 ? "PM" : "AM";
   let hours12 = hours24 % 12;
   if (hours12 === 0) hours12 = 12;
   return { hours12, period };
 }
 function to24Hour(hours12, period) {
+  if (!Number.isInteger(hours12) || hours12 < 1 || hours12 > 12) {
+    throw new RangeError(`[to24Hour] hours12 must be an integer in [1, 12], got ${hours12}`);
+  }
   if (period === "AM") {
     return hours12 === 12 ? 0 : hours12;
   }
@@ -495,8 +501,8 @@ function generateHours(format = "24h") {
   return Array.from({ length: 24 }, (_, i) => i);
 }
 function generateMinutes(step = 1) {
-  if (step < 1 || step > 30) {
-    throw new Error(`[generateMinutes] step must be between 1 and 30, got ${step}`);
+  if (step < 1 || step > 60) {
+    throw new Error(`[generateMinutes] step must be between 1 and 60, got ${step}`);
   }
   const result = [];
   for (let i = 0; i < 60; i += step) {
@@ -590,7 +596,9 @@ var DEFAULT_RANGEPICKER_LABELS = {
   popoverLabel: "Choose date range",
   startInput: "Start date",
   endInput: "End date",
-  presetsGroup: "Date range presets"
+  presetsGroup: "Date range presets",
+  selectingEnd: "Now select end date",
+  rangeSelected: "Range selected"
 };
 var DEFAULT_TIMEPICKER_LABELS = {
   timeInput: "Time",

package/dist/index.d.cts

@@ -218,7 +218,13 @@ declare function parseTimeString(input: string): TimeValue | null;
 declare function formatTimeString(time: TimeValue, withSeconds?: boolean): string;
 /**
  * Converts a 24-hour value to 12-hour form.
- * 0 → 12 AM, 12 → 12 PM
+ *
+ * - `0` → `{ hours12: 12, period: 'AM' }` (midnight)
+ * - `12` → `{ hours12: 12, period: 'PM' }` (noon)
+ *
+ * Throws if `hours24` isn't an integer in `[0, 23]`. The previous silent
+ * modulo behaviour mapped invalid inputs (e.g. `24`, `25`, `-1`) onto
+ * arbitrary valid-looking outputs, which masked caller bugs.
  */
 declare function to12Hour(hours24: number): {
     hours12: number;
@@ -226,6 +232,10 @@ declare function to12Hour(hours24: number): {
 };
 /**
  * Converts a 12-hour value to 24-hour form.
+ *
+ * Throws if `hours12` isn't an integer in `[1, 12]`. The previous silent
+ * arithmetic produced out-of-range outputs (e.g. `to24Hour(13, 'PM')` → `25`).
+ * `period` is constrained at the type level to `'AM' | 'PM'`.
  */
 declare function to24Hour(hours12: number, period: 'AM' | 'PM'): number;
 /**
@@ -234,9 +244,15 @@ declare function to24Hour(hours12: number, period: 'AM' | 'PM'): number;
 declare function generateHours(format?: '12h' | '24h'): number[];
 /**
  * Builds a minutes list at the given step.
- * step=1 → [0, 1, 2, ..., 59]
- * step=15 → [0, 15, 30, 45]
- * step=5 → [0, 5, 10, ..., 55]
+ *
+ * - `step=1` → `[0, 1, 2, ..., 59]`
+ * - `step=15` → `[0, 15, 30, 45]`
+ * - `step=5` → `[0, 5, 10, ..., 55]`
+ * - `step=45` → `[0, 45]`
+ * - `step=60` → `[0]` (on-the-hour only)
+ *
+ * Steps above 60 are rejected because they always collapse to `[0]` — useful UX
+ * is impossible past that point.
  */
 declare function generateMinutes(step?: number): number[];
 /**
@@ -399,6 +415,16 @@ interface RangePickerLabels extends DatePickerLabels {
     startInput: string;
     endInput: string;
     presetsGroup: string;
+    /**
+     * Screen-reader prompt appended after the start date is picked, telling the user
+     * the next click commits the end of the range.
+     */
+    selectingEnd: string;
+    /**
+     * Screen-reader prefix announced when both endpoints are committed, e.g.
+     * `"Range selected: Jan 5, 2026 – Jan 12, 2026"`.
+     */
+    rangeSelected: string;
 }
 interface TimePickerLabels {
     timeInput: string;

package/dist/index.d.ts

@@ -218,7 +218,13 @@ declare function parseTimeString(input: string): TimeValue | null;
 declare function formatTimeString(time: TimeValue, withSeconds?: boolean): string;
 /**
  * Converts a 24-hour value to 12-hour form.
- * 0 → 12 AM, 12 → 12 PM
+ *
+ * - `0` → `{ hours12: 12, period: 'AM' }` (midnight)
+ * - `12` → `{ hours12: 12, period: 'PM' }` (noon)
+ *
+ * Throws if `hours24` isn't an integer in `[0, 23]`. The previous silent
+ * modulo behaviour mapped invalid inputs (e.g. `24`, `25`, `-1`) onto
+ * arbitrary valid-looking outputs, which masked caller bugs.
  */
 declare function to12Hour(hours24: number): {
     hours12: number;
@@ -226,6 +232,10 @@ declare function to12Hour(hours24: number): {
 };
 /**
  * Converts a 12-hour value to 24-hour form.
+ *
+ * Throws if `hours12` isn't an integer in `[1, 12]`. The previous silent
+ * arithmetic produced out-of-range outputs (e.g. `to24Hour(13, 'PM')` → `25`).
+ * `period` is constrained at the type level to `'AM' | 'PM'`.
  */
 declare function to24Hour(hours12: number, period: 'AM' | 'PM'): number;
 /**
@@ -234,9 +244,15 @@ declare function to24Hour(hours12: number, period: 'AM' | 'PM'): number;
 declare function generateHours(format?: '12h' | '24h'): number[];
 /**
  * Builds a minutes list at the given step.
- * step=1 → [0, 1, 2, ..., 59]
- * step=15 → [0, 15, 30, 45]
- * step=5 → [0, 5, 10, ..., 55]
+ *
+ * - `step=1` → `[0, 1, 2, ..., 59]`
+ * - `step=15` → `[0, 15, 30, 45]`
+ * - `step=5` → `[0, 5, 10, ..., 55]`
+ * - `step=45` → `[0, 45]`
+ * - `step=60` → `[0]` (on-the-hour only)
+ *
+ * Steps above 60 are rejected because they always collapse to `[0]` — useful UX
+ * is impossible past that point.
  */
 declare function generateMinutes(step?: number): number[];
 /**
@@ -399,6 +415,16 @@ interface RangePickerLabels extends DatePickerLabels {
     startInput: string;
     endInput: string;
     presetsGroup: string;
+    /**
+     * Screen-reader prompt appended after the start date is picked, telling the user
+     * the next click commits the end of the range.
+     */
+    selectingEnd: string;
+    /**
+     * Screen-reader prefix announced when both endpoints are committed, e.g.
+     * `"Range selected: Jan 5, 2026 – Jan 12, 2026"`.
+     */
+    rangeSelected: string;
 }
 interface TimePickerLabels {
     timeInput: string;

package/dist/index.js

@@ -426,12 +426,18 @@ function formatTimeString(time, withSeconds = false) {
   return `${hh}:${mm}`;
 }
 function to12Hour(hours24) {
+  if (!Number.isInteger(hours24) || hours24 < 0 || hours24 > 23) {
+    throw new RangeError(`[to12Hour] hours24 must be an integer in [0, 23], got ${hours24}`);
+  }
   const period = hours24 >= 12 ? "PM" : "AM";
   let hours12 = hours24 % 12;
   if (hours12 === 0) hours12 = 12;
   return { hours12, period };
 }
 function to24Hour(hours12, period) {
+  if (!Number.isInteger(hours12) || hours12 < 1 || hours12 > 12) {
+    throw new RangeError(`[to24Hour] hours12 must be an integer in [1, 12], got ${hours12}`);
+  }
   if (period === "AM") {
     return hours12 === 12 ? 0 : hours12;
   }
@@ -444,8 +450,8 @@ function generateHours(format = "24h") {
   return Array.from({ length: 24 }, (_, i) => i);
 }
 function generateMinutes(step = 1) {
-  if (step < 1 || step > 30) {
-    throw new Error(`[generateMinutes] step must be between 1 and 30, got ${step}`);
+  if (step < 1 || step > 60) {
+    throw new Error(`[generateMinutes] step must be between 1 and 60, got ${step}`);
   }
   const result = [];
   for (let i = 0; i < 60; i += step) {
@@ -539,7 +545,9 @@ var DEFAULT_RANGEPICKER_LABELS = {
   popoverLabel: "Choose date range",
   startInput: "Start date",
   endInput: "End date",
-  presetsGroup: "Date range presets"
+  presetsGroup: "Date range presets",
+  selectingEnd: "Now select end date",
+  rangeSelected: "Range selected"
 };
 var DEFAULT_TIMEPICKER_LABELS = {
   timeInput: "Time",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kalyx/core",
-  "version": "1.0.0-rc.10",
+  "version": "1.0.0-rc.12",
   "description": "Kalyx core — platform-agnostic date logic, IANA timezone helpers, and the DateAdapter contract used by @kalyx/react",
   "license": "MIT",
   "author": "jiji-hoon96",