Skip to content

Comments

refactor: using DI structure#145

Open
langningchen wants to merge 180 commits intomainfrom
refactor/di
Open

refactor: using DI structure#145
langningchen wants to merge 180 commits intomainfrom
refactor/di

Conversation

@langningchen
Copy link
Owner

This pull request introduces Dependency Injection (DI) scaffolding using tsyringe, adds new domain-aligned port interfaces.

@langningchen langningchen self-assigned this Dec 17, 2025
@langningchen langningchen added the enhancement New feature or request label Dec 17, 2025
Copilot AI review requested due to automatic review settings December 17, 2025 22:38
@github-project-automation github-project-automation bot moved this to Todo in CPH-NG Dec 17, 2025

This comment was marked as outdated.

…rategies

- Added TempStorageAdapter for managing temporary storage paths.
- Introduced CompilerAdapter for compiling problems.
- Created JudgeCoordinator to handle judging logic for execution results.
- Developed RunStrategyFactoryAdapter to create different run strategies.
- Implemented CheckerRunnerAdapter for running checkers on outputs.
- Added RunnerProviderAdapter to manage runner binary compilation and retrieval.
- Created SolutionRunnerAdapter for executing solutions with different strategies.
- Developed ExternalRunnerStrategy for executing solutions externally.
- Implemented NormalStrategy and WrapperStrategy for different execution methods.
- Added IRunStrategy interface for defining run strategies.
- Created LoggerAdapter for logging through VSCode's output channel.
- Implemented TelemetryAdapter for sending telemetry events.
- Developed TranslatorAdapter for localization support.
Copilot AI review requested due to automatic review settings December 21, 2025 09:13

This comment was marked as outdated.

@langningchen langningchen moved this from Todo to In Progress in CPH-NG Dec 21, 2025
Copilot AI review requested due to automatic review settings December 22, 2025 12:15

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings December 23, 2025 01:06

This comment was marked as outdated.

@langningchen langningchen marked this pull request as ready for review February 21, 2026 02:49
Copilot AI review requested due to automatic review settings February 21, 2026 02:49

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings February 22, 2026 02:23

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings February 22, 2026 02:46

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants