René's URL Explorer Experiment

Title: Fix DOM order reversal, stale dependency tracking, localStorage SafariError, and reactive bq-for updates by Copilot · Pull Request #26 · bQuery/bQuery · GitHub

Open Graph Title: Fix DOM order reversal, stale dependency tracking, localStorage SafariError, and reactive bq-for updates by Copilot · Pull Request #26 · bQuery/bQuery

X Title: Fix DOM order reversal, stale dependency tracking, localStorage SafariError, and reactive bq-for updates by Copilot · Pull Request #26 · bQuery/bQuery

Description: Addresses 4 critical issues from PR #16 review: multi-element insertion order, reactive subscription leaks, Safari private mode crashes, and stale bq-for bindings. DOM Insertion Order insertContent() now reverses element arrays for afterbegin/afterend positions since each insertAdjacentElement() is relative to the original target: // Before: [A, B, C] → C B A (reversed) // After: [A, B, C] → A B C (correct) element.prepend([divA, divB, divC]); element.after([divA, divB, divC]); Test Coverage: Added 4 comprehensive tests verifying correct insertion order for append(), prepend(), before(), and after() methods when passing arrays of elements. Reactive Dependency Tracking Implemented proper cleanup to prevent stale subscriptions when observers conditionally read different signals: Added ReactiveSource interface for Signal and Computed clearDependencies() unsubscribes from old sources before observer re-runs Fixed infinite loops by snapshotting subscriber sets during iteration // Previously: observer stayed subscribed to both branches effect(() => flag.value ? signalA.value : signalB.value); flag.value = !flag.value; // Now correctly unsubscribes from unused branch Safari Private Mode Compatibility persistedSignal() now catches SecurityError when accessing localStorage (Safari throws on property access, not just usage): // Wrapped in try/catch with actual access test and cleanup try { storage = globalThis.localStorage; storage.setItem('__bquery_test__', '__bquery_test__'); storage.getItem('__bquery_test__'); hasLocalStorage = true; } finally { storage.removeItem('__bquery_test__'); // Ensure cleanup } Test Coverage: Added test simulating Safari private mode by mocking localStorage to throw SecurityError, verifying graceful fallback to in-memory signal. Reactive bq-for Items Both item and index are now signals, enabling updates when list items are reused:

Open Graph Description: Addresses 4 critical issues from PR #16 review: multi-element insertion order, reactive subscription leaks, Safari private mode crashes, and stale bq-for bindings. DOM Insertion Order insertContent...

X Description: Addresses 4 critical issues from PR #16 review: multi-element insertion order, reactive subscription leaks, Safari private mode crashes, and stale bq-for bindings. DOM Insertion Order insertContent...

Opengraph URL: https://github.com/bQuery/bQuery/pull/26

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width