Skip to content

chore: add migrate-pr script, skill, and docs#920

Open
kayibal wants to merge 3 commits into
mainfrom
chore/migrate-pr-tooling
Open

chore: add migrate-pr script, skill, and docs#920
kayibal wants to merge 3 commits into
mainfrom
chore/migrate-pr-tooling

Conversation

@kayibal

@kayibal kayibal commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • scripts/migrate-pr.sh: auto-detects path mappings per source repo (tycho-protocol-sdk, tycho-simulation, tycho-execution), strips unmapped diffs (.github/, Cargo.lock), rewrites path literals on added content lines (include_str! etc.), and falls back to --reject on failure so partial application is preserved
  • scripts/Readme.md: documents usage, known manual fixups, and the wiggle workflow for resolving .rej files into inline conflict markers
  • .claude/skills/migrate-pr/SKILL.md: agent skill covering end-to-end migration with a dedicated conflict-resolution section explaining how git am --continue resumes remaining patches after each fix

Test plan

  • Script lints clean: shellcheck scripts/migrate-pr.sh
  • ./scripts/migrate-pr.sh ../tycho-protocol-sdk <branch> applies cleanly for a known-good branch
  • Unknown repo without explicit maps prints a clear error

- scripts/migrate-pr.sh: auto-detect path mappings per source repo,
  strip unmapped/.github/Cargo.lock diffs, fall back to --reject on
  failure (drops unsafe -C0 that could silently misplace hunks)
- scripts/Readme.md: document usage, manual fixups, and wiggle workflow
  for resolving .rej files into inline conflict markers
- .claude/skills/migrate-pr/SKILL.md: agent skill covering end-to-end
  migration with a dedicated conflict-resolution section

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

louise-poole and others added 2 commits April 20, 2026 14:11
…o crates in migrate-pr

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…te-pr

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-project-automation github-project-automation Bot moved this from Todo to In Progress in Tycho Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants