Refactor/v0.2.0 best practices

[arvarik]

Updating dashboard to include new stats and displaying onto dashboard

[Copilot]

Pull request overview

This PR refactors the eero-stats daemon toward a more production-ready v0.2.0 setup: tiered polling with structured writers, build-time version metadata, improved CI/dev tooling, and a regenerated Grafana dashboard.

Changes:

• Implemented tiered polling (fast/medium/slow) with new InfluxDB writer functions and retry logic (+ tests).
• Added build/version metadata plumbing (Go internal/version, Makefile ldflags, Dockerfile build args) and improved operational boot/shutdown logging.
• Replaced the previous Grafana dashboard with a generated “Eero Network Telemetry” dashboard + generator script; updated docs and tooling (CI, golangci-lint config, pre-commit hook).

Reviewed changes

Copilot reviewed 21 out of 22 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
internal/poller/poller.go	Refactors polling loop into fast/medium/slow tiers with panic-safe wrappers.
internal/poller/writers.go	Adds InfluxDB measurements/tags/fields for devices, nodes, network health/config, profiles.
internal/poller/retry.go + *_test.go	Adds exponential backoff retry helper with unit tests.
internal/auth/auth.go	Improves session caching behavior (mkdir, error handling) and clarifies path handling.
internal/db/influx.go	Documents and configures aggressive batching; adds Shutdown API (context parameter).
scripts/build_dashboard.py	New dashboard generator producing Grafana JSON with Flux queries and panels.
grafana/dashboards/eero.json	New provisioned dashboard JSON (generated).
grafana/dashboards/eero-overview.json	Removes old dashboard.
internal/version/version.go	Adds build-time version/commit/date variables for ldflags injection.
cmd/eero-stats/main.go	Logs version metadata; improves graceful shutdown and Influx shutdown helper.
README.md	Updates architecture, setup instructions, and new dashboard positioning.
Makefile / Dockerfile	Adds ldflags/versioning, tests target, dashboard target; pins Alpine runtime.
.github/workflows/ci.yml / .golangci.yml / .githooks/pre-commit	Tightens CI checks, lint config, and local pre-commit verification.
.dockerignore	Adds ignores for local data and env files.

Comments suppressed due to low confidence (1)

internal/db/influx.go:56

• Shutdown accepts a context and the comment states it can be used for timeout control, but the implementation ignores ctx. Either remove the context parameter/comment, or honor ctx (e.g., by running Flush/Close in a goroutine and returning early when ctx.Done() fires).

// Shutdown flushes all buffered writes to disk and closes the InfluxDB
// connection. The provided context can be used for timeout control.
func (i *InfluxClient) Shutdown(ctx context.Context) {
	slog.Info("Flushing InfluxDB memory buffers to disk...")
	i.WriteAPI.Flush()
	i.Client.Close()
	slog.Info("InfluxDB connection closed")

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.