Skip to content

fix: fork install docs, scripts and settings stop pointing at upstream - #24

Merged
eyeveil merged 3 commits into
mainfrom
fork/install-points-at-fork
Nov 7, 2026
Merged

eyeveil merged 3 commits into
mainfrom
fork/install-points-at-fork

Conversation

@yvesoddly

@yvesoddly yvesoddly commented Nov 2, 2026 •

Copy link
Copy Markdown

Why

The fork publishes only nightly builds, but its install docs, install script headers, and the desktop update-track switch still sent fork users to upstream installs, upstream releases, or a stable channel the fork doesn't have. #23 moved the release source to the fork. This covers what it left.

What changed

  • install.sh and install.ps1 headers give the fork's raw script URL with T3CODE_CHANNEL=nightly. Their errors no longer point at t3.codes/download or npm install -g t3. README.md and docs/user/install.md link the fork's releases, label package managers as installing upstream, and say the fork ships Linux x86_64 only.
  • FORK_HAS_UPDATE_TRACKS = false in @t3tools/shared/fork hides the desktop update-track switch and its settings search entry. The switch took its target from upstream's npm t3, which fork releases don't have. docs/user/updating.md explains this.
  • fork.test.ts fails if the README, install guide, or scripts mention t3.codes/install, t3.codes/download, a bare t3.codes link, or upstream's releases page.

Scope

  • Covers docs, install scripts, and the update-track switch.
  • Leaves about 10 in-app npx t3 … strings, plus updating.md's npx t3@<client-version> line, pointing at upstream npm. Fixing them means rewording shared upstream UI, so they need their own change.
  • Fork desktop users who once chose Stable have no in-app way back. That comes from reading the code and wasn't reproduced.

Tradeoffs

  • npx t3 still runs upstream, and the docs now say so. Publishing a fork npm package needs the maintainer's OK.

Blast Radius

Docs and scripts change for fork users only. Hiding the switch also hides its "machines on the other track" review path for connected remote machines.

Verification

  • scripts/install.sh, run as its header says in an isolated home, installed 0.1.8-nightly.20261106.100, and t3 --version reported it. Without T3CODE_CHANNEL it stops with "could not find a stable release".
  • vp run check:changed fork/main passes, and fork.test.ts passes 9 of 9.
  • Not run: the PowerShell one-liner (no Windows machine), and install.sh on Linux arm64, where it should fail with "no release archive" because the fork publishes none.

Model and harness: Claude Opus 5.5 (implementation, update onto main, review) via Claude Code in T3 Code.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Nov 2, 2026
@greptile-apps

greptile-apps Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge within its stated scope.

Summary

This PR points install instructions and release links at the fork’s nightly builds.

  • Fork install directions point to nightly builds from issuelt/t3code.

yvesoddly explicitly acknowledged that users who previously selected Stable lose their in-app route back, and that hiding the setting removes the remote-machine Review path. The remaining upstream npx t3 instructions are explicitly deferred.

No new actionable issues were found. Review was based on source inspection; tests and UI checks were not run.

Reviews (2) · Last reviewed commit: "Point the new Linux install notes and RE..." · Reviewed by Greptile

Upstream's Linux section promised ARM64 desktop downloads the fork does not
publish, and the README's desktop app link still opened t3.codes. The fork
test now also rejects t3.codes download links.
@eyeveil
eyeveil changed the base branch from fork/constants-and-carried-prs to main November 7, 2026 12:48
@eyeveil
eyeveil marked this pull request as ready for review November 7, 2026 12:48
@eyeveil

eyeveil commented Nov 7, 2026

Copy link
Copy Markdown
Collaborator

Note

Written by claude-opus-5-5 on behalf of eyeveil

Retargeted to main, with main merged in at b23f45a6fe4. @greptileai review

@eyeveil
eyeveil merged commit 4879d0d into main Nov 7, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants