Repository navigation
feat: Pip is Ada's logo across every surface - #235
Merged
Merged
Conversation
jayleaton
force-pushed
the
feat/rebrand-ada-cli-20261113
branch
from
November 13, 2026 10:51
5ac6b54 to
6a58e77
Compare
jayleaton
force-pushed
the
feat/rebrand-ada-logo-20261113
branch
from
November 13, 2026 10:51
cdc6d4c to
84bbec0
Compare
jayleaton
marked this pull request as ready for review
November 13, 2026 10:51
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: unavailable · PR result: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
assets/agents/pip.svg is the only hand-edited source. icons:export draws it into each channel's Icon Composer backdrop (prod navy, nightly sky, dev blueprint) and writes the desktop, web, fork and marketing sets, a padded macOS icon and the social card. icons:export:android renders the mobile launcher, splash and widget marks. Legacy T3 sources nothing references are removed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The README picks the dark or light lockup with prefers-color-scheme. The social card nests the dark lockup. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…releases GitHub redirects /releases/latest to the release list when no release is marked latest, which answers 200 without a tag_name, so every stable check showed a raw schema error. Read the Atom feed for every channel, fall back to the paged API index as cliRelease and npmRelease do, and report an unreadable list in plain words. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Mobile About -> Legal becomes License and opens LICENSE on GitHub. Web About gains a License row beside the privacy policy, which still covers the telemetry sent to upstream. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Pairing, auth and triage hints name the ada command; the T3 Connect sentence leaves Notifications; the T3 Chat theme is Orchid (its id stays t3-chat); Labs and Storage say Ada home folder; new settings default the branch prefix to ada; Android saves images to Pictures/Ada. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Allowlist the intentional ones, each with its reason. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The fork reported to upstream's ingest by default. Telemetry now defaults off, including after the desktop reads the login shell, and setting T3CODE_TELEMETRY_ENABLED=true still opts in. The web About row and Welcome notice replace the t3.codes privacy-policy link with one line saying so, and the user docs describe it as opt-in. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
jayleaton
force-pushed
the
feat/rebrand-ada-logo-20261113
branch
from
November 13, 2026 12:21
73c33a7 to
0d3a5ed
Compare
Owner
Author
jayleaton
changed the base branch from
feat/rebrand-ada-cli-20261113
to
main
November 13, 2026 12:32
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.
Stacked on #234 (the
adaCLI), which is stacked on #233 (copy and names). Merge those first.Summary
JJ chose Pip as Ada's logo. Every logo, icon and favicon now comes from one master SVG,
assets/agents/pip.svg, via the existing generators. The README and the social card use the approved "A" lockup.Variants keep the same Pip with each channel's existing backdrop. Production is solid navy with a lavender Pip, nightly is the night sky with clouds and a white Pip, and dev is the blueprint with a white Pip. That is the same way the T3 variants differed. Mobile still uses one set for every variant, as the fork already did.
Trade-off: macOS icons. Icon Composer's command line has no "macOS pre-Tahoe" preset, so upstream hand-exported those PNGs. They are now generated: the render sits on Apple's 824-on-1024 safe area. They don't have Icon Composer's native drop shadow. A hand export from Icon Composer can replace them if the shadow matters.
Evidence
The captures compare #234 (
5ac6b54) with this branch. They come from a real web client on fresh dev databases and are in a secret gist: https://gist.github.com/jayleaton/b719528fd65934a59b4d11ac7a50e569/favicon-32x32.png),brand-mark.png, sidebar headerSocial card (
apps/marketing/src/assets/social-card.webp):README rendered on GitHub: https://github.com/jayleaton/t3code/blob/feat/rebrand-ada-logo-20261113/README.md. It switches between the dark and light lockups with your GitHub theme.
Checks:
vp run check:changedat73c33a7→ pass: format, lint, knip, typecheck (41 packages), test (257 files, 4,844 passed), react compilervp run icons:check→ all 55 generated assets match their sources. The renders are byte-reproducible.scripts/{apply-web-brand-assets,build-desktop-artifact,upstream-install-commands}.test.ts→ pass (58)Mobile: not captured here. Captain runs the MacBook iOS pass. Capture these:
t3_*MCP calls) in a thread.Android, if checked: the adaptive icon (Pip inside the mask over navy), the themed monochrome icon, and the Android 12 splash.
Leftovers from the MacBook iOS pass of #234
These come from https://gist.github.com/jayleaton/5955335593e5703ddf636bcd85702f3f. They are new commits on top, so #233 and #234 are unchanged.
t3 pairada triagehints now sayada. The server, desktop and docs got the same sweep: Ada home,ada update,ada service …,ada serve,ada policy checkand others.LICENSE_URL(https://github.com/${CLI_RELEASE_REPOSITORY}/blob/main/LICENSE), and the in-app viewer only allows that page. Web About → Legal adds a License row. The privacy policy stays because it covers the telemetry sent toingest.t3.codes.docs/user/mobile-notifications.md.t3-chatis unchanged.~/.t3.branchNamePrefixdecoding default is nowada. Stored values are not migrated, so a saved prefix stays as it is.Pictures/Ada.Bug: mobile "Check for updates" showed
Missing key at ["tag_name"](also on main).git.832008.xyz/jayleaton/t3code/releases/latestredirects to/releases. That page answers 200 with JSON that has notag_name, anddecodeReleasethrew.asErrorpassed the schema message straight to the UI.findEnvironmentUpdatereads the Atom feed for every channel. If the feed has no match, it falls back to the paged API index, ascliReleaseandnpmReleasedo. An unreadable list now says "Could not read the release list. Try again."environment-maintenance.test.tsfailed first with the rawMissing key at ["tag_name"]. They now pass.Guard.
scripts/upstream-install-commands.test.tsnow also flagsT3 <Word>phrases andt3 <subcommand>hints. Before the fixes, it failed on every missed item:The widened pattern found 144 lines. Most were fixed. The rest are allowlisted, each with a reason:
t3 updatein the copied command for hosts older thanada;t3link, whichadasits beside as an alias;t3service preflight;Kept on purpose: with the desktop's
t3link on PATH, the triage hint still sayst3 triage. That link is the one the CLI state proves exists, andadabeside it is best-effort.t3@<version>(the npm package name) and paths such as~/.t3/bin/t3are also unchanged.MacBook re-pass: the Pip list below, plus Connections → New (
ada pair), About → License, Notifications, the Orchid theme, Thread settings → Ada home folder, Branch naming, and Check for updates on a stable host.Usage data off by default
JJ's call: the fork doesn't report to upstream unless asked.
T3CODE_TELEMETRY_ENABLEDnow defaults to false inAnalyticsService, and that includes the desktop's login-shell path.T3CODE_TELEMETRY_ENABLED=true(in the environment or the shell profile) still opts in, and a managed policy can still force it off. The key and host defaults are unchanged.AnalyticsService.test.tsfailed first. With nothing set, 20 events were sent; now none are. An unset shell value counted as on; now only an explicittruefrom the shell turns it on.ingest.t3.codes. Desktop and web OTLP export only runs to endpoints the user configures. The server still downloads its model manifest fromraw.githubusercontent.com/pingdotgg/t3code. That isn't telemetry, so it's left as a separate decision.legalLinks.tsis deleted). Web About → Legal → Usage data and the Welcome page footer now say "Ada doesn't collect usage data unless you turn it on with T3CODE_TELEMETRY_ENABLED." Under a policy that turns usage data off, both show the organization's notice instead.docs/user/telemetry.md(now "Turn it on") andfaq.mddescribe it as opt-in.Merge Danger
Door: two-way. The logo is assets and generators only. The leftovers change copy and one settings default (
branchNamePrefixfor new settings), with no stored-data migration. Reverting restores the T3 artwork.Blast Radius: platform
Usage data stops by default: installs that never set
T3CODE_TELEMETRY_ENABLEDstop reporting after this update.Every surface's icon changes: desktop app and installers, web favicons and manifest, mobile launcher, splash and widget, DMG backgrounds, marketing, and README.
macOS Dock icon loses Icon Composer's native shadow. See the trade-off above.
Upstream merges: upstream edits to the
.iconprojects or the T3 PNGs will conflict here. Resolve by keeping Pip and rerunningvp run icons:exportandvp run icons:export:android.Model and harness: Claude Opus 5.5 in Claude Code (T3 Code).
🤖 Generated with Claude Code