docs(tend): lead bug-triage asks with wt -vv diagnostic#2415
Merged
Conversation
Per #2410: the diagnostic report covers version, OS, git, shell integration, `wt config show`, `git worktree list --porcelain`, and a full git trace in one file. Asking for that gist URL up front is faster than chaining version/config/repro questions. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
3 tasks
max-sixty
pushed a commit
to max-sixty/tend
that referenced
this pull request
Apr 25, 2026
) ## Summary Update the `running-in-ci` "How to propose" recipe so a bot updating `.claude/skills/` can actually write to disk. The current recipe (`cd .claude && mv /tmp/... SKILL.md`) satisfies the Claude Code harness write-guard ([anthropics/claude-code#37157](anthropics/claude-code#37157)) but fails the read-only bind-mount the sandbox places on `.claude/`. Mirror the pattern `review-runs` already documents: do the edit, commit, and push from a git worktree under `$TMPDIR`. Both restrictions are now explained inline so a future reader knows why the worktree is mandatory. ## Why `worktrunk-bot` hit this in worktrunk run [24926120125](https://github.com/max-sixty/worktrunk/actions/runs/24926120125) (a `tend-mention` session triggered by an explicit maintainer instruction to update repo skill guidance). The bot followed the bundled recipe, hit `mv: ... Read-only file system`, retried with `cp` (same error), retried with `touch` (same error), then ad-libbed `git hash-object -w` plus `git update-index --cacheinfo` to write the blob and stage it directly to the index — a workaround that left the working tree showing the file as modified after the fact. ~7 wasted bash cycles before producing [worktrunk PR #2415](max-sixty/worktrunk#2415). `worktrunk-bot` followed the new bundled-skill-defect flow (introduced in #324 yesterday) and opened permission-request issue [max-sixty/worktrunk#2416](max-sixty/worktrunk#2416) on the consumer side. This PR is the corresponding tend-side fix surfaced by `review-reviewers` rather than waiting on the permission round-trip — the diagnosis is unambiguous and the fix is small. ## What changes The "Draft a minimal edit" step (step 3 of "How to propose"): - Drops the `cd .claude && mv /tmp/... SKILL.md` recipe that fails on the read-only mount. - Adds the same prose `review-runs` already carries explaining (a) the read-only bind-mount of `.claude/` and (b) the harness write-guard from claude-code#37157, so a reader sees both barriers and why a worktree clears both. - Replaces the recipe with the `git worktree add "$TMPDIR/skill-fix"` pattern, parameterised on `<topic>-$GITHUB_RUN_ID` for the branch name. - Keeps the existing TODO referencing claude-code#37157. The frontmatter snippet is moved up so the recipe sits at the bottom of step 3 with no interleaving. ## Test plan - [x] Eaten own dogfood: this PR's branch was authored from a worktree at `$TMPDIR/skill-fix` using exactly the recipe being landed; commit and push worked end-to-end. Worktree under `$TMPDIR` is writable, no `Read-only file system` errors. - [ ] CI on this PR passes (actionlint, ruff, typos, uv-lock). - [ ] Visual diff vs `review-runs` SKILL.md prose to confirm the two skills now agree on the recipe shape. Independent of [#327](#327) (different concern in the same file: external-tool verification, lines ~498 — no overlap with this edit at lines ~568). --------- Co-authored-by: continuous-bot <269947486+continuous-bot@users.noreply.github.com> Co-authored-by: tend-agent <270458913+tend-agent@users.noreply.github.com> Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Per maintainer feedback in #2410, reframe the Issue Triage section so the primary ask is
wt -vv <command>. The diagnostic report covers wt/git/OS versions, shell integration,wt config show,git worktree list --porcelain, and a full git trace in one file, plus agh gist create --web …hint, so a single gist URL replaces what was previously chained across multiple round-trips.wt --versionandwt config showremain as narrower asks for the cases where the full diagnostic is overkill.Test plan