# GuruRMM — Agents Sidebar Nav Link + Mike's Spec Decisions

## User
- **User:** Howard Enos (howard)
- **Machine:** Howard-Home
- **Role:** tech

---

## Session Summary

Session opened with an unread coord message from Mike (GURU-5070, e72b2145) responding to the SPEC-013 and SPEC-014 feature specs submitted earlier in the day. Mike's decisions: SPEC-013 (file browser) deferred until file transfer (P2) ships first — both features share agent-side filesystem logic and building the browser now wastes work; SPEC-014 (event log viewer) approved with Phase 1 PowerShell `Get-WinEvent` relay (native Rust Phase 2 deferred). Mike also noted that the log analysis deduplication pattern (one row + count badge, expandable, resolve-all-duplicates) should be applied to SPEC-014 event grouping in the Events tab UI.

SPEC-013 spec status updated to "Deferred — pending file transfer (P2) implementation" and the roadmap entry updated to flag DEFERRED. Coord message marked as read.

Howard submitted a feature request: the sidebar navigation has no Agents quicklink — the `/agents` page is reachable by URL but not from the left nav, which jumps from Clients directly to Scripts. Fix applied directly to `dashboard/src/components/Layout.tsx`: imported `Monitor` from lucide-react and added `{ path: "/agents", label: "Agents", icon: Monitor }` to the MANAGEMENT nav section between Clients and Scripts. The `SEARCH_SCOPES` map at line 394 already had an `/agents` entry — no change needed there. Change committed to gururmm as `daa14e2`, pushed, submodule advanced in claudetools.

---

## Key Decisions

- **Direct fix, no spec** — adding a single nav link entry is a UI gap fix, not a feature; spec overhead not warranted for a one-line change.
- **Monitor icon** — most semantically appropriate for "agent machines being monitored"; not already used in the sidebar nav (Server icon is used in the brand logo header only).
- **SPEC-013 deferred per Mike** — updating the spec status and roadmap entry immediately keeps them accurate for future sprint planning; no further action needed until file transfer (P2) ships.
- **SPEC-014 Phase 1 confirmed** — PowerShell relay is approved; note Mike's guidance that event deduplication UI should follow the log analysis pattern (count badge + expandable row + resolve-all).

---

## Configuration Changes

- `projects/msp-tools/guru-rmm/dashboard/src/components/Layout.tsx` — added `Monitor` import, added Agents nav entry (MANAGEMENT section, between Clients and Scripts)
- `projects/msp-tools/guru-rmm/docs/specs/SPEC-013-file-browser-disk-explorer.md` — status updated to Deferred; Mike's decision noted
- `projects/msp-tools/guru-rmm/docs/FEATURE_ROADMAP.md` — SPEC-013 entry updated to flag DEFERRED
- `projects/msp-tools/guru-rmm` (submodule) — advanced to commit daa14e2

---

## Credentials & Secrets

None created or modified this session.

---

## Infrastructure & Servers

- **GuruRMM dashboard:** https://rmm.azcomputerguru.com
- **Coord API:** http://172.16.3.30:8001/api/coord

---

## Commands & Outputs

```bash
# Mark Mike's coord message read
PUT /api/coord/messages/e72b2145-b518-4f16-b78e-5ff84f29126c/read → OK

# gururmm commit
git commit -m "feat(dashboard): add Agents link to sidebar nav (MANAGEMENT section)"
# → [main daa14e2] — 3 files changed, 5 insertions(+), 3 deletions(-)

# Push (no rebase needed)
git push origin main
# → 90e37f2..daa14e2

# Submodule advance
git commit -m "chore(submodule): advance guru-rmm — Agents sidebar nav link"
# → [main ba2e16d]
```

---

## Pending / Incomplete Tasks

- **SPEC-014 implementation (P2):** Event log viewer approved (PowerShell Phase 1). Ready for sprint. Note Mike's guidance: event deduplication in the Events tab should use count-badge + expandable-row + resolve-all pattern (from log analysis thread).
- **SPEC-012 implementation (P2):** Sortable table headers, 4h estimate, no blockers.
- **SPEC-013 (P3):** Deferred — revisit after file transfer (P2) ships.
- **WinRing0 cleanup:** Closed by Mike (todo 42c08298). Done.
- **Cascades pending:** Ashley Jensen folder redirect verification, RECEPTIONIST-PC Q:/W: drives, NURSESTATION-PC HIPAA GPO, Nurses credential vault, Phase 3 domain joins, Entra Connect OU=Administrative expansion, M365 relicensing (31 seats Business Standard → Business Premium — time-sensitive).

---

## Reference Information

- **Layout.tsx change:** `dashboard/src/components/Layout.tsx:77` (Agents nav entry)
- **gururmm commit:** daa14e2 (Agents nav + SPEC-013 deferral)
- **claudetools commit:** ba2e16d (submodule advance)
- **Coord message marked read:** e72b2145 (Mike's SPEC-013/014 decisions)
- **SPEC-013:** `docs/specs/SPEC-013-file-browser-disk-explorer.md` — Status: Deferred
- **SPEC-014:** `docs/specs/SPEC-014-windows-event-log-viewer.md` — Status: Proposed, P2, Phase 1 PowerShell approved
- **Event dedup pattern for SPEC-014:** one row + count badge, expandable, resolve-all-duplicates (per Mike, from log analysis thread)
