chore(chat): clean up code on large file detection (#8032)

clean up code on large file detection
## Test plan
Green CI
<!-- Required. See
https://docs-legacy.sourcegraph.com/dev/background-information/testing_principles.
-->

Author: julialeex

vscode/src/editor/utils/editor-context.test.ts

@@ -138,11 +138,9 @@ describe('filterContextItemFiles', () => {
 
         expect(filtered).toEqual([
             {
-                isTooLarge: true,
-                isTooLargeReason: 'File exceeds 1MB size limit',
-                size: 222222,
+                size: 222222, // Math.floor(1000001 / 4.5)
                 type: 'file',
-                uri: largeFile.uri,
+                uri: largeFile.uri, // Use the same URI reference
             },
         ])
     })

vscode/src/editor/utils/editor-context.ts

@@ -378,10 +378,6 @@ export async function filterContextItemFiles(
         if (cf.type !== 'file' || fileStat?.type !== vscode.FileType.File) {
             continue
         }
-
-        // Mark files larger than 1MB as too large but still include them
-        const isTooLarge = fileStat.size > 1000000
-
         // TODO (bee) consider a better way to estimate the token size of a file
         // We cannot get the exact token size without parsing the file, which is expensive.
         // Instead, we divide the file size in bytes by 4.5 for non-markdown as a rough estimate of the token size.
@@ -393,12 +389,6 @@ export async function filterContextItemFiles(
         // the file content on-demand instead of in bulk. We would then label the file size more accurately with the tokenizer.
         cf.size = Math.floor(fileStat.size / (cf.uri.fsPath.endsWith('.md') ? 3.5 : 4.5))
 
-        // Mark the file as too large if it exceeds the size limit
-        if (isTooLarge) {
-            cf.isTooLarge = true
-            cf.isTooLargeReason = 'File exceeds 1MB size limit'
-        }
-
         filtered.push(cf)
     }
     return filtered