Skip to content

Conversation

@trycatchkamal
Copy link
Contributor

@trycatchkamal trycatchkamal commented Jul 10, 2025

Removes unused test-related classes, functions, and types identified by static analysis

These symbols are not referenced anywhere in the codebase, including other tests

Used ts-prune to detect unused exports in the codebase

Manually verified that these symbols are not referenced anywhere else

Summary

Component File
ChunkingEndpointClientCache, ISerializedFileChunk, ISerializedFileChunkWithEmbedding, ICacheEntry test/base/cachingChunksEndpointClient.ts
CodeOrDocSearchCache, ICacheEntry test/base/cachingCodeSearchClient.ts
EmbeddingsCache, ICacheEntry test/base/embeddingsCache.ts
forEachCombination (function) test/base/stest.ts
MockProgress (class) test/base/testProgress.ts
assertDeepStrictEqualsAny test/simulation/stestUtil.ts

@trycatchkamal
Copy link
Contributor Author

@microsoft-github-policy-service agree

Copy link
Contributor

@mjbvz mjbvz left a comment

Choose a reason for hiding this comment

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

Thanks!

@mjbvz mjbvz enabled auto-merge July 11, 2025 05:46
@vs-code-engineering vs-code-engineering bot added this to the July 2025 milestone Jul 11, 2025
@mjbvz mjbvz added this pull request to the merge queue Jul 11, 2025
Merged via the queue into microsoft:main with commit 91dc2d6 Jul 11, 2025
6 checks passed
connor4312 pushed a commit that referenced this pull request Jul 15, 2025
Co-authored-by: trycatchkamal <=>
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