Repository navigation
Merges Past Their Checks reads the family's repositories alone - #280
Merged
Merged
Conversation
The entry said the KPI reads "my repositories", and the App was installed on the whole robertblust organization, so the weekly object also carried xiny, cdo-server and cdo-web, none of them a family member. The entry now says the App reads the family's repositories in robertblust and is installed on those alone, and What it can hide names what that costs: a new family repository counts only once it is added to the installation. Verified: `npx -y github:companygraph/meta-model#v0.88.0 check`, `sh conventions/conventions-check` and `sh conventions/conventions-format check` passed, and the KPI schema's writing rules were read against the change. Process: Delivery Phase: Implement Track: Prose Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The entry for Merges Past Their Checks said the weekly workflow reads "my repositories", and the KPI App was installed on the whole robertblust organization, so the object for 2026-W40 also carried robertblust/xiny (archived, listed as unread), cdo-server and cdo-web, none of which is a member of the family in conventions' REPOSITORIES.md. The entry now says the App reads the family's repositories in robertblust and is installed on those alone.
What it can hide gains the cost of that choice: a family repository the App is not installed on is never read, so a new member counts only once it is added to the installation.
The installation itself is changed by the owner in the organization's settings (GitHub Apps, the KPI App, Repository access, only the six members); until then the entry is ahead of what the workflow reads. W40's counts do not move, since its one bypass was in mental-model.
Verified:
npx -y github:companygraph/meta-model#v0.88.0 check,sh conventions/conventions-checkandsh conventions/conventions-format checkpassed, and the KPI schema's writing rules were read against the change.🤖 Generated with Claude Code