Address TODO comments in the codebase #6131

New issue

Open

opened 2026-04-09 15:19:37 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-09 15:19:37 +00:00

Owner

There are several TODO comments in the codebase that represent technical debt and should be addressed.

List of TODOs:

src/cleveragents/tool/runner.py:295: pass pref.target_resource to the resolver /
src/cleveragents/mcp/adapter.py:611: resource_slots are stored in source_metadata but
src/cleveragents/cli/commands/skill.py:1023: When MCP adapter integration is available, call
src/cleveragents/cli/commands/plan.py:367: derive strategy from plan's actual sandbox configuration
src/cleveragents/cli/commands/plan.py:377: derive strategy from plan's actual sandbox configuration
src/cleveragents/cli/commands/plan.py:1812: Migrate to EstimationResult.as_display_dict() to reduce duplication
src/cleveragents/application/services/uko_loader.py:331: (perf): precompute layer index when scaling demands it.
src/cleveragents/application/services/uko_indexer_internals.py:333: (#578): integrate real embedding model — placeholder vector
src/cleveragents/application/services/service_retry_wiring.py:96: This class conflates factory/config, execution, and caching
src/cleveragents/application/services/plan_lifecycle_service.py:343: Replace EstimationStubActor with real actor dispatch
src/cleveragents/application/services/phase_gating.py:79: (pg-migration): TOCTOU — The phase is read here but the
src/cleveragents/application/services/llm_actors.py:150: (#650): Wire actor-configured response_format into provider calls
src/cleveragents/application/services/llm_actors.py:393: (#650): Wire actor-configured response_format into provider calls
src/cleveragents/application/services/correction_service.py:282: affected_files and artifacts_to_archive use synthetic
src/cleveragents/application/services/cleanup_service.py:311: (CONC3): Wire DB session query when Container is
src/cleveragents/application/services/cleanup_service.py:319: (CONC3): Wire DB session deletion when Container is
src/cleveragents/a2a/facade.py:477: Wire to ACMS ContextAssemblyPipeline once available.

Recommendation:
Create individual issues for each of these TODO comments and address them in a timely manner.

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

There are several `TODO` comments in the codebase that represent technical debt and should be addressed. **List of TODOs**: * `src/cleveragents/tool/runner.py:295`: pass pref.target_resource to the resolver / * `src/cleveragents/mcp/adapter.py:611`: resource_slots are stored in source_metadata but * `src/cleveragents/cli/commands/skill.py:1023`: When MCP adapter integration is available, call * `src/cleveragents/cli/commands/plan.py:367`: derive strategy from plan's actual sandbox configuration * `src/cleveragents/cli/commands/plan.py:377`: derive strategy from plan's actual sandbox configuration * `src/cleveragents/cli/commands/plan.py:1812`: Migrate to EstimationResult.as_display_dict() to reduce duplication * `src/cleveragents/application/services/uko_loader.py:331`: (perf): precompute layer index when scaling demands it. * `src/cleveragents/application/services/uko_indexer_internals.py:333`: (#578): integrate real embedding model — placeholder vector * `src/cleveragents/application/services/service_retry_wiring.py:96`: This class conflates factory/config, execution, and caching * `src/cleveragents/application/services/plan_lifecycle_service.py:343`: Replace EstimationStubActor with real actor dispatch * `src/cleveragents/application/services/phase_gating.py:79`: (pg-migration): TOCTOU — The phase is read here but the * `src/cleveragents/application/services/llm_actors.py:150`: (#650): Wire actor-configured response_format into provider calls * `src/cleveragents/application/services/llm_actors.py:393`: (#650): Wire actor-configured response_format into provider calls * `src/cleveragents/application/services/correction_service.py:282`: affected_files and artifacts_to_archive use synthetic * `src/cleveragents/application/services/cleanup_service.py:311`: (CONC3): Wire DB session query when Container is * `src/cleveragents/application/services/cleanup_service.py:319`: (CONC3): Wire DB session deletion when Container is * `src/cleveragents/a2a/facade.py:477`: Wire to ACMS ContextAssemblyPipeline once available. **Recommendation**: Create individual issues for each of these `TODO` comments and address them in a timely manner. --- **Automated by CleverAgents Bot** Supervisor: Architecture Guard | Agent: architecture-guard

Rows
Columns