Skip to content

chore: regenerate tend workflows with 0.1.13 - #6126

Merged
max-sixty merged 2 commits into
PRQL:mainfrom
max-sixty:tend-0.1.13
Aug 3, 2026
Merged

chore: regenerate tend workflows with 0.1.13#6126
max-sixty merged 2 commits into
PRQL:mainfrom
max-sixty:tend-0.1.13

Conversation

@max-sixty

Copy link
Copy Markdown
Member

Regenerates the tend-*.yaml workflows against tend 0.1.13. Generated output only — no hand edits.

The change that matters is that every workflow now names environment: tend, so the bot token and the model credential can live in a deployment environment gated to main instead of as repo-level secrets, which any workflow on any branch can read. tend-mention gains a secretless relay job to go with it: a review or review-comment event runs on refs/pull/N/merge, a ref no deployment-branch policy can admit, so relay holds no secrets and re-posts only identifiers as a repository_dispatch, which verify then re-reads from the API.

Merging this on its own changes nothing operationally. The tend environment already exists on this repo with TEND_BOT_TOKEN in it, and a job naming an environment can still read repo-level secrets, so the gate closes only when the repo-level copies are deleted — a follow-up once the model credential is in the environment too.

This was written by Claude Code on behalf of max-sixty

@prql-bot prql-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generated-output review: the bump is consistent (all eight tend-*.yaml files move 0.1.11 → 0.1.13 in both the header and the action ref), and the substantive 0.1.13 additions — the secretless relay job + repository_dispatch re-entry, environment: tend on the secret-bearing jobs, the tend-review base-tree setup checkout, and the tend-notifications retry-with-logging — all read as coherent.

One blocker, though: this branch was generated against a 0.1.11 base (the diff is 0.1.11 → 0.1.13), but main already moved to 0.1.12 via #6101. So the PR conflicts with main (mergeable: CONFLICTING), and that's what pre-commit.ci - pr is surfacing as error during mergeable check rather than a real lint failure — every tend-*.yaml was touched on both sides.

It's not a duplicate — main at 0.1.12 has neither the relay job nor environment: gating, so 0.1.13 is a genuine new step. It just needs regenerating on top of current main so the diff becomes 0.1.12 → 0.1.13. Since these are generated files, a plain git rebase will fight the overlapping hunks; cleaner to branch off fresh origin/main and re-run uvx tend@0.1.13 init, then force-push. Happy to do that regeneration and push it to the branch if you'd like — just say the word.

# Conflicts:
#	.github/workflows/tend-ci-fix.yaml
#	.github/workflows/tend-mention.yaml
#	.github/workflows/tend-nightly.yaml
#	.github/workflows/tend-notifications.yaml
#	.github/workflows/tend-review-runs.yaml
#	.github/workflows/tend-review.yaml
#	.github/workflows/tend-triage.yaml
#	.github/workflows/tend-weekly.yaml
@max-sixty
max-sixty merged commit b787ce2 into PRQL:main Aug 3, 2026
43 checks passed
@max-sixty
max-sixty deleted the tend-0.1.13 branch August 3, 2026 06:15
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