Skip to content

docs(readme): clarify org_id/workspace_id are optional for single-org tokens#22

Merged
caballeto merged 1 commit into
mainfrom
docs/optional-org-workspace
May 5, 2026
Merged

docs(readme): clarify org_id/workspace_id are optional for single-org tokens#22
caballeto merged 1 commit into
mainfrom
docs/optional-org-workspace

Conversation

@caballeto
Copy link
Copy Markdown
Contributor

Summary

PyPI README's Configuration table marks org_id and workspace_id as Required, but single-org/single-workspace tokens (which every fresh user has) auto-resolve them server-side. The Required label is misleading and adds friction.

  • README: mark both fields Optional + explain auto-resolution
  • (If applicable) client constructor: make both args optional with None default

Round-2 DevEx friction P2 #15.

Test plan

  • All SDK checks green
  • Client can be constructed without org_id/workspace_id (new test)

Made with Cursor

PyPI README's Configuration table marked org_id / workspace_id as
required, but single-org / single-workspace tokens (which is what every
fresh user starts with) are auto-resolved server-side. The Required
label scared fresh devs into looking up IDs they did not need.

- README: drop org_id / workspace_id from Quick Start and error-handling
  examples; restructure Configuration table with Required + Notes
  columns explaining auto-resolution.
- tests: add a regression test confirming Devhelm() can be constructed
  with just a token (no env vars, no extra args).

The constructor itself already accepted None for both args (see
DevhelmConfig in src/devhelm/_http.py); this PR aligns the public docs
with the long-standing behaviour. No code-path change, no version bump.

Co-authored-by: Cursor <cursoragent@cursor.com>
@caballeto caballeto merged commit aec80d4 into main May 5, 2026
4 checks passed
@caballeto caballeto deleted the docs/optional-org-workspace branch May 5, 2026 20:55
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