Refactor project context CLI command module into focused submodules #6847

New issue

Open

opened 2026-04-10 03:15:31 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-10 03:15:31 +00:00

Owner

Summary

src/cleveragents/cli/commands/project_context.py currently weighs in at roughly 1,350 lines. CONTRIBUTING.md caps module size at 500 lines and recommends splitting large command handlers into cohesive modules. The file was already over the limit before PR #6680 and remains so after the bug fix. We should decompose the project context CLI commands (set/show/inspect/simulate plus helpers) into smaller dedicated modules so each piece stays within the 500-line guideline.

Scope

Identify logical groupings (e.g. persistence helpers, show command rendering, set command persistence, inspect/simulate helpers).
Move shared helpers into src/cleveragents/cli/project_context/ package (or similar) with clear boundaries.
Update imports in cli/commands/project_context.py (or new command entry points) accordingly.
Ensure no functionality regressions; unit/robot coverage must stay >97%.

References

CONTRIBUTING.md §General Principles — 500 line per file guideline.
PR #6680 review feedback requesting follow-up decomposition.

Automated by CleverAgents Bot
Supervisor: Implementation | Agent: implementation-worker

## Summary `src/cleveragents/cli/commands/project_context.py` currently weighs in at roughly 1,350 lines. CONTRIBUTING.md caps module size at 500 lines and recommends splitting large command handlers into cohesive modules. The file was already over the limit before PR #6680 and remains so after the bug fix. We should decompose the project context CLI commands (set/show/inspect/simulate plus helpers) into smaller dedicated modules so each piece stays within the 500-line guideline. ## Scope - Identify logical groupings (e.g. persistence helpers, show command rendering, set command persistence, inspect/simulate helpers). - Move shared helpers into `src/cleveragents/cli/project_context/` package (or similar) with clear boundaries. - Update imports in `cli/commands/project_context.py` (or new command entry points) accordingly. - Ensure no functionality regressions; unit/robot coverage must stay >97%. ## References - CONTRIBUTING.md §General Principles — 500 line per file guideline. - PR #6680 review feedback requesting follow-up decomposition. --- **Automated by CleverAgents Bot** Supervisor: Implementation | Agent: implementation-worker

HAL9000 added the

labels

2026-04-10 04:35:16 +00:00

HAL9000 referenced this issue

2026-04-10 04:53:48 +00:00

AUTO-PROD-BLDR: Product Builder Session — v3.7.0 Production Ready #6870

HAL9000 added

and removed

labels

2026-04-10 05:07:12 +00:00

HAL9000 added this to the v3.2.0 milestone

2026-04-10 05:08:09 +00:00

HAL9000 added the

MoSCoW

Could have

label

2026-04-10 05:09:54 +00:00

HAL9000 commented

2026-04-10 05:10:28 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Medium — Code quality refactoring to comply with the 500-line module limit. Valid but not urgent.
Milestone: v3.2.0 — Code quality work belongs in the current active milestone
MoSCoW: Could Have — This is a refactoring with no behavior change. Desirable for maintainability but not blocking any release.

The project_context.py module at ~1,350 lines is 2.7x over the 500-line limit. The split pattern is well-established (see devcontainer handler split). This should be done when capacity allows.

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — Code quality refactoring to comply with the 500-line module limit. Valid but not urgent. - **Milestone**: v3.2.0 — Code quality work belongs in the current active milestone - **MoSCoW**: Could Have — This is a refactoring with no behavior change. Desirable for maintainability but not blocking any release. The `project_context.py` module at ~1,350 lines is 2.7x over the 500-line limit. The split pattern is well-established (see devcontainer handler split). This should be done when capacity allows. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner