Problem
Validated from Claude third-pass finding [84].
cli/python/base_projects/engine.py is still 1,076 lines and still carries # pylint: disable=too-many-lines near the top, even after the command_helpers.py extraction. The file remains a broad dispatcher plus workspace status/check/doctor/clone/pull/init/configure implementation surface.
This is low-risk technical debt, but the suppression makes it easy for new workspace behavior to keep accumulating in one module.
Desired outcome
Remove the need for the too-many-lines suppression by continuing the existing module split along established workspace command boundaries.
Scope
- Identify the next cohesive extraction from
base_projects.engine.
- Prefer existing boundaries such as workspace init, clone, pull, reports, or command parsing.
- Avoid behavior changes unless covered by focused tests.
- Remove or reduce the
too-many-lines suppression when the split is complete.
Acceptance criteria
base_projects.engine is smaller and no longer needs a broad too-many-lines suppression, or the remaining suppression is justified by a narrower follow-up issue.
- Existing
base_projects tests continue to pass.
- Extracted helpers follow the patterns already used by
command_helpers.py and existing workspace modules.
Classification
Review finding taxonomy: refactoring/code organization.
Problem
Validated from Claude third-pass finding
[84].cli/python/base_projects/engine.pyis still 1,076 lines and still carries# pylint: disable=too-many-linesnear the top, even after thecommand_helpers.pyextraction. The file remains a broad dispatcher plus workspace status/check/doctor/clone/pull/init/configure implementation surface.This is low-risk technical debt, but the suppression makes it easy for new workspace behavior to keep accumulating in one module.
Desired outcome
Remove the need for the
too-many-linessuppression by continuing the existing module split along established workspace command boundaries.Scope
base_projects.engine.too-many-linessuppression when the split is complete.Acceptance criteria
base_projects.engineis smaller and no longer needs a broadtoo-many-linessuppression, or the remaining suppression is justified by a narrower follow-up issue.base_projectstests continue to pass.command_helpers.pyand existing workspace modules.Classification
Review finding taxonomy: refactoring/code organization.