Skip to content

Commit c349236

Browse files
committed
feat: add collapsable list for skipped checks
1 parent 7107afd commit c349236

4 files changed

Lines changed: 22 additions & 20 deletions

File tree

.github/workflows/release.yml

Lines changed: 3 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
# Triggered by: python scripts/release.py --push
2-
# (tags vX.Y.Z and pushes, which triggers this workflow)
32

43
name: Release
54

@@ -31,13 +30,11 @@ jobs:
3130
- name: Check dist is up to date
3231
run: git diff --exit-code -- dist/ ':!dist/**/*.d.ts.map'
3332

34-
# --- Artifact signing (dogfood: sign dist/index.js using auths-dev/sign@v1) ---
35-
- name: Sign and verify dist/index.js
33+
# Dogfood: sign dist/index.js using auths-dev/sign@v1 (ephemeral, no secrets)
34+
- name: Sign dist/index.js
3635
uses: auths-dev/sign@v1
3736
with:
38-
token: ${{ secrets.AUTHS_CI_TOKEN }}
3937
files: 'dist/index.js'
40-
verify: true
4138
note: 'GitHub Actions release — ${{ github.ref_name }}'
4239

4340
- name: Generate SHA256 checksums
@@ -60,22 +57,12 @@ jobs:
6057
body: |
6158
## Auths Verify GitHub Action
6259
63-
Verify commit signatures and artifact attestations in your CI pipeline using [Auths](https://github.com/auths-dev/auths) identity keys.
60+
Verify commit signatures and artifact attestations in CI.
6461
6562
### Usage
6663
6764
```yaml
6865
- uses: auths-dev/verify@v1
69-
with:
70-
token: '.auths/allowed_signers'
71-
```
72-
73-
**New: Artifact verification**
74-
```yaml
75-
- uses: auths-dev/verify@v1
76-
with:
77-
token: $\{{ secrets.AUTHS_CI_TOKEN }}
78-
files: 'dist/*.tar.gz'
7966
```
8067
8168
See the [README](https://github.com/auths-dev/verify#readme) for full configuration options.

.github/workflows/sign-commits.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,5 +21,4 @@ jobs:
2121

2222
- uses: auths-dev/sign@v1
2323
with:
24-
token: ${{ secrets.AUTHS_CI_TOKEN }}
2524
commits: 'HEAD~1..HEAD'

action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ description: 'Verify cryptographic commit and artifact signatures on every PR us
33
author: 'auths-dev'
44

55
inputs:
6-
token:
7-
description: 'Identity for verification. Accepts: AUTHS_CI_TOKEN JSON, identity bundle JSON, file path to bundle, or path to allowed_signers file. Default: .auths/allowed_signers'
6+
identity:
7+
description: 'Identity for verification. Accepts: identity bundle JSON, file path to bundle, or path to allowed_signers file. Default: .auths/allowed_signers'
88
required: false
99
default: ''
1010
commits:

src/main.ts

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -356,11 +356,13 @@ export function buildSummaryMarkdown(
356356
lines.push('| Commit | Status | Details |');
357357
lines.push('|--------|--------|---------|');
358358

359+
const skippedRows: string[] = [];
360+
359361
for (const result of results) {
360362
const shortSha = `\`${result.commit.substring(0, 8)}\``;
361363

362364
if (result.skipped) {
363-
lines.push(`| ${shortSha} | Skipped | ${result.skipReason || 'N/A'} |`);
365+
skippedRows.push(`| ${shortSha} | Skipped | ${result.skipReason || 'N/A'} |`);
364366
} else if (result.valid) {
365367
const signer = result.signer || 'verified';
366368
lines.push(`| ${shortSha} | \u2705 Verified | Signed by ${signer} |`);
@@ -370,6 +372,20 @@ export function buildSummaryMarkdown(
370372
}
371373
}
372374

375+
// Collapse skipped commits if there are more than 3
376+
if (skippedRows.length > 0 && skippedRows.length <= 3) {
377+
lines.push(...skippedRows);
378+
} else if (skippedRows.length > 3) {
379+
lines.push('');
380+
lines.push(`<details><summary>${skippedRows.length} skipped commits (merge commits)</summary>`);
381+
lines.push('');
382+
lines.push('| Commit | Status | Details |');
383+
lines.push('|--------|--------|---------|');
384+
lines.push(...skippedRows);
385+
lines.push('');
386+
lines.push('</details>');
387+
}
388+
373389
lines.push('');
374390
const resultEmoji = failed === 0 ? '\u2705' : '\u274c';
375391
let resultLine = `**Result:** ${resultEmoji} ${passed}/${total} commits verified`;

0 commit comments

Comments
 (0)