@mui/internal-code-infra 0.0.4-canary.73 → 0.0.4-canary.75

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-code-infra",
-  "version": "0.0.4-canary.73",
+  "version": "0.0.4-canary.75",
   "author": "MUI Team",
   "description": "Infra scripts and configs to be used across MUI repos.",
   "license": "MIT",
@@ -136,8 +136,8 @@
     "yaml": "^2.9.0",
     "yargs": "^18.0.0",
     "@mui/internal-babel-plugin-display-name": "1.0.4-canary.21",
-    "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.38",
-    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.28"
+    "@mui/internal-babel-plugin-minify-errors": "2.0.8-canary.28",
+    "@mui/internal-babel-plugin-resolve-imports": "2.0.7-canary.38"
   },
   "peerDependencies": {
     "@next/eslint-plugin-next": "*",
@@ -181,12 +181,14 @@
     "src",
     "vale",
     "README.md",
-    "LICENSE"
+    "LICENSE",
+    "!src/**/__fixtures__/**",
+    "!src/**/*.test.*"
   ],
   "publishConfig": {
     "access": "public"
   },
-  "gitSha": "a646322dc5697683ef464c5a05a0de6240de9f05",
+  "gitSha": "80047ad2e0a9da7085a7605f4b1ef6ea137ebf73",
   "scripts": {
     "build": "tsgo -p tsconfig.build.json",
     "typescript": "tsgo -noEmit",

package/src/brokenLinksChecker/crawlWorker.mjs

@@ -189,7 +189,30 @@ if (pageData.status < 200 || pageData.status >= 400) {
             rules: {
               // TODO: Enable when subresource integrity is adopted across projects
               'require-sri': 'off',
+              // Components that portal their content (menus, selects, listboxes,
+              // etc.) wire up aria-controls/aria-labelledby targets only after
+              // client hydration, so those targets are absent from the
+              // statically rendered HTML. Disable rather than report false
+              // positives.
+              'no-missing-references': 'off',
             },
+            elements: [
+              'html5',
+              {
+                form: {
+                  attributes: {
+                    // React renders `action="javascript:throw new Error(...)"` on
+                    // a <form> with a function action during SSR (a sentinel that
+                    // throws if the form is submitted before hydration wires up
+                    // the real handler). Permit that value while keeping the
+                    // default check (`^\s*\S+\s*$`) for every other action value.
+                    action: {
+                      enum: ['/^\\s*\\S+\\s*$/', '/^\\s*javascript:throw new Error\\(/'],
+                    },
+                  },
+                },
+              },
+            ],
           },
           ...overridePresets,
         },

package/src/cli/cmdPublish.mjs

