From dabf513dc328a89d4e589773a4994fea6c4cd89a Mon Sep 17 00:00:00 2001 From: Diego Maranhao Date: Thu, 20 Aug 2026 22:55:40 -0400 Subject: [PATCH 1/2] fix(app): resolve AppEntry TDZ from Kessel circular init --- package-lock.json | 1 - src/AppEntry.js | 25 +++++++++++++++---------- 2 files changed, 15 insertions(+), 11 deletions(-) diff --git a/package-lock.json b/package-lock.json index 7c8ffc350..5700f11db 100644 --- a/package-lock.json +++ b/package-lock.json @@ -6048,7 +6048,6 @@ "resolved": "https://registry.npmjs.org/@project-kessel/react-kessel-access-check/-/react-kessel-access-check-0.5.7.tgz", "integrity": "sha512-hxT7NfemD/fbRkPO6JERUEvgs8nGj1mcarZC7B7m97gH8Jj5FV0f5NhacdsY/R+hlwcv2s63OfpBTAvy/1TRVw==", "license": "Apache-2.0", - "peer": true, "peerDependencies": { "react": "^16.8.0 || ^17.0.0 || ^18.0.0 || ^19.0.0" } diff --git a/src/AppEntry.js b/src/AppEntry.js index e06710dc2..99c66a9ff 100644 --- a/src/AppEntry.js +++ b/src/AppEntry.js @@ -1,7 +1,6 @@ -import React from 'react'; +import React, { Suspense } from 'react'; import { Provider } from 'react-redux'; import { store } from './redux'; -import App from './app'; import './styles/index.scss'; /** @@ -9,19 +8,25 @@ import './styles/index.scss'; * @module AppEntry */ -/** - * ToDo: review consolidating between this and app.js - */ + +const App = React.lazy(() => import('./app')); + /** * Application entry. * - A platform required file, including how it's cased. + * Function declaration is hoisted so Chrome can resolve this export during circular init. + * App is lazy-loaded so Kessel/auth are not evaluated in the same module pass. * * @returns {JSX.Element} */ -const AppEntry = () => ( - - - -); +function AppEntry() { + return ( + + + + + + ); +} export { AppEntry as default, AppEntry }; From 393acc8cc470d8ed7cb0abfd8d46cc59c324b9b8 Mon Sep 17 00:00:00 2001 From: Diego Maranhao Date: Fri, 21 Aug 2026 08:55:07 -0400 Subject: [PATCH 2/2] chore(app): fix AppEntry lint and refresh dist snapshot Co-authored-by: Cursor --- src/AppEntry.js | 21 ++++++++++----------- src/README.md | 1 + tests/__snapshots__/dist.test.js.snap | 16 ++++++++++------ 3 files changed, 21 insertions(+), 17 deletions(-) diff --git a/src/AppEntry.js b/src/AppEntry.js index 99c66a9ff..21439288c 100644 --- a/src/AppEntry.js +++ b/src/AppEntry.js @@ -8,25 +8,24 @@ import './styles/index.scss'; * @module AppEntry */ - const App = React.lazy(() => import('./app')); +/** + * ToDo: review consolidating between this and app.js + */ /** * Application entry. * - A platform required file, including how it's cased. - * Function declaration is hoisted so Chrome can resolve this export during circular init. * App is lazy-loaded so Kessel/auth are not evaluated in the same module pass. * * @returns {JSX.Element} */ -function AppEntry() { - return ( - - - - - - ); -} +const AppEntry = () => ( + + + + + +); export { AppEntry as default, AppEntry }; diff --git a/src/README.md b/src/README.md index 1a1af5aee..18fb424be 100644 --- a/src/README.md +++ b/src/README.md @@ -46,6 +46,7 @@ Curiosity application start. ### AppEntry~AppEntry() ⇒ JSX.Element Application entry. - A platform required file, including how it's cased. +App is lazy-loaded so Kessel/auth are not evaluated in the same module pass. **Kind**: inner method of [AppEntry](#Base.module_AppEntry) diff --git a/tests/__snapshots__/dist.test.js.snap b/tests/__snapshots__/dist.test.js.snap index 0b2e938c7..57161650c 100644 --- a/tests/__snapshots__/dist.test.js.snap +++ b/tests/__snapshots__/dist.test.js.snap @@ -3,7 +3,8 @@ exports[`Build distribution should match a specific file output 1`] = ` [ "", - "./dist/css/632*css", + "./dist/css/374*css", + "./dist/css/819*css", "./dist/css/827*css", "./dist/fed-mods.json", "./dist/fonts/graph2x.png", @@ -11,19 +12,23 @@ exports[`Build distribution should match a specific file output 1`] = ` "./dist/js/133*js", "./dist/js/139*js", "./dist/js/146*js", + "./dist/js/191*js", + "./dist/js/191*txt", "./dist/js/215*js", "./dist/js/223*js", + "./dist/js/232*js", "./dist/js/247*js", "./dist/js/251*js", "./dist/js/335*js", + "./dist/js/374*js", "./dist/js/388*js", "./dist/js/393*js", - "./dist/js/395*js", - "./dist/js/395*txt", "./dist/js/398*js", "./dist/js/436*js", "./dist/js/454*js", "./dist/js/474*js", + "./dist/js/480*js", + "./dist/js/480*txt", "./dist/js/486*js", "./dist/js/514*js", "./dist/js/520*js", @@ -33,14 +38,13 @@ exports[`Build distribution should match a specific file output 1`] = ` "./dist/js/628*js", "./dist/js/630*js", "./dist/js/630*txt", - "./dist/js/632*js", "./dist/js/7*js", "./dist/js/716*js", "./dist/js/73*js", - "./dist/js/765*js", "./dist/js/766*js", "./dist/js/794*js", - "./dist/js/824*js", + "./dist/js/810*js", + "./dist/js/819*js", "./dist/js/827*js", "./dist/js/93*js", "./dist/js/945*js",