remove uneccesary pass through getMaxAllowedSize; fix tests

Author: ocasta181

src/integrations/misc/extract-text.test.ts

@@ -29,12 +29,12 @@ describe("extract-text", () => {
 	})
 
 	it("throws ContentTooLargeError when file would exceed max allowed size", async () => {
-		// Create content that would exceed max allowed size for deepseek (64k - 27k tokens)
-		const largeContent = "x".repeat(148000) // 37k tokens > (64k - 27k) tokens
+		// Create content that would exceed max allowed size (37k tokens)
+		const largeContent = "x".repeat(148000) // 37k tokens
 		await fs.writeFile(tempFilePath, largeContent)
 
 		try {
-			await extractTextFromFile(tempFilePath, 64_000)
+			await extractTextFromFile(tempFilePath, 37_000) // Pass pre-processed maxAllowedSize
 			throw new Error("Should have thrown error")
 		} catch (error) {
 			expect(error).to.be.instanceOf(ContentTooLargeError)

src/utils/content-size.test.ts

@@ -16,9 +16,11 @@ describe("content-size", () => {
 
 	describe("wouldExceedSizeLimit", () => {
 		it("checks if byte count would exceed max allowed size", () => {
-			expect(wouldExceedSizeLimit(100, 64_000)).to.equal(false) // 25 tokens < (64k - 27k) tokens
-			expect(wouldExceedSizeLimit(148000, 64_000)).to.equal(true) // 37k tokens > (64k - 27k) tokens
-			expect(wouldExceedSizeLimit(392000, 128_000)).to.equal(true) // 98k tokens > (128k - 30k) tokens
+			// For deepseek (64k - 27k = 37k tokens)
+			expect(wouldExceedSizeLimit(100, 37_000)).to.equal(false) // 25 tokens < 37k tokens
+			expect(wouldExceedSizeLimit(148000, 37_000)).to.equal(true) // 37k tokens = 37k tokens
+			// For standard models (128k - 30k = 98k tokens)
+			expect(wouldExceedSizeLimit(392000, 98_000)).to.equal(true) // 98k tokens = 98k tokens
 		})
 	})
 

src/utils/content-size.ts

@@ -23,8 +23,7 @@ export function estimateTokens(bytes: number): number {
  */
 export function wouldExceedSizeLimit(byteCount: number, contextLimit: number): boolean {
 	const estimatedTokenCount = estimateTokens(byteCount)
-	const maxAllowedSize = getMaxAllowedSize(contextLimit)
-	return estimatedTokenCount >= maxAllowedSize
+	return estimatedTokenCount >= getMaxAllowedSize(contextLimit)
 }
 
 /**