Skip to content

fix(runner): batch auto-install and add CI - #2

Merged
taigrr merged 1 commit into
masterfrom
cd/ci-and-auto-install-fixes
Apr 12, 2026
Merged

taigrr merged 1 commit into
masterfrom
cd/ci-and-auto-install-fixes

Conversation

@taigrr

@taigrr taigrr commented Apr 5, 2026

Copy link
Copy Markdown
Owner

Changes

Bug fixes

  • Auto-install race condition: Multiple plugins registering binaries simultaneously would each defer a separate runner.install() call. The second call would get rejected with "tasks already running" warning. Now uses a batched 200ms debounce timer — all registrations within the window are collected and installed in a single batch.
  • Silent auto-install opens UI: When auto_install.silent = true, the Glaze floating window would still pop open because runner.install() unconditionally called view.open(). Now respects a { silent = true } option to suppress the UI.
  • Empty if branch lint warning: Refactored the install filter in runner.lua to eliminate the empty if branch.

CI

  • Added GitHub Actions workflow with StyLua format checking and luacheck linting.
  • Added .stylua.toml and .luacheckrc configs.
  • Applied stylua formatting to checker.lua and health.lua.

- Auto-install now batches all registrations into a single deferred
  install call (200ms debounce), preventing race conditions where
  multiple deferred installs would reject each other
- Silent auto-install no longer pops open the Glaze UI window
- runner.install() accepts { silent: true } option to suppress UI
  and notifications
- Add CI workflow (stylua format check + luacheck lint)
- Add .stylua.toml and .luacheckrc configs
- Fix empty if branch lint warning in runner.lua
- Apply stylua formatting to checker.lua and health.lua
@taigrr
taigrr merged commit 9463ba2 into master Apr 12, 2026
1 check passed
@taigrr
taigrr deleted the cd/ci-and-auto-install-fixes branch May 4, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant