Codestin Search App

PixPMusic · 2026-07-29T15:27:12Z

Summary

add a host-local Host name setting under Settings → Connections for web and desktop
use the configured name in live environment descriptors and as the default for newly paired manual connections
refresh relay-managed environment labels from verified health responses so renamed hosts propagate to other clients
fall back to the existing OS-detected computer name when the setting is cleared
keep web and mobile environment presentation in sync with the live server descriptor

Why

T3 Connect environment names were derived from the host operating system with no supported override. Relay-managed machines, especially remote hosts, could not choose a useful display name from the host itself.

Before / After

Before

After

The screenshots use an isolated test environment and the neutral example “Studio Mac”; no real machine hostname is shown.

Verification

vp test run packages/contracts/src/settings.test.ts apps/server/src/environment/ServerEnvironment.test.ts packages/client-runtime/src/state/server.test.ts packages/client-runtime/src/relay/discovery.test.ts infra/relay/src/environments/EnvironmentConnector.test.ts infra/relay/src/environments/EnvironmentLinks.test.ts infra/relay/src/agentActivity/AgentActivityPublisher.test.ts infra/relay/src/agentActivity/MobileRegistrations.test.ts infra/relay/src/environments/EnvironmentLinker.test.ts infra/relay/src/http/Api.test.ts — 105 tests passed
vp run --filter @t3tools/contracts --filter @t3tools/client-runtime --filter t3 --filter @t3tools/web --filter @t3tools/mobile --filter t3code-relay --concurrency-limit 2 typecheck
targeted formatting and lint for all changed files
git diff --check
isolated test-t3-app pass: verified the new row is absent on main, setting “Studio Mac” persists, and Connections renders the configured name without exposing the system hostname

Checklist

This PR is small and focused
I explained what changed and why
I added regression coverage
I included before/after UI evidence

Note

Add `environmentLabel` to `ServerSettings` and let hosts set their display name

Adds environmentLabel to ServerSettings and ServerSettingsPatch schemas in settings.ts, defaulting to an empty trimmed string so legacy configs decode unchanged.
ServerEnvironment.getDescriptor now reads the current configured label at runtime (no restart), and the new getDescriptorForSettings returns a descriptor snapshot for a given settings object in ServerEnvironment.ts.
The settingsUpdated websocket event includes an environment descriptor in ws.ts, and clients project it into config state in server.ts.
Web users with orchestration-operate scope can edit the "Host name" field in ConnectionsSettings.tsx; mobile and web presentation utils now prefer the server-reported label.
The relay persists label updates from verified health responses via EnvironmentLinks.updateLabelForUser in EnvironmentConnector.ts, and client discovery updates labels from status descriptors in discovery.ts.
Behavioral Change: ServerEnvironment.getDescriptor now depends on ServerSettings; if settings cannot be read it logs a warning and falls back to defaultDescriptor. The ServerConfigSettingsUpdatedPayload schema gains an optional environment field that older clients will ignore.

^{Macroscope summarized 4ae38b7.}

Model: GPT-5.6 Sol | Harness: Codex in T3 Code

Note

Medium Risk
Touches live server descriptors, WebSocket config events, and relay link persistence. Label updates are gated by verified health responses and a timestamp compare-and-set, but still change shared environment identity shown across clients.

Overview
Hosts can now set a Host name in Settings → Connections. That override is stored as environmentLabel in server settings, applied live to the environment descriptor, and used as the default name for new manual connections. Clearing it falls back to the OS-detected computer name.

settingsUpdated WebSocket events now include the environment descriptor so web/mobile presentation tracks the live label without a restart. Relay health checks persist a renamed label onto the user’s environment link (updateLabelForUser) when the verified descriptor differs, so T3 Connect clients pick up the new name.

^{Reviewed by Cursor Bugbot for commit 4ae38b7. Bugbot is set up for automated code reviews on this repo. Configure here.}

coderabbitai · 2026-07-29T15:27:20Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7d98afac-d0d6-468c-901e-773cd450170a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-29T15:33:51Z

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces a new user-facing feature allowing hosts to customize their display name. The changes span server, relay, web, mobile, and contracts packages with new database operations and modified data flow for environment labels. New features of this scope warrant human review.

^{You can add or adjust custom eligibility rules. Learn more.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 1b689f955fae75ffded96473ca0e13e695cabbcd. Configure here.}

inayayousfi · 2026-08-17T12:48:05Z

Written by inayayousfi, typed by gpt-5.6-sol running in OpenCode.
Every call here is inayayousfi's, and no agent acted on its own.

Thanks for working through the server-owned naming design. PR #7267 now implements the same core approach and extends it with authorized remote editing, a native mobile rename screen, duplicate-name confirmation, an explicit live label event, and direct environment-authenticated relay synchronization with bounded retries.

Would you be open to closing this PR in favor of #7267? The overlap is substantial, and keeping one implementation should reduce the review and integration burden for maintainers. The remote-access documentation added here is useful and could still be carried over separately.

github-actions Bot added size:L 100-499 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Jul 29, 2026

cursor Bot reviewed Jul 29, 2026

View reviewed changes

Comment thread apps/web/src/components/settings/ConnectionsSettings.tsx

PixPMusic force-pushed the pixpmusic/connect-host-name branch from f7d7c28 to 1b689f9 Compare August 3, 2026 16:07

cursor Bot reviewed Aug 3, 2026

View reviewed changes

Comment thread apps/server/src/ws.ts

PixPMusic force-pushed the pixpmusic/connect-host-name branch from a718716 to f03974b Compare August 10, 2026 08:03

inayayousfi mentioned this pull request Aug 16, 2026

feat: let users rename environments #7267

Open

PixPMusic added 3 commits August 19, 2026 19:51

feat(connect): let hosts set their display name

03fc5eb

fix(connect): gate host name edits on server readiness

1633551

fix(server): keep environment labels consistent

1ef1fec

PixPMusic force-pushed the pixpmusic/connect-host-name branch from f03974b to 1ef1fec Compare August 20, 2026 03:07

macroscopeapp Bot reviewed Aug 20, 2026

View reviewed changes

Comment thread infra/relay/src/environments/EnvironmentConnector.ts

fix(relay): order environment label updates

4ae38b7

PixPMusic closed this Aug 20, 2026

PixPMusic deleted the pixpmusic/connect-host-name branch August 20, 2026 18:20

Navigation Menu

Uh oh!

feat(connect): let hosts set their display name - #4850

feat(connect): let hosts set their display name#4850
PixPMusic wants to merge 4 commits into
pingdotgg:mainfrom
PixPMusic:pixpmusic/connect-host-name

PixPMusic commented Jul 29, 2026 •

edited by macroscopeapp Bot

Loading

Uh oh!

coderabbitai Bot commented Jul 29, 2026 •

edited

Loading

Review skipped

Uh oh!

Uh oh!

macroscopeapp Bot commented Jul 29, 2026 •

edited

Loading

Uh oh!

cursor Bot left a comment

Uh oh!

Uh oh!

inayayousfi commented Aug 17, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

PixPMusic commented Jul 29, 2026 • edited by macroscopeapp Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Why

Before / After

Before

After

Verification

Checklist

Add environmentLabel to ServerSettings and let hosts set their display name

Uh oh!

coderabbitai Bot commented Jul 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Uh oh!

macroscopeapp Bot commented Jul 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Approvability

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

inayayousfi commented Aug 17, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

PixPMusic commented Jul 29, 2026 •

edited by macroscopeapp Bot

Loading

Add `environmentLabel` to `ServerSettings` and let hosts set their display name

coderabbitai Bot commented Jul 29, 2026 •

edited

Loading

macroscopeapp Bot commented Jul 29, 2026 •

edited

Loading