feat(activity): coordinate heatmap refresh with other graphs #314

Merged
fuzzy merged 1 commit from phase05-heatmap-coordination into main 2026-07-07 11:11:20 +00:00
Owner

What

Add heatmap to the graph tracking system, completing #238 and Phase 5.

Changes

  • Added graphHeatmap to the graphKind enum
  • Heatmap now tracked in StaleGraphs() alongside PRs, Issues, Contributions
  • Uses same cache strategy (shared RefreshContext + GetRefreshWindow)
  • Re-renders on new data via shared renderSparklines() pipeline
  • Respects 7-day window via v.window.Days

Phase 5 complete

# Task Status
#234 Heatmap integration
#238 Heatmap coordination

References: #238, #235, #236, #237

Testing

  • 2 new tests: render includes heatmap section, heatmap shares cache with sparklines
  • go test -race ./... — all pass (0 races)
## What Add heatmap to the graph tracking system, completing #238 and **Phase 5**. ### Changes - Added `graphHeatmap` to the `graphKind` enum - Heatmap now tracked in `StaleGraphs()` alongside PRs, Issues, Contributions - Uses same cache strategy (shared `RefreshContext` + `GetRefreshWindow`) - Re-renders on new data via shared `renderSparklines()` pipeline - Respects 7-day window via `v.window.Days` ### Phase 5 complete | # | Task | Status | |---|------|--------| | #234 | Heatmap integration | ✓ | | #238 | Heatmap coordination | ✓ | References: #238, #235, #236, #237 ## Testing - [x] 2 new tests: render includes heatmap section, heatmap shares cache with sparklines - [x] `go test -race ./...` — all pass (0 races)
feat(activity): coordinate heatmap refresh with other graphs
All checks were successful
Test and Release / test (pull_request) Successful in 3m32s
Test and Release / lint (pull_request) Successful in 4m29s
6b407769c2
- Add graphHeatmap to graphKind enum for stale tracking
- Heatmap uses same cache strategy as sparklines (shared
  RefreshContext, GetRefreshWindow, Cache)
- Triggered re-render on new data via shared renderSparklines()
- Ensures 7-day window by using v.window.Days
- Heatmap section verified in render output via tests

This completes Phase 5.

Ref: #238, #235, #236, #237
fuzzy added this to the Steeper project 2026-07-07 11:10:05 +00:00
fuzzy self-assigned this 2026-07-07 11:10:05 +00:00
the.auditor left a comment

Review

Result: Approved — no blocking issues.

Summary: Adds graphHeatmap to the graphKind enum, tracking it alongside PR, Issue, and Contribution graphs. Heatmap now participates in StaleGraphs(), shares the same cache refresh strategy, and re-renders via the shared renderSparklines() pipeline. ROADMAP2.md updated — Phase 5 fully complete.

Checklist:

  • Correctness: graphHeatmap added to enum at position 0, all iteration loops pick it up automatically via graphCount
  • Completeness: Closes #238 — heatmap uses same cache, re-renders on data refresh, respects 7-day window
  • Test coverage: 2 new tests (heatmap section present in render, heatmap shares cache with sparklines)
  • Race-safe: go test -race passes
## Review **Result: Approved** — no blocking issues. **Summary:** Adds `graphHeatmap` to the `graphKind` enum, tracking it alongside PR, Issue, and Contribution graphs. Heatmap now participates in `StaleGraphs()`, shares the same cache refresh strategy, and re-renders via the shared `renderSparklines()` pipeline. ROADMAP2.md updated — Phase 5 fully complete. **Checklist:** - Correctness: ✅ graphHeatmap added to enum at position 0, all iteration loops pick it up automatically via `graphCount` - Completeness: ✅ Closes #238 — heatmap uses same cache, re-renders on data refresh, respects 7-day window - Test coverage: ✅ 2 new tests (heatmap section present in render, heatmap shares cache with sparklines) - Race-safe: ✅ `go test -race` passes
fuzzy merged commit 6b407769c2 into main 2026-07-07 11:11:20 +00:00
fuzzy deleted branch phase05-heatmap-coordination 2026-07-07 11:11:20 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
thwap/steeper!314
No description provided.