Skip to content

fix(euclid): restore div_saved on boot, refresh mute LED on scene load#523

Merged
chmanie merged 1 commit into
mainfrom
fix/euclid-app
May 16, 2026
Merged

fix(euclid): restore div_saved on boot, refresh mute LED on scene load#523
chmanie merged 1 commit into
mainfrom
fix/euclid-app

Conversation

@ArthurGibert
Copy link
Copy Markdown
Member

Summary

  • Boot path now restores div_saved from storage. Previously, the user's clock division setting was saved correctly but never read back on app boot — div_glob always initialized to the hardcoded default (6 / 16th notes), so any custom division reverted on layout reload. The scene-load path already handled this; only the initial run() path was missing it.
  • Scene load refreshes the channel-1 button LED to match the loaded mute state immediately, instead of leaving it stale until the next interaction.
  • CI compat: adds configurator/pnpm-workspace.yaml and pins packageManager to pnpm 11.1.1 — required for the configurator CI step to install successfully under pnpm 11's new build-script approval policy.

Test plan

  • Set a non-default clock division (Third layer: button 0 + fader 0)
  • Power-cycle / reload the layout
  • Confirm the saved division is restored (LED color in Third layer + observed clock rate)
  • Mute via channel-1 button, save scene, load a different scene with unmuted state, confirm button LED reflects loaded state immediately

🤖 Generated with Claude Code

Copy link
Copy Markdown
Member

@chmanie chmanie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

…load

The clock division was saved correctly but never restored when the app
booted — div_glob always initialized to the hardcoded default of 6,
losing the user's preference across reloads. Scene-load already did
this; only the initial boot path was missing it.

Also refreshes the channel-1 button LED on scene load so it matches
the loaded mute state immediately instead of waiting for the next
button-press interaction to repaint it.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@chmanie chmanie merged commit 2567890 into main May 16, 2026
3 checks passed
@chmanie chmanie deleted the fix/euclid-app branch May 16, 2026 10:43
chmanie pushed a commit that referenced this pull request May 16, 2026
…on scene load (#523)

The clock division was saved correctly but never restored when the app
booted — div_glob always initialized to the hardcoded default of 6,
losing the user's preference across reloads. Scene-load already did
this; only the initial boot path was missing it.

Also refreshes the channel-1 button LED on scene load so it matches
the loaded mute state immediately instead of waiting for the next
button-press interaction to repaint it.
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.

2 participants