@rex0220/kintone-sql-tools 3.56.0 → 3.56.1

package/dist-cli/ksql.js

@@ -17978,9 +17978,29 @@ async function executeSelect(stmt, client, options, cacheContext, cteCache, capt
   }
   return mergeSelectWarnings(result, [...defaultRangeWarnings, ...subtableFieldWarnings]);
 }
+function subtableGroupingAdvice(fieldName, owners) {
+  if (owners.length === 1) {
+    const { appId, owner } = owners[0];
+    return `ArgumentError: ${fieldName} \u306F\u30B5\u30D6\u30C6\u30FC\u30D6\u30EB\u300C${owner}\u300D\uFF08APP${appId}\uFF09\u306E\u4E2D\u306E\u9805\u76EE\u3067\u3059\u3002\u89AA\u30C6\u30FC\u30D6\u30EB\u306E GROUP BY \u306B\u306F\u6307\u5B9A\u3067\u304D\u307E\u305B\u3093\u3002APP${appId}$${owner} \u304B\u3089\u96C6\u8A08\u3057\u3066\u304F\u3060\u3055\u3044\uFF08\u89AA\u306E\u30EC\u30B3\u30FC\u30C9 ID \u306F _pid\u3001\u89AA\u9805\u76EE\u306F _p.<\u30D5\u30A3\u30FC\u30EB\u30C9\u30B3\u30FC\u30C9> \u306B\u306A\u308A\u307E\u3059\uFF09\u3002`;
+  }
+  const list = owners.map(({ appId, owner }) => `APP${appId}$${owner}`).join(" / ");
+  return `ArgumentError: ${fieldName} \u306F\u30B5\u30D6\u30C6\u30FC\u30D6\u30EB\u306E\u4E2D\u306E\u9805\u76EE\u3067\u3059\uFF08${list}\uFF09\u3002\u89AA\u30C6\u30FC\u30D6\u30EB\u306E GROUP BY \u306B\u306F\u6307\u5B9A\u3067\u304D\u307E\u305B\u3093\u3002\u3069\u306E\u30B5\u30D6\u30C6\u30FC\u30D6\u30EB\u304B\u3089\u96C6\u8A08\u3059\u308B\u304B\u3092 FROM \u3067\u6307\u5B9A\u3057\u3066\u304F\u3060\u3055\u3044\u3002`;
+}
+function dedupeSubtableOwners(owners) {
+  const seen = /* @__PURE__ */ new Set();
+  const result = [];
+  for (const entry of owners) {
+    const key = `${entry.appId}\0${entry.owner}`;
+    if (seen.has(key)) continue;
+    seen.add(key);
+    result.push(entry);
+  }
+  return result;
+}
 async function buildRuntimePlainGroupByPlan(stmt, client, cacheContext, materializedTables) {
   if (stmt.groupBy.length === 0) return void 0;
   const sources = [stmt.from, ...stmt.joins.map((join2) => join2.table)];
+  const subtableOwners = /* @__PURE__ */ new Map();
   const inputs = await Promise.all(sources.map(async (source) => {
     if (source.cteName !== null) {
       const materialized = materializedTables?.get(source.cteName);
@@ -17991,6 +18011,12 @@ async function buildRuntimePlainGroupByPlan(stmt, client, cacheContext, material
     }
     const fields = await getFieldsCached(source.appId, client, cacheContext);
     if (!source.subtableCode) {
+      for (const field of fields) {
+        if (field.inSubtable !== true || !field.subtableCode) continue;
+        const entries = subtableOwners.get(field.code) ?? [];
+        entries.push({ appId: source.appId, owner: field.subtableCode });
+        subtableOwners.set(field.code, dedupeSubtableOwners(entries));
+      }
       return {
         kind: "APP",
         fieldCodes: fields.filter((field) => !field.inSubtable).map((field) => field.code)
@@ -18012,10 +18038,10 @@ async function buildRuntimePlainGroupByPlan(stmt, client, cacheContext, material
   );
   const groupBy = stmt.groupBy;
   const plan = planPlainGroupByResolution(groupBy, stmt.columns, schemas);
-  assertRuntimePlainGroupByPlan(stmt, groupBy, plan);
+  assertRuntimePlainGroupByPlan(stmt, groupBy, plan, subtableOwners);
   return plan;
 }
-function assertRuntimePlainGroupByPlan(stmt, groupBy, plan) {
+function assertRuntimePlainGroupByPlan(stmt, groupBy, plan, subtableOwners) {
   const physicalAppIds = [
     stmt.from,
     ...stmt.joins.map((join2) => join2.table)
@@ -18028,6 +18054,12 @@ function assertRuntimePlainGroupByPlan(stmt, groupBy, plan) {
     const name = key?.type === "FIELD_NAME" ? key.name : "(expression)";
     if (item.kind === "UNKNOWN") {
       const appSuffix = primaryPhysicalAppId === null ? "" : ` (APP${primaryPhysicalAppId})`;
+      const dot = item.name.indexOf(".");
+      const bare = dot >= 0 ? item.name.slice(dot + 1) : item.name;
+      const owners = subtableOwners.get(item.name) ?? subtableOwners.get(bare) ?? [];
+      if (owners.length > 0) {
+        throw new Error(subtableGroupingAdvice(item.name, owners));
+      }
       throw new Error(`ArgumentError: unknown field code(s): ${item.name}${appSuffix}`);
     }
     if (item.kind === "ALIAS_REJECT") {
@@ -18078,9 +18110,15 @@ async function buildGroupingFieldResolver(stmt, client, cacheContext, materializ
       return [appId, new Map(infos.map((info) => [info.code, info]))];
     }))
   );
+  const subtableFieldOf = (table, code) => {
+    if (table.subtableCode) return null;
+    const info = infosByApp.get(table.appId)?.get(code);
+    return info?.inSubtable === true ? info.subtableCode ?? "" : null;
+  };
   const physicalMatch = (table, field) => {
     if (field === "$id") return "$id";
     const code = fieldCodeForTypeLookup(table, field);
+    if (subtableFieldOf(table, code) !== null) return null;
     return infosByApp.get(table.appId)?.has(code) ? code : null;
   };
   const materializedHas = (table, field) => {
@@ -18115,6 +18153,10 @@ async function buildGroupingFieldResolver(stmt, client, cacheContext, materializ
       }
       const code = physicalMatch(table, field.field);
       if (code === null) {
+        const owner = subtableFieldOf(table, fieldCodeForTypeLookup(table, field.field));
+        if (owner !== null && owner !== "") {
+          throw new Error(subtableGroupingAdvice(field.field, [{ appId: table.appId, owner }]));
+        }
         throw new Error(`ArgumentError: B65 field ${field.tableAlias}.${field.field} does not exist in APP${table.appId}.`);
       }
       return resolved(table, tableIndex, field, code);
@@ -18137,6 +18179,15 @@ async function buildGroupingFieldResolver(stmt, client, cacheContext, materializ
         `ArgumentError: B65 field ${field.field} resolves to a materialized CTE/temp column; physical APP fields are required.`
       );
     }
+    const owners = tables.flatMap((table) => {
+      if (table.cteName !== null) return [];
+      const owner = subtableFieldOf(table, fieldCodeForTypeLookup(table, field.field));
+      return owner === null || owner === "" ? [] : [{ appId: table.appId, owner }];
+    });
+    const uniqueOwners = dedupeSubtableOwners(owners);
+    if (uniqueOwners.length > 0) {
+      throw new Error(subtableGroupingAdvice(field.field, uniqueOwners));
+    }
     throw new Error(`ArgumentError: B65 field ${field.field} does not exist in a physical APP source.`);
   };
 }