Skip to content

docs: fix Smart Denoising (Krisp) default to true - #1387

Open
manish-vapi wants to merge 2 commits into
VapiAI:mainfrom
manish-vapi:docs/fix-smart-denoising-default
Open

manish-vapi wants to merge 2 commits into
VapiAI:mainfrom
manish-vapi:docs/fix-smart-denoising-default

Conversation

@manish-vapi

Copy link
Copy Markdown

The background speech denoising page listed smartDenoisingPlan.enabled as default false. The API spec and server behavior default it to true: Krisp runs even when backgroundSpeechDenoisingPlan is omitted. Correct the default and add a note showing how to disable it.

Description

  • Fix smartDenoisingPlan.enabled default on the Background speech denoising page from false to true, matching the API reference (SmartDenoisingPlan.enabled.default = true) and server behavior.
  • Add a note clarifying that Krisp runs even when backgroundSpeechDenoisingPlan is omitted, with an example of disabling it via smartDenoisingPlan: { enabled: false }.
  • Fourier denoising default (false) is unchanged and already correct.

Testing Steps

  • Run the app locally using fern docs dev or navigate to preview deployment
  • Ensure that the changed pages and code snippets work

The background speech denoising page listed smartDenoisingPlan.enabled
as default false. The API spec and server behavior default it to true:
Krisp runs even when backgroundSpeechDenoisingPlan is omitted. Correct
the default and add a note showing how to disable it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. default="true" matches SmartDenoisingPlan.enabled.default: true in fern/apis/api/openapi.json, and the new note covers omit-means-on plus how to disable. The failing preview/run checks are only missing FERN_TOKEN (fork PR, no secrets); fern-check passes.

Written by Devin

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.

3 participants