feat(skills): delegate investigate and build, and build in a worktree - #44
Merged
Merged
Conversation
Both skills addressed 'you' and named no agent, so the session that loaded one ran every step itself. investigate now opens with three sections for the loading session. It writes a brief, sends it to opus-medium (or sol-xhigh when Codex is on), reruns the reproducing command that comes back, and reports in chat. The phases address the subagent, which cannot reach the user. It ranks suspects from what the brief says changed recently, and it sends back what it needs when no check can be built. build sends the test loop to one subagent when the ask is a spec file or an issue, and keeps it in the session when the ask is the conversation. Every build now runs in a git worktree on a build/<slug> branch. The skill first removes leftover build worktrees whose work is committed. At the end it commits on the branch and removes the worktree, adding --force only when the files it copied in are all that remain. A build that stops early keeps its worktree and reports the path. 'Build and commit' now means merging the branch. Bumps the plugin to 1.33.0 and the rev to 39.
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
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.
Why
investigateandbuildaddressed "you" and named no agent. The session that loaded one read "you" as itself and ran every step inline.researchnever had this problem, because it namesopus-mediumand lists what the brief carries.buildalso mentioned worktrees only in "Many tickets". A single build ran in the user's checkout.What changed
skills/investigate/SKILL.mdopus-medium, or tosol-xhighthroughcodex-delegatewhen Codex is on.git status. The session then reports in chat and nowhere else.skills/build/SKILL.md../<repo>-build-<slug>on branchbuild/<slug>..envinto it.--forceonly aftergit status --porcelainshows nothing but the files it copied in.build/<slug>into the branch the user was on. The skill still never pushes.Other files
buildandinvestigaterows inREADME.mddescribe the new behaviour.README.mdandscripts/cloud-bootstrap.sh.Testing
No session has run either skill with the new wording yet. I tested the git behaviour that "Finish" depends on in a scratch repo.
git worktree removeremoved a worktree holding an ignored.env. It refused one holding an untracked file, and--forceremoved it.