Import changes from previous project iterations. #2
Conversation
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
* Imported the primary domain interfaces from previous project iterations. * Resolved issue where ILayer did not condition TDomain to extend from IDomain. * Added a latest build shield to the README.md * Imported the .editorconfig from previous project iterations.
* Imported the primary layer interfaces from previous project iterations.
* Imported the primary connector interfaces from previous project iterations.
* Imported the primary interaction interfaces from previous project iterations.
* Imported the core debugging classes from previous project iterations.
* Major modifications were made to CatalystDebug.cs to extract and unify certain sections of code, including but not limited to:
* CatalystDebugConfiguration
* CatalystDebugOptions
* Imported the profiling project from previous project iterations. * No additional changes were made.
* Imported the collections project from previous project iterations. * No additional changes were made other than bumping the version to beta.2.
* Imported the threading and supporting projects from previous project iterations, including:
* Core/CatalystUI.Threading
* Core/CatalystUI.Attributes
* Tooling/CatalystUI.Analyzers
* Tooling/CatalystUI.CodeFix
* Bumped all versions to beta.2.
* Updated all project authors from "FireController#1847" to "CatalystUI LLC"
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: FireController#1847 <usfirepilot123@gmail.com>
* Temporarily remove dependency on Catalyst.Debug for the Tooling/CatalystUI.Profiling project, since that has not yet been re-added.
Signed-off-by: FireController#1847 <usfirepilot123@gmail.com>
* Imported the builder pattern and associated classes from previous project iterations. * Imported the model registry and native and associated classes from previous project iterations. * Fixed incorrect import in Core/CatalystUI.Core * Added debugging statements to the model registry.
* Imported the debug project from previous project iterations. * Minor changes were made across the board to bring consistency with the rest of the changes made. * Bumped Catalyst.Debug to 1.0.0-beta.2 * Un-commented the Catalyst.Debug dependency in the profiling project. *
* Imported the supplementary project from previous project iterations.
* Imported the mathematics project from previous project iterations. * No additional changes were made.
* Added the `IInputData` interface to Catalyst.Core to represent interaction data.
* Also added an associated property for `IInteraction`.
* Added various supplementary default interactions and their associated data.
* Added device-specific supplementary default interactions for Keyboards and Mice.
* Added the Arcane module to the solution. * Added various new solution folders. * Added the INI Arcane module implementation from previous project iterations.
FireController1847
requested review from
ItzNxthaniel,
Copilot and
thecakeisalie05
FireController1847
added
ACT: Review
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR imports several core components and modules from previous project iterations, primarily adding new functionality for debugging, mathematics, supplementary utilities, and Arcane INI configuration file handling. The changes represent a significant expansion of the framework's capabilities while maintaining consistency with existing CatalystUI architecture.
Key Changes:
- Added five new core projects: Debug, Supplementary, Mathematics, Arcane.Core, and Arcane.Ini
- Introduced keyboard and mouse input handling infrastructure
- Implemented file data connector patterns and INI file support
- Enhanced debugging capabilities with Serilog integration
Reviewed Changes
Copilot reviewed 83 out of 83 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| CatalystUI.sln | Added new projects and module folder structure |
| CatalystUI.Core.csproj | Updated package version and added Mathematics dependency |
| CatalystUI.Profiling.csproj | Uncommented debug package reference and updated version |
| CatalystUI.Debug/* | New debugging infrastructure with Serilog integration |
| CatalystUI.Mathematics/* | New vector and geometry types for mathematical operations |
| CatalystUI.Supplementary/* | New input handling, file connectors, and system detection |
| CatalystUI.Modules.Arcane.Core/* | New core types for file-based module system |
| CatalystUI.Modules.Arcane.Ini/* | New INI configuration file handling implementation |
| .scripts/Setup.ps1 | Updated project build configuration |
| .github/workflows/dotnet-manual.yml | Added manual build workflow |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
FireController1847
force-pushed
the
experimental
branch
from
f39aae9 to
bc0e831
Compare
FireController1847
force-pushed
the
experimental
branch
from
bc0e831 to
d8ce1e2
Compare
Member
Author
|
Closing this because something funky is going on with the Git history. |
FireController1847
added a commit
that referenced
this pull request
Nov 3, 2025
* Add builder pattern classes
* Imported the builder pattern and associated classes from previous project iterations.
* Imported the model registry and native and associated classes from previous project iterations.
* Fixed incorrect import in Core/CatalystUI.Core
* Added debugging statements to the model registry.
* Add Core/CatalystUI.Debug
* Imported the debug project from previous project iterations.
* Minor changes were made across the board to bring consistency with the rest of the changes made.
* Bumped Catalyst.Debug to 1.0.0-beta.2
* Un-commented the Catalyst.Debug dependency in the profiling project.
*
* Add Core/CatalystUI.Supplementary
* Imported the supplementary project from previous project iterations.
* Add Core/CatalystUI.Mathematics
* Imported the mathematics project from previous project iterations.
* No additional changes were made.
* Add interactions supplementary
* Added the `IInputData` interface to Catalyst.Core to represent interaction data.
* Also added an associated property for `IInteraction`.
* Added various supplementary default interactions and their associated data.
* Added device-specific supplementary default interactions for Keyboards and Mice.
* Add the Arcane project
* Added the Arcane module to the solution.
* Added various new solution folders.
* Added the INI Arcane module implementation from previous project iterations.
* Add manual .NET action trigger
* Update CatalystUI/Modules/Arcane/CatalystUI.Modules.Arcane.Ini/Semantics/IniSemantic.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: FireController#1847 <usfirepilot123@gmail.com>
* Resolve comments on PR by @thecakeisalie05
* Resolve comments on PR by @ItzNxthaniel
---------
Signed-off-by: FireController#1847 <usfirepilot123@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
📑 Summary
Additional process for importing previous project iterations with minor changes made across the board for unification.
⚒️ Changes
Caution
Accurately describing the changes made in this PR will greatly improve the speed at which we can review your PR. Semantic Versioning can be helpful in determining the scope of your change, and will help in determining the appropriate version number, merge location, and timing of the integration of your change.
➕ Additions
Added all of the following projects from previous project iterations:
Additionally, changes from about four months ago in the original CatalystUI
developmentproject have now been migrated into the latest version. Basic interactions for Keyboard and Mice were added to theCatalystUI.Supplementaryproject.✖️ Modifications
All changes are noted in the individual commits. Most changes were minor. The most extensive changes made were to the
CatalystUI.Supplementaryproject.➖ Removals
💬 Review
Important
Accurately listing key items for review will help us review your PR more quickly. Please include large changes, breaking changes, and any other changes that may require special attention.
Reviewers will check off items in the list as they are reviewed, in addition to the normal GitHub review process. They may request changes if any items are not adequately addressed or add additional items as necessary.