Skip to content

Count views from both Clarity and GitHub folder traffic - #582

Merged
Alejandro Lopez (soyalejolopez) merged 1 commit into
masterfrom
combine-github-and-clarity-views
Nov 2, 2026
Merged

Alejandro Lopez (soyalejolopez) merged 1 commit into
masterfrom
combine-github-and-clarity-views

Conversation

@soyalejolopez

Copy link
Copy Markdown
Member

Clarity replaced a resource's GitHub-based view count as soon as it recorded a single view, which wiped out the older, larger counts (PowerClaw fell from 1,632 views to 4, Purview Audit Copilot Report from 656 to 2) and left Update-VivaEngageLicensingToggle, which Clarity has never seen, on top of Popular with a GitHub-only 222.

Views are now the sum of the two surfaces: catalog page views from Clarity and views of the resource's GitHub folder from the saved traffic snapshots. Neither sees the other's visits.

  • combineViews() in stats-state.js, used by build-stats.js, with unit tests
  • resource-stats.json re-rendered from the persisted data
  • hover text on every view count explains the sources and that the count is a minimum
  • traffic-data/README.md documents how views are counted and which days were never captured

Clarity replaced a resource's GitHub-based view count as soon as it recorded a single view, which
wiped out the older, larger counts (PowerClaw fell from 1,632 views to 4, Purview Audit Copilot
Report from 656 to 2) and left Update-VivaEngageLicensingToggle, which Clarity has never seen,
on top of Popular with a GitHub-only 222.

Views are now the sum of the two surfaces: catalog page views from Clarity and views of the
resource's GitHub folder from the saved traffic snapshots. Neither sees the other's visits.

- combineViews() in stats-state.js, used by build-stats.js, with unit tests
- resource-stats.json re-rendered from the persisted data
- hover text on every view count explains the sources and that the count is a minimum
- traffic-data/README.md documents how views are counted and which days were never captured

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@soyalejolopez
Alejandro Lopez (soyalejolopez) merged commit 7dd34d8 into master Nov 2, 2026
4 checks passed
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

🛰️ PR Sweeper report

Risk: 🟢 LOW · Security gate: ✅ passing · Files: 6

🔒 Automated guardrails (authoritative)

Category Location Finding
ℹ️ scope — Change(s) outside known content roots: index.html, resource-stats.json.

🤖 Dual-model AI review (advisory)

AI review unavailable for this run (models not reachable or no diff). Guardrails above are unaffected.

  • GPT-class reviewer (openai/gpt-5): skipped — request failed: Unexpected token 'O', "OK
    " is not valid JSON
  • Deep-reasoning reviewer (openai/o3): skipped — request failed: Unexpected token 'O', "OK
    " is not valid JSON

The automated guardrails are authoritative and gate the security status. The AI review is advisory and never auto-merges. Thanks for contributing to FastTrack! 🛩️

@github-actions github-actions Bot added sweeper:risk-low PR Sweeper: low risk sweeper:scope-review PR Sweeper: scope/sensitive-path review labels Nov 2, 2026
@soyalejolopez
Alejandro Lopez (soyalejolopez) deleted the combine-github-and-clarity-views branch November 2, 2026 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweeper:risk-low PR Sweeper: low risk sweeper:scope-review PR Sweeper: scope/sensitive-path review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant