feat: add model-specific instructions

Author: root

src/node/services/systemMessage.test.ts

@@ -269,7 +269,8 @@ General details only.
     );
 
     expect(systemMessage).toContain("<model-openai-gpt-5-1-codex>");
-    expect(systemMessage).toContain("OpenAI's GPT-5.1 Codex models already default to terse replies.");
+    expect(systemMessage).toContain(
+      "OpenAI's GPT-5.1 Codex models already default to terse replies."
+    );
   });
-
 });

src/node/utils/main/markdown.ts

@@ -1,43 +1,37 @@
 import MarkdownIt from "markdown-it";
 
-/**
- * Extract the content under a heading titled "Mode: <mode>" (case-insensitive).
- * - Matches any heading level (#..######)
- * - Returns raw markdown content between this heading and the next heading
- *   of the same or higher level in the same document
- * - If multiple sections match, the first one wins
- * - The heading line itself is excluded from the returned content
- */
-export function extractModeSection(markdown: string, mode: string): string | null {
-  if (!markdown || !mode) return null;
+type HeadingMatcher = (headingText: string, level: number) => boolean;
+
+function extractSectionByHeading(
+  markdown: string,
+  headingMatcher: HeadingMatcher
+): string | null {
+  if (!markdown) return null;
 
   const md = new MarkdownIt({ html: false, linkify: false, typographer: false });
   const tokens = md.parse(markdown, {});
   const lines = markdown.split(/\r?\n/);
-  const target = `mode: ${mode}`.toLowerCase();
 
   for (let i = 0; i < tokens.length; i++) {
-    const t = tokens[i];
-    if (t.type !== "heading_open") continue;
+    const token = tokens[i];
+    if (token.type !== "heading_open") continue;
 
-    const level = Number(t.tag?.replace(/^h/, "")) || 1;
+    const level = Number(token.tag?.replace(/^h/, "")) || 1;
     const inline = tokens[i + 1];
     if (inline?.type !== "inline") continue;
 
-    const text = (inline.content || "").trim().toLowerCase();
-    if (text !== target) continue;
+    const headingText = (inline.content || "").trim();
+    if (!headingMatcher(headingText, level)) continue;
 
-    // Start content after the heading block ends
-    const headingEndLine = inline.map?.[1] ?? t.map?.[1] ?? (t.map?.[0] ?? 0) + 1;
+    const headingEndLine = inline.map?.[1] ?? token.map?.[1] ?? (token.map?.[0] ?? 0) + 1;
 
-    // Find the next heading of same or higher level to bound the section
-    let endLine = lines.length; // exclusive
+    let endLine = lines.length;
     for (let j = i + 1; j < tokens.length; j++) {
-      const tt = tokens[j];
-      if (tt.type === "heading_open") {
-        const nextLevel = Number(tt.tag?.replace(/^h/, "")) || 1;
+      const nextToken = tokens[j];
+      if (nextToken.type === "heading_open") {
+        const nextLevel = Number(nextToken.tag?.replace(/^h/, "")) || 1;
         if (nextLevel <= level) {
-          endLine = tt.map?.[0] ?? endLine;
+          endLine = nextToken.map?.[0] ?? endLine;
           break;
         }
       }
@@ -50,6 +44,16 @@ export function extractModeSection(markdown: string, mode: string): string | nul
   return null;
 }
 
+/**
+ * Extract the content under a heading titled "Mode: <mode>" (case-insensitive).
+ */
+export function extractModeSection(markdown: string, mode: string): string | null {
+  if (!markdown || !mode) return null;
+
+  const expectedHeading = `mode: ${mode}`.toLowerCase();
+  return extractSectionByHeading(markdown, (headingText) => headingText.toLowerCase() === expectedHeading);
+}
+
 /**
  * Extract the first section whose heading matches "Model: <regex>" and whose regex matches
  * the provided model identifier. Matching is case-insensitive by default unless the regex
@@ -58,16 +62,12 @@ export function extractModeSection(markdown: string, mode: string): string | nul
 export function extractModelSection(markdown: string, modelId: string): string | null {
   if (!markdown || !modelId) return null;
 
-  const md = new MarkdownIt({ html: false, linkify: false, typographer: false });
-  const tokens = md.parse(markdown, {});
-  const lines = markdown.split(/\r?\n/);
   const headingPattern = /^model:\s*(.+)$/i;
 
   const compileRegex = (pattern: string): RegExp | null => {
     const trimmed = pattern.trim();
     if (!trimmed) return null;
 
-    // Allow optional /pattern/flags syntax; default to case-insensitive matching otherwise
     if (trimmed.startsWith("/") && trimmed.lastIndexOf("/") > 0) {
       const lastSlash = trimmed.lastIndexOf("/");
       const source = trimmed.slice(1, lastSlash);
@@ -86,38 +86,10 @@ export function extractModelSection(markdown: string, modelId: string): string |
     }
   };
 
-  for (let i = 0; i < tokens.length; i++) {
-    const token = tokens[i];
-    if (token.type !== "heading_open") continue;
-
-    const level = Number(token.tag?.replace(/^h/, "")) || 1;
-    const inline = tokens[i + 1];
-    if (inline?.type !== "inline") continue;
-
-    const match = headingPattern.exec((inline.content || "").trim());
-    if (!match) continue;
-
+  return extractSectionByHeading(markdown, (headingText) => {
+    const match = headingPattern.exec(headingText);
+    if (!match) return false;
     const regex = compileRegex(match[1] ?? "");
-    if (!regex) continue;
-    if (!regex.test(modelId)) continue;
-
-    const headingEndLine = inline.map?.[1] ?? token.map?.[1] ?? (token.map?.[0] ?? 0) + 1;
-
-    let endLine = lines.length;
-    for (let j = i + 1; j < tokens.length; j++) {
-      const nextToken = tokens[j];
-      if (nextToken.type === "heading_open") {
-        const nextLevel = Number(nextToken.tag?.replace(/^h/, "")) || 1;
-        if (nextLevel <= level) {
-          endLine = nextToken.map?.[0] ?? endLine;
-          break;
-        }
-      }
-    }
-
-    const slice = lines.slice(headingEndLine, endLine).join("\n").trim();
-    return slice.length > 0 ? slice : null;
-  }
-
-  return null;
+    return Boolean(regex && regex.test(modelId));
+  });
 }