@embroider/shared-internals - npm Package Compare versions

Comparing version 2.6.1-unstable.fad2387 to 2.6.1

package.json

 {
   "name": "@embroider/shared-internals",
-  "version": "2.6.1-unstable.fad2387",
+  "version": "2.6.1",
   "private": false,
@@ -27,5 +27,2 @@ "description": "Utilities shared among the other embroider packages",
   ],
-  "scripts": {
-    "test": "jest"
-  },
   "dependencies": {
@@ -45,3 +42,2 @@ "babel-import-util": "^2.0.0",
     "broccoli-node-api": "^1.7.0",
-    "@embroider/test-support": "workspace:*",
     "@types/babel__core": "^7.1.14",
@@ -58,7 +54,11 @@ "@types/babel__traverse": "^7.18.5",
     "tmp": "^0.1.0",
-    "typescript": "^5.1.6"
+    "typescript": "^5.1.6",
+    "@embroider/test-support": "0.36.0"
   },
   "engines": {
     "node": "12.* || 14.* || >= 16"
+  },
+  "scripts": {
+    "test": "jest"
   }
-}
+}

src/ember-standard-modules.js

@@ -51,4 +51,2 @@ "use strict";
 exports.emberVirtualPackages.add('@glimmer/manager');
-// ember-data pre 4.12 failed to declare this as a peer dependency of @ember-data/debug
-exports.emberVirtualPeerDeps.add('@ember-data/store');
 // These are the known names that people use to import template precomiplation
@@ -55,0 +53,0 @@ // macros from.

src/index.d.ts

 export { AppMeta, AddonMeta, PackageInfo } from './metadata';
-export { explicitRelative, extensionsPattern, unrelativize, cleanUrl, getUrlQueryParams, correspondingTemplate, } from './paths';
+export { explicitRelative, extensionsPattern, unrelativize, cleanUrl, correspondingTemplate } from './paths';
 export { getOrCreate } from './get-or-create';
@@ -4,0 +4,0 @@ export { default as Package, V2AddonPackage as AddonPackage, V2AppPackage as AppPackage, V2Package } from './package';

src/index.js

@@ -20,3 +20,3 @@ "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.locateEmbroiderWorkingDir = exports.cacheBustingPluginVersion = exports.cacheBustingPluginPath = exports.cacheBustingPlugin = exports.templateColocationPluginPath = exports.templateColocationPlugin = exports.hbsToJS = exports.tmpdir = exports.packageName = exports.babelFilter = exports.RewrittenPackageCache = exports.PackageCache = exports.Package = exports.getOrCreate = exports.correspondingTemplate = exports.getUrlQueryParams = exports.cleanUrl = exports.unrelativize = exports.extensionsPattern = exports.explicitRelative = void 0;
+exports.locateEmbroiderWorkingDir = exports.cacheBustingPluginVersion = exports.cacheBustingPluginPath = exports.cacheBustingPlugin = exports.templateColocationPluginPath = exports.templateColocationPlugin = exports.hbsToJS = exports.tmpdir = exports.packageName = exports.babelFilter = exports.RewrittenPackageCache = exports.PackageCache = exports.Package = exports.getOrCreate = exports.correspondingTemplate = exports.cleanUrl = exports.unrelativize = exports.extensionsPattern = exports.explicitRelative = void 0;
 var paths_1 = require("./paths");
@@ -27,3 +27,2 @@ Object.defineProperty(exports, "explicitRelative", { enumerable: true, get: function () { return paths_1.explicitRelative; } });
 Object.defineProperty(exports, "cleanUrl", { enumerable: true, get: function () { return paths_1.cleanUrl; } });
-Object.defineProperty(exports, "getUrlQueryParams", { enumerable: true, get: function () { return paths_1.getUrlQueryParams; } });
 Object.defineProperty(exports, "correspondingTemplate", { enumerable: true, get: function () { return paths_1.correspondingTemplate; } });
@@ -30,0 +29,0 @@ var get_or_create_1 = require("./get-or-create");

