Skip to content

skills(running-tend): tighten transient-failure disposition to an allowlist#2741

Closed
tend-agent wants to merge 1 commit into
max-sixty:mainfrom
tend-agent:skills/transient-disposition-allowlist
Closed

skills(running-tend): tighten transient-failure disposition to an allowlist#2741
tend-agent wants to merge 1 commit into
max-sixty:mainfrom
tend-agent:skills/transient-disposition-allowlist

Conversation

@tend-agent

Copy link
Copy Markdown

Tighten the transient-failure disposition rule in the running-tend overlay so it implicitly excludes future bundled recipes — including the bundled ci-fix skill's section 3a open-and-close template that has been firing as the default.

The previous text forbade a PR explicitly ("do not create a PR") but only suggested "comment on the run or exit silently" as alternatives without ruling out other artifacts. Bundled tend-ci-fix section 3a (added in max-sixty/tend#367) instructs the bot to open and immediately self-close an audit issue when it classifies a failure as transient — and over the last 24h that template has fired three times on max-sixty/worktrunk (#2694, #2716, #2720), each opened and self-closed within ~4 seconds.

Switching the overlay to an allowlist of permitted dispositions (comment on the run, exit silently) names the surface that's allowed rather than enumerating what's forbidden. Anything the bundled skill suggests outside that set is implicitly excluded.

Motivated by the discussion at max-sixty/tend#426 — closed in favor of this tightening rather than a bundled-skill change, since "bundled skills as defaults, overlays override" is the model. This PR is the overlay override.

- **Transient infrastructure** (...) — do **not** create a PR. The maintainer will rerun CI. Comment on the run or exit silently; a permanent config change for a one-off timeout is churn the maintainer will close.
+ **Transient infrastructure** (...) — permitted dispositions are (1) comment on the run, or (2) exit silently. Do not open a PR, an audit issue, or any other artifact. The maintainer will rerun CI; a permanent config change for a one-off timeout is churn the maintainer will close.

…owlist

The previous text forbade a PR but did not explicitly forbid an audit
issue, so bundled tend ci-fix section 3a's open-and-close template
fired as the default. Switch to an allowlist (comment on the run, or
exit silently) so future bundled recipes the overlay hasn't anticipated
are implicitly excluded.

See max-sixty/tend#426 for the discussion that motivated this.
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.

3 participants