Desktop release builds on Linux and Windows embed no web UI
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 76/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Domain
- build-system, devops
Research direction
Read .github/actions/setup-desktop-build, scripts/stage-webui.sh, and the desktop job in cd.yaml to trace where the UI is staged relative to $GITHUB_WORKSPACE; check the Makefile, GoReleaser hooks, and operator-chart e2e callers before choosing a compatible change. Then verify a Linux or Windows build-mode desktop release stages the UI in the release checkout and fails if pkg/webui/dist contains only .gitkeep before building.
Written by the indexing model from the issue text.
Description
🤖 Generated by the Agentic Engineer
Problem
The Linux and Windows desktop releases can ship without the web UI. The app would then show "KSail web UI not built" instead of the interface.
In cd.yaml, the desktop job uses setup-desktop-build as a remote action (devantler-tech/ksail/.github/actions/setup-desktop-build@<sha>). Its last step runs scripts/stage-webui.sh through $GITHUB_ACTION_PATH, and the script cds to its own repository root. For a remote action, that root is the runner's downloaded copy of the action repository, not the release checkout in $GITHUB_WORKSPACE. So the UI is built and staged into the downloaded copy, and go build in the release checkout embeds only the placeholder pkg/webui/dist/.gitkeep.
The macOS desktop leg (GoReleaser's before hook) and make ui stage inside the checkout, so they are not affected.
Found by CodeRabbit as an outside-diff finding on #8182 (it is older than that PR). I confirmed it by reading the action and the script.
Acceptance criteria
- A build-mode Linux or Windows desktop release embeds the built web UI from the release checkout.
stage-webui.shstages into the checkout that calls it. For example, it accepts a target root, or the action passes$GITHUB_WORKSPACE. Other callers (Makefile, GoReleaser hooks, operator-chart e2e) keep working.- A check fails the desktop leg when
pkg/webui/distholds only.gitkeepbefore the build.
- Dominant language
- Go
- Stars
- 166
- Forks
- 13
- Avg merge
- 7h 4m
- Merged PRs (30d)
- 388
Getting set up
- Ships a Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from devantler-tech/ksail
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
devantler-tech/ksail#8256 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
devantler-tech/ksail#8255 ·
Maintainers usually reply within 1 day
-
Homebrew installs of ksail come with shell completionsPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 Half a day Newbie friendliness 86/100
devantler-tech/ksail#7847 · 1 comment ·
Maintainers usually reply within 1 day
-
bug(ci): a failed autoscaler inventory trial uploads no diagnosticsPossibly taken @devantler claimed this 22 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
devantler-tech/ksail#7700 · 1 assignee ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
devantler-tech/ksail#8257 ·
Maintainers usually reply within 1 day
All issues in devantler-tech/ksail
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
[Bug] K8s reconcile still resets explicit 0 to defaults for response_api, memory and flow statePossibly taken @Bevisy claimed this today. Openneeds-acceptance
Difficulty 2/5 1-3 hours Newbie friendliness 87/100
vllm-project/semantic-router#5998 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day
-
desktop v3 windows
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
esengine/DeepSeek-Reasonix#15016 · 1 comment ·
Maintainers usually reply within 1 day
-
tech-debt: credits.sql — twelve generated queries over the retired AI-credit ledger have no callerOpentech-debt
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day