Report CdnPreconnect for cdn.shopify.com without a path

.changeset/cdn-preconnect-bare-origin.md

@@ -0,0 +1,5 @@
+---
+'@shopify/theme-check-common': patch
+---
+
+Report `CdnPreconnect` for `href="https://cdn.shopify.com"` and `href="//cdn.shopify.com"` without a trailing path, and stop reporting hosts that only contain `cdn.shopify.com` in their path.

packages/theme-check-common/src/checks/cdn-preconnect/index.spec.ts

@@ -27,6 +27,26 @@ describe('Module: CdnPreconnect', () => {
     );
   });
 
+  it('should report preconnect to cdn.shopify.com without a trailing slash', async () => {
+    const sourceCode = `
+      <link rel="preconnect" href="https://cdn.shopify.com" crossorigin>
+      <link rel="preconnect" href="//cdn.shopify.com">
+    `;
+
+    const offenses = await runLiquidCheck(CdnPreconnect, sourceCode);
+    expect(offenses).to.have.length(2);
+  });
+
+  it('should not report preconnect to other hosts', async () => {
+    const sourceCode = `
+      <link rel="preconnect" href="https://fonts.shopifycdn.com" crossorigin>
+      <link rel="preconnect" href="https://example.com/cdn.shopify.com.html">
+    `;
+
+    const offenses = await runLiquidCheck(CdnPreconnect, sourceCode);
+    expect(offenses).to.have.length(0);
+  });
+
   it('should not report other link types', async () => {
     const sourceCode = `
       <link rel="preload" href="https://example.com/foo.css" as="style"/>

packages/theme-check-common/src/checks/cdn-preconnect/index.ts

@@ -29,7 +29,10 @@ export const CdnPreconnect: LiquidCheckDefinition = {
 
       const isShopifyCdn = node.attributes
         .filter(isValuedHtmlAttribute)
-        .some((attr) => isAttr(attr, 'href') && valueIncludes(attr, '.+cdn.shopify.com.+'));
+        .some(
+          (attr) =>
+            isAttr(attr, 'href') && valueIncludes(attr, '(https?:)?//cdn\\.shopify\\.com(/\\S*)?'),
+        );
 
       if (!isShopifyCdn) return;