@query-doctor/core 0.0.4 → 0.1.0

package/dist/index.cjs

@@ -860,6 +860,10 @@ var _PgIdentifier = class _PgIdentifier {
     this.value = value;
     this.quoted = quoted;
   }
+  /**
+   * Constructs an identifier from a single part (column or table name).
+   * When quoting identifiers like `select table.col` use {@link fromParts} instead
+   */
   static fromString(identifier) {
     const identifierRegex = /^[a-z_][a-zA-Z0-9_]*$/;
     const match = identifier.match(/^"(.+)"$/);
@@ -869,6 +873,18 @@ var _PgIdentifier = class _PgIdentifier {
     const quoted = !identifierRegex.test(identifier) || this.reservedKeywords.has(identifier.toLowerCase());
     return new _PgIdentifier(identifier, quoted);
   }
+  /**
+   * Quotes parts of an identifier like `select schema.table.col`.
+   * A separate function is necessary because postgres will treat
+   * `select "HELLO.WORLD"` as a column name. It has to be like
+   * `select "HELLO"."WORLD"` instead.
+   */
+  static fromParts(...identifiers) {
+    return new _PgIdentifier(
+      identifiers.map((identifier) => _PgIdentifier.fromString(identifier)).join("."),
+      false
+    );
+  }
   toString() {
     if (this.quoted) {
       return `"${this.value.replace(/"/g, '""')}"`;
@@ -876,36 +892,24 @@ var _PgIdentifier = class _PgIdentifier {
     return this.value;
   }
 };
+// Every keyword that's not explicitly marked as
+// unreserved in src/include/parser/kwlist.h
 __publicField(_PgIdentifier, "reservedKeywords", /* @__PURE__ */ new Set([
-  "abort",
-  "analyze",
-  "binary",
-  "cluster",
-  "constraint",
-  "copy",
-  "do",
-  "explain",
-  "extend",
-  "listen",
-  "load",
-  "lock",
-  "move",
-  "new",
-  "none",
-  "notify",
-  "offset",
-  "reset",
-  "setof",
-  "show",
-  "unlisten",
-  "until",
-  "vacuum",
-  "verbose",
   "all",
+  "analyse",
+  "analyze",
+  "and",
   "any",
+  "array",
+  "as",
   "asc",
+  "asymmetric",
+  "authorization",
   "between",
+  "bigint",
+  "binary",
   "bit",
+  "boolean",
   "both",
   "case",
   "cast",
@@ -914,76 +918,147 @@ __publicField(_PgIdentifier, "reservedKeywords", /* @__PURE__ */ new Set([
   "check",
   "coalesce",
   "collate",
+  "collation",
   "column",
+  "concurrently",
   "constraint",
+  "create",
   "cross",
-  "current",
+  "current_catalog",
   "current_date",
+  "current_role",
+  "current_schema",
   "current_time",
   "current_timestamp",
   "current_user",
   "dec",
   "decimal",
   "default",
+  "deferrable",
   "desc",
   "distinct",
+  "do",
   "else",
   "end",
   "except",
   "exists",
   "extract",
   "false",
+  "fetch",
   "float",
   "for",
   "foreign",
+  "freeze",
   "from",
   "full",
-  "global",
+  "grant",
+  "greatest",
   "group",
+  "grouping",
   "having",
+  "ilike",
   "in",
+  "initially",
   "inner",
+  "inout",
+  "int",
+  "integer",
   "intersect",
+  "interval",
   "into",
   "is",
+  "isnull",
   "join",
+  "json",
+  "json_array",
+  "json_arrayagg",
+  "json_exists",
+  "json_object",
+  "json_objectagg",
+  "json_query",
+  "json_scalar",
+  "json_serialize",
+  "json_table",
+  "json_value",
+  "lateral",
   "leading",
+  "least",
   "left",
   "like",
-  "local",
+  "limit",
+  "localtime",
+  "localtimestamp",
+  "merge_action",
+  "national",
   "natural",
   "nchar",
+  "none",
+  "normalize",
   "not",
+  "notnull",
   "null",
   "nullif",
   "numeric",
+  "offset",
   "on",
+  "only",
   "or",
   "order",
+  "out",
   "outer",
   "overlaps",
+  "overlay",
+  "placing",
   "position",
   "precision",
   "primary",
-  "public",
+  "real",
   "references",
+  "returning",
   "right",
+  "row",
   "select",
   "session_user",
+  "setof",
+  "similar",
+  "smallint",
   "some",
   "substring",
+  "symmetric",
+  "system_user",
   "table",
+  "tablesample",
   "then",
+  "time",
+  "timestamp",
   "to",
-  "transaction",
+  "trailing",
+  "treat",
   "trim",
   "true",
   "union",
   "unique",
   "user",
+  "using",
+  "values",
   "varchar",
+  "variadic",
+  "verbose",
   "when",
-  "where"
+  "where",
+  "window",
+  "with",
+  "xmlattributes",
+  "xmlconcat",
+  "xmlelement",
+  "xmlexists",
+  "xmlforest",
+  "xmlnamespaces",
+  "xmlparse",
+  "xmlpi",
+  "xmlroot",
+  "xmlserialize",
+  "xmltable"
 ]));
 var PgIdentifier = _PgIdentifier;
 
@@ -1130,7 +1205,10 @@ var _IndexOptimizer = class _IndexOptimizer {
       if (!isIndexProbablyDroppable(index)) {
         continue;
       }
-      const indexName = `${index.schema_name}.${index.index_name}`;
+      const indexName = PgIdentifier.fromParts(
+        index.schema_name,
+        index.index_name
+      );
       await dropIndex(tx, indexName);
     }
   }
@@ -1181,9 +1259,12 @@ var _IndexOptimizer = class _IndexOptimizer {
       await this.db.transaction(async (tx) => {
         await f?.(tx);
         await this.statistics.restoreStats(tx);
-        const flags = ["format json", "trace"];
+        const flags = ["format json"];
         if (options && !options.genericPlan) {
           flags.push("analyze");
+          if (this.config.trace) {
+            flags.push("trace");
+          }
         } else {
           flags.push("generic_plan");
         }