Skip to content

Clean up dead autocomplete code#8654

Merged
markijbema merged 5 commits into
mainfrom
session/agent_54a63e1a-9980-4a7d-b2d4-13233973f500
Apr 9, 2026
Merged

Clean up dead autocomplete code#8654
markijbema merged 5 commits into
mainfrom
session/agent_54a63e1a-9980-4a7d-b2d4-13233973f500

Conversation

@kilo-code-bot

@kilo-code-bot kilo-code-bot Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove unused autocomplete docs, package NLS files, non-runtime locale dictionaries, and orphaned chat handler modules.
  • Drop the dormant ContinueDev completion provider pipeline plus its unused generation, filtering, templating, diff, LLM, and test-support code.
  • Remove unused autocomplete exports/types and include src/services/autocomplete/** in the package knip coverage.

Tests

  • bun run format
  • bun run check-types:extension
  • bun run compile-tests
  • bun run lint
  • bun run knip -- --reporter compact --max-show-issues 1000
  • bun run check-kilocode-change
  • bun run script/extract-source-links.ts
  • bun test tests/unit/autocomplete-error-backoff.test.ts tests/unit/autocomplete-inline-utils.test.ts tests/unit/autocomplete-statusbar-utils.test.ts tests/unit/autocomplete-telemetry-utils.test.ts tests/unit/chat-autocomplete-utils.test.ts tests/unit/contextual-skip.test.ts tests/unit/useless-suggestion-filter.test.ts tests/unit/visible-code-utils.test.ts tests/unit/file-ignore-controller.test.ts tests/unit/i18n-shim.test.ts tests/unit/i18n-keys.test.ts
  • bun test src/services/autocomplete/continuedev/core/llm/countTokens.test.ts src/services/autocomplete/continuedev/core/diff/util.test.ts src/services/autocomplete/continuedev/core/autocomplete/filtering/streamTransforms/lineStream.test.ts src/services/autocomplete/continuedev/core/autocomplete/templating/__tests__/formatOpenedFilesContext.test.ts src/services/autocomplete/continuedev/core/autocomplete/postprocessing/index.test.ts src/services/autocomplete/continuedev/core/autocomplete/postprocessing/__tests__/removePrefixOverlap.test.ts
  • Pre-push hook: bun turbo typecheck

Notes

  • A direct broad bun test src/services/autocomplete ... invocation is not a supported target here; it fails on existing VS Code mock setup in source-local tests, so the supported unit files and remaining pure ContinueDev tests were run explicitly.

@kilo-code-bot

kilo-code-bot Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: 3 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 3

Fix these issues in Kilo Cloud

Issue Details (click to expand)

No inline issues on diff-commentable lines.

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
packages/kilo-vscode/src/services/autocomplete/continuedev/INTEGRATION.md 154 Still imports deleted core/autocomplete/CompletionProvider.ts and links deleted API_REFERENCE.md, so this doc now contains broken references.
packages/kilo-vscode/src/services/autocomplete/continuedev/ARCHITECTURE.md 123 Still documents deleted core/autocomplete/CompletionProvider.ts and later links deleted API_REFERENCE.md, leaving stale navigation after this cleanup.
packages/kilo-vscode/src/services/autocomplete/continuedev/EXAMPLES.md 20 Still shows imports from deleted CompletionProvider / MinimalConfig modules and later links deleted API_REFERENCE.md, so the examples are no longer runnable as written.
Files Reviewed (11 files)
  • packages/kilo-vscode/knip.json - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/chat-autocomplete/ChatTextAreaAutocomplete.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/context/root-path-context/__test-cases__/python.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/filtering/streamTransforms/lineStream.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/util/types.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/diff/util.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/countTokens.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/test/fixtures.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/test/testDir.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/util/parameters.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/util/uri.ts - 0 issues

Reviewed by gpt-5.4-20260305 · 725,902 tokens

@markijbema markijbema enabled auto-merge April 9, 2026 10:37
@markijbema markijbema merged commit d65ee14 into main Apr 9, 2026
17 checks passed
@markijbema markijbema deleted the session/agent_54a63e1a-9980-4a7d-b2d4-13233973f500 branch April 9, 2026 10:40
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
…-9980-4a7d-b2d4-13233973f500

Clean up dead autocomplete code
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.

2 participants