From 9b175f860eef3b2c78fad1de5c61edfeb88dd821 Mon Sep 17 00:00:00 2001
From: Tobias Bieniek <tobias@bieniek.cloud>
Date: Sun, 6 Jul 2025 21:07:22 +0200
Subject: [PATCH] trustpub: Show "Trusted Publishing" crate settings for
 everyone

The feedback on our semi-public testing period has been positive, so let's make this a public testing period now.
---
 app/templates/crate/settings/index.hbs | 81 +++++++++++++-------------
 e2e/routes/crate/settings.spec.ts      |  8 +--
 tests/routes/crate/settings-test.js    |  8 +--
 3 files changed, 45 insertions(+), 52 deletions(-)

diff --git a/app/templates/crate/settings/index.hbs b/app/templates/crate/settings/index.hbs
index a495b704fb5..f5321ee8be2 100644
--- a/app/templates/crate/settings/index.hbs
+++ b/app/templates/crate/settings/index.hbs
@@ -54,51 +54,48 @@
   {{/each}}
 </div>
 
-{{! The "Trusted Publishing" section is hidden for now until we make this feature publicly available. }}
-{{#if this.githubConfigs}}
-  <div local-class="header">
-    <h2>Trusted Publishing</h2>
-    <div>
-      <LinkTo @route="docs.trusted-publishing" class="button button--tan button--small" data-test-trusted-publishing-docs-button>
-        Learn more
-      </LinkTo>
-      <LinkTo @route="crate.settings.new-trusted-publisher" class="button button--small" data-test-add-trusted-publisher-button>
-        Add
-      </LinkTo>
-    </div>
+<div local-class="header">
+  <h2>Trusted Publishing</h2>
+  <div>
+    <LinkTo @route="docs.trusted-publishing" class="button button--tan button--small" data-test-trusted-publishing-docs-button>
+      Learn more
+    </LinkTo>
+    <LinkTo @route="crate.settings.new-trusted-publisher" class="button button--small" data-test-add-trusted-publisher-button>
+      Add
+    </LinkTo>
   </div>
+</div>
 
-  <table local-class="trustpub" data-test-trusted-publishing>
-    <thead>
-    <tr>
-      <th>Publisher</th>
-      <th>Details</th>
-      <th></th>
+<table local-class="trustpub" data-test-trusted-publishing>
+  <thead>
+  <tr>
+    <th>Publisher</th>
+    <th>Details</th>
+    <th><span class="sr-only">Actions</span></th>
+  </tr>
+  </thead>
+  <tbody>
+  {{#each this.githubConfigs as |config|}}
+    <tr data-test-github-config={{config.id}}>
+      <td>GitHub</td>
+      <td local-class="details">
+        <strong>Repository:</strong> <a href="https://github.com/{{config.repository_owner}}/{{config.repository_name}}" target="_blank" rel="noopener noreferrer">{{config.repository_owner}}/{{config.repository_name}}</a><br>
+        <strong>Workflow:</strong> <a href="https://github.com/{{config.repository_owner}}/{{config.repository_name}}/blob/HEAD/.github/workflows/{{config.workflow_filename}}" target="_blank" rel="noopener noreferrer">{{config.workflow_filename}}</a><br>
+        {{#if config.environment}}
+          <strong>Environment:</strong> {{config.environment}}
+        {{/if}}
+      </td>
+      <td local-class="actions">
+        <button type="button" class="button button--small" data-test-remove-config-button {{on "click" (perform this.removeConfigTask config)}}>Remove</button>
+      </td>
     </tr>
-    </thead>
-    <tbody>
-    {{#each this.githubConfigs as |config|}}
-      <tr data-test-github-config={{config.id}}>
-        <td>GitHub</td>
-        <td local-class="details">
-          <strong>Repository:</strong> <a href="https://github.com/{{config.repository_owner}}/{{config.repository_name}}" target="_blank" rel="noopener noreferrer">{{config.repository_owner}}/{{config.repository_name}}</a><br>
-          <strong>Workflow:</strong> <a href="https://github.com/{{config.repository_owner}}/{{config.repository_name}}/blob/HEAD/.github/workflows/{{config.workflow_filename}}" target="_blank" rel="noopener noreferrer">{{config.workflow_filename}}</a><br>
-          {{#if config.environment}}
-            <strong>Environment:</strong> {{config.environment}}
-          {{/if}}
-        </td>
-        <td local-class="actions">
-          <button type="button" class="button button--small" data-test-remove-config-button {{on "click" (perform this.removeConfigTask config)}}>Remove</button>
-        </td>
-      </tr>
-    {{else}}
-      <tr data-test-no-config>
-        <td colspan="3">No trusted publishers configured for this crate.</td>
-      </tr>
-    {{/each}}
-    </tbody>
-  </table>
-{{/if}}
+  {{else}}
+    <tr data-test-no-config>
+      <td colspan="3">No trusted publishers configured for this crate.</td>
+    </tr>
+  {{/each}}
+  </tbody>
+</table>
 
 <h2 local-class="header">Danger Zone</h2>
 
diff --git a/e2e/routes/crate/settings.spec.ts b/e2e/routes/crate/settings.spec.ts
index f7b1c3769f5..2eba1136335 100644
--- a/e2e/routes/crate/settings.spec.ts
+++ b/e2e/routes/crate/settings.spec.ts
@@ -51,11 +51,9 @@ test.describe('Route | crate.settings', { tag: '@routes' }, () => {
     await expect(page.locator(`[data-test-owner-user="${user.login}"]`)).toBeVisible();
     await expect(page.locator('[data-test-remove-owner-button]')).toBeVisible();
 
-    // Disabled for now, until we make this feature publicly available
-    // await expect(page.locator('[data-test-trusted-publishing]')).toBeVisible();
-    // await expect(page.locator('[data-test-no-config]')).toBeVisible();
-    // await expect(page.locator('[data-test-github-config]')).not.toBeVisible();
-    await expect(page.locator('[data-test-trusted-publishing]')).not.toBeVisible();
+    await expect(page.locator('[data-test-trusted-publishing]')).toBeVisible();
+    await expect(page.locator('[data-test-no-config]')).toBeVisible();
+    await expect(page.locator('[data-test-github-config]')).not.toBeVisible();
 
     await expect(page.locator('[data-test-delete-button]')).toBeVisible();
   });
diff --git a/tests/routes/crate/settings-test.js b/tests/routes/crate/settings-test.js
index bf428e2006f..c05da7d94ff 100644
--- a/tests/routes/crate/settings-test.js
+++ b/tests/routes/crate/settings-test.js
@@ -55,11 +55,9 @@ module('Route | crate.settings', hooks => {
     assert.dom(`[data-test-owner-user="${user.login}"]`).exists();
     assert.dom('[data-test-remove-owner-button]').exists();
 
-    // Disabled for now, until we make this feature publicly available
-    // assert.dom('[data-test-trusted-publishing]').exists();
-    // assert.dom('[data-test-no-config]').exists();
-    // assert.dom('[data-test-github-config]').doesNotExist();
-    assert.dom('[data-test-trusted-publishing]').doesNotExist();
+    assert.dom('[data-test-trusted-publishing]').exists();
+    assert.dom('[data-test-no-config]').exists();
+    assert.dom('[data-test-github-config]').doesNotExist();
 
     assert.dom('[data-test-delete-button]').exists();
   });