Skip to content

[codex] Fix post-merge health checks - #112

Merged
blove merged 1 commit into
mainfrom
codex/fix-post-merge-health-ci
May 8, 2026
Merged

[codex] Fix post-merge health checks#112
blove merged 1 commit into
mainfrom
codex/fix-post-merge-health-ci

Conversation

@blove

@blove blove commented May 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • install npm 11 into a temporary prefix for the release workflow instead of mutating the runner's active npm installation
  • move OpenSSF Scorecard write scopes to job-level permissions while keeping workflow-global permissions read-only
  • scope the drawer smoke test to the exact in-drawer /docs link to avoid WebKit pointer interception

Verification

  • ruby -e 'require "yaml"; ARGV.each { |file| YAML.load_file(file); puts file }' .github/workflows/release.yml .github/workflows/scorecard.yml .github/workflows/ci.yml\n- temp-prefix npm install --global npm@11.14.1 returned 11.14.1\n- pnpm --filter @pretable/app-website exec playwright test e2e/smoke.spec.ts --project=webkit -g "docs brand link returns" --repeat-each=5\n- pnpm --filter @pretable/app-website exec playwright test e2e/smoke.spec.ts e2e/docs.spec.ts\n- pnpm format\n- pnpm lint\n- pnpm typecheck

@vercel

vercel Bot commented May 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
pretable Ignored Ignored May 8, 2026 7:11pm

Request Review

@blove
blove enabled auto-merge (squash) May 8, 2026 19:11
@blove
blove merged commit c26d2a6 into main May 8, 2026
13 checks passed
@blove
blove deleted the codex/fix-post-merge-health-ci branch May 8, 2026 19:14
@github-actions

github-actions Bot commented May 8, 2026

Copy link
Copy Markdown
Contributor

Vercel preview ready

Preview: https://pretable-lfort6qtf-cacheplane.vercel.app
Commit: d7387dced8f2b1e0b934cc563597e5d5962c3f3b

Updated automatically by the deploy-preview job.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant