Merge branch 'main' into split-playwright-runs

pixiebrix · Jun 10, 2024 · adaa8b0 · adaa8b0
2 parents 2ffd8e7 + 87f0bbd
commit adaa8b0
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 31 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -271,7 +271,7 @@
     "jest-webextension-mock": "^3.9.0",
     "jsdom": "^24.1.0",
     "jsdom-testing-mocks": "^1.13.0",
-    "knip": "^5.16.0",
+    "knip": "^5.18.2",
     "mini-css-extract-plugin": "^2.6.1",
     "mockdate": "^3.0.5",
     "msw": "^1.3.3",

diff --git a/src/contentScript/lifecycle.ts b/src/contentScript/lifecycle.ts
@@ -237,13 +237,15 @@ export function getActiveExtensionPoints(): StarterBrick[] {
 
 /**
  * Test helper to get internal persisted extension state
+ * @knip used in tests, see lifecycle.test.ts
  */
 export function TEST_getPersistedExtensions(): Map<UUID, StarterBrick> {
   return _persistedExtensions;
 }
 
 /**
  * Test helper to get internal editor extension state
+ * @knip used in tests, see lifecycle.test.ts
  */
 export function TEST_getEditorExtensions(): Map<UUID, StarterBrick> {
   return _editorExtensions;
@@ -561,6 +563,7 @@ function decideRunReason({ force }: { force?: boolean }): RunReason {
 
 /**
  * Handle a website navigation, e.g., page load or a URL change in an SPA.
+ * @knip export used in tests, see lifecycle.test.ts
  */
 export async function handleNavigate({
   force,