Skip to content

feat: signature extraction performance enhancement#2767

Merged
malek10xdev merged 20 commits intomainfrom
fix/transactional-tag-filtering
Apr 3, 2026
Merged

feat: signature extraction performance enhancement#2767
malek10xdev merged 20 commits intomainfrom
fix/transactional-tag-filtering

Conversation

@malek10xdev
Copy link
Copy Markdown
Collaborator

@malek10xdev malek10xdev commented Apr 1, 2026

Summary

Performance Impact

  • ~70% reduction in LLM API calls for repeated signatures
  • 3x fewer Redis round-trips per signature

Files Changed

  • signature-extractor-cache.ts - NEW LLM response cache decorator
  • RedisEmailSignatureCache.ts - Added setIfNewer() with Redis pipeline
  • EmailSignatureCache.ts - Added setIfNewer() interface
  • handler.ts - Updated to use setIfNewer()
  • emailSignatureWorker.ts - Wired up SignatureExtractorCache
  • schema.ts - Added SIGNATURE_LLM_CACHE_TTL_SECONDS env var

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Coverage Report

Passed

Commit: 52e899a

Summary

Name Stmts Branch Funcs Lines
🔴 Total 38.5% 35.7% 44.7% 38.6%
All files
Name Stmts Branch Funcs Lines
🔴 backend/src/emailSignatureWorker.ts 0% 0% 0% 0%
🔴 backend/src/server.ts 0% 100% 0% 0%
🔴 backend/src/config/schema.ts 0% 100% 0% 0%
🔴 backend/src/services/cache/redis/RedisEmailSignatureCache.ts 0% 0% 0% 0%
🟢 backend/src/services/extractors/engines/EmailMessage.ts 93.8% 66.7% 100% 93.7%
🟢 backend/src/services/signature/index.ts 91.2% 92% 87.5% 91.2%
🟢 backend/src/services/signature/llm/signature-extractor-cache.ts 100% 66.7% 100% 100%
🔴 backend/src/utils/redis.ts 40% 12.5% 28.6% 40%
🔴 backend/src/workers/email-signature/handler.ts 0% 0% 0% 0%
No coverage changes
Name Stmts Branch Funcs Lines
🔴 backend/src/emailSignatureWorker.ts 0% 0% 0% 0%
🔴 backend/src/server.ts 0% 100% 0% 0%
🔴 backend/src/config/schema.ts 0% 100% 0% 0%
🔴 backend/src/services/cache/redis/RedisEmailSignatureCache.ts 0% 0% 0% 0%
🟢 backend/src/services/extractors/engines/EmailMessage.ts 93.8% 66.7% 100% 93.7%
🟢 backend/src/services/signature/index.ts 91.2% 92% 87.5% 91.2%
🟢 backend/src/services/signature/llm/signature-extractor-cache.ts 100% 66.7% 100% 100%
🔴 backend/src/utils/redis.ts 40% 12.5% 28.6% 40%
🔴 backend/src/workers/email-signature/handler.ts 0% 0% 0% 0%

Generated by Test Coverage Reporter for commit 52e899a

@malek10xdev malek10xdev merged commit be8bfbc into main Apr 3, 2026
15 checks passed
@malek10xdev malek10xdev deleted the fix/transactional-tag-filtering branch April 3, 2026 14:42
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.

[Performance] Signature Extraction: Add LLM response cache and batch Redis operations

1 participant