Review nits on the native merge: README pin, uuid as a direct dependency
native/README.md said both sides pin Rastrillo Native 33ffe99. Only the Go server module does; Package.swift pins 2295259, five commits later, which the recent-accounts client needs. Say both. native.go imports github.com/google/uuid directly, but go.mod still listed it // indirect. go mod tidy, with both direct requirements the merge brought in moved into the main require block. Fable review of #13, nits 1 and 2.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2 files changed,
+6
−4
go.mod+2 −3native/README.md+4 −1
CI passed — run details
CI log
=== vet === go vet ./... go: downloading amadan.net/rastrillo/native/server v0.0.0-20260912204600-33ffe9941734 === fmt === === test === go test ./... ? amadan.net/oficina/memoria/cmd/memoria [no test files] ok amadan.net/oficina/memoria/internal/memoria 0.681s ok amadan.net/oficina/memoria/internal/memoria/chunk 0.004s ? amadan.net/oficina/memoria/internal/memoria/icons [no test files] ok amadan.net/oficina/memoria/internal/memoria/index 0.031s ok amadan.net/oficina/memoria/internal/memoria/notion 0.017s ok amadan.net/oficina/memoria/internal/memoria/query 0.003s ok amadan.net/oficina/memoria/internal/memoria/seal 0.003s ok amadan.net/oficina/memoria/internal/memoria/sibling 3.020s ok amadan.net/oficina/memoria/internal/memoriatest 1.294s ? amadan.net/oficina/memoria/tools/bench [no test files] ? amadan.net/oficina/memoria/tools/drive [no test files] ? amadan.net/oficina/memoria/tools/seed [no test files] ? amadan.net/oficina/memoria/tools/vendorassets [no test files] === migration-check === go run github.com/carlosframework/rastrillo/cmd/rastrillo migration check rastrillo migration check: models and migrations agree
diff --git a/go.mod b/go.mod| index 210a388..76b6613 100644 |
| --- a/go.mod |
| +++ b/go.mod |
| @@ -9,8 +9,10 @@ require ( |
| amadan.net/oficina/home/connect v0.0.0-20260910000424-5b8d8410775f |
| amadan.net/oficina/home/handoff v0.0.0-20260908102137-481d2d57f489 |
| amadan.net/oficina/suite v0.0.0-20260905215529-60b01a8a0d6d |
| + amadan.net/rastrillo/native/server v0.0.0-20260912204600-33ffe9941734 |
| github.com/carlosframework/rastrillo v0.23.0 |
| github.com/go-chi/chi/v5 v5.3.2 |
| + github.com/google/uuid v1.6.0 |
| golang.org/x/crypto v0.55.0 |
| gorm.io/gorm v1.31.2 |
| modernc.org/sqlite v1.55.0 |
| @@ -24,7 +26,6 @@ require ( |
| require ( |
| github.com/BurntSushi/toml v1.6.0 // indirect |
| github.com/dustin/go-humanize v1.0.1 // indirect |
| - github.com/google/uuid v1.6.0 // indirect |
| github.com/jinzhu/inflection v1.0.0 // indirect |
| github.com/jinzhu/now v1.1.5 // indirect |
| github.com/keymaildev/signin v0.1.1 // indirect |
| @@ -40,5 +41,3 @@ require ( |
| ) |
| tool github.com/carlosframework/rastrillo/cmd/rastrillo |
| - |
| -require amadan.net/rastrillo/native/server v0.0.0-20260912204600-33ffe9941734 |
diff --git a/native/README.md b/native/README.md| index 5a02551..3296646 100644 |
| --- a/native/README.md |
| +++ b/native/README.md |
| @@ -15,7 +15,10 @@ the iOS project with `xcodegen generate --spec native/ios/project.yml`, then bui |
| `MemoriaPhone` in Xcode. Physical devices require your development team. |
| The generated projects and build products are ignored. |
| -The Swift and Go dependencies pin Rastrillo Native revision `33ffe994173438bbb45e88d16061eba2ac4f6d4c`. |
| +The Swift package pins Rastrillo Native revision `229525909e1d6ac3b6a353d1c4d022798b0e9a24` |
| +(`native/apple/Package.swift`); the Go server module pins `33ffe994173438bbb45e88d16061eba2ac4f6d4c` |
| +(`go.mod`, `amadan.net/rastrillo/native/server`). `2295259` is five commits past `33ffe99` |
| +and carries the saved-accounts refresh the recent-accounts client needs. |
| Browser sign-in requires the new backend `/native/` routes. A server that has |
| not received this update reports that native support is unavailable. |