# C-1 Windows-Specific Bug Fix Implementation

**Date:** 2026-03-29
**Branch:** culurien

---

## Files Changed

| File | Change |
|------|--------|
| `service/windows.go` | Proportional jitter + exponential backoff (F-C1-5), emojis removed (F-C1-7) |
| `scanner/winget.go` | Multi-location path search (F-C1-1), column-position text parser (F-C1-2), fmt.Printf replaced with log.Printf (F-C1-6) |
| `installer/windows.go` | RebootRequired post-install marker (F-C1-3) |
| `installer/types.go` | Added RebootRequired field to InstallResult |

## Approach Notes

**F-C1-5 (polling loop):** Applied B-2 fixes directly to service/windows.go rather than extracting a shared function. The structural differences (stop channel, Windows Event Log, different config types) make extraction high-risk. TODO comment added for future deduplication. Documented as DEV-030.

**F-C1-1 (winget path):** Added `findWingetPath()` that checks PATH first, then known system-wide locations via `filepath.Glob`. Cross-platform safe — glob patterns are just strings on Linux.

**F-C1-3 (ghost updates):** Used Option A — post-install `RebootRequired` flag on InstallResult. The scanner's existing `IsInstalled=0` criteria is correct; the ghost update occurs because Windows hasn't committed the install state yet.

**F-C1-2 (text parser):** Column-position parsing using header line keyword positions instead of whitespace splitting. Falls back to simple parser if header format is unrecognized.

**F-C1-4:** Already resolved — service has `SetRecoveryActions` configured.

## Linux Safety

All modified SHARED files (winget.go, installer/windows.go, types.go) compile on Linux. Windows-specific service/windows.go has `//go:build windows` tag.
