can you take a look again, we renamed the package name and I rebased onto main, also tests seem to fail

Claude Code·Opus 4.7[1m]·Soph·4mo ago·10min·1 Checkpoint·2 file changes·+11/-3·10.7K tokens

can you take a look again, we renamed the package name and I rebased onto main, also tests seem to fail

4mo ago·9m

Everything is green: 200x cancellation tests pass, full suite passes, and lint is clean.

Two issues found and fixed:

  1. CI test flake (real cancellation race) — TestRun_IntegrationV{1,2}FetchCanceledMidStreamFails raced between context cancellation propagating to the client and the malformed zzzz bytes being parsed by go-git's pkt-line reader / network close surfacing first. Under go-git v6 the timing shifted enough to flake in CI.

    • Fixed in internal/gitproto/fetch.go:55 — FetchToStore now joins ctx.Err() with the underlying error when the context was canceled, so errors.Is(err, context.Canceled) is deterministic regardless of which read failure won the race. Underlying error is preserved via errors.Join.
  2. Lint config still references old module path — .golangci.yaml:100 had github.com/entirehq/git-sync/internal/auth.Method in the ireturn allow list, missed during the rename to entire.io/git-sync. Two auth.Method ireturn warnings were leaking through.

    • Fixed by updating to entire.io/git-sync/internal/auth.Method.

Branch is now 10 commits ahead of origin/soph/bump-go-git-v6-alpha2 from the rebase + these fixes — when you're ready, git push --force-with-lease to update the PR.

can you commit?

4mo ago·18.0s

Committed as 98a36eb. Branch is now 11 ahead of origin — push with git push --force-with-lease when ready.