@7365admin1/layer-common 4.1.3-staging.252 → 4.1.3-staging.253

package/components/HidAccessLogDashboard.vue

@@ -89,15 +89,9 @@
 
       <!-- Staging's hover-the-full-name tooltip, kept. -->
       <template #[`item.name`]="{ item }">
+        <!-- Client requirement (2026-08-26): no face photo on an access log row.
+             The reader keeps the image; we neither fetch nor hold one. -->
         <span class="hid-log-name app-cell--strong">
-          <v-avatar
-            v-if="item.faceImage"
-            size="34"
-            rounded="lg"
-            class="hid-face-avatar"
-          >
-            <v-img :src="item.faceImage" cover />
-          </v-avatar>
           <span>{{ item.name }}</span>
           <v-tooltip
             v-if="item.name && item.name !== 'N/A'"
@@ -314,7 +308,6 @@ type AccessRow = {
   name: string;
   hidUserId: string;
   facialData: string;
-  faceImage?: string;
   unit: string;
   method: string;
   lastAccessTime: string;
@@ -371,7 +364,6 @@ type JsPdf = {
 const {
   getReaders,
   getLogs,
-  getUserImage,
   getReaderUsers,
   getReaderAccessLogs,
 } = useHidAmico();
@@ -384,7 +376,6 @@ const visitorRecords = ref<Record<string, HidAccessRecord>>({});
 const logs = ref<HidAccessRecord[]>([]);
 const liveAccessLogs = ref<HidAccessRecord[]>([]);
 const administratorUserIds = ref<Set<number>>(new Set());
-const userImages = ref<Record<string, string>>({});
 const selectedReaderId = ref("");
 const activeTab = ref<AccessTab>("all");
 const search = ref("");
@@ -470,7 +461,6 @@ const selectedHistoryRows = computed<AccessRow[]>(() => {
     name: selectedHistoryRow.value?.name || "N/A",
     hidUserId: selectedHistoryRow.value?.hidUserId || "N/A",
     facialData: selectedHistoryRow.value?.facialData || "N/A",
-    faceImage: selectedHistoryRow.value?.faceImage,
     unit: selectedHistoryRow.value?.unit || "N/A",
     method: getAccessMethod({}, log),
     lastAccessTime: formatDate(getLogAccessDateValue(log)),
@@ -486,7 +476,6 @@ const selectedHistoryRows = computed<AccessRow[]>(() => {
     name: selectedHistoryRow.value?.name || "N/A",
     hidUserId: selectedHistoryRow.value?.hidUserId || "N/A",
     facialData: selectedHistoryRow.value?.facialData || "N/A",
-    faceImage: selectedHistoryRow.value?.faceImage,
     unit: selectedHistoryRow.value?.unit || "N/A",
     method: getAccessMethod({}, { payload: { rawAccessLog: log } }),
     lastAccessTime: formatDate(getAccessLogTime(log, {})),
@@ -700,14 +689,6 @@ async function loadDashboard() {
     serverPageRange.value = String(
       hidAccessLogResponse?.pageRange ?? hidAccessLogResponse?.data?.pageRange ?? "",
     );
-    const accessLogUserIds = getAccessLogUserIds(liveItems);
-    const accessLogImageCandidates = [...accessLogUserIds].map((hidUserId) => ({
-      hidUserId: String(hidUserId),
-    }));
-    await loadUserImages(
-      [...readerUsers, ...accessLogImageCandidates],
-      accessLogUserIds,
-    );
   } finally {
     if (sequence === loadSequence.value) {
       loading.value = false;
@@ -724,47 +705,8 @@ async function loadDashboardSource<T>(request: Promise<T>, fallback: T, label: s
   }
 }
 
-async function loadUserImages(items: HidAccessRecord[], forceUserIds = new Set<number>()) {
-  if (!selectedReaderId.value) return;
-
-  await Promise.all(items.map(async (identity) => {
-    const hidUserId = toHidNumericId(identity.hidUserId);
-    const key = String(hidUserId || "");
-    if (
-      !hidUserId ||
-      key in userImages.value ||
-      (!forceUserIds.has(hidUserId) && !hasHidImageHint(identity))
-    ) return;
-
-    try {
-      const response = await getUserImage(selectedReaderId.value, hidUserId);
-      const image = response?.data || response;
-      userImages.value[key] = image?.base64
-        ? `data:${image.contentType || "image/jpeg"};base64,${image.base64}`
-        : "";
-    } catch {
-      userImages.value[key] = "";
-    }
-  }));
-}
 
-function getAccessLogUserIds(items: HidAccessRecord[]) {
-  return new Set(
-    items
-      .map((item) => toHidNumericId(item.user_id ?? item.userId ?? item.hidUserId))
-      .filter((userId): userId is number => userId !== undefined),
-  );
-}
 
-function hasHidImageHint(identity: HidAccessRecord) {
-  return Boolean(
-    identity?.metadata?.imageTimestamp ||
-      identity?.metadata?.facialData ||
-      identity?.imageTimestamp ||
-      identity?.image_timestamp ||
-      identity?.facialData,
-  );
-}
 
 async function onTabChange(tab: AccessTab) {
   if (tab) {
@@ -783,7 +725,6 @@ function resetDashboardData() {
   visitorRecords.value = {};
   logs.value = [];
   liveAccessLogs.value = [];
-  userImages.value = {};
   administratorUserIds.value = new Set();
   serverPages.value = 1;
   serverPageRange.value = "";
@@ -1128,7 +1069,6 @@ function mapAccessLogToRow(rawLog: HidAccessRecord, event: HidAccessRecord, inde
     name: getRawLogName(rawLog, event, identity),
     hidUserId: formatHidUid(rawLog.user_id ?? rawLog.userId ?? rawLog.hidUserId ?? identity?.hidUserId),
     facialData: identity ? getFacialData(identity) : "N/A",
-    faceImage: logHidUserId ? userImages.value[String(logHidUserId)] || "" : "",
     unit: identity ? getUnit(identity) : "N/A",
     method: getAccessMethod(identity || {}, { ...event, payload: { ...event.payload, rawAccessLog: rawLog } }),
     lastAccessTime: formatDate(rawAccessTime),
@@ -1773,10 +1713,6 @@ function escapeHtml(value: unknown) {
   border-radius: 0;
 }
 
-.hid-face-avatar {
-  border: 1px solid var(--border);
-  background: var(--hover);
-}
 
 .hid-log-name {
   align-items: center;

package/components/HidReaderUserRoster.vue

@@ -56,16 +56,10 @@
         <span class="app-cell--num">{{ item.registration || "N/A" }}</span>
       </template>
 
+      <!-- Client requirement (2026-08-26): the face photo is not shown once
+           registration is complete. Enrollment STATE only. -->
       <template #[`item.facialData`]="{ item }">
-        <v-avatar
-          v-if="getUserImageSrc(item)"
-          size="34"
-          rounded="lg"
-          class="hid-face-avatar"
-        >
-          <v-img :src="getUserImageSrc(item)" cover />
-        </v-avatar>
-        <span v-else :class="hasFacialData(item) ? 'facial-data--available' : 'app-cell--muted'">
+        <span :class="hasFacialData(item) ? 'facial-data--available' : 'app-cell--muted'">
           <v-icon
             :icon="hasFacialData(item) ? 'mdi-face-recognition' : 'mdi-face-man-shimmer-outline'"
             size="19"
@@ -208,14 +202,12 @@ type HidReaderUser = Record<string, unknown> & {
 const {
   getReaders,
   getReaderUsers,
-  getUserImage,
   getUserPinStatus,
   setUserPin,
   deleteUserPin,
 } = useHidAmico();
 const readers = ref<HidReader[]>([]);
 const users = ref<HidReaderUser[]>([]);
-const userImages = ref<Record<string, string>>({});
 const selectedReaderId = ref("");
 const search = ref("");
 const userType = ref<"all" | "user" | "visitor">("all");
@@ -328,7 +320,6 @@ async function loadUsers() {
       ? matchingItems.slice(pageStart, pageStart + 10)
       : matchingItems;
     users.value = pageItems.map(toReaderUser);
-    await loadUserImages(users.value);
     total.value = clientFiltersUserType
       ? matchingItems.length
       : toNumber(response.total ?? data.total, users.value.length);
@@ -356,7 +347,6 @@ function goToPage(nextPage: number) {
 
 function reloadFromFirstPage() {
   page.value = 1;
-  userImages.value = {};
   loadUsers();
 }
 
@@ -364,31 +354,6 @@ function hasFacialData(user: HidReaderUser) {
   return user.metadata?.facialEnrolled === true || Boolean(toText(user.metadata?.facialData));
 }
 
-async function loadUserImages(items: HidReaderUser[]) {
-  if (!selectedReaderId.value) return;
-
-  await Promise.all(items.map(async (user) => {
-    const hidUserId = toHidNumericId(user.hidUserId);
-    const key = String(hidUserId || "");
-    if (!hidUserId || key in userImages.value) return;
-
-    try {
-      const response = toRecord(await getUserImage(selectedReaderId.value, hidUserId));
-      const image = toRecord(response.data ?? response);
-      const base64 = toText(image.base64);
-      userImages.value[key] = base64
-        ? `data:${toText(image.contentType) || "image/jpeg"};base64,${base64}`
-        : "";
-    } catch {
-      userImages.value[key] = "";
-    }
-  }));
-}
-
-function getUserImageSrc(user: HidReaderUser) {
-  const hidUserId = toHidNumericId(user.hidUserId);
-  return (hidUserId ? userImages.value[String(hidUserId)] : "") || "";
-}
 
 async function openPinDialog(user: HidReaderUser) {
   const hidUserId = toHidNumericId(user.hidUserId);
@@ -583,9 +548,6 @@ function toNumber(value: unknown, fallback: number) {
   color: var(--warning, #9a6b00);
 }
 
-.hid-face-avatar {
-  border: 1px solid var(--border-color, #e0e3e8);
-}
 
 .facial-data--available {
   align-items: center;

package/components/HidUserEnrollment.vue

@@ -85,18 +85,11 @@
         <span class="app-cell--strong">{{ getName(item) }}</span>
       </template>
 
-      <!-- The face thumbnail, unchanged. It prints whatever `getUserImageSrc`
-           already returned; this screen does not read or export a template. -->
+      <!-- Client requirement (2026-08-26): the face photo is not shown once
+           registration is complete. The column reports enrollment STATE only --
+           the reader holds the image and the biometric template, we hold neither. -->
       <template #[`item.facialData`]="{ item }">
-        <v-avatar
-          v-if="getUserImageSrc(item)"
-          size="34"
-          rounded="lg"
-          class="hid-face-avatar"
-        >
-          <v-img :src="getUserImageSrc(item)" cover />
-        </v-avatar>
-        <span v-else class="app-cell--muted">{{ getFacialData(item) }}</span>
+        <span class="app-cell--muted">{{ getFacialData(item) }}</span>
       </template>
 
       <template #[`item.hidUserId`]="{ item }">
@@ -192,6 +185,12 @@
                   <span v-if="form.photoPreview" class="photo-edit-icon">
                     <v-icon icon="mdi-pencil" size="16" />
                   </span>
+                  <template v-else-if="facialEnrolled">
+                    <!-- Enrolled, but the photo is deliberately not shown or held.
+                         Re-enrolling needs a fresh capture. -->
+                    <v-icon icon="mdi-face-recognition" size="40" />
+                    <span>Face Enrolled</span>
+                  </template>
                   <template v-else>
                     <!-- Was `color="#a4acb5"`, the same grey on both themes.
                          `.photo-button` already sets `--muted`, so it inherits. -->
@@ -207,7 +206,7 @@
                   @click="triggerCameraInput"
                 />
                 <v-list-item
-                  v-if="form.photoPreview"
+                  v-if="form.photoPreview || facialEnrolled"
                   title="Remove Photo"
                   class="text-error"
                   @click="removePhoto"
@@ -538,7 +537,6 @@ const {
   getReaders,
   getIdentities,
   getReaderUsers,
-  getUserImage,
   setUserImage,
   deleteUserImage,
   getUserCards,
@@ -617,7 +615,6 @@ type HidIdentityPayload = Record<string, unknown> & {
 
 const readers = ref<HidReader[]>([]);
 const users = ref<HidUser[]>([]);
-const userImages = ref<Record<string, string>>({});
 const administratorUserIds = ref<Set<number>>(new Set());
 const selectedReaderId = ref("");
 const search = ref("");
@@ -650,6 +647,20 @@ const photoInput = ref<HTMLInputElement | null>(null);
 const cameraDialog = ref(false);
 const selectedPhotoFile = ref<File | null>(null);
 const removePhotoRequested = ref(false);
+/**
+ * Client requirement (2026-08-26): the photo is neither shown nor kept once registration
+ * completes, so the form tracks only WHETHER this person is enrolled. Re-enrolling needs a
+ * fresh capture -- there is no stored image to re-send.
+ */
+const facialEnrolled = ref(false);
+
+function hasFacialEnrollment(user?: HidUser | null) {
+  return Boolean(
+    user?.metadata?.facialEnrolled ||
+    user?.metadata?.facialData ||
+    user?.metadata?.imageTimestamp
+  );
+}
 const removePinRequested = ref(false);
 const snackbar = reactive({
   show: false,
@@ -832,8 +843,6 @@ async function loadUsers() {
     serverPageRange.value = String(responseRecord.pageRange ?? responseData.pageRange ?? "");
     if (props.cardManagement) {
       await loadVisibleUserCards(users.value);
-    } else {
-      await loadUserImages(users.value);
     }
   } catch (error) {
     console.error("Unable to load HID reader users:", error);
@@ -850,27 +859,6 @@ async function loadUsers() {
   }
 }
 
-async function loadUserImages(items: HidUser[]) {
-  if (!selectedReaderId.value) return;
-
-  await Promise.all(
-    items.map(async (user) => {
-      const hidUserId = toHidNumericId(user.hidUserId);
-      const key = String(hidUserId || "");
-      if (!hidUserId || userImages.value[key] || user?.metadata?.photo) return;
-
-      try {
-        const response = await getUserImage(selectedReaderId.value, hidUserId);
-        const image = response?.data || response;
-        userImages.value[key] = image?.base64
-          ? `data:${image.contentType || "image/jpeg"};base64,${image.base64}`
-          : "";
-      } catch {
-        userImages.value[key] = "";
-      }
-    })
-  );
-}
 
 function resetForm() {
   form.reader = selectedReaderId.value || readers.value[0]?._id || "";
@@ -888,6 +876,7 @@ function resetForm() {
   form.photoPreview = "";
   selectedPhotoFile.value = null;
   removePhotoRequested.value = false;
+  facialEnrolled.value = false;
   form.subjectCategory = "resident";
   form.subjectId = "";
 }
@@ -944,7 +933,8 @@ async function openEdit(user: HidUser) {
   form.accessPin = "";
   pinEnrolled.value = Boolean(user.metadata?.pinEnrolled);
   removePinRequested.value = false;
-  form.photoPreview = getUserImageSrc(user);
+  form.photoPreview = "";
+  facialEnrolled.value = hasFacialEnrollment(user);
   form.subjectCategory = user.person
     ? "resident"
     : user.serviceProvider
@@ -966,12 +956,6 @@ async function openEdit(user: HidUser) {
     // Preserve the last known status if the reader is temporarily offline.
   }
 
-  if (!form.photoPreview) {
-    await loadUserImages([user]);
-    if (selectedUser.value?._rowKey === user._rowKey) {
-      form.photoPreview = getUserImageSrc(user);
-    }
-  }
 }
 
 function openView(user: HidUser) {
@@ -1017,6 +1001,7 @@ function removePhoto() {
   form.photoPreview = "";
   selectedPhotoFile.value = null;
   removePhotoRequested.value = true;
+  facialEnrolled.value = false;
 }
 
 function onPhotoChange(event: Event) {
@@ -1094,7 +1079,12 @@ async function saveUser() {
         level: form.level,
         unit: form.unit,
         unitLabel: buildUnitLabel(),
-        facialData: form.photoPreview ? form.hidUserId : "",
+        // Enrollment STATE carries forward on its own. It used to be derived from the
+        // photo loaded into the form, which now stays empty -- editing a name would have
+        // silently un-enrolled the person.
+        facialEnrolled: hasFacialEnrollment(selectedUser.value),
+        facialEnrolledAt: selectedUser.value?.metadata?.facialEnrolledAt || "",
+        facialData: selectedUser.value?.metadata?.facialData || "",
         imageTimestamp: selectedUser.value?.metadata?.imageTimestamp || "",
         facialScores: selectedUser.value?.metadata?.facialScores || {},
         pinEnrolled: pinEnrolled.value,
@@ -1145,6 +1135,11 @@ async function saveUser() {
     showToast(getHidErrorMessage(error), "error");
   } finally {
     saving.value = false;
+    // Discard the captured image on every path. On a part-way failure this is what stops
+    // the photo lingering in the open form after a retry or a cancel.
+    form.photoPreview = "";
+    selectedPhotoFile.value = null;
+    removePhotoRequested.value = false;
   }
 }
 
@@ -1185,13 +1180,11 @@ async function syncFacialImage(
       }
     );
     const result = response?.data;
-    userImages.value[String(hidUserId)] = form.photoPreview;
     return { action: "enrolled", timestamp, scores: result?.scores };
   }
 
   if (removePhotoRequested.value) {
     await deleteUserImage(readerId, hidUserId);
-    delete userImages.value[String(hidUserId)];
     return { action: "removed" };
   }
 
@@ -1203,12 +1196,15 @@ function applyFacialResult(
   result: FacialSyncResult
 ) {
   if (result.action === "enrolled") {
+    metadata.facialEnrolled = true;
+    metadata.facialEnrolledAt = new Date().toISOString();
     metadata.facialData = form.hidUserId;
     metadata.imageTimestamp = String(result.timestamp || "");
     metadata.facialScores = result.scores || {};
   } else if (result.action === "removed") {
+    metadata.facialEnrolled = false;
+    metadata.facialEnrolledAt = "";
     metadata.facialData = "";
-    metadata.photo = "";
     metadata.imageTimestamp = "";
     metadata.facialScores = {};
   }
@@ -1675,16 +1671,7 @@ function getFacialData(user: HidUser) {
   return (
     user?.metadata?.facialData ||
     user?.metadata?.imageTimestamp ||
-    (user?.metadata?.photo ? user.hidUserId : "N/A")
-  );
-}
-
-function getUserImageSrc(user: HidUser) {
-  const hidUserId = toHidNumericId(user?.hidUserId);
-  return (
-    user?.metadata?.photo ||
-    (hidUserId ? userImages.value[String(hidUserId)] : "") ||
-    ""
+    (user?.metadata?.facialEnrolled ? user.hidUserId : "N/A")
   );
 }
 
@@ -2016,10 +2003,6 @@ function toHidCard(value: unknown): HidCard {
   box-shadow: var(--shadow-menu);
 }
 
-.hid-face-avatar {
-  border: 1px solid var(--border);
-  background: var(--hover);
-}
 
 .admin-rule-indicator {
   display: inline-flex;

package/composables/useHidAmico.ts

@@ -299,11 +299,6 @@ export default function useHidAmico() {
     });
   }
 
-  function getUserImage(readerId: string, hidUserId: string | number) {
-    return useNuxtApp().$api<HidApiRecord>(`${basePath}/readers/${readerId}/users/${hidUserId}/image`, {
-      method: "GET",
-    });
-  }
 
   function setUserImage(
     readerId: string,
@@ -525,7 +520,6 @@ export default function useHidAmico() {
     getReaderConfiguration,
     setReaderConfiguration,
     getLogs,
-    getUserImage,
     setUserImage,
     deleteUserImage,
     getUserCards,

package/package.json

@@ -2,7 +2,7 @@
   "name": "@7365admin1/layer-common",
   "license": "MIT",
   "type": "module",
-  "version": "4.1.3-staging.252",
+  "version": "4.1.3-staging.253",
   "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.",