🪛 Refactoring and validation

src/client/actions/fileops.actions.tsx

@@ -11,9 +11,8 @@ import {
   CV_CLEANUP,
 } from "../constants/action-types";
 
-import { refineQuery } from "../shared/utils/nlp.utils";
+import { refineQuery } from "../shared/utils/filenameparser.utils";
 import { matchScorer } from "../shared/utils/searchmatchscorer.utils";
-import { assign, isNull } from "lodash";
 
 export async function walkFolder(path: string): Promise<Array<IFolderData>> {
   return axios

src/client/components/Card.tsx

@@ -6,6 +6,7 @@ import {
 } from "../shared/utils/formatting.utils";
 import { isUndefined, isEmpty } from "lodash";
 import { Link } from "react-router-dom";
+import ellipsize from "ellipsize";
 
 interface IProps {
   comicBookCoversMetadata: IExtractedComicBookCoverFile;
@@ -43,7 +44,9 @@ class Card extends React.Component<IProps, IState> {
             <div className="card-content">
               <ul>
                 <Link to={"/comic/details/" + this.props.mongoObjId}>
-                  <li className="has-text-weight-semibold">{metadata.name}</li>
+                  <li className="has-text-weight-semibold">
+                    {ellipsize(metadata.name, 18)}
+                  </li>
                 </Link>
               </ul>
             </div>

src/client/shared/utils/filenameparser.utils.ts

@@ -3,6 +3,7 @@ import { default as dates } from "compromise-dates";
 import { default as sentences } from "compromise-sentences";
 import { default as numbers } from "compromise-numbers";
 import xregexp from "xregexp";
+import voca from "voca";
 import { map, xor, isEmpty, isNull } from "lodash";
 
 nlp.extend(sentences);
@@ -88,22 +89,23 @@ export const tokenize = (inputString: string) => {
   // also, if the name has 3-6, remove the -6 part.  note that we'll
   // try to handle the word "of" in a few common languages, like french/
   // spanish (de), italian (di), german (von), dutch (van) or polish (z)
-  replaceRecursive(inputString, "\\(", "\\)", (match) => "");
-  replaceRecursive(inputString, "\\[", "\\]", (match) => "");
-  replaceRecursive(inputString, "\\{", "\\}", (match) => "");
-  const parantheses = inputString.replace(/\([^\(]*?\)/gi, "");
-  const curlyBraces = inputString.replace(/\{[^\{]*?\}/gi, "");
-  const squareBrackets = inputString.replace(/\[[^\[]*?\]/gi, "");
+  replaceRecursive(inputString, "\\(", "\\)", () => "");
+  replaceRecursive(inputString, "\\[", "\\]", () => "");
+  replaceRecursive(inputString, "\\{", "\\}", () => "");
+  inputString.replace(/\([^\(]*?\)/gi, "");
+  inputString.replace(/\{[^\{]*?\}/gi, "");
+  inputString.replace(/\[[^\[]*?\]/gi, "");
+
+  inputString.replace(/([^\d]+)(\s*(of|de|di|von|van|z)\s*#*\d+)/gi, "");
 
-  const genericNumericRange = inputString.replace(
-    /([^\d]+)(\s*(of|de|di|von|van|z)\s*#*\d+)/gi,
-    "",
-  );
   const hyphenatedIssueRange = inputString.match(/(\d)(-\d+)/gi);
   if (!isNull(hyphenatedIssueRange) && hyphenatedIssueRange.length > 2) {
     let issueNumber = hyphenatedIssueRange[0];
   }
 
+  if (voca.includes(inputString, "_") && !voca.includes(inputString, " ")) {
+    inputString.replace(/[-_#]/gi, "");
+  }
   const readingListIndicators = inputString.match(
     /^\s*\d+(\.\s+?|\s*-?\s*)/gim,
   );
@@ -127,10 +129,7 @@ export const tokenize = (inputString: string) => {
       issueNumbers,
       chapters,
       pageCounts,
-      parantheses,
-      curlyBraces,
-      squareBrackets,
-      genericNumericRange,
+
       readingListIndicators,
       volumes,
     },
@@ -145,6 +144,12 @@ export const tokenize = (inputString: string) => {
   return queryObject;
 };
 
+export const extractNumerals = (inputString: string): string => {
+  // Searches through the given string left-to-right, building an ordered list of
+  //  "issue number-like" re.match objects.  For example, this method finds
+  //  matches substrings like:  3, #4, 5a, 6.00, 10.0b, .5, -1.0
+};
+
 export const refineQuery = (inputString) => {
   const queryObj = tokenize(inputString);
   const removedYears = xor(

src/client/shared/utils/formatting.utils.ts

@@ -6,6 +6,5 @@ export const removeLeadingPeriod = (input: string): string => {
 };
 
 export const escapePoundSymbol = (input: string): string => {
-  console.log(input.replace(/\#/gi, "%23"));
   return input.replace(/\#/gi, "%23");
 };

src/client/shared/utils/searchmatchscorer.utils.ts

@@ -8,7 +8,6 @@ export const matchScorer = (searchMatches, searchQuery) => {
   // 4. Check if issue covers hash match strongly (score: +++)
   // 5. Check if issue year matches strongly (score: +)
   const score = 0;
-  console.log("yedvadkar", searchMatches);
   each(searchMatches, (match, idx) => {
     if (!isNull(searchQuery.issue.meta.normalized) && !isNull(match.name)) {
       const issueNameScore = stringSimilarity.compareTwoStrings(