@saptools/cf-hana 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.3.1 - 2026-07-01
+
+- Add actionable HANA LOB `ORDER BY`/`GROUP BY` hints that recommend removing LOB columns or wrapping them with `TO_VARCHAR(<column>)`.
+- Add invalid-column typo suggestions that inspect target table columns and print close matches to stderr.
+- Increase the default saved result TTL from 60 minutes to 7 days.
+
 ## 0.3.0 - 2026-07-01
 
 - Add invalid table/view recovery suggestions for failed `query` statements, with nearby table and view names printed to stderr so stdout remains parseable.

package/README.md

@@ -129,7 +129,7 @@ ID,CONTENT
 ```
 
 The ref is not a CSV column. Exact returned rows are stored under
-`~/.saptools/cf-hana/results/` for 60 minutes by default. Only returned rows are
+`~/.saptools/cf-hana/results/` for 7 days by default. Only returned rows are
 stored; rows beyond the selected `--limit` are not fetched or saved.
 
 Follow-up commands:

package/dist/cli.js

@@ -26,12 +26,36 @@ var CredentialsNotFoundError = class extends CfHanaError {
 };
 var QueryError = class extends CfHanaError {
   sqlState;
+  databaseCode;
   constructor(message, options) {
     super("QUERY", message, options);
     this.name = "QueryError";
     this.sqlState = options?.sqlState;
+    this.databaseCode = options?.databaseCode;
   }
 };
+function isDatabaseErrorShape(error) {
+  return typeof error === "object" && error !== null && "code" in error;
+}
+function safeDatabaseCode(value) {
+  if (typeof value === "number" && Number.isSafeInteger(value)) {
+    return value;
+  }
+  if (typeof value !== "string" || !/^\d+$/.test(value)) {
+    return void 0;
+  }
+  const parsed = Number(value);
+  return Number.isSafeInteger(parsed) ? parsed : void 0;
+}
+function databaseCode(error) {
+  if (error instanceof QueryError) {
+    return error.databaseCode ?? databaseCode(error.cause);
+  }
+  if (isDatabaseErrorShape(error)) {
+    return safeDatabaseCode(error.code);
+  }
+  return void 0;
+}
 var ReadOnlyViolationError = class extends CfHanaError {
   constructor(message, options) {
     super("READ_ONLY_VIOLATION", message, options);
@@ -674,7 +698,7 @@ async function listColumns(connection, schema, table) {
 
 // src/config.ts
 var CLI_NAME = "cf-hana";
-var CLI_VERSION = "0.3.0";
+var CLI_VERSION = "0.3.1";
 var ENV_PREFIX = "CF_HANA";
 var DEFAULT_QUERY_TIMEOUT_MS = 3e4;
 var DEFAULT_CONNECT_TIMEOUT_MS = 3e4;
@@ -683,7 +707,7 @@ var DEFAULT_POOL_IDLE_MS = 6e4;
 var DEFAULT_AUTO_LIMIT = 100;
 var DEFAULT_CELL_LIMIT = 128;
 var MAX_CELL_LIMIT = 1e4;
-var DEFAULT_RESULT_TTL_MINUTES = 60;
+var DEFAULT_RESULT_TTL_MINUTES = 10080;
 var DEFAULT_RESULT_SEARCH_LIMIT = 20;
 var MAX_RESULT_STORE_BYTES = 256 * 1024 * 1024;
 function envName(suffix) {
@@ -1142,7 +1166,15 @@ function catalogObjects() {
     { SCHEMA_NAME: "APP_SCHEMA", OBJECT_NAME: "EXISTING_TABLE", OBJECT_TYPE: "TABLE" },
     { SCHEMA_NAME: "APP_SCHEMA", OBJECT_NAME: "MISSING_TABLE_FIXED", OBJECT_TYPE: "TABLE" },
     { SCHEMA_NAME: "APP_SCHEMA", OBJECT_NAME: "MISSING_TABLE_VIEW", OBJECT_TYPE: "VIEW" },
-    { SCHEMA_NAME: "APP_SCHEMA", OBJECT_NAME: "STATUS_ITEMS", OBJECT_TYPE: "TABLE" }
+    { SCHEMA_NAME: "APP_SCHEMA", OBJECT_NAME: "STATUS_ITEMS", OBJECT_TYPE: "TABLE" },
+    { SCHEMA_NAME: "APP_SCHEMA", OBJECT_NAME: "CORE_AUTH_SCOPE", OBJECT_TYPE: "TABLE" }
+  ];
+}
+function tableColumns() {
+  return [
+    { COLUMN_NAME: "ID", DATA_TYPE_NAME: "INTEGER", LENGTH: null, SCALE: null, IS_NULLABLE: "FALSE", POSITION: 1 },
+    { COLUMN_NAME: "IS_ACTIVE", DATA_TYPE_NAME: "BOOLEAN", LENGTH: null, SCALE: null, IS_NULLABLE: "TRUE", POSITION: 2 },
+    { COLUMN_NAME: "SCOPE_NAME", DATA_TYPE_NAME: "NVARCHAR", LENGTH: 255, SCALE: null, IS_NULLABLE: "TRUE", POSITION: 3 }
   ];
 }
 function fakeExec(sql) {
@@ -1167,9 +1199,32 @@ function fakeExec(sql) {
       affectedRows: 0
     };
   }
+  if (upperSql.includes("SYS.TABLE_COLUMNS")) {
+    return {
+      rows: tableColumns(),
+      columns: [
+        { name: "COLUMN_NAME", typeName: "NVARCHAR" },
+        { name: "DATA_TYPE_NAME", typeName: "NVARCHAR" },
+        { name: "LENGTH", typeName: "INTEGER" },
+        { name: "SCALE", typeName: "INTEGER" },
+        { name: "IS_NULLABLE", typeName: "NVARCHAR" },
+        { name: "POSITION", typeName: "INTEGER" }
+      ],
+      affectedRows: 0
+    };
+  }
   if (upperSql.includes("MISSING_TABLE") || upperSql.includes("MISSING_TABLES")) {
     throw new QueryError("invalid table name: MISSING_TABLE", { sqlState: "42S02" });
   }
+  if (upperSql.includes("ISACTIVE")) {
+    throw new QueryError("invalid column name: ISACTIVE: line 1 col 8 (at pos 7)", { databaseCode: 260 });
+  }
+  if (upperSql.includes("LOB_ORDER_ERROR")) {
+    throw new QueryError("inconsistent datatype: LOB type is not allowed in ORDER BY clause", { databaseCode: 266 });
+  }
+  if (upperSql.includes("LOB_GROUP_ERROR")) {
+    throw new QueryError("inconsistent datatype: LOB type is not allowed in GROUP BY clause", { databaseCode: 274 });
+  }
   if (sql.toUpperCase().includes("LOB_FIXTURE")) {
     return {
       rows: [
@@ -1294,7 +1349,12 @@ function extractSqlState(error) {
 }
 function toQueryError(error) {
   const sqlState = extractSqlState(error);
-  return sqlState === void 0 ? new QueryError(error.message, { cause: error }) : new QueryError(error.message, { cause: error, sqlState });
+  const code = databaseCode(error);
+  return new QueryError(error.message, {
+    cause: error,
+    ...sqlState === void 0 ? {} : { sqlState },
+    ...code === void 0 ? {} : { databaseCode: code }
+  });
 }
 function toColumns(metadata) {
   if (metadata === void 0) {
@@ -3375,6 +3435,14 @@ function score(requested, candidate) {
   }
   return value;
 }
+function extractInvalidColumnNameFromError(error) {
+  if (!(error instanceof QueryError)) {
+    return void 0;
+  }
+  const match = /invalid column name:\s*([^:]+)/i.exec(error.message);
+  const name = match?.[1]?.trim();
+  return name === void 0 || name.length === 0 ? void 0 : name;
+}
 function rankCatalogSuggestions(requested, candidates, limit = 5) {
   return candidates.map((candidate) => ({ candidate, score: score(requested, candidate) })).filter((item) => item.score >= 45).sort(
     (left, right) => right.score - left.score || left.candidate.schema.localeCompare(right.candidate.schema) || (left.candidate.type === right.candidate.type ? 0 : left.candidate.type === "TABLE" ? -1 : 1) || left.candidate.name.localeCompare(right.candidate.name)
@@ -3389,6 +3457,24 @@ function formatSuggestions(suggestions) {
     ...suggestions.map((item) => `  ${item.schema}.${item.name} (${item.type})`)
   ].join("\n");
 }
+function rankNameSuggestions(requested, candidates, limit = 5) {
+  const requestedName = { name: requested };
+  return candidates.map((candidate) => ({
+    candidate,
+    score: score(requestedName, { schema: "", name: candidate.name, type: "TABLE" })
+  })).filter((item) => item.score >= 45).sort(
+    (left, right) => right.score - left.score || left.candidate.name.localeCompare(right.candidate.name)
+  ).slice(0, limit).map((item) => item.candidate);
+}
+function formatColumnSuggestions(suggestions) {
+  if (suggestions.length === 0) {
+    return void 0;
+  }
+  return [
+    "Did you mean column:",
+    ...suggestions.map((item) => `  ${item.name}`)
+  ].join("\n");
+}
 
 // src/cli.ts
 function print2(text) {
@@ -3536,13 +3622,50 @@ async function loadSuggestionCatalogObjects(client, refresh) {
     return await client.listCatalogObjects(client.info.schema);
   }
 }
-async function enrichAndRethrowQueryError(error, client, sql, refresh) {
+function isLobSortOrGroupError(error) {
+  const code = databaseCode(error);
+  if (code !== 266 && code !== 274) {
+    return false;
+  }
+  return error instanceof QueryError && /LOB type is not allowed in (?:ORDER BY|GROUP BY) clause/i.test(error.message);
+}
+function printLobSortOrGroupHint() {
+  const lines = [
+    `${CLI_NAME}: HANA cannot ORDER BY or GROUP BY NCLOB/CLOB/BLOB columns directly.`,
+    `${CLI_NAME}: Remove the LOB column from ORDER BY/GROUP BY or wrap it as TO_VARCHAR(<column>).`
+  ];
+  process.stderr.write(`${lines.join("\n")}
+`);
+}
+async function printColumnSuggestions(error, client, sql) {
+  if (databaseCode(error) !== 260) {
+    return;
+  }
+  const columnName = extractInvalidColumnNameFromError(error);
+  const tableName = extractMissingObjectName(sql);
+  if (columnName === void 0 || tableName === void 0) {
+    return;
+  }
+  try {
+    const columns = await client.listColumns(
+      tableName.schema ?? client.info.schema,
+      tableName.name
+    );
+    const text = formatColumnSuggestions(rankNameSuggestions(columnName, columns));
+    if (text !== void 0) {
+      process.stderr.write(`${text}
+`);
+    }
+  } catch {
+  }
+}
+async function printCatalogObjectSuggestions(error, client, sql, refresh) {
   if (!isInvalidCatalogObjectError(error)) {
-    throw error;
+    return;
   }
   const requested = extractMissingObjectNameFromError(error) ?? extractMissingObjectName(sql);
   if (requested === void 0) {
-    throw error;
+    return;
   }
   try {
     const objects = await loadSuggestionCatalogObjects(client, refresh);
@@ -3553,6 +3676,14 @@ async function enrichAndRethrowQueryError(error, client, sql, refresh) {
     }
   } catch {
   }
+}
+async function enrichAndRethrowQueryError(error, client, sql, refresh) {
+  if (isLobSortOrGroupError(error)) {
+    printLobSortOrGroupHint();
+    throw error;
+  }
+  await printColumnSuggestions(error, client, sql);
+  await printCatalogObjectSuggestions(error, client, sql, refresh);
   throw error;
 }
 async function runQuery(selector, sql, command) {