fix: GPG workflow signing, auto-merge, PWA icon fix, and perf optimizations - #70
Merged
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This PR tightens automation and performance for the site: it updates GitHub Actions workflows to better support GPG-signed automation PRs + auto-merge, tweaks frontend boot timing to defer non-critical work, and adjusts PWA/manifest icon references and related build-time asset generation behavior.
Changes:
- Frontend performance: add external-origin preconnects + preload landing hero assets; defer some JS init until after first paint; remove console banner.
- PWA/icon pipeline: update manifest icon filenames and adjust icon generation / Font Awesome subset generation scripts.
- Automation: ensure bot PR commits use imported GPG identity and attempt to enable auto-merge; broaden CI/linter workflow path ignores.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| templates/macros/head.html | Adds preconnect/dns-prefetch and homepage hero image preloads. |
| static/js/core/main.js | Defers non-prepaint JS initialization to post-first-paint. |
| static/icons/site.webmanifest | Switches “any” purpose icons to transparent variants. |
| scripts/generate-icons.js | Changes behavior when ImageMagick is missing during icon generation. |
| scripts/generate-fa-subset.js | Injects font-display: swap into kept @font-face blocks when absent. |
| LLM_RULES.md | Reformats and expands repo conventions (workflows, perf directives, icons/PWA). |
| .github/workflows/sync-data.yml | Uses imported GPG identity for commits and attempts to enable auto-merge for sync PRs. |
| .github/workflows/linter.yml | Expands paths-ignore patterns for PR-triggered lint workflow. |
| .github/workflows/ci.yml | Expands paths-ignore patterns for PR-triggered CI workflow. |
| .github/workflows/bump-versions.yml | Uses imported GPG identity for commits and attempts to enable auto-merge for bump PRs. |
Comment on lines
+84
to
+87
| {% if current_url == config.base_url or current_url == config.base_url ~ "/" %} | ||
| <link rel="preload" as="image" href="{{ get_url(path="images/landing/whood-light.jpg") | safe }}" fetchpriority="high"> | ||
| <link rel="preload" as="image" href="{{ get_url(path="images/landing/whood.jpg") | safe }}" fetchpriority="high"> | ||
| {% endif %} |
| branches: [main] | ||
| paths-ignore: | ||
| - "README.md" | ||
| - "**.md" |
| branches: [main] | ||
| paths-ignore: | ||
| - "README.md" | ||
| - "**.md" |
Comment on lines
214
to
220
| const command = resolveImageMagickCommand(); | ||
| if (!command) { | ||
| throw new Error( | ||
| "ImageMagick is required for icon generation (magick/convert not found).", | ||
| console.log( | ||
| "Icon generation skipped: ImageMagick is not installed (magick/convert not found).", | ||
| ); | ||
| return; | ||
| } |
kascit
force-pushed
the
fix/workflows-perf-pwa
branch
from
August 2, 2026 12:29
06ba633 to
7499dac
Compare
… and optimize performance
kascit
force-pushed
the
fix/workflows-perf-pwa
branch
from
August 2, 2026 12:38
7499dac to
d7e5f30
Compare
This was referenced Aug 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.