Merge: HedgeBuddy UI Modernization (v0.10.0)

Phase 1 (visual + chrome refresh) + Phase 2 (behaviors + correctness)
plus iterative QA-driven fixes across 1.1-1.3 and 2.1-2.3.

Summary:
- Sidebar shell + right-side drawers replace the legacy top toolbar
- Design tokens, OS-native fonts, Lucide icons, reactive renderer pattern
- Component library: CardRow, Sidebar, Drawer, IconButton, InlineStateButton,
  Modal, FieldRow
- No-toast feedback contract via inline button states + row flashes
- Inline validation, fsnotify auto-reload, case-normalized storage dir
- CRITICAL: rename collision check (was silently overwriting variables)
- Python lib drops Linux branch; matches Windows + macOS scope
- 61 commits, ~12 affected packages

See CHANGELOG.md for the full delta.

Author: shakedex

CHANGELOG.md

@@ -8,9 +8,56 @@ Format based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.10.0] - 2026-05-25
+
+Major UI modernization. Old top-toolbar layout replaced with a sidebar shell + right-side drawers. The 5 legacy views (`*view.go`) have been removed and rebuilt as drawers and modals composed from a shared component library.
+
+### Added
+
+- **Sidebar shell** — left-aligned 200px sidebar with PROFILES section (active profile highlighted, inline `+` composer and inline rename), FILTERS section (`All / String / Path / URL / Secret` with live counts and Lucide icons), and Settings/About footer links.
+- **Right-side drawer overlay** — replaces the previous full-pane navigation for Edit/New/Import/Export. Esc and scrim-tap both close. Drawer content swaps without re-mounting the shell.
+- **Design token packages** — `app/internal/ui/tokens/` defines colors, spacing, and radii as the single source of truth. The theme proxies to tokens.
+- **OS-native font loader** — Segoe UI on Windows (`segoeui.ttf` + `segoeuib.ttf`), SF Pro on macOS, with safe fallback to Fyne's default when files are missing. Path uses `%SystemRoot%` for portability.
+- **Lucide icon set** — 28 outlined SVGs vendored via `tools/icons` fetcher tool and embedded via `//go:embed`. Replaces all `theme.*Icon()` callsites in the new UI surfaces.
+- **Reusable component primitives** (`app/internal/ui/components/`): `Sidebar`+`SidebarItem`, `CardRow`, `Drawer`, `IconButton` (with hover-tint and tooltip), `InlineStateButton` (idle→busy→done/error state machine), `Modal`+`ShowDeleteConfirm`, `FieldRow` with inline error caption.
+- **No-toast feedback contract** — `InlineStateButton` morphs on save/import/export; `CardRow.Flash()` tints rows on save/import/duplicate/copy; `CardRow.ConfirmCopy()` swaps copy icon to a check for 1s. Replaces transient toast notifications.
+- **Inline validation** — `Entry.Validator` + `FieldRow.SetError` surface variable name and value errors under the field as the user types. Modal `dialog.ShowError` reduced to disk-IO last resort.
+- **Auto-reload via fsnotify** — `vars.json` is watched; external edits refresh the list automatically.
+- **Variable name collision check** — `SaveVariable` refuses to rename or create a variable that would clobber an existing one. Surfaces inline under the Name field via `ErrVariableExists` sentinel.
+- **Auto-scroll to flashed row** — duplicate/save/import scrolls the list to bring the affected row into view (uses real `Position().Y`, not estimated row height).
+- **Inline profile composer** — new-profile and rename happen as inline editable rows in the sidebar. Modal forms remain only for Import-as-Profile.
+- **Sidebar filter icons** — Lucide `type/folder/link/lock` icons next to filter labels.
+- **Code-block styling for variable values** — recessed `Surface1` background distinguishes value from description.
+- **Tooltip layer** — IconButtons embed `ttwidget.ToolTipWidget` for hover tooltips on all actionable icons.
+- **Browse File / Browse Folder split** — path-type variables now offer both file and folder pickers.
+- **Phase 2 manual QA checklist** — `docs/superpowers/specs/2026-05-24-hedgebuddy-ui-modernization-qa.md`.
+
 ### Changed
 
