feat(sql-orm-client): require junction namespaceId on the include through descriptor

The contract now carries the junction's namespaceId on through (slice 0);
the include descriptor passes it through as a required field instead of
re-deriving it from storage. Re-emit the M:N fixture so the User<->Tag
through block carries it.

Signed-off-by: Alexey Orlenko's AI Agent <robot@aqrln.net>

Author: tensordreams

packages/3-extensions/sql-orm-client/src/collection-contract.ts

@@ -250,13 +250,11 @@ export function resolveIncludeRelation(
     );
     through = {
       table: relation.through.table,
+      namespaceId: relation.through.namespaceId,
       parentColumns: relation.through.parentColumns,
       childColumns: relation.through.childColumns,
       targetColumns: relation.through.targetColumns,
       parentLocalColumns,
-      ...(relation.through.namespaceId !== undefined
-        ? { namespaceId: relation.through.namespaceId }
-        : {}),
     };
   }
 

packages/3-extensions/sql-orm-client/src/types.ts

@@ -52,6 +52,8 @@ export interface IncludeCombine<ResultShape extends Record<string, unknown>>
 
 export interface IncludeThroughDescriptor {
   readonly table: string;
+  /** Namespace the junction table lives in, as declared in the contract. */
+  readonly namespaceId: string;
   /** FK columns in the junction table that point to the parent. */
   readonly parentColumns: readonly string[];
   /** FK columns in the junction table that point to the target (child). */
@@ -60,8 +62,6 @@ export interface IncludeThroughDescriptor {
   readonly targetColumns: readonly string[];
   /** Resolved column names in the parent table that junction.parentColumns reference. */
   readonly parentLocalColumns: readonly string[];
-  /** Namespace of the junction table, resolved from storage. */
-  readonly namespaceId?: string;
 }
 
 export interface IncludeExpr {

packages/3-extensions/sql-orm-client/test/query-plan-select.test.ts

@@ -638,6 +638,7 @@ describe('M:N include correlated subquery', () => {
       cardinality: 'N:M',
       through: {
         table: opts.junctionTable,
+        namespaceId: 'public',
         parentColumns: opts.parentColumns,
         childColumns: opts.childColumns,
         targetColumns: opts.targetColumns,
@@ -912,6 +913,7 @@ describe('M:N include correlated subquery', () => {
       cardinality: 'N:M',
       through: {
         table: 'parent_child',
+        namespaceId: 'public',
         parentColumns: ['parent_id'],
         childColumns: ['child_id'],
         targetColumns: ['id'],