Assistant: Disable long sentences Breve feature by default (#38508)

* disable long sentences by default * changelog
Automattic · Jul 24, 2024 · 904df75 · 904df75
1 parent 4c6d134
commit 904df75
Show file tree

Hide file tree

Showing 6 changed files with 17 additions and 1 deletion.
diff --git a/projects/plugins/jetpack/changelog/update-jetpack-ai-breve-disable-long-sentences-default b/projects/plugins/jetpack/changelog/update-jetpack-ai-breve-disable-long-sentences-default
@@ -0,0 +1,4 @@
+Significance: patch
+Type: enhancement
+
+AI Assistant: Disable long sentences Breve feature by default
diff --git a/...k/extensions/plugins/ai-assistant-plugin/components/breve/features/complex-words/index.ts b/...k/extensions/plugins/ai-assistant-plugin/components/breve/features/complex-words/index.ts
@@ -15,6 +15,7 @@ export const COMPLEX_WORDS: BreveFeatureConfig = {
 	title: 'Complex words',
 	tagName: 'span',
 	className: 'has-proofread-highlight--complex-words',
+	defaultEnabled: true,
 };
 
 const list = new RegExp(

diff --git a/.../extensions/plugins/ai-assistant-plugin/components/breve/features/long-sentences/index.ts b/.../extensions/plugins/ai-assistant-plugin/components/breve/features/long-sentences/index.ts
@@ -12,6 +12,7 @@ export const LONG_SENTENCES: BreveFeatureConfig = {
 	title: 'Long sentences',
 	tagName: 'span',
 	className: 'has-proofread-highlight--long-sentences',
+	defaultEnabled: false,
 };
 
 const sentenceRegex = /[^\s][^.!?]+[.!?]+/g;

diff --git a/...tensions/plugins/ai-assistant-plugin/components/breve/features/unconfident-words/index.ts b/...tensions/plugins/ai-assistant-plugin/components/breve/features/unconfident-words/index.ts
@@ -13,6 +13,7 @@ export const UNCONFIDENT_WORDS: BreveFeatureConfig = {
 	title: 'Unconfident words',
 	tagName: 'span',
 	className: 'has-proofread-highlight--unconfident-words',
+	defaultEnabled: true,
 };
 
 const list = new RegExp( `\\b(${ words.map( escapeRegExp ).join( '|' ) })\\b`, 'gi' );

diff --git a/.../plugins/jetpack/extensions/plugins/ai-assistant-plugin/components/breve/store/reducer.ts b/.../plugins/jetpack/extensions/plugins/ai-assistant-plugin/components/breve/store/reducer.ts
@@ -2,6 +2,10 @@
  * WordPress dependencies
  */
 import { combineReducers } from '@wordpress/data';
+/**
+ * Internal dependencies
+ */
+import features from '../features';
 /**
  * Types
  */
@@ -14,7 +18,11 @@ const disabledFeaturesFromLocalStorage = window.localStorage.getItem(
 const initialConfiguration = {
 	enabled: enabledFromLocalStorage === 'true' || enabledFromLocalStorage === null,
 	disabled:
-		disabledFeaturesFromLocalStorage !== null ? JSON.parse( disabledFeaturesFromLocalStorage ) : [],
+		disabledFeaturesFromLocalStorage !== null
+			? JSON.parse( disabledFeaturesFromLocalStorage )
+			: features
+					.filter( feature => ! feature.config.defaultEnabled )
+					.map( feature => feature.config.name ),
 };
 
 export function configuration(

diff --git a/projects/plugins/jetpack/extensions/plugins/ai-assistant-plugin/components/breve/types.ts b/projects/plugins/jetpack/extensions/plugins/ai-assistant-plugin/components/breve/types.ts
@@ -91,6 +91,7 @@ export type BreveFeatureConfig = {
 	title: string;
 	tagName: string;
 	className: string;
+	defaultEnabled: boolean;
 };
 
 export type BreveFeature = {