site: adopt weft design system + fix Pages (Actions) config#12
Conversation
Design-system consistency pass (audit-driven): - .eyebrow re-implementation -> shared .weft-eyebrow atom (canonical 11px/medium overline; corrects a 12px/regular drift). - token hygiene: 64px->var(--space-10), 3px->var(--radius-sm), raw rgba(0,0,0,.08) inset -> var(--border-hairline). The D4 display-font fix (Space Grotesk retired -> IBM Plex Sans) lands via the shared @weft/site-kit on the next vendor refresh. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The deploy job used deploy-pages@v4 with no configure-pages step, so it did not self-heal the repo's Pages source — which was still 'Deploy from a branch' (legacy: gh-pages / rc5-docs), causing the Actions deploy to fail or be ignored. Add actions/configure-pages@v5 (enablement: true) to the build job so the source is pinned to build_type=workflow on every run. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c2903fcbee
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # still set to "Deploy from a branch". | ||
| uses: actions/configure-pages@v5 | ||
| with: | ||
| enablement: true |
There was a problem hiding this comment.
Use a PAT/App token for Pages enablement
In the workflow I inspected, this step does not pass token:, so actions/configure-pages@v5 uses its default github.token. The action's own action.yml says enablement requires a token other than GITHUB_TOKEN; when the repo is still set to “Deploy from a branch” or Pages is not enabled, the enable/configure API call is rejected and the build fails before artifact upload instead of self-healing. Either pass a PAT/GitHub App token with the required Pages/admin permissions or leave enablement off and require manual Pages source setup.
Useful? React with 👍 / 👎.
Design-system consistency pass + Pages config fix (see commits). Site/CI only; no src changes.
🤖 Generated with Claude Code