Skip to content

test(server): a neutral Environment label in the label override test - #59

Merged
immakermatty merged 1 commit into
mainfrom
agent/public-safe-environment-label
Oct 13, 2026
Merged

immakermatty merged 1 commit into
mainfrom
agent/public-safe-environment-label

Conversation

@immakermatty

Copy link
Copy Markdown

Problem

The test of the T3CODE_ENVIRONMENT_LABEL override in apps/server/src/environment/ServerEnvironmentLabel.test.ts used a real organization's name as its sample label. This repository is public.

Fix

The sample label is now the neutral Acme / Management, the form the fork runbook already uses. Input and expectation change together. Behaviour is unchanged.

The name stays in the history of main. Nothing here rewrites published history.

Verification

  • vp test run src/environment/ServerEnvironmentLabel.test.ts (apps/server): 9 passed.
  • git grep over the tree finds no other organization names.

🤖 Generated with Claude Code (Claude Opus 5.5)

The test of the T3CODE_ENVIRONMENT_LABEL override used a real organization's
name as its sample label. This repository is public, so the sample is now the
neutral "Acme / Management", the form the fork runbook already uses.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@agentrozjedemeai agentrozjedemeai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@immakermatty Exact-head review of 9a416af: approved. The diff only replaces the real organization sample with the neutral Acme / Management in both the environment override input and expectation; the trimming assertion remains intact and the runbook uses the same neutral example. The current tree has no occurrence of the previous name; published history is deliberately not rewritten. git diff --check is clean. CI on this SHA passed, including ServerEnvironmentLabel.test.ts (9 tests). No unresolved review threads. No local package install/test was run in the isolated checkout; test evidence is from the exact-head CI run.

@immakermatty
immakermatty merged commit 96a87f0 into main Oct 13, 2026
5 checks passed
@immakermatty
immakermatty deleted the agent/public-safe-environment-label branch October 13, 2026 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants