Skip to content

refactor(background-task): delete dead code in modules/ directory#1852

Open
iyoda wants to merge 1 commit intocode-yeongyu:devfrom
iyoda:refactor/pr0-delete-dead-code
Open

refactor(background-task): delete dead code in modules/ directory#1852
iyoda wants to merge 1 commit intocode-yeongyu:devfrom
iyoda:refactor/pr0-delete-dead-code

Conversation

@iyoda
Copy link
Contributor

@iyoda iyoda commented Feb 15, 2026

Summary

Delete the entire src/tools/background-task/modules/ directory (6 files, ~827 lines) which is confirmed dead code — zero imports from anywhere in the codebase.

Changes

  • Deleted: src/tools/background-task/modules/background-cancel.ts (116 lines)
  • Deleted: src/tools/background-task/modules/background-output.ts (137 lines)
  • Deleted: src/tools/background-task/modules/background-task.ts (105 lines)
  • Deleted: src/tools/background-task/modules/formatters.ts (315 lines)
  • Deleted: src/tools/background-task/modules/message-processing.ts (75 lines)
  • Deleted: src/tools/background-task/modules/utils.ts (65 lines)

Verification

  • bun run typecheck
  • bun test ✅ (2693 pass, 0 fail)
  • bun run build

PR Stack

This is PR 0 of 9 in the code deduplication refactor series. Merge this first.

# PR Description
0 This PR Delete dead code in modules/
2 (next) Consolidate formatDuration (4→1)
3 Consolidate getMessageDir (7→1)
4 Unify fuzzyMatchModel
5 Add json-cache utility
6 Apply json-cache to consumers
7 Add ensure-directory utility
8 Apply ensure-directory to production
9 Consolidate ToolContextWithMetadata

Summary by cubic

Removed dead background-task modules. Trims ~827 LOC with no behavior changes.

  • Refactors
    • Deleted src/tools/background-task/modules/ (6 unused files).

Written for commit f2be9b4. Summary will update on new commits.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 10 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Requires human review: PR removes background-task modules. Despite no AI issues, this significantly changes functionality and may cause regressions. Not 100% sure no regressions.

@iyoda iyoda force-pushed the refactor/pr0-delete-dead-code branch from 69f7fb7 to f2be9b4 Compare February 15, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant