Skip to content

ci: open nightly-failure issues with prql-bot PAT#5770

Merged
max-sixty merged 3 commits intomainfrom
fix/nightly-failure-issue-bot-token
Apr 5, 2026
Merged

ci: open nightly-failure issues with prql-bot PAT#5770
max-sixty merged 3 commits intomainfrom
fix/nightly-failure-issue-bot-token

Conversation

@prql-bot
Copy link
Copy Markdown
Collaborator

@prql-bot prql-bot commented Apr 4, 2026

Summary

Fixes the gap @max-sixty flagged in #5769: the nightly-failure issue was opened via the default GITHUB_TOKEN, whose events do not cascade to other workflows, so tend-triage never ran on it.

Two small changes:

  1. .github/workflows/tests.yaml — create the nightly-failure issue with PRQL_BOT_GITHUB_TOKEN instead of GITHUB_TOKEN, so the issues: opened event cascades.
  2. .github/workflows/tend-triage.yaml — the existing filter github.event.issue.user.login != 'prql-bot' would still skip the newly-bot-authored issue, so narrow it to allow issues labeled github_actions (the label set by .github/nightly-failure.md).

Note: tend-triage.yaml is normally regenerated by uvx tend init; I added a comment flagging the local customization so it isn't silently clobbered on regen. If there's a better upstream place for this, happy to move it.

Test plan

  • Next nightly-upstream failure opens an issue authored by prql-bot, which triggers tend-triage.
  • Existing non-github_actions prql-bot issues (e.g. devcontainer bumps) continue to be skipped by triage.

🤖 Generated with Claude Code

prql-bot and others added 2 commits April 4, 2026 22:23
The nightly-failure issue was created with the default GITHUB_TOKEN,
whose events do not cascade to other workflows — so tend-triage never
ran on it. Switch to PRQL_BOT_GITHUB_TOKEN so triage fires.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The previous commit switched nightly-failure issues to be opened by
prql-bot so the event would cascade to tend-triage, but tend-triage's
own filter skips all prql-bot-authored issues. Narrow that filter to
still allow issues labeled `github_actions` (the nightly-failure label)
through.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@max-sixty
Copy link
Copy Markdown
Member

the first one is generated by tend. put an issue into the tend repo that we need better guidance in the workflow headers on how repos should make changes: they should edit their skills and not change the workflows, which are autogenerated

@prql-bot
Copy link
Copy Markdown
Collaborator Author

prql-bot commented Apr 5, 2026

Filed max-sixty/tend#149 for the header-guidance request.

On the local tend-triage.yaml edit in this PR: happy to revert it once tend supports the label-based filter upstream, but leaving it for now so the nightly gets triaged in the meantime — let me know if you'd rather drop it and wait.

@max-sixty
Copy link
Copy Markdown
Member

drop the first change

@prql-bot
Copy link
Copy Markdown
Collaborator Author

prql-bot commented Apr 5, 2026

Dropped the tend-triage.yaml edit (reverted in 1a8649c) — will wait for the upstream fix in max-sixty/tend#149. The tests.yaml token change remains.

@max-sixty max-sixty merged commit 043b53e into main Apr 5, 2026
40 checks passed
@max-sixty max-sixty deleted the fix/nightly-failure-issue-bot-token branch April 5, 2026 03:13
max-sixty pushed a commit to max-sixty/tend that referenced this pull request Apr 7, 2026
… YAML (#150)

Closes #149.

Extends the generated-workflow header with explicit guidance pointing
downstream maintainers at repo-local skills or upstream issues, rather
than hand-editing the YAML (which gets silently reverted on the next
`uvx tend init`). Motivated by
[PRQL/prql#5770](PRQL/prql#5770), where a
triage-filter fix went into the generated workflow file.

New header (applied to all generated workflows):

```
# Generated by tend. Regenerate with: uvx tend@latest init
#
# Do not edit this file directly — it will be overwritten on regeneration.
# To customize behavior, edit the relevant skill in this repo's
# .claude/skills/ directory, or open an issue at
# https://github.com/max-sixty/tend/issues for changes that need to
# happen upstream in the tend-ci-runner plugin.
```

## Test plan

- [x] `cd generator && uv run pytest` (112 passed)
- [x] `uvx tend init` regenerates all 8 workflows with the new header

---------

Co-authored-by: tend-agent <270458913+tend-agent@users.noreply.github.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants