fix(config): watch this fork, not only upstream - #72
Open
Svector-anu wants to merge 1 commit into
Open
Conversation
memory/watched-repos.md still held the untouched upstream default of just aeonfun/aeon — git log on the file shows only inherited upstream commits, none fork-specific. every skill that resolves an empty var through it was therefore pointed at upstream rather than this fork: - auto-merge logged "Repo(s): aeonfun/aeon" and "0/0 qualifying" on every run. 20 runs, 19 successes, 95% success rate, quality score 3, and it has merged nothing, ever. a daily no-op reporting healthy. - pr-review with an empty var reviewed upstream's queue instead of ours. - feature's default `watched` selector resolved to upstream, which is why aeon.yml keeps it dispatch-only with a note that a bare dispatch "would auto-PR upstream" — a symptom of this, not a separate choice. add the fork alongside upstream rather than replacing it, so nothing that currently happens stops happening. immediate blast radius is small and was verified before making the change: auto-merge requires a bot author AND green checks, and the only bot-authored PRs here (#3, #4, #5, #20, #22) are all mergeable=false / dirty, so they are skipped as blocked. no new bot PRs are being created either — skills now commit straight to main rather than opening PRs, so the old PR-per-run pattern that produced those five is gone. feature is workflow_dispatch-only and does not act on its own. the real behavioural change is pr-review reviewing this fork's own queue, which posts comments. revert this commit to restore the previous targeting.
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.
what
memory/watched-repos.mdstill held the untouched upstream default — justaeonfun/aeon.git logon that file shows only inherited upstream commits, none fork-specific. So every skill resolving an emptyvarthrough it was pointed at upstream, not this fork.what that broke
auto-mergeis a no-op that reports healthy. It logsRepo(s): aeonfun/aeonandAUTO_MERGE_SKIP: 0/0 qualifyingevery run. 20 runs, 19 successes, 95% success rate, quality score 3 — and it has merged nothing, ever.pr-reviewwith an empty var reviewed upstream's queue instead of ours.feature's defaultwatchedselector resolved to upstream — which is whyaeon.ymlkeeps it dispatch-only with the note that a bare dispatch "would auto-PR upstream." That caveat is a symptom of this, not an independent design choice.the change
One line: add the fork alongside upstream rather than replacing it, so nothing that currently happens stops happening.
blast radius — verified before changing, not assumed
mergeable=false/dirty→ skipped as blocked.main(chore(<skill>): auto-commit) rather than opening a PR per run — the pattern that produced those five is gone.featureisworkflow_dispatch-only and does not act on its own.pr-reviewreviewing this fork's own queue, which posts comments.Revert this commit to restore the previous targeting.