zitejs 0.9.101 → 0.9.102

package/dist/cjs/sync/lib.js

@@ -151,7 +151,9 @@ const DATE_FORMAT_EXAMPLES = {
 };
 const withDateFormatExample = (format) => {
     const example = DATE_FORMAT_EXAMPLES[format];
-    return example ? `"${format}" format (e.g. ${example})` : `"${format}" format`;
+    return example
+        ? `"${format}" format (e.g. ${example})`
+        : `"${format}" format`;
 };
 function toPascalCase(name) {
     const pascal = name
@@ -196,7 +198,9 @@ function tsTypeForSchemaField(def, variant = "read") {
                 .map((o) => `"${o.label.replace(/"/g, '\\"')}"`)
                 .join(" | ");
             const union = `${literals} | string`;
-            const read = def.type === "multiple_select" ? `(${union})[] | null` : `${union} | null`;
+            const read = def.type === "multiple_select"
+                ? `(${union})[] | null`
+                : `${union} | null`;
             if (variant === "write") {
                 return def.type === "multiple_select"
                     ? `${union} | (${union})[] | null`
@@ -447,7 +451,7 @@ function generateDbTs(schema) {
     lines.push("//   .findAll({ filters?, sort?, offset?, limit?, fields? }) → { records: T[], hasMore: boolean }");
     lines.push("//     filters: { fieldName: value } for equality, or { fieldName: { <op>: value } }");
     lines.push("//     operators: contains, not, in, notIn, lt, lte, gt, gte");
-    lines.push('//       `not: null` means "is set". An empty `in: []` matches nothing; an', '//       empty `notIn: []` applies no filter.');
+    lines.push('//       `not: null` means "is set". An empty `in: []` matches nothing; an', "//       empty `notIn: []` applies no filter.");
     lines.push("//     sort is currently IGNORED here — it is stripped before the request.", "//       (it does work on zite.auth.findAllUsers). Order in SQL instead.");
     lines.push("//     limit defaults to 500, max 2000; offset is a row count (a number)");
     lines.push("//   .findOne({ id?, filters?, fields? }) → T | undefined");
@@ -620,12 +624,52 @@ function inspectEndpointFile(source) {
  * words are legal object keys.
  */
 const RESERVED_IDENTIFIERS = new Set([
-    "await", "break", "case", "catch", "class", "const", "continue", "debugger",
-    "default", "delete", "do", "else", "enum", "export", "extends", "false",
-    "finally", "for", "function", "if", "implements", "import", "in",
-    "instanceof", "interface", "let", "new", "null", "package", "private",
-    "protected", "public", "return", "static", "super", "switch", "this",
-    "throw", "true", "try", "typeof", "var", "void", "while", "with", "yield",
+    "await",
+    "break",
+    "case",
+    "catch",
+    "class",
+    "const",
+    "continue",
+    "debugger",
+    "default",
+    "delete",
+    "do",
+    "else",
+    "enum",
+    "export",
+    "extends",
+    "false",
+    "finally",
+    "for",
+    "function",
+    "if",
+    "implements",
+    "import",
+    "in",
+    "instanceof",
+    "interface",
+    "let",
+    "new",
+    "null",
+    "package",
+    "private",
+    "protected",
+    "public",
+    "return",
+    "static",
+    "super",
+    "switch",
+    "this",
+    "throw",
+    "true",
+    "try",
+    "typeof",
+    "var",
+    "void",
+    "while",
+    "with",
+    "yield",
 ]);
 const toSafeIdentifier = (camelName) => RESERVED_IDENTIFIERS.has(camelName) ? `_${camelName}` : camelName;
 function generateApiTs(endpointFiles) {
@@ -955,7 +999,7 @@ function generateAirtableTs(lock) {
         '//   each carries its current user-facing name in quotes (// "Task Name"), which is',
         "//   the source of truth for what it means.",
         "//   Values are RAW; format them for display using each field's comment.",
-        "//   Fields marked \"Links to X\" hold Airtable record ids (rec...) from",
+        '//   Fields marked "Links to X" hold Airtable record ids (rec...) from',
         "//   findOne/findAll — never invent one.",
         "//   Read-only fields are absent from the *RecordInput types: Airtable rejects a",
         "//   write to a computed one (formula, rollup, lookup, autonumber, the created/",
@@ -1066,11 +1110,31 @@ function generateBackendWrapperTs(envVarNames = []) {
         " * compile and throw. `Omit` keeps any index signature merged in by",
         " * `.zite/user-extensions.d.ts`, so a migrated user-sync app still reads its",
         " * synced columns off this.",
+        " *",
+        " * The `Pick` intersection is not redundant. On an app that DOES merge in",
+        " * `[key: string]: any`, `keyof User` widens to `string`, `Exclude` then",
+        " * removes nothing, and the `Omit` collapses to a bare index signature —",
+        " * which would silently degrade `user.id` from `string` to `any`. The `Pick`",
+        " * names those members explicitly, so they survive the collapse.",
+        " *",
+        " * Only `id` and `email` are pinned that way, deliberately. On a user-sync",
+        " * app `context.user` IS the synced row (`buildUserContext` returns it",
+        " * whole), and only those two are guaranteed by the runtime: the id is the",
+        " * record id and the email is merged in explicitly. `firstName`/`lastName`",
+        " * come from whatever the app mapped those columns to, so pinning them to",
+        " * `string` would reject a legitimate read on a table that types them",
+        " * differently. Non-widened apps get both from the `Omit` regardless.",
+        " *",
+        " * On such an app `user.createdAt` still compiles, as `any`. That is",
+        " * deliberate: a synced user table may legitimately have its own",
+        " * `createdAt` column, and typing it `never` to close the hole would break",
+        " * exactly the legacy apps `user-extensions.d.ts` exists to keep compiling.",
         " */",
         "export type ZiteEndpointUser = Omit<",
         "  User,",
         "  'name' | 'emailVerified' | 'image' | 'createdAt' | 'updatedAt'",
-        ">;",
+        "> &",
+        "  Pick<User, 'id' | 'email'>;",
         "",
         'export interface ZiteRequestContext extends Omit<_ZiteRequestContext, "user"> {',
         "  user: ZiteEndpointUser;",

package/dist/esm/sync/lib.js

@@ -141,7 +141,9 @@ const DATE_FORMAT_EXAMPLES = {
 };
 const withDateFormatExample = (format) => {
     const example = DATE_FORMAT_EXAMPLES[format];
-    return example ? `"${format}" format (e.g. ${example})` : `"${format}" format`;
+    return example
+        ? `"${format}" format (e.g. ${example})`
+        : `"${format}" format`;
 };
 export function toPascalCase(name) {
     const pascal = name
@@ -186,7 +188,9 @@ function tsTypeForSchemaField(def, variant = "read") {
                 .map((o) => `"${o.label.replace(/"/g, '\\"')}"`)
                 .join(" | ");
             const union = `${literals} | string`;
-            const read = def.type === "multiple_select" ? `(${union})[] | null` : `${union} | null`;
+            const read = def.type === "multiple_select"
+                ? `(${union})[] | null`
+                : `${union} | null`;
             if (variant === "write") {
                 return def.type === "multiple_select"
                     ? `${union} | (${union})[] | null`
@@ -437,7 +441,7 @@ export function generateDbTs(schema) {
     lines.push("//   .findAll({ filters?, sort?, offset?, limit?, fields? }) → { records: T[], hasMore: boolean }");
     lines.push("//     filters: { fieldName: value } for equality, or { fieldName: { <op>: value } }");
     lines.push("//     operators: contains, not, in, notIn, lt, lte, gt, gte");
-    lines.push('//       `not: null` means "is set". An empty `in: []` matches nothing; an', '//       empty `notIn: []` applies no filter.');
+    lines.push('//       `not: null` means "is set". An empty `in: []` matches nothing; an', "//       empty `notIn: []` applies no filter.");
     lines.push("//     sort is currently IGNORED here — it is stripped before the request.", "//       (it does work on zite.auth.findAllUsers). Order in SQL instead.");
     lines.push("//     limit defaults to 500, max 2000; offset is a row count (a number)");
     lines.push("//   .findOne({ id?, filters?, fields? }) → T | undefined");
@@ -610,12 +614,52 @@ function inspectEndpointFile(source) {
  * words are legal object keys.
  */
 const RESERVED_IDENTIFIERS = new Set([
-    "await", "break", "case", "catch", "class", "const", "continue", "debugger",
-    "default", "delete", "do", "else", "enum", "export", "extends", "false",
-    "finally", "for", "function", "if", "implements", "import", "in",
-    "instanceof", "interface", "let", "new", "null", "package", "private",
-    "protected", "public", "return", "static", "super", "switch", "this",
-    "throw", "true", "try", "typeof", "var", "void", "while", "with", "yield",
+    "await",
+    "break",
+    "case",
+    "catch",
+    "class",
+    "const",
+    "continue",
+    "debugger",
+    "default",
+    "delete",
+    "do",
+    "else",
+    "enum",
+    "export",
+    "extends",
+    "false",
+    "finally",
+    "for",
+    "function",
+    "if",
+    "implements",
+    "import",
+    "in",
+    "instanceof",
+    "interface",
+    "let",
+    "new",
+    "null",
+    "package",
+    "private",
+    "protected",
+    "public",
+    "return",
+    "static",
+    "super",
+    "switch",
+    "this",
+    "throw",
+    "true",
+    "try",
+    "typeof",
+    "var",
+    "void",
+    "while",
+    "with",
+    "yield",
 ]);
 const toSafeIdentifier = (camelName) => RESERVED_IDENTIFIERS.has(camelName) ? `_${camelName}` : camelName;
 export function generateApiTs(endpointFiles) {
@@ -945,7 +989,7 @@ export function generateAirtableTs(lock) {
         '//   each carries its current user-facing name in quotes (// "Task Name"), which is',
         "//   the source of truth for what it means.",
         "//   Values are RAW; format them for display using each field's comment.",
-        "//   Fields marked \"Links to X\" hold Airtable record ids (rec...) from",
+        '//   Fields marked "Links to X" hold Airtable record ids (rec...) from',
         "//   findOne/findAll — never invent one.",
         "//   Read-only fields are absent from the *RecordInput types: Airtable rejects a",
         "//   write to a computed one (formula, rollup, lookup, autonumber, the created/",
@@ -1056,11 +1100,31 @@ export function generateBackendWrapperTs(envVarNames = []) {
         " * compile and throw. `Omit` keeps any index signature merged in by",
         " * `.zite/user-extensions.d.ts`, so a migrated user-sync app still reads its",
         " * synced columns off this.",
+        " *",
+        " * The `Pick` intersection is not redundant. On an app that DOES merge in",
+        " * `[key: string]: any`, `keyof User` widens to `string`, `Exclude` then",
+        " * removes nothing, and the `Omit` collapses to a bare index signature —",
+        " * which would silently degrade `user.id` from `string` to `any`. The `Pick`",
+        " * names those members explicitly, so they survive the collapse.",
+        " *",
+        " * Only `id` and `email` are pinned that way, deliberately. On a user-sync",
+        " * app `context.user` IS the synced row (`buildUserContext` returns it",
+        " * whole), and only those two are guaranteed by the runtime: the id is the",
+        " * record id and the email is merged in explicitly. `firstName`/`lastName`",
+        " * come from whatever the app mapped those columns to, so pinning them to",
+        " * `string` would reject a legitimate read on a table that types them",
+        " * differently. Non-widened apps get both from the `Omit` regardless.",
+        " *",
+        " * On such an app `user.createdAt` still compiles, as `any`. That is",
+        " * deliberate: a synced user table may legitimately have its own",
+        " * `createdAt` column, and typing it `never` to close the hole would break",
+        " * exactly the legacy apps `user-extensions.d.ts` exists to keep compiling.",
         " */",
         "export type ZiteEndpointUser = Omit<",
         "  User,",
         "  'name' | 'emailVerified' | 'image' | 'createdAt' | 'updatedAt'",
-        ">;",
+        "> &",
+        "  Pick<User, 'id' | 'email'>;",
         "",
         'export interface ZiteRequestContext extends Omit<_ZiteRequestContext, "user"> {',
         "  user: ZiteEndpointUser;",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zitejs",
-  "version": "0.9.101",
+  "version": "0.9.102",
   "description": "The Zite framework — build apps on Zite Database",
   "type": "module",
   "main": "./dist/cjs/index.js",