Technical Debt: Address TODO/FIXME comments in codebase #3141

Closed
opened 2026-04-05 06:48:47 +00:00 by freemo · 3 comments
Owner

During a recent scan of the codebase, a significant number of TODO and FIXME comments were identified. These comments indicate areas of the code that require further attention, and they represent a backlog of technical debt that should be addressed.

This issue is created to track all of these comments in a single place, so that they can be prioritized and addressed in a structured manner.

List of TODO/FIXME Comments

Core Logic:

  • src/cleveragents/tool/runner.py:177: Pass pref.target_resource to the resolver.
  • src/cleveragents/mcp/adapter.py:596: resource_slots are stored in source_metadata.
  • src/cleveragents/application/services/uko_loader.py:331: Precompute layer index for performance.
  • src/cleveragents/application/services/uko_indexer_internals.py:333: Integrate real embedding model.
  • src/cleveragents/application/services/service_retry_wiring.py:96: 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: TOCTOU issue with phase reading.
  • src/cleveragents/application/services/llm_actors.py:123,325: Wire actor-configured response_format into provider calls.
  • src/cleveragents/application/services/correction_service.py:282: affected_files and artifacts_to_archive use synthetic data.
  • src/cleveragents/application/services/cleanup_service.py:311,319: Wire DB session query and deletion when Container is implemented.
  • src/cleveragents/a2a/facade.py:471: Wire to ACMS ContextAssemblyPipeline.

CLI:

  • src/cleveragents/cli/commands/skill.py:1023: Call MCP adapter integration when available.
  • src/cleveragents/cli/commands/plan.py:1278: Migrate to EstimationResult.as_display_dict().

Testing:

  • robot/helper_wf03_plan_prompt_confidence.py:95: Verify guidance_added.scope, state_transition.
  • robot/helper_wf07_cicd.py:472: automation_profile not yet propagated.
  • robot/e2e/wf12_hierarchical.robot:61,153,175,201,218,433: Various test improvements and implementations.
  • robot/e2e/tdd_acms_behavioral_validation.robot:180: Indexing pipeline wiring.
  • features/wf03_plan_prompt_confidence.feature:17: Verify guidance_added.scope, state_transition.
  • features/steps/tdd_exec_env_resolution_precedence_steps.py:82: 6-level precedence chain is built manually.
  • features/steps/estimation_actor_steps.py:368: Use public save_plan() once test helpers are refactored.
  • features/consolidated_routing.feature:848,856,865,874: Uncomment when step definitions are implemented.
  • features/consolidated_domain_models.feature:67,83,94: Uncomment when step definitions are implemented.

Documentation:

  • docs/reference/lsp_stub.md:151: Wire _dispatch_stub() through self._facade.
  • docs/development/review_playbook.md:45: TODO placement.

Changelog:

  • CHANGELOG.md:1204: TODO comments in runtime code.

Internal Agent Definitions:

  • .opencode/agents/product-builder.md:943: No TODO/FIXME/HACK markers remain in code.
  • .opencode/agents/ca-product-verifier.md:119,121,166: Search for TODO, FIXME, HACK, XXX comments.
  • .opencode/agents/ca-milestone-reviewer.md:166: Incomplete TODO/FIXME/HACK comments.
  • .opencode/agents/ca-architecture-guard.md:143: TODO/FIXME comments.

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

