@khanacademy/wonder-blocks-core - npm Package Compare versions

Comparing version 4.6.2 to 4.7.0

CHANGELOG.md

 # @khanacademy/wonder-blocks-core
 
+## 4.7.0
+
+### Minor Changes
+
+-   91cb727c: Convert enums to POJOs
+
+### Patch Changes
+
+-   91cb727c: Remove file extensions from imports
+
 ## 4.6.2
@@ -4,0 +14,0 @@

dist/es/index.js

@@ -199,7 +199,7 @@ import _extends from '@babel/runtime/helpers/extends';
 
-const RenderState = require("flow-enums-runtime")({
+const RenderState = {
   Root: "root",
   Initial: "initial",
   Standard: "standard"
-});
+};
 const RenderStateContext = React.createContext(RenderState.Root);
@@ -206,0 +206,0 @@

dist/index.js

@@ -226,7 +226,7 @@ 'use strict';
 
-const RenderState = require("flow-enums-runtime")({
+const RenderState = {
   Root: "root",
   Initial: "initial",
   Standard: "standard"
-});
+};
 const RenderStateContext = React__namespace.createContext(RenderState.Root);
@@ -233,0 +233,0 @@

package.json

 {
   "name": "@khanacademy/wonder-blocks-core",
-  "version": "4.6.2",
+  "version": "4.7.0",
   "design": "v1",
@@ -27,3 +27,3 @@ "publishConfig": {
   "devDependencies": {
-    "wb-dev-build-settings": "^0.7.0"
+    "wb-dev-build-settings": "^0.7.1"
   },
@@ -30,0 +30,0 @@ "author": "",

src/components/__docs__/id-provider.stories.js

@@ -8,3 +8,3 @@ // @flow
 
-import ComponentInfo from "../../../../../.storybook/components/component-info.js";
+import ComponentInfo from "../../../../../.storybook/components/component-info";
 import {name, version} from "../../../package.json";
@@ -11,0 +11,0 @@

src/components/__docs__/unique-id-provider.stories.js

@@ -11,3 +11,3 @@ // @flow
 
-import ComponentInfo from "../../../../../.storybook/components/component-info.js";
+import ComponentInfo from "../../../../../.storybook/components/component-info";
 import {name, version} from "../../../package.json";
@@ -14,0 +14,0 @@

src/components/__docs__/view.stories.js

@@ -16,5 +16,5 @@ // @flow
 
-import ComponentInfo from "../../../../../.storybook/components/component-info.js";
+import ComponentInfo from "../../../../../.storybook/components/component-info";
 import {name, version} from "../../../package.json";
-import viewArgTypes from "./view.argtypes.js";
+import viewArgTypes from "./view.argtypes";
 
@@ -21,0 +21,0 @@ export default {

src/components/__docs__/with-ssr-placeholder.stories.js

@@ -8,3 +8,3 @@ // @flow
 
-import ComponentInfo from "../../../../../.storybook/components/component-info.js";
+import ComponentInfo from "../../../../../.storybook/components/component-info";
 import {name, version} from "../../../package.json";
@@ -11,0 +11,0 @@

src/components/__tests__/id-provider.test.js

@@ -5,3 +5,3 @@ // @flow
 
-import IDProvider from "../id-provider.js";
+import IDProvider from "../id-provider";
 
@@ -8,0 +8,0 @@ const mockIDENTIFIER = `uid-component-0-${IDProvider.defaultId}`;

src/components/__tests__/render-state-root.test.js

@@ -5,8 +5,4 @@ // @flow
 
-import {RenderStateRoot} from "../render-state-root.js";
-// TODO(somewhatabstract, FEI-4174): Update eslint-plugin-import when they
-// have fixed:
-// https://github.com/import-js/eslint-plugin-import/issues/2073
-// eslint-disable-next-line import/named
-import {RenderState, RenderStateContext} from "../render-state-context.js";
+import {RenderStateRoot} from "../render-state-root";
+import {RenderState, RenderStateContext} from "../render-state-context";
 
@@ -13,0 +9,0 @@ const {useContext} = React;

src/components/__tests__/unique-id-provider.test.js

 // @flow
 import * as React from "react";
-import * as ReactDOMServer from "react-dom/server.js";
+import * as ReactDOMServer from "react-dom/server";
 import {render} from "@testing-library/react";
 
-import View from "../view.js";
+import View from "../view";
 
-import SsrIDFactory from "../../util/ssr-id-factory.js";
-import UniqueIDFactory from "../../util/unique-id-factory.js";
-import UniqueIDProvider from "../unique-id-provider.js";
-import WithSSRPlaceholder from "../with-ssr-placeholder.js";
-import {RenderStateRoot} from "../render-state-root.js";
+import SsrIDFactory from "../../util/ssr-id-factory";
+import UniqueIDFactory from "../../util/unique-id-factory";
+import UniqueIDProvider from "../unique-id-provider";
+import WithSSRPlaceholder from "../with-ssr-placeholder";
+import {RenderStateRoot} from "../render-state-root";
 
@@ -14,0 +14,0 @@ describe("UniqueIDProvider", () => {

src/components/__tests__/view.test.js

 import * as React from "react";
 import renderer from "react-test-renderer";
 
-import View from "../view.js";
+import View from "../view";
 
@@ -6,0 +6,0 @@ describe("View", () => {

src/components/__tests__/with-ssr-placeholder.test.js

 // @flow
 import * as React from "react";
-import * as ReactDOMServer from "react-dom/server.js";
+import * as ReactDOMServer from "react-dom/server";
 import {render} from "@testing-library/react";
 
-import WithSSRPlaceholder from "../with-ssr-placeholder.js";
-import {RenderStateRoot} from "../render-state-root.js";
+import WithSSRPlaceholder from "../with-ssr-placeholder";
+import {RenderStateRoot} from "../render-state-root";
 
@@ -9,0 +9,0 @@ describe("WithSSRPlaceholder", () => {

src/components/id-provider.js

 // @flow
 import * as React from "react";
 
-import UniqueIDProvider from "./unique-id-provider.js";
+import UniqueIDProvider from "./unique-id-provider";
 
-import type {IIdentifierFactory} from "../util/types.js";
+import type {IIdentifierFactory} from "../util/types";
 
@@ -8,0 +8,0 @@ type Props = {|

src/components/render-state-context.js

 // @flow
 import * as React from "react";
 
-export enum RenderState {
-    Root = "root",
-    Initial = "initial",
-    Standard = "standard",
-}
+// TODO(FEI-5000): Convert to TS enum after all codebases have been migrated
+export const RenderState = {
+    Root: ("root": "root"),
+    Initial: ("initial": "initial"),
+    Standard: ("standard": "standard"),
+};
 
@@ -25,3 +26,3 @@ /**
  */
-export const RenderStateContext: React.Context<RenderState> =
+export const RenderStateContext: React.Context<$Values<typeof RenderState>> =
     React.createContext(RenderState.Root);

src/components/render-state-root.js

 // @flow
 import * as React from "react";
 
-// TODO(somewhatabstract, FEI-4174): Update eslint-plugin-import when they
-// have fixed:
-// https://github.com/import-js/eslint-plugin-import/issues/2073
-// eslint-disable-next-line import/named
-import {RenderState, RenderStateContext} from "./render-state-context.js";
-import {useRenderState} from "../hooks/use-render-state.js";
+import {RenderState, RenderStateContext} from "./render-state-context";
+import {useRenderState} from "../hooks/use-render-state";
 
@@ -11,0 +7,0 @@ const {useEffect, useState} = React;

src/components/text.js

@@ -5,5 +5,5 @@ // @flow
 
-import {processStyleList} from "../util/util.js";
+import {processStyleList} from "../util/util";
 
-import type {TextViewSharedProps} from "../util/types.js";
+import type {TextViewSharedProps} from "../util/types";
 
@@ -10,0 +10,0 @@ // NOTE(jeresig): We want to leave the props for these open so that we can

src/components/unique-id-provider.js

 // @flow
 import * as React from "react";
 
-import WithSSRPlaceholder from "./with-ssr-placeholder.js";
+import WithSSRPlaceholder from "./with-ssr-placeholder";
 
-import UniqueIDFactory from "../util/unique-id-factory.js";
-import SsrIDFactory from "../util/ssr-id-factory.js";
+import UniqueIDFactory from "../util/unique-id-factory";
+import SsrIDFactory from "../util/ssr-id-factory";
 
-import type {IIdentifierFactory} from "../util/types.js";
+import type {IIdentifierFactory} from "../util/types";
 
@@ -11,0 +11,0 @@ // TODO(FEI-4202): update to use `useUniqueId`

src/components/view.js

@@ -5,5 +5,5 @@ // @flow
 
-import addStyle from "../util/add-style.js";
+import addStyle from "../util/add-style";
 
-import type {TextViewSharedProps} from "../util/types.js";
+import type {TextViewSharedProps} from "../util/types";
 
@@ -10,0 +10,0 @@ const styles = StyleSheet.create({

src/components/with-ssr-placeholder.js

 // @flow
 import * as React from "react";
 
-// TODO(FEI-4202): update to use `useContext(RenderStateContext)`
-// TODO(somewhatabstract, FEI-4174): Update eslint-plugin-import when they
-// have fixed:
-// https://github.com/import-js/eslint-plugin-import/issues/2073
-// eslint-disable-next-line import/named
-import {RenderState, RenderStateContext} from "./render-state-context.js";
+import {RenderState, RenderStateContext} from "./render-state-context";
 
@@ -120,3 +115,3 @@ /**
 
-    _maybeRender(renderState: RenderState): React.Node {
+    _maybeRender(renderState: $Values<typeof RenderState>): React.Node {
         const {children, placeholder} = this.props;
@@ -123,0 +118,0 @@

src/hooks/__tests__/use-force-update.test.js

@@ -6,3 +6,3 @@ // @flow
 
-import {useForceUpdate} from "../use-force-update.js";
+import {useForceUpdate} from "../use-force-update";
 
@@ -9,0 +9,0 @@ describe("#useForceUpdate", () => {

src/hooks/__tests__/use-is-mounted.test.js

 // @flow
 import {renderHook} from "@testing-library/react-hooks";
 
-import {useIsMounted} from "../use-is-mounted.js";
+import {useIsMounted} from "../use-is-mounted";
 
@@ -6,0 +6,0 @@ describe("useIsMounted", () => {

src/hooks/__tests__/use-on-mount-effect.test.js

 // @flow
 import {renderHook} from "@testing-library/react-hooks";
 
-import {useOnMountEffect} from "../use-on-mount-effect.js";
+import {useOnMountEffect} from "../use-on-mount-effect";
 
@@ -6,0 +6,0 @@ describe("#useOnMountEffect", () => {

src/hooks/__tests__/use-online.test.js

@@ -6,3 +6,3 @@ // @flow
 
-import {useOnline} from "../use-online.js";
+import {useOnline} from "../use-online";
 
@@ -9,0 +9,0 @@ describe("useOnline", () => {

src/hooks/__tests__/use-render-state.test.js

@@ -6,9 +6,5 @@ // @flow
 
-import {useRenderState} from "../use-render-state.js";
-import {RenderStateRoot} from "../../components/render-state-root.js";
-// TODO(somewhatabstract, FEI-4174): Update eslint-plugin-import when they
-// have fixed:
-// https://github.com/import-js/eslint-plugin-import/issues/2073
-// eslint-disable-next-line import/named
-import {RenderState} from "../../components/render-state-context.js";
+import {useRenderState} from "../use-render-state";
+import {RenderStateRoot} from "../../components/render-state-root";
+import {RenderState} from "../../components/render-state-context";
 
@@ -15,0 +11,0 @@ describe("useRenderState", () => {

src/hooks/__tests__/use-unique-id.test.js

@@ -6,6 +6,6 @@ // @flow
 
-import SsrIDFactory from "../../util/ssr-id-factory.js";
-import UniqueIDFactory from "../../util/unique-id-factory.js";
-import {useUniqueIdWithMock, useUniqueIdWithoutMock} from "../use-unique-id.js";
-import {RenderStateRoot} from "../../components/render-state-root.js";
+import SsrIDFactory from "../../util/ssr-id-factory";
+import UniqueIDFactory from "../../util/unique-id-factory";
+import {useUniqueIdWithMock, useUniqueIdWithoutMock} from "../use-unique-id";
+import {RenderStateRoot} from "../../components/render-state-root";
 
@@ -12,0 +12,0 @@ describe("useUniqueIdWithoutMock", () => {

src/hooks/use-is-mounted.js

 // @flow
 import * as React from "react";
 
-import {useOnMountEffect} from "./use-on-mount-effect.js";
+import {useOnMountEffect} from "./use-on-mount-effect";
 
@@ -6,0 +6,0 @@ /**

src/hooks/use-online.js

 // @flow
 import {useEffect} from "react";
-import {useForceUpdate} from "./use-force-update.js";
+import {useForceUpdate} from "./use-force-update";
 
@@ -5,0 +5,0 @@ /**

src/hooks/use-render-state.js

 // @flow
 import {useContext} from "react";
 
-import {RenderStateContext} from "../components/render-state-context.js";
+import {
+    RenderState,
+    RenderStateContext,
+} from "../components/render-state-context";
 
-import type {RenderState} from "../components/render-state-context";
-
-export const useRenderState = (): RenderState => useContext(RenderStateContext);
+export const useRenderState = (): $Values<typeof RenderState> =>
+    useContext(RenderStateContext);

src/hooks/use-unique-id.js

 // @flow
 import {useRef} from "react";
 
-import {useRenderState} from "./use-render-state.js";
-import SsrIDFactory from "../util/ssr-id-factory.js";
-import UniqueIDFactory from "../util/unique-id-factory.js";
+import {useRenderState} from "./use-render-state";
+import SsrIDFactory from "../util/ssr-id-factory";
+import UniqueIDFactory from "../util/unique-id-factory";
 
-import {
-    // TODO(somewhatabstract, FEI-4174): Update eslint-plugin-import when they
-    // have fixed:
-    // https://github.com/import-js/eslint-plugin-import/issues/2073
-    // eslint-disable-next-line import/named
-    RenderState,
-} from "../components/render-state-context.js";
+import {RenderState} from "../components/render-state-context";
 
-import type {IIdentifierFactory} from "../util/types.js";
+import type {IIdentifierFactory} from "../util/types";
 
@@ -18,0 +12,0 @@ /**

src/index.js

 // @flow
-import type {AriaProps, IIdentifierFactory, StyleType} from "./util/types.js";
+import type {AriaProps, IIdentifierFactory, StyleType} from "./util/types";
 
-export {default as Text} from "./components/text.js";
-export {default as View} from "./components/view.js";
-export {default as WithSSRPlaceholder} from "./components/with-ssr-placeholder.js";
-export {default as IDProvider} from "./components/id-provider.js";
-export {default as UniqueIDProvider} from "./components/unique-id-provider.js";
-export {default as addStyle} from "./util/add-style.js";
-export {default as Server} from "./util/server.js";
+export {default as Text} from "./components/text";
+export {default as View} from "./components/view";
+export {default as WithSSRPlaceholder} from "./components/with-ssr-placeholder";
+export {default as IDProvider} from "./components/id-provider";
+export {default as UniqueIDProvider} from "./components/unique-id-provider";
+export {default as addStyle} from "./util/add-style";
+export {default as Server} from "./util/server";
 export {
     useUniqueIdWithMock,
     useUniqueIdWithoutMock,
-} from "./hooks/use-unique-id.js";
-export {useForceUpdate} from "./hooks/use-force-update.js";
-export {useIsMounted} from "./hooks/use-is-mounted.js";
-export {useOnMountEffect} from "./hooks/use-on-mount-effect.js";
-export {useOnline} from "./hooks/use-online.js";
-export {useRenderState} from "./hooks/use-render-state.js";
-export {RenderStateRoot} from "./components/render-state-root.js";
-// TODO(somewhatabstract, FEI-4174): Update eslint-plugin-import when they
-// have fixed:
-// https://github.com/import-js/eslint-plugin-import/issues/2073
-// eslint-disable-next-line import/named
-export {RenderState} from "./components/render-state-context.js";
+} from "./hooks/use-unique-id";
+export {useForceUpdate} from "./hooks/use-force-update";
+export {useIsMounted} from "./hooks/use-is-mounted";
+export {useOnMountEffect} from "./hooks/use-on-mount-effect";
+export {useOnline} from "./hooks/use-online";
+export {useRenderState} from "./hooks/use-render-state";
+export {RenderStateRoot} from "./components/render-state-root";
+export {RenderState} from "./components/render-state-context";
 
 export type {AriaProps, IIdentifierFactory, StyleType};

src/util/__tests__/add-style.test.js

@@ -6,3 +6,3 @@ // @flow
 
-import addStyle from "../add-style.js";
+import addStyle from "../add-style";
 
@@ -9,0 +9,0 @@ const StyledDiv = addStyle<"div">("div");

src/util/__tests__/server.test.js

 // @flow
-import Server from "../server.js";
+import Server from "../server";
 
@@ -4,0 +4,0 @@ describe("./server.js", () => {

src/util/__tests__/ssr-id-factory.test.js

@@ -6,3 +6,3 @@ // @flow
         const id = "this-is-the-id";
-        const {default: SsrIDFactory} = await import("../ssr-id-factory.js");
+        const {default: SsrIDFactory} = await import("../ssr-id-factory");
 
@@ -9,0 +9,0 @@ // Act

src/util/__tests__/unique-id-factory.test.js

 // @flow
-import UniqueIDFactory from "../unique-id-factory.js";
+import UniqueIDFactory from "../unique-id-factory";
 
@@ -4,0 +4,0 @@ describe("UniqueIDFactory", () => {

src/util/add-style.js

@@ -5,5 +5,5 @@ // @flow
 
-import {processStyleList} from "./util.js";
+import {processStyleList} from "./util";
 
-import type {StyleType} from "./types.js";
+import type {StyleType} from "./types";
 
@@ -10,0 +10,0 @@ // TODO(kevinb): have an a version which uses exact object types

src/util/add-styles.flowtest.js

@@ -5,3 +5,3 @@ // @flow
 
-import addStyle from "./add-style.js";
+import addStyle from "./add-style";
 
@@ -8,0 +8,0 @@ const styles = StyleSheet.create({

src/util/ssr-id-factory.js

 // @flow
-import type {IIdentifierFactory} from "./types.js";
+import type {IIdentifierFactory} from "./types";
 
@@ -4,0 +4,0 @@ /**

src/util/unique-id-factory.js

 // @flow
-import type {IIdentifierFactory} from "./types.js";
+import type {IIdentifierFactory} from "./types";
 
@@ -4,0 +4,0 @@ /**

src/util/util.js

@@ -5,3 +5,3 @@ // @flow
 
-import type {StyleType} from "./types.js";
+import type {StyleType} from "./types";
 
@@ -8,0 +8,0 @@ type StyledExport = {|

No alert changes

Worsened metrics

Total package byte prevSize

151870

decreased by-0.85%

Number of downloads

6931

decreased by-21.94%

Lines of code

3845

decreased by-0.59%

Number of versions

285

decreased by-0.35%

Published versions last two months

57

decreased by-1.72%

Published versions last week

8

decreased by-11.11%

Published versions last year

174

decreased by-0.57%

No dependency changes