Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Common SDK + CLI Demo #2

Merged
merged 28 commits into from
Mar 26, 2025
Merged

feat: Common SDK + CLI Demo #2

merged 28 commits into from
Mar 26, 2025

Conversation

Chriztiaan
Copy link
Collaborator

@Chriztiaan Chriztiaan commented Mar 11, 2025

This PR introduces the first iteration of the Closed Alpha SDK for .NET support.
It includes the following:

  • Common SDK that works for CLI/binary usage, supported by a Microsoft.Data.Sqlite driver adapter (See MDSQLiteAdapter.cs)
    • Uses one write connection and one read connection
  • A C# CLI Demo
  • Unit tests that cover BucketStorage, CRUD, and transactions against PowerSyncDatabase

Work to be done in future iterations:

  • Batched execution
  • Bucket Priorities
  • Comprehensive unit test suite
  • Proper NuGet release process + CI/CD
  • Schema validation
  • ORM support
  • MAUI/WPF demos/packages

The root README.md covers setup, which includes the step for pulling the appropriate PowerSync core extension for your OS/Architecture.

Demo:
image

@Chriztiaan Chriztiaan changed the title Feat/common sdk feat: Common SDK + CLI Demo Mar 12, 2025
@Chriztiaan Chriztiaan marked this pull request as ready for review March 12, 2025 13:22
benitav
benitav previously approved these changes Mar 12, 2025
@Chriztiaan
Copy link
Collaborator Author

Chriztiaan commented Mar 26, 2025

I have made notes about the current feedback, to be addressed in a follow-up PR.

Copy link
Collaborator

@stevensJourney stevensJourney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy with the changes so far. Remaining items can be addressed later.

@Chriztiaan Chriztiaan merged commit 2ae759e into main Mar 26, 2025
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.

3 participants