Skip to content

Commit 0881d02

Browse files
chrisutheclaude
andcommitted
chore(sdk): bump to v3.0.0 with breaking interface change
BREAKING: IClockSynchronizer now requires HasMinimalSync property. Update release notes with migration guide for implementers. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent a83fe73 commit 0881d02

1 file changed

Lines changed: 17 additions & 1 deletion

File tree

src/Sendspin.SDK/Sendspin.SDK.csproj

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
<!-- NuGet Package Metadata -->
1010
<PackageId>Sendspin.SDK</PackageId>
11-
<Version>2.2.1</Version>
11+
<Version>3.0.0</Version>
1212
<Authors>Sendspin Contributors</Authors>
1313
<Company>Sendspin</Company>
1414
<Product>Sendspin SDK</Product>
@@ -20,6 +20,22 @@
2020
<PackageLicenseExpression>MIT</PackageLicenseExpression>
2121
<PackageReadmeFile>README.md</PackageReadmeFile>
2222
<PackageReleaseNotes>
23+
v3.0.0 - Breaking Changes:
24+
- BREAKING: IClockSynchronizer interface now requires HasMinimalSync property
25+
- Returns true after 2+ time sync measurements (vs IsConverged which requires 5+)
26+
- Existing implementations must add this property
27+
28+
New Features:
29+
- Quick-start playback: reduced time-to-playback from ~5 seconds to ~300-500ms
30+
- AudioPipeline now uses HasMinimalSync instead of IsConverged for playback gating
31+
- Matches JS/CLI player behavior for faster startup
32+
- Sync correction system handles any initial estimation errors
33+
- Default convergence timeout reduced from 5000ms to 1000ms
34+
35+
Migration Guide:
36+
- If you implement IClockSynchronizer, add: bool HasMinimalSync => MeasurementCount >= 2;
37+
- If you only use KalmanClockSynchronizer (most users), no changes needed
38+
2339
v2.2.1:
2440
Bug Fixes:
2541
- CRITICAL: Fixed sync error calculation that prevented resampling correction from converging

0 commit comments

Comments
 (0)