@dxos/web-context-react 0.8.4-main.bc674ce → 0.8.4-main.bcb3aa67d6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxos/web-context-react",
-  "version": "0.8.4-main.bc674ce",
+  "version": "0.8.4-main.bcb3aa67d6",
   "description": "React integration with web context protocol",
   "homepage": "https://dxos.org",
   "bugs": "https://github.com/dxos/dxos/issues",
@@ -29,7 +29,7 @@
     "src"
   ],
   "dependencies": {
-    "@dxos/web-context": "0.8.4-main.bc674ce"
+    "@dxos/web-context": "0.8.4-main.bcb3aa67d6"
   },
   "devDependencies": {
     "@testing-library/react": "^16.3.0",

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

@@ -175,11 +175,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();
@@ -203,7 +203,7 @@ export const ContextProtocolProvider = <T extends UnknownContext>({
     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 +215,7 @@ export const ContextProtocolProvider = <T extends UnknownContext>({
 
   return (
     <ContextRequestHandlerContext.Provider value={handleRequest}>
-      <div ref={containerRef} style={{ display: 'contents' }}>
+      <div role='none' className='contents' ref={containerRef}>
         {children}
       </div>
     </ContextRequestHandlerContext.Provider>