Skip to content

feat(settings): telemetry privacy toggle + image-gen defaults#1021

Merged
thinmintdev merged 1 commit into
mainfrom
fix/wave8c-settings
Jul 4, 2026
Merged

feat(settings): telemetry privacy toggle + image-gen defaults#1021
thinmintdev merged 1 commit into
mainfrom
fix/wave8c-settings

Conversation

@thinmintdev

Copy link
Copy Markdown
Contributor

Wave 8C — Settings gaps

  • Privacy: telemetry.enabled opt-in toggle added to the previously-dead General section (PUT /api/settings {telemetry:{enabled}} via the existing deep-merge hook + ApplyBadge; reads the current value, preserves all other config).
  • Image-gen: surfaced default_size / default_steps / idle_restore_minutes in the Image-gen section (the "no slot-config path" rationale was stale — settings(image-gen): persist size/steps/workflow (needs SlotConfig fields) #599 shipped ImageGenConfig). Saved to the img slot's [image] config via the one-level slot-config merge (preserves siblings), int-coerced, and degrades gracefully (disabled + hint) when no img slot exists.

Verification

  • vite build clean; settings e2e 5 passed (8 sections intact); data-write review clean (correct body shapes, deep-merge/one-level-merge preserve siblings, no field wipes).

🤖 Generated with Claude Code

Repurpose the dead General section to host the telemetry.enabled opt-in
(PUT /api/settings {telemetry:{enabled}} via the deep-merge settings hook
+ ApplyBadge). Unhide the image-gen default_size/default_steps/
idle_restore_minutes controls (stale 'no slot-config path' rationale —
#599 added ImageGenConfig), saved to the img slot's [image] config via
the one-level slot-config merge; degrades gracefully when no img slot.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@thinmintdev thinmintdev enabled auto-merge (squash) July 4, 2026 04:21
@thinmintdev thinmintdev merged commit 583ac95 into main Jul 4, 2026
3 checks passed
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