fix(onboarding): address Copilot review on PR #57

- ui.Toggle: default an empty value to "on" inside the component (via
  toggleValue helper) so single-boolean callers submit a meaningful
  form value matching the HTML default. Previously the component
  emitted value="" verbatim, so r.FormValue(name) == "on" was always
  false even when checked. Multi-select groups still pass an explicit
  non-empty value as before.

- InjectIntegrationStatus: restore fail-open semantics on
  box-gear DB load errors. Previously this PR set a partial gear
  status/order context (system gears only) on error, collapsing the
  sidebar instead of falling back to the default full-rendering
  branch. Now box-gear errors short-circuit before any context is
  set, matching the original behavior.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: sarg3nt

gearbox/internal/framework/handler/handler.go

@@ -221,16 +221,22 @@ func (h *Handler) InjectIntegrationStatus(next http.Handler) http.Handler {
 		if boxID := h.getDefaultServerID(); boxID != "" {
 			integrations, err := h.db.GetGears(boxID)
 			if err != nil {
-				h.logger.Error("failed to get integrations for sidebar", "error", err)
-			} else {
-				for _, i := range integrations {
-					status[i.Name] = i.Enabled
-					orderedIntegrations = append(orderedIntegrations, auth.SidebarIntegration{
-						Name:      i.Name,
-						Enabled:   i.Enabled,
-						SortOrder: i.SortOrder,
-					})
-				}
+				// Fail-open: a partial gear list could collapse the sidebar
+				// to system-gears-only, hiding box features the user
+				// actually has. Leave the gear-status/order context unset
+				// so OrderedIntegrationLinks falls back to its default
+				// (full) rendering branch.
+				h.logger.Error("failed to get box integrations for sidebar", "error", err)
+				next.ServeHTTP(w, r.WithContext(ctx))
+				return
+			}
+			for _, i := range integrations {
+				status[i.Name] = i.Enabled
+				orderedIntegrations = append(orderedIntegrations, auth.SidebarIntegration{
+					Name:      i.Name,
+					Enabled:   i.Enabled,
+					SortOrder: i.SortOrder,
+				})
 			}
 		} else {
 			// No box configured — explicitly mark box-scoped gears off so

gearbox/internal/framework/ui/toggle.templ

@@ -1,10 +1,24 @@
 package ui
 
+// toggleValue defaults an empty value to "on" so the underlying checkbox
+// submits a non-empty form value when checked. This matches the HTML
+// default for a value-less checkbox and keeps `r.FormValue(name) == "on"`
+// callers working without forcing every site to pass an explicit value.
+func toggleValue(v string) string {
+	if v == "" {
+		return "on"
+	}
+	return v
+}
+
 // Toggle creates a switch/toggle input component.
 //
 // id: unique identifier for the input
 // name: form field name
-// value: form value submitted when checked (empty string → defaults to "on")
+// value: form value submitted when checked. If empty, defaults to "on"
+//        (the HTML default for a checkbox without a value attribute). Pass
+//        a unique non-empty value when multiple toggles share a name
+//        (multi-select checkbox group).
 // checked: initial state
 // disabled: whether the toggle is disabled
 //
@@ -17,7 +31,7 @@ templ Toggle(id string, name string, value string, checked bool, disabled bool)
 			type="checkbox"
 			id={ id }
 			name={ name }
-			value={ value }
+			value={ toggleValue(value) }
 			class="sr-only peer"
 			checked?={ checked }
 			disabled?={ disabled }