windmill-components 1.677.0 → 1.687.0

package/package/githubApp.js

@@ -94,7 +94,11 @@ export async function loadGithubInstallations(state, currentWorkspace) {
                 const hostname = new URL(ghesBaseUrl).hostname;
                 const isGHES = hostname !== 'github.com' && !hostname.endsWith('.ghe.com');
                 const appsPath = isGHES ? 'github-apps' : 'apps';
-                state.githubInstallationUrl = `${ghesBaseUrl}/${appsPath}/${ghesConfig.app_slug}/installations/new?state=${stateParam}`;
+                // GHE Cloud (*.ghe.com) requires the app owner (org/user) in the URL path
+                const appPath = hostname.endsWith('.ghe.com') && ghesConfig.app_owner
+                    ? `${ghesConfig.app_owner}/${ghesConfig.app_slug}`
+                    : ghesConfig.app_slug;
+                state.githubInstallationUrl = `${ghesBaseUrl}/${appsPath}/${appPath}/installations/new?state=${stateParam}`;
             }
             else {
                 state.githubInstallationUrl = `https://github.com/apps/windmill-sync-helper/installations/new?state=${stateParam}`;

package/package/hubPaths.json

@@ -27,10 +27,7 @@
 	"deprecated_gitSync_latest": "hub/28180/sync-script-to-git-repo-windmill",
 	"deprecated_gitSync_25": "hub/28183/sync-script-to-git-repo-windmill",
 	"gitSyncTest": "hub/28184/git-repo-test-read-write-windmill",
-	"gitInitRepo_0": "hub/28134/git-sync%3A-init-repository-windmill",
-	"gitInitRepo_1": "hub/28158/git-sync%3A-init-repository-windmill",
-	"gitInitRepo_2": "hub/28174/git-sync%3A-init-repository-windmill",
-	"gitInitRepo": "hub/28181/git-sync%3A-init-repository-windmill",
+	"gitInitRepo": "hub/28201/git-sync%3A-init-repository-windmill",
 	"slackErrorHandler": "hub/19741/workspace-or-schedule-error-handler-slack",
 	"slackErrorHandler_0": "hub/9079/workspace-or-schedule-error-handler-slack",
 	"slackErrorHandler_1": "hub/9206/workspace-or-schedule-error-handler-slack",

package/package/infer.js

@@ -1,6 +1,6 @@
 import { ScriptService, FlowService } from './gen';
 import { get, writable } from 'svelte/store';
-import { emptySchema, sortObject } from './utils.js';
+import { emptySchema, getHubFlowIdFromPath, isHubFlowPath, sortObject } from './utils.js';
 import { tick } from 'svelte';
 import initTsParser, { parse_deno, parse_outputs } from 'windmill-parser-wasm-ts';
 import initRegexParsers, { parse_sql, parse_mysql, parse_oracledb, parse_duckdb, parse_bigquery, parse_snowflake, parse_graphql, parse_mssql, parse_db_resource, parse_bash, parse_powershell } from 'windmill-parser-wasm-regex';
@@ -439,6 +439,18 @@ export async function loadSchema(workspace, path, runType) {
         return { schema: script.schema, summary: script.summary };
     }
     else if (runType === 'flow') {
+        if (isHubFlowPath(path)) {
+            const hubFlowId = getHubFlowIdFromPath(path);
+            if (hubFlowId === undefined) {
+                throw new Error(`Invalid hub flow path: ${path}`);
+            }
+            const hub = await FlowService.getHubFlowById({ id: hubFlowId });
+            const flow = hub.flow;
+            const schema = flow?.schema && typeof flow.schema === 'object' && Object.keys(flow.schema).length > 0
+                ? flow.schema
+                : emptySchema();
+            return { schema, summary: flow?.summary };
+        }
         const flow = await FlowService.getFlowByPath({
             workspace,
             path

package/package/infer.svelte.js

@@ -43,6 +43,13 @@ export function usePreparedAssetSqlQueries(_getQueries, getWorkspace) {
         }
     };
 }
+// DuckDB prepare replaces $N params with NULL. Some functions (read_parquet, read_csv, etc.)
+// reject NULL arguments, which is expected — the query will work at execution time with real args.
+function isNullParamSubstitutionError(error) {
+    if (!error)
+        return false;
+    return error.includes('cannot take NULL') || error.includes('Could not choose a best candidate');
+}
 function mapPrepareResults(res, chunk) {
     if (res.length !== chunk.length) {
         throw new Error(`Prepare results count mismatch: got ${res.length}, expected ${chunk.length}`);
@@ -53,7 +60,9 @@ function mapPrepareResults(res, chunk) {
             ? {
                 columns: Object.fromEntries(r.columns.map(({ name, type: t }) => [name, sqlDataTypeToJsTypeHeuristic(t)]))
             }
-            : { error: r.error ?? "Couldn't prepare query " }
+            : isNullParamSubstitutionError(r.error)
+                ? { columns: {} }
+                : { error: r.error ?? "Couldn't prepare query " }
     ]);
 }
 async function prepareDatatableQueries(queries, getWorkspace) {

package/package/monaco_workers/sqlTypePlugin.worker.d.ts

@@ -8,6 +8,7 @@ declare class SqlAwareTypeScriptWorker {
     _sqlQueriesByFile: Map<any, any>;
     _fileVersions: Map<any, any>;
     _transformedCodeCache: Map<any, any>;
+    _bypassTransform: boolean;
     /**
      * Gets or computes the cached transformation result for a file
      * This centralizes all transformation logic and caching
@@ -132,6 +133,15 @@ declare class SqlAwareTypeScriptWorker {
     _createSqlErrorDiagnostics(fileName: string): any[];
     getSyntacticDiagnostics(fileName: any): Promise<any[]>;
     getSemanticDiagnostics(fileName: any): Promise<any[]>;
+    /**
+     * Override formatting methods to operate on original code (without injected SQL types).
+     * The formatter only cares about syntax/spacing, not types, so the original code is correct.
+     * Without this, formatting edits use positions from the transformed code, causing
+     * character offsets to be wrong and truncating SQL strings.
+     */
+    getFormattingEditsForDocument(fileName: any, options: any): Promise<any>;
+    getFormattingEditsForRange(fileName: any, start: any, end: any, options: any): Promise<any>;
+    getFormattingEditsAfterKeystroke(fileName: any, position: any, ch: any, options: any): Promise<any>;
     getSuggestionDiagnostics(fileName: any): Promise<any[]>;
     getCompilerOptionsDiagnostics(fileName: any): Promise<any[]>;
     /**

package/package/monaco_workers/sqlTypePlugin.worker.js

@@ -53,6 +53,8 @@ class SqlAwareTypeScriptWorker extends TypeScriptWorker {
 		// Cache of transformed code and offset maps per file version
 		// Structure: fileUri -> {version, originalText, transformed, offsetMap, offsetMapEntries}
 		this._transformedCodeCache = new Map()
+		// When true, getScriptSnapshot returns original code (used during formatting)
+		this._bypassTransform = false
 	}
 
 	/**
@@ -110,6 +112,10 @@ class SqlAwareTypeScriptWorker extends TypeScriptWorker {
 	 * This is called by TypeScript when it needs to read source files
 	 */
 	getScriptSnapshot(fileName) {
+		if (this._bypassTransform) {
+			return super.getScriptSnapshot(fileName)
+		}
+
 		const cached = this._getTransformResult(fileName)
 
 		if (!cached) {
@@ -718,6 +724,39 @@ class SqlAwareTypeScriptWorker extends TypeScriptWorker {
 		return [...mappedDiagnostics, ...sqlDiagnostics]
 	}
 
+	/**
+	 * Override formatting methods to operate on original code (without injected SQL types).
+	 * The formatter only cares about syntax/spacing, not types, so the original code is correct.
+	 * Without this, formatting edits use positions from the transformed code, causing
+	 * character offsets to be wrong and truncating SQL strings.
+	 */
+	async getFormattingEditsForDocument(fileName, options) {
+		this._bypassTransform = true
+		try {
+			return await super.getFormattingEditsForDocument(fileName, options)
+		} finally {
+			this._bypassTransform = false
+		}
+	}
+
+	async getFormattingEditsForRange(fileName, start, end, options) {
+		this._bypassTransform = true
+		try {
+			return await super.getFormattingEditsForRange(fileName, start, end, options)
+		} finally {
+			this._bypassTransform = false
+		}
+	}
+
+	async getFormattingEditsAfterKeystroke(fileName, position, ch, options) {
+		this._bypassTransform = true
+		try {
+			return await super.getFormattingEditsAfterKeystroke(fileName, position, ch, options)
+		} finally {
+			this._bypassTransform = false
+		}
+	}
+
 	async getSuggestionDiagnostics(fileName) {
 		const diagnostics = await super.getSuggestionDiagnostics(fileName)
 		return this._mapDiagnostics(diagnostics, fileName)

package/package/script_helpers.d.ts

@@ -115,10 +115,16 @@ export declare const INITIAL_CODE: {
     wac_typescript: {
         script: string;
     };
+    ci_test_bun: {
+        script: string;
+    };
+    ci_test_python: {
+        script: string;
+    };
 };
 export declare function isInitialCode(content: string): boolean;
-export declare function initialCode(language: SupportedLanguage | 'bunnative' | undefined, kind: Script['kind'] | undefined, subkind: 'pgsql' | 'mysql' | 'flow' | 'script' | 'fetch' | 'docker' | 'powershell' | 'bunnative' | 'claudesandbox' | 'wac_python' | 'wac_typescript' | undefined, templateScript?: boolean): string;
-export declare function getResetCode(language: SupportedLanguage | 'bunnative' | undefined, kind: Script['kind'] | undefined, subkind: 'pgsql' | 'mysql' | 'flow' | 'script' | 'fetch' | 'docker' | 'powershell' | 'bunnative' | 'claudesandbox' | 'wac_python' | 'wac_typescript' | undefined): string;
+export declare function initialCode(language: SupportedLanguage | 'bunnative' | undefined, kind: Script['kind'] | undefined, subkind: 'pgsql' | 'mysql' | 'flow' | 'script' | 'fetch' | 'docker' | 'powershell' | 'bunnative' | 'claudesandbox' | 'wac_python' | 'wac_typescript' | 'ci_test_bun' | 'ci_test_python' | undefined, templateScript?: boolean): string;
+export declare function getResetCode(language: SupportedLanguage | 'bunnative' | undefined, kind: Script['kind'] | undefined, subkind: 'pgsql' | 'mysql' | 'flow' | 'script' | 'fetch' | 'docker' | 'powershell' | 'bunnative' | 'claudesandbox' | 'wac_python' | 'wac_typescript' | 'ci_test_bun' | 'ci_test_python' | undefined): string;
 export declare const PREPROCESSOR_SUPPORTED_LANGUAGES: readonly ["typescript", "python", "python3", "deno", "bun", "php"];
 export declare function canHavePreprocessor(language: string | undefined): boolean;
 export declare function canHaveTrigger(language: SupportedLanguage | undefined): boolean;

package/package/script_helpers.js

@@ -2,6 +2,8 @@ import {} from './gen';
 import CLAUDE_SANDBOX_INIT_CODE from './templates/claude_sandbox.ts.template?raw';
 import WAC_PYTHON_INIT_CODE from './templates/wac_python.py.template?raw';
 import WAC_TYPESCRIPT_INIT_CODE from './templates/wac_typescript.ts.template?raw';
+import CI_TEST_BUN_INIT_CODE from './templates/ci_test_bun.ts.template?raw';
+import CI_TEST_PYTHON_INIT_CODE from './templates/ci_test_python.py.template?raw';
 const PYTHON_FAILURE_MODULE_CODE = `import os
 
 def main(message: str, name: str, step_id: str):
@@ -1355,6 +1357,12 @@ export const INITIAL_CODE = {
     },
     wac_typescript: {
         script: WAC_TYPESCRIPT_INIT_CODE
+    },
+    ci_test_bun: {
+        script: CI_TEST_BUN_INIT_CODE
+    },
+    ci_test_python: {
+        script: CI_TEST_PYTHON_INIT_CODE
     }
     // for related places search: ADD_NEW_LANG
 };
@@ -1406,6 +1414,12 @@ export function initialCode(language, kind, subkind, templateScript) {
             return INITIAL_CODE.deno.script;
         }
     }
+    else if (subkind === 'ci_test_bun') {
+        return INITIAL_CODE.ci_test_bun.script;
+    }
+    else if (subkind === 'ci_test_python') {
+        return INITIAL_CODE.ci_test_python.script;
+    }
     else if (subkind === 'wac_python') {
         return INITIAL_CODE.wac_python.script;
     }

package/package/stores.d.ts

@@ -68,6 +68,10 @@ export declare const relativeLineNumbers: import("svelte/store").Writable<boolea
 export declare const codeCompletionSessionEnabled: import("svelte/store").Writable<boolean>;
 export declare const usedTriggerKinds: import("svelte/store").Writable<string[]>;
 export declare let globalDbManagerDrawer: StateStore<DbManagerUriState | undefined>;
+export declare let globalForkModal: StateStore<GlobalForkModalState | undefined>;
+export type GlobalForkModalState = {
+    opened: true;
+};
 type SQLBaseSchema = {
     [schemaKey: string]: {
         [tableKey: string]: {

package/package/stores.js

@@ -80,6 +80,7 @@ export const relativeLineNumbers = writable(getLocalSetting(RELATIVE_LINE_NUMBER
 export const codeCompletionSessionEnabled = writable(getLocalSetting(CODE_COMPLETION_SETTING_NAME) != 'false');
 export const usedTriggerKinds = writable([]);
 export let globalDbManagerDrawer = { val: undefined };
+export let globalForkModal = createState({ val: undefined });
 export const SQLSchemaLanguages = [
     'mysql',
     'bigquery',