Skip to content

Add wtf_transcribe link for WTF transcription#131

Merged
pavanputhra merged 1 commit intomainfrom
feature/wtf-transcribe-link
Mar 9, 2026
Merged

Add wtf_transcribe link for WTF transcription#131
pavanputhra merged 1 commit intomainfrom
feature/wtf-transcribe-link

Conversation

@howethomas
Copy link
Contributor

@howethomas howethomas commented Mar 2, 2026

Note

Medium Risk
Adds a new link that fetches/decodes audio (including file:// and remote URLs) and sends it to an external transcription service, then persists derived analysis back to Redis; failures or misconfiguration could impact processing and introduces new outbound network/file access paths.

Overview
Adds a new wtf_transcribe link that scans vCon dialogs, extracts audio from body (base64/base64url) or url (including file:///HTTP), POSTs it to a configurable vfun /transcribe endpoint (optional bearer api-key, diarize, timeout), and writes back a generated WTF-compliant wtf_transcription analysis entry per dialog.

The link skips non-recording dialogs, dialogs below min-duration, and dialogs already containing a wtf_transcription, and includes documentation for configuration, chain usage, and expected output format.

Written by Cursor Bugbot for commit 950bc3c. Configure here.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Free Tier Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

# Send audio to vfun server
files = {"file": (filename, audio_content, mimetype)}
data = {
"diarize": str(opts.get("diarize", True)),
Copy link

Choose a reason for hiding this comment

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

str(True) produces "True" not "true" for API

Medium Severity

Python's str(True) returns "True" (capital T), not "true". The diarize form field sent to the vfun server will be "True" instead of the expected lowercase "true". Note that the adjacent block field on line 284 correctly uses the lowercase string literal "true", making this inconsistency especially likely to cause issues. Most HTTP APIs parse boolean strings as lowercase, so diarization may silently fail to activate.

Fix in Cursor Fix in Web

@pavanputhra pavanputhra merged commit 04ff18d into main Mar 9, 2026
2 checks passed
@pavanputhra pavanputhra deleted the feature/wtf-transcribe-link branch March 9, 2026 00:25
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