Fix Protocol E2E sync test timeouts

[Copilot]

CI failing on test/local-first/Protocol.test.ts with test timeouts in E2E sync suite. The "client and relay have all data" test was timing out at the default 5000ms limit.

Changes

• Added { timeout: 15000 } to four E2E sync tests that perform reconciliation with multiple async disposable runners:
  • "client and relay have all data"
  • "client has all data - many steps"
  • "relay has all data - many steps"
  • "client and relay each have a random half of the data - many steps"

The timeout increase follows the pattern from commit 0a8f3ea, which fixed a similar flaky test in the same suite. These tests invoke the reconcile function which creates disposable runners in a loop, causing them to exceed the default timeout after the structured concurrency migration.

it("client and relay have all data", { timeout: 15000 }, async () => {
  await using run = testCreateRunner();
  const [clientStorage, relayStorage] = await createStorages();
  await run(clientStorage.writeMessages(testOwnerIdBytes, messages));
  await run(relayStorage.writeMessages(testOwnerIdBytes, messages));

  const syncSteps = await reconcile(clientStorage, relayStorage);
  // ...
});

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

• 🔍 Trigger a full review

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}