rondevu-client

mirror of https://github.com/xtr-dev/rondevu-client.git synced 2025-12-15 21:33:23 +00:00

Author	SHA1	Message	Date
Bas van den Aakster	9a4fbb63f8	v0.18.7 - Revert to v0.18.3 answer processing logic	2025-12-14 14:39:26 +01:00
Bas van den Aakster	f8fb842935	Revert to v0.18.3 answer processing logic Reverted pollInternal to exactly match v0.18.3 which was the last working version. The changes in v0.18.5 and v0.18.6 that moved the answered flag and timestamp updates were causing issues. v0.18.3 working logic restored: - Check !activeOffer.answered - Call setRemoteDescription (no try/catch) - Set answered = true AFTER - Update lastPollTimestamp AFTER - No pre-emptive timestamp updates The only difference from v0.18.3 is the eventemitter3 import which should not affect answer processing behavior. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>	2025-12-14 14:39:15 +01:00
Bas van den Aakster	50d49d80d3	v0.18.6 - Fix duplicate answer bug (timestamp update)	2025-12-14 14:29:06 +01:00
Bas van den Aakster	b3b1751f63	Fix duplicate answer bug - update timestamp before processing Root cause: When setRemoteDescription failed, lastPollTimestamp was never updated, causing the server to return the same answer repeatedly. Solution: 1. Update lastPollTimestamp BEFORE processing answers 2. Calculate max timestamp from all received answers upfront 3. Don't throw on setRemoteDescription errors - just log and continue 4. This ensures we advance the timestamp even if processing fails This prevents the infinite loop of: - Poll returns answer - Processing fails - Timestamp not updated - Next poll returns same answer - Repeat Now the timestamp advances regardless of processing success, preventing duplicate answer fetches from the server. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>	2025-12-14 14:28:57 +01:00
Bas van den Aakster	e48b3bb17a	v0.18.5 - Fix duplicate answer processing regression	2025-12-14 14:19:44 +01:00
Bas van den Aakster	8f7e15e633	Fix duplicate answer processing regression The activeOffer.answered flag was being set AFTER the async setRemoteDescription() call, creating a race condition window where the same answer could be processed multiple times. Root cause: - Check `!activeOffer.answered` happens - setRemoteDescription() starts (async operation) - Before it completes, another check could happen - Same answer gets processed twice → "stable" state error Fix: - Set activeOffer.answered = true BEFORE setRemoteDescription - Add try/catch to reset flag if setRemoteDescription fails - This prevents duplicate processing while allowing retries on error This regression was introduced when the answered flag assignment was not moved along with other polling logic changes. Fixes: #6 regression 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>	2025-12-14 14:19:34 +01:00

2 changed files with 3 additions and 3 deletions

									
										4

package-lock.json
									
										generated
									
												View File
												
				@@ -1,12 +1,12 @@

				{

				  "name": "@xtr-dev/rondevu-client",

				  "version": "0.18.4",

				  "version": "0.18.7",

				  "lockfileVersion": 3,

				  "requires": true,

				  "packages": {

				    "": {

				      "name": "@xtr-dev/rondevu-client",

				      "version": "0.18.4",

				      "version": "0.18.7",

				      "license": "MIT",

				      "dependencies": {

				        "@noble/ed25519": "^3.0.0",

									
										2

package.json
									
												View File
												
				@@ -1,6 +1,6 @@

				{

				  "name": "@xtr-dev/rondevu-client",

				  "version": "0.18.4",

				  "version": "0.18.7",

				  "description": "TypeScript client for Rondevu with durable WebRTC connections, automatic reconnection, and message queuing",

				  "type": "module",

				  "main": "dist/index.js",

Compare commits

6 Commits

v0.18.4 ... v0.18.7

4

package-lock.json generated

View File

2

package.json

View File

Compare commits

6 Commits v0.18.4 ... v0.18.7

4 package-lock.json generated Unescape Escape View File

2 package.json Unescape Escape View File

6 Commits

v0.18.4 ... v0.18.7

4

package-lock.json generated

View File

2

package.json

View File