Skip to content

feat: Add Post Call Transcription to Archives - #270

Open
superchilled wants to merge 7 commits into
mainfrom
feat-add-post-call-transcription-to-archives
Open

feat: Add Post Call Transcription to Archives#270
superchilled wants to merge 7 commits into
mainfrom
feat-add-post-call-transcription-to-archives

Conversation

@superchilled

Copy link
Copy Markdown
Contributor

This PR updates the implementation for the Archives feature in order to add support for post-call transcription.

@vonage-ai-assistant

vonage-ai-assistant Bot commented Aug 24, 2026

Copy link
Copy Markdown

AI Code: 100%

AI Adoption Report

Lines %
AI-generated 10 100%
Human 0 0%
Untracked 97
Total 107

💰 Cost Breakdown

Agent Model AI Lines Input Tokens Output Tokens Cost Source
GitHub Copilot github-copilot 10 112 34 $0.0000 🤖 Copilot (subscription)

Total Estimated Cost: $0.0000

Data Sources:

  • 📐 3 commits with estimated tokens (~3.5 tokens/line)

Bedrock pricing: eu-central-1 region

Per-commit breakdown (7 commits)
Commit Message Added AI Human AI %
3b896b8 Only include hasTranscription and transcriptionProperties in 5 5 0 🤖 100% (Copilot)
8b99a8e Potential fix for pull request finding 1 1 0 🤖 100% (Copilot)
109c596 Potential fix for pull request finding 4 4 0 🤖 100% (Copilot)
559568f Updating doc blocks for transcription properties 8 untracked
429c330 Updating Archives.init 2 untracked
ba484f0 Updating start_archive method 15 untracked
c5736c1 Adding unit test for archive with transcription params 72 untracked

Warning

The following commits have no Git AI authorship notes — AI adoption cannot be tracked for these changes.
Install git-ai locally (curl -sSL https://usegitai.com/install.sh | bash) to enable automatic tracking.

  • 559568f Updating doc blocks for transcription properties
  • 429c330 Updating Archives.init
  • ba484f0 Updating start_archive method
  • c5736c1 Adding unit test for archive with transcription params

Powered by Git AI Standard v3.0.0 — authorship data from refs/notes/ai (supports sessions-v2 format)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the Archives feature to support post-call transcription by adding transcription parameters to start_archive, parsing transcription-related fields onto the Archive model, and validating the behavior via a new unit test.

Changes:

  • Added has_transcription and transcription_properties parameters to OpenTok.start_archive() and included them in the archive start request payload.
  • Extended Archive to expose has_transcription and transcription_properties from the API response.
  • Added a new test covering archive creation with transcription enabled and verifying request/response handling.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
tests/test_archive_api.py Adds coverage for starting an archive with transcription enabled and validating request/response fields.
opentok/opentok.py Adds new transcription-related parameters and payload fields to start_archive, plus corresponding docstring updates.
opentok/archives.py Adds transcription-related fields to the Archive model initialization.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread opentok/opentok.py
Comment thread opentok/opentok.py Outdated
Comment thread opentok/archives.py Outdated
superchilled and others added 2 commits August 24, 2026 12:46
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…when transcription is enabled

Co-authored-by: superchilled <5489922+superchilled@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Over 80% AI contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants