Skip to content

Refactor project dependencies and enhance API documentation - #63

Merged
Allann merged 2 commits into
masterfrom
features/llm-understandings
May 13, 2026
Merged

Refactor project dependencies and enhance API documentation#63
Allann merged 2 commits into
masterfrom
features/llm-understandings

Conversation

@Allann

@Allann Allann commented May 12, 2026

Copy link
Copy Markdown
Collaborator

This pull request primarily updates and expands unit tests across several components, with a focus on improving coverage for agent state transitions and decision management. It also updates a project dependency and makes minor test refactorings for clarity and correctness.

Unit test improvements and expansions:

  • Refactored and significantly expanded AgentInfoTests to cover the new AgentInfoIdle, AgentInfoRunning, and AgentInfoFailed types, including status transitions, error handling, and immutability via with-expressions.
  • Added a comprehensive new test suite for DecisionsService, covering creation, listing, retrieval, resolution, error cases, and round-trip parsing of decisions.

Test correctness and modernization:

  • Updated DigestServiceTests to use DateOnly for date parameters and assertions, improving type safety and clarity across multiple test cases. [1] [2] [3] [4] [5] [6] [7]
  • Fixed BoardServiceCompleteTaskTests to use the correct SessionStatus enum instead of a string literal for session status.
  • Updated AgentServiceTests to assert using the strongly-typed AgentInfoFailed and its Failure property, improving test robustness.
  • Modernized LINQ usage in LmStudioExecutorTests by using collection expressions for conciseness.

Dependency updates:

  • Bumped the Aspire.AppHost.Sdk version from 13.2.4 to 13.3.0 in the main project file.

Allann added 2 commits May 11, 2026 01:55
Adds XML docs to all public APIs, refactors AgentInfo to immutable records, introduces AgentFailure/AgentFailover, and adds new value types (SessionStatus, SessionSizeRating, TaskClassification, TestOutcome) with JSON converters and tests. Replaces custom JSON converters with dedicated files. Refactors services for [LoggerMessage] logging, uses collection expressions, and splits WinUI converters. Improves code comments and updates tests/view models for new types. No business logic changes; improves maintainability and API clarity.
@Allann
Allann requested a review from auleewilliams May 12, 2026 13:04
@Allann
Allann merged commit e6901c7 into master May 13, 2026
1 check passed
@Allann
Allann deleted the features/llm-understandings branch May 13, 2026 13:43
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