@7365admin1/layer-common 4.141.0 → 4.143.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @iservice365/layer-common
 
+## 4.143.0
+
+### Minor Changes
+
+- 87617ca: NFC patrol: route timing On/Off switch on the route form (Off = no start times or tolerance, patrol at any time; routes list and preview show "Anytime (timing off)"); "Select all / Unselect all" for the route's days; any number of checkpoints (was a 1-10 list) and every tag of the site offered (was the first 100); tolerance sent as a number. Reports: a checkpoint still Pending after its timed round's window closed counts as missed and the round reads "Incomplete" (was "Completed, 0 missed"), a round still open reads "In progress", untimed rounds are never missed for time and show "Anytime"; the Patrol tab offers an untimed route's rounds of the day; the report's route picker reads every page. Routes search returns to page 1.
+
+## 4.142.0
+
+### Minor Changes
+
+- 38e504d: Every app now reports browser crashes, failed API calls and successful saves to the Seven365 Logger through API-core (POST /api/client-logs), filed under the app's own folder.
+
 ## 4.141.0
 
 ### Minor Changes
@@ -3401,11 +3413,11 @@ Monthly report: a month that had patrols opens the list of that month's patrol r
   `utils/console-tier.ts` + `composables/useConsoleTier.ts` mirror the server's
   own rule from two endpoints the console already calls, unprojected:
 
-                                                                                                                                                                                                                                                                                                                                                                                                                                                                                          GET /api/members/user/:user/app/admin    the Seven365 staff membership
-                                                                                                                                                                                                                                                                                                                                                                                                                                                                                          GET /api/roles/id/:role                  that membership's role document
+                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              GET /api/members/user/:user/app/admin    the Seven365 staff membership
+                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              GET /api/roles/id/:role                  that membership's role document
 
-                                                                                                                                                                                                                                                                                                                                                                                                                                                                                          owner  = member.type === "admin" && role.type === "admin" && role.default === true
-                                                                                                                                                                                                                                                                                                                                                                                                                                                                                          staff  = member.type === "admin" && role.type === "admin"
+                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              owner  = member.type === "admin" && role.type === "admin" && role.default === true
+                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              staff  = member.type === "admin" && role.type === "admin"
 
   `role.default` is the marker because it is the only property of a platform
   staff role no API caller can set - `role.controller.ts` validates create and

package/components/Nfc/NFCPatrolRouteForm.vue

@@ -33,17 +33,21 @@
             <label class="text-caption screen-hint mb-1 d-block">
               Checkpoints <span class="text-error">*</span>
             </label>
-            <v-select
-              v-model="form.checkPointNumber"
-              :items="checkpointOptions"
+            <!-- Any number of checkpoints (it was a 1-10 list). -->
+            <v-text-field
+              :model-value="form.checkPointNumber"
+              type="number"
+              min="1"
+              inputmode="numeric"
               variant="outlined"
               density="comfortable"
-              hide-details
-              @update:model-value="updateCheckpointsCount"
+              :rules="[rules.checkpointCount]"
+              hide-details="auto"
+              @update:model-value="onCheckpointCountInput"
             />
           </v-col>
 
-          <v-col cols="12" md="4" class="mb-4 px-md-1">
+          <v-col v-if="form.timed" cols="12" md="4" class="mb-4 px-md-1">
             <label class="text-caption screen-hint mb-1 d-block">
               Tolerance (mins) <span class="text-error">*</span>
             </label>
@@ -74,11 +78,46 @@
               density="comfortable"
               hide-details
               chips
+            >
+              <template #prepend-item>
+                <v-list-item
+                  :title="allDaysSelected ? 'Unselect all' : 'Select all'"
+                  @click="toggleAllDays"
+                >
+                  <template #prepend>
+                    <v-checkbox-btn
+                      :model-value="allDaysSelected"
+                      :indeterminate="someDaysSelected"
+                      @click.stop="toggleAllDays"
+                    />
+                  </template>
+                </v-list-item>
+                <v-divider class="mt-1" />
+              </template>
+            </v-select>
+          </v-col>
+
+          <!-- Route timing. Off: no start times or tolerance, and the route can be
+               patrolled at any time of the day. -->
+          <v-col cols="12" class="mb-4">
+            <v-switch
+              v-model="form.timed"
+              color="primary"
+              inset
+              hide-details
+              :label="form.timed ? 'Route timing: On' : 'Route timing: Off'"
             />
+            <div class="text-caption screen-hint">
+              {{
+                form.timed
+                  ? "Patrols start at the route's start times, within the tolerance."
+                  : "No time window: this route can be patrolled at any time."
+              }}
+            </div>
           </v-col>
 
           <!-- Start Times -->
-          <v-col cols="12" class="mb-4">
+          <v-col v-if="form.timed" cols="12" class="mb-4">
             <v-card variant="outlined" class="pa-4" rounded="lg">
               <div class="d-flex align-center justify-space-between mb-3">
                 <span class="text-h6 font-weight-medium">Start Times</span>
@@ -264,6 +303,8 @@
 <script setup lang="ts">
 import useNFCPatrolRoute from "../../composables/useNFCPatrolRoute";
 import useNFCPatrolTag from "../../composables/useNFCPatrolTag";
+import { collectAllPages } from "../../utils/nfc-patrol-report";
+import { allDaysToggled, checkpointCountError } from "../../utils/nfc-patrol-schedule";
 
 const props = defineProps({
   title: {
@@ -329,11 +370,12 @@ const form = ref({
   startTimes: ["08:00"] as string[],
   endTimes: ["09:00"] as string[],
   checkPoints: [] as Checkpoint[],
+  // Route timing; false = no time window (patrol at any time).
+  timed: true,
 });
 
 const startTimeKeys = ref<number[]>([Date.now()]);
 
-const checkpointOptions = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10];
 const dayOptions = [
   { title: "Monday", value: 1 },
   { title: "Tuesday", value: 2 },
@@ -344,10 +386,29 @@ const dayOptions = [
   { title: "Sunday", value: 0 },
 ];
 
+const allDaysSelected = computed(() => form.value.days.length === dayOptions.length);
+const someDaysSelected = computed(
+  () => form.value.days.length > 0 && !allDaysSelected.value,
+);
+
+function toggleAllDays() {
+  form.value.days = allDaysToggled(
+    form.value.days,
+    dayOptions.map((day) => day.value),
+  );
+}
+
+function onCheckpointCountInput(value: string | number) {
+  const count = Number(value);
+  form.value.checkPointNumber = count;
+  if (Number.isInteger(count) && count >= 1) updateCheckpointsCount(count);
+}
+
 const MAX_TOLERANCE_MINUTES = 60;
 
 const rules = {
   required: (v: any) => !!v || "This field is required",
+  checkpointCount: (v: any) => checkpointCountError(v) ?? true,
   tolerance: (v: any) => {
     if (!/^\d+$/.test(String(v ?? ""))) return "Numbers only";
     if (Number(v) > MAX_TOLERANCE_MINUTES)
@@ -374,8 +435,11 @@ const { getAll: getAllNFCTags } = useNFCPatrolTag();
 
 async function fetchNFCTags() {
   try {
-    const response = await getAllNFCTags({ site: props.site, limit: 100 });
-    nfcTags.value = response.items || [];
+    // Every page: a site can have any number of tags (it read the first 100).
+    nfcTags.value = await collectAllPages(
+      (page) => getAllNFCTags({ site: props.site, page, limit: 100 }) as any,
+      1000,
+    );
   } catch (error) {
     console.error("Error fetching NFC tags:", error);
   }
@@ -554,6 +618,7 @@ async function initializeEditForm(newRoute: any) {
   form.value.startTimes = startTimes;
   form.value.endTimes = endTimes;
   form.value.checkPoints = existingCheckPoints;
+  form.value.timed = newRoute.timed !== false;
 
   // Reinitialize keys for each start time
   startTimeKeys.value = startTimes.map(() => Date.now() + Math.random());
@@ -598,14 +663,17 @@ async function submit() {
       ),
     ];
 
+    // Timing off keeps the tolerance as it was (the server requires one) but
+    // sends no start times: the route has no schedule.
     const payload = {
       name: form.value.name,
-      checkPointNumber: form.value.checkPointNumber,
-      tolerance: form.value.tolerance,
+      checkPointNumber: Number(form.value.checkPointNumber),
+      tolerance: Number(form.value.tolerance) || 0,
       days: form.value.days,
-      startTimes: uniqueStartTimes,
+      startTimes: form.value.timed ? uniqueStartTimes : [],
       checkPoints: form.value.checkPoints,
       site: props.site,
+      timed: form.value.timed,
     };
 
     if (props.mode === "create") {

package/components/Nfc/NFCPatrolRouteMain.vue

@@ -50,14 +50,15 @@
 
         <template #item.tolerance="{ item }">
           <div class="text-center">
-            {{ item.tolerance ? `${item.tolerance} mins` : "--" }}
+            {{ item.timed && item.tolerance ? `${item.tolerance} mins` : "--" }}
           </div>
         </template>
 
         <!-- Earliest first, one unbreakable pill per window; past three the
              rest fold into "+N more" so a busy route keeps a normal row. -->
-        <template #item.schedules="{ value }">
-          <NFCRouteScheduleTimes :windows="value || []" :limit="3" />
+        <template #item.schedules="{ item, value }">
+          <span v-if="!item.timed">Anytime (timing off)</span>
+          <NFCRouteScheduleTimes v-else :windows="value || []" :limit="3" />
         </template>
 
         <template #item.repeatDays="{ value }">
@@ -132,7 +133,12 @@
               </span>
             </v-col>
 
-            <v-col cols="12" class="mb-2" v-if="selectedRoute.tolerance">
+            <v-col cols="12" class="mb-2" v-if="selectedRoute.timed === false">
+              Security Check Schedule:
+              <span class="font-weight-bold">Anytime (timing off)</span>
+            </v-col>
+
+            <v-col cols="12" class="mb-2" v-if="selectedRoute.timed !== false && selectedRoute.tolerance">
               Tolerance:
               <span class="font-weight-bold">
                 {{ selectedRoute.tolerance }} mins
@@ -149,7 +155,7 @@
             <v-col
               cols="12"
               class="mb-2"
-              v-if="selectedRoute.schedules && selectedRoute.schedules.length"
+              v-if="selectedRoute.timed !== false && selectedRoute.schedules && selectedRoute.schedules.length"
             >
               <div class="mb-2">
                 Security Check Schedule
@@ -376,6 +382,7 @@ watchEffect(() => {
       days: item.days,
       startTimes: item.startTimes || [],
       checkPoints: item.checkPoints || [],
+      timed: item.timed !== false,
       // Display fields for the table
       routeName: item.name,
       numberOfCheckpoints: item.checkPointNumber,
@@ -392,6 +399,11 @@ watchEffect(() => {
   }
 });
 
+// A new search starts on page 1 (from page 2 or later it could show nothing).
+watch(searchInput, () => {
+  page.value = 1;
+});
+
 const dialogAdd = ref(false);
 const dialogEdit = ref(false);
 const dialogPreview = ref(false);

package/components/Nfc/PatrolReport/SummaryReportTable.vue

@@ -108,6 +108,7 @@ function skipped(row: NFCPatrolReportSummaryRow) {
 }
 
 function statusClass(status: NFCPatrolReportSummaryStatus) {
+  if (status === "In progress") return "text-warning";
   return status === "Completed" ? "text-success" : "text-error";
 }
 </script>

package/composables/useNFCPatrolReportFilters.ts

@@ -3,6 +3,8 @@ import type {
   NFCPatrolReportFilters,
 } from "../types/nfc-patrol-report";
 import useNFCPatrolRoute from "./useNFCPatrolRoute";
+import useNFCPatrolLog from "./useNFCPatrolLog";
+import { collectAllPages } from "../utils/nfc-patrol-report";
 
 function getTodayDate() {
   const now = new Date();
@@ -13,6 +15,7 @@ function getTodayDate() {
 }
 export function useNFCPatrolReportFilters(site: string) {
   const { getAll: getPatrolRoutes } = useNFCPatrolRoute();
+  const { getAll: getPatrolLogs } = useNFCPatrolLog();
 
   const filters = reactive<NFCPatrolReportFilters>({
     route: "",
@@ -34,17 +37,25 @@ export function useNFCPatrolReportFilters(site: string) {
       // Deleted routes too, labelled, so their past patrols can still be
       // reported on. An API from before route deletion refuses the parameter
       // with a 400: ask again without it, and the list is what it was.
-      const response = await getPatrolRoutes({
-        site,
-        page: 1,
-        limit: 100,
-        includeDeleted: true,
-      }).catch((error: any) => {
-        const status = Number(error?.statusCode ?? error?.response?.status ?? 0);
-        if (status !== 400) throw error;
-        return getPatrolRoutes({ site, page: 1, limit: 100 });
-      });
-      const items: Array<Record<string, any>> = response.items ?? [];
+      // Every page: a site may have more than 100 routes (deleted ones count).
+      const allRoutes = (includeDeleted: boolean) =>
+        collectAllPages(
+          (page) =>
+            getPatrolRoutes({
+              site,
+              page,
+              limit: 100,
+              ...(includeDeleted ? { includeDeleted: true } : {}),
+            }) as any,
+          1000,
+        );
+      const items: Array<Record<string, any>> = await allRoutes(true).catch(
+        (error: any) => {
+          const status = Number(error?.statusCode ?? error?.response?.status ?? 0);
+          if (status !== 400) throw error;
+          return allRoutes(false);
+        },
+      );
       const isDeleted = (route: Record<string, any>) => route.status === "Deleted";
       // Routes still in use first, so the report opens on one of them.
       const routes = [
@@ -54,6 +65,7 @@ export function useNFCPatrolReportFilters(site: string) {
         title: isDeleted(route) ? `${route.name} (deleted)` : route.name,
         value: route._id,
         startTimes: route.startTimes ?? [],
+        timed: route.timed !== false,
       }));
 
       options.value.routes = routes;
@@ -62,22 +74,50 @@ export function useNFCPatrolReportFilters(site: string) {
         filters.route = routes[0]?.value ?? "";
       }
 
-      updateStartTimes();
+      await updateStartTimes();
     } finally {
       loading.value = false;
     }
   }
 
-  function updateStartTimes() {
+  let startTimesRequest = 0;
+  async function updateStartTimes() {
     const route = options.value.routes.find(({ value }) => value === filters.route);
-    options.value.timeRanges = route?.startTimes ?? [];
+    const request = ++startTimesRequest;
+
+    // An untimed route has no start times: its rounds start whenever the guard
+    // starts them, so offer the start times of that day's rounds instead.
+    let times: string[] = route?.startTimes ?? [];
+    if (route && route.timed === false && filters.date) {
+      try {
+        const logs = await collectAllPages(
+          (page) =>
+            getPatrolLogs({
+              page,
+              limit: 100,
+              site,
+              date: filters.date,
+              routeId: route.value,
+            }) as any,
+          1000,
+        );
+        times = [...new Set(logs.map((log) => String(log.route?.startTime ?? "")))]
+          .filter(Boolean)
+          .sort();
+      } catch {
+        times = [];
+      }
+      if (request !== startTimesRequest) return;
+    }
+
+    options.value.timeRanges = times;
 
     if (!options.value.timeRanges.includes(filters.timeRange)) {
       filters.timeRange = options.value.timeRanges[0] ?? "";
     }
   }
 
-  watch(() => filters.route, updateStartTimes);
+  watch(() => [filters.route, filters.date], () => void updateStartTimes());
 
   return {
     filters,

package/composables/useNFCPatrolRoute.ts

@@ -32,11 +32,12 @@ export default function useNFCPatrolRoute() {
   function add(value: {
     name: string;
     checkPointNumber: number;
-    tolerance: string;
+    tolerance: string | number;
     days: number[];
     startTimes: string[];
     checkPoints: Array<{ nfcTag_id: string }>;
     site: string;
+    timed?: boolean;
   }) {
     return useNuxtApp().$api<Record<string, any>>("/api/nfc-patrol-route", {
       method: "POST",
@@ -61,11 +62,12 @@ export default function useNFCPatrolRoute() {
     value: {
       name: string;
       checkPointNumber: number;
-      tolerance: string;
+      tolerance: string | number;
       days: number[];
       startTimes: string[];
       checkPoints: Array<{ nfcTag_id: string }>;
       site: string;
+      timed?: boolean;
     }
   ) {
     return useNuxtApp().$api(`/api/nfc-patrol-route/id/${id}`, {

package/nuxt.config.ts

@@ -17,6 +17,10 @@ export default defineNuxtConfig({
     ...(process.env.API_CORE
       ? { "/api/storage/**": { proxy: `${process.env.API_CORE}/api/files/object/**` } }
       : {}),
+    // Browser logs (utils/client-log.ts), forwarded by API-core to the Seven365 Logger.
+    ...(process.env.API_CORE
+      ? { "/api/client-logs": { proxy: `${process.env.API_CORE}/api/client-logs` } }
+      : {}),
   },
   build: {
     transpile: ["vuetify"],

package/package.json

@@ -2,7 +2,7 @@
   "name": "@7365admin1/layer-common",
   "license": "MIT",
   "type": "module",
-  "version": "4.141.0",
+  "version": "4.143.0",
   "author": "7365admin1",
   "main": "./nuxt.config.ts",
   "//files": "What a consumer extending this layer actually loads. Without this npm ships the whole working tree - the changesets, the CI workflows, the render harness in tools/ and any scratch directory that happened to exist at publish time. Nuxt resolves a layer by directory, so every runtime directory below has to stay listed; adding a new top-level runtime directory means adding it here too.",

package/plugins/API.ts

@@ -1,5 +1,9 @@
 import { isPermissionRefusal } from "../utils/permission-refusal";
 import { accessSuspendedMessage, SIGNIN_NOTICE_COOKIE } from "../utils/access-suspended";
+import { apiPath, clientLog } from "../utils/client-log";
+
+// Expected answers, not failures: the session lapsed, no permission, nothing there.
+const UNLOGGED_REFUSALS = new Set([401, 403, 404]);
 
 export default defineNuxtPlugin((nuxtApp) => {
   const { cookieConfig } = useRuntimeConfig().public;
@@ -33,7 +37,33 @@ export default defineNuxtPlugin((nuxtApp) => {
     // Both refusal shapes count: API-core's 403 + PERMISSION_DENIED and core's
     // 401 + "You do not have permission ..." (vehicle writes and every other
     // `requireSiteReach` gate). See utils/permission-refusal.ts.
-    onResponseError({ response }) {
+    // Saves that went through, so a client's "I clicked save" can be checked.
+    onResponse({ request, options, response }) {
+      const method = String(options.method ?? "GET").toUpperCase();
+      if (method === "GET" || !response.ok) return;
+      clientLog("info", `Saved: ${method} ${apiPath(String(request))}`, { status: response.status });
+    },
+    // Offline or the API unreachable: the click did nothing for the client.
+    onRequestError({ request, options, error }) {
+      const method = String(options.method ?? "GET").toUpperCase();
+      clientLog("error", `Request failed: ${method} ${apiPath(String(request))}: ${error?.message ?? "network error"}`, {
+        method,
+        route: apiPath(String(request)),
+        status: 0,
+        expected: !navigator.onLine || undefined,
+      });
+    },
+    onResponseError({ request, options, response }) {
+      const status = response?.status ?? 0;
+      if (!UNLOGGED_REFUSALS.has(status)) {
+        const method = String(options.method ?? "GET").toUpperCase();
+        const reason = (response as any)?._data?.message;
+        clientLog(status >= 500 ? "error" : "warn", `${method} ${apiPath(String(request))} answered ${status}${reason ? `: ${reason}` : ""}`, {
+          method,
+          route: apiPath(String(request)),
+          status,
+        });
+      }
       // Every membership suspended (2026-10-01): the server already ended the
       // session. Sign out here and let the sign-in page say why, once.
       const suspended = accessSuspendedMessage((response as any)?._data);

package/plugins/client-logs.client.ts

@@ -0,0 +1,47 @@
+import { clientLog, configureClientLog, flushClientLogs } from "../utils/client-log";
+
+// Crashes anywhere on the page go to the Seven365 Logger. Failed and successful
+// API calls are logged from plugins/API.ts, where every request passes.
+export default defineNuxtPlugin((nuxtApp) => {
+  configureClientLog(String(useRuntimeConfig().public.APP ?? ""));
+
+  const describe = (error: unknown) => {
+    const e = error as { message?: string; stack?: string; name?: string } | undefined;
+    return { message: e?.message || String(error), stack: e?.stack, name: e?.name };
+  };
+
+  // An old tab asking for a file the last deploy replaced: the page needs a
+  // reload, the code is not broken. Logged, never ticketed.
+  const isStaleBuild = (message: string) =>
+    /Failed to fetch dynamically imported module|Importing a module script failed|error loading dynamically imported module/i.test(message);
+
+  nuxtApp.hook("vue:error", (error, _instance, info) => {
+    const { message, stack } = describe(error);
+    clientLog("error", message, { stack, source: `vue:${info}`, expected: isStaleBuild(message) || undefined });
+  });
+
+  nuxtApp.hook("app:error", (error) => {
+    const { message, stack } = describe(error);
+    clientLog("error", message, { stack, source: "app", expected: isStaleBuild(message) || undefined });
+  });
+
+  window.addEventListener("error", (event) => {
+    if (!event.message) return;
+    clientLog("error", event.message, {
+      stack: (event.error as Error | undefined)?.stack,
+      source: `${event.filename}:${event.lineno}:${event.colno}`,
+      expected: isStaleBuild(event.message) || undefined,
+    });
+  });
+
+  window.addEventListener("unhandledrejection", (event) => {
+    const { message, stack, name } = describe(event.reason);
+    // Failed $api calls are already logged with their status by plugins/API.ts.
+    if (name === "FetchError") return;
+    clientLog("error", message, { stack, source: "promise", expected: isStaleBuild(message) || undefined });
+  });
+
+  document.addEventListener("visibilitychange", () => {
+    if (document.visibilityState === "hidden") void flushClientLogs();
+  });
+});

package/types/nfc-patrol-report.ts

@@ -1,6 +1,6 @@
 export type NFCPatrolReportTab = "patrol" | "daily" | "monthly";
 
-export type NFCPatrolActivityStatus = "Completed" | "Skipped";
+export type NFCPatrolActivityStatus = "Completed" | "Skipped" | "Pending";
 
 export interface NFCPatrolReportFilters {
   route: string;
@@ -17,6 +17,8 @@ export interface NFCPatrolReportRouteOption {
   title: string;
   value: string;
   startTimes: string[];
+  /** false = route timing switched off (rounds start any time). */
+  timed?: boolean;
 }
 
 export interface NFCPatrolReportFilterOptions {
@@ -69,7 +71,7 @@ export interface NFCPatrolReport {
   activities: NFCPatrolActivity[];
 }
 
-export type NFCPatrolReportSummaryStatus = "Completed" | "Incomplete";
+export type NFCPatrolReportSummaryStatus = "Completed" | "Incomplete" | "In progress";
 
 export interface NFCPatrolReportSummaryRow {
   id: string;

package/utils/client-log.ts

@@ -0,0 +1,71 @@
+// Browser logs for the Seven365 Logger. Entries are batched and sent to API-core
+// (POST /api/client-logs), which files them under this app's own folder and adds
+// the signed-in user from the session. Nothing is sent in development.
+type Level = "info" | "warn" | "error";
+
+type Entry = { level: Level; message: string; context: Record<string, unknown>; timestamp: string };
+
+const queue: Entry[] = [];
+const recent = new Map<string, number>();
+const sessionId = Math.random().toString(36).slice(2, 12);
+let app = "";
+let timer: ReturnType<typeof setTimeout> | null = null;
+
+const BATCH = 20;
+const FLUSH_MS = 5000;
+const REPEAT_WINDOW_MS = 10000;
+
+export function configureClientLog(appName: string) {
+  app = appName;
+}
+
+export function apiPath(url: string) {
+  return String(url)
+    .split("?")[0]
+    .replace(/\b[0-9a-f]{24}\b/gi, ":id")
+    .replace(/\/\d+(?=\/|$)/g, "/:n");
+}
+
+function sid() {
+  const match = document.cookie.match(/(?:^|;\s*)sid=([^;]+)/);
+  return match ? decodeURIComponent(match[1]) : "";
+}
+
+export function clientLog(level: Level, message: string, context: Record<string, unknown> = {}) {
+  if (!import.meta.client || import.meta.dev || !app || !message) return;
+
+  // The same line from the same tab within 10s is one event (a re-rendering
+  // component can throw the same error dozens of times a second).
+  const key = `${level}|${message}`;
+  const now = Date.now();
+  if ((recent.get(key) ?? 0) > now - REPEAT_WINDOW_MS) return;
+  if (recent.size > 200) recent.clear();
+  recent.set(key, now);
+
+  queue.push({
+    level,
+    message: String(message).slice(0, 2000),
+    context: { ...context, page: location.pathname, sessionId },
+    timestamp: new Date().toISOString(),
+  });
+
+  if (queue.length >= BATCH) void flushClientLogs();
+  else if (!timer) timer = setTimeout(() => void flushClientLogs(), FLUSH_MS);
+}
+
+export async function flushClientLogs() {
+  if (timer) clearTimeout(timer);
+  timer = null;
+  const logs = queue.splice(0, 50);
+  if (!logs.length) return;
+  try {
+    await fetch("/api/client-logs", {
+      method: "POST",
+      keepalive: true,
+      headers: { "Content-Type": "application/json", Authorization: sid() },
+      body: JSON.stringify({ app, logs }),
+    });
+  } catch {
+    // Logging must never break the page; a lost batch is acceptable.
+  }
+}

package/utils/nfc-patrol-report.ts

@@ -78,6 +78,30 @@ export async function collectAllPages(
   return items;
 }
 
+/** A round of a route whose timing is switched off: no window, never late. */
+export function isUntimedLog(log: Record<string, any>) {
+  return log?.route?.timed === false;
+}
+
+/**
+ * Was this checkpoint missed? A skip always is. A checkpoint still Pending
+ * once its timed round's window has closed is too: nothing on the server ever
+ * moves it off Pending, so a round the guard walked away from used to read
+ * "Completed, 0 missed". A round of an untimed route has no window and is
+ * never missed for time.
+ */
+export function isMissedCheckpoint(
+  log: Record<string, any>,
+  checkpoint: Record<string, any>,
+  now: Date = new Date(),
+) {
+  if (checkpoint?.status === "Skipped") return true;
+  if (checkpoint?.status === "Completed") return false;
+  if (isUntimedLog(log)) return false;
+  const end = new Date(log?.endDateTime).getTime();
+  return Number.isFinite(end) && now.getTime() > end;
+}
+
 /** The month (0-11) a patrol log counts under; NaN for a date that is not one. */
 export function monthIndexOf(log: Record<string, any>) {
   return new Date(log?.date).getMonth();
@@ -91,6 +115,7 @@ export function monthIndexOf(log: Record<string, any>) {
  */
 export function mapLogsToMonthlyRows(
   logs: Record<string, any>[],
+  now: Date = new Date(),
 ): TPatrolMonthRow[] {
   const rows: TPatrolMonthRow[] = MONTHS.map((month) => ({
     month,
@@ -106,7 +131,7 @@ export function mapLogsToMonthlyRows(
 
     for (const checkpoint of log?.checkPoints ?? []) {
       if (checkpoint?.status === "Completed") rows[monthIndex].checked++;
-      else if (checkpoint?.status === "Skipped") rows[monthIndex].missed++;
+      else if (isMissedCheckpoint(log, checkpoint, now)) rows[monthIndex].missed++;
     }
   }
 
@@ -168,15 +193,20 @@ export function mapLogToPatrolRun(
     };
   });
 
+  const untimed = isUntimedLog(log);
+
   return {
     routeSummary: {
       routeName: log.route?.name,
-      tolerance: Number(log.route?.tolerance) || 0,
-      schedule: {
-        start: log.route?.startTime,
-        end: log.route?.endTime,
-        totalTime: diffMinutes(log.startDateTime, log.endDateTime),
-      },
+      tolerance: untimed ? 0 : Number(log.route?.tolerance) || 0,
+      // An untimed round has no schedule to compare against.
+      schedule: untimed
+        ? { start: "Anytime", end: "-", totalTime: "-" }
+        : {
+            start: log.route?.startTime,
+            end: log.route?.endTime,
+            totalTime: diffMinutes(log.startDateTime, log.endDateTime),
+          },
       actual: {
         start: clockTime(actual.start, timezone),
         end: clockTime(actual.end, timezone),
@@ -196,21 +226,28 @@ export function mapLogToSummaryRow(
   log: Record<string, any>,
   index: number,
   siteTimezone?: string,
+  now: Date = new Date(),
 ): NFCPatrolReportSummaryRow {
   const checkpoints: Record<string, any>[] = log.checkPoints ?? [];
   const checked = checkpoints.filter((cp) => cp.status === "Completed").length;
-  const missed = checkpoints.filter((cp) => cp.status === "Skipped").length;
+  const missed = checkpoints.filter((cp) => isMissedCheckpoint(log, cp, now)).length;
+  const allDone =
+    checkpoints.length > 0 &&
+    checkpoints.every((cp) => cp.status === "Completed" || cp.status === "Skipped");
+  const status = missed > 0 ? "Incomplete" : allDone ? "Completed" : "In progress";
 
   return {
     id: log._id,
     date: log.date,
     routeName: `${index + 1}. ${log.route?.name ?? "-"}`,
-    securityCheckSchedule: log.route?.startTime ?? "-",
+    securityCheckSchedule: isUntimedLog(log)
+      ? `Anytime (started ${log.route?.startTime ?? "-"})`
+      : log.route?.startTime ?? "-",
     guardName: log.sign?.guardName,
     guardSignatureFileId: log.sign?.guardSignatureFileId,
     checked,
     missed,
-    status: missed === 0 ? "Completed" : "Incomplete",
+    status,
     detail: mapLogToPatrolRun(log, siteTimezone),
   };
 }

package/utils/nfc-patrol-schedule.ts

@@ -71,3 +71,23 @@ export function formatRouteDays(days: number[] = []) {
   }
   return parts.join(", ");
 }
+
+/**
+ * The route form's "Select all" for days: every day when not all are
+ * selected yet, none when they all are.
+ */
+export function allDaysToggled(selected: number[], allDays: number[]): number[] {
+  const everyDay = allDays.every((day) => selected.includes(day));
+  return everyDay ? [] : [...allDays];
+}
+
+/**
+ * Why a checkpoint count is not usable, or null when it is. Any whole number
+ * from 1 up: a route takes as many NFC tags as the site has (no cap).
+ */
+export function checkpointCountError(value: unknown): string | null {
+  const count = Number(value);
+  if (value === "" || value === null || value === undefined) return "This field is required";
+  if (!Number.isInteger(count) || count < 1) return "Enter a whole number of 1 or more";
+  return null;
+}