Skip to content

fix: support Anthropic keys requiring a workspace ID - #33

Merged
Kinfe123 merged 1 commit into
mainfrom
fix/anthropic-workspace-header
Oct 23, 2026
Merged

Kinfe123 merged 1 commit into
mainfrom
fix/anthropic-workspace-header

Conversation

@Kinfe123

@Kinfe123 Kinfe123 commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

Anthropic personal/service-account keys that are not bound to a workspace reject model requests unless the host supplies anthropic-workspace-id. Tress currently sends only the API key and API version.

Add optional server-only ANTHROPIC_WORKSPACE_ID configuration and reuse the resulting headers in agent execution, browser model forwarding/probes, and model health checks. Existing workspace-scoped keys continue to work without the setting, and incoming browser headers cannot override the host's workspace. Document local and Vercel configuration.

Validation:

  • Site TypeScript and production build passed.
  • Full host integration suite passed: 46 unit tests and all browser/terminal/managed/folder integration scenarios; three database-dependent cases skipped locally.
  • Integration coverage verifies the workspace header on both generation and metadata requests.
  • GitHub CI passed for 2fc7218; no actionable review feedback is present.
  • Direct Anthropic verification with the configured workspace returned HTTP 200 and the expected response.
  • Staged production deployment returned HTTP 200 and TRESS_DEPLOY_OK from its model endpoint before promotion.
  • After promotion, the public browser resumed the previously failing Managed Harness test thread and returned TRESS_PRODUCTION_OK. Public storage/model/Harness health checks all passed; the browser reported no console errors. The brief runtime-log stream returned no events, so it is not additional log evidence.

Production: deployed this exact commit as dpl_Dr733AXdc6ymVP4qQ3678ksT5mvg and promoted it to https://tress-theta.vercel.app on October 23, 2026 UTC. The host has the validated model key and workspace setting; credentials remain server-only and are not included in this PR. The public site's low-credit error is resolved with the replacement credential.

This fix is based directly on main and does not include the larger Harness runtime or pairing UI PRs. The PR still needs merging so subsequent main-branch deployments retain the fix.

@vercel

vercel Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
tress Ready Ready Preview Oct 23, 2026 2:52am UTC

Request Review

@Kinfe123
Kinfe123 marked this pull request as ready for review October 23, 2026 03:36
@Kinfe123
Kinfe123 merged commit 0e7a92d into main Oct 23, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 2fc72186 Deployed Oct 23, 2026 by vercel[bot]
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.

1 participant