(janitor/dead-code): remove orphaned SSEStreamProcessor in auto-fix-infra - #5762
Merged
Merged
Conversation
…nfra SSEStreamProcessor (services/auto-fix-infra/src/services/sse-stream-processor.ts) is dead code. It is never imported, never registered, and never exercised: - No import of the symbol `SSEStreamProcessor` or the path `sse-stream-processor` anywhere in the repo (source, tests, or config). - No test references it. No barrel re-exports it (src/services/ has no index.ts). The service entry point (src/index.ts) and fix-orchestrator.ts only import prompt-builder and cloud-agent-next-client. - The auto-fix Worker's actual cloud-agent client (cloud-agent-next-client.ts) uses HTTP fetch for prepare/initiate, not SSE. - Not referenced by wrangler.jsonc (main is src/index.ts; the only Durable Object is AutoFixOrchestrator), nor by any dynamic import, cron, queue, or RPC config. - The file has no imports and no module-level side effects (only type declarations and a class), so removal changes no initialization behavior. - Kilo-Org/kilocode (external API consumer) contains no reference. Verified with repo-wide grep, the service typecheck, lint, and unit tests.
Contributor
Author
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 files)
Reviewed by grok-4.6 · Input: 55.6K · Output: 3.6K · Cached: 91.3K Review guidance: REVIEW.md from base branch |
pandemicsyn
approved these changes
Aug 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What was deleted
services/auto-fix-infra/src/services/sse-stream-processor.ts— a single 204-line file containing theSSEStreamProcessorclass (a generic Server-Sent Events parser/buffer manager) plus two module-private type declarations (StreamEventHandler,StreamMetrics).Evidence it was unreachable
SSEStreamProcessorhas zero references outside its own file (only its definition and internal method calls).sse-stream-processor(relative, package subpath, or@/alias) anywhere in the repo, including.ts,.tsx,.js,.mjs,.json, and.jsonc.services/auto-fix-infra/src/services/has noindex.ts; nothing re-exports the module.src/index.ts(Worker entry) andsrc/fix-orchestrator.tsimport only./services/prompt-builderand./services/cloud-agent-next-client. The actual cloud-agent client (cloud-agent-next-client.ts) talks to cloud-agent-next over HTTPfetch(prepare/initiate), not SSE.typedeclarations and a class), so deleting it cannot remove any initialization behavior or side effects.SSEStreamProcessororsse-stream-processor.Dynamic and configured entry points checked
wrangler.jsonc:mainissrc/index.ts; the only Durable Object binding isAutoFixOrchestrator; no cron/queue/RPC/alarm/workflow binding references this module.import(...)references the path.apps/web).package.jsonscripts.Contract and compatibility considerations
kilo-auto-fix-workeris aprivate: truepackage with no published exports;SSEStreamProcessoris not in its surface.AutoFixOrchestratorDurable Object contract.Validation performed
pnpm --filter kilo-auto-fix-worker typecheck— passes (no errors).pnpm --filter kilo-auto-fix-worker lint— 0 warnings, 0 errors.pnpm --filter kilo-auto-fix-worker test— 1 passed.