fix(pr-pipeline): name the missing dependencies permission when an intake edge write gets 403
Maintainers usually reply within 1 day
A pull request for this has already been merged.
- #10310 by @melodic-automation-lanes — merged
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- github-actions, javascript
- Domain
- ci-cd
Research direction
Look at .github/scripts/intake-triage/apply-verdict.mjs where the blocked-by edge write happens and the failure is handled (context in the review thread on #9799). When the GitHub API returns 403, keep failing closed but add the missing dependencies App permission and grant instructions to the run summary and any issue comment. Add a test covering the 403 path in the existing *.test.mjs files in the same directory and run them to confirm.
Written by the indexing model from the issue text.
Description
Refs: #9799
Problem
Intake's verdict step writes native blocked-by edges (#9799) and fails closed when the edge write fails. If the lanes App lacks the dependencies permission, the write returns 403 and every rerun of that issue fails the same way, with nothing that names the missing permission.
Done when
- A 403 on the blocked-by edge write still fails closed, and the run summary (and the issue comment, if one is posted) names the missing App permission and how to grant it.
- A test in
.github/scripts/intake-triage/*.test.mjscovers the 403 path.
Related
- Raised during review of #9799 (thread on
apply-verdict.mjsedge-write failure).
🤖 Generated with Claude Code
- Dominant language
- Shell
- Stars
- 22
- Forks
- 2
- Avg merge
- 2h 48m
- Merged PRs (30d)
- 939
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
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.
More from melodic-software/claude-code-plugins
-
priority: medium status: ready work-class: scoped
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
melodic-software/claude-code-plugins#10227 · 1 comment ·
Maintainers usually reply within 1 day
-
fix(evals): give the methodology rewrite answer a template whose target slot needs a stated basisOpenpriority: medium status: ready work-class: scoped
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
melodic-software/claude-code-plugins#10217 · 1 comment ·
Maintainers usually reply within 1 day
-
priority: medium status: ready work-class: scoped
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
melodic-software/claude-code-plugins#10128 · 1 comment ·
Maintainers usually reply within 1 day
-
pr-pipeline: clear a feedback-not-landed escalation once the head moves and no lane thread is openOpenpriority: medium status: needs-decision work-class: scoped
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
melodic-software/claude-code-plugins#10074 · 4 comments ·
Maintainers usually reply within 1 day
-
priority: medium work-class: scoped
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
melodic-software/claude-code-plugins#9842 · 5 comments ·
Maintainers usually reply within 1 day
All issues in melodic-software/claude-code-plugins
Similar issues
-
Update migenOpen
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
YosysHQ/oss-cad-suite-build#220 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
docker-library/redmine#430 ·
-
fm-spawn: isolation timeout hides the real treehouse get error and closes the pane that holds itOpen
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
kunchenguid/firstmate#7848 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Maintainers usually reply within 1 day
-
docs: state the LF Projects antitrust policy in the repositoryPossibly taken @vinoo999 claimed this today. Open
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
a2aproject/A2A#2394 ·
Maintainers usually reply within 3 days