@deephaven-enterprise/query-utils 2026.1.54 → 2026.1.57

package/dist/ArgumentUtils.d.ts

@@ -11,6 +11,10 @@
  * @returns An array of tokens. Returns an empty array for empty/whitespace input.
  */
 export declare function tokenizeArgs(args: string): string[];
+/** Removes blank list entries while preserving every nonblank value verbatim. */
+export declare function removeBlankEntries(entries: readonly string[]): string[];
+/** Removes fully blank name/value pairs while preserving nonblank pairs. */
+export declare function removeBlankEnvVarEntries(entries: readonly string[]): string[];
 /**
  * Joins a flat alternating key/value array into a space-separated
  * `KEY=VALUE` string. Values are shell-quoted as needed so the output can be
@@ -19,7 +23,7 @@ export declare function tokenizeArgs(args: string): string[];
  * @param entries Flat alternating array: `["KEY1", "VALUE1", "KEY2", "VALUE2", …]`
  * @returns Shell-safe string: `"KEY1=VALUE1 KEY2='value with spaces'"`
  */
-export declare function joinEnvVarEntries(entries: string[]): string;
+export declare function joinEnvVarEntries(entries: readonly string[]): string;
 /**
  * Parses a space-separated `KEY=VALUE` string into a flat alternating
  * key/value array. Uses shell quoting rules so that quoted spans are
@@ -33,13 +37,15 @@ export declare function joinEnvVarEntries(entries: string[]): string;
  * @throws {Error} If any token has an empty key or is a shell operator/glob.
  */
 export declare function splitEnvVarEntries(envVars: string): string[];
+/** Returns an error for invalid environment variable entries, or null. */
+export declare function getEnvVarEntriesValidationError(entries: readonly string[]): string | null;
 /**
  * Joins an array of classpath entries into a colon-separated string.
  *
  * @param entries Array of classpath entries, one path per element.
  * @returns Colon-separated string: `"path1:path2:path3"`
  */
-export declare function joinClasspathAdditions(entries: string[]): string;
+export declare function joinClasspathAdditions(entries: readonly string[]): string;
 /**
  * Splits a colon-separated classpath string into an array of entries,
  * filtering out any empty segments.

package/dist/ArgumentUtils.js

@@ -15,6 +15,23 @@ export function tokenizeArgs(args) {
     var _a;
     return (_a = args.match(/(?:[^\s"']+|"[^"]*"|'[^']*'|["'])+/g)) !== null && _a !== void 0 ? _a : [];
 }
+/** Removes blank list entries while preserving every nonblank value verbatim. */
+export function removeBlankEntries(entries) {
+    return entries.filter(entry => entry.trim() !== '');
+}
+/** Removes fully blank name/value pairs while preserving nonblank pairs. */
+export function removeBlankEnvVarEntries(entries) {
+    var _a, _b;
+    const result = [];
+    for (let index = 0; index < entries.length; index += 2) {
+        const name = (_a = entries[index]) !== null && _a !== void 0 ? _a : '';
+        const value = (_b = entries[index + 1]) !== null && _b !== void 0 ? _b : '';
+        if (name.trim() !== '' || value.trim() !== '') {
+            result.push(name, value);
+        }
+    }
+    return result;
+}
 /**
  * Joins a flat alternating key/value array into a space-separated
  * `KEY=VALUE` string. Values are shell-quoted as needed so the output can be
@@ -28,8 +45,9 @@ export function joinEnvVarEntries(entries) {
     for (let i = 0; i + 1 < entries.length; i += 2) {
         const key = entries[i];
         const value = entries[i + 1];
+        const quotedKey = ShellQuote.quote([key]);
         const quotedValue = ShellQuote.quote([value]);
-        pairs.push(`${key}=${quotedValue}`);
+        pairs.push(`${quotedKey}=${quotedValue}`);
     }
     return pairs.join(' ');
 }
@@ -66,6 +84,27 @@ export function splitEnvVarEntries(envVars) {
     }
     return entries;
 }
+/** Returns an error for invalid environment variable entries, or null. */
+export function getEnvVarEntriesValidationError(entries) {
+    const seenNames = new Set();
+    for (let i = 0; i + 1 < entries.length; i += 2) {
+        const name = entries[i].trim();
+        const value = entries[i + 1];
+        if (name === '' && value !== '') {
+            return 'Environment variable name cannot be empty';
+        }
+        if (name.includes('=')) {
+            return `Environment variable name cannot contain "=": ${name}`;
+        }
+        if (name !== '' && seenNames.has(name)) {
+            return `Duplicate environment variable name: ${name}`;
+        }
+        if (name !== '') {
+            seenNames.add(name);
+        }
+    }
+    return null;
+}
 /**
  * Joins an array of classpath entries into a colon-separated string.
  *

package/dist/QueryScheduler.d.ts

@@ -273,6 +273,21 @@ export declare class QueryScheduler {
     set stopTime(value: number);
     get overnight(): boolean;
     set overnight(value: boolean);
+    /**
+     * Reconciles the base times with the overnight flag when delay-start (useMinStartTime) is
+     * enabled on a dependent schedule. While delay-start was off, overnight was a free manual flag
+     * and there was no start endpoint; once it becomes live the previously unused start time may be
+     * inconsistent with the ordering that validation now enforces.
+     *
+     * The start time is the field the delay-start checkbox governs, so it is adjusted in preference
+     * to the user's chosen End Time and overnight flag:
+     * - Non-overnight requires start < stop: if the stale start is not before the End Time, snap it
+     *   to midnight, preserving the End Time and the overnight flag.
+     * - Overnight requires start > stop: there is no clean single start value that keeps the End
+     *   Time, so fall back to swapping the two real clock values (the only sensible overnight
+     *   window). This is the one case that changes the End Time.
+     */
+    alignTimesForDelayStart(): void;
     get stopTimeDisabled(): boolean;
     set stopTimeDisabled(value: boolean);
     /**

package/dist/QueryScheduler.js

@@ -159,6 +159,13 @@ export class QueryScheduler {
             this.overnightInternal = false;
             return;
         }
+        // Dependent scheduling only has a start endpoint when delay-start (useMinStartTime) is enabled.
+        // While it is disabled there is no ordering constraint at all — both this client validator
+        // and the controller skip start/stop ordering — so overnight is a free manual flag and must
+        // not be auto-derived from the (unused) base times.
+        if (type === QueryScheduler.TYPES.DEPENDENT && !this.useMinStartTime) {
+            return;
+        }
         if (!overnightInternal && stopTimeInternal <= startTimeInternal) {
             this.overnightInternal = true;
         }
@@ -186,12 +193,45 @@ export class QueryScheduler {
     set overnight(value) {
         const { startTimeInternal, stopTimeInternal } = this;
         this.overnightInternal = value;
+        // Dependent scheduling only has a start endpoint when delay-start (useMinStartTime) is enabled.
+        // While it is disabled there is no start time to order against, so overnight is a free manual
+        // flag and toggling it must not swap the base times. The times are reconciled later, if and
+        // when delay-start is enabled, via alignTimesForDelayStart().
+        if (this.type === QueryScheduler.TYPES.DEPENDENT && !this.useMinStartTime) {
+            return;
+        }
         if ((!value && stopTimeInternal < startTimeInternal) ||
             (value && startTimeInternal < stopTimeInternal)) {
             this.startTimeInternal = stopTimeInternal;
             this.stopTimeInternal = startTimeInternal;
         }
     }
+    /**
+     * Reconciles the base times with the overnight flag when delay-start (useMinStartTime) is
+     * enabled on a dependent schedule. While delay-start was off, overnight was a free manual flag
+     * and there was no start endpoint; once it becomes live the previously unused start time may be
+     * inconsistent with the ordering that validation now enforces.
+     *
+     * The start time is the field the delay-start checkbox governs, so it is adjusted in preference
+     * to the user's chosen End Time and overnight flag:
+     * - Non-overnight requires start < stop: if the stale start is not before the End Time, snap it
+     *   to midnight, preserving the End Time and the overnight flag.
+     * - Overnight requires start > stop: there is no clean single start value that keeps the End
+     *   Time, so fall back to swapping the two real clock values (the only sensible overnight
+     *   window). This is the one case that changes the End Time.
+     */
+    alignTimesForDelayStart() {
+        const { overnightInternal, startTimeInternal, stopTimeInternal } = this;
+        if (!overnightInternal) {
+            if (startTimeInternal >= stopTimeInternal) {
+                this.startTimeInternal = QueryScheduler.parseLocalTime('00:00:00');
+            }
+        }
+        else if (startTimeInternal <= stopTimeInternal) {
+            this.startTimeInternal = stopTimeInternal;
+            this.stopTimeInternal = startTimeInternal;
+        }
+    }
     get stopTimeDisabled() {
         return this.stopTimeDisabledInternal;
     }

