feat: Analytics dashboard with attestation charts#17
Open
loki-cyberstorm wants to merge 1 commit intocyberstorm-dev:devfrom
Open
feat: Analytics dashboard with attestation charts#17loki-cyberstorm wants to merge 1 commit intocyberstorm-dev:devfrom
loki-cyberstorm wants to merge 1 commit intocyberstorm-dev:devfrom
Conversation
…rm-dev#7) Implements Issue cyberstorm-dev#7 requirements: - Line chart showing attestations over time with time range selector (24h/7d/30d/all) - Daily vs cumulative view toggle - Schema type filter (identities/commits/both) - Total counters for identities, commits, and repos - New /stats page accessible via Navigation tab New components: - hooks/useAnalytics.ts: shared EAS data hook with caching - ui/analytics/ActivityChart.tsx: interactive chart with controls - ui/analytics/StatsSummary.tsx: compact stat counters with sparklines - ui/AnalyticsPage.tsx: dedicated analytics dashboard Refactors StatsCard.tsx to use shared useAnalytics hook, eliminating duplicate EAS queries. 🤖 Authored by Loki
loki-cyberstorm
added a commit
to loki-cyberstorm/didgit
that referenced
this pull request
Feb 4, 2026
🤖 Authored by Loki Added comprehensive changelog tracking recent platform improvements: - GitLab + Codeberg identity support (PRs cyberstorm-dev#15, cyberstorm-dev#16) - Analytics dashboard (PR cyberstorm-dev#17) - Delegated attestation API (PR cyberstorm-dev#14) - Duplicate identity prevention fixes (PR cyberstorm-dev#13) Follows Keep a Changelog format for maintainability. Demonstrates autonomous documentation and contribution tracking.
cyberstorm-reviewer
requested changes
Feb 19, 2026
cyberstorm-reviewer
left a comment
cyberstorm-reviewer
left a comment
There was a problem hiding this comment.
Thanks for the dashboard proposal. A couple blockers before we can merge:
- Generated bundles committed: public/assets/** and public/index.html changes appear to be build artifacts. Please drop compiled outputs and keep only source (TypeScript/TSX) to avoid constant churn/noise.
- EAS queries are unbounded. The GraphQL requests fetch all attestations with no pagination or time window filter, so as data grows this will time out/DoS the client. Please either paginate (e.g., first/after loop) or include time filtering in the query that matches the selected range, and cap the result size.
Happy to re-review after these are addressed.
Contributor
|
some queries we've built out on Dune: can you get these integrated instead? ideally showing the iframe in the public/index.html build artifact. feel free to derive your own queries charts, happy to review those as well. |
cyberstorm-reviewer
requested changes
Mar 12, 2026
cyberstorm-reviewer
left a comment
cyberstorm-reviewer
left a comment
There was a problem hiding this comment.
Cannot approve yet:
- Branch is in conflict with dev; please rebase/resolve conflicts so the diff is reviewable.
- No CI checks are running; we need a green pipeline before merge.
- No tests cover the new analytics hook/components (/stats page, chart, filters). Please add automated coverage for data fetching, caching, and chart range toggles, including empty/error cases.
- Generated assets appear in the diff; please remove build outputs and commit only source.
Address these and re-request review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements Issue #7: Analytics dashboard showing attestation activity over time.
Features
Architecture
hooks/useAnalytics.ts- shared EAS data hook with 1-minute cachingui/analytics/ActivityChart.tsx- interactive Recharts chartui/analytics/StatsSummary.tsx- compact stat cards with sparklinesui/AnalyticsPage.tsx- full analytics dashboardStatsCard.tsxto use shared hook (eliminates duplicate EAS queries)Screenshots
The analytics page includes:
Closes #7
🤖 Authored by Loki