fix: add warning message for ember-source below v4.1

Author: aklkv

ember-engines-router-service/package.json

@@ -36,7 +36,8 @@
     "start": "rollup --config --watch"
   },
   "dependencies": {
-    "@embroider/addon-shim": "^1.8.4"
+    "@embroider/addon-shim": "^1.8.4",
+    "@embroider/macros": "^1.13.2"
   },
   "devDependencies": {
     "@babel/core": "^7.20.7",
@@ -55,8 +56,11 @@
     "prettier": "^2.8.1",
     "rollup": "^3.8.1"
   },
+  "peerDependencies": {
+    "ember-source": "^3.28.0 || ^4.0.0"
+  },
   "engines": {
-    "node": "14.* || 16.* || >= 18"
+    "node": "16.* || 18.* || >= 20"
   },
   "publishConfig": {
     "registry": "https://registry.npmjs.org"

ember-engines-router-service/src/services/engine-router-service.js

@@ -5,6 +5,7 @@ import { action, computed } from '@ember/object';
 import { reads } from '@ember/object/computed';
 import { getOwner } from '@ember/application';
 import Evented from '@ember/object/evented';
+import { macroCondition, dependencySatisfies } from '@embroider/macros';
 import { namespaceEngineRouteName } from '../utils/namespace-engine-route-name';
 import { getRootOwner } from '../utils/root-owner';
 import { resemblesURL } from '../utils/resembles-url';
@@ -66,21 +67,29 @@ export default class EngineRouterService extends Service.extend(Evented) {
   }
 
   refresh(routeName = this.currentRouteName, ...args) {
-    if (resemblesURL(routeName)) {
-      return this.externalRouter.refresh(routeName);
+    if (macroCondition(dependencySatisfies('ember-source', '>= 4.1.0'))) {
+      if (resemblesURL(routeName)) {
+        return this.externalRouter.refresh(routeName);
+      }
+
+      return this.externalRouter.refresh(
+        namespaceEngineRouteName(this._mountPoint, routeName),
+        ...args
+      );
+    } else {
+      assert('Refresh method is not available in ember-source below v4.1');
     }
-
-    return this.externalRouter.refresh(
-      namespaceEngineRouteName(this._mountPoint, routeName),
-      ...args
-    );
   }
 
   refreshExternal(routeName, ...args) {
-    return this.externalRouter.refresh(
-      this.getExternalRouteName(routeName),
-      ...args
-    );
+    if (macroCondition(dependencySatisfies('ember-source', '>= 4.1.0'))) {
+      return this.externalRouter.refresh(
+        this.getExternalRouteName(routeName),
+        ...args
+      );
+    } else {
+      assert('Refresh method is not available in ember-source below v4.1');
+    }
   }
 
   transitionTo(routeName, ...args) {

yarn.lock

@@ -1189,7 +1189,7 @@
     typescript-memoize "^1.0.1"
     walk-sync "^3.0.0"
 
-"@embroider/macros@1.13.2", "@embroider/macros@^1.0.0", "@embroider/macros@^1.10.0", "@embroider/macros@^1.13.0", "@embroider/macros@^1.3.0":
+"@embroider/macros@1.13.2", "@embroider/macros@^1.0.0", "@embroider/macros@^1.10.0", "@embroider/macros@^1.13.0", "@embroider/macros@^1.13.2", "@embroider/macros@^1.3.0":
   version "1.13.2"
   resolved "https://registry.yarnpkg.com/@embroider/macros/-/macros-1.13.2.tgz#07dda11313a2539f403404881b729e622a80ca17"
   integrity sha512-AUgJ71xG8kjuTx8XB1AQNBiebJuXRfhcHr318dCwnQz9VRXdYSnEEqf38XRvGYIoCvIyn/3c72LrSwzaJqknOA==