@fabriktor/fx 0.0.119 → 0.0.121

package/dist/src/chat/send-queue.d.ts

@@ -1,5 +1,6 @@
 import type { ChatOperator } from "@fabriktor/client";
 import type { Chat, InChat } from "@fabriktor/schema";
+import { type EventListener } from "../core/events.js";
 import { type QueueRetry, type QueueStore, type QueueLock } from "../core/queue.js";
 export declare const ChatSendStatus: {
     readonly Saving: "saving";
@@ -55,7 +56,8 @@ export declare class ChatSendQueue {
     private entries;
     private confirmed;
     private confirmedSequences;
-    private listeners;
+    private readonly events;
+    private readonly changed;
     private readonly queue;
     private uploads;
     private confirmations;
@@ -72,7 +74,7 @@ export declare class ChatSendQueue {
     constructor(opts: ChatSendQueueOptions);
     get hasUnsavedMessages(): boolean;
     get snapshot(): ChatSendSnapshot;
-    subscribe(listener: (snapshot: ChatSendSnapshot) => void): () => void;
+    subscribe(listener: EventListener<ChatSendSnapshot>): () => void;
     start(ownerID: string): Promise<void>;
     pause(): void;
     enqueue(input: InChat, attachments: ChatSendAttachments): string;

package/dist/src/chat/send-queue.js

@@ -4,6 +4,7 @@
 // not use this file except in compliance with the License. You may obtain
 // a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
 import { requiredOperationID } from "../core/id.js";
+import { createEvent, EventBus } from "../core/events.js";
 import { PersistentQueue, QueueWakeup, nextQueueRetry, } from "../core/queue.js";
 import { newOptimisticChat } from "./optimistic.js";
 export const ChatSendStatus = {
@@ -27,7 +28,8 @@ export class ChatSendQueue {
     entries = [];
     confirmed = new Map();
     confirmedSequences = new Map();
-    listeners = new Set();
+    events = new EventBus();
+    changed = createEvent();
     queue;
     uploads = new Set();
     confirmations = new Set();
@@ -59,9 +61,7 @@ export class ChatSendQueue {
         };
     }
     subscribe(listener) {
-        this.listeners.add(listener);
-        listener(this.snapshot);
-        return () => this.listeners.delete(listener);
+        return this.events.subscribe(this.changed, listener, () => this.snapshot);
     }
     async start(ownerID) {
         if (this.enabled && this.owner === ownerID) {
@@ -690,9 +690,7 @@ export class ChatSendQueue {
     }
     publish() {
         this.revision++;
-        for (const listener of this.listeners) {
-            listener(this.snapshot);
-        }
+        this.events.publish(this.changed, this.snapshot);
     }
 }
 function chatSendCanConfirm(entry) {

package/dist/src/core/events.d.ts

@@ -0,0 +1,13 @@
+declare const eventPayload: unique symbol;
+export type EventKey<T> = symbol & {
+    readonly [eventPayload]: (value: T) => T;
+};
+export type EventListener<T> = (value: T) => void | Promise<void>;
+export declare function createEvent<T>(): EventKey<T>;
+export declare function notifyObserver(run: () => unknown): void;
+export declare class EventBus {
+    private readonly listeners;
+    subscribe<T>(event: EventKey<T>, listener: EventListener<NoInfer<T>>, initial?: () => T): () => void;
+    publish<T>(event: EventKey<T>, value: NoInfer<T>): void;
+}
+export {};

package/dist/src/core/events.js

@@ -0,0 +1,47 @@
+// Copyright (C) Fabriktor, Inc. 2025-present.
+//
+// Licensed under the Apache License, Version 2.0 (the "License"); you may
+// not use this file except in compliance with the License. You may obtain
+// a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
+export function createEvent() {
+    return Symbol();
+}
+export function notifyObserver(run) {
+    try {
+        void Promise.resolve(run()).catch(observerError);
+    }
+    catch (err) {
+        observerError(err);
+    }
+}
+function observerError(err) {
+    console.error("Event observer failed.", err);
+}
+export class EventBus {
+    listeners = new Map();
+    subscribe(event, listener, initial) {
+        let listeners = this.listeners.get(event);
+        if (!listeners) {
+            listeners = new Set();
+            this.listeners.set(event, listeners);
+        }
+        const observer = listener;
+        listeners.add(observer);
+        if (initial)
+            notifyObserver(() => listener(initial()));
+        return () => {
+            listeners.delete(observer);
+            if (!listeners.size && this.listeners.get(event) === listeners)
+                this.listeners.delete(event);
+        };
+    }
+    publish(event, value) {
+        const listeners = this.listeners.get(event);
+        if (!listeners)
+            return;
+        for (const listener of [...listeners]) {
+            if (listeners.has(listener))
+                notifyObserver(() => listener(value));
+        }
+    }
+}

package/dist/src/core/multi-create.d.ts

@@ -1,3 +1,4 @@
+import { type EventListener } from "./events.js";
 export declare const multiCreateMaximumEntries = 40;
 export declare const multiCreateConcurrency = 5;
 export declare const MultiCreateStatus: {
@@ -52,13 +53,14 @@ export type MultiCreateOptions<Input, Result> = {
 export declare class MultiCreator<Input, Result> {
     private readonly opts;
     private entries;
-    private listeners;
+    private readonly events;
+    private readonly changed;
     private running;
     private controller;
     private completion;
     constructor(opts: MultiCreateOptions<Input, Result>);
     get snapshot(): MultiCreateSnapshot<Input, Result>;
-    subscribe(listener: (snapshot: MultiCreateSnapshot<Input, Result>) => void): () => void;
+    subscribe(listener: EventListener<MultiCreateSnapshot<Input, Result>>): () => void;
     add(input: Input): string;
     edit(id: string, input: Input): boolean;
     remove(id: string): boolean;
@@ -70,5 +72,4 @@ export declare class MultiCreator<Input, Result> {
     private run;
     private replace;
     private publish;
-    private notify;
 }

package/dist/src/core/multi-create.js

@@ -3,6 +3,7 @@
 // Licensed under the Apache License, Version 2.0 (the "License"); you may
 // not use this file except in compliance with the License. You may obtain
 // a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
+import { createEvent, EventBus, notifyObserver, } from "./events.js";
 export const multiCreateMaximumEntries = 40;
 export const multiCreateConcurrency = 5;
 export const MultiCreateStatus = {
@@ -47,7 +48,8 @@ const sharedMultiCreatePool = new MultiCreatePool();
 export class MultiCreator {
     opts;
     entries = [];
-    listeners = new Set();
+    events = new EventBus();
+    changed = createEvent();
     running = false;
     controller = new AbortController();
     completion = Promise.resolve();
@@ -58,9 +60,7 @@ export class MultiCreator {
         return { entries: this.entries, running: this.running };
     }
     subscribe(listener) {
-        this.listeners.add(listener);
-        listener(this.snapshot);
-        return () => this.listeners.delete(listener);
+        return this.events.subscribe(this.changed, listener, () => this.snapshot);
     }
     add(input) {
         if (this.running || this.entries.length >= multiCreateMaximumEntries)
@@ -170,14 +170,14 @@ export class MultiCreator {
             catch (error) {
                 current = { ...current, status: MultiCreateStatus.Failed, error };
                 this.replace(current);
-                this.notify(() => this.opts.on_failed?.(current));
+                notifyObserver(() => this.opts.on_failed?.(current));
             }
         })));
         this.running = false;
         this.publish();
         const ids = new Set(pending.map((entry) => entry.id));
         const entries = this.entries.filter((entry) => ids.has(entry.id));
-        this.notify(() => this.opts.on_settled?.({
+        notifyObserver(() => this.opts.on_settled?.({
             entries,
             created: entries.filter((entry) => entry.status === MultiCreateStatus.Created).length,
             failed: entries.filter((entry) => entry.status === MultiCreateStatus.Failed).length,
@@ -188,16 +188,6 @@ export class MultiCreator {
         this.publish();
     }
     publish() {
-        for (const listener of this.listeners)
-            this.notify(() => listener(this.snapshot));
-    }
-    notify(run) {
-        // Presentation errors cannot turn a successful creation into a failed POST.
-        try {
-            run();
-        }
-        catch (err) {
-            console.error("Multi-creation observer failed.", err);
-        }
+        this.events.publish(this.changed, this.snapshot);
     }
 }

package/dist/src/fx.d.ts

@@ -4,6 +4,7 @@ import type { BillableFX, BillableFXOperator } from "./billable/billable.js";
 import type { CallsFX, CallsFXOperator } from "./calls/calls.js";
 import type { ChatFX, ChatFXOperator } from "./chat/chat.js";
 import type { ContactsFX, ContactsFXOperator } from "./contacts/contacts.js";
+import { EventBus } from "./core/events.js";
 import type { FeedFX, FeedFXOperator } from "./feed/feed.js";
 import type { FulfillmentsFX, FulfillmentsFXOperator } from "./fulfillments/fulfillments.js";
 import type { JobsFX, JobsFXOperator } from "./jobs/jobs.js";
@@ -56,6 +57,7 @@ export type DefaultFXOptions = {
     marketplace_keep_alive_interval_ms: number;
 };
 export interface FXOperator {
+    readonly events: EventBus;
     billable: BillableFXOperator;
     calls: CallsFXOperator;
     chat: ChatFXOperator;
@@ -78,6 +80,7 @@ export interface FXOperator {
     users: UsersFXOperator;
 }
 export declare class FX implements FXOperator {
+    readonly events: EventBus;
     billable: BillableFX;
     calls: CallsFX;
     chat: ChatFX;

package/dist/src/fx.js

@@ -8,6 +8,7 @@ import { newBillableFX } from "./billable/billable.js";
 import { newCallsFX } from "./calls/calls.js";
 import { newChatFX } from "./chat/chat.js";
 import { newContactsFX } from "./contacts/contacts.js";
+import { EventBus } from "./core/events.js";
 import { newFeedFX } from "./feed/feed.js";
 import { newFulfillmentsFX } from "./fulfillments/fulfillments.js";
 import { newJobsFX } from "./jobs/jobs.js";
@@ -25,6 +26,7 @@ import { newStorageFX } from "./storage/storage.js";
 import { newTimesheetsFX } from "./timesheets/timesheets.js";
 import { newUsersFX } from "./users/users.js";
 export class FX {
+    events = new EventBus();
     billable;
     calls;
     chat;
@@ -83,6 +85,7 @@ export class FX {
         });
         this.preferences = newPreferencesFX({
             preferences: opts.preferences,
+            events: this.events,
         });
         this.privacy = newPrivacyFX({
             privacy: opts.privacy,

package/dist/src/index.d.ts

@@ -8,6 +8,7 @@ export * from "./contacts/contacts.js";
 export * from "./contacts/scope.js";
 export * from "./core/date.js";
 export * from "./core/err.js";
+export * from "./core/events.js";
 export * from "./core/id.js";
 export * from "./core/multi-create.js";
 export * from "./core/pre-action.js";

package/dist/src/index.js

@@ -8,6 +8,7 @@ export * from "./contacts/contacts.js";
 export * from "./contacts/scope.js";
 export * from "./core/date.js";
 export * from "./core/err.js";
+export * from "./core/events.js";
 export * from "./core/id.js";
 export * from "./core/multi-create.js";
 export * from "./core/pre-action.js";

package/dist/src/preferences/preferences.d.ts

@@ -1,5 +1,9 @@
 import type { PreferencesOperator } from "@fabriktor/client";
 import { type InPrefClient, type InPrefCompany, type InPrefEmployee, type OperationResult, type PrefPayment, type PrefClient, type PrefEmployee, type PrefCompany, type PrefVisibility, type PrefLanguage, type PrefNotif, type PrefEmail, type PrefTutorial } from "@fabriktor/schema";
+import { EventBus } from "../core/events.js";
+export declare const PreferencesEvents: {
+    readonly CompanySaved: import("../index.js").EventKey<PrefCompany>;
+};
 type PrefPaymentInsertOptions = Parameters<PreferencesOperator["insertOnePrefPayment"]>[0];
 type PrefPaymentReplaceOptions = Parameters<PreferencesOperator["replaceOnePrefPayment"]>[0];
 type PrefClientInsertOptions = Parameters<PreferencesOperator["insertOnePrefClient"]>[0];
@@ -112,9 +116,11 @@ export interface PreferencesFXOperator {
 }
 export type PreferencesFXOptions = {
     preferences: PreferencesOperator;
+    events?: EventBus;
 };
 export declare class PreferencesFX implements PreferencesFXOperator {
     private preferences;
+    private readonly events;
     constructor(opts: PreferencesFXOptions);
     createPrefPayment(opts: CreatePrefPaymentOptions): Promise<PrefPayment>;
     replacePrefPayment(opts: ReplacePrefPaymentOptions): Promise<PrefPayment>;

package/dist/src/preferences/preferences.js

@@ -6,7 +6,11 @@
 import { queryParam } from "@fabriktor/client";
 import { OwnerIDKey, } from "@fabriktor/schema";
 import { errResolveFailed } from "../core/err.js";
+import { createEvent, EventBus } from "../core/events.js";
 import { requiredOperationID } from "../core/id.js";
+export const PreferencesEvents = {
+    CompanySaved: createEvent(),
+};
 const resourcePrefPayment = "pref_payment";
 const msgCreatedPrefPaymentIDMissing = "Created preference payment response did not contain an ID.";
 const msgPrefPaymentMissing = "Preference payment could not be resolved after the mutation.";
@@ -40,8 +44,10 @@ export const PrefCompanyDocumentNote = {
 };
 export class PreferencesFX {
     preferences;
+    events;
     constructor(opts) {
         this.preferences = opts.preferences;
+        this.events = opts.events ?? new EventBus();
     }
     async createPrefPayment(opts) {
         const out = await this.preferences.insertOnePrefPayment(opts);
@@ -102,11 +108,15 @@ export class PreferencesFX {
     async createPrefCompany(opts) {
         const out = await this.preferences.insertOnePrefCompany(opts);
         const id = requiredOperationID(out, msgCreatedPrefCompanyIDMissing);
-        return await this.findPrefCompany(id);
+        const pref = await this.findPrefCompany(id);
+        this.events.publish(PreferencesEvents.CompanySaved, pref);
+        return pref;
     }
     async replacePrefCompany(opts) {
         await this.preferences.replaceOnePrefCompany(opts);
-        return await this.findPrefCompany(opts.id);
+        const pref = await this.findPrefCompany(opts.id);
+        this.events.publish(PreferencesEvents.CompanySaved, pref);
+        return pref;
     }
     async getPrefCompany(opts) {
         const pref = await this.queryPrefCompany(opts.owner_id);

package/dist/src/timesheets/access.d.ts

@@ -13,9 +13,16 @@ export type QueryTimesheetSyncDestinationsOptions = {
     source: WeekSheet;
     signal?: AbortSignal;
 };
+export type QueryTimesheetSyncSourcesOptions = Pick<TimesheetsReadOptions, "context" | "owner_ids" | "period_indexes" | "signal">;
+export type TimesheetSyncTarget = {
+    owner_id: string;
+    week_sheet?: WeekSheet;
+};
 type TimesheetRecord = WeekSheet | Summary;
 export declare function timesheetCanReadResource(context: TimesheetsActor): boolean;
 export declare function timesheetCanCreateWeek(context: TimesheetsActor): boolean;
+export declare function timesheetCanManageSync(context: TimesheetsActor): boolean;
+export declare function timesheetCanSyncRecord(context: TimesheetsActor, record: WeekSheet | null | undefined): boolean;
 export declare function timesheetCanReadRecord(context: TimesheetsActor, record: TimesheetRecord | null | undefined): boolean;
 export declare function timesheetCanWriteRecord(context: TimesheetsActor, record: TimesheetRecord | null | undefined): boolean;
 export declare function timesheetCanDeleteRecord(context: TimesheetsActor, record: TimesheetRecord | null | undefined): boolean;

package/dist/src/timesheets/access.js

@@ -1,13 +1,20 @@
-import { PunchStatusWorking, UserTypeEmployee, } from "@fabriktor/schema";
+import { PunchStatusWorking, UserTypeEmployee, UserTypeEmployer, } from "@fabriktor/schema";
 import { employmentActivityOwnerIDs, employmentCanRead, employmentEmployeeIDs, employmentEmployerIDs, employmentManagesRecord, employmentOwnsRecord, } from "../contacts/scope.js";
 import { restrictOwnerIDs } from "../core/scope.js";
-import { timesheetCanApprove, timesheetCanSync, timesheetHasWorkingPunches, timesheetIsMutable, } from "./workspace.js";
+import { timesheetCanApprove, timesheetCanSync, timesheetCanSyncSource, timesheetHasWorkingPunches, timesheetIsMutable, } from "./workspace.js";
 export function timesheetCanReadResource(context) {
     return employmentCanRead(context);
 }
 export function timesheetCanCreateWeek(context) {
     return timesheetCanReadResource(context) && context.active_mode === UserTypeEmployee;
 }
+export function timesheetCanManageSync(context) {
+    return timesheetCanReadResource(context) && context.active_mode === UserTypeEmployer;
+}
+export function timesheetCanSyncRecord(context, record) {
+    return record != null && timesheetCanManageSync(context) &&
+        employmentManagesRecord(context, record) && timesheetCanSyncSource(record);
+}
 export function timesheetCanReadRecord(context, record) {
     return timesheetCanReadResource(context) && record != null && (employmentOwnsRecord(context, record) || employmentManagesRecord(context, record));
 }
@@ -30,7 +37,7 @@ export function timesheetCanEditDay(context, record, week_day) {
         day.punches.every((punch) => punch.punch_status !== PunchStatusWorking);
 }
 export function timesheetCanSyncDestination(context, source, destination) {
-    return timesheetCanApproveRecord(context, source) &&
+    return timesheetCanSyncRecord(context, source) &&
         timesheetCanWriteRecord(context, destination) && timesheetCanSync(source, destination);
 }
 export function timesheetEmployeeScopeIDs(context) {

package/dist/src/timesheets/timesheets.d.ts

@@ -1,7 +1,7 @@
 import { type Date as SchemaDate, type OperationResult, type Summary, type WeekSheet } from "@fabriktor/schema";
 import type { BillableOperator, TimesheetsOperator } from "@fabriktor/client";
 import type { ContactsFXOperator } from "../contacts/contacts.js";
-import { type LoadTimesheetsReadContextOptions, type QueryTimesheetSyncDestinationsOptions, type TimesheetsReadContext, type TimesheetsReadOptions } from "./access.js";
+import { type LoadTimesheetsReadContextOptions, type QueryTimesheetSyncDestinationsOptions, type QueryTimesheetSyncSourcesOptions, type TimesheetsReadContext, type TimesheetsReadOptions, type TimesheetSyncTarget } from "./access.js";
 import { TimesheetsWorkspace } from "./workspace.js";
 type PunchPayload = Parameters<TimesheetsOperator["punch"]>[0];
 type WeekSheetReplaceOptions = Parameters<TimesheetsOperator["replaceOneWeekSheet"]>[0];
@@ -57,7 +57,9 @@ export type PunchControls = {
 };
 export type SyncWeekOptions = {
     from: WeekSheet;
-    to: WeekSheet[];
+    to?: WeekSheet[];
+    to_owner_ids?: string[];
+    actor?: LoadTimesheetsReadContextOptions;
 };
 export type SyncDaysOptions = SyncWeekOptions & {
     days: number[];
@@ -72,6 +74,8 @@ export interface TimesheetsFXOperator {
     queryReadableWeekSheets(opts: TimesheetsReadOptions): Promise<WeekSheet[]>;
     queryReadableSummaries(opts: TimesheetsReadOptions): Promise<Summary[]>;
     querySyncDestinations(opts: QueryTimesheetSyncDestinationsOptions): Promise<WeekSheet[]>;
+    querySyncSources(opts: QueryTimesheetSyncSourcesOptions): Promise<WeekSheet[]>;
+    querySyncTargets(opts: QueryTimesheetSyncDestinationsOptions): Promise<TimesheetSyncTarget[]>;
     replaceWeekSheet(opts: ReplaceWeekSheetOptions): Promise<WeekSheet>;
     replaceSummary(opts: ReplaceSummaryOptions): Promise<Summary>;
     punchControls(opts: PunchControlsOptions): PunchControls;
@@ -96,6 +100,8 @@ export declare class TimesheetsFX implements TimesheetsFXOperator {
     queryReadableWeekSheets(opts: TimesheetsReadOptions): Promise<WeekSheet[]>;
     queryReadableSummaries(opts: TimesheetsReadOptions): Promise<Summary[]>;
     querySyncDestinations(opts: QueryTimesheetSyncDestinationsOptions): Promise<WeekSheet[]>;
+    querySyncSources(opts: QueryTimesheetSyncSourcesOptions): Promise<WeekSheet[]>;
+    querySyncTargets(opts: QueryTimesheetSyncDestinationsOptions): Promise<TimesheetSyncTarget[]>;
     replaceWeekSheet(opts: ReplaceWeekSheetOptions): Promise<WeekSheet>;
     replaceSummary(opts: ReplaceSummaryOptions): Promise<Summary>;
     punchControls(opts: PunchControlsOptions): PunchControls;

package/dist/src/timesheets/timesheets.js

@@ -9,8 +9,8 @@ import { errRequired, errResolveFailed, errValidation, } from "../core/err.js";
 import { normalizeIDs } from "../core/id.js";
 import { loadEmploymentReadContext } from "../contacts/scope.js";
 import { PreActionBlocked } from "../core/pre-action.js";
-import { timesheetCanApproveRecord, timesheetCanSyncDestination, timesheetReadFilters, } from "./access.js";
-import { TimesheetsWorkspace, TimesheetsBlock, timesheetCanApprove, timesheetCanSync, timesheetIsMutable, } from "./workspace.js";
+import { timesheetCanSyncRecord, timesheetCanSyncDestination, timesheetEmployeeScopeIDs, timesheetReadFilters, } from "./access.js";
+import { TimesheetsWorkspace, TimesheetsBlock, timesheetCanApprove, timesheetCanSync, timesheetCanSyncSource, timesheetIsMutable, } from "./workspace.js";
 const allWeekDays = [0, 1, 2, 3, 4, 5, 6];
 const minuteStep = 15;
 const msgPunchInProgress = "A punch action is already in progress.";
@@ -55,7 +55,7 @@ export class TimesheetsFX {
         const source = await this.workspace.week(opts.source.id, opts.signal);
         if (source.updated_at !== opts.source.updated_at)
             throw new PreActionBlocked(TimesheetsBlock.Changed);
-        if (!timesheetCanApproveRecord(opts.context, source))
+        if (!timesheetCanSyncRecord(opts.context, source))
             throw new PreActionBlocked(TimesheetsBlock.InvalidSync);
         const weeks = await this.queryReadableWeekSheets({
             context: opts.context,
@@ -66,6 +66,41 @@ export class TimesheetsFX {
         });
         return weeks.filter((week) => timesheetCanSyncDestination(opts.context, source, week));
     }
+    async querySyncSources(opts) {
+        // Full weeks are needed for eligibility; never use cleaned day/punch results.
+        const weeks = await this.queryReadableWeekSheets({
+            context: opts.context,
+            owner_ids: opts.owner_ids,
+            period_indexes: opts.period_indexes,
+            signal: opts.signal,
+        });
+        return weeks.filter((week) => timesheetCanSyncRecord(opts.context, week));
+    }
+    async querySyncTargets(opts) {
+        const source = await this.workspace.week(opts.source.id, opts.signal);
+        if (source.updated_at !== opts.source.updated_at)
+            throw new PreActionBlocked(TimesheetsBlock.Changed);
+        if (!timesheetCanSyncRecord(opts.context, source))
+            throw new PreActionBlocked(TimesheetsBlock.InvalidSync);
+        // Include locked records: hiding them would mistake them for missing weeks.
+        const weeks = await this.queryReadableWeekSheets({
+            context: opts.context,
+            employer_ids: [source.employer_id],
+            period_indexes: [source.period_index],
+            signal: opts.signal,
+        });
+        const targets = [];
+        for (const owner_id of timesheetEmployeeScopeIDs(opts.context)) {
+            if (owner_id === source.owner_id)
+                continue;
+            const matches = weeks.filter((week) => week.owner_id === owner_id);
+            if (matches.length === 0)
+                targets.push({ owner_id });
+            else if (matches.length === 1 && timesheetCanSyncDestination(opts.context, source, matches[0]))
+                targets.push({ owner_id, week_sheet: matches[0] });
+        }
+        return targets;
+    }
     async replaceWeekSheet(opts) {
         await this.timesheets.replaceOneWeekSheet(opts);
         return await this.findWeekSheet(opts.id);
@@ -212,8 +247,7 @@ export class TimesheetsFX {
     }
     async syncWeek(opts) {
         await this.syncSheets({
-            from: opts.from,
-            to: opts.to,
+            ...opts,
             days: [...allWeekDays],
         });
     }
@@ -226,8 +260,7 @@ export class TimesheetsFX {
             });
         }
         await this.syncSheets({
-            from: opts.from,
-            to: opts.to,
+            ...opts,
             days,
         });
     }
@@ -308,18 +341,36 @@ export class TimesheetsFX {
         const source = await this.findWeekSheet(opts.from.id);
         if (source.updated_at !== opts.from.updated_at)
             throw new PreActionBlocked(TimesheetsBlock.Changed);
-        if (!timesheetCanApprove(source))
+        if (!timesheetCanSyncSource(source))
             throw new PreActionBlocked(TimesheetsBlock.InvalidSync);
         opts = { ...opts, from: source };
         const from_id = requiredID(opts.from.id, "from", msgWeekSheetIDRequired);
-        if (opts.to.length === 0) {
+        const owner_ids = normalizeIDs(opts.to_owner_ids ?? []);
+        if (!opts.to?.length && owner_ids.length === 0) {
             throw errRequired({
                 field: "to",
                 msg: msgDestinationWeekSheetsRequired,
             });
         }
+        if (owner_ids.length > 0) {
+            if (!opts.actor)
+                throw new PreActionBlocked(TimesheetsBlock.InvalidSync);
+            const context = await this.loadReadContext(opts.actor);
+            const targets = await this.querySyncTargets({ context, source });
+            if (owner_ids.some((id) => !targets.some((target) => target.owner_id === id)))
+                throw new PreActionBlocked(TimesheetsBlock.InvalidSync);
+            // Refetch full records immediately before submission. The service checks
+            // again inside its transaction and creates missing destinations there.
+            for (const target of targets) {
+                if (!owner_ids.includes(target.owner_id) || !target.week_sheet)
+                    continue;
+                const fresh = await this.findWeekSheet(target.week_sheet.id);
+                if (!timesheetCanSync(source, fresh))
+                    throw new PreActionBlocked(TimesheetsBlock.InvalidSync);
+            }
+        }
         const to_ids = [];
-        for (const expected of opts.to) {
+        for (const expected of opts.to ?? []) {
             const week_sheet = await this.findWeekSheet(expected.id);
             if (week_sheet.updated_at !== expected.updated_at)
                 throw new PreActionBlocked(TimesheetsBlock.Changed);
@@ -337,6 +388,7 @@ export class TimesheetsFX {
         const input = {
             from_id,
             to_ids: normalizeIDs(to_ids),
+            to_owner_ids: owner_ids,
             days: normalizeDays(opts.days),
         };
         await this.timesheets.sync(input);

package/dist/src/timesheets/workspace.d.ts

@@ -50,6 +50,7 @@ export declare function timesheetPunches(sheet: WeekSheet): Punch[];
 export declare function timesheetHasWorkingPunches(sheet: WeekSheet): boolean;
 export declare function timesheetIsMutable(sheet: WeekSheet): boolean;
 export declare function timesheetCanApprove(sheet: WeekSheet): boolean;
+export declare function timesheetCanSyncSource(sheet: WeekSheet): boolean;
 export declare function timesheetCanSync(source: WeekSheet, destination: WeekSheet): boolean;
 export declare function timesheetStateKey(state: TimesheetPunchState): string;
 export declare function timesheetWeekDay(date: InDate): number;

package/dist/src/timesheets/workspace.js

@@ -32,12 +32,18 @@ export function timesheetCanApprove(sheet) {
         punches.length > 0 &&
         punches.every((p) => p.punch_status === PunchStatusPending));
 }
+export function timesheetCanSyncSource(sheet) {
+    const punches = timesheetPunches(sheet);
+    return punches.length > 0 && punches.every((punch) => punch.punch_status === PunchStatusPending || punch.punch_status === PunchStatusApproved);
+}
 export function timesheetCanSync(source, destination) {
-    return timesheetCanApprove(source) && timesheetIsMutable(destination) &&
-        !timesheetHasWorkingPunches(destination) &&
+    return timesheetCanSyncSource(source) && !destination.is_archived &&
+        timesheetPunches(destination).every((punch) => punch.punch_status === PunchStatusPending) &&
         destination.employer_id === source.employer_id &&
         destination.id !== source.id &&
         destination.period_index === source.period_index &&
+        destination.period === source.period &&
+        destination.period_year === source.period_year &&
         destination.timezone === source.timezone;
 }
 export function timesheetStateKey(state) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fabriktor/fx",
-  "version": "0.0.119",
+  "version": "0.0.121",
   "description": "![Fabriktor Logo](./img/fabriktor-character.gif)",
   "type": "module",
   "exports": {
@@ -17,8 +17,8 @@
     "typescript": "^6.0.3"
   },
   "dependencies": {
-    "@fabriktor/client": "0.0.88",
+    "@fabriktor/client": "0.0.90",
     "@fabriktor/fx": "0.0.94",
-    "@fabriktor/schema": "0.0.64"
+    "@fabriktor/schema": "0.0.65"
   }
 }