clivly 0.9.7 → 0.10.0

package/drizzle.cjs

@@ -1,7 +1,7 @@
 Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
-const require_projection = require("./projection-DEwL-D8R.cjs");
+const require_projection = require("./projection-Dz6_i1-w.cjs");
 const require_drizzle_meta = require("./drizzle-meta-Bb1JWYqf.cjs");
-const require_projection_sql = require("./projection-sql-iHy-Cx6V.cjs");
+const require_projection_sql = require("./projection-sql-CbXCKt3s.cjs");
 const require_drizzle = require("./drizzle-DcB7OfBp.cjs");
 let drizzle_orm = require("drizzle-orm");
 //#region ../sdk/src/drizzle-projection.ts

package/drizzle.d.cts

@@ -1,6 +1,6 @@
-import { F as ProjectionRef, P as ProjectionMapping } from "./entity-config-B2RA53nw.cjs";
-import { n as buildUniqueConstraints, r as discoverFromDrizzle } from "./drizzle-ixVV4pPN.cjs";
-import { t as NamespaceIntrospector, v as SyncSource } from "./namespace-probe-DZIGdvlm.cjs";
+import { F as ProjectionMapping, I as ProjectionRef } from "./entity-config-DNQAMbMS.cjs";
+import { n as buildUniqueConstraints, r as discoverFromDrizzle } from "./drizzle-DTOuGXEn.cjs";
+import { t as NamespaceIntrospector, v as SyncSource } from "./namespace-probe-DC4tbmLj.cjs";
 import { Table } from "drizzle-orm";
 //#region ../sdk/src/drizzle-projection.d.ts
 /**

package/drizzle.d.mts

@@ -1,6 +1,6 @@
-import { F as ProjectionRef, P as ProjectionMapping } from "./entity-config-B2RA53nw.mjs";
-import { n as buildUniqueConstraints, r as discoverFromDrizzle } from "./drizzle-BmDDgAIF.mjs";
-import { t as NamespaceIntrospector, v as SyncSource } from "./namespace-probe-BQ4imnV7.mjs";
+import { F as ProjectionMapping, I as ProjectionRef } from "./entity-config-DNQAMbMS.mjs";
+import { n as buildUniqueConstraints, r as discoverFromDrizzle } from "./drizzle-SGapoly2.mjs";
+import { t as NamespaceIntrospector, v as SyncSource } from "./namespace-probe-9ScMDR0t.mjs";
 import { Table } from "drizzle-orm";
 //#region ../sdk/src/drizzle-projection.d.ts
 /**

package/drizzle.mjs

@@ -1,6 +1,6 @@
-import { t as BASE_REF } from "./projection-hnPSgH6G.mjs";
+import { t as BASE_REF } from "./projection-CSGJbZlZ.mjs";
 import { t as DRIZZLE_META } from "./drizzle-meta-D-5k3n0i.mjs";
-import { a as compileProjectionPreview, i as compileProjectionJoins, n as compileProjectionFrom, r as compileProjectionIdentity, s as quoteIdent } from "./projection-sql-9zzTVGvf.mjs";
+import { a as compileProjectionPreview, i as compileProjectionJoins, n as compileProjectionFrom, r as compileProjectionIdentity, s as quoteIdent } from "./projection-sql-iHoiYr1R.mjs";
 import { n as buildUniqueConstraints, r as discoverFromDrizzle, t as buildColumnsMeta } from "./drizzle-wbcuG-cy.mjs";
 import { asc, getTableColumns, getTableName, sql } from "drizzle-orm";
 //#region ../sdk/src/drizzle-projection.ts

package/{entity-config-C8IIkrmX.mjs → entity-config-C4d3K9CP.mjs}

@@ -1,4 +1,4 @@
-import { a as projectionSchema, c as boolean, d as record, f as strictObject, i as identityRefs, l as literal, m as union, o as _enum, p as string, s as array, t as BASE_REF, u as number } from "./projection-hnPSgH6G.mjs";
+import { a as _enum, c as literal, d as strictObject, f as string, i as projectionSchema, l as number, o as array, p as union, r as identityRefs, s as boolean, t as BASE_REF, u as record } from "./projection-CSGJbZlZ.mjs";
 //#region ../core/src/schema-meta.ts
 /**
 * Accessors over a discovered table's schema-v2 metadata.
@@ -29,7 +29,17 @@ function coversUniqueKey(table, columns) {
 	const present = new Set(columns);
 	return (table.uniqueConstraints ?? []).some((key) => key.length > 0 && key.every((column) => present.has(column)));
 }
-/** Is this table's payload rich enough to validate a projection against? */
+/**
+* Is this table's payload rich enough to validate a projection against?
+*
+* A type predicate rather than a plain boolean so the narrowing is visible to
+* the compiler: callers that guard on this can iterate `columnsMeta` directly
+* instead of casting, and a cast that outlives its guard becomes a type error
+* rather than a silent walk over `undefined`.
+*
+* The invariant is unchanged: this reports whether the metadata is *known*,
+* never whether the table has keys. Absent metadata stays UNKNOWN.
+*/
 function hasRichMeta(table) {
 	return Boolean(table.columnsMeta && table.uniqueConstraints);
 }
