Skip to content

feat(sync): add gRPC client dependencies and generated protobuf stubs#22

Merged
kickthemoon0817 merged 2 commits intomainfrom
feature/sync-client-deps
Feb 18, 2026
Merged

feat(sync): add gRPC client dependencies and generated protobuf stubs#22
kickthemoon0817 merged 2 commits intomainfrom
feature/sync-client-deps

Conversation

@kickthemoon0817
Copy link
Copy Markdown
Owner

Summary

Add gRPC client dependencies to the app's Package.swift and copy generated protobuf client stubs into Core/Sync/Generated/.

Changes

  • Package.swift: Added grpc-swift-2, grpc-swift-nio-transport, grpc-swift-protobuf, and swift-protobuf as package dependencies. Wired GRPCCore, GRPCNIOTransportHTTP2, GRPCProtobuf, and SwiftProtobuf into the Pedef target.
  • Core/Sync/Generated/: Copied 5 generated protobuf files from PedefSync providing client stubs for SyncService (Pull/Push/Status RPCs) and PaperService (CRUD + streaming PDF upload/download).

Testing

  • Build verified with xcodebuild -scheme Pedef -configuration Debug build -destination 'platform=macOS'BUILD SUCCEEDED
  • No runtime behavior changes; this is dependency/scaffolding only.

Notes

  • Generated files have @available(macOS 15.0, iOS 18.0, ...) on gRPC client types — sync features will be gated behind availability checks in subsequent PRs.
  • Phase 4, Branch 1 of the multi-platform sync implementation.

Add grpc-swift-2, grpc-swift-nio-transport, grpc-swift-protobuf, and
swift-protobuf as package dependencies. Wire GRPCCore,
GRPCNIOTransportHTTP2, GRPCProtobuf, and SwiftProtobuf into the Pedef
executable target.
Copy pedef_models.pb.swift, pedef_sync.pb.swift, pedef_sync.grpc.swift,
pedef_papers.pb.swift, and pedef_papers.grpc.swift from PedefSync into
Core/Sync/Generated/ so the app can use the gRPC client types for
SyncService and PaperService.
@kickthemoon0817 kickthemoon0817 merged commit 2352198 into main Feb 18, 2026
1 check passed
@kickthemoon0817 kickthemoon0817 deleted the feature/sync-client-deps branch February 18, 2026 14:36
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