docs: PR workflow — branch naming convention and template-based PR creation#106
Closed
aaronbrezel wants to merge 8 commits into
Closed
docs: PR workflow — branch naming convention and template-based PR creation#106aaronbrezel wants to merge 8 commits into
aaronbrezel wants to merge 8 commits into
Conversation
Main back into develop
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* docs: update GitHub auth section for Docker Sandbox environment Documents the proxy-based credential injection model accurately: - Explains why gh CLI fails (proxy tunnel incompatibility in Go's net/http) - Explains why git push works (libcurl handles the proxy tunnel correctly) - Removes stale TLS workarounds (-k flag, TOKEN=$(gh auth token)) - Adds troubleshooting step for git push credential failures Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: clarify HTTPS-only git support in Docker Sandbox SSH remotes require host-side SSH agent setup and network policy changes; HTTPS works out of the box via proxy credential injection. Adds remote URL switch command and link to Docker Sandbox SSH docs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * chore: ignore Docker Sandbox dotfiles written to project root The sandbox writes shell and tool config files (.bashrc, .zshrc, .gitconfig, .mcp.json, etc.) to the project directory. Also fixes .vscode/.idea patterns by removing trailing slashes which weren't matching correctly in the sandbox git environment. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
### Branch Namingsection to CLAUDE.md documenting theAI-{n}-descriptionconvention; instructs Claude to prompt for a Linear issue ID at branch creation and to warn (with confirmation gate) at PR creation if the branch name lacks an issue ID### Creating PRsto a three-step workflow: branch name check, PR body assembly from.github/PULL_REQUEST_TEMPLATE.md(Summary from diff, feature-specific QA steps + regression checklist, Notes), and curl-based creation with Python JSON escapingdocs/superpowers/documenting the decisions (public repo → no Linear URLs in PR bodies; branch name alone triggers Linear integration)Manual QA
CLAUDE.md→## GitHub→### Branch Naming: confirm theAI-{n}-descriptionformat, examples, Linear auto-detection note, and enforcement instructions for branch creation and PR creation are clear and completeCLAUDE.md→## GitHub→### Creating PRs: confirm Steps 1–3 (branch check, body assembly, curl command) are unambiguous and actionable for a future Claude sessiondocs/superpowers/specs/2026-06-02-pr-workflow-design.mdanddocs/superpowers/plans/2026-06-02-pr-workflow.mdare present and readableNotes
Documentation-only change — no source code or tests modified. All 489 tests pass.