Skip to content

fix: scope governance activity tracking to repository-specific events#373

Open
Sanchit2662 wants to merge 1 commit into
LF-Decentralized-Trust-labs:mainfrom
Sanchit2662:fix/governance-activity-tracking
Open

fix: scope governance activity tracking to repository-specific events#373
Sanchit2662 wants to merge 1 commit into
LF-Decentralized-Trust-labs:mainfrom
Sanchit2662:fix/governance-activity-tracking

Conversation

@Sanchit2662
Copy link
Copy Markdown

Summary

I fixed the governance activity tracking bug by scoping it to only track activity in the gitmesh repository. Instead of checking a user's global GitHub activity (which included commits to any public repo), I now query four repo-specific endpoints: commits, pull requests, issue comments, and PR review comments. This ensures contributors are only marked inactive if they haven't done anything in this repository for 90+ days, not in other projects.

Related Issue

Fixes #356

Type of Change

  • fix

How Was This Tested?

  • Not tested (explain why)

Notes:
The governance workflow runs on a GitHub Actions schedule (weekly Friday at 01:30 UTC) and requires: 1. Live GitHub API access with a valid GITHUB_TOKEN
2. A real gitmesh repository clone with governance files
3. Network calls to GitHub's API endpoints

CE & Security Check

  • Targets GitMesh CE only (no EE code)
  • No secrets or credentials committed

  Replace global user GitHub activity API (/users/{username}/events/public) with
  repo-scoped queries for commits, PRs, and comments. This ensures contributors
  are only marked active/inactive based on their activity in the gitmesh repository,
  not unrelated activity in other public repositories.
                                                                                                                                                                       Fixes LF-Decentralized-Trust-labs#356

Signed-off-by: Sanchit2662 <sanchit2662@gmail.com>
@Sanchit2662
Copy link
Copy Markdown
Author

@RAWx18 , Please review the fix.

@RAWx18
Copy link
Copy Markdown
Member

RAWx18 commented Apr 20, 2026

@Ronit-Raj9 @Parvm1102

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.

bug: Governance workflow updates dates for users even if they have had no repo activity

2 participants