docs(ai-team): Record .slnx-only directive#11
Conversation
- Added MongoDB.Bson 3.5.2 to Directory.Packages.props for centralized version management - Added MongoDB.Bson PackageReference to src/Shared/Shared.csproj to resolve missing MongoDB.Bson type references Fixes compilation errors: CS0246 - MongoDB namespace and ObjectId type not found Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR documents a team directive to use the .slnx solution file format exclusively (no legacy .sln conversions) and adds MongoDB.Bson package support to the Shared project. The directive establishes that all workflows, documentation, and tooling must reference IssueManager.slnx going forward. Additionally, the solution file receives a standard XML declaration, and MongoDB.Bson v3.5.2 is added to support ObjectId types used in DTOs.
Changes:
- Added team decision document mandating .slnx-only usage across all processes
- Added MongoDB.Bson package reference to Shared project (centrally versioned at 3.5.2)
- Added XML declaration to IssueManager.slnx file
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .ai-team/decisions/inbox/copilot-directive-slnx-only.md | Documents team directive requiring .slnx format exclusively, with implementation guidelines for workflows and tooling |
| Directory.Packages.props | Adds MongoDB.Bson v3.5.2 to centralized package version management |
| src/Shared/Shared.csproj | Adds MongoDB.Bson package reference (used by ObjectId types in DTOs) |
| IssueManager.slnx | Adds standard XML 1.0 UTF-8 declaration header |
| **Why:** User preference—ensures consistency with .NET 10.0 standard and prevents tooling confusion. | ||
|
|
||
| **Implementation:** | ||
| - All build workflows target `IssueManager.slnx` (not any `.sln` equivalent) |
There was a problem hiding this comment.
The directive states "All build workflows target IssueManager.slnx (not any .sln equivalent)", but the actual implementation is incomplete. The file .github/workflows/test.yml still references IssueManager.sln in 7 places (lines 62, 91, 145, 198, 265, 319, and 373). These references need to be updated to IssueManager.slnx to align with this directive.
While this file is not part of the current diff, the directive being added explicitly promises that this change has been made, creating a discrepancy between the documented decision and the actual codebase state.
Records team decision to use .slnx format exclusively with no conversions to legacy .sln format