@nordicsemiconductor/pc-nrfconnect-shared 246.0.0 → 247.0.0

package/dist/typings/scripts/check-app-properties.d.ts

@@ -1,4 +1,5 @@
 #!/usr/bin/env tsx
+export declare const sameRepoURLs: (url1: string, url2: string) => boolean;
 declare const runChecks: ({ checkChangelogHasCurrentEntry, }: {
     checkChangelogHasCurrentEntry: boolean;
 }) => void;

package/dist/typings/scripts/check-app-properties.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"check-app-properties.d.ts","sourceRoot":"","sources":["../../../scripts/check-app-properties.ts"],"names":[],"mappings":";AAkLA,QAAA,MAAM,SAAS,GAAI,oCAEhB;IACC,6BAA6B,EAAE,OAAO,CAAC;CAC1C,SAKA,CAAC;AAOF,eAAe,SAAS,CAAC"}
+{"version":3,"file":"check-app-properties.d.ts","sourceRoot":"","sources":["../../../scripts/check-app-properties.ts"],"names":[],"mappings":";AAmEA,eAAO,MAAM,YAAY,GAAI,MAAM,MAAM,EAAE,MAAM,MAAM,YAUtD,CAAC;AAuGF,QAAA,MAAM,SAAS,GAAI,oCAEhB;IACC,6BAA6B,EAAE,OAAO,CAAC;CAC1C,SAKA,CAAC;AAOF,eAAe,SAAS,CAAC"}

package/dist/typings/scripts/check-app-properties.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=check-app-properties.test.d.ts.map

package/dist/typings/scripts/check-app-properties.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"check-app-properties.test.d.ts","sourceRoot":"","sources":["../../../scripts/check-app-properties.test.ts"],"names":[],"mappings":""}

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@nordicsemiconductor/pc-nrfconnect-shared",
-    "version": "246.0.0",
+    "version": "247.0.0",
     "description": "Shared commodities for developing pc-nrfconnect-* packages",
     "repository": {
         "type": "git",

package/release_notes.md

@@ -1,3 +1,3 @@
 ### Changed
 
-- Aligned Card component styles with old implementation.
+- In the repo URL check, ignore the hostname for ssh git URLs.

package/scripts/check-app-properties.test.ts

@@ -0,0 +1,60 @@
+/*
+ * Copyright (c) 2026 Nordic Semiconductor ASA
+ *
+ * SPDX-License-Identifier: LicenseRef-Nordic-4-Clause
+ */
+
+import { sameRepoURLs } from './check-app-properties';
+
+describe('sameRepoURLs', () => {
+    it('accepts equal URLs', () => {
+        expect(
+            sameRepoURLs(
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk',
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk',
+            ),
+        ).toBe(true);
+    });
+
+    it('rejects different URLs', () => {
+        expect(
+            sameRepoURLs(
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk',
+                'https://github.com/nordicsemi/pc-nrfconnect-shared',
+            ),
+        ).toBe(false);
+    });
+
+    it('ignores a .git postfix on either URL', () => {
+        expect(
+            sameRepoURLs(
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk.git',
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk',
+            ),
+        ).toBe(true);
+        expect(
+            sameRepoURLs(
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk',
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk.git',
+            ),
+        ).toBe(true);
+    });
+
+    it('treats the protocol https and git as the same', () => {
+        expect(
+            sameRepoURLs(
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk',
+                'git@github.com:nordicsemi/pc-nrfconnect-ppk.git',
+            ),
+        ).toBe(true);
+    });
+
+    it('ignores the host name for the git protocol', () => {
+        expect(
+            sameRepoURLs(
+                'https://github.com/nordicsemi/pc-nrfconnect-ppk',
+                'git@nordic-git:nordicsemi/pc-nrfconnect-ppk.git',
+            ),
+        ).toBe(true);
+    });
+});

package/scripts/check-app-properties.ts

@@ -65,6 +65,18 @@ const mustContainOneOf = (
     }
 };
 
+export const sameRepoURLs = (url1: string, url2: string) => {
+    const withoutPostfix = (gitUrl: string) => gitUrl.replace(/\.git$/, '');
+
+    const withoutProtocol = (gitUrl: string) =>
+        gitUrl.replace(/^git@[^:]+:/, 'github.com/').replace(/^https:\/\//, '');
+
+    const stripped = (gitUrl: string) =>
+        withoutProtocol(withoutPostfix(gitUrl));
+
+    return stripped(url1) === stripped(url2);
+};
+
 const checkRepoUrl = (packageJson: PackageJsonApp) => {
     if (!existsSync('./.git')) {
         return;
@@ -75,17 +87,7 @@ const checkRepoUrl = (packageJson: PackageJsonApp) => {
     }).trimEnd();
     const declaredGitUrl = packageJson.repository?.url;
 
-    const withoutPostfix = (gitUrl?: string) => gitUrl?.replace(/\.git$/, '');
-
-    const withoutProtocol = (gitUrl?: string) =>
-        gitUrl
-            ?.replace(/^git@github\.com:/, 'github.com/')
-            .replace(/^https:\/\//, '');
-
-    const stripped = (gitUrl?: string) =>
-        withoutProtocol(withoutPostfix(gitUrl));
-
-    if (stripped(realGitUrl) !== stripped(declaredGitUrl)) {
+    if (declaredGitUrl == null || !sameRepoURLs(realGitUrl, declaredGitUrl)) {
         fail(
             `package.json says the repository is located at \`${declaredGitUrl}\` but \`git remote get-url origin\` says it is at \`${realGitUrl}\`.`,
         );