-- **Quills extracted to its own repository at [shakedex/quills](https://github.com/shakedex/quills).** The `service/` and `quills/` directories, the Quills release workflow, and the combined "HedgeBuddy Suite" installer have been removed. Historical entries below that describe Quills work remain for context, but Quills development continues in the new repo.
+- **Storage directory case-normalized to lowercase `hedgebuddy/`** on Windows and macOS. Existing `HedgeBuddy/` directories migrate automatically on first launch. Aligns Go side with Python side; eliminates case-sensitive filesystem mismatches.
+- **Python check + update check sequenced** — update dialog only fires after the Python check completes or dismisses, eliminating modal stacking.
+- **Export warning** — now mentions both `.env` and JSON formats (both write secrets in plain text).
+- **Delete confirm dialog** — action-specific button label (`Delete API_KEY`) and terse single-line body.
+- **Python/update dialogs** — two-button layout with inline "Don't remind me again" checkbox (was 3-4 stacked buttons).
+- **Install Update** — shows "Launching updater…" inline for 600ms before quit instead of disappearing instantly.
+- **Validator messages** — human language ("URLs must start with http:// or https://"; "can't find this path on this machine").
+- **About modal** — softened disclaimer; less shouty.
+- **Default and active profile** — Rename and Delete options disabled in the ⋯ menu instead of throwing errors after the click.
+
+### Removed
+
+- **6 legacy view files** — `aboutview.go`, `formview.go`, `importview.go`, `exportview.go`, `profileview.go`, `helpers.go`. Replaced by `aboutmodal.go`, `editdrawer.go`, `importdrawer.go`, `exportdrawer.go`, `profilemodal.go`, `settingsmodal.go`, and the new components package.
+- **Linux branch of Python lib's `_get_base_dir()`** — raises `StorageNotFoundError` on Linux. The Go GUI doesn't ship for Linux, so the silent fallback was creating drift.
+- **Toast notifications and `ShowStatus`** — replaced by the inline feedback contract.
+- **Manual "Refresh" toolbar button** — fsnotify replaces it.
+
+### Fixed
+
+- **CRITICAL: rename collision data loss** — renaming variable `A` to an existing variable `B`'s name no longer silently overwrites `B`. (Pre-Phase-2 behavior would merge into `B`, losing its value.)
+- **Drop-handler leak across views** — drop handler scoped to drawer lifecycle.
+- **Confirm-delete modal text wrapping** — uses `widget.Label` with `TextWrapWord`.
+- **`container.NewMax`** deprecated calls replaced with `container.NewStack`.
+- **Manual string truncation** replaced with `widget.Label.Truncation = TextTruncateEllipsis`.
 
 ## [0.9.1] - 2026-04-12
 

VERSION

@@ -1 +1 @@
-0.9.1
+0.10.0

app/FyneApp.toml

@@ -4,5 +4,5 @@ Website = "https://shaked.co"
 Icon = "icon.png"
 Name = "HedgeBuddy"
 ID = "co.hedge.hedgebuddy"
-Version = "0.9.1"
+Version = "0.10.0"
 Build = 7

app/go.mod

@@ -5,6 +5,7 @@ go 1.25.0
 require (
 	fyne.io/fyne/v2 v2.7.3
 	github.com/dweymouth/fyne-tooltip v0.4.0
+	github.com/fsnotify/fsnotify v1.10.1
 	github.com/ncruces/zenity v0.10.14
 	github.com/tidwall/gjson v1.18.0
 	github.com/tidwall/sjson v1.2.5
@@ -17,7 +18,6 @@ require (
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/dchest/jsmin v1.0.0 // indirect
 	github.com/fredbi/uri v1.1.1 // indirect
-	github.com/fsnotify/fsnotify v1.9.0 // indirect
 	github.com/fyne-io/gl-js v0.2.0 // indirect
 	github.com/fyne-io/glfw-js v0.3.0 // indirect
 	github.com/fyne-io/image v0.1.1 // indirect

app/go.sum

@@ -16,8 +16,8 @@ github.com/felixge/fgprof v0.9.3 h1:VvyZxILNuCiUCSXtPtYmmtGvb65nqXh2QFWc0Wpf2/g=
 github.com/felixge/fgprof v0.9.3/go.mod h1:RdbpDgzqYVh/T9fPELJyV7EYJuHB55UTEULNun8eiPw=
 github.com/fredbi/uri v1.1.1 h1:xZHJC08GZNIUhbP5ImTHnt5Ya0T8FI2VAwI/37kh2Ko=
 github.com/fredbi/uri v1.1.1/go.mod h1:4+DZQ5zBjEwQCDmXW5JdIjz0PUA+yJbvtBv+u+adr5o=
-github.com/fsnotify/fsnotify v1.9.0 h1:2Ml+OJNzbYCTzsxtv8vKSFD9PbJjmhYF14k/jKC7S9k=
-github.com/fsnotify/fsnotify v1.9.0/go.mod h1:8jBTzvmWwFyi3Pb8djgCCO5IBqzKJ/Jwo8TRcHyHii0=
+github.com/fsnotify/fsnotify v1.10.1 h1:b0/UzAf9yR5rhf3RPm9gf3ehBPpf0oZKIjtpKrx59Ho=
+github.com/fsnotify/fsnotify v1.10.1/go.mod h1:TLheqan6HD6GBK6PrDWyDPBaEV8LspOxvPSjC+bVfgo=
 github.com/fyne-io/gl-js v0.2.0 h1:+EXMLVEa18EfkXBVKhifYB6OGs3HwKO3lUElA0LlAjs=
 github.com/fyne-io/gl-js v0.2.0/go.mod h1:ZcepK8vmOYLu96JoxbCKJy2ybr+g1pTnaBDdl7c3ajI=
 github.com/fyne-io/glfw-js v0.3.0 h1:d8k2+Y7l+zy2pc7wlGRyPfTgZoqDf3AI4G+2zOWhWUk=

app/internal/profile/profile.go

@@ -30,13 +30,13 @@ func GetBaseDir() (string, error) {
 		if appData == "" {
 			return "", fmt.Errorf("APPDATA environment variable not found")
 		}
-		return filepath.Join(appData, "HedgeBuddy"), nil
+		return filepath.Join(appData, "hedgebuddy"), nil
 	case "darwin":
 		homeDir, err := os.UserHomeDir()
 		if err != nil {
 			return "", err
 		}
-		return filepath.Join(homeDir, "Library", "Application Support", "HedgeBuddy"), nil
+		return filepath.Join(homeDir, "Library", "Application Support", "hedgebuddy"), nil
 	default:
 		return "", fmt.Errorf("unsupported platform: %s", runtime.GOOS)
 	}
@@ -113,6 +113,57 @@ func SaveIndex(idx *ProfileIndex) error {
 	return nil
 }
 
+// migrateCapitalizedDir performs a one-shot rename of an existing capitalized
+// "HedgeBuddy" directory to the new lowercase "hedgebuddy" name. Safe to call
+// repeatedly — it no-ops when there's nothing to migrate.
+//
+// This addresses the historical Go (capital H) / Python (lowercase) mismatch
+// that breaks on case-sensitive filesystems.
+func migrateCapitalizedDir() error {
+	var oldDir string
+	switch runtime.GOOS {
+	case "windows":
+		oldDir = filepath.Join(os.Getenv("APPDATA"), "HedgeBuddy")
+	case "darwin":
+		homeDir, err := os.UserHomeDir()
+		if err != nil {
+			return err
+		}
+		oldDir = filepath.Join(homeDir, "Library", "Application Support", "HedgeBuddy")
+	default:
+		return nil
+	}
+
+	newDir, err := GetBaseDir()
+	if err != nil {
+		return err
+	}
+
+	// Nothing to do if paths are identical (case-insensitive FS or already lowercase).
+	if oldDir == newDir {
+		return nil
+	}
+
+	// Skip if no old dir.
+	oldInfo, err := os.Stat(oldDir)
+	if os.IsNotExist(err) {
+		return nil
+	}
+	if err != nil {
+		return err
+	}
+	if !oldInfo.IsDir() {
+		return nil
+	}
+
+	// Skip if new dir already exists (don't clobber it).
+	if _, err := os.Stat(newDir); err == nil {
+		return nil
+	}
+
+	return os.Rename(oldDir, newDir)
+}
+
 // Migrate performs a one-time migration from the flat vars.json layout to
 // the profiles-based layout. If profiles/ already exists, it's a no-op.
 //
@@ -121,6 +172,12 @@ func SaveIndex(idx *ProfileIndex) error {
 //  2. Move existing vars.json into profiles/default/vars.json
 //  3. Create profiles.json with active = "default"
 func Migrate() error {
+	// One-shot rename from capitalized HedgeBuddy dir to lowercase hedgebuddy.
+	// No-op when there's nothing to migrate.
+	if err := migrateCapitalizedDir(); err != nil {
+		fmt.Println("Warning: migrating HedgeBuddy → hedgebuddy:", err.Error())
+	}
+
 	base, err := GetBaseDir()
 	if err != nil {
 		return err

app/internal/ui/aboutmodal.go

@@ -0,0 +1,65 @@
+package ui
+
+import (
+	"net/url"
+
+	"fyne.io/fyne/v2"
+	"fyne.io/fyne/v2/canvas"
+	"fyne.io/fyne/v2/container"
+	"fyne.io/fyne/v2/layout"
+	"fyne.io/fyne/v2/widget"
+
+	"app/internal/ui/components"
+	"app/internal/ui/icons"
+	"app/internal/ui/tokens"
+)
+
+// ShowAboutModal opens the About centered modal.
+func ShowAboutModal(c *AppController) {
+	logo := canvas.NewImageFromResource(AppIcon())
+	logo.FillMode = canvas.ImageFillContain
+	logo.SetMinSize(fyne.NewSize(80, 80))
+
+	title := canvas.NewText(AppName, tokens.Accent)
+	title.TextSize = 24
+	title.TextStyle = fyne.TextStyle{Bold: true}
+	title.Alignment = fyne.TextAlignCenter
+
+	version := canvas.NewText("v"+AppVersion, tokens.TextMuted)
+	version.TextSize = 12
+	version.Alignment = fyne.TextAlignCenter
+
+	author := widget.NewLabel("Created by Shaked Lipszyc")
+	author.Alignment = fyne.TextAlignCenter
+
+	websiteBtn := widget.NewButtonWithIcon("shaked.co", icons.ExternalLink, func() {
+		u, _ := url.Parse(WebsiteURL)
+		_ = fyne.CurrentApp().OpenURL(u)
+	})
+	githubBtn := widget.NewButtonWithIcon("GitHub", icons.ExternalLink, func() {
+		u, _ := url.Parse(GithubURL)
+		_ = fyne.CurrentApp().OpenURL(u)
+	})
+
+	disclaimer := widget.NewLabel("Independent project. Not affiliated with Hedge (hedge.co). MIT licensed.")
+	disclaimer.Wrapping = fyne.TextWrapWord
+	disclaimer.Alignment = fyne.TextAlignCenter
+	disclaimer.Importance = widget.LowImportance
+
+	body := container.NewVBox(
+		container.NewCenter(logo),
+		title,
+		version,
+		widget.NewSeparator(),
+		author,
+		container.NewCenter(container.NewHBox(websiteBtn, githubBtn)),
+		widget.NewSeparator(),
+		disclaimer,
+	)
+
+	closeBtn := widget.NewButton("Close", nil)
+	d := components.ShowCustomModal(c.Window, "About", body, []fyne.CanvasObject{
+		layout.NewSpacer(), closeBtn,
+	})
+	closeBtn.OnTapped = func() { d.Hide() }
+}