Add a setting to confirm prompts before using them.

Signed-off-by: Katharine Berry <ktbry@google.com>

Author: Katharine

app/package.json

@@ -105,7 +105,8 @@
       "IMAGE_ID",
       "MAP_WIDGET",
       "MAP_WIDGET_IMAGE_ID",
-      "MAP_WIDGET_USER_LOCATION"
+      "MAP_WIDGET_USER_LOCATION",
+      "CONFIRM_TRANSCRIPTS"
     ],
     "resources": {
       "media": [

app/src/c/converse/session_window.c

@@ -19,6 +19,7 @@
 #include "conversation.h"
 #include "conversation_manager.h"
 #include "segments/segment_layer.h"
+#include "../settings/settings.h"
 #include "../util/thinking_layer.h"
 #include "../util/style.h"
 #include "../util/action_menu_crimes.h"
@@ -135,7 +136,7 @@ static void prv_window_load(Window *window) {
   conversation_manager_set_handler(sw->manager, prv_conversation_manager_handler, sw);
   conversation_manager_set_deletion_handler(sw->manager, prv_conversation_entry_deleted_handler);
   sw->dictation = dictation_session_create(0, prv_dictation_status_callback, sw);
-  dictation_session_enable_confirmation(sw->dictation, false);
+  dictation_session_enable_confirmation(sw->dictation, settings_get_should_confirm_transcripts());
 
   sw->segment_space = 3;
   sw->segment_count = 0;

app/src/c/settings/settings.c

@@ -56,6 +56,11 @@ VibePatternSetting settings_get_timer_vibe_pattern() {
   return result;
 }
 
+bool settings_get_should_confirm_transcripts() {
+  // the default is false, so we don't have to check whether it exists.
+  return persist_read_bool(PERSIST_KEY_CONFIRM_TRANSCRIPTS);
+}
+
 static void prv_app_message_handler(DictionaryIterator *iter, void *context) {
   for (Tuple *tuple = dict_read_first(iter); tuple; tuple = dict_read_next(iter)) {
     if (tuple->key == MESSAGE_KEY_QUICK_LAUNCH_BEHAVIOUR) {
@@ -65,6 +70,8 @@ static void prv_app_message_handler(DictionaryIterator *iter, void *context) {
       persist_write_int(PERSIST_KEY_ALARM_VIBE_PATTERN, atoi(tuple->value->cstring));
     } else if (tuple->key == MESSAGE_KEY_TIMER_VIBE_PATTERN) {
       persist_write_int(PERSIST_KEY_TIMER_VIBE_PATTERN, atoi(tuple->value->cstring));
+    } else if (tuple->key == MESSAGE_KEY_CONFIRM_TRANSCRIPTS) {
+      persist_write_bool(PERSIST_KEY_CONFIRM_TRANSCRIPTS, tuple->value->int8);
     }
   }
 }

app/src/c/settings/settings.h

@@ -16,6 +16,8 @@
 
 #pragma once
 
+#include <pebble.h>
+
 typedef enum {
   QuickLaunchBehaviourConverseWithTimeout = 1,
   QuickLaunchBehaviourConverseForever = 2,
@@ -35,3 +37,4 @@ void settings_deinit();
 QuickLaunchBehaviour settings_get_quick_launch_behaviour();
 VibePatternSetting settings_get_alarm_vibe_pattern();
 VibePatternSetting settings_get_timer_vibe_pattern();
+bool settings_get_should_confirm_transcripts();

app/src/c/util/persist_keys.h

@@ -20,7 +20,7 @@
 // These keys are stored centrally so we can avoid accidental collisions.
 // Remember: these numbers can *never* be changed.
 
-// next key: 13
+// next key: 14
 
 // We write the alarm count twice - once before doing any work, and once after.
 // If they disagree we assume the lower number is correct.
@@ -44,5 +44,6 @@
 #define PERSIST_KEY_QUICK_LAUNCH_BEHAVIOUR 9
 #define PERSIST_KEY_ALARM_VIBE_PATTERN 10
 #define PERSIST_KEY_TIMER_VIBE_PATTERN 11
+#define PERSIST_KEY_CONFIRM_TRANSCRIPTS 13
 
 #endif //APP_PERSIST_KEYS_H

app/src/pkjs/config.json

@@ -113,6 +113,13 @@
                     {"label": "Start conversation then stay open", "value": "2"},
                     {"label": "Open home screen", "value": "3"}
                 ]
+            },
+            {
+                "type": "toggle",
+                "messageKey": "CONFIRM_TRANSCRIPTS",
+                "defaultValue": false,
+                "label": "Confirm prompts before responding",
+                "description": "When enabled, after dictating to the watch, you'll always be asked to confirm your input before Bobby takes action. When disabled, Bobby will respond immediately to whatever was recognised."
             }
         ]
     },