@@ -143,8 +143,19 @@ async function checkGitHubReleaseExists(owner, repo, version) {
  */
 async function createGitTag(version, dryRun = false) {
   const tagName = `v${version}`;
+  // The CI smoke-test runs with a read-only token that can't push. It sets this
+  // so we tolerate the push failure explicitly, instead of assuming any dry-run
+  // permission error is expected.
+  const isPublishTest = process.env.IS_PUBLISH_TEST === 'true';
 
   try {
+    // Skip if the tag already exists locally. Tag creation is local and fails if it exists.
+    const { stdout: existingTag } = await $`git tag -l ${tagName}`;
+    if (existingTag.trim()) {
+      console.log(`🏷️  Git tag ${tagName} already exists, skipping`);
+      return;
+    }
+
     await $({
       env: {
         ...process.env,
@@ -152,10 +163,28 @@ async function createGitTag(version, dryRun = false) {
         GIT_COMMITTER_EMAIL: 'code-infra@mui.com',
       },
     })`git tag -a ${tagName} -m ${`Version ${version}`}`;
-    const pushArgs = dryRun ? ['--dry-run'] : [];
-    await $({ stdio: 'inherit' })`git push origin ${tagName} ${pushArgs}`;
 
-    console.log(`🏷️  Created and pushed git tag ${tagName}${dryRun ? ' (dry-run)' : ''}`);
+    if (dryRun) {
+      // `git push --dry-run` still authenticates and needs push access. When the token
+      // can push, validate it for real; when it can't, only tolerate the failure if this
+      // is an explicit publish smoke-test and the error is a genuine permission denial.
+      try {
+        // Don't inherit stdio so stderr is captured and matchable below.
+        await $`git push origin ${tagName} --dry-run`;
+        console.log(`🏷️  Created git tag ${tagName} (dry-run)`);
+      } catch (/** @type {any} */ pushError) {
+        const message = pushError.stderr || pushError.message || '';
+        if (isPublishTest && /permission|403|denied/i.test(message)) {
+          console.log(`🏷️  Created git tag ${tagName} (dry-run, no push permission, skipped push)`);
+          return;
+        }
+        throw pushError;
+      }
+      return;
+    }
+    await $({ stdio: 'inherit' })`git push origin ${tagName}`;
+
+    console.log(`🏷️  Created and pushed git tag ${tagName}`);
   } catch (/** @type {any} */ error) {
     throw new Error(`Failed to create git tag: ${error.message}`);
   }

package/src/brokenLinksChecker/__fixtures__/static-site/broken-links.html

@@ -1,20 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with Broken Links</title>
-  </head>
-  <body>
-    <h1>Page with Broken Links</h1>
-    <p>This page contains links to non-existent pages.</p>
-    <nav>
-      <ul>
-        <li><a href="/">Home (valid)</a></li>
-        <li><a href="/does-not-exist.html">This page does not exist</a></li>
-        <li><a href="/another-missing-page.html">Another missing page</a></li>
-        <li><a href="/valid.html">Valid link</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/broken-targets.html

@@ -1,22 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with Broken Targets</title>
-  </head>
-  <body>
-    <h1>Page with Broken Targets</h1>
-    <p>This page contains links to valid pages but with invalid hash targets.</p>
-    <nav>
-      <ul>
-        <li><a href="/">Home (valid)</a></li>
-        <li><a href="/with-anchors.html#section1">Valid anchor</a></li>
-        <li><a href="/with-anchors.html#nonexistent">Non-existent anchor</a></li>
-        <li><a href="/valid.html#missing-target">Valid page, missing target</a></li>
-        <li><a href="/with-anchors.html#section2">Another valid anchor</a></li>
-        <li><a href="/with-anchors.html#also-missing">Also missing</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/example.md

@@ -1,20 +0,0 @@
-# Example Markdown File
-
-This is a markdown file with links:
-
-- [Link to valid page](/valid.html)
-- [Link to page with anchors](/with-anchors.html#section1)
-- [Broken markdown link](/broken-from-markdown.html)
-- [Relative link to valid page](valid.html)
-- [Relative link with dot-slash](./with-anchors.html)
-- [Relative broken link](broken-relative.html)
-
-## Markdown Section
-
-This section has an anchor target (`#markdown-section`) generated by rehype-slug.
-
-This is a code block with a link that should NOT be extracted:
-
-```html
-<a href="/this-should-not-be-checked.html">Link in code block</a>
-```

package/src/brokenLinksChecker/__fixtures__/static-site/external-links.html

@@ -1,21 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with External Links</title>
-  </head>
-  <body>
-    <h1>Page with External Links</h1>
-    <p>This page contains external links that should be ignored.</p>
-    <nav>
-      <ul>
-        <li><a href="/">Home (valid internal)</a></li>
-        <li><a href="https://example.com">External link 1</a></li>
-        <li><a href="https://github.com/mui">External link 2</a></li>
-        <li><a href="http://localhost:9999/fake">External localhost</a></li>
-        <li><a href="/valid.html">Valid internal link</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/ignored-page.html

@@ -1,17 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Ignored Page</title>
-  </head>
-  <body>
-    <h1>Ignored Page</h1>
-    <p>This page should be ignored by the crawler.</p>
-    <nav>
-      <ul>
-        <li><a href="/this-link-should-not-be-checked.html">Should not be checked</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/index.html

@@ -1,29 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Test Site Home</title>
-  </head>
-  <body>
-    <h1>Test Site Home</h1>
-    <nav>
-      <ul>
-        <li><a href="/valid.html">Valid Page</a></li>
-        <li><a href="/with-anchors.html">Page with Anchors</a></li>
-        <li><a href="/broken-links.html">Page with Broken Links</a></li>
-        <li><a href="/broken-targets.html">Page with Broken Targets</a></li>
-        <li><a href="/external-links.html">Page with External Links</a></li>
-        <li><a href="/nested/page.html">Nested Page</a></li>
-        <li><a href="/ignored-page.html">Ignored Page</a></li>
-        <li><a href="/page-with-ignored-content.html">Page with Ignored Content</a></li>
-        <li><a href="/page-with-custom-targets.html">Page with Custom Targets</a></li>
-        <li><a href="/page-with-known-target-links.html">Page with Known Target Links</a></li>
-        <li><a href="/page-with-api-links.html">Page with API Links</a></li>
-        <li><a href="/example.md">Example Markdown</a></li>
-        <li><a href="/unclosed-tags.html">Page with Unclosed Tags</a></li>
-        <li><a href="/invalid-html.html">Invalid HTML Page</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/invalid-html.html

@@ -1,15 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <title>Invalid HTML Page</title>
-  </head>
-  <body>
-    <h1>Invalid HTML</h1>
-    <!-- Duplicate ID (violates no-duplicate-id rule) -->
-    <div id="dup">First</div>
-    <div id="dup">Second</div>
-    <!-- Raw ampersand (violates no-raw-characters rule) -->
-    <p>Tom & Jerry</p>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/known-targets.json

@@ -1,5 +0,0 @@
-{
-  "targets": {
-    "/api-page.html": ["#method1", "#method2", "#method3"]
-  }
-}

package/src/brokenLinksChecker/__fixtures__/static-site/nested/page.html

@@ -1,21 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Nested Page</title>
-  </head>
-  <body>
-    <h1>Nested Page</h1>
-    <p>This is a page in a nested directory.</p>
-    <nav>
-      <ul>
-        <li><a href="/">Home</a></li>
-        <li><a href="/valid.html">Valid page</a></li>
-        <li><a href="/with-anchors.html#section1">Link to anchor</a></li>
-        <li><a href="../valid.html">Relative link to valid page</a></li>
-        <li><a href="../broken-relative-html.html">Relative broken link from HTML</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/orphaned-page.html

@@ -1,20 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Orphaned Page</title>
-  </head>
-  <body>
-    <h1>Orphaned Page</h1>
-    <p>This page is not linked from anywhere and can only be discovered via seedUrls.</p>
-    <nav>
-      <ul>
-        <li><a href="/valid.html">Valid Page</a></li>
-        <li><a href="/valid.html/">Valid Page with trailing slash</a></li>
-        <li><a href="/orphaned-broken-link.html">Broken link from orphaned page</a></li>
-        <li><a href="/example.md">Markdown file with HTML snippet</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/page-with-api-links.html

@@ -1,20 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with API Links</title>
-  </head>
-  <body>
-    <h1>Page with API Links</h1>
-    <p>This page links to API documentation with known targets.</p>
-    <nav>
-      <ul>
-        <li><a href="/">Home</a></li>
-        <li><a href="/api-page.html#method1">API Method 1 (known)</a></li>
-        <li><a href="/api-page.html#method2">API Method 2 (known)</a></li>
-        <li><a href="/api-page.html#unknown-method">Unknown API method</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/page-with-custom-targets.html

@@ -1,24 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with Custom Targets</title>
-  </head>
-  <body>
-    <h1>Page with Custom Targets</h1>
-    <nav>
-      <ul>
-        <li><a href="/">Home</a></li>
-        <li><a href="#custom-id">Link to custom ID</a></li>
-        <li><a href="#__should-be-ignored">Link to ignored ID</a></li>
-      </ul>
-    </nav>
-    <section id="custom-id">
-      <h2>Custom ID Section</h2>
-    </section>
-    <section id="__should-be-ignored">
-      <h2>This target should be ignored</h2>
-    </section>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/page-with-ignored-content.html

@@ -1,28 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with Ignored Content</title>
-  </head>
-  <body>
-    <h1>Page with Ignored Content</h1>
-    <nav>
-      <ul>
-        <li><a href="/">Home</a></li>
-      </ul>
-    </nav>
-    <aside class="sidebar">
-      <h2>Sidebar (should be ignored)</h2>
-      <ul>
-        <li><a href="/sidebar-broken-link.html">Broken link in sidebar</a></li>
-      </ul>
-    </aside>
-    <main>
-      <p>Main content</p>
-      <p>
-        <a href="/broken-link-with-sidebar-class.html" class="sidebar">Link with sidebar class directly</a>
-      </p>
-    </main>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/page-with-known-target-links.html

@@ -1,19 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with Known Target Links</title>
-  </head>
-  <body>
-    <h1>Page with Known Target Links</h1>
-    <p>This page links to external pages with known targets.</p>
-    <nav>
-      <ul>
-        <li><a href="/">Home</a></li>
-        <li><a href="/external-page.html#valid-target">Valid external target</a></li>
-        <li><a href="/external-page.html#invalid-target">Invalid external target</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/unclosed-tags.html

@@ -1 +0,0 @@
-<!DOCTYPE html><html lang="en"><head><meta charset="UTF-8"><title>Unclosed Tags</title></head><body><div class="wrapper"><main class="content"><a href="/broken-inside-unclosed-main.html">Broken link inside unclosed main</a><a href="/valid.html">Valid link inside unclosed main</a></main></div></body></html>

package/src/brokenLinksChecker/__fixtures__/static-site/valid.html

@@ -1,20 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Valid Page</title>
-  </head>
-  <body>
-    <h1>Valid Page</h1>
-    <p>This page has only valid internal links.</p>
-    <nav>
-      <ul>
-        <li><a href="/">Home</a></li>
-        <li><a href="/with-anchors.html#section1">Link to section 1</a></li>
-        <li><a href="/with-anchors.html#section2">Link to section 2</a></li>
-        <li><a href="/nested/page.html">Nested page</a></li>
-      </ul>
-    </nav>
-  </body>
-</html>

package/src/brokenLinksChecker/__fixtures__/static-site/with-anchors.html

@@ -1,31 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>Page with Anchors</title>
-  </head>
-  <body>
-    <h1>Page with Anchors</h1>
-    <nav>
-      <ul>
-        <li><a href="/">Home</a></li>
-        <li><a href="#section1">Jump to section 1</a></li>
-        <li><a href="#section2">Jump to section 2</a></li>
-        <li><a href="#section3">Jump to section 3</a></li>
-      </ul>
-    </nav>
-    <section id="section1">
-      <h2>Section 1</h2>
-      <p>Content for section 1</p>
-    </section>
-    <section id="section2">
-      <h2>Section 2</h2>
-      <p>Content for section 2</p>
-    </section>
-    <section id="section3">
-      <h2>Section 3</h2>
-      <p>Content for section 3</p>
-    </section>
-  </body>
-</html>