Skip to content

fix: align repo auto init with GitHub API#99

Merged
zhangyangyu merged 1 commit into
mainfrom
codex/align-repo-auto-init-20260615
Jun 14, 2026
Merged

fix: align repo auto init with GitHub API#99
zhangyangyu merged 1 commit into
mainfrom
codex/align-repo-auto-init-20260615

Conversation

@zhangyangyu

Copy link
Copy Markdown
Collaborator

Description

Align repository creation with the GitHub REST API by removing the service-specific add_readme create-repo option, using only auto_init, and seeding an empty README.md when auto-init is requested.

Updates service tests, E2E scripts, smoke coverage, and docs to use the standard field.

Tracking

N/A

Tests

  • go test ./internal/gitstore
  • go test ./internal/service -run '^$'
  • go test ./internal/rest -run '^$'
  • go test ./internal/githttp -run '^$'
  • go test ./... -run '^$'
  • bash -n e2e/code-search-e2e.sh e2e/repo-transfer-lifecycle.sh e2e/vector-search-e2e.sh scripts/backend_smoke.sh
  • python3 scripts/doc_lint.py
  • git diff --check

Risks / Reviewer Notes

Clients that send the non-standard add_readme field will no longer trigger README seeding; callers should send GitHub-compatible auto_init: true. Unknown JSON fields continue to be ignored by the existing decoder.

SSOT Drift Prevention Checklist

  • Checked that this PR does not violate boundaries in docs/architecture.md or docs/module-contracts.md
  • If this PR changes package structure or dependencies, updated the relevant SSOT doc
  • If this PR changes test conventions or coverage expectations, updated docs/test-strategy.md
  • If this PR changes CI, smoke, regression, or workflow expectations, updated docs/ci.md

@zhangyangyu zhangyangyu merged commit e447969 into main Jun 14, 2026
15 of 16 checks passed
@zhangyangyu zhangyangyu deleted the codex/align-repo-auto-init-20260615 branch June 14, 2026 18:03
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