During a recent scan of the codebase, a significant number of `TODO` and `FIXME` comments were identified. These comments indicate areas of the code that require further attention, and they represent a backlog of technical debt that should be addressed. This issue is created to track all of these comments in a single place, so that they can be prioritized and addressed in a structured manner. ### List of TODO/FIXME Comments **Core Logic:** - `src/cleveragents/tool/runner.py:177`: Pass `pref.target_resource` to the resolver. - `src/cleveragents/mcp/adapter.py:596`: `resource_slots` are stored in `source_metadata`. - `src/cleveragents/application/services/uko_loader.py:331`: Precompute layer index for performance. - `src/cleveragents/application/services/uko_indexer_internals.py:333`: Integrate real embedding model. - `src/cleveragents/application/services/service_retry_wiring.py:96`: 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`: TOCTOU issue with phase reading. - `src/cleveragents/application/services/llm_actors.py:123,325`: Wire actor-configured `response_format` into provider calls. - `src/cleveragents/application/services/correction_service.py:282`: `affected_files` and `artifacts_to_archive` use synthetic data. - `src/cleveragents/application/services/cleanup_service.py:311,319`: Wire DB session query and deletion when Container is implemented. - `src/cleveragents/a2a/facade.py:471`: Wire to ACMS `ContextAssemblyPipeline`. **CLI:** - `src/cleveragents/cli/commands/skill.py:1023`: Call MCP adapter integration when available. - `src/cleveragents/cli/commands/plan.py:1278`: Migrate to `EstimationResult.as_display_dict()`. **Testing:** - `robot/helper_wf03_plan_prompt_confidence.py:95`: Verify `guidance_added.scope`, `state_transition`. - `robot/helper_wf07_cicd.py:472`: `automation_profile` not yet propagated. - `robot/e2e/wf12_hierarchical.robot:61,153,175,201,218,433`: Various test improvements and implementations. - `robot/e2e/tdd_acms_behavioral_validation.robot:180`: Indexing pipeline wiring. - `features/wf03_plan_prompt_confidence.feature:17`: Verify `guidance_added.scope`, `state_transition`. - `features/steps/tdd_exec_env_resolution_precedence_steps.py:82`: 6-level precedence chain is built manually. - `features/steps/estimation_actor_steps.py:368`: Use public `save_plan()` once test helpers are refactored. - `features/consolidated_routing.feature:848,856,865,874`: Uncomment when step definitions are implemented. - `features/consolidated_domain_models.feature:67,83,94`: Uncomment when step definitions are implemented. **Documentation:** - `docs/reference/lsp_stub.md:151`: Wire `_dispatch_stub()` through `self._facade`. - `docs/development/review_playbook.md:45`: TODO placement. **Changelog:** - `CHANGELOG.md:1204`: `TODO` comments in runtime code. **Internal Agent Definitions:** - `.opencode/agents/product-builder.md:943`: No `TODO`/`FIXME`/`HACK` markers remain in code. - `.opencode/agents/ca-product-verifier.md:119,121,166`: Search for `TODO`, `FIXME`, `HACK`, `XXX` comments. - `.opencode/agents/ca-milestone-reviewer.md:166`: Incomplete `TODO`/`FIXME`/`HACK` comments. - `.opencode/agents/ca-architecture-guard.md:143`: `TODO`/`FIXME` comments. --- **Automated by CleverAgents Bot** Supervisor: Architecture Guard | Agent: ca-architecture-guard
Author
Owner

⚠️ Potential duplicate detected — This issue appears to cover the same work as:

  • #3154 — Refactor: Address outstanding TODOs and FIXMEs in the codebase

Both issues describe addressing TODO/FIXME comments. Please review and close this issue as a duplicate of #3154 if confirmed.


Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: ca-backlog-groomer

⚠️ **Potential duplicate detected** — This issue appears to cover the same work as: - #3154 — Refactor: Address outstanding TODOs and FIXMEs in the codebase Both issues describe addressing TODO/FIXME comments. Please review and close this issue as a duplicate of #3154 if confirmed. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: ca-backlog-groomer
Author
Owner

Closing as duplicate of #3154. Both issues address outstanding TODO/FIXME comments in the codebase. #3154 is the canonical issue.


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

Closing as duplicate of #3154. Both issues address outstanding TODO/FIXME comments in the codebase. #3154 is the canonical issue. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
freemo 2026-04-05 07:39:25 +00:00
Author
Owner

Closing as duplicate of #3154. Both issues track the same work: addressing TODO/FIXME comments in the codebase. #3154 has a more detailed description.


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

Closing as duplicate of #3154. Both issues track the same work: addressing TODO/FIXME comments in the codebase. #3154 has a more detailed description. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core#3141
No description provided.