fix(dev): write bootstrap warnings directly to stderr

[maria-rcks]

main's typecheck fails with TS377064 because the standalone worktree setup script uses console.warn. write the same warning directly to node's stderr stream, preserving its text and newline without a diagnostic suppression or a dependency on effect. the env-file safeguards remain unchanged.

verified on blacksmith: scripts typecheck, targeted lint and formatting, and all 74 existing dev-runner tests passed. a direct node runtime comparison confirmed matching warning output and exit status.

model: gpt-6.1-sol. harness: codex in t3 code.

[macroscopeapp]

Approvability

Verdict: Approved at 9ca829f

Macroscope's review found this PR approvable — This narrowly scoped development fix preserves the existing warning text and control flow while writing directly to stderr during worktree setup. It does not affect product defaults or production request behavior, and it removes rather than adds a static-analysis suppression.

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

[maria-rcks]

Note

Written by gpt-6.1-sol on behalf of Maria

addressed in 9ca829f: removed the diagnostic suppression and replaced console.warn with process.stderr.write. the warning text and newline match, and the script still uses only node built-ins before dependencies are installed. scripts typecheck, targeted lint/formatting, and 74 existing dev-runner tests pass on blacksmith.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 687a592a-5a52-4358-bc61-fa44342d2513

📥 Commits

Reviewing files that changed from the base of the PR and between 1e2ecbd and 9ca829f.

📒 Files selected for processing (1)

• scripts/setup-worktree.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 6 remain after this review.

---

📝 Walkthrough

Walkthrough

When the environment-file source is not a regular file, setup-worktree.ts writes the skip message to stderr instead of calling console.warn. The setup flow otherwise remains unchanged.

Changes

Setup warning output

Layer / File(s)	Summary
Write skip message to stderr scripts/setup-worktree.ts	When the environment-file source is not a regular file, the skip message is written to stderr instead of being emitted with console.warn.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 9ca82

The focused stderr change is mergeable with normal checks.

Architecture Summary

Architecture risk: 🔵 Low · up to 9ca82

The change affects 1 system.

Changed systems: scripts

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

• observed — scripts (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

• observed — Modified behavior in scripts/setup-worktree.ts: The non-regular-file warning is now written to stderr rather than emitted with console.warn.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly describes the main change: writing bootstrap warnings directly to stderr.
Description check	✅ Passed	The description explains the typecheck problem, the stderr change, and the verification results. It also states that the env-file safeguards remain unchanged, which makes the focused scope clear.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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