Note clock abstraction as Step 8 polish item.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
2026-03-23 21:45:20 -07:00
parent c552a3657f
commit 0d53ca34aa
2 changed files with 4 additions and 0 deletions

View File

@@ -45,6 +45,9 @@ Step 7: Remaining Commands (remove, verify, list, diff).
replication is deferred to a future phase.
- **gRPC remote mode**: Phase 2. Package structure is designed to accommodate
it (garden core separates logic from CLI wiring).
- **Clock abstraction**: Inject a clock interface (e.g. `jonboulle/clockwork`)
into Garden instead of calling `time.Now()` directly. Improves timestamp
test determinism. Deferred to Step 8 (Polish).
## Change Log

View File

@@ -87,6 +87,7 @@ Depends on Step 5.
## Step 8: Polish
- [ ] Lint setup (golangci-lint config)
- [ ] Clock abstraction: inject `jonboulle/clockwork` into Garden for deterministic timestamp tests
- [ ] End-to-end test: init → add → checkpoint → modify file → status → restore → verify
- [ ] Ensure `go vet ./...` and `go test ./...` pass clean
- [ ] Update CLAUDE.md, ARCHITECTURE.md, PROGRESS.md