Skip to content

fix(stats): wire clearpath as a live producer (statsPath)#113

Merged
GmanFooFoo merged 1 commit into
mainfrom
fix/clearpath-live-producer-statspath
Jul 5, 2026
Merged

fix(stats): wire clearpath as a live producer (statsPath)#113
GmanFooFoo merged 1 commit into
mainfrom
fix/clearpath-live-producer-statspath

Conversation

@GmanFooFoo

@GmanFooFoo GmanFooFoo commented Jul 5, 2026

Copy link
Copy Markdown
Member

What

The estate stats showed clearpath as total 0 / seeded — its stats-config.json entry had no statsPath, so update-stats never fetched its producer stats.json. Pre-existing (clearpath-52 was seeded 0 too), but it hid the real count.

The clearpath repo now ships a working stats.yml producer emitting stats.json (total 39, repo neckarshore-mmps/clearpath). Adding "statsPath": "stats.json" makes update-stats fetch it; the aggregate's live-wins-by-slug dedup drops the redundant seed row automatically (no double-count). The estate number becomes honest (~2611 → ~2650).

Verification

stats-config.json valid · test:unit 81/81 · check:privacy OK.

🤖 Generated with Claude Code

https://claude.ai/code/session_013Sxyx5TvFWqVXYMfZoLztw

Summary by CodeRabbit

  • Chores
    • Updated repository configuration so the Clearpath project now includes the additional stats information needed for reporting.

The estate stats showed clearpath as total 0 / seeded — because its stats-config
entry had no statsPath, so update-stats never fetched its producer stats.json.
This was pre-existing (clearpath-52 was seeded 0 too) but hid the real count: the
clearpath repo now ships a working stats.yml producer emitting stats.json
(total 39, repo neckarshore-mmps/clearpath). Adding statsPath makes update-stats
fetch it; the aggregate's live-wins-by-slug dedup drops the redundant seed row
automatically (no double-count), and the estate number becomes honest.

Verified: stats-config.json valid, test:unit 81/81, check:privacy OK.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013Sxyx5TvFWqVXYMfZoLztw
@vercel

vercel Bot commented Jul 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
neckarshore-website Ready Ready Preview, Comment Jul 5, 2026 2:50pm

Request Review

@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b616775c-1a76-45dc-9bc1-b4ad97265992

📥 Commits

Reviewing files that changed from the base of the PR and between ec323b1 and b76a1d4.

📒 Files selected for processing (1)
  • stats-config.json

📝 Walkthrough

Walkthrough

The clearpath repository entry in stats-config.json was updated to include a new statsPath field set to "stats.json", previously absent from the configuration.

Changes

Stats Config Update

Layer / File(s) Summary
Add statsPath field
stats-config.json
Adds a statsPath property ("stats.json") to the clearpath repo entry, which previously only had owner, name, and private fields.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Compact metadata

  • Type: Configuration change
  • Files changed: 1
  • Lines changed: +1/-1
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: wiring clearpath as a live producer by adding statsPath.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/clearpath-live-producer-statspath

Comment @coderabbitai help to get the list of available commands.

@GmanFooFoo GmanFooFoo merged commit 453278e into main Jul 5, 2026
10 checks passed
@GmanFooFoo GmanFooFoo deleted the fix/clearpath-live-producer-statspath branch July 5, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant