Skip to content

fix(sdk): default both SDKs to the www.sim.ai host - #8581

Merged
waleedlatif1 merged 3 commits into
stagingfrom
fix/sdk-default-www-host
Oct 2, 2026
Merged

fix(sdk): default both SDKs to the www.sim.ai host#8581
waleedlatif1 merged 3 commits into
stagingfrom
fix/sdk-default-www-host

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The apex https://sim.ai returns 301 -> https://www.sim.ai (verified: curl -X POST https://sim.ai/api/health → 301 https://www.sim.ai:443/api/health). fetch and requests replay a 301 POST as a body-less GET, so default-configured SDK calls such as POST /api/v2/workflows/<id>/execute break.
  • TypeScript SDK (config.baseUrl || ...) and Python SDK (base_url default + docstring) now default to https://www.sim.ai; READMEs and the TS example updated. Remaining apex mentions are browser links/homepage metadata (GET, redirect is harmless).
  • Patch-bumped both SDKs 0.2.0 → 0.2.1 (package.json + SDK_VERSION + bun.lock workspace entry; pyproject.toml + __version__). The publish workflows release on push to main when the version is new, so this ships on the next main merge. setup.py stays at its stale, unread 0.1.1 as in fix(sdk): make the 0.2.0 SDK release safe to publish #6616.
  • API reference (python.mdx, typescript.mdx) now states the www default and notes that versions before 0.2.1 default to the apex, which redirects.

Type of Change

  • Bug fix

Testing

  • packages/ts-sdk: bun run test — 10 passed
  • packages/python-sdk: PYTHONPATH=. pytest tests/ — 41 passed (default-URL assertions updated)
  • bun run lint, bun run check:audits (incl. check:site-urls), bun run docs-manifest:check — pass

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests pass locally
  • Docs updated

🤖 Generated with Claude Code

@waleedlatif1
fix(sdk): default both SDKs to the www.sim.ai host
5411d74 
The apex https://sim.ai answers every request with a 301 to
https://www.sim.ai, and fetch and requests replay a 301 POST as a GET with
no body, so a default-configured execute call never reaches the API. Both
SDKs now default to https://www.sim.ai. Patch-bump both to 0.2.1 so the
publish workflows release the new default on merge to main, and note in the
API reference that earlier versions still default to the apex.
@waleedlatif1
waleedlatif1 requested a review from a team as a code owner October 2, 2026 19:21
@vercel

vercel Bot commented Oct 2, 2026 •
edited
Loading

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Oct 2, 2026 7:28pm UTC

Request Review

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 2, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 11 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Oct 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Medium risk] Changes default API endpoint for both SDKs.

The SDK host changes appear sound, but the Python test must satisfy the repository’s explicit testing requirement before merging.

Findings

  1. P2 Mock-call assertions in Python tests ▶

Summary

The PR changes both SDKs’ default API host to https://www.sim.ai, bumps them to 0.2.1, and updates tests, examples, and documentation.

  • The new TypeScript response mock is compatible with the supported test runtime.
  • An updated Python test still violates the repository’s mock-assertion requirement.

Reviews (4) · Last reviewed commit: "test(ts-sdk): use a typed Response in th..."

greptile-apps[bot]
greptile-apps Bot reviewed Oct 2, 2026
View reviewed changes
Comment thread packages/ts-sdk/src/index.ts
@waleedlatif1

waleedlatif1 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

waleedlatif1 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 2, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 12 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@waleedlatif1

waleedlatif1 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

waleedlatif1 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 2, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 12 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

greptile-apps[bot]
greptile-apps Bot reviewed Oct 2, 2026
View reviewed changes
Comment thread packages/ts-sdk/src/index.test.ts Outdated
greptile-apps[bot]
greptile-apps Bot reviewed Oct 2, 2026
View reviewed changes
Comment thread packages/ts-sdk/src/index.test.ts
@waleedlatif1

waleedlatif1 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

greptile-apps[bot]
greptile-apps Bot reviewed Oct 2, 2026
View reviewed changes
Comment thread packages/python-sdk/tests/test_client.py
@waleedlatif1
waleedlatif1 merged commit ba0c8c7 into staging Oct 2, 2026
36 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/sdk-default-www-host branch October 2, 2026 19:51
@waleedlatif1 waleedlatif1 mentioned this pull request Oct 2, 2026
Merged

This branch was previously deployed

1 inactive deployment
Preview — 838bf8d1 Deployed Oct 2, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments
@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@waleedlatif1