Skip to content

fix: preserve workflow handoffs across branches#414

Open
malikrohail wants to merge 1 commit intogarrytan:mainfrom
malikrohail:malikrohail/fix-workflow-handoff-state
Open

fix: preserve workflow handoffs across branches#414
malikrohail wants to merge 1 commit intogarrytan:mainfrom
malikrohail:malikrohail/fix-workflow-handoff-state

Conversation

@malikrohail
Copy link
Contributor

Summary

Fixes the still-open root causes from #280 that make multi-step workflows lose context when skills run on different branches.

What changed

  • Add bin/gstack-find-artifact to resolve the best design doc or CEO plan for the current repo/branch
  • Make gstack-review-log write to a project-scoped reviews.jsonl in addition to the legacy branch file
  • Make gstack-review-read prefer the project-scoped log and fall back to all legacy per-branch logs when needed
  • Update /plan-eng-review to load both the current design doc and the active CEO plan via the helper instead of branch-fragile globs
  • Update /plan-ceo-review and /autoplan to use the shared design-doc lookup helper
  • Add behavioral tests covering cross-branch review visibility, legacy log fallback, branch-aware design doc lookup, and CEO plan lookup

Why this fixes the bug

Current main already has the narrow /review logging fix, but workflow continuity still breaks because:

  • review discovery is effectively branch-scoped
  • design doc lookup still depends on branch-specific filenames
  • /plan-eng-review has no direct CEO-plan discovery path

This PR moves those handoffs to project scope and gives the planning skills a shared artifact lookup path instead of ad hoc globs.

Verification

  • bun run gen:skill-docs
  • bun run gen:skill-docs --host codex
  • bun run gen:skill-docs --dry-run
  • bun test test/workflow-handoff.test.ts
  • bun test

Refs #280

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