v0.18.6 - Fix duplicate answer bug (timestamp update)

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>

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@xtr-dev/rondevu-client",
-  "version": "0.18.4",
+  "version": "0.18.6",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@xtr-dev/rondevu-client",
-      "version": "0.18.4",
+      "version": "0.18.6",
       "license": "MIT",
       "dependencies": {
         "@noble/ed25519": "^3.0.0",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xtr-dev/rondevu-client",
-  "version": "0.18.4",
+  "version": "0.18.6",
   "description": "TypeScript client for Rondevu with durable WebRTC connections, automatic reconnection, and message queuing",
   "type": "module",
   "main": "dist/index.js",

src/rondevu.ts

@@ -645,23 +645,37 @@ export class Rondevu extends EventEmitter {
         try {
             const result = await this.api.poll(this.lastPollTimestamp)
 
+            // Update timestamp FIRST to prevent re-fetching same answers if processing fails
+            if (result.answers.length > 0) {
+                const maxAnswerTimestamp = Math.max(...result.answers.map(a => a.answeredAt))
+                this.lastPollTimestamp = Math.max(this.lastPollTimestamp, maxAnswerTimestamp)
+            }
+
             // Process answers
             for (const answer of result.answers) {
                 const activeOffer = this.activeOffers.get(answer.offerId)
                 if (activeOffer && !activeOffer.answered) {
                     this.debug(`Received answer for offer ${answer.offerId}`)
 
+                    // Mark as answered BEFORE setRemoteDescription to prevent race condition
+                    activeOffer.answered = true
+
+                    try {
                         await activeOffer.pc.setRemoteDescription({
                             type: 'answer',
                             sdp: answer.sdp
                         })
 
-                    activeOffer.answered = true
-                    this.lastPollTimestamp = answer.answeredAt
                         this.emit('offer:answered', answer.offerId, answer.answererId)
 
                         // Create replacement offer
                         this.fillOffers()
+                    } catch (err) {
+                        // If setRemoteDescription fails, reset the answered flag
+                        activeOffer.answered = false
+                        this.debug(`Failed to set remote description for offer ${answer.offerId}:`, err)
+                        // Don't throw - continue processing other answers
+                    }
                 }
             }