refactor: Move getHelpfeels function to a separate file

Author: takker99

websocket/getHelpfeels.ts

@@ -0,0 +1,30 @@
+import type { BaseLine } from "@cosense/types/userscript";
+
+/** Extract Helpfeel entries from text
+ *
+ * Helpfeel is a Scrapbox notation for questions and help requests.
+ * Lines starting with "?" are considered Helpfeel entries and are
+ * used to collect questions and support requests within a project.
+ *
+ * ```ts
+ * import { assertEquals } from "@std/assert/equals";
+ *
+ * const text = `test page
+ * [normal]link
+ *  but \`this [link]\` is not a link
+ *
+ * code:code
+ *  Links [link] and images [https://scrapbox.io/files/65f29c0c9045b5002522c8bb.svg] in code blocks should be ignored
+ *
+ *    ? Need help with setup!!
+ * `;
+ *
+ * assertEquals(getHelpfeels(text.split("\n").map((text) => ({ text }))), [
+ *   "Need help with setup!!",
+ * ]);
+ * ```
+ */
+export const getHelpfeels = (lines: Pick<BaseLine, "text">[]): string[] =>
+  lines.flatMap(({ text }) =>
+    /^\s*\? .*$/.test(text) ? [text.trimStart().slice(2)] : []
+  );

websocket/getPageMetadataFromLines.ts

@@ -1,5 +1,4 @@
 import { type Node, parse } from "@progfay/scrapbox-parser";
-import type { BaseLine } from "@cosense/types/userscript";
 import { toTitleLc } from "../title.ts";
 import { parseYoutube } from "../parser/youtube.ts";
 
@@ -279,32 +278,3 @@ const makeInlineCodeForDescription = (text: string): `\`${string}\`` =>
   `\`${text.trim().replaceAll("`", "\\`").slice(0, 198)}\``;
 
 const cutId = (link: string): string => link.replace(/#[a-f\d]{24,32}$/, "");
-
-/** Extract Helpfeel entries from text
- *
- * Helpfeel is a Scrapbox notation for questions and help requests.
- * Lines starting with "?" are considered Helpfeel entries and are
- * used to collect questions and support requests within a project.
- *
- * ```ts
- * import { assertEquals } from "@std/assert/equals";
- *
- * const text = `test page
- * [normal]link
- *  but \`this [link]\` is not a link
- *
- * code:code
- *  Links [link] and images [https://scrapbox.io/files/65f29c0c9045b5002522c8bb.svg] in code blocks should be ignored
- *
- *    ? Need help with setup!!
- * `;
- *
- * assertEquals(getHelpfeels(text.split("\n").map((text) => ({ text }))), [
- *   "Need help with setup!!",
- * ]);
- * ```
- */
-export const getHelpfeels = (lines: Pick<BaseLine, "text">[]): string[] =>
-  lines.flatMap(({ text }) =>
-    /^\s*\? .*$/.test(text) ? [text.trimStart().slice(2)] : []
-  );

websocket/makeChanges.ts

@@ -1,12 +1,10 @@
 import { diffToChanges } from "./diffToChanges.ts";
 import type { Page } from "@cosense/types/rest";
 import type { ChangeToPush } from "@cosense/types/websocket";
-import {
-  getHelpfeels,
-  getPageMetadataFromLines,
-} from "./getPageMetadataFromLines.ts";
+import { getPageMetadataFromLines } from "./getPageMetadataFromLines.ts";
 import { isSameArray } from "./isSameArray.ts";
 import { isString } from "@core/unknownutil/is/string";
+import { getHelpfeels } from "./getHelpfeels.ts";
 
 export function* makeChanges(
   before: Page,