Repository navigation
docs: T3 0.1.1-nightly.20261019.3373 update review - #10
Merged
Merged
Conversation
Two entries ready (setup skipped for local threads, thread read lists PRs), the archived-threads fix tracked to the next nightly, and evidence from nine nightlies written into the active entries. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 19, 2026
leomontigatti
added a commit
to leomontigatti/rubens
that referenced
this pull request
Oct 19, 2026
The `setup-script-cwd` entry said T3 runs the setup script in the project root for a thread with no worktree. Since T3 0.1.1-nightly.20261018.3331 (pingdotgg/t3code#18891) such a thread runs no setup. The one case left that runs the script in the main checkout is a thread launched as `existing_worktree` on the project root, which `t3.json`'s `pwd -P` guard still covers, so `t3.json` is unchanged. Found by the shared T3 update review (leomontigatti/agent-workflow#10, adopted in leomontigatti/agent-workflow#12). ## Review Tier: readback, reviewed 6759d47 - readback (Luna): no findings 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
leomontigatti
added a commit
to leomontigatti/enescena
that referenced
this pull request
Oct 19, 2026
Since T3 0.1.1-nightly.20261018.3331 (pingdotgg/t3code#18891) a thread with no worktree runs no setup. The `setup-script-cwd` entry now says so, and names the one case that still runs the script in the main checkout: a thread launched as `existing_worktree` on the project root. `t3.json` and its `pwd -P` guard stay as they are, since the guard still covers that case. Found by the shared T3 update review (leomontigatti/agent-workflow#10, adopted in leomontigatti/agent-workflow#12). ## Review Tier: readback, reviewed 2f23c96 - readback (Luna): no findings 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <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
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.
T3 update review: 0.1.1-nightly.20261017.3238 → 0.1.1-nightly.20261019.3373 (9 releases)
Needs you
~/.local/state/t3-update-review/state.jsondidn't exist, so this run started from 0.1.1-nightly.20261017.3238, the version PR docs: T3 0.1.1-nightly.20261017.3238 update review #6 reviewed. It is marked now. If an earlier run reviewed a later build, some items were looked at twice. Nothing was skipped.setup-runs-in-main-checkoutandthread-read-lists-prs(see Fixes / ready). Each one, once adopted, is a small edit toskills/implement/THREADS.md(andcontract.mdfor the first).pr-auto-link: open a PR withghfrom a thread and don't link it. Then check whether T3's automatic branch association shows its status, can be watched and settles the thread on merge. See Improves.Fixes / ready
setup-runs-in-main-checkout→ ready. pingdotgg/t3code#18891 (merged 2026-10-18, in 0.1.1-nightly.20261018.3331; closes #18882). "ThreadLaunchServiceskips the setup action, and its 'Starting setup script' progress step, when the thread has no worktree path." watched.diff:worktreePath === null ? ({ status: "no-script" } as const) : yield* setupScripts.runForThread(...). One case is still open: "anexisting_worktreelaunch whose path is the project root still runs setup."skills/implement/THREADS.md(Read-only threads): remove "One started in 'local' mode, with no worktree, still runs therunOnWorktreeCreatescript, in the main checkout; a setup terminal that fails there is expected, and is no reason to change the main checkout."contract.md(t3.jsonrow): keep "safe to run in the main checkout", with the new reason "where T3 still runs it for anexisting_worktreelaunch on the project root."setup-script-cwd) and pcp (setup-links-env) say T3 runs the script "the project root for a thread with no worktree". That stopped being true in 0.1.1-nightly.20261018.3331. Proposed edit for each project'sdocs/agents/t3-dependencies.md: the main-checkout case is now only anexisting_worktreelaunch on the root. Keep the scripts'pwd -Pguard.thread-read-lists-prs. pingdotgg/t3code#17454 (0.1.1-nightly.20261018.3331).t3_thread_listandt3_thread_readnow returnpullRequests. The tool text says "every linked pull request; linkedPullRequest is a legacy single link and can be null while pullRequests still has entries". Proposed edit,skills/implement/THREADS.md("A lost context is recovered from T3"): take the PRs from the samet3_thread_readcall. Keeplist_thread_pull_requestsfor watch and check detail.Improves
pr-auto-link(watching, lead added). watched.diff,ThreadPullRequestService.ts, shows T3 discovering a thread's PR by branch after each turn and saving it asbranchPullRequest. pingdotgg/t3code#20223 describes it as an "automatic association" that shows in the sidebar. It is not a link (sourcehas no branch value). Nobody has checked whether it settles the thread or can be watched. Proposed: the trial in Needs you. If it passes,implement§ 7 keepslink_pull_requestonly for PRs from another branch.storage-cleanup-removes-thread-worktrees: #20449 recreates a missing worktree only inside the managed folder (<managed>/<repo>/<folder>).t3_worktree_handofflostpath; no shared file uses it.renamed-branch-in-thread: #20223 drops only automatic associations whose head branch is no longer the thread's. Explicit links are unchanged, so the entry still holds.pr-watch-is-native: #21155. Watching again starts fromendedWatch, so a re-watch no longer repeats "all required checks passed".babysit-prreads the round itself on each wake, so nothing breaks.delegate-task-reviews: #19058 buildssummaryfrom the newest assistant message. #21487 makestask_cancelalso cancel the task's own queued turn.implementer-worktrees: #20213 refusesisolation: worktreewhen the project repo lacks the parent's commit.preview-tools-verification: theServerBrowserPage.tsrewrite (#20446) masks password values and retries a snapshot once.MAX_SCREENSHOT_WIDTHand the scaled clip are unchanged, and #20899 is still open, so the entry stays watching.cleanup-unlinks-env: cleanup now runsgit worktree remove -- <path>(still no--force), so the entry holds.scheduled-reviews: bound runs now follow the thread's provider (#18467). Ours are unbound. Webhook values are now framed as untrusted (#20447); we use no webhook.Watching, moved but not ready
storage-cleanup-archived-threads. Fix pingdotgg/t3code#16473 merged 2026-10-19T18:37Z (034b506; closes #17811). 3373 was cut from300a356, andcompare 034b506...300a356returnsbehind, so the fix isn't installed yet. The Watch line now says ready once the next nightly is installed. Thenhousekeeping§ 1 can drop the archived-thread sweep.Not covered
~/.t3/projects/infra): nodocs/agents/t3-dependencies.mdon any branch.Still ready
setup-runs-in-main-checkout: THREADS.md drops the local-thread setup sentence; the contract keeps the safe-script rule forexisting_worktreeon the root.thread-read-lists-prs: recovering a lost context takes the PRs fromt3_thread_read.Method: 696 release items were triaged to 382 (web-perf, mobile and marketing dropped). Four
gpt-6-lunaminers checked them against the dependency files: 36 hits, all evidence or minor. I read watched.diff (20 changed files) in full. Drift check: every shared Used by file and heading exists, and no uncovered T3 rule was found in the shared repo or the four projects.Projects affected: none by this diff. The rubens, enescena and pcp edits above are proposals for those projects.
🤖 Generated with Claude Code