Refactor: Decompose project.py into smaller modules #4162

New issue

Closed

opened 2026-04-06 12:04:42 +00:00 by freemo · 1 comment

freemo commented

2026-04-06 12:04:42 +00:00

Owner

The src/cleveragents/domain/models/core/project.py file has been identified as a violation of the Single Responsibility Principle and the 500-line limit guideline.

Issues:

Mixed Concerns: The file mixes several distinct concerns:
- Data Models: ParsedName, ContextConfig, LinkedResource, and NamespacedProject.
- Enum: TemporalScope.
- Business Logic: The parse_namespaced_name function contains business logic for parsing namespaced names.
File Length: The file is over 550 lines long, which makes it difficult to read and maintain.

Recommendation:

Refactor the project.py file by splitting it into smaller, more focused modules:

Keep the data models in src/cleveragents/domain/models/core/project.py.
Move the TemporalScope enum to src/cleveragents/domain/models/core/enums.py.
Move the parse_namespaced_name function to the src/cleveragents/application layer, possibly in a new projects package or a utils module.

This refactoring will improve the modularity, maintainability, and testability of the project-related functionality, and it will also resolve the file length issue.

Automated by CleverAgents Bot
Supervisor: Architecture Guard | Agent: ca-architecture-guard

The `src/cleveragents/domain/models/core/project.py` file has been identified as a violation of the Single Responsibility Principle and the 500-line limit guideline. **Issues:** 1. **Mixed Concerns:** The file mixes several distinct concerns: * **Data Models:** `ParsedName`, `ContextConfig`, `LinkedResource`, and `NamespacedProject`. * **Enum:** `TemporalScope`. * **Business Logic:** The `parse_namespaced_name` function contains business logic for parsing namespaced names. 2. **File Length:** The file is over 550 lines long, which makes it difficult to read and maintain. **Recommendation:** Refactor the `project.py` file by splitting it into smaller, more focused modules: * Keep the data models in `src/cleveragents/domain/models/core/project.py`. * Move the `TemporalScope` enum to `src/cleveragents/domain/models/core/enums.py`. * Move the `parse_namespaced_name` function to the `src/cleveragents/application` layer, possibly in a new `projects` package or a `utils` module. This refactoring will improve the modularity, maintainability, and testability of the project-related functionality, and it will also resolve the file length issue. --- **Automated by CleverAgents Bot** Supervisor: Architecture Guard | Agent: ca-architecture-guard

freemo referenced this issue

2026-04-06 12:08:56 +00:00

[Automated] CleverAgents Build Session - 2026-04-06 #3775

freemo added the

Points

label

2026-04-06 12:13:02 +00:00

freemo referenced this issue

2026-04-06 16:19:03 +00:00

Epic: Code Architecture Cleanup - Single Responsibility Principle Violations #4179

freemo commented

2026-04-06 17:37:51 +00:00

Author

Owner

DUPLICATE: Consolidated into Epic #4179 "Code Architecture Cleanup - Single Responsibility Principle Violations"

This refactoring task for decomposing project.py into smaller modules is part of the broader architectural cleanup effort. All Single Responsibility Principle violation fixes are being tracked collectively in the epic.

Action: Closing as duplicate in favor of Epic #4179

**DUPLICATE**: Consolidated into Epic #4179 "Code Architecture Cleanup - Single Responsibility Principle Violations" This refactoring task for decomposing `project.py` into smaller modules is part of the broader architectural cleanup effort. All Single Responsibility Principle violation fixes are being tracked collectively in the epic. **Action**: Closing as duplicate in favor of Epic #4179

freemo closed this issue

2026-04-06 17:37:59 +00:00