fix: affected-alert remediation (private fork) never produces a fork — doesn't remediate
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 6
- Forks
- 2
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 17
Description
Problem
The affected-alert path (create advisory → private fork → clone → Claude-fix in the fork) doesn't work. create_advisory_and_fork frequently returns an empty fork name (the polling loop never finds the private fork), so the clone step got https://github.com/.git/ and hard-failed. That failure was the source of the re-investigation loop on mozilla/fxa.
The loop itself is now mitigated (#129/#130 tag on failure; empty fork routes to advisory_only), but affected alerts still get zero remediation — the path just fails cleanly instead of looping.
Root question
Why is the private fork empty? Candidates: advisory fork creation is async and the poll window/field path is wrong, or a permissions gap on repository-advisories.
Proposal
Two options:
- Fix fork detection/polling so the private-fork flow actually completes.
- Simpler + more robust: for affected alerts, open a normal bump PR on the main repo (same mechanism as transitive bumps) and only use the private-fork + Claude-fix flow when the fix genuinely needs code changes beyond a version bump. Most 'affected' transitive advisories are still resolved by bumping the dependency, so the fork dance is overkill for them.
Impact
Enables real remediation of affected alerts (currently none).
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at create_advisory_and_fork and trace the affected-alert path through fork polling, cloning, and the advisory_only fallback. Compare it with the existing transitive-bump mechanism and determine whether affected alerts should use that path or reliable private-fork detection. Done means affected alerts produce real remediation instead of an empty fork or clean failure.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github, python
- Domain
- devops, security
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100