github.split_branch_in_PRs

Goal

• Propose how to split the changes between current HEAD of the branch and origin/master into small PRs to minimize:

  • The review complexity
  • The risk of breaking the unit tests

• If the task is not perfectly clear or complex, you MUST not perform it, but ask for clarifications and create a plan for review

  • Create a plan.md with 5 bullet points explaining what the plan is

Workflow

Read Rules

• Read rules about:
  • Coding: .claude/skills/coding.rules.md
  • Unit tests: .claude/skills/testing.rules.md

Read the Changes in the Current Git Client

• Obtain the files that need to be merged with:

  git diff --name-status origin/master HEAD
  ...
  

• Analyze the difference between the current client and master (or origin/master):

  git diff master...HEAD --name-only
  # or if master is not available locally:
  git diff origin/master...HEAD --name-only
  

Propose PRs

• Propose a set of PRs that decompose the changes in the current Git client, so that there are coherent and tightly coupled changes

• Prefer PRs that change entire files, not split a file across PRs

• Keep each file and its test file in the same PR

• Merge low risk / complexity PRs to have fewer PRs, even if the merged PR is the sum of simpler ones

  • Describe each merged unit as its own bullet

Order the PRs

• Start with the PRs that have low risk and touch most files

Create pytest Commands

• For each PR create:
  • A file pr<NUM>.files.txt with the files changed, e.g.,

    .claude/skills
    .claude/templates
    .claude/notify.sh
    .claude/settings.local.json
    

  • A file pr<NUM>.pytest.sh with the pytest command to run to exercise the changes by running the tests in all the modified files and make sure the PR is running successfully, e.g.,

    #!/bin/bash
    pytest_log \
      dev_scripts_helpers/coding_tools/test \
      dev_scripts_helpers/dockerize/test \
      linters2/test \
      $@
    

  • Make this script executable with chmod +x pr<NUM>.pytest.sh