Skip to content

feat(projects): add pre-build stats fetching#68

Merged
CalvinAllen merged 1 commit intomainfrom
feat/projects/prebuild-stats
Jan 31, 2026
Merged

feat(projects): add pre-build stats fetching#68
CalvinAllen merged 1 commit intomainfrom
feat/projects/prebuild-stats

Conversation

@CalvinAllen
Copy link
Collaborator

Summary

  • Add scripts/fetch-project-stats.js to fetch GitHub and marketplace stats as a pre-build step
  • Stats are fetched based on project category:
    • nuget-package: GitHub + NuGet
    • vs-extension/vscode-extension: GitHub + VS Marketplace
    • cli-tool/desktop-app/documentation/github-action: GitHub only
  • Update workflow to use build:with-stats with GITHUB_TOKEN for authenticated GitHub API calls
  • Update projects.ts to read from cached stats JSON instead of fetching at build time
  • Remove marketplace-stats.ts (replaced by pre-build script)
  • Fix NuGet package URLs to use correct CodingWithCalvin. prefix
  • Add npm scripts: fetch:stats, build:with-stats

Test plan

  • Verify npm run fetch:stats fetches stats correctly with category-based logic
  • Verify npm run build:with-stats runs fetch then build
  • Verify GitHub Actions builds successfully with GITHUB_TOKEN
  • Verify stats appear on project listing and detail pages

- Add scripts/fetch-project-stats.js to fetch GitHub and marketplace stats before build
- Stats are fetched based on project category:
  - nuget-package: GitHub + NuGet
  - vs-extension/vscode-extension: GitHub + VS Marketplace
  - cli-tool/desktop-app/documentation/github-action: GitHub only
- Update workflow to use build:with-stats with GITHUB_TOKEN
- Update projects.ts to read from cached stats JSON
- Remove marketplace-stats.ts (replaced by pre-build script)
- Fix NuGet package URLs to use correct CodingWithCalvin. prefix
- Add npm scripts: fetch:stats, build:with-stats
@CalvinAllen CalvinAllen merged commit 4b2bc12 into main Jan 31, 2026
1 check passed
@CalvinAllen CalvinAllen deleted the feat/projects/prebuild-stats branch January 31, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant