Conversation
Swap temporarily frozen .fernignore entries to .bak paths so the Fern generator can overwrite the originals. Our manually-patched versions are preserved as .bak files for post-regen comparison.
There was a problem hiding this comment.
Pull request overview
This PR appears to prepare the repo for a Fern SDK regeneration by “freezing” the current manually-patched versions of several generated files (as *.bak) and updating .fernignore to point at those backups.
Changes:
- Added
*.bakcopies of several previously patched Fern-generated files (socket clients, listen types, listen clients). - Added
*.bakcopies of WireMock mappings and a Wire test file used in the wiremock-based suite. - Updated
.fernignoreentries to ignore the.bakpaths (temporarily freezing them during regeneration).
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.fernignore |
Switches ignore targets from original files to *.bak backups to preserve patches during regen. |
wiremock/wiremock-mappings.json.bak |
Backup of WireMock mappings (to preserve pre-regen patched version). |
tests/wire/test_listen_v1_media.py.bak |
Backup of wire test (to preserve pre-regen patched version). |
src/deepgram/speak/v1/socket_client.py.bak |
Backup of patched Speak WS client. |
src/deepgram/listen/v1/socket_client.py.bak |
Backup of patched Listen v1 WS client. |
src/deepgram/listen/v2/socket_client.py.bak |
Backup of patched Listen v2 WS client. |
src/deepgram/agent/v1/socket_client.py.bak |
Backup of patched Agent WS client (incl. numeric sanitization patch). |
src/deepgram/listen/v1/client.py.bak |
Backup of patched Listen v1 client (query array param support). |
src/deepgram/listen/v2/client.py.bak |
Backup of patched Listen v2 client (query array param support). |
src/deepgram/types/listen_v1response_results_utterances_item.py.bak |
Backup of patched type (int vs float corrections). |
src/deepgram/types/listen_v1response_results_utterances_item_words_item.py.bak |
Backup of patched type (int vs float corrections). |
src/deepgram/types/listen_v1response_results_channels_item_alternatives_item_paragraphs_paragraphs_item.py.bak |
Backup of patched type (int vs float corrections). |
src/deepgram/types/listen_v1redact.py.bak |
Backup of patched redact type (Union/Sequence support). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # [temporarily frozen — .bak preserves our patches during regen] | ||
| src/deepgram/speak/v1/socket_client.py.bak | ||
| src/deepgram/listen/v1/socket_client.py.bak | ||
| src/deepgram/listen/v2/socket_client.py.bak | ||
| src/deepgram/agent/v1/socket_client.py.bak |
There was a problem hiding this comment.
These newly added src/deepgram/**.py.bak files live inside the published deepgram package directory (see pyproject.toml package include config). With no packaging excludes configured, they are likely to be shipped in the wheel/sdist. If these backups are intended to be temporary for regeneration, consider keeping them outside src/ or adding an explicit packaging exclude for *.bak, and make sure they’re removed before merging/releasing.
| # WireMock mappings: removed duplicate empty-body /v1/listen stub that causes | ||
| # non-deterministic matching failures | ||
| wiremock/wiremock-mappings.json | ||
| # [temporarily frozen — .bak preserves our patch during regen] | ||
| wiremock/wiremock-mappings.json.bak | ||
|
|
||
| # Wire test with manual fix: transcribe_file() requires request=bytes parameter | ||
| tests/wire/test_listen_v1_media.py | ||
| # [temporarily frozen — .bak preserves our patch during regen] | ||
| tests/wire/test_listen_v1_media.py.bak |
There was a problem hiding this comment.
PR description says this adds support for the new "Think" agent message type, but the only functional change here is swapping .fernignore entries to point at .bak backups (plus adding those backup files). If this PR is just the "prepare repo for regeneration" step, please update the PR description/title accordingly and ensure a follow-up commit in this PR restores .fernignore to the real paths and removes the .bak files before merging; otherwise this PR won’t actually deliver the stated feature support.
efdc2b3 to
78846e8
Compare
This pull request primarily adds support for new "Think" agent message type.