docs: refresh docs for the shipped NERV theme, icon, and release pipeline - #12
Merged
Conversation
…line Post-ship documentation sweep — the README and website were already accurate (theme, schemes, git+python3 prereqs, license); these were the remaining gaps: - CLAUDE.md: note src/lib/theme/ (the Classic↔NERV theme system) and the website/ marketing site in the architecture map, and add the NERV nerv.css `:global()` footgun to the gotchas. - DESIGN.md: the theming work shipped — drop the future tense, mark the "open decisions" resolved (lane palettes, NERV Shiki theme, Classic light/dark), and record that the app icon shipped (favicon is still stock SvelteKit). - README.md: point the Distribution section at docs/RELEASING.md (the CI/CD release pipeline + in-app updater), which it never mentioned. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ment The comment said the Homebrew PATH prepend exists so child processes can find git-filter-repo — but it's bundled now (run via python3, not resolved on PATH). The real remaining reason is Homebrew-installed tools like `gh` (the GitHub screen). Corrected the rationale; behavior unchanged. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Post-ship documentation sweep (from a read-only review of README, website, DESIGN.md, CLAUDE.md, docs/). The README and website were found accurate — theme/schemes,
git+python3prerequisites (no Homebrew), and the CC BY-NC-SA license ("not open source") are all current. These were the remaining gaps:src/lib/theme/(the Classic↔NERV theme system) andwebsite/; added the NERVnerv.css:global()footgun to the gotchas (it silently drops selectors — bit the build once).SettingsPanel.svelte, the "open decisions" are marked resolved (per-scheme lane palettes ingraph/colors.ts, NERV Shiki theme indiff/nervShikiTheme.ts, Classic light/dark independence), and the app icon is recorded as shipped (favicon is still stock SvelteKit).docs/RELEASING.mdfor the signed+notarized CI/CD pipeline and the in-app updater.Verified every code reference cited above exists. Not included (need the running app or a separate call): recapturing the stale
docs/screenshots/settings.png(it predates the tabbed Settings + Appearance tab) and a possible NERV-themed gallery screenshot.🤖 Generated with Claude Code