sync: auto-sync from HOWARD-HOME at 2026-05-31 21:42:06

Author: Howard Enos
Machine: HOWARD-HOME
Timestamp: 2026-05-31 21:42:06
This commit is contained in:
2026-05-31 21:42:17 -07:00
parent 424d61057b
commit f9fdf9f598

View File

@@ -234,3 +234,48 @@ Code Review: APPROVE. Gitea Agent branched fix/clear-site-notes, committed 04830
- PR #33: merge 873e9ca; main tip 342d3a1; submodule branch commit 048306f.
- Root pointer-bump commit: fa57fab.
- Coord: component gururmm/dashboard -> deployed v0.2.35; lock 427369c8 released; todo ab091bae done.
---
## Update: 21:41 MST — Next-tier quick-wins: a11y fix deployed + BUG-015 ARP MSI shipped
### User
- **User:** Howard Enos (howard)
- **Machine:** Howard-Home
- **Role:** tech
### Session Summary
Started the "next tier" roadmap quick-wins (todo 15a5440f) and found it largely stale: verified in current code that BUG-008 (metrics.rs leak), BUG-009/010 (isError handling), and BUG-011 (`: any` — now 0 dashboard-wide) are all already fixed. Only two items had real work left.
1) **a11y (LogAnalysis FindingCard):** the toggle button's `aria-controls={detailsId}` pointed at a conditionally-rendered panel, so the id didn't exist when collapsed. Fixed by always rendering the container with `hidden={!expanded}` (heavy inner content still gated by `expanded` inside). Reviewed APPROVE. Shipped as PR #34 (merge 4923e9f), deployed via RMM agent → dashboard v0.2.36, live bundle index-BAUBO9gN.js.
2) **BUG-015 / SPEC-011 (agent missing from Programs and Features):** installer-only fix per spec — added 6 ARP properties + `<Icon>` + AgentIcon component + pinned AgentBinary GUID to installer/gururmm-agent.wxs, and generated installer/gururmm-agent.ico (multi-res 16/32/48/256, 8802 bytes, via Pillow). Reviewed APPROVE WITH NITS (key: .ico must be committed with .wxs or Pluto build fails — it was). Shipped as PR #35 (merge 4bca9f3, .ico in commit 07d14bd). Root submodule pointer bumped (424d610).
Both merged cleanly; Howard explicitly directed fixing BUG-015 despite the usual defer-agent/installer-to-Mike rule.
### Problems Encountered
- **Build-pipeline gate gap:** BUG-015's WiX change merged but the Windows MSI did NOT rebuild — build-windows.sh's change-gate (line ~31) only diffs `agent/`, so installer-only changes log "No agent changes — Windows build skipped" (last-built-commit-windows stuck at 9b34393). Forced a clean one-off build by emptying /opt/gururmm/last-built-commit-windows (the skip condition requires `-n "$LAST_SHA"`), backed up first. Ran build-windows.sh as root (matches webhook context; root holds the Pluto SSH key). Build completed in 623s, signed, marked stable. Routed the durable gate fix to Mike (todo a47884d7 + coord message) since the pipeline is his domain.
- Initial detached-launch RMM command timed out because the backgrounded wrapper held the agent's stdout pipe; verified the build was actually running (setsid inner-redirect worked) and watched it via a background poller instead.
### Configuration Changes
- gururmm: dashboard/src/components/LogAnalysis.tsx (PR #34); installer/gururmm-agent.wxs + new installer/gururmm-agent.ico (PR #35).
- Production: dashboard rebuilt+rsynced (v0.2.36, index-BAUBO9gN.js); Windows agent base MSI rebuilt on Pluto with ARP metadata.
- Root submodule pointer bumps: 424d610 (both PRs).
### Commands & Outputs
- Forced Windows build: emptied last-built-commit-windows → `build-windows.sh` → "Windows build complete: v0.6.50 in 623s", gururmm-agent-base-0.6.50.msi marked stable.
- MSI verification (msiinfo Property table): ARPPRODUCTICON=GuruRMMIcon.ico, ARPURLINFOABOUT=https://rmm.azcomputerguru.com, ARPHELPLINK=https://azcomputerguru.com, ARPCONTACT=Arizona Computer Guru LLC, ARPNOREPAIR=1, ARPNOMODIFY=1.
- MSI: /var/www/gururmm/downloads/gururmm-agent-base-0.6.50.msi (2,654,208 bytes, built 04:03 UTC). last-built-commit-windows now 4bca9f3.
### Infrastructure & Servers
- PLUTO Windows build host: agent id 07a11ece-c744-4cf5-a845-2c5366e36471; build SSH from .30 as root using /root/.ssh/id_ed25519, pinned via /opt/gururmm/pluto_known_hosts.
- gururmm-webhook runs as root, invokes `sudo /opt/gururmm/build-windows.sh`.
### Pending / Incomplete Tasks
- **BUG-015 real-world check:** install gururmm-agent-base-0.6.50.msi on a Win10/11 box, confirm it appears in Programs and Features with icon + uninstall works. Needs a clean Windows test machine (existing agents are site-configured; base MSI has no site key).
- **Mike (todo a47884d7):** extend the build change-gate to watch installer/ (and reconcile stale scripts/ repo copies vs /opt/gururmm).
- **Optional:** flip FEATURE_ROADMAP.md BUG-015 status Open→Fixed (doc PR).
### Reference Information
- PRs: #34 (a11y, merge 4923e9f), #35 (BUG-015, merge 4bca9f3, .ico commit 07d14bd). Root bump 424d610. gururmm main tip 4bca9f3.
- Coord: dashboard component v0.2.36; todo 15a5440f done; todo a47884d7 (gate fix, mike). PLUTO agent 07a11ece.