no-mistakes(document): Fixed stale documentation: README.md and metaobjects-authoring/SKILL.md updated to reflect report list hook generation; all other docs accurate.

Author: dmealing

agent-context/skills/metaobjects-authoring/SKILL.md

@@ -752,7 +752,7 @@ no `source.*` is checked at load and generates nothing.
 A report `@from` a TPH subtype is refused when its view is derived (the subtype shares its
 base's table): declare it `@from` the base with an `@filter` on the discriminator field.
 
-What does not exist: no client hook, grid or form for a report yet, no way to narrow which
+What does not exist: no grid or form for a report (TypeScript generates a list hook; no other port has a client tier), no way to narrow which
 derived fields are filterable, no `measure.derived`
 (arithmetic between measures beyond `measure.ratio`), no query-time choice of dimensions or
 measures (a report is a fixed, compiled combination), and no time-zone vocabulary (grains and

docs/README.md

@@ -66,7 +66,7 @@ this tree is documentation, not the source of truth.
 | Build on a metadata model another repository publishes (`dependencies`, `meta deps sync`, overlay/extend across the boundary) | [`features/metadata-dependencies.md`](features/metadata-dependencies.md) |
 | Adopt a design MetaObjects already ships — users/groups/roles, an LLM trace envelope — instead of authoring it (`libraries`, `meta eject <library>`) | [`features/libraries.md`](features/libraries.md) |
 | Record what the system is supposed to do, and stop agents reviving retired features | [`features/requirements.md`](features/requirements.md) |
-| Declare what a dashboard groups by and counts (`dimension`, `measure`, `segment`, `object.report`; load-time checked; a view-backed report becomes a SQL view served by a generated read-only list route; no client hook yet) | [`features/reporting.md`](features/reporting.md) |
+| Declare what a dashboard groups by and counts (`dimension`, `measure`, `segment`, `object.report`; load-time checked; a view-backed report becomes a SQL view served by a generated read-only list route and a TypeScript list hook) | [`features/reporting.md`](features/reporting.md) |
 | Wire prompt construction (FR-004) | [`features/templates-and-payloads.md`](features/templates-and-payloads.md) |
 | Share a metadata shape across multiple instances (abstracts, `extends:`) | [`features/abstracts-and-inheritance.md`](features/abstracts-and-inheritance.md) |
 | Add a custom metamodel subtype or attribute to a downstream project | [`features/extending-with-providers.md`](features/extending-with-providers.md) + [`recipes/extending-metaobjects-with-providers.md`](recipes/extending-metaobjects-with-providers.md) |

server/typescript/packages/codegen-ts/test/tmp-owned-skew-sqlite-uTNCWG/Article.queries.ts

@@ -0,0 +1,88 @@
+// @generated by @metaobjectsdev/codegen-ts — DO NOT EDIT.
+// Source metadata: Article (Article)
+// Extend in your own module (e.g. Article.extra.ts) — nothing imports it for you.
+import { eq } from "drizzle-orm";
+import type { z } from "zod";
+
+import type { BaseSQLiteDatabase } from "drizzle-orm/sqlite-core";
+type Db = BaseSQLiteDatabase<
+  "sync" | "async",
+  unknown,
+  Record<string, unknown>
+>;
+
+import {
+  type Article,
+  ArticleInsertPreservingSchema,
+  ArticleInsertSchema,
+  type ArticlePatch,
+  articles,
+  ArticleUpdateSchema,
+} from "./Article";
+export async function findArticleById(
+  db: Db,
+  id: number,
+): Promise<Article | null> {
+  const [article] = await db
+    .select()
+    .from(articles)
+    .where(eq(articles.id, id))
+    .limit(1);
+  return article ?? null;
+}
+export async function listArticles(
+  db: Db,
+  opts?: { limit?: number; offset?: number },
+): Promise<Article[]> {
+  let q = db.select().from(articles).$dynamic();
+  if (opts?.limit !== undefined) {
+    q = q.limit(opts.limit);
+  }
+  if (opts?.offset !== undefined) {
+    q = q.offset(opts.offset);
+  }
+  return q;
+}
+export async function createArticle(
+  db: Db,
+  data: z.input<typeof ArticleInsertSchema>,
+): Promise<Article> {
+  const validated = ArticleInsertSchema.parse(data);
+  const [article] = await db.insert(articles).values(validated).returning();
+  return article!;
+}
+export async function insertPreservingArticle(
+  db: Db,
+  data: z.input<typeof ArticleInsertPreservingSchema>,
+): Promise<Article> {
+  const validated = ArticleInsertPreservingSchema.parse(data);
+  const [article] = await db.insert(articles).values(validated).returning();
+  return article!;
+}
+export async function updateArticle(
+  db: Db,
+  id: number,
+  patch: ArticlePatch,
+): Promise<Article | null> {
+  const validated = ArticleUpdateSchema.parse(patch);
+  // PATCH-5: an empty patch is a no-op — return the current row rather than let
+  // Drizzle throw on an empty SET clause.
+  if (Object.keys(validated).length === 0) {
+    return findArticleById(db, id);
+  }
+  const [article] = await db
+    .update(articles)
+    .set(validated)
+    .where(eq(articles.id, id))
+    .returning();
+  return article ?? null;
+}
+export async function deleteArticleById(db: Db, id: number): Promise<boolean> {
+  // Use .returning() unconditionally — supported on SQLite ≥3.35 (covers D1, libsql/Turso)
+  // and Postgres. Result is an array of deleted rows; presence implies success.
+  const deleted = await db
+    .delete(articles)
+    .where(eq(articles.id, id))
+    .returning();
+  return deleted.length > 0;
+}

server/typescript/packages/codegen-ts/test/tmp-owned-skew-sqlite-uTNCWG/Article.ts

@@ -0,0 +1,159 @@
+// @generated by @metaobjectsdev/codegen-ts — hand edits are kept by a three-way merge on regeneration; meta verify --codegen checks only the generated parts.
+// Source metadata: Article (Article)
+// Extend in your own module (e.g. Article.extra.ts) — nothing imports it for you.
+import type { InferInsertModel, InferSelectModel } from "drizzle-orm";
+import { integer, sqliteTable, text } from "drizzle-orm/sqlite-core";
+import { z } from "zod";
+
+export const articles = sqliteTable("articles", {
+  id: integer("id").primaryKey({ autoIncrement: true }),
+  title: text("title").notNull(),
+  createdAt: text("created_at").$defaultFn(() => new Date().toISOString()),
+  updatedAt: text("updated_at").$defaultFn(() => new Date().toISOString()),
+});
+export type Article = InferSelectModel<typeof articles>;
+export type ArticleInsert = InferInsertModel<typeof articles>;
+export type ArticleUpdate = Partial<ArticleInsert>;
+/** A field.timestamp instant in UTC (`YYYY-MM-DDTHH:MM:SS[.fff]Z`): a value already spelled so is kept
+ *  as sent, an offset is applied, a zoneless value is UTC and a date alone is midnight UTC.
+ *  SQLite/D1 compare timestamps as text, so instants are stored in UTC. Generated. */
+function utcIsoTimestamp(v: string): string {
+  if (/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(?:\.\d+)?Z$/.test(v)) {
+    return v;
+  }
+  const m =
+    /^(\d{4}-\d{2}-\d{2})(?:[Tt ](\d{2}:\d{2})(?::(\d{2})(?:\.(\d+))?)?(?:([Zz])|([+-]\d{2}):?(\d{2})?)?)?$/.exec(
+      v,
+    );
+  if (m === null) {
+    return v;
+  }
+  const t = Date.parse(
+    `${m[1]}T${m[2] ?? "00:00"}:${m[3] ?? "00"}.${(m[4] ?? "").slice(0, 3).padEnd(3, "0")}${
+      m[6] === undefined ? "Z" : `${m[6]}:${m[7] ?? "00"}`
+    }`,
+  );
+  return Number.isNaN(t)
+    ? v
+    : new Date(t).toISOString().replace(/\.?0+Z$/, "Z");
+}
+
+export const ArticleInsertSchema = z.object({
+  title: z.string().min(1).max(200),
+  createdAt: z
+    .string()
+    .optional()
+    .transform(() => new Date().toISOString()),
+  updatedAt: z
+    .string()
+    .optional()
+    .transform(() => new Date().toISOString()),
+});
+
+export const ArticleUpdateSchema = z.object({
+  title: z.string().min(1).max(200).optional(),
+  updatedAt: z
+    .string()
+    .optional()
+    .transform(() => new Date().toISOString()),
+});
+
+/** Typed create shape for Article: the insert schema's INPUT (pre-transform) type. A
+ * renamed/dropped/misspelt field is a compile error at every `createArticle` call site;
+ * the schema still validates at runtime. */
+export type ArticleCreate = z.input<typeof ArticleInsertSchema>;
+
+/** Typed patch shape for Article: every settable field, optional (FR-035 PATCH). A
+ * renamed/dropped field is a compile error at every `updateArticle` call site. */
+export type ArticlePatch = z.input<typeof ArticleUpdateSchema>;
+
+/** Insert-shape for import / restore / replication of Article: identical to
+ * ArticleInsertSchema, but the @autoSet timestamp columns are written VERBATIM
+ * (no create-time now() stamp) so the caller's original values are preserved. */
+export const ArticleInsertPreservingSchema = z.object({
+  title: z.string().min(1).max(200),
+  createdAt: z
+    .string()
+    .regex(
+      /^(?:\d{4}-(?:(?:0[13578]|1[02])-(?:0[1-9]|[12]\d|3[01])|(?:0[469]|11)-(?:0[1-9]|[12]\d|30)|02-(?:0[1-9]|1\d|2[0-8]))|(?:\d{2}(?:0[48]|[2468][048]|[13579][26])|(?:[02468][048]|[13579][26])00)-02-29)(?:[Tt ](?:[01]\d|2[0-3]):[0-5]\d(?::[0-5]\d(?:\.\d+)?)?(?:[Zz]|[+-](?:[01]\d|2[0-3])(?::?[0-5]\d)?)?)?$/,
+      "must be an ISO 8601 timestamp (YYYY-MM-DD[THH:MM[:SS[.fff]]][Z|±HH:MM])",
+    )
+    .transform(utcIsoTimestamp)
+    .optional(),
+  updatedAt: z
+    .string()
+    .regex(
+      /^(?:\d{4}-(?:(?:0[13578]|1[02])-(?:0[1-9]|[12]\d|3[01])|(?:0[469]|11)-(?:0[1-9]|[12]\d|30)|02-(?:0[1-9]|1\d|2[0-8]))|(?:\d{2}(?:0[48]|[2468][048]|[13579][26])|(?:[02468][048]|[13579][26])00)-02-29)(?:[Tt ](?:[01]\d|2[0-3]):[0-5]\d(?::[0-5]\d(?:\.\d+)?)?(?:[Zz]|[+-](?:[01]\d|2[0-3])(?::?[0-5]\d)?)?)?$/,
+      "must be an ISO 8601 timestamp (YYYY-MM-DD[THH:MM[:SS[.fff]]][Z|±HH:MM])",
+    )
+    .transform(utcIsoTimestamp)
+    .optional(),
+});
+
+/** Typed input of `insertPreservingArticle` — ArticleInsertPreservingSchema's pre-transform shape. */
+export type ArticleCreatePreserving = z.input<
+  typeof ArticleInsertPreservingSchema
+>;
+/**
+ * Metadata constants for Article.
+ *
+ * Use these instead of magic strings so TS catches typos and refactors stay
+ * coherent. Each non-dollar-prefixed key is a per-field object carrying
+ * name, label, view, an htmlType where the field's view maps to one, and the
+ * RHF-shaped validation rules derived from the field's validator children.
+ *
+ * placeholder and helpText are NOT emitted — no provider registers an attribute
+ * for them, so nothing here could derive one. (htmlType is emitted, but derived
+ * from the view subtype; the @htmlType override was removed for the same reason.)
+ * useEntityForm still honours placeholder if you add it: this file is generated,
+ * hand edits inside it survive regeneration through the three-way merge, and the
+ * consumers read this const rather than the metadata. That is the intended way
+ * to set it.
+ *
+ * Typical usage with the metaobjects React form helper:
+ *
+ *   import { useEntityForm } from '@metaobjectsdev/react';
+ *   const form = useEntityForm(Article, ArticleInsertSchema);
+ *   <input {...form.input.id} />
+ */
+export const Article = {
+  $entity: "Article",
+  $table: "articles",
+  $path: "/articles",
+  id: { name: "id", label: "Id", view: "number", htmlType: "number" },
+  title: {
+    name: "title",
+    label: "Title",
+    view: "text",
+    htmlType: "text",
+    rules: {
+      required: "Title is required",
+      maxLength: { value: 200, message: "Must be 200 characters or fewer" },
+    },
+  },
+  createdAt: {
+    name: "createdAt",
+    label: "Created At",
+    view: "date",
+    htmlType: "datetime-local",
+  },
+  updatedAt: {
+    name: "updatedAt",
+    label: "Updated At",
+    view: "date",
+    htmlType: "datetime-local",
+  },
+} as const;
+import type { FilterAllowlist } from "@metaobjectsdev/runtime-ts/drizzle-fastify";
+
+export const ArticleFilterAllowlist = {} as const satisfies FilterAllowlist;
+import type { SortAllowlist } from "@metaobjectsdev/runtime-ts/drizzle-fastify";
+
+export const ArticleSortAllowlist = {} as const satisfies SortAllowlist;
+export type ArticleFilter = {
+  limit?: number;
+  offset?: number;
+  sort?: string;
+  or?: ArticleFilter[];
+  and?: ArticleFilter[];
+};