Skip to content

Consolidate architecture updates, JS utilities, tests, and agent guidance#929

Merged
neozhu merged 3 commits intomainfrom
merge
Apr 24, 2026
Merged

Consolidate architecture updates, JS utilities, tests, and agent guidance#929
neozhu merged 3 commits intomainfrom
merge

Conversation

@neozhu
Copy link
Copy Markdown
Owner

@neozhu neozhu commented Apr 24, 2026

Summary

  • add repository guidance for AI coding agents with a new root AGENT.md
  • update README.md to reflect the current workflow, EF Core migration command, and superpowers-based planning guidance
  • merge broader application and UI updates already present on this branch, including JS utilities, infrastructure wiring, and expanded unit/integration test coverage

What Changed

  • documentation and repository guidance
    • add AGENT.md for AI coding agents
    • add docs/superpowers/specs/2026-04-24-agent-md-design.md
    • refresh README.md to remove OpenSpec, recommend superpowers, and update EF Core migration instructions
  • solution and runtime composition
    • update solution and project wiring, including Server.UI references and Docker-related files
    • adjust application, infrastructure, and UI integration points across the solution
  • application and domain-adjacent code
    • expand constants, interfaces, mappings, exception handling, security, caching, and supporting models
    • refine feature implementations across application modules
  • UI and client assets
    • add or update JS interop utilities, routes, navigation, theme assets, and static files
  • tests
    • add and revise unit, integration, and infrastructure tests to cover the newer runtime and behavior paths

Commits Included

  • dc4412ff Add JS utilities and comprehensive unit/integration tests
  • c2edb565 Add Infrastructure project reference to Server.UI
  • ea672683 Update README.md to reflect architectural changes and add AGENT.md for AI coding agents

Verification

  • not run in this PR creation step

Notes For Reviewers

  • this branch is not limited to documentation; it also includes substantial application, UI, infrastructure, and test changes that were already committed before the README/AGENT updates
  • the diff versus main is large, so review by commit or by area is recommended
  • suggested review order: application/runtime wiring, UI/runtime integration, then documentation

neozhu added 3 commits April 24, 2026 08:14
Added new JavaScript modules for authentication, file download, QR code, passkey (WebAuthn), and theme management. Introduced extensive unit and integration tests for application and infrastructure features (picklist, exception handling, domain events, mapping, notifications, mail service, security analysis, login audit, role import). Added a new test project targeting .NET 10.0 with xUnit and Moq. No changes to existing code files; all additions are new utilities and tests.
Reorganize project references in Server.UI.csproj by adding Infrastructure.csproj and restructuring the ItemGroup for Migrators. Email template resources remain unchanged.
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
0.0% Coverage on New Code (required ≥ 80%)
6.4% Duplication on New Code (required ≤ 3%)
E Security Rating on New Code (required ≥ A)
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Copy link
Copy Markdown
Contributor

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@neozhu neozhu merged commit 3bcf7f8 into main Apr 24, 2026
3 of 5 checks passed
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