mirror of
https://github.com/windmill-labs/windmill.git
synced 2026-10-13 08:02:47 +00:00
Guilhem LemouelandClaude Opus 5
45c0a35981
Seven conflicts, all in the app-write path and all the same shape: main added a
code diff to each app tool's result (`finishAppDraftWrite` grew a `codeDiff`
argument, and the file tools capture the before-text), while this branch changed
those same calls to address a resolved target and report the name the app deploys
under. Both kept: the diff argument and `inlineRunnableDiff` from main, the
resolved target and `appPath` closure from here. Main's side of each hunk still
called `loadAppDraftValue(args.path, …)` and read `{ value }` from it, which no
longer type-checks — the merged helpers take a target and return the value — so
the call shape had to come from this side.
The last conflict is the ACTIVE EDITOR block: main added an `isNew` hint telling
the model not to create a second item over a template, this branch emits the
storage key as `path` with the chosen name as `draft_path`. Both are now emitted.
Main also added `quickjs-emscripten-core`, so this worktree needed `npm install`;
without it `tsc` reports the module missing.
One test, `test_run_step opens an aiagent step on the step's own configuration`,
hangs after this merge. It hangs the same way on pristine `origin/main`
(`eab3dd5a52`, 30s with the timeout raised), so it arrived with main rather than
with this resolution — a seventh pre-existing failure, not a regression.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Windmill frontend
The Windmill frontend written in Svelte 5 + Tailwind CSS
The frontend is under AGPL, see the LICENSE file at the root of this repo