Adding better error handling to /simple/search/

Author: coddingtonbear

src/constants.ts

@@ -40,6 +40,9 @@ export const ERROR_CODE_MESSAGES: Record<ErrorCode, string> = {
     "The 'Operation' header you provided was invalid.",
   [ErrorCode.PatchFailed]:
     "The patch you provided could not be applied to the target content.",
+  [ErrorCode.InvalidSearch]: "The search query you provided is not valid.",
+  [ErrorCode.ErrorPreparingSimpleSearch]:
+    "Error encountered while calling Obsidian `prepareSimpleSearch` API.",
 };
 
 export enum ContentTypes {

src/requestHandler.ts

@@ -991,9 +991,24 @@ export default class RequestHandler {
     const results: SearchResponseItem[] = [];
 
     const query: string = req.query.query as string;
+    if (!String.isString(query)) {
+      return this.returnCannedResponse(res, {
+        message: "A single '?query=' parameter is required.",
+        errorCode: ErrorCode.InvalidSearch,
+      });
+    }
     const contextLength: number =
       parseInt(req.query.contextLength as string, 10) ?? 100;
-    const search = prepareSimpleSearch(query);
+    let search: ReturnType<typeof prepareSimpleSearch>;
+    try {
+      search = prepareSimpleSearch(query);
+    } catch (e) {
+      console.error("Could not prepare simple search: ", e);
+      return this.returnCannedResponse(res, {
+        message: `${e}`,
+        errorCode: ErrorCode.ErrorPreparingSimpleSearch,
+      });
+    }
 
     for (const file of this.app.vault.getMarkdownFiles()) {
       const cachedContents = await this.app.vault.cachedRead(file);

src/types.ts

@@ -17,10 +17,12 @@ export enum ErrorCode {
   PeriodIsNotEnabled = 40060,
   InvalidFilterQuery = 40070,
   PatchFailed = 40080,
+  InvalidSearch = 40090,
   ApiKeyAuthorizationRequired = 40101,
   PeriodDoesNotExist = 40460,
   PeriodicNoteDoesNotExist = 40461,
   RequestMethodValidOnlyForFiles = 40510,
+  ErrorPreparingSimpleSearch = 50010,
 }
 
 export interface LocalRestApiSettings {