th-df7007/th-e7ef23: stop swallowing the exception behind INTERNAL_ERROR; fix .NET gateway env vars

[brentrager]

Problem

Running the smooth-bench agentic suite across all five engines, the .NET server booted fine, accepted the WebSocket upgrade (HTTP 101 on /ws), and then returned INTERNAL_ERROR: Internal error processing the request. for every send_message. Zero tool calls, every scenario INCONCLUSIVE.

Worse: the .NET server logged nothing about the failure. No exception, no stack, at any level. Both places FrameDispatcher maps an exception to INTERNAL_ERROR discarded it outright.

Fix 1 — observability (th-e7ef23)

FrameDispatcher already carried an ILogger? (wired from DI at SmoothOperatorWebSocketExtensions.cs:107) but never used it at either catch site. Both now route through LogInternalError, which records action + requestId + the exception with its stack at Error level, falling back to stderr when no logger is wired. The wire message is unchanged — still generic, still leaks nothing to the client.

Same swallow existed in the siblings: the Go dispatcher dropped err at all seven of its INTERNAL_ERROR sites (now one internalError chokepoint that slog.Errors first), and the Python dispatcher dropped the exception at both of its own (now logging.exception). TypeScript already logged via console.error; Rust puts the detail on the wire.

Fix 2 — root cause (th-df7007)

The new logging named it on the first run:

INTERNAL_ERROR handling action 'send_message' (requestId r2): HTTP 401 (auth_error: 401)
LiteLLM Virtual Key expected. Received=****, expected to start with 'sk-'.

dotnet/server/host/Program.cs read SMOOTH_GATEWAY_URL / SMOOTH_GATEWAY_KEY / SMOOTH_MODEL. Rust, Go, TS and Python all read the SMOOAI_* spelling — the actual cross-engine contract. Every launcher and bench exporting the shared names left the .NET host keyless, so it called the gateway with the literal key "unset". SMOOAI_* now wins; SMOOTH_* kept as a fallback for existing deployments. README + Dockerfile updated.

Verification

Before (bench, no-over-refusal, deepseek-v4-flash, host isolation): INCONCLUSIVE, INTERNAL_ERROR, 0 tool calls.

After: /health reports deepseek-v4-flash (was silently claude-haiku-4-5), turns stream tokens, and the bench scenario runs conclusively with real tool calls (read_file, bash, list_files, grep). Zero INTERNAL_ERROR across every scenario's engine log.

Tests

• dotnet/server/tests/InternalErrorLoggingTests.cs — 3 tests: turn-path logging, dispatcher-path logging, and the no-logger fallback. Asserts the generic message on the wire AND the detail in the log.
• go/server/internal_error_log_test.go — same contract over slog.
• python/server/tests/test_internal_error_logging.py — same contract over caplog.

Full suites green: .NET 294 passed, Go all packages ok, Python 231 passed / 1 skipped.

🤖 Generated with Claude Code

https://claude.ai/code/session_01P83HrAKaqp2ehxQcW7m6QU

[changeset-bot]

🦋 Changeset detected

Latest commit: bcfe4a9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@smooai/smooth-operator	Patch
@smooai/smooth-operator-web-chat-example	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR