Skip to content

fix(pipelines): wave-land must create feature branch before committing#200

Merged
nextlevelshit merged 1 commit intomainfrom
fix/wave-land-branch-workflow
Mar 1, 2026
Merged

fix(pipelines): wave-land must create feature branch before committing#200
nextlevelshit merged 1 commit intomainfrom
fix/wave-land-branch-workflow

Conversation

@nextlevelshit
Copy link
Collaborator

@nextlevelshit nextlevelshit commented Mar 1, 2026

Summary

  • wave-land branch workflow: The commit step now creates a feature branch before committing, enabling the standard push → PR → merge flow instead of pushing directly to main
  • Preflight tool validation: Added requires.tools to all 22 platform pipelines (gh-*, gl-*, gt-*, bb-*) and wave-land so preflight catches missing CLIs before execution starts
  • TodoWrite avoidance: Added soft instruction to base-protocol.md to avoid TodoWrite (hard denial via settings.json is ineffective under --dangerously-skip-permissions)

Test plan

  • go test -race ./... passes
  • Preflight validation proven: wave run gh-implement without gh on PATH fails with clear error and recovery instructions
  • Run wave run wave-land with dirty working tree — verify feature branch is created and PR workflow works
  • Run a pipeline and verify TodoWrite usage is reduced after base-protocol instruction

The commit step was committing directly to whatever branch the user
was on (typically main), so the ship step pushed to main without a PR.
Now the commit step creates a feature branch first, enabling the
standard push → PR → merge workflow.
@nextlevelshit nextlevelshit merged commit 45ee6c0 into main Mar 1, 2026
3 checks passed
@nextlevelshit nextlevelshit deleted the fix/wave-land-branch-workflow branch March 1, 2026 09:27
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