8000 fix(ingest): stop ffmpeg, antiword and pdftotext runs that never finish by adeelahmad · Pull Request #202 · adeelahmad/lense · GitHub
[go: up one dir, main page]

Skip to content

fix(ingest): stop ffmpeg, antiword and pdftotext runs that never finish - #202

Merged
adeelahmad merged 2 commits into
mainfrom
claude/project-thread-qt650x-converter-timeouts
Oct 6, 2026
Merged

adeelahmad merged 2 commits into
mainfrom
claude/project-thread-qt650x-converter-timeouts

Conversation

@adeelahmad
Copy link
Copy Markdown
Owner

Requested by Adeel · project thread

Description

Before: decoding a recording with ffmpeg, encoding audio for a speech service, and reading .doc files (antiword, catdoc) or PDFs without pypdf (pdftotext) ran with no time limit. A file that made one of them hang held its worker forever.

After: each has a limit (6 hours for decoding a recording, 10 minutes for encoding a piece of speech, 5 minutes for reading a document). Past it the tool is stopped and the import fails with "took longer than ... s", like the converters that already had limits. Normal files finish far inside these limits.

Motivation and Context

This answers threat-model question Q20: every external tool Lens runs on uploaded files has a time limit.

Steps to reproduce (if appropriate):

test_stuck_converters_are_stopped in tests/domain/test_speech.py puts tools that never finish on PATH and checks each one is given up on.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change; ! in the title)

Checklist:

  • The title is a Conventional Commit.
  • My code follows the code style of this project (make hooks checks it on commit).
  • I added or updated tests.
  • My change requires documentation updates, and I have updated docs/ accordingly.
  • Users need more than the title to understand it, so I wrote it under ## Unreleased in CHANGELOG.md.

🤖 Generated with Claude Code

https://claude.ai/code/session_01W42z4FbekWr33b6791svQQ


Generated by Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01W42z4FbekWr33b6791svQQ
@adeelahmad adeelahmad self-assigned this Oct 6, 2026
@adeelahmad
adeelahmad merged commit a504d3c into main Oct 6, 2026
5 checks passed
@adeelahmad
adeelahmad deleted the claude/project-thread-qt650x-converter-timeouts branch October 6, 2026 11:05
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.

2 participants

0