@kernhq/module-tracker 0.7.0 → 0.7.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kernhq/module-tracker",
-  "version": "0.7.0",
+  "version": "0.7.2",
   "description": "Kern tracker module: projects, work item types, custom fields, workflows, issues, KQL, cycles, views, reports, intake, time tracking (server + client skeleton)",
   "license": "AGPL-3.0-only",
   "type": "module",

package/src/client/format.test.ts

@@ -0,0 +1,29 @@
+import { describe, expect, it } from 'vitest'
+import { formatDuration } from './format.js'
+
+describe('formatDuration', () => {
+  it('keeps minutes below an hour', () => {
+    // A timer running for twenty minutes used to read "0h".
+    expect(formatDuration(60)).toBe('1m')
+    expect(formatDuration(20 * 60)).toBe('20m')
+    expect(formatDuration(59 * 60)).toBe('59m')
+  })
+
+  it('keeps minutes below a day', () => {
+    // 90 logged minutes used to read "2h", which is not a rounding, it is wrong.
+    expect(formatDuration(90 * 60)).toBe('1h 30m')
+    expect(formatDuration(2 * 3600)).toBe('2h')
+    expect(formatDuration(7 * 3600 + 45 * 60)).toBe('7h 45m')
+  })
+
+  it('drops minutes above a day, where they stop being the point', () => {
+    expect(formatDuration(26 * 3600)).toBe('1d 2h')
+    expect(formatDuration(48 * 3600)).toBe('2d')
+    expect(formatDuration(26 * 3600 + 30 * 60)).toBe('1d 2h')
+  })
+
+  it('says nothing rather than something wrong for nothing at all', () => {
+    expect(formatDuration(0)).toBe('0h')
+    expect(formatDuration(-5)).toBe('0h')
+  })
+})

package/src/client/format.ts

@@ -153,12 +153,25 @@ export function daysUntil(dueDate: string, today = new Date()): number {
   return Math.round((due - start) / 86_400_000)
 }
 
-/** Seconds of logged time as the board footer shows it ("0h", "3h", "1d 2h"). */
+/**
+ * Seconds of time as somebody would say it: `45m`, `1h 30m`, `3h`, `1d 2h`.
+ *
+ * Minutes matter below a day. Rounding to whole hours reported 90 logged minutes as "2h" and a
+ * timer running for twenty as "0h" — for an estimate that is a rounding, for time somebody tracked
+ * it is wrong, and the same function shows both.
+ *
+ * Above a day, minutes stop being the point and are dropped.
+ */
 export function formatDuration(seconds: number): string {
   if (seconds <= 0) return '0h'
-  const hours = Math.round(seconds / 3600)
-  if (hours < 24) return `${hours}h`
-  const days = Math.floor(hours / 24)
-  const rest = hours % 24
-  return rest ? `${days}d ${rest}h` : `${days}d`
+  const totalMinutes = Math.round(seconds / 60)
+  if (totalMinutes < 60) return `${totalMinutes}m`
+
+  const totalHours = Math.floor(totalMinutes / 60)
+  const minutes = totalMinutes % 60
+  if (totalHours < 24) return minutes ? `${totalHours}h ${minutes}m` : `${totalHours}h`
+
+  const days = Math.floor(totalHours / 24)
+  const hours = totalHours % 24
+  return hours ? `${days}d ${hours}h` : `${days}d`
 }

package/src/client/types.ts

@@ -55,6 +55,8 @@ export type {
   StatusHistoryEntry,
   StatusInfo,
   SystemFieldId,
+  Timer,
+  TimesheetRow,
   UpdateIssue,
   Version,
   View,
@@ -62,4 +64,5 @@ export type {
   ViewLayout,
   Workflow,
   WorkItemType,
+  Worklog,
 } from '../contract/index.js'