Skip to content

Remove old tfidf worker code#311536

Merged
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/eerie-raven
Apr 20, 2026
Merged

Remove old tfidf worker code#311536
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/eerie-raven

Conversation

@mjbvz
Copy link
Copy Markdown
Collaborator

@mjbvz mjbvz commented Apr 20, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 20, 2026 23:16
@mjbvz mjbvz enabled auto-merge April 20, 2026 23:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the legacy TF‑IDF implementation and its worker wiring from the Copilot extension codebase, simplifying the build/package outputs by dropping unused search/indexing code.

Changes:

  • Deleted the TF‑IDF SQLite-backed index implementation, worker entrypoint, and messaging helpers.
  • Removed the TF‑IDF unit tests.
  • Updated build/packaging configuration to stop producing and shipping the tfidfWorker bundle, and removed ESLint config references.
Show a summary per file
File Description
extensions/copilot/src/platform/tfidf/node/tfidfWorker.ts Removed the TF‑IDF worker implementation and RPC host/worker plumbing.
extensions/copilot/src/platform/tfidf/node/tfidfMessaging.ts Removed the object rewrite helper used for worker message serialization.
extensions/copilot/src/platform/tfidf/node/tfidf.ts Removed the SQLite-backed PersistentTfIdf implementation and search logic.
extensions/copilot/src/platform/tfidf/node/test/tfidf.spec.ts Removed TF‑IDF-specific Vitest unit tests.
extensions/copilot/eslint.config.mjs Removed references to the deleted TF‑IDF files from the ESLint config.
extensions/copilot/.vscodeignore Stopped including dist/tfidfWorker.js in the packaged extension.
extensions/copilot/.esbuild.ts Removed the TF‑IDF worker entry from the esbuild worker bundle list.

Copilot's findings

  • Files reviewed: 7/7 changed files
  • Comments generated: 0

@mjbvz mjbvz merged commit a13c4de into microsoft:main Apr 20, 2026
30 checks passed
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 20, 2026
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.

3 participants