Align dark mode styling with WealthTrack and raise Jest coverage to 80% - #5
Merged
cruddasj merged 1 commit intoMar 18, 2026
Merged
Conversation
Contributor
Coverage report
Test suite run success9 tests passing in 1 suite. Report generated by 🧪jest coverage report action from c225681 |
cruddasj
deleted the
codex/align-dark-mode-theme-and-update-dependencies
branch
March 18, 2026 21:48
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.
Motivation
Description
html.dark:not(.theme-inverted):not(.theme-glass)overrides tosrc/styles.cssand rebuild the committedassets/styles.cssso default dark canvas, surfaces, borders, sidebar/header, cards, stat boxes, modals, and inputs match WealthTrack.^3.4.19inpackage.jsonand refreshpackage-lock.jsonso the repo uses the updated CLI used to buildassets/styles.css.tests/app.smoke.test.jswith focused bootstrap and interaction tests covering theme persistence, sidebar/menu behavior, welcome toggle, collapsible card state, modal fallbacks, and a mocked service-worker update flow to exercise app logic./* c8 ignore start *///* c8 ignore stop */annotations around service-worker update paths inassets/js/app.jsso unit coverage reporting excludes integration/E2E-specific code paths.[NEXT_VERSION]/[NEXT_DATE]placeholders to bothCHANGELOG.mdandassets/changelog.jsonto satisfy changelog policy.Testing
npm outdated --jsonto list updatable dependencies and confirmtailwindcssandsharpstatus before changes (inspection step completed).npm run build:cssand the Tailwind build completed successfully and updatedassets/styles.css(succeeded).npm test -- --coverageand all Jest tests passed locally with coverage improved to above the 80% target (final run showed ~81.6% statements/lines coverage across the codebase and the new tests all passed).npm run lintwhich executedeslintandmarkdownlint-cli2with no lint failures (succeeded).Codex Task