Skip to content

Conversation

@iscekic
Copy link
Collaborator

@iscekic iscekic commented Dec 5, 2025

Context

Add back the update-contributors script. We want to highlight the amazing members of our community!

@iscekic iscekic self-assigned this Dec 5, 2025
@changeset-bot
Copy link

changeset-bot bot commented Dec 5, 2025

⚠️ No Changeset found

Latest commit: 47f2b70

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@iscekic
Copy link
Collaborator Author

iscekic commented Dec 5, 2025

I ran the script, this is what it produced: https://github.com/Kilo-Org/kilocode/blob/d490746ff09a4f2e94f60501b7b26b489af3ef9c/README.md#contributing

The contributors are sorted by contributions desc.

@iscekic iscekic marked this pull request as ready for review December 5, 2025 13:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR reintroduces the update-contributors script to automatically fetch and display GitHub contributors in the README. The script replaces a previous implementation that fetched data from an external JSON endpoint with a direct GitHub API integration.

Key changes:

  • Rewrites scripts/update-contributors.js to use the GitHub API directly instead of fetching from kilo.ai/contributors.json
  • Changes from ES modules to CommonJS for consistency with other scripts in the repository
  • Adds an npm script update-contributors to package.json for easy execution
  • Updates README.md with a new table-based contributor display format (6 columns) and filters out bot accounts

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
scripts/update-contributors.js Complete rewrite of the contributors script using GitHub API, CommonJS modules, and improved error handling
package.json Adds update-contributors npm script for convenient execution
README.md Updates contributors section with new auto-generated table format displaying all contributors

Copy link
Contributor

@kiloconnect kiloconnect bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No Issues Found

3 files reviewed | Confidence: 90% | Recommendation: Merge

Review Details

Files: scripts/update-contributors.js, package.json, README.md

Checked: Security, bugs, error handling, code quality

Notes:

  • Previous Copilot review comments have been addressed or intentionally dismissed by the author
  • The Bearer token format is now correct (line 19)
  • The script properly handles GitHub API errors with status code checks
  • Bot filtering logic correctly excludes [bot] and R00-B0T accounts
  • The 100 contributor limit is acceptable per author's decision
  • README.md changes are auto-generated content with proper markers

Minor observations (not blocking):

  • The updateReadme function returns undefined when markers are missing, but author confirmed this fail-hard behavior is intentional
  • No success message is logged after completion, but author confirmed this is acceptable

@iscekic iscekic enabled auto-merge December 9, 2025 20:01
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.

3 participants