v0.18.7 - Revert to v0.18.3 answer processing logic

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 21:03:23 +00:00 · 2025-12-14 14:39:26 +01:00 · 2025-12-14 14:39:15 +01:00 · 2025-12-14 14:29:06 +01:00 · 2025-12-14 14:28:57 +01:00 · 2025-12-14 14:19:44 +01:00
3 changed files with 19 additions and 9 deletions
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,19 +1,19 @@
 {
  "name": "@xtr-dev/rondevu-client",
-  "version": "0.18.1",
+  "version": "0.18.7",
  "lockfileVersion": 3,
  "requires": true,
  "packages": {
    "": {
      "name": "@xtr-dev/rondevu-client",
-      "version": "0.18.1",
+      "version": "0.18.7",
      "license": "MIT",
      "dependencies": {
-        "@noble/ed25519": "^3.0.0"
+        "@noble/ed25519": "^3.0.0",
+        "eventemitter3": "^5.0.1"
      },
      "devDependencies": {
        "@eslint/js": "^9.39.1",
-        "@types/node": "^25.0.2",
        "@typescript-eslint/eslint-plugin": "^8.48.1",
        "@typescript-eslint/parser": "^8.48.1",
        "eslint": "^9.39.1",
@@ -1082,6 +1082,8 @@
      "integrity": "sha512-gWEkeiyYE4vqjON/+Obqcoeffmk0NF15WSBwSs7zwVA2bAbTaE0SJ7P0WNGoJn8uE7fiaV5a7dKYIJriEqOrmA==",
      "dev": true,
      "license": "MIT",
+      "optional": true,
+      "peer": true,
      "dependencies": {
        "undici-types": "~7.16.0"
      }
@@ -2001,6 +2003,12 @@
        "node": ">=0.10.0"
      }
    },
+    "node_modules/eventemitter3": {
+      "version": "5.0.1",
+      "resolved": "https://registry.npmjs.org/eventemitter3/-/eventemitter3-5.0.1.tgz",
+      "integrity": "sha512-GWkBvjiSZK87ELrYOSESUYeVIc9mvLLf/nXalMOS5dYrgZq9o5OVkbZAVM06CVxYsCwH9BDZFPlQTlPA1j4ahA==",
+      "license": "MIT"
+    },
    "node_modules/fast-deep-equal": {
      "version": "3.1.3",
      "resolved": "https://registry.npmjs.org/fast-deep-equal/-/fast-deep-equal-3.1.3.tgz",
@@ -2844,7 +2852,9 @@
      "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-7.16.0.tgz",
      "integrity": "sha512-Zz+aZWSj8LE6zoxD+xrjh4VfkIG8Ya6LvYkZqtUQGJPZjYl53ypCaUwWqo7eI0x66KBGeRo+mlBEkMSeSZ38Nw==",
      "dev": true,
-      "license": "MIT"
+      "license": "MIT",
+      "optional": true,
+      "peer": true
    },
    "node_modules/update-browserslist-db": {
      "version": "1.2.2",
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "@xtr-dev/rondevu-client",
-  "version": "0.18.1",
+  "version": "0.18.7",
  "description": "TypeScript client for Rondevu with durable WebRTC connections, automatic reconnection, and message queuing",
  "type": "module",
  "main": "dist/index.js",
@@ -25,7 +25,6 @@
  "license": "MIT",
  "devDependencies": {
    "@eslint/js": "^9.39.1",
-    "@types/node": "^25.0.2",
    "@typescript-eslint/eslint-plugin": "^8.48.1",
    "@typescript-eslint/parser": "^8.48.1",
    "eslint": "^9.39.1",
@@ -42,6 +41,7 @@
    "README.md"
  ],
  "dependencies": {
-    "@noble/ed25519": "^3.0.0"
+    "@noble/ed25519": "^3.0.0",
+    "eventemitter3": "^5.0.1"
  }
 }
--- a/src/rondevu.ts
+++ b/src/rondevu.ts
@@ -1,6 +1,6 @@
 import { RondevuAPI, Keypair, IceCandidate, BatcherOptions } from './api.js'
 import { CryptoAdapter } from './crypto-adapter.js'
-import { EventEmitter } from 'events'
+import { EventEmitter } from 'eventemitter3'

 // ICE server preset names
 export type IceServerPreset = 'ipv4-turn' | 'hostname-turns' | 'google-stun' | 'relay-only'
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
Bas van den Aakster	fcd0f8ead0	0.18.4	2025-12-14 14:06:57 +01:00
Bas van den Aakster	8fd4b249de	Fix EventEmitter for cross-platform compatibility (v0.18.3) Replace Node.js 'events' module with 'eventemitter3' package to ensure compatibility in both browser and Node.js environments. Changes: - Replace import from 'events' to 'eventemitter3' - Add eventemitter3 as dependency - Remove @types/node (no longer needed) Fixes browser bundling error where 'events' module was not available. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>	2025-12-14 14:05:36 +01:00
Bas van den Aakster	275c156c64	0.18.3	2025-12-14 13:35:45 +01:00
Bas van den Aakster	c60a5f332a	Merge remote-tracking branch 'origin/main'	2025-12-14 13:35:38 +01:00
Bas	ecd6be7f8a	Merge pull request #7 from xtr-dev/claude/fix-issue-6-CTKj9 Fix issue #6	2025-12-14 13:35:19 +01:00
Bas van den Aakster	0f469e234d	0.18.2	2025-12-14 12:41:22 +01:00