Revert "feat: add LocationSensor helper component for next AppRouter and PagesRouter" (#793)

Author: thomaswhyyou

.changeset/cuddly-grapes-lose.md

@@ -0,0 +1,6 @@
+---
+"@knocklabs/client": patch
+"@knocklabs/react": patch
+---
+
+revert 4b888c4 to remove nextjs helper components and fix module resolution error

packages/client/src/clients/guide/client.ts

@@ -341,7 +341,7 @@ export class KnockGuideClient {
 
   cleanup() {
     this.unsubscribe();
-    this.removeLocationChangeEventListeners();
+    this.removeEventListeners();
     this.clearGroupStage();
     this.clearCounterInterval();
   }
@@ -1144,7 +1144,6 @@ export class KnockGuideClient {
 
   // Define as an arrow func property to always bind this to the class instance.
   private handleLocationChange = () => {
-    this.knock.log(`[Guide] .handleLocationChange`);
     const win = checkForWindow();
     if (!win?.location) return;
 
@@ -1224,7 +1223,7 @@ export class KnockGuideClient {
     }
   }
 
-  removeLocationChangeEventListeners() {
+  private removeEventListeners() {
     const win = checkForWindow();
     if (!win?.history) return;
 

packages/react/package.json

@@ -46,15 +46,9 @@
     "url": "https://github.com/knocklabs/javascript/issues"
   },
   "peerDependencies": {
-    "next": "^13.0.0 || ^14.0.0 || ^15.0.0 || ^16.0.0",
     "react": "^16.11.0 || ^17.0.0 || ^18.0.0 || ^19.0.0",
     "react-dom": "^16.11.0 || ^17.0.0 || ^18.0.0 || ^19.0.0"
   },
-  "peerDependenciesMeta": {
-    "next": {
-      "optional": true
-    }
-  },
   "dependencies": {
     "@knocklabs/client": "workspace:^",
     "@knocklabs/react-core": "workspace:^",
@@ -85,7 +79,6 @@
     "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-react-refresh": "^0.4.14",
     "jsdom": "^27.0.0",
-    "next": "15.3.3",
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
     "rimraf": "^6.0.1",

packages/react/src/index.ts

@@ -44,8 +44,6 @@ export {
   CardView,
   KnockGuideProvider,
   GuideToolbar as KnockGuideToolbar,
-  LocationSensorNextPagesRouter as KnockGuideLocationSensorNextPagesRouter,
-  LocationSensorNextAppRouter as KnockGuideLocationSensorNextAppRouter,
   Modal,
   ModalView,
 } from "./modules/guide";

packages/react/src/modules/guide/components/LocationSensor/NextAppRouter.tsx

@@ -1,8 +1,4 @@
 export { Banner, BannerView } from "./Banner";
 export { Card, CardView } from "./Card";
 export { GuideToolbar } from "./GuideToolbar";
-export {
-  LocationSensorNextAppRouter,
-  LocationSensorNextPagesRouter,
-} from "./LocationSensor";
 export { Modal, ModalView } from "./Modal";

packages/react/src/modules/guide/components/LocationSensor/NextPagesRouter.tsx

@@ -6,7 +6,5 @@ export {
   Modal,
   ModalView,
   GuideToolbar,
-  LocationSensorNextAppRouter,
-  LocationSensorNextPagesRouter,
 } from "./components";
 export { KnockGuideProvider } from "./providers";

packages/react/src/modules/guide/components/LocationSensor/index.ts

@@ -4228,7 +4228,6 @@ __metadata:
     eslint-plugin-react-refresh: "npm:^0.4.14"
     jsdom: "npm:^27.0.0"
     lodash.debounce: "npm:^4.0.8"
-    next: "npm:15.3.3"
     react: "npm:^19.0.0"
     react-dom: "npm:^19.0.0"
     rimraf: "npm:^6.0.1"
@@ -4241,12 +4240,8 @@ __metadata:
     vitest: "npm:^3.1.1"
     vitest-axe: "npm:^0.1.0"
   peerDependencies:
-    next: ^13.0.0 || ^14.0.0 || ^15.0.0 || ^16.0.0
     react: ^16.11.0 || ^17.0.0 || ^18.0.0 || ^19.0.0
     react-dom: ^16.11.0 || ^17.0.0 || ^18.0.0 || ^19.0.0
-  peerDependenciesMeta:
-    next:
-      optional: true
   languageName: unknown
   linkType: soft