@happyvertical/smrt-timesheets 0.56.0 → 0.56.1

package/AGENTS.md

@@ -48,6 +48,40 @@ files.
 through a `ServiceCommercialResolver`), and corrects. `smrt-projects` supplies
 `SubscriptionServiceCommercialResolver` for #1925 pricing.
 
+## Generated write actions (#4001)
+
+Generated CRUD writes stay closed. `ServiceTimeEntryCollection` hosts the only
+generated writes, `record` / `submit` / `approve` (`POST
+/servicetimeentries/<action>`, body `{ entryId }` for submit/approve), on REST
+only; MCP and CLI stay read-only until per-operation MCP authorization (#4004).
+`services/time-entry-actions.ts` delegates each to `ServiceEvidenceService`, so
+the trail and snapshots are set in one place. `services/action-principal.ts`
+takes identity from the tenancy ALS, then the smrt-users session ALS, never the
+body; authorizes `servicetimeentries.<action>` with `assertOperationPermission`
+(system context never bypasses; `approve` refuses super-admin bypass); and
+stamps the user's linked profile. Only the participant records/submits their
+own time unless the caller holds `approve`. `approve` prices with the host's
+`setServiceTimeEntryCommercialResolver()` — a factory called per request with
+the collection's options, so a writing resolver shares the request executor;
+without one it is a 409 and writes nothing. The runtime REST transport gates
+a collection-hosted action on the ITEM's `api.include`, SvelteKit on the
+collection's, so both list the actions;
+a consumer closing them overrides both registrations. The collection and both
+snapshot collections carry their own `@smrt()` surface: undecorated, they
+published generated create/update/delete MCP tools. `reject` waits for #3994.
+
+## Recipe
+
+`src/recipes.ts` declares one `SmrtRecipe`, `timesheets.time-entries`
+(label "Time entries", section `work`), emitted into the built
+`manifest.json` and `smrt-knowledge.json` `recipes`. It shows
+`ServiceTimeEntry` alone, with the generic work reference, evidence, agent
+and approval-trail fields hidden; attendance and rollup models stay out of
+it. `status` and its trail are hidden and locked, and the help never says to
+set them: `save()` accepts a bare transition, so submit and approve must go
+through `ServiceEvidenceService` (the recipe declares no surface for that). Help lives in `src/time-entries.recipe.md` (#3591); keep it and the recipe
+in plain language. See [core recipes](../core/agents/recipes.md).
+
 ## Invariants
 
 - **No domain foreign key on the base.** Work context is the generic

package/README.md

@@ -131,10 +131,41 @@ recording facade remains seconds-based; its shared entry view uses
 `TimeEntryApprovalQueue` are props-driven: hosts load entries and pass plain
 view objects (`TimeEntry`, `TimeEntryApprovalView`).
 
+## Generated write actions
+
+Generated CRUD writes are closed. The write path a generated app can call is
+three service-backed REST actions on `ServiceTimeEntryCollection` (#4001):
+
+| Route | Body | Permission |
+|---|---|---|
+| `POST /servicetimeentries/record` | work ref, `source` (`manual`/`timer`), times or `durationSeconds`/`durationHours`, optional `participantProfileId` | `servicetimeentries.record` |
+| `POST /servicetimeentries/submit` | `{ entryId }` | `servicetimeentries.submit` |
+| `POST /servicetimeentries/approve` | `{ entryId }` | `servicetimeentries.approve` (no super-admin bypass) |
+
+Each delegates to `ServiceEvidenceService`, so `submittedAt`/`submittedBy`,
+`approvedAt`/`approvedBy`/`approvalPath` (`operator`) and both snapshots are
+set exactly as the service sets them. The tenant and the acting profile come
+from the authenticated session, never the body. Recording or submitting
+someone else's time needs `approve` as well. Install pricing for `approve`
+as a factory, so a resolver that writes (client charges) runs on the request's
+executor and transaction:
+
+```ts
+import { setServiceTimeEntryCommercialResolver } from '@happyvertical/smrt-timesheets';
+
+setServiceTimeEntryCommercialResolver(({ options }) =>
+  SubscriptionServiceCommercialResolver.create(options, provider),
+);
+```
+
+Without a resolver, `approve` answers 409 `approval_not_configured`. MCP and
+CLI stay read-only; there is no `reject` action yet (#3994).
+
 ## Closing the generated surface
 
 The entry and both snapshots expose generated `list` / `get` on REST, CLI,
-and MCP, with optional tenant scoping. An application that routes all time
+and MCP, with optional tenant scoping, and the entry's collection the write
+actions above. An application that routes all time
 and money through its own permission-gated services can close those surfaces
 without declaring replacement models:
 
@@ -144,6 +175,8 @@ import '@happyvertical/smrt-timesheets';
 
 for (const name of [
   'ServiceTimeEntry',
+  // Hosts the record/submit/approve actions on generated SvelteKit routes.
+  'ServiceTimeEntryCollection',
   'ServiceChargeSnapshot',
   'ServiceCompensationSnapshot',
 ]) {

package/dist/attendance.js

@@ -1,3 +1,3 @@
-import { a as AttendanceReplay, c as AttendanceSegmentCollection, i as AttendancePunchCollection, n as AttendanceBreakCollection, o as AttendanceReplayCollection, r as AttendancePunch, s as AttendanceSegment, t as AttendanceBreak } from "./chunks/models-0-YD5F_o.js";
-import { a as continueSegment, c as openSegment, d as validateAttendanceWork, f as ATTENDANCE_CORRECTION_KINDS, i as closeOpenSegment, l as replaceSegment, m as AttendanceCorrectionCollection, n as AttendanceService, o as endSegment, p as AttendanceCorrection, r as beginSegment, s as listSegments, t as AttendanceCorrectionService, u as AttendanceConflict } from "./chunks/attendance-DGVy-iHI.js";
+import { a as AttendanceReplay, c as AttendanceSegmentCollection, i as AttendancePunchCollection, n as AttendanceBreakCollection, o as AttendanceReplayCollection, r as AttendancePunch, s as AttendanceSegment, t as AttendanceBreak } from "./chunks/models-tUePZ5x7.js";
+import { a as continueSegment, c as openSegment, d as validateAttendanceWork, f as ATTENDANCE_CORRECTION_KINDS, i as closeOpenSegment, l as replaceSegment, m as AttendanceCorrectionCollection, n as AttendanceService, o as endSegment, p as AttendanceCorrection, r as beginSegment, s as listSegments, t as AttendanceCorrectionService, u as AttendanceConflict } from "./chunks/attendance-DJNytMTN.js";
 export { ATTENDANCE_CORRECTION_KINDS, AttendanceBreak, AttendanceBreakCollection, AttendanceConflict, AttendanceCorrection, AttendanceCorrectionCollection, AttendanceCorrectionService, AttendancePunch, AttendancePunchCollection, AttendanceReplay, AttendanceReplayCollection, AttendanceSegment, AttendanceSegmentCollection, AttendanceService, beginSegment, closeOpenSegment, continueSegment, endSegment, listSegments, openSegment, replaceSegment, validateAttendanceWork };

package/dist/chunks/{attendance-B1jCbqSb.js → attendance-Cw75uJnT.js}

@@ -1,5 +1,5 @@
-import { i as AttendancePunchCollection, n as AttendanceBreakCollection } from "./models-0-YD5F_o.js";
-import { i as integer, n as clipSource, r as instant } from "./rollup-CDqzCxI2.js";
+import { i as AttendancePunchCollection, n as AttendanceBreakCollection } from "./models-tUePZ5x7.js";
+import { i as integer, n as clipSource, r as instant } from "./rollup-CzCb-DxI.js";
 //#region src/rollup/attendance.ts
 async function attendanceSources(db, actor, period) {
 	const punches = await AttendancePunchCollection.create({ db });
@@ -57,4 +57,4 @@ async function attendanceSources(db, actor, period) {
 //#endregion
 export { attendanceSources };
 
-//# sourceMappingURL=attendance-B1jCbqSb.js.map
+//# sourceMappingURL=attendance-Cw75uJnT.js.map

package/dist/chunks/{attendance-B1jCbqSb.js.map → attendance-Cw75uJnT.js.map}

@@ -1 +1 @@
-{"version":3,"file":"attendance-B1jCbqSb.js","names":[],"sources":["../../src/rollup/attendance.ts"],"sourcesContent":["import type { DatabaseInterface } from '@happyvertical/sql';\nimport {\n  AttendanceBreakCollection,\n  AttendancePunchCollection,\n} from '../attendance/models.js';\nimport { clipSource, instant, integer } from './calculation.js';\nimport type { TimecardActor, TimecardPeriod, TimecardSource } from './types.js';\n\n/** Optional attendance reader. Called on the rollup's transaction executor. */\nexport async function attendanceSources(\n  db: DatabaseInterface,\n  actor: TimecardActor,\n  period: TimecardPeriod,\n): Promise<{ sources: TimecardSource[]; linkedEntryIds: Set<string> }> {\n  const punches = await AttendancePunchCollection.create({ db });\n  const breaks = await AttendanceBreakCollection.create({ db });\n  const sources: TimecardSource[] = [];\n  const linkedEntryIds = new Set<string>();\n  for (const punch of await punches.list({ where: { ...actor } })) {\n    // A linked punch is authoritative even when pending review: never count its\n    // evidence entry behind the review gate, or outside the punch's period.\n    if (punch.serviceTimeEntryId) linkedEntryIds.add(punch.serviceTimeEntryId);\n    if (!punch.endedAt || punch.reviewRequired) continue;\n    const start = instant(punch.startedAt);\n    const end = instant(punch.endedAt);\n    if (end < start) throw new Error('Invalid attendance interval.');\n    let cursor = start;\n    let unpaidMs = 0;\n    const spans: [number, number][] = [];\n    for (const pause of await breaks.list({\n      where: { tenantId: actor.tenantId, punchId: punch.id },\n      orderBy: 'started_at ASC',\n    })) {\n      if (!pause.endedAt)\n        throw new Error('Closed attendance has an open break.');\n      const left = instant(pause.startedAt);\n      const right = instant(pause.endedAt);\n      if (left < cursor || right < left || right > end)\n        throw new Error('Invalid or overlapping attendance breaks.');\n      if (!pause.paid) {\n        spans.push([cursor, left]);\n        unpaidMs += right - left;\n      } else {\n        spans.push([cursor, right]);\n      }\n      cursor = right;\n    }\n    spans.push([cursor, end]);\n    const net = Math.round((end - start - unpaidMs) / 1000);\n    if (integer(punch.durationSeconds) !== net)\n      throw new Error('Attendance duration does not match its breaks.');\n    // Round elapsed paid prefixes once, preserving conservation across breaks.\n    let paidMs = 0;\n    for (const [left, right] of spans) {\n      const seconds =\n        Math.round((paidMs + right - left) / 1000) - Math.round(paidMs / 1000);\n      paidMs += right - left;\n      const source = clipSource(\n        {\n          kind: 'attendance',\n          id: String(punch.id),\n          startsAt: new Date(left).toISOString(),\n          endsAt: new Date(right).toISOString(),\n          seconds,\n        },\n        period,\n      );\n      if (source?.seconds) sources.push(source);\n    }\n  }\n  return { sources, linkedEntryIds };\n}\n"],"mappings":";;;AASA,eAAsB,kBACpB,IACA,OACA,QACqE;CACrE,MAAM,UAAU,MAAM,0BAA0B,OAAO,EAAE,GAAG,CAAC;CAC7D,MAAM,SAAS,MAAM,0BAA0B,OAAO,EAAE,GAAG,CAAC;CAC5D,MAAM,UAA4B,CAAC;CACnC,MAAM,iCAAiB,IAAI,IAAY;CACvC,KAAA,MAAW,SAAS,MAAM,QAAQ,KAAK,EAAE,OAAO,EAAE,GAAG,MAAM,EAAE,CAAC,GAAG;EAG/D,IAAI,MAAM,oBAAoB,eAAe,IAAI,MAAM,kBAAkB;EACzE,IAAI,CAAC,MAAM,WAAW,MAAM,gBAAgB;EAC5C,MAAM,QAAQ,QAAQ,MAAM,SAAS;EACrC,MAAM,MAAM,QAAQ,MAAM,OAAO;EACjC,IAAI,MAAM,OAAO,MAAM,IAAI,MAAM,8BAA8B;EAC/D,IAAI,SAAS;EACb,IAAI,WAAW;EACf,MAAM,QAA4B,CAAC;EACnC,KAAA,MAAW,SAAS,MAAM,OAAO,KAAK;GACpC,OAAO;IAAE,UAAU,MAAM;IAAU,SAAS,MAAM;GAAG;GACrD,SAAS;EACX,CAAC,GAAG;GACF,IAAI,CAAC,MAAM,SACT,MAAM,IAAI,MAAM,sCAAsC;GACxD,MAAM,OAAO,QAAQ,MAAM,SAAS;GACpC,MAAM,QAAQ,QAAQ,MAAM,OAAO;GACnC,IAAI,OAAO,UAAU,QAAQ,QAAQ,QAAQ,KAC3C,MAAM,IAAI,MAAM,2CAA2C;GAC7D,IAAI,CAAC,MAAM,MAAM;IACf,MAAM,KAAK,CAAC,QAAQ,IAAI,CAAC;IACzB,YAAY,QAAQ;GACtB,OACE,MAAM,KAAK,CAAC,QAAQ,KAAK,CAAC;GAE5B,SAAS;EACX;EACA,MAAM,KAAK,CAAC,QAAQ,GAAG,CAAC;EACxB,MAAM,MAAM,KAAK,OAAO,MAAM,QAAQ,YAAY,GAAI;EACtD,IAAI,QAAQ,MAAM,eAAe,MAAM,KACrC,MAAM,IAAI,MAAM,gDAAgD;EAElE,IAAI,SAAS;EACb,KAAA,MAAW,CAAC,MAAM,UAAU,OAAO;GACjC,MAAM,UACJ,KAAK,OAAO,SAAS,QAAQ,QAAQ,GAAI,IAAI,KAAK,MAAM,SAAS,GAAI;GACvE,UAAU,QAAQ;GAClB,MAAM,SAAS,WACb;IACE,MAAM;IACN,IAAI,OAAO,MAAM,EAAE;IACnB,UAAU,IAAI,KAAK,IAAI,CAAA,CAAE,YAAY;IACrC,QAAQ,IAAI,KAAK,KAAK,CAAA,CAAE,YAAY;IACpC;GACF,GACA,MACF;GACA,IAAI,QAAQ,SAAS,QAAQ,KAAK,MAAM;EAC1C;CACF;CACA,OAAO;EAAE;EAAS;CAAe;AACnC"}
+{"version":3,"file":"attendance-Cw75uJnT.js","names":[],"sources":["../../src/rollup/attendance.ts"],"sourcesContent":["import type { DatabaseInterface } from '@happyvertical/sql';\nimport {\n  AttendanceBreakCollection,\n  AttendancePunchCollection,\n} from '../attendance/models.js';\nimport { clipSource, instant, integer } from './calculation.js';\nimport type { TimecardActor, TimecardPeriod, TimecardSource } from './types.js';\n\n/** Optional attendance reader. Called on the rollup's transaction executor. */\nexport async function attendanceSources(\n  db: DatabaseInterface,\n  actor: TimecardActor,\n  period: TimecardPeriod,\n): Promise<{ sources: TimecardSource[]; linkedEntryIds: Set<string> }> {\n  const punches = await AttendancePunchCollection.create({ db });\n  const breaks = await AttendanceBreakCollection.create({ db });\n  const sources: TimecardSource[] = [];\n  const linkedEntryIds = new Set<string>();\n  for (const punch of await punches.list({ where: { ...actor } })) {\n    // A linked punch is authoritative even when pending review: never count its\n    // evidence entry behind the review gate, or outside the punch's period.\n    if (punch.serviceTimeEntryId) linkedEntryIds.add(punch.serviceTimeEntryId);\n    if (!punch.endedAt || punch.reviewRequired) continue;\n    const start = instant(punch.startedAt);\n    const end = instant(punch.endedAt);\n    if (end < start) throw new Error('Invalid attendance interval.');\n    let cursor = start;\n    let unpaidMs = 0;\n    const spans: [number, number][] = [];\n    for (const pause of await breaks.list({\n      where: { tenantId: actor.tenantId, punchId: punch.id },\n      orderBy: 'started_at ASC',\n    })) {\n      if (!pause.endedAt)\n        throw new Error('Closed attendance has an open break.');\n      const left = instant(pause.startedAt);\n      const right = instant(pause.endedAt);\n      if (left < cursor || right < left || right > end)\n        throw new Error('Invalid or overlapping attendance breaks.');\n      if (!pause.paid) {\n        spans.push([cursor, left]);\n        unpaidMs += right - left;\n      } else {\n        spans.push([cursor, right]);\n      }\n      cursor = right;\n    }\n    spans.push([cursor, end]);\n    const net = Math.round((end - start - unpaidMs) / 1000);\n    if (integer(punch.durationSeconds) !== net)\n      throw new Error('Attendance duration does not match its breaks.');\n    // Round elapsed paid prefixes once, preserving conservation across breaks.\n    let paidMs = 0;\n    for (const [left, right] of spans) {\n      const seconds =\n        Math.round((paidMs + right - left) / 1000) - Math.round(paidMs / 1000);\n      paidMs += right - left;\n      const source = clipSource(\n        {\n          kind: 'attendance',\n          id: String(punch.id),\n          startsAt: new Date(left).toISOString(),\n          endsAt: new Date(right).toISOString(),\n          seconds,\n        },\n        period,\n      );\n      if (source?.seconds) sources.push(source);\n    }\n  }\n  return { sources, linkedEntryIds };\n}\n"],"mappings":";;;AASA,eAAsB,kBACpB,IACA,OACA,QACqE;CACrE,MAAM,UAAU,MAAM,0BAA0B,OAAO,EAAE,GAAG,CAAC;CAC7D,MAAM,SAAS,MAAM,0BAA0B,OAAO,EAAE,GAAG,CAAC;CAC5D,MAAM,UAA4B,CAAC;CACnC,MAAM,iCAAiB,IAAI,IAAY;CACvC,KAAA,MAAW,SAAS,MAAM,QAAQ,KAAK,EAAE,OAAO,EAAE,GAAG,MAAM,EAAE,CAAC,GAAG;EAG/D,IAAI,MAAM,oBAAoB,eAAe,IAAI,MAAM,kBAAkB;EACzE,IAAI,CAAC,MAAM,WAAW,MAAM,gBAAgB;EAC5C,MAAM,QAAQ,QAAQ,MAAM,SAAS;EACrC,MAAM,MAAM,QAAQ,MAAM,OAAO;EACjC,IAAI,MAAM,OAAO,MAAM,IAAI,MAAM,8BAA8B;EAC/D,IAAI,SAAS;EACb,IAAI,WAAW;EACf,MAAM,QAA4B,CAAC;EACnC,KAAA,MAAW,SAAS,MAAM,OAAO,KAAK;GACpC,OAAO;IAAE,UAAU,MAAM;IAAU,SAAS,MAAM;GAAG;GACrD,SAAS;EACX,CAAC,GAAG;GACF,IAAI,CAAC,MAAM,SACT,MAAM,IAAI,MAAM,sCAAsC;GACxD,MAAM,OAAO,QAAQ,MAAM,SAAS;GACpC,MAAM,QAAQ,QAAQ,MAAM,OAAO;GACnC,IAAI,OAAO,UAAU,QAAQ,QAAQ,QAAQ,KAC3C,MAAM,IAAI,MAAM,2CAA2C;GAC7D,IAAI,CAAC,MAAM,MAAM;IACf,MAAM,KAAK,CAAC,QAAQ,IAAI,CAAC;IACzB,YAAY,QAAQ;GACtB,OACE,MAAM,KAAK,CAAC,QAAQ,KAAK,CAAC;GAE5B,SAAS;EACX;EACA,MAAM,KAAK,CAAC,QAAQ,GAAG,CAAC;EACxB,MAAM,MAAM,KAAK,OAAO,MAAM,QAAQ,YAAY,GAAI;EACtD,IAAI,QAAQ,MAAM,eAAe,MAAM,KACrC,MAAM,IAAI,MAAM,gDAAgD;EAElE,IAAI,SAAS;EACb,KAAA,MAAW,CAAC,MAAM,UAAU,OAAO;GACjC,MAAM,UACJ,KAAK,OAAO,SAAS,QAAQ,QAAQ,GAAI,IAAI,KAAK,MAAM,SAAS,GAAI;GACvE,UAAU,QAAQ;GAClB,MAAM,SAAS,WACb;IACE,MAAM;IACN,IAAI,OAAO,MAAM,EAAE;IACnB,UAAU,IAAI,KAAK,IAAI,CAAA,CAAE,YAAY;IACrC,QAAQ,IAAI,KAAK,KAAK,CAAA,CAAE,YAAY;IACpC;GACF,GACA,MACF;GACA,IAAI,QAAQ,SAAS,QAAQ,KAAK,MAAM;EAC1C;CACF;CACA,OAAO;EAAE;EAAS;CAAe;AACnC"}

package/dist/chunks/{attendance-DGVy-iHI.js → attendance-DJNytMTN.js}

@@ -1,5 +1,5 @@
-import { i as ServiceTimeEntryCollection, o as parseTimesheetJson } from "./service-time-entry-t7P-bkOQ.js";
-import { c as AttendanceSegmentCollection, d as persistAutoClose, f as persistCorrection, i as AttendancePunchCollection, l as deleteCorrection, n as AttendanceBreakCollection, o as AttendanceReplayCollection, r as AttendancePunch, s as AttendanceSegment, t as AttendanceBreak, u as isCorrectionWrite } from "./models-0-YD5F_o.js";
+import { i as ServiceTimeEntryCollection, o as parseTimesheetJson } from "./service-time-entry-BJVSGx33.js";
+import { c as AttendanceSegmentCollection, d as persistAutoClose, f as persistCorrection, i as AttendancePunchCollection, l as deleteCorrection, n as AttendanceBreakCollection, o as AttendanceReplayCollection, r as AttendancePunch, s as AttendanceSegment, t as AttendanceBreak, u as isCorrectionWrite } from "./models-tUePZ5x7.js";
 import { SmrtCollection, SmrtObject, crossPackageRef, field, foreignKey, isEmbeddedDatabase, isPostgresDatabase, smrt, withEmbeddedWriteTransaction } from "@happyvertical/smrt-core";
 import { TenantScoped, getTenantId, tenantId, withTenant } from "@happyvertical/smrt-tenancy";
 //#region src/attendance/correction-models.ts
@@ -1243,4 +1243,4 @@ var AttendanceCorrectionService = class {
 //#endregion
 export { continueSegment as a, openSegment as c, validateAttendanceWork as d, ATTENDANCE_CORRECTION_KINDS as f, closeOpenSegment as i, replaceSegment as l, AttendanceCorrectionCollection as m, AttendanceService as n, endSegment as o, AttendanceCorrection as p, beginSegment as r, listSegments as s, AttendanceCorrectionService as t, AttendanceConflict as u };
 
-//# sourceMappingURL=attendance-DGVy-iHI.js.map
+//# sourceMappingURL=attendance-DJNytMTN.js.map