@@ -273,6 +283,64 @@ function checkFanOut(errors, projection, resolver, base, basePk) {
 	});
 }
 /**
+* One error naming a filter column that is not on the projection's base table,
+* and the base table it would have to be on.
+*/
+function columnOnBase(column, baseTable, path) {
+	if (baseTable?.columns.includes(column)) return [];
+	return [{
+		path,
+		message: `filter column "${column}" does not exist on "${baseTable?.name ?? "the base table"}" — a derived object is filtered by its base table's columns only, so name one of those, or move the condition into a join's \`on\``
+	}];
+}
+/**
+* A derived object's filter is evaluated in the cloud against pushed rows, so
+* it can only name columns of the base table.
+*
+* Every rejected shape here fails *silently* today, which is the whole reason
+* the check exists — a filter that cannot be honoured must error, never quietly
+* return the wrong rows. The three shapes fail in different directions:
+*
+* - `$raw` is opaque SQL meant for the embedded model. The cloud path cannot
+*   evaluate it and passes every row, so the filter is ignored and the object
+*   syncs UNFILTERED.
+* - A joined-table column resolves to nothing on a pushed row, so under a
+*   positive operator (`eq`, `in`) no row matches and the object syncs EMPTY.
+* - The same unresolvable column under a NEGATIVE operator (`ne`, `isNull`,
+*   `notIn`) is the dangerous one: the missing cell reads as null, so the
+*   predicate is true for every row and a filter meant to narrow the sync
+*   ADMITS EVERYTHING. Nobody notices extra rows arriving.
+*
+* Rejecting an unresolvable column closes the last two together.
+*/
+function validateProjectionFilter(filter, projection, schema, base) {
+	if (!filter) return [];
+	const path = `${base}.filter`;
+	if ("$raw" in filter) return [{
+		path,
+		message: "a $raw filter cannot be evaluated against a derived object's rows — it is opaque SQL, so the cloud passes every row through unfiltered. Express it as column conditions on the base table instead"
+	}];
+	const baseTable = schema.find((t) => t.name === projection.baseTable);
+	const errors = [];
+	for (const column of Object.keys(filter)) {
+		if (column.includes(".")) {
+			const [ref, ...rest] = column.split(".");
+			const bare = rest.join(".");
+			if (ref !== "base") {
+				errors.push({
+					path,
+					message: `filter column "${column}" names the join "${ref}", but a derived object is filtered by its base table ("${projection.baseTable}") only — filter on a base column, or move the condition into that join's \`on\``
+				});
+				continue;
+			}
+			errors.push(...columnOnBase(bare, baseTable, path));
+			continue;
+		}
+		errors.push(...columnOnBase(column, baseTable, path));
+	}
+	return errors;
+}
+/**
 * Validate a projection against the discovered schema. Returns every error
 * found (not just the first) so a builder UI can show them all at once.
 */
