@dxos/web-context-react 0.8.4-main.59c2e9b → 0.8.4-main.60689f5b1c

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxos/web-context-react",
-  "version": "0.8.4-main.59c2e9b",
+  "version": "0.8.4-main.60689f5b1c",
   "description": "React integration with web context protocol",
   "homepage": "https://dxos.org",
   "bugs": "https://github.com/dxos/dxos/issues",
@@ -8,7 +8,7 @@
     "type": "git",
     "url": "https://github.com/dxos/dxos"
   },
-  "license": "MIT",
+  "license": "FSL-1.1-Apache-2.0",
   "author": "DXOS.org",
   "sideEffects": true,
   "type": "module",
@@ -21,15 +21,12 @@
     }
   },
   "types": "dist/types/src/index.d.ts",
-  "typesVersions": {
-    "*": {}
-  },
   "files": [
     "dist",
     "src"
   ],
   "dependencies": {
-    "@dxos/web-context": "0.8.4-main.59c2e9b"
+    "@dxos/web-context": "0.8.4-main.60689f5b1c"
   },
   "devDependencies": {
     "@testing-library/react": "^16.3.0",
@@ -37,7 +34,7 @@
     "@types/react-dom": "~19.2.3",
     "react": "~19.2.3",
     "react-dom": "~19.2.3",
-    "vitest": "3.2.4"
+    "vitest": "4.1.6"
   },
   "peerDependencies": {
     "react": "~19.2.3"

package/src/consumer.test.tsx

@@ -5,7 +5,7 @@
 // @vitest-environment jsdom
 
 import { act, cleanup, render, screen } from '@testing-library/react';
-import React from 'react';
+import React, { ReactNode, useEffect, useRef, useState } from 'react';
 import { afterEach, describe, expect, test } from 'vitest';
 
 import { CONTEXT_REQUEST_EVENT, createContext } from '@dxos/web-context';
@@ -37,7 +37,7 @@ describe('useWebComponentContext', () => {
 
   test('consumes context from updates (subscription)', async () => {
     const Container = () => {
-      const [val, setVal] = React.useState('initial');
+      const [val, setVal] = useState('initial');
       return (
         <>
           <button onClick={() => setVal('updated')}>Update</button>
@@ -59,9 +59,9 @@ describe('useWebComponentContext', () => {
   });
 
   test('consumes context from DOM parent (outside React tree)', () => {
-    const Wrapper = ({ children }: { children: React.ReactNode }) => {
-      const ref = React.useRef<HTMLDivElement>(null);
-      React.useEffect(() => {
+    const Wrapper = ({ children }: { children: ReactNode }) => {
+      const ref = useRef<HTMLDivElement>(null);
+      useEffect(() => {
         const handler = (e: Event) => {
           const event = e as any;
           if (event.context === ctx) {

package/src/provider.tsx

@@ -149,8 +149,12 @@ export const ContextProtocolProvider = <T extends UnknownContext>({
   const handleContextProviderEvent = useCallback(
     (e: Event) => {
       const event = e as ContextProviderEvent<UnknownContext>;
-      if (event.context !== context) return;
-      if (containerRef.current && event.contextTarget === containerRef.current) return;
+      if (event.context !== context) {
+        return;
+      }
+      if (containerRef.current && event.contextTarget === containerRef.current) {
+        return;
+      }
 
       const seen = new Set<ContextCallback<ContextType<T>>>();
       for (const ref of subscriptionRefs) {
@@ -161,9 +165,13 @@ export const ContextProtocolProvider = <T extends UnknownContext>({
         }
 
         const info = subscriptions.get(callback);
-        if (!info) continue;
+        if (!info) {
+          continue;
+        }
 
-        if (seen.has(callback)) continue;
+        if (seen.has(callback)) {
+          continue;
+        }
         seen.add(callback);
 
         info.consumerHost.dispatchEvent(
@@ -175,11 +183,11 @@ export const ContextProtocolProvider = <T extends UnknownContext>({
     [context],
   );
 
-  // Notify subscribers when value changes
+  // Notify subscribers when value changes.
   useEffect(() => {
     // Skip notification if value hasn't changed? React does this for us if we use dependencies correctly?
     // No, we need to imperatively call callbacks.
-    // value constraint is in the dependency array
+    // value constraint is in the dependency array.
 
     for (const ref of subscriptionRefs) {
       const callback = ref.deref();
@@ -198,12 +206,14 @@ export const ContextProtocolProvider = <T extends UnknownContext>({
 
   useEffect(() => {
     const el = containerRef.current;
-    if (!el) return;
+    if (!el) {
+      return;
+    }
 
     el.addEventListener(CONTEXT_REQUEST_EVENT, handleContextRequestEvent);
     el.addEventListener(CONTEXT_PROVIDER_EVENT, handleContextProviderEvent);
 
-    // Announce provider
+    // Announce provider.
     el.dispatchEvent(new ContextProviderEvent(context, el));
 
     return () => {
@@ -215,7 +225,7 @@ export const ContextProtocolProvider = <T extends UnknownContext>({
 
   return (
     <ContextRequestHandlerContext.Provider value={handleRequest}>
-      <div ref={containerRef} style={{ display: 'contents' }}>
+      <div className='contents' ref={containerRef}>
         {children}
       </div>
     </ContextRequestHandlerContext.Provider>