Skip to content

Extract settings page navigation config - #78

Merged
ichwars merged 1 commit into
mainfrom
dev-split-settings-page
Aug 4, 2026
Merged

Extract settings page navigation config#78
ichwars merged 1 commit into
mainfrom
dev-split-settings-page

Conversation

@ichwars

@ichwars ichwars commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • move static settings headers, sub-tab metadata, legacy routing maps, and URL resolver helpers into settings-page-config.ts
  • reduce SettingsPage.tsx from 7,618 to 7,074 lines without changing page state or rendered settings content
  • lower the source-size allowlist ceiling to the exact new size; the extracted module is 610 lines and below budget

Local verification

  • npm.cmd exec -- tsc -b
  • npm.cmd run lint
  • SettingsPage and settingsNavigation tests: 116 passed
  • full frontend suite: 244 files / 2,848 tests passed
  • all 11 locales in parity
  • isolated Vite production build: 3,045 modules transformed
  • source size budget: 1,573 files passed
  • source budget unit tests: 14 passed

@ichwars
ichwars merged commit b113e73 into main Aug 4, 2026
30 checks passed
@ichwars
ichwars deleted the dev-split-settings-page branch August 4, 2026 04:16
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