Consolidate Views/ into Pages/ and standardize module UI organization#79
Merged
antosubash merged 4 commits intomainfrom Apr 5, 2026
Merged
Consolidate Views/ into Pages/ and standardize module UI organization#79antosubash merged 4 commits intomainfrom
antosubash merged 4 commits intomainfrom
Conversation
- Merge all Views/ directories into Pages/ across all modules (tsx pages, cs endpoints, and utility files) - Move shared components to module-root components/ directory (Admin, Users) - Update all Pages/index.ts imports to use relative ./paths - Update .editorconfig CA1812 suppression to cover Pages/*Endpoint.cs - Remove duplicate OpenIddict page files - Update CLAUDE.md example to reflect new import pattern
…ndpoint-per-file check - Remove ViewPageAttribute from Core and all 76 endpoint files - Update generator to derive page names from namespace segments between module base namespace and class name (stripping Views/Pages segments) - Update ViewPagesEmitter to generate ./ imports instead of ../Views/ - Change all endpoint namespaces from *.Views.* to *.Pages.* to match file locations - Move OpenIddict client endpoints to Pages/OpenIddict/ subdirectory for correct namespace-based inference of OpenIddict/OpenIddict/* pages - Add SM0049 diagnostic: error when multiple IViewEndpoint classes exist in a single file - Update ViewDiscoveryTests and ViewPagesEmitterTests for new behavior - Add nested namespace test for Users/Account/Login pattern
…use Path.GetFileName
- Add TypeMappingHelpers.ExtractNamespace() to eliminate repeated
LastIndexOf('.') + Substring pattern
- Cache moduleNs per owner in view inference loop to avoid redundant
string manipulation per view
- Use Path.GetFileName in SM0049 diagnostic instead of manual
slash-detection logic
- Update FindClosestModuleName to use the new helper
Deploying simplemodule-website with
|
| Latest commit: |
ee41eaf
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://10096003.simplemodule-website.pages.dev |
| Branch Preview URL: | https://claude-improve-ui-organizati.simplemodule-website.pages.dev |
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
2 participants
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.
cs endpoints, and utility files)