guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

fix(ci): use github.run_number in firebase-tests RUN_URL #561

Open
guettlibot wants to merge 1 commits from issue-560-fix-firebase-run-url into main
pull from: issue-560-fix-firebase-run-url
merge into: :main
Conversation 0 Commits 1 Files Changed 1
+1 -1
guettlibot commented 2026-06-11 03:12:39 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

The Firebase Tests workflow's auto-created failure issue links the run with ${{ github.run_id }}, but Forgejo's web URL for a run is /{owner}/{repo}/actions/runs/{run_number}. The result is that every Firebase-test failure issue ships with a broken (404) log link — issue #560 is the latest example: its link /actions/runs/4657175 404s instead of the working /actions/runs/2403.

Without a working log URL, neither agents nor humans can diagnose what actually failed in the run, so we cannot find the real Firebase test root cause for issue #560 (the failure may well be transient infra, but the diagnostic link is missing either way).

Change

Switch RUN_URL to use ${{ github.run_number }}, matching how run_number is already used in the runner-wait-time API queries elsewhere in this same workflow.

Test plan

  • CI passes.
  • Next Firebase Tests failure issue contains a working log link (/actions/runs/<run_number>).

Refs #560

🤖 Generated with Claude Code

## Summary The Firebase Tests workflow's auto-created failure issue links the run with `${{ github.run_id }}`, but Forgejo's web URL for a run is `/{owner}/{repo}/actions/runs/{run_number}`. The result is that every Firebase-test failure issue ships with a broken (404) log link — issue #560 is the latest example: its link `/actions/runs/4657175` 404s instead of the working `/actions/runs/2403`. Without a working log URL, neither agents nor humans can diagnose what actually failed in the run, so we cannot find the real Firebase test root cause for issue #560 (the failure may well be transient infra, but the diagnostic link is missing either way). ## Change Switch `RUN_URL` to use `${{ github.run_number }}`, matching how `run_number` is already used in the runner-wait-time API queries elsewhere in this same workflow. ## Test plan - [ ] CI passes. - [ ] Next Firebase Tests failure issue contains a working log link (`/actions/runs/<run_number>`). Refs #560 🤖 Generated with [Claude Code](https://claude.com/claude-code)
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin issue-560-fix-firebase-run-url:issue-560-fix-firebase-run-url
git checkout issue-560-fix-firebase-run-url
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#561