@uniweb/kit 0.12.1 → 0.12.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/kit",
-  "version": "0.12.1",
+  "version": "0.12.3",
   "description": "Standard component library for Uniweb foundations",
   "type": "module",
   "exports": {
@@ -44,8 +44,8 @@
     "shiki": "^3.0.0",
     "tailwind-merge": "^3.6.0",
     "temml": "^0.13.2",
-    "@uniweb/core": "^0.9.0",
-    "@uniweb/semantic-parser": "^1.2.2",
+    "@uniweb/core": "^0.10.1",
+    "@uniweb/semantic-parser": "^1.2.3",
     "@uniweb/scene": "^0.1.3"
   },
   "peerDependencies": {

package/src/hooks/index.js

@@ -39,7 +39,7 @@ export {
 export { useFormSubmit } from './useFormSubmit.js'
 export { useFormValues, valueAt } from './useFormValues.js'
 
-// Site tracking — one event stream (`kb/framework/plans/tracking.md`).
+// Site tracking — one event stream.
 // `block.track(name, data)` is the common case and needs no hook; these cover
 // events with no block in hand, the consent gate, and opt-in scroll reporting.
 export { useTracker } from './useTracker.js'

package/src/hooks/useScrollDepth.js

@@ -24,6 +24,7 @@
 
 import { useEffect, useRef } from 'react'
 import { getUniweb } from '@uniweb/core'
+import { useRouting } from './useRouting.js'
 
 const MILESTONES = [25, 50, 75, 100]
 
@@ -47,6 +48,21 @@ export function useScrollDepth(options = {}) {
   const lastCheck = useRef(0)
   const reported = useRef(new Set())
 
+  // ⛔ **Keyed on the path, or "once each per page" is not true.** The milestone
+  // set lives in a ref, so without this the effect runs once at mount and the
+  // set is never cleared again — a second page reports nothing at all. That bites
+  // exactly where this hook is most naturally called: once, in a layout
+  // component, which persists across SPA navigation (`PageRenderer` does not
+  // remount — the router declares a single catch-all route). Calling it in every
+  // section would have hidden the bug and multiplied the events instead.
+  //
+  // ⭐ `pathname` rather than a route object, deliberately: it is the same
+  // boundary `usePageView` emits on, so a `scroll_depth` always pairs with the
+  // `page_view` it belongs to. `useRouting` returns a default location when there
+  // is no Router, so this stays SSG-safe.
+  const { useLocation } = useRouting()
+  const { pathname } = useLocation()
+
   useEffect(() => {
     const tracking = getUniweb()?.tracking
     // Skip the listener entirely when there is nowhere to report — the calls
@@ -54,6 +70,9 @@ export function useScrollDepth(options = {}) {
     if (!enabled || !tracking?.isEnabled?.()) return
 
     reported.current.clear()
+    // Reset the throttle too: a navigation within the throttle window would
+    // otherwise make the immediate check below return early and miss the fold.
+    lastCheck.current = 0
 
     const handleScroll = () => {
       const now = Date.now()
@@ -72,7 +91,7 @@ export function useScrollDepth(options = {}) {
     handleScroll() // a page that fits the viewport is already at 100
     window.addEventListener('scroll', handleScroll, { passive: true })
     return () => window.removeEventListener('scroll', handleScroll)
-  }, [enabled, throttleMs, event])
+  }, [enabled, throttleMs, event, pathname])
 }
 
 export default useScrollDepth

package/src/hooks/useTrackingConsent.js

@@ -50,8 +50,12 @@ export function useTrackingConsent() {
 
   const set = useCallback((granted) => {
     getUniweb()?.tracking?.setConsent(granted)
-    // Read back rather than assuming: a tracker with no destination stays
-    // 'granted' and never moves, and the banner should not claim otherwise.
+    // Read back rather than assuming. The tracker is the authority on its own
+    // state, and a site that never asked for a gate starts — and stays —
+    // 'granted', so a banner following the pattern above never renders and this
+    // never runs. Recording a decision does not depend on the tracker having
+    // anywhere to send, so the status moves in a framed document too — where
+    // the tracker is disabled and nothing is transmitted either way.
     setStatus(readStatus())
   }, [])