Repository navigation
Release 0.6.0 - #252
Merged
Merged
Release 0.6.0#252
Conversation
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.
Summary
0.6.0 ships the changes merged since v0.5.0. The headline is the Architect run (
thirdshift architect [<focus>]): an Architecture review of the Base branch publishes its top recommendation as a plan, and thirdshift marks the plan ready and dispatches it asthirdshift <plan URL>would (#241). The other big change is that a Run now recognises an Inherited failure, a red check that is also red on the Base branch, and either fails at once or, when allowed, starts a Base fix (#226).New command:
thirdshift architectthirdshift architect [<focus>] --plan-onlyruns the Architecture review in a detached<repo>-architectworktree and reads the plan from its final line. It checks the plan, swapsneeds-triageforready-for-agentand prints the plan's URL. A failure after the review starts exits1and changes no label (thirdshift architect --plan-only: the Architecture review publishes a plan and thirdshift marks it ready #240).--plan-only, the Architect run goes on to the Spec run (a plan with sub-issues) or Run thatthirdshift <plan URL>would start, and ends as that run does. It takesmerge,no-mergeandparallel <n>, with the User config'smerge.alwaysandspec.parallelas defaults; these flags are an argument error with--plan-only(Architect run dispatches its plan as a Spec run or a Run #244).0and prints the idea issue the review filed, or the open issue that already covers its top recommendation. No label is changed (Architect run with no Strong candidate: idea issue or already filed #243).improve-codebase-architecture,to-spec,to-ticketsandcodebase-design(Headless Factory skills for the Architecture review #238).email [address]or byemail.alwayswithoutno-email, sends exactly one however it ends. The run it dispatches sends none (One Run notification per Architect run #245).architecttakesbase-fixandno-base-fix, or the User config'sbase.fix, and hands them to the dispatched run. Both are rejected with--plan-only(Pass Base fix through an Architect run to its dispatched run #251).Inherited failures and the Base fix
CI red on <check>, which also fails on <base> at <short sha>; fix <base> first. When the branch also has failures of its own, the CI-fix Repair gets only those, with the Inherited failures listed as not to fix (Inherited failures fail the Run without a Repair #225).base-fix(or--base-fix), such a Run instead writes a Base fix issue labelledbase-fixandready-for-agent, adding either label if the repository lacks it. It starts a child Merge run into the Base branch, then merges the Base branch in and watches CI again. A Run gets one Base fix. On a Spec, a Ticket's Base fix goes into the Spec branch and the Spec PR's into the Base branch (A Base fix, when the base-fix flag is given #236).base.fixin the User config allows a Base fix for every Run, andno-base-fixforbids one for a single Run; giving both flags is an argument error. Setup asks about Base fixes only when every Run is a Merge run, and writes[base] fix = falseotherwise (Base fix in the User config and Setup #239).base-fixissue for the same Base branch and checks waits on it, with no time limit if it was written from another clone or machine. If the Base fix was started from the same Launch directory and has ended with its issue still open, the Run starts it again. Tickets of a Spec run that meet the same Inherited failure share one Base fix (Runs share one Base fix #242).Base fix: <issue URL> merged. A Ticket that landed after one says so in the Tickets checklist and the Spec run's Run notification (Inherited failures: fail fast on a red Base branch, or fix it with a Base fix #226).Also: docs, test suite and release script updates (#209, #210, #213, #214, #215, #216, #217, #218, #220, #227, #235, #250).
Version diff