@jskit-ai/users-core 0.1.159 → 0.1.161

package/package.descriptor.mjs

@@ -1,7 +1,7 @@
 export default Object.freeze({
   packageVersion: 1,
   packageId: "@jskit-ai/users-core",
-  version: "0.1.159",
+  version: "0.1.161",
   kind: "runtime",
   description: "Users/account runtime plus HTTP routes for account features.",
   dependsOn: [
@@ -146,16 +146,16 @@ export default Object.freeze({
   mutations: {
     dependencies: {
       runtime: {
-        "@jskit-ai/auth-core": "0.1.144",
-        "@jskit-ai/crud-core": "0.1.156",
-        "@jskit-ai/database-runtime": "0.1.145",
-        "@jskit-ai/http-runtime": "0.1.144",
-        "@jskit-ai/json-rest-api-core": "0.1.90",
-        "@jskit-ai/kernel": "0.1.146",
-        "@jskit-ai/resource-core": "0.1.89",
-        "@jskit-ai/resource-crud-core": "0.1.89",
+        "@jskit-ai/auth-core": "0.1.146",
+        "@jskit-ai/crud-core": "0.1.158",
+        "@jskit-ai/database-runtime": "0.1.147",
+        "@jskit-ai/http-runtime": "0.1.146",
+        "@jskit-ai/json-rest-api-core": "0.1.92",
+        "@jskit-ai/kernel": "0.1.147",
+        "@jskit-ai/resource-core": "0.1.90",
+        "@jskit-ai/resource-crud-core": "0.1.90",
         "@local/users": "file:packages/users",
-        "@jskit-ai/uploads-runtime": "0.1.122"
+        "@jskit-ai/uploads-runtime": "0.1.123"
       },
       dev: {}
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/users-core",
-  "version": "0.1.159",
+  "version": "0.1.161",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -11,14 +11,14 @@
     "./shared/resources/userSettingsResource": "./src/shared/resources/userSettingsResource.js"
   },
   "dependencies": {
-    "@jskit-ai/auth-core": "0.1.144",
-    "@jskit-ai/database-runtime": "0.1.145",
-    "@jskit-ai/http-runtime": "0.1.144",
-    "@jskit-ai/json-rest-api-core": "0.1.90",
-    "@jskit-ai/kernel": "0.1.146",
-    "@jskit-ai/resource-crud-core": "0.1.89",
-    "@jskit-ai/resource-core": "0.1.89",
-    "@jskit-ai/uploads-runtime": "0.1.122",
-    "json-rest-schema": "1.x.x"
+    "@jskit-ai/auth-core": "0.1.146",
+    "@jskit-ai/database-runtime": "0.1.147",
+    "@jskit-ai/http-runtime": "0.1.146",
+    "@jskit-ai/json-rest-api-core": "0.1.92",
+    "@jskit-ai/kernel": "0.1.147",
+    "@jskit-ai/resource-crud-core": "0.1.90",
+    "@jskit-ai/resource-core": "0.1.90",
+    "@jskit-ai/uploads-runtime": "0.1.123",
+    "json-rest-schema": "^1.0.17"
   }
 }

package/src/server/common/repositories/repositoryUtils.js

@@ -1,24 +1,11 @@
 import {
   normalizeDbRecordId,
-  toInsertDateTime,
-  toNullableDateTime,
   toIsoString,
   createWithTransaction
 } from "@jskit-ai/database-runtime/shared";
 import { isDuplicateEntryError } from "@jskit-ai/database-runtime/shared/duplicateEntry";
 import { normalizeLowerText, normalizeRecordId, normalizeText } from "@jskit-ai/kernel/shared/support/normalize";
 
-function nowDb() {
-  return toInsertDateTime();
-}
-
-function toNullableIso(value) {
-  if (!value) {
-    return null;
-  }
-  return toIsoString(value);
-}
-
 function uniqueSorted(values) {
   return [...new Set(values)].sort((left, right) => String(left).localeCompare(String(right)));
 }
@@ -43,15 +30,12 @@ function toDbJson(value, fallback = {}) {
 }
 
 export {
-  toNullableDateTime,
   toIsoString,
   isDuplicateEntryError,
   normalizeText,
   normalizeLowerText,
   normalizeRecordId,
   normalizeDbRecordId,
-  nowDb,
-  toNullableIso,
   uniqueSorted,
   parseJson,
   toDbJson,

package/src/server/common/repositories/userProfilesRepository.js

@@ -5,7 +5,6 @@ import {
   normalizeLowerText,
   normalizeRecordId,
   normalizeText,
-  nowDb,
   toIsoString
 } from "./repositoryUtils.js";
 import {
@@ -89,7 +88,7 @@ function normalizeCreatePayload(payload = {}) {
     normalized.avatarVersion = normalizeNullableVersion(source.avatarVersion);
   }
   if (Object.hasOwn(source, "avatarUpdatedAt")) {
-    normalized.avatarUpdatedAt = source.avatarUpdatedAt == null ? null : new Date(source.avatarUpdatedAt);
+    normalized.avatarUpdatedAt = source.avatarUpdatedAt == null ? null : toIsoString(source.avatarUpdatedAt);
   }
 
   return normalized;
@@ -238,7 +237,7 @@ function createRepository({ api, knex } = {}) {
           RESOURCE_TYPE,
           {
             displayName,
-            updatedAt: new Date()
+            updatedAt: new Date().toISOString()
           },
           {
             id: normalizedUserId
@@ -265,8 +264,8 @@ function createRepository({ api, knex } = {}) {
           {
             avatarStorageKey: avatar.avatarStorageKey ?? null,
             avatarVersion: avatar.avatarVersion ?? null,
-            avatarUpdatedAt: avatar.avatarUpdatedAt ?? nowDb(),
-            updatedAt: new Date()
+            avatarUpdatedAt: toIsoString(avatar.avatarUpdatedAt ?? new Date()),
+            updatedAt: new Date().toISOString()
           },
           {
             id: normalizedUserId
@@ -294,7 +293,7 @@ function createRepository({ api, knex } = {}) {
             avatarStorageKey: null,
             avatarVersion: null,
             avatarUpdatedAt: null,
-            updatedAt: new Date()
+            updatedAt: new Date().toISOString()
           },
           {
             id: normalizedUserId
@@ -350,7 +349,7 @@ function createRepository({ api, knex } = {}) {
                   email,
                   displayName,
                   username,
-                  updatedAt: new Date()
+                  updatedAt: new Date().toISOString()
                 },
                 {
                   id: normalizeDbRecordId(existing.id, { fallback: null })
@@ -378,7 +377,7 @@ function createRepository({ api, knex } = {}) {
               email,
               displayName,
               username,
-              createdAt: new Date()
+              createdAt: new Date().toISOString()
             }),
             transaction: trx
           },

package/src/server/common/repositories/userSettingsRepository.js

@@ -1,6 +1,5 @@
 import {
   normalizeRecordId,
-  nowDb,
   isDuplicateEntryError,
   createWithTransaction
 } from "./repositoryUtils.js";
@@ -146,7 +145,7 @@ function createRepository({ api, knex } = {}) {
           RESOURCE_TYPE,
           {
             ...updatePayload,
-            updatedAt: nowDb()
+            updatedAt: new Date().toISOString()
           },
           {
             id: normalizedUserId

package/test/repositoryContracts.test.js

@@ -235,9 +235,9 @@ test("userProfilesRepository.upsert patches existing profiles with resource-back
   assert.equal(attributes.email, "ada.renamed@example.com");
   assert.equal(attributes.displayName, "Ada Renamed");
   assert.equal(attributes.username, "ada");
-  assert.ok(attributes.updatedAt instanceof Date);
+  assert.equal(typeof attributes.updatedAt, "string");
   assert.equal(record?.id, "7");
-  assert.equal(record?.updatedAt, attributes.updatedAt.toISOString());
+  assert.equal(record?.updatedAt, attributes.updatedAt);
 });
 
 test("userProfilesRepository profile patch helpers stamp updatedAt through the resource contract", async () => {
@@ -285,7 +285,7 @@ test("userProfilesRepository profile patch helpers stamp updatedAt through the r
   assert.equal(calls.length, 3);
   for (const call of calls) {
     assert.equal(call.transaction, trx);
-    assert.ok(call.inputRecord?.data?.attributes?.updatedAt instanceof Date);
+    assert.equal(typeof call.inputRecord?.data?.attributes?.updatedAt, "string");
   }
   assert.deepEqual(Object.keys(calls[0].inputRecord.data.attributes).sort(), ["displayName", "updatedAt"]);
   assert.deepEqual(Object.keys(calls[1].inputRecord.data.attributes).sort(), [
@@ -296,7 +296,7 @@ test("userProfilesRepository profile patch helpers stamp updatedAt through the r
   ]);
   assert.equal(calls[1].inputRecord.data.attributes.avatarStorageKey, "avatars/7.png");
   assert.equal(calls[1].inputRecord.data.attributes.avatarVersion, "v1");
-  assert.equal(calls[1].inputRecord.data.attributes.avatarUpdatedAt, avatarUpdatedAt);
+  assert.equal(calls[1].inputRecord.data.attributes.avatarUpdatedAt, avatarUpdatedAt.toISOString());
   assert.deepEqual(calls[2].inputRecord.data.attributes.avatarStorageKey, null);
   assert.deepEqual(calls[2].inputRecord.data.attributes.avatarVersion, null);
   assert.deepEqual(calls[2].inputRecord.data.attributes.avatarUpdatedAt, null);