Skip to content

[AAASM-4959] πŸ”§ (config): Migrate default branch master β†’ main (outbound refs)#293

Merged
Chisanan232 merged 2 commits into
mainfrom
v0.0.1/AAASM-4959/config/master_to_main
Jul 23, 2026
Merged

[AAASM-4959] πŸ”§ (config): Migrate default branch master β†’ main (outbound refs)#293
Chisanan232 merged 2 commits into
mainfrom
v0.0.1/AAASM-4959/config/master_to_main

Conversation

@Chisanan232

Copy link
Copy Markdown
Contributor

Description

Default-branch migration for python-sdk (master β†’ main) per ADR 0016 (Accepted). The GitHub branch rename has already been performed (default is now main, protection moved, master redirects). This PR sweeps the outbound in-tree references the rename does not touch.

Functional refs flipped:

  • CI branch filters β€” push/pull_request/workflow_run branches: on all workflows (ci.yaml, benchmarks.yml, codeql.yml, documentation.yaml, framework-resolution-check.yml, native-pin-consistency.yml, quickstart-tabs-check.yml, readme-version-check.yml, release-python-conversion-test.yml, type-check.yml, wheel-python-matrix.yml).
  • github.ref_name == 'master' conditionals in rw_build_and_test.yaml / rw_run_all_test_and_record.yaml.
  • actions/checkout ref: master in documentation.yaml + docs-backfill.yaml.
  • Operator dispatch commands gh workflow run … --ref master in docs/release/RUNBOOK.md and the sdk-only-release / release-runbook skills (these hard-break after rename β€” the ref is gone).
  • README shields.io badges (?branch=master), codecov.yml (branch / strict_yaml_branch), mkdocs.yml (edit_uri).
  • Convention docs β€” CONTRIBUTING.md base-branch + worktree commands, .claude/CLAUDE.md base-branch + remote/master guidance.

Intentionally NOT changed:

  • External Chisanan232/…@master reusable-workflow refs (different repo's default branch β€” ADR 0016 Β§6).
  • Redirect-covered github.com/…/blob/master Β· /tree/master Β· /commits/master web links in docs/** (cosmetic per ADR 0016 Β§6; GitHub 301-redirects them).
  • Historical verification-reports/** (records of past runs).

Type of Change

  • πŸ”§ Bug fix (release/CI migration β€” config)
  • πŸ“š Documentation update

Breaking Changes

  • No β€” the rename itself moved the default + protection and installs a masterβ†’main redirect. This PR keeps CI firing on main and keeps operator runbooks accurate.

Related Issues

  • Related JIRA ticket: AAASM-4959 (Epic AAASM-4955)

Lockstep dependency

The release-coupling with agent-assembly is handled in a separate PR on agent-assembly (base master, since that repo is not yet migrated): it flips the PYTHON_SDK_BOT_TOKEN create-pull-request step base: master β†’ main in .github/workflows/release.yml and the check-release-completeness.sh drift guard. Both land in the same window as this rename.

Testing

  • Static: git grep confirms no functional master ref remains (branch filters / ref: / --ref / ?branch= / ref_name); only cosmetic redirect-covered web links and external @master refs remain.
  • Rename verified: default = main; branch protection (1 approval + code-owner review) re-verified on main; no master branch recreated.
  • No unit/integration tests required β€” config/docs only.

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated

Chisanan232 and others added 2 commits July 21, 2026 16:20
…→ main (AAASM-4959)

Default-branch migration per ADR 0016. Repoints push/pull_request/
workflow_run branch filters, github.ref_name conditionals, and
actions/checkout ref: to main. External Chisanan232 reusable-workflow
@master refs are intentionally left unchanged (different repo).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
… (AAASM-4959)

Default-branch migration per ADR 0016. Updates README CI/docs badges
(?branch=), codecov branch config, mkdocs edit_uri, CONTRIBUTING base-branch
convention + worktree commands, .claude/CLAUDE.md convention, and the
operator release runbooks/skills (--ref dispatch commands). Redirect-covered
github.com blob/commits web links are left as cosmetic per ADR 0016 Β§6.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 21, 2026

Copy link
Copy Markdown

Codecov Report

βœ… All modified and coverable lines are covered by tests.

πŸ“’ Thoughts on this report? Let us know!

@Chisanan232

Copy link
Copy Markdown
Contributor Author

πŸ€– Claude Code review (evidence only β€” merge is a Pioneer reviewer's call)

Part of the python-sdk master β†’ main migration (AAASM-4959, ADR 0016). This PR fixes python-sdk's own branch-name refs after the default-branch rename.

  • Rename: default_branch=main, protection (1 review) moved to main, master redirects, no master branch remains.
  • Scope: outbound branch-filter refs + docs only. Cross-repo links use rename-proof /blob/HEAD/.
  • Release lockstep (separate PR): the agent-assembly release.yml python bot-PR base: + the drift-guard map are flipped to main in agent-assembly#1624 β€” must merge in the same window so the next release opens the python pin-bump PR against main.
  • CI: green (27 pass / 0 fail).

Not self-merged. Evidence only.

@Chisanan232

Copy link
Copy Markdown
Contributor Author

πŸ€– Claude Code review (evidence only β€” merge is a Pioneer reviewer's call)

1. CI β€” βœ… green (27 pass / 0 fail / 7 skip). 2. CodeCov/SonarCloud β€” βœ… no issues (the codecov.yml change is only branch/strict_yaml_branch masterβ†’main, i.e. the coverage default-branch β€” no threshold/gate change).

3. Scope vs AAASM-4959 β€” βœ… complete. python-sdk masterβ†’main migration: 22 files, 54 -master / 54 +main (perfectly balanced) β€” workflow branch filters, docs, README, CONTRIBUTING, mkdocs, codecov default-branch. Release lockstep is in the paired agent-assembly#1624 (must merge together).

4. Side effects β€” βœ… none. Pure branch-reference migration (every removed master replaced by main; zero logic/job/threshold edits). No product/business-logic code touched.

5. Front-end β€” n/a. No FE source changed (workflows + Markdown + CI config only) β†’ Playwright not applicable.

Verdict: ready for approval + merge. Not self-merged.

@Chisanan232
Chisanan232 merged commit bd8dea0 into main Jul 23, 2026
34 checks passed
@Chisanan232
Chisanan232 deleted the v0.0.1/AAASM-4959/config/master_to_main branch July 23, 2026 00:56
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