Remove ambiguous string scope (#1661)

And add notification for when we break it again

## Checklist

- [ ] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [ ] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [ ] I have not broken the cheatsheet

Author: pokey

Diff for: cursorless-talon/src/csv_overrides.py

@@ -143,6 +143,10 @@ def update_dicts(
     results_map = {}
     for list_name, dict in default_values.items():
         for key, value in dict.items():
+            if value in results_map:
+                warning = f"WARNING: Duplicate value `{value}` in {list_name}.csv"
+                print(warning)
+                app.notify(warning)
             results_map[value] = {"key": key, "value": value, "list": list_name}
 
     # Update result with current values

Diff for: cursorless-talon/src/modifiers/scopes.py

@@ -46,7 +46,6 @@
     "-six section": "sectionLevelSix",
     "selector": "selector",
     "state": "statement",
-    "string": "string",
     "branch": "branch",
     "type": "type",
     "value": "value",