src/package-cache.js

@@ -66,19 +66,23 @@ "use strict";
     ownerOfFile(filename) {
-        let segments = filename.replace(/\\/g, '/').split(path_1.posix.sep);
+        let candidate = filename;
         // first we look through our cached packages for any that are rooted right
         // at or above the file.
-        for (let length = segments.length; length >= 0; length--) {
-            if (segments[length - 1] === 'node_modules' || segments[length - 1] === '') {
-                // once we hit a node_modules or the filesystem root, we're leaving the
+        while (true) {
+            if ((0, path_1.basename)(candidate) === 'node_modules') {
+                // once we hit a node_modules, we're leaving the
                 // package we were in, so any higher caches don't apply to us
                 break;
             }
-            let usedSegments = segments.slice(0, length);
-            let candidate = usedSegments.join(path_1.posix.sep);
             if (this.rootCache.has(candidate)) {
                 return this.rootCache.get(candidate);
             }
-            if (getCachedExists([...usedSegments, 'package.json'].join(path_1.posix.sep))) {
+            if (getCachedExists((0, path_1.join)(candidate, 'package.json'))) {
                 return this.get(candidate);
             }
+            let nextCandidate = (0, path_1.resolve)(candidate, '..');
+            if (nextCandidate === candidate) {
+                // got to the top
+                break;
+            }
+            candidate = nextCandidate;
         }
@@ -85,0 +89,0 @@ }

src/paths.d.ts

@@ -5,4 +5,3 @@ import type Package from './package';
 export declare function unrelativize(pkg: Package, specifier: string, fromFile: string): string;
-export declare function cleanUrl(url: string, includeHashSign?: boolean): string;
-export declare function getUrlQueryParams(url: string, includeHashSign?: boolean): string;
+export declare function cleanUrl(url: string): string;
 export declare function correspondingTemplate(filename: string): string;

src/paths.js

 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.correspondingTemplate = exports.getUrlQueryParams = exports.cleanUrl = exports.unrelativize = exports.extensionsPattern = exports.explicitRelative = void 0;
+exports.correspondingTemplate = exports.cleanUrl = exports.unrelativize = exports.extensionsPattern = exports.explicitRelative = void 0;
 const path_1 = require("path");
@@ -46,22 +46,8 @@ // by "explicit", I mean that we want "./local/thing" instead of "local/thing"
 const postfixRE = /[?#].*$/s;
-// this pattern includes URL query params (ex: ?direct)
-// but excludes specifiers starting with # (ex: #embroider_compats/components/fancy)
-// so when using this pattern, #embroider_compat/fancy would be consider a pathname
-// without any params.
-const postfixREQueryParams = /[?].*$/s;
 // this is the same implementation Vite uses internally to keep its
 // cache-busting query params from leaking where they shouldn't.
-// includeHashSign true means #my-specifier is considered part of the pathname
-function cleanUrl(url, includeHashSign = false) {
-    const regexp = includeHashSign ? postfixREQueryParams : postfixRE;
-    return url.replace(regexp, '');
+function cleanUrl(url) {
+    return url.replace(postfixRE, '');
 }
 exports.cleanUrl = cleanUrl;
-// includeHashSign true means #my-specifier is considered part of the pathname
-function getUrlQueryParams(url, includeHashSign = false) {
-    var _a, _b;
-    const regexp = includeHashSign ? postfixREQueryParams : postfixRE;
-    return (_b = (_a = url.match(regexp)) === null || _a === void 0 ? void 0 : _a[0]) !== null && _b !== void 0 ? _b : '';
-}
-exports.getUrlQueryParams = getUrlQueryParams;
 // given a filename, returns it with the hbs extension
@@ -68,0 +54,0 @@ // for instance, passing filename.js returns filename.hbs

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Improved metrics

Number of package files

59

increased by1.72%

Number of low quality alerts

1

decreased by-50%

Worsened metrics

Total package byte prevSize

154208

decreased by-1.29%

Lines of code

1560

decreased by-0.89%

No dependency changes