Address PR review cleanup comments · Entire

Address PR review cleanup comments

ac68fed→main·

Soph·3mo ago·3 files·+9 added/-9 removed

Sessions

36113cbd7ee5View transcript

Changes

3

197 unmodified lines

198
199
200
201
201
202
203
204
152 unmodified lines

357
358
359
360
360
361
362
363
364
365
366
366
367
368
368
369
370
370

197 unmodified lines

- `--scenario bootstrap`
- `--scenario sync`

For large-repo measurements, use a local bare mirror as the source so the benchmark reflects `git-sync` behavior rather than internet variance. See [docs/benchmarking.md](/Users/soph/Work/entire/devenv/git-sync/docs/benchmarking.md) for details.
For large-repo measurements, use a local bare mirror as the source so the benchmark reflects `git-sync` behavior rather than internet variance. See [docs/benchmarking.md](docs/benchmarking.md) for details.

## Sync Behavior

152 unmodified lines

env GOCACHE=/tmp/go-build go test ./...
```

Extended and environment-specific test instructions are in [docs/testing.md](/Users/soph/Work/entire/devenv/git-sync/docs/testing.md).
Extended and environment-specific test instructions are in [docs/testing.md](docs/testing.md).

## Design Notes

`bootstrap` is the dedicated path for large initial syncs into an empty target. The goal is to relay a fetched source pack directly into target `receive-pack` instead of decoding the full object graph into local memory first.

Current architectural summary and package boundaries are in [docs/architecture.md](/Users/soph/Work/entire/devenv/git-sync/docs/architecture.md).
Current architectural summary and package boundaries are in [docs/architecture.md](docs/architecture.md).

The design note is in [docs/bootstrap.md](/Users/soph/Work/entire/devenv/git-sync/docs/bootstrap.md).
The design note is in [docs/bootstrap.md](docs/bootstrap.md).

For very large single-branch repositories, there is also a batching design and initial implementation note in [docs/bootstrap-batching.md](/Users/soph/Work/entire/devenv/git-sync/docs/bootstrap-batching.md).
For very large single-branch repositories, there is also a batching design and initial implementation note in [docs/bootstrap-batching.md](docs/bootstrap-batching.md).

MREADME.md+5/-5

92 unmodified lines

93
94
95
96
96
97
98
99

92 unmodified lines

fs.StringVar(&workDir, "work-dir", "", "directory for temporary target repositories")
    fs.IntVar(&repeat, "repeat", 1, "number of runs to execute")
    fs.BoolVar(&keepTargets, "keep-targets", false, "retain generated target repositories after the run")
    fs.BoolVar(&jsonOutput, "json", true, "print JSON output")
    fs.BoolVar(&jsonOutput, "json", false, "print JSON output")

branches := fs.String("branch", "", "comma-separated branch list; default is all source branches")
    fs.Var(&mappings, "map", "ref mapping in src:dst form; short names map branches, full refs map exact refs")

Mcmd/git-sync-bench/main.go+1/-1

151 unmodified lines

152
153
154
155
156
157
155
156
157

151 unmodified lines

## Related Notes

- [bootstrap.md](/Users/soph/Work/entire/devenv/git-sync/docs/bootstrap.md)
- [bootstrap-batching.md](/Users/soph/Work/entire/devenv/git-sync/docs/bootstrap-batching.md)
- [benchmarking.md](/Users/soph/Work/entire/devenv/git-sync/docs/benchmarking.md)
- [bootstrap.md](bootstrap.md)
- [bootstrap-batching.md](bootstrap-batching.md)
- [benchmarking.md](benchmarking.md)