Skip to content

Modernize application structure and add mock server - #30

Merged
sepaseh merged 9 commits into
mainfrom
codex/sync-banking-structure
Aug 28, 2026
Merged

Modernize application structure and add mock server#30
sepaseh merged 9 commits into
mainfrom
codex/sync-banking-structure

Conversation

@sepaseh

@sepaseh sepaseh commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • simplify route access and navigation around pure permission policies instead of permission hooks
  • improve provider boundaries, router error handling, API/form typing, FormData support, and shared form components
  • add regression coverage for application and route-level error fallbacks
  • add a collection-driven mock server with authentication, pagination, saved examples, tests, CI integration, and operational documentation
  • expand architecture, testing, component coverage, quality, collection, deployment, and security documentation
  • update compatible runtime and development dependencies while keeping TypeScript 6, Stryker 9, and Node 24 types to avoid major-version conflicts

Validation

  • npm run audit
  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm run test:coverage (166 passed, 7 skipped)
  • npm run test:contract (2 passed)
  • npm run server:test (6 passed)
  • npm run knip
  • npm run performance
  • npm run build-storybook
  • npm run test:storybook (47 passed)
  • npm run test:e2e -- --project=chromium (19 passed)

Review notes

  • useGoBack and useFilterParams were intentionally left unchanged for separate review.
  • Storybook still emits the known non-failing Ant Design cssVar warning and the intentional ErrorBoundary story error output.

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 810af81f-4007-4ab4-8ed4-30ad0f377157


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Comment thread server/server.cjs Fixed
Comment thread server/server.cjs Fixed
Comment thread server/server.cjs Fixed
@sepaseh

sepaseh commented Aug 26, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@sepaseh

sepaseh commented Aug 26, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai full review

@sepaseh
sepaseh marked this pull request as draft August 28, 2026 05:03
@sonarqubecloud

Copy link
Copy Markdown

@sepaseh
sepaseh marked this pull request as ready for review August 28, 2026 05:49
@sepaseh
sepaseh merged commit 239bef9 into main Aug 28, 2026
7 checks passed
@sepaseh
sepaseh deleted the codex/sync-banking-structure branch August 28, 2026 05:55
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.

2 participants