package/dist/QueryUtils.d.ts

@@ -29,6 +29,7 @@ export declare const QUERY_CONFIG_TABLE: "QueryInfo";
 export declare const SCOPE_CONFIG_TABLE: "ScopeNamesAndType";
 export declare const CATALOG_TABLE: "catalog";
 export declare const FACTORY_SERVICE_TABLES: Set<"QueryInfo" | "ScopeNamesAndType" | "catalog">;
+export type FactoryServiceTableName = typeof FACTORY_SERVICE_TABLES extends Set<infer T> ? T : never;
 export type TemporaryDraftQueryConfig = Omit<DraftQueryConstructorObject, 'scheduling' | 'workerKind' | 'type'> & {
     type: string;
     engine: string;
@@ -172,7 +173,7 @@ export declare function hasCommunitySupport(queryConfigurationTypes: ReadonlyMap
  * @param tableName the name of the table
  * @returns true if the table is a factory service table, false otherwise
  */
-export declare function isFactoryServiceTable(tableName: string): boolean;
+export declare function isFactoryServiceTable(tableName: string): tableName is FactoryServiceTableName;
 /**
  * Turns a URI into a QueryVariableDescriptor
  * @param params Function parameters

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deephaven-enterprise/query-utils",
-  "version": "2026.1.54",
+  "version": "2026.1.57",
   "description": "Deephaven Enterprise Query Utils",
   "author": "Deephaven Data Labs LLC",
   "license": "SEE LICENSE IN LICENSE.md",
@@ -17,11 +17,11 @@
     "build": "tsc --build"
   },
   "dependencies": {
-    "@deephaven-enterprise/client-utils": "2026.1.54",
-    "@deephaven-enterprise/jsapi-types": "2026.1.54",
+    "@deephaven-enterprise/client-utils": "2026.1.57",
+    "@deephaven-enterprise/jsapi-types": "2026.1.57",
     "@deephaven/jsapi-types": "1.0.0-dev0.40.5",
-    "@deephaven/log": "^0.97.0",
-    "@deephaven/utils": "^0.97.0",
+    "@deephaven/log": "^1.8.0",
+    "@deephaven/utils": "^1.29.0",
     "@internationalized/date": "^3.12.2",
     "nanoid": "^5.1.6",
     "shell-quote": "^1.8.4"