@kernhq/module-hr 0.10.4 → 0.11.0

package/src/client/pages/leave-and-attendance.test.ts

@@ -198,7 +198,13 @@ const attendance = page('./AttendancePage.svelte')
 const leave = page('./LeavePage.svelte')
 
 /** A day sheet row, shaped enough for the `{#if}`s inside the month list. */
-const day = { overtimeMinutes: 0, anomalies: [] as unknown[] }
+const day = { overtimeMinutes: 0, anomalies: [] as unknown[], businessDate: '2026-08-03' }
+/**
+ * What an expandable row reads besides the day itself: whether a correction is waiting on this
+ * date, and whether this row is the open one. Both live outside the `{#each}`, so the resolver has
+ * to be handed them or every render of the month throws.
+ */
+const row = { correctionDates: new Set<string>(), open: false }
 
 describe('the month on the attendance page', () => {
   /**
@@ -211,7 +217,7 @@ describe('the month on the attendance page', () => {
   })
 
   it('draws a day-shaped skeleton while the first fetch is in flight', () => {
-    const body = attendance.render('loading', { loading: true, days: [], daysQuery: {}, day })
+    const body = attendance.render('loading', { loading: true, days: [], daysQuery: {}, day, ...row })
     expect(body).toContain('Skeleton')
     expect(body).not.toContain('EmptyState')
   })
@@ -227,6 +233,7 @@ describe('the month on the attendance page', () => {
       days: [],
       daysQuery: { isError: true },
       day,
+      ...row,
     })
     expect(body).toContain("t('attendance_error')")
     expect(body).toContain("t('retry')")
@@ -239,6 +246,7 @@ describe('the month on the attendance page', () => {
       days: [],
       daysQuery: { isError: false },
       day,
+      ...row,
     })
     expect(body).toContain("t('attendance_none')")
     expect(body).not.toContain("t('attendance_error')")
@@ -255,6 +263,7 @@ describe('the month on the attendance page', () => {
       days: [{ id: 'a' }],
       daysQuery: { isError: true },
       day,
+      ...row,
     })
     expect(body).toContain('{#each days as day (day.id)}')
     expect(body).toContain("t('attendance_stale')")
@@ -268,10 +277,28 @@ describe('the month on the attendance page', () => {
       days: [{ id: 'a' }],
       daysQuery: { isError: false },
       day,
+      ...row,
     })
     expect(body).toContain('{#each days as day (day.id)}')
     expect(body).not.toContain("t('attendance_stale')")
   })
+
+  /**
+   * The anomaly badge used to be `{day.anomalies.length}` — a number in a warning tone with no
+   * noun beside it and nothing to click. It counts things a person has to look at, so it says so,
+   * and the sentences behind it are one row-click away.
+   */
+  it('names what the anomaly badge counts rather than showing a bare number', () => {
+    const body = attendance.render('loading', {
+      loading: false,
+      days: [{ id: 'a' }],
+      daysQuery: { isError: false },
+      day: { ...day, anomalies: ['missing_clock_out'] },
+      ...row,
+    })
+    expect(body).toContain("t('att_anomalies_count', { count: day.anomalies.length })")
+    expect(body).not.toContain('<Badge tone="warning">{day.anomalies.length}</Badge>')
+  })
 })
 
 describe('the attendance tiles', () => {

package/src/client/query.ts

@@ -50,6 +50,27 @@ export const hrKeys = {
   entities: (ws: string) => ['hr', 'entities', ws] as const,
   officePeople: (ws: string, officeId: string, primaryOnly: boolean) =>
     ['hr', 'office-people', ws, officeId, primaryOnly ? 'primary' : 'all'] as const,
+  /**
+   * The ledger is keyed by the leave type *and* the year as well as the person: the panel anchors
+   * its running balance on the balance the server computed for one entitlement year, so serving it
+   * another year's entries from cache would draw a column of numbers reconciling to nothing.
+   */
+  leaveLedger: (ws: string, personId: string, leaveTypeId: string, periodYear: number) =>
+    ['hr', 'leave-ledger', ws, personId, leaveTypeId, periodYear] as const,
+  employmentHistory: (ws: string, personId: string) => ['hr', 'employment-history', ws, personId] as const,
+  documents: (ws: string, personId: string) => ['hr', 'documents', ws, personId] as const,
+  /** Fetched only where the viewer holds `hr.person.view_sensitive`, so it is its own entry. */
+  sensitive: (ws: string, personId: string) => ['hr', 'sensitive', ws, personId] as const,
+  /**
+   * The org editor asks for archived units too, so it cannot share `orgUnits` with the chart —
+   * a toggle that changes what a query asks for has to change the key, or the cache answers the
+   * question it was asked last time and the switch appears to do nothing.
+   */
+  orgUnitsAll: (ws: string) => ['hr', 'org-units', ws, 'with-archived'] as const,
+  positions: (ws: string) => ['hr', 'positions', ws, 'with-archived'] as const,
+  /** Every period in one read: `periods.list` returns no cursor, so there is nothing to page. */
+  periods: (ws: string) => ['hr', 'periods', ws] as const,
+  approvalChains: (ws: string) => ['hr', 'approval-chains', ws] as const,
 } as const
 
 /** `YYYY-MM-DD` for a date, in the viewer's own zone rather than UTC. */