Add migration instruction to snippets error message (#2833)

AndreasArvidsson · web-flow · commit 3b8f2828943b · 2025-02-06T16:44:41.000Z
diff --git a/cursorless-talon/src/snippets/snippets_deprecated.py b/cursorless-talon/src/snippets/snippets_deprecated.py
@@ -30,7 +30,7 @@ class Actions:
     def cursorless_insert_snippet_by_name(name: str):  # pyright: ignore [reportGeneralTypeIssues]
         """[DEPRECATED] Cursorless: Insert named snippet <name>"""
         raise NotImplementedError(
-            "Cursorless snippets are deprecated. Please use community snippets."
+            "Cursorless snippets are deprecated. Please use community snippets. Update to latest cursorless-talon and say 'cursorless migrate snippets'."
         )
 
     def cursorless_wrap_with_snippet_by_name(
@@ -40,7 +40,7 @@ def cursorless_wrap_with_snippet_by_name(
     ):
         """[DEPRECATED] Cursorless: Wrap target with a named snippet <name>"""
         raise NotImplementedError(
-            "Cursorless snippets are deprecated. Please use community snippets."
+            "Cursorless snippets are deprecated. Please use community snippets. Update to latest cursorless-talon and say 'cursorless migrate snippets'."
         )
 
 
diff --git a/packages/cursorless-engine/src/core/getPreferredSnippet.ts b/packages/cursorless-engine/src/core/getPreferredSnippet.ts
@@ -22,7 +22,7 @@ export function getPreferredSnippet(
 ) {
   if (snippetDescription.type === "named") {
     throw new Error(
-      "Cursorless snippets are deprecated. Please use community snippets.",
+      "Cursorless snippets are deprecated. Please use community snippets. Update to latest cursorless-talon and say 'cursorless migrate snippets'.",
     );
   }
   if (snippetDescription.type === "custom") {

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ class Actions:`
`30`	`30`	`def cursorless_insert_snippet_by_name(name: str): # pyright: ignore [reportGeneralTypeIssues]`
`31`	`31`	`"""[DEPRECATED] Cursorless: Insert named snippet <name>"""`
`32`	`32`	`raise NotImplementedError(`
`33`		`- "Cursorless snippets are deprecated. Please use community snippets."`
	`33`	`+ "Cursorless snippets are deprecated. Please use community snippets. Update to latest cursorless-talon and say 'cursorless migrate snippets'."`
`34`	`34`	`)`
`35`	`35`
`36`	`36`	`def cursorless_wrap_with_snippet_by_name(`
`@@ -40,7 +40,7 @@ def cursorless_wrap_with_snippet_by_name(`
`40`	`40`	`):`
`41`	`41`	`"""[DEPRECATED] Cursorless: Wrap target with a named snippet <name>"""`
`42`	`42`	`raise NotImplementedError(`
`43`		`- "Cursorless snippets are deprecated. Please use community snippets."`
	`43`	`+ "Cursorless snippets are deprecated. Please use community snippets. Update to latest cursorless-talon and say 'cursorless migrate snippets'."`
`44`	`44`	`)`
`45`	`45`
`46`	`46`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ export function getPreferredSnippet(`
`22`	`22`	`) {`
`23`	`23`	`if (snippetDescription.type === "named") {`
`24`	`24`	`throw new Error(`
`25`		`- "Cursorless snippets are deprecated. Please use community snippets.",`
	`25`	`+ "Cursorless snippets are deprecated. Please use community snippets. Update to latest cursorless-talon and say 'cursorless migrate snippets'.",`
`26`	`26`	`);`
`27`	`27`	`}`
`28`	`28`	`if (snippetDescription.type === "custom") {`