Skip to content

feat(sdk): support server-generated branch names - #57

Open
necolas wants to merge 1 commit into
devfrom
necolas/generated-branch-names
Open

feat(sdk): support server-generated branch names#57
necolas wants to merge 1 commit into
devfrom
necolas/generated-branch-names

Conversation

@necolas

@necolas necolas commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Why

The HTTP API can allocate a unique branch name when target_branch is absent, but the SDKs still require callers to choose a name. SDK callers also need access to the optional literal prefix supported by the API.

Scope

  • Make the target branch optional in the TypeScript, Python, and Go SDKs.
  • Add the optional target prefix to each SDK.
  • Reject requests that set both target options.
  • Add request-payload tests and examples for server-generated names.
  • Update the Code Storage skill to match the public SDK contract.

Blast Radius

Existing calls with an explicit target branch keep their current request payload. The new behavior applies when a caller omits the target branch or supplies the new target prefix.

Verification

  • pnpm --filter @pierre/storage build: passed.
  • pnpm --filter @pierre/storage exec vitest --run: passed, 204 tests.
  • ./venv/bin/pytest -q: passed, 184 tests.
  • ./venv/bin/ruff check pierre_storage: passed.
  • ./venv/bin/ruff format --check pierre_storage: passed.
  • go test ./...: passed.
  • python3 scripts/sync_versions.py --check: passed.
  • git diff --check origin/main...HEAD: passed.

./venv/bin/mypy pierre_storage still reports the same three existing errors on main; this change does not add a type-check error.

Make createBranch targets optional in the TypeScript, Python, and Go SDKs. Add the optional prefix field, payload coverage, package examples, and agent-facing guidance.
@notion-workspace

Copy link
Copy Markdown

@necolas
necolas changed the base branch from main to dev September 8, 2026 18:16
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