@@ -537,6 +605,15 @@ function customObjectDeclarations(config) {
 * Qualified rather than bare: two bindings can pull the same column name from
 * different tables (`cl.full_name` and `adv.full_name`), which a bare column
 * name cannot tell apart.
+*
+* This is the single derivation of that map, and both consumers depend on it
+* emitting ref tokens. `mappingsToEntitiesConfig` populates a projection
+* entity's required `fields`; the cloud ingest path needs the same map because
+* a projection's persisted `crm_entity_mappings.field_map` is empty by design,
+* so `rowToSourceRow` derives the effective field map from the projection
+* instead. The tokens emitted here are exactly how `fromDrizzleProjection` keys
+* the rows it pushes (`drizzle-projection.ts` `refToken`), so the two line up.
+* Deriving this a second time anywhere would let the copies drift.
 */
 function projectionFields(projection) {
 	const fields = {};
@@ -675,7 +752,7 @@ function validateEntity(v, config, key, entity, schema) {
 		if (entity.source !== entity.projection.baseTable) v.report(`${base}.source`, `"source" ("${entity.source}") must match "projection.baseTable" ("${entity.projection.baseTable}") — the view compiler reads baseTable and the sync engine reads source, so a mismatch makes them disagree about which table this entity is`);
 		for (const error of validateProjection(entity.projection, schema, base)) v.report(error.path, error.message);
 		validateReadOnly(v, base, entity);
-		validateFilter(v, base, entity);
+		for (const error of validateProjectionFilter(entity.filter, entity.projection, schema, base)) v.report(error.path, error.message);
 		return;
 	}
 	validateFields(v, base, entity);
@@ -701,4 +778,4 @@ function validateEntitiesConfig(config, schema) {
 	};
 }
 //#endregion
-export { columnMetaFor as _, MATERIALIZING_CONCEPTS as a, primaryKeyColumns as b, entitiesConfigSchema as c, relationshipSchema as d, relationshipsSchema as f, joinFansOut as g, fanOutJoins as h, ClivlyConfigError as i, filterSchema as l, validateProjection as m, CRM_CONCEPTS as n, customObjectDeclarations as o, validateEntitiesConfig as p, CUSTOM_CONCEPT as r, defineClivlyConfig as s, CANONICAL_FIELDS as t, projectionFields as u, coversUniqueKey as v, hasRichMeta as y };
+export { joinFansOut as _, MATERIALIZING_CONCEPTS as a, hasRichMeta as b, entitiesConfigSchema as c, relationshipSchema as d, relationshipsSchema as f, fanOutJoins as g, validateProjectionFilter as h, ClivlyConfigError as i, filterSchema as l, validateProjection as m, CRM_CONCEPTS as n, customObjectDeclarations as o, validateEntitiesConfig as p, CUSTOM_CONCEPT as r, defineClivlyConfig as s, CANONICAL_FIELDS as t, projectionFields as u, columnMetaFor as v, primaryKeyColumns as x, coversUniqueKey as y };

package/{entity-config-B2RA53nw.d.cts → entity-config-DNQAMbMS.d.cts}

@@ -2137,6 +2137,23 @@ declare const ZodReadonly: $constructor<ZodReadonly>;
  */
 /** The reserved ref name for the projection's base table. */
 declare const BASE_REF = "base";
+/**
+ * Entity keys become view names (`crm_src_<entityKey>`), so keep them slugs.
+ *
+ * Accepts hyphens as well as underscores, matching `OBJECT_TYPE_SLUG_PATTERN`
+ * on the server. Those two were widened apart: the server learned hyphens
+ * because the SDK's own `slugify` collapses non-alphanumerics to them and
+ * `clivly status` reports that form as canonical — but this pattern was missed,
+ * so `clivly status` would tell you to key an entity `client-accounts` and then
+ * `defineClivlyConfig` would reject the config it had just asked for. Same bug,
+ * opposite half of the product.
+ *
+ * Widening is the safe direction: every key that was valid before still is, so
+ * nothing existing is renamed. Safe at all because neither use is ever a bare
+ * SQL identifier — the view name is quoted (`view-compiler.ts`, `quoteIdent`),
+ * and so is a join's `key` when it is emitted as an alias.
+ */
+declare const PROJECTION_KEY_PATTERN: RegExp;
 declare const projectionRefSchema: ZodObject<{
   table: ZodString;
   column: ZodString;
@@ -2638,6 +2655,15 @@ declare function customObjectDeclarations(config: ClivlyEntitiesConfig): CustomO
  * Qualified rather than bare: two bindings can pull the same column name from
  * different tables (`cl.full_name` and `adv.full_name`), which a bare column
  * name cannot tell apart.
+ *
+ * This is the single derivation of that map, and both consumers depend on it
+ * emitting ref tokens. `mappingsToEntitiesConfig` populates a projection
+ * entity's required `fields`; the cloud ingest path needs the same map because
+ * a projection's persisted `crm_entity_mappings.field_map` is empty by design,
+ * so `rowToSourceRow` derives the effective field map from the projection
+ * instead. The tokens emitted here are exactly how `fromDrizzleProjection` keys
+ * the rows it pushes (`drizzle-projection.ts` `refToken`), so the two line up.
+ * Deriving this a second time anywhere would let the copies drift.
  */
 declare function projectionFields(projection: ProjectionMapping): Record<string, string>;
 declare function defineClivlyConfig(config: ClivlyEntitiesConfigInput, options?: {
@@ -2698,4 +2724,4 @@ interface ConfigValidationResult {
  */
 declare function validateEntitiesConfig(config: ClivlyEntitiesConfig, schema: DiscoveredSchemaTable[]): ConfigValidationResult;
 //#endregion
-export { BASE_REF as A, defineClivlyConfig as C, relationshipSchema as D, projectionFields as E, ProjectionRef as F, identityRefs as I, projectionSchema as L, ProjectionIdentity as M, ProjectionJoin as N, relationshipsSchema as O, ProjectionMapping as P, customObjectDeclarations as S, filterSchema as T, MATERIALIZING_CONCEPTS as _, ClivlyEntitiesConfig as a, RelationshipVia as b, ClivlyEntityConfigInput as c, CrmConcept as d, CustomObjectDeclaration as f, FilterValue as g, FilterExpr as h, ClivlyConfigError as i, ProjectionFieldBinding as j, validateEntitiesConfig as k, ConfigValidationError as l, DiscoveredSchemaTable as m, CRM_CONCEPTS as n, ClivlyEntitiesConfigInput as o, DiscoveredColumnMeta as p, CUSTOM_CONCEPT as r, ClivlyEntityConfig as s, CANONICAL_FIELDS as t, ConfigValidationResult as u, MaterializingConcept as v, entitiesConfigSchema as w, RelationshipsMap as x, RelationshipSpec as y };
+export { BASE_REF as A, defineClivlyConfig as C, relationshipSchema as D, projectionFields as E, ProjectionMapping as F, ProjectionRef as I, identityRefs as L, ProjectionFieldBinding as M, ProjectionIdentity as N, relationshipsSchema as O, ProjectionJoin as P, projectionSchema as R, customObjectDeclarations as S, filterSchema as T, MATERIALIZING_CONCEPTS as _, ClivlyEntitiesConfig as a, RelationshipVia as b, ClivlyEntityConfigInput as c, CrmConcept as d, CustomObjectDeclaration as f, FilterValue as g, FilterExpr as h, ClivlyConfigError as i, PROJECTION_KEY_PATTERN as j, validateEntitiesConfig as k, ConfigValidationError as l, DiscoveredSchemaTable as m, CRM_CONCEPTS as n, ClivlyEntitiesConfigInput as o, DiscoveredColumnMeta as p, CUSTOM_CONCEPT as r, ClivlyEntityConfig as s, CANONICAL_FIELDS as t, ConfigValidationResult as u, MaterializingConcept as v, entitiesConfigSchema as w, RelationshipsMap as x, RelationshipSpec as y };

package/{entity-config-B2RA53nw.d.mts → entity-config-DNQAMbMS.d.mts}

@@ -2137,6 +2137,23 @@ declare const ZodReadonly: $constructor<ZodReadonly>;
  */
 /** The reserved ref name for the projection's base table. */
 declare const BASE_REF = "base";
+/**
+ * Entity keys become view names (`crm_src_<entityKey>`), so keep them slugs.
+ *
+ * Accepts hyphens as well as underscores, matching `OBJECT_TYPE_SLUG_PATTERN`
+ * on the server. Those two were widened apart: the server learned hyphens
+ * because the SDK's own `slugify` collapses non-alphanumerics to them and
+ * `clivly status` reports that form as canonical — but this pattern was missed,
+ * so `clivly status` would tell you to key an entity `client-accounts` and then
+ * `defineClivlyConfig` would reject the config it had just asked for. Same bug,
+ * opposite half of the product.
+ *
+ * Widening is the safe direction: every key that was valid before still is, so
+ * nothing existing is renamed. Safe at all because neither use is ever a bare
+ * SQL identifier — the view name is quoted (`view-compiler.ts`, `quoteIdent`),
+ * and so is a join's `key` when it is emitted as an alias.
+ */
+declare const PROJECTION_KEY_PATTERN: RegExp;
 declare const projectionRefSchema: ZodObject<{
   table: ZodString;
   column: ZodString;
@@ -2638,6 +2655,15 @@ declare function customObjectDeclarations(config: ClivlyEntitiesConfig): CustomO
  * Qualified rather than bare: two bindings can pull the same column name from
  * different tables (`cl.full_name` and `adv.full_name`), which a bare column
  * name cannot tell apart.
+ *
+ * This is the single derivation of that map, and both consumers depend on it
+ * emitting ref tokens. `mappingsToEntitiesConfig` populates a projection
+ * entity's required `fields`; the cloud ingest path needs the same map because
+ * a projection's persisted `crm_entity_mappings.field_map` is empty by design,
+ * so `rowToSourceRow` derives the effective field map from the projection
+ * instead. The tokens emitted here are exactly how `fromDrizzleProjection` keys
+ * the rows it pushes (`drizzle-projection.ts` `refToken`), so the two line up.
+ * Deriving this a second time anywhere would let the copies drift.
  */
 declare function projectionFields(projection: ProjectionMapping): Record<string, string>;
 declare function defineClivlyConfig(config: ClivlyEntitiesConfigInput, options?: {
@@ -2698,4 +2724,4 @@ interface ConfigValidationResult {
  */
 declare function validateEntitiesConfig(config: ClivlyEntitiesConfig, schema: DiscoveredSchemaTable[]): ConfigValidationResult;
 //#endregion
-export { BASE_REF as A, defineClivlyConfig as C, relationshipSchema as D, projectionFields as E, ProjectionRef as F, identityRefs as I, projectionSchema as L, ProjectionIdentity as M, ProjectionJoin as N, relationshipsSchema as O, ProjectionMapping as P, customObjectDeclarations as S, filterSchema as T, MATERIALIZING_CONCEPTS as _, ClivlyEntitiesConfig as a, RelationshipVia as b, ClivlyEntityConfigInput as c, CrmConcept as d, CustomObjectDeclaration as f, FilterValue as g, FilterExpr as h, ClivlyConfigError as i, ProjectionFieldBinding as j, validateEntitiesConfig as k, ConfigValidationError as l, DiscoveredSchemaTable as m, CRM_CONCEPTS as n, ClivlyEntitiesConfigInput as o, DiscoveredColumnMeta as p, CUSTOM_CONCEPT as r, ClivlyEntityConfig as s, CANONICAL_FIELDS as t, ConfigValidationResult as u, MaterializingConcept as v, entitiesConfigSchema as w, RelationshipsMap as x, RelationshipSpec as y };
+export { BASE_REF as A, defineClivlyConfig as C, relationshipSchema as D, projectionFields as E, ProjectionMapping as F, ProjectionRef as I, identityRefs as L, ProjectionFieldBinding as M, ProjectionIdentity as N, relationshipsSchema as O, ProjectionJoin as P, projectionSchema as R, customObjectDeclarations as S, filterSchema as T, MATERIALIZING_CONCEPTS as _, ClivlyEntitiesConfig as a, RelationshipVia as b, ClivlyEntityConfigInput as c, CrmConcept as d, CustomObjectDeclaration as f, FilterValue as g, FilterExpr as h, ClivlyConfigError as i, PROJECTION_KEY_PATTERN as j, validateEntitiesConfig as k, ConfigValidationError as l, DiscoveredSchemaTable as m, CRM_CONCEPTS as n, ClivlyEntitiesConfigInput as o, DiscoveredColumnMeta as p, CUSTOM_CONCEPT as r, ClivlyEntityConfig as s, CANONICAL_FIELDS as t, ConfigValidationResult as u, MaterializingConcept as v, entitiesConfigSchema as w, RelationshipsMap as x, RelationshipSpec as y };

package/{entity-config-CPM-Tp-d.cjs → entity-config-DgAe0_YD.cjs}

@@ -1,4 +1,4 @@
-const require_projection = require("./projection-DEwL-D8R.cjs");
+const require_projection = require("./projection-Dz6_i1-w.cjs");
 //#region ../core/src/schema-meta.ts
 /**
 * Accessors over a discovered table's schema-v2 metadata.
@@ -29,7 +29,17 @@ function coversUniqueKey(table, columns) {
 	const present = new Set(columns);
 	return (table.uniqueConstraints ?? []).some((key) => key.length > 0 && key.every((column) => present.has(column)));
 }
-/** Is this table's payload rich enough to validate a projection against? */
+/**
+* Is this table's payload rich enough to validate a projection against?
+*
+* A type predicate rather than a plain boolean so the narrowing is visible to
+* the compiler: callers that guard on this can iterate `columnsMeta` directly
+* instead of casting, and a cast that outlives its guard becomes a type error
+* rather than a silent walk over `undefined`.
+*
+* The invariant is unchanged: this reports whether the metadata is *known*,
+* never whether the table has keys. Absent metadata stays UNKNOWN.
+*/
 function hasRichMeta(table) {
 	return Boolean(table.columnsMeta && table.uniqueConstraints);
 }
@@ -273,6 +283,64 @@ function checkFanOut(errors, projection, resolver, base, basePk) {
 	});
 }
 /**
+* One error naming a filter column that is not on the projection's base table,
+* and the base table it would have to be on.
+*/
+function columnOnBase(column, baseTable, path) {
+	if (baseTable?.columns.includes(column)) return [];
+	return [{
+		path,
+		message: `filter column "${column}" does not exist on "${baseTable?.name ?? "the base table"}" — a derived object is filtered by its base table's columns only, so name one of those, or move the condition into a join's \`on\``
+	}];
+}
+/**
+* A derived object's filter is evaluated in the cloud against pushed rows, so
+* it can only name columns of the base table.
+*
+* Every rejected shape here fails *silently* today, which is the whole reason
+* the check exists — a filter that cannot be honoured must error, never quietly
+* return the wrong rows. The three shapes fail in different directions:
+*
+* - `$raw` is opaque SQL meant for the embedded model. The cloud path cannot
+*   evaluate it and passes every row, so the filter is ignored and the object
+*   syncs UNFILTERED.
+* - A joined-table column resolves to nothing on a pushed row, so under a
+*   positive operator (`eq`, `in`) no row matches and the object syncs EMPTY.
+* - The same unresolvable column under a NEGATIVE operator (`ne`, `isNull`,
+*   `notIn`) is the dangerous one: the missing cell reads as null, so the
+*   predicate is true for every row and a filter meant to narrow the sync
+*   ADMITS EVERYTHING. Nobody notices extra rows arriving.
+*
+* Rejecting an unresolvable column closes the last two together.
+*/
+function validateProjectionFilter(filter, projection, schema, base) {
+	if (!filter) return [];
+	const path = `${base}.filter`;
+	if ("$raw" in filter) return [{
+		path,
+		message: "a $raw filter cannot be evaluated against a derived object's rows — it is opaque SQL, so the cloud passes every row through unfiltered. Express it as column conditions on the base table instead"
+	}];
+	const baseTable = schema.find((t) => t.name === projection.baseTable);
+	const errors = [];
+	for (const column of Object.keys(filter)) {
+		if (column.includes(".")) {
+			const [ref, ...rest] = column.split(".");
+			const bare = rest.join(".");
+			if (ref !== "base") {
+				errors.push({
+					path,
+					message: `filter column "${column}" names the join "${ref}", but a derived object is filtered by its base table ("${projection.baseTable}") only — filter on a base column, or move the condition into that join's \`on\``
+				});
+				continue;
+			}
+			errors.push(...columnOnBase(bare, baseTable, path));
+			continue;
+		}
+		errors.push(...columnOnBase(column, baseTable, path));
+	}
+	return errors;
+}
+/**
 * Validate a projection against the discovered schema. Returns every error
 * found (not just the first) so a builder UI can show them all at once.
 */
@@ -545,6 +613,15 @@ function customObjectDeclarations(config) {
 * Qualified rather than bare: two bindings can pull the same column name from
 * different tables (`cl.full_name` and `adv.full_name`), which a bare column
 * name cannot tell apart.
+*
+* This is the single derivation of that map, and both consumers depend on it
+* emitting ref tokens. `mappingsToEntitiesConfig` populates a projection
+* entity's required `fields`; the cloud ingest path needs the same map because
+* a projection's persisted `crm_entity_mappings.field_map` is empty by design,
+* so `rowToSourceRow` derives the effective field map from the projection
+* instead. The tokens emitted here are exactly how `fromDrizzleProjection` keys
+* the rows it pushes (`drizzle-projection.ts` `refToken`), so the two line up.
+* Deriving this a second time anywhere would let the copies drift.
 */
 function projectionFields(projection) {
 	const fields = {};
@@ -683,7 +760,7 @@ function validateEntity(v, config, key, entity, schema) {
 		if (entity.source !== entity.projection.baseTable) v.report(`${base}.source`, `"source" ("${entity.source}") must match "projection.baseTable" ("${entity.projection.baseTable}") — the view compiler reads baseTable and the sync engine reads source, so a mismatch makes them disagree about which table this entity is`);
 		for (const error of validateProjection(entity.projection, schema, base)) v.report(error.path, error.message);
 		validateReadOnly(v, base, entity);
-		validateFilter(v, base, entity);
+		for (const error of validateProjectionFilter(entity.filter, entity.projection, schema, base)) v.report(error.path, error.message);
 		return;
 	}
 	validateFields(v, base, entity);
@@ -829,3 +906,9 @@ Object.defineProperty(exports, "validateProjection", {
 		return validateProjection;
 	}
 });
+Object.defineProperty(exports, "validateProjectionFilter", {
+	enumerable: true,
+	get: function() {
+		return validateProjectionFilter;
+	}
+});

package/index.cjs

@@ -1,9 +1,9 @@
 #!/usr/bin/env node
 Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
-const require_entity_config = require("./entity-config-CPM-Tp-d.cjs");
+const require_entity_config = require("./entity-config-DgAe0_YD.cjs");
 require("./mapping-score-CaB3F4Uu.cjs");
 const require_internal_entity_heuristics = require("./internal-entity-heuristics.cjs");
-const require_sdk = require("./sdk-BL9BCzyD.cjs");
+const require_sdk = require("./sdk-B1vX4kUJ.cjs");
 let node_child_process = require("node:child_process");
 let node_fs = require("node:fs");
 let node_module = require("node:module");
@@ -306,16 +306,27 @@ function addCommand(pm, packages) {
 */
 const CLIVLY_TICK_PATH = "/api/clivly/tick";
 /**
-* What `clivly login` writes into `.env` for `CLIVLY_SYNC_TRIGGER_URL`.
+* The example shape `clivly init`/`clivly login` show for `CLIVLY_SYNC_TRIGGER_URL`
+* — commented out in `.env`, not written as a live value.
 *
-* A visible template beats an empty line: the developer sees the shape they
-* eventually need — a public origin plus the tick path — instead of an unhelpful
-* `CLIVLY_SYNC_TRIGGER_URL=` they have to go and look up.
+* It used to be scaffolded uncommented, on the theory that a visible template
+* beats an empty line. The SDK's `isPlaceholderUrl` guard
+* (`@clivly/sdk/trigger-url`) already keeps a bracketed placeholder like this
+* one out of `syncTrigger.url` on every heartbeat — `resolveTriggerUrl` skips
+* it in favour of the dev tunnel / platform-derived origin — and
+* `assertSafeTriggerUrl` on the API side rejects it outright if it were ever
+* sent, since `<yourDomain.com>` doesn't even parse as a URL. So it was never
+* actually fireable (F-12 is not an SSRF/overwrite bug).
 *
-* It is safe to ship a truthy non-address here only because the SDK refuses to
-* treat a bracketed value as configured (`isPlaceholderUrl` in
-* `@clivly/sdk/trigger-url`). Change the brackets here and that guard stops
-* matching, so the two must stay in step.
+* What it DID break: readiness reporting only checked *presence*, not
+* validity, so an unedited placeholder made the dashboard's setup ladder and
+* `triggerConfigured` report "done" while `clivly status` correctly flagged
+* it — two surfaces disagreeing about a workspace that could never run "Run
+* sync". See `hasWorkingTrigger` (`@clivly.com/api/sync-trigger`) for the
+* fix. Commenting the example out here removes the confusing truthy-but-fake
+* value at the source, on top of that fix, while keeping the shape visible
+* for the (uncommon) case a developer needs it — most platforms are
+* auto-detected, and `clivly dev` handles local development.
 */
 const TRIGGER_URL_PLACEHOLDER = `https://<yourDomain.com>${CLIVLY_TICK_PATH}`;
 //#endregion
@@ -1440,8 +1451,8 @@ function credentialEntries(credentials) {
 		},
 		{
 			key: "CLIVLY_SYNC_TRIGGER_URL",
-			value: TRIGGER_URL_PLACEHOLDER,
-			comment: "Public URL of your tick route — replace <yourDomain.com> once deployed. Local dev: run `clivly dev` instead."
+			value: "",
+			comment: `Public URL of your tick route. Only needed when the platform's own URL can't be auto-detected (see \`clivly status\`). Uncomment and set your deployed origin if so:\n# CLIVLY_SYNC_TRIGGER_URL=${TRIGGER_URL_PLACEHOLDER}`
 		}
 	];
 }
@@ -1810,8 +1821,7 @@ const CLIVLY_ENV_KEYS = [
 	},
 	{
 		key: "CLIVLY_SYNC_TRIGGER_URL",
-		value: TRIGGER_URL_PLACEHOLDER,
-		comment: "Public URL of your tick route — replace <yourDomain.com> once deployed. Local dev: run `clivly dev` instead."
+		comment: `Public URL of your tick route. Only needed when the platform's own URL can't be auto-detected (see \`clivly status\`). Uncomment and set your deployed origin if so:\n# CLIVLY_SYNC_TRIGGER_URL=${TRIGGER_URL_PLACEHOLDER}`
 	},
 	{
 		key: "CLIVLY_SYNC_TRIGGER_SECRET",

package/index.mjs

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
-import { t as CANONICAL_FIELDS } from "./entity-config-C8IIkrmX.mjs";
+import { t as CANONICAL_FIELDS } from "./entity-config-C4d3K9CP.mjs";
 import "./mapping-score-C68uAqGB.mjs";
 import { matchesConcept, requiredFieldsFor, scoreFieldMap, scoreRelationships, suggestCursorField, toFieldMap } from "./internal-entity-heuristics.mjs";
-import { s as isPlaceholderUrl, t as createClivlySDK } from "./sdk-CubnsyD9.mjs";
+import { s as isPlaceholderUrl, t as createClivlySDK } from "./sdk-D6gX0TXS.mjs";
 import { createRequire } from "node:module";
 import { spawn } from "node:child_process";
 import { accessSync, constants, existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, statSync, watch, writeFileSync } from "node:fs";
@@ -305,16 +305,27 @@ function addCommand(pm, packages) {
 */
 const CLIVLY_TICK_PATH = "/api/clivly/tick";
 /**
-* What `clivly login` writes into `.env` for `CLIVLY_SYNC_TRIGGER_URL`.
+* The example shape `clivly init`/`clivly login` show for `CLIVLY_SYNC_TRIGGER_URL`
+* — commented out in `.env`, not written as a live value.
 *
-* A visible template beats an empty line: the developer sees the shape they
-* eventually need — a public origin plus the tick path — instead of an unhelpful
-* `CLIVLY_SYNC_TRIGGER_URL=` they have to go and look up.
+* It used to be scaffolded uncommented, on the theory that a visible template
+* beats an empty line. The SDK's `isPlaceholderUrl` guard
+* (`@clivly/sdk/trigger-url`) already keeps a bracketed placeholder like this
+* one out of `syncTrigger.url` on every heartbeat — `resolveTriggerUrl` skips
+* it in favour of the dev tunnel / platform-derived origin — and
+* `assertSafeTriggerUrl` on the API side rejects it outright if it were ever
+* sent, since `<yourDomain.com>` doesn't even parse as a URL. So it was never
+* actually fireable (F-12 is not an SSRF/overwrite bug).
 *
-* It is safe to ship a truthy non-address here only because the SDK refuses to
-* treat a bracketed value as configured (`isPlaceholderUrl` in
-* `@clivly/sdk/trigger-url`). Change the brackets here and that guard stops
-* matching, so the two must stay in step.
+* What it DID break: readiness reporting only checked *presence*, not
+* validity, so an unedited placeholder made the dashboard's setup ladder and
+* `triggerConfigured` report "done" while `clivly status` correctly flagged
+* it — two surfaces disagreeing about a workspace that could never run "Run
+* sync". See `hasWorkingTrigger` (`@clivly.com/api/sync-trigger`) for the
+* fix. Commenting the example out here removes the confusing truthy-but-fake
+* value at the source, on top of that fix, while keeping the shape visible
+* for the (uncommon) case a developer needs it — most platforms are
+* auto-detected, and `clivly dev` handles local development.
 */
 const TRIGGER_URL_PLACEHOLDER = `https://<yourDomain.com>${CLIVLY_TICK_PATH}`;
 //#endregion
@@ -1439,8 +1450,8 @@ function credentialEntries(credentials) {
 		},
 		{
 			key: "CLIVLY_SYNC_TRIGGER_URL",
-			value: TRIGGER_URL_PLACEHOLDER,
-			comment: "Public URL of your tick route — replace <yourDomain.com> once deployed. Local dev: run `clivly dev` instead."
+			value: "",
+			comment: `Public URL of your tick route. Only needed when the platform's own URL can't be auto-detected (see \`clivly status\`). Uncomment and set your deployed origin if so:\n# CLIVLY_SYNC_TRIGGER_URL=${TRIGGER_URL_PLACEHOLDER}`
 		}
 	];
 }
@@ -1809,8 +1820,7 @@ const CLIVLY_ENV_KEYS = [
 	},
 	{
 		key: "CLIVLY_SYNC_TRIGGER_URL",
-		value: TRIGGER_URL_PLACEHOLDER,
-		comment: "Public URL of your tick route — replace <yourDomain.com> once deployed. Local dev: run `clivly dev` instead."
+		comment: `Public URL of your tick route. Only needed when the platform's own URL can't be auto-detected (see \`clivly status\`). Uncomment and set your deployed origin if so:\n# CLIVLY_SYNC_TRIGGER_URL=${TRIGGER_URL_PLACEHOLDER}`
 	},
 	{
 		key: "CLIVLY_SYNC_TRIGGER_SECRET",