Skip to content

Comments

Migrate issue delivery skills to plan-issue binaries#169

Merged
graysurf merged 1 commit intomainfrom
feat/plan-issue-cli-skill-migration
Feb 25, 2026
Merged

Migrate issue delivery skills to plan-issue binaries#169
graysurf merged 1 commit intomainfrom
feat/plan-issue-cli-skill-migration

Conversation

@graysurf
Copy link
Owner

Migrate issue delivery skills to plan-issue binaries

Summary

This PR migrates issue/plan delivery workflows from legacy shell wrappers to the Rust CLI binaries plan-issue and plan-issue-local, and updates dependent skills, docs, and tests to the new binary-first contract.

Changes

  • Added migration references:
    • docs/migrations/plan-issue-cli-skill-mapping.md
    • docs/plans/plan-issue-cli-skill-migration-plan.md
  • Updated docs to remove stale wrapper references and use binary-first examples:
    • docs/plans/duck-issue-loop-test-plan.md
    • docs/runbooks/skills/TOOLING_INDEX_V2.md
  • Rewrote orchestration skills to binary-first command contracts:
    • skills/automation/plan-issue-delivery-loop/SKILL.md
    • skills/automation/issue-delivery-loop/SKILL.md
  • Reworked issue subagent execution contract to scriptless native git/gh workflow:
    • skills/workflows/issue/issue-subagent-pr/SKILL.md
  • Updated issue-pr-review to use internal PR body hygiene validation and removed runtime dependency on deleted subagent wrapper:
    • skills/workflows/issue/issue-pr-review/SKILL.md
    • skills/workflows/issue/issue-pr-review/scripts/manage_issue_pr_review.sh
  • Removed deprecated wrapper scripts and obsolete script-spec fixtures:
    • skills/automation/plan-issue-delivery-loop/scripts/plan-issue-delivery-loop.sh
    • skills/automation/issue-delivery-loop/scripts/manage_issue_delivery_loop.sh
    • skills/workflows/issue/issue-subagent-pr/scripts/manage_issue_subagent_pr.sh
    • corresponding deleted entries under tests/script_specs/...
  • Updated affected tests to assert binary-first/scriptless contracts.

Testing

  • scripts/check.sh --all (pass)
  • Full pytest suite from scripts/check.sh --all: 358 passed

Risk / Notes

  • Breaking change: the three legacy wrapper scripts above were deleted; downstream callers must use plan-issue / plan-issue-local or documented native git/gh commands.
  • issue-pr-review PR body validation behavior changed to internal checks (required headings + placeholder rejection), so error text and failure shape are updated accordingly.

@graysurf graysurf marked this pull request as ready for review February 25, 2026 10:55
@graysurf graysurf merged commit c4fb49a into main Feb 25, 2026
13 checks passed
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@graysurf graysurf deleted the feat/plan-issue-cli-skill-migration branch February 25, 2026 10: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