@oh-my-pi/pi-utils 14.9.9 → 15.0.0

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-utils",
-	"version": "14.9.9",
+	"version": "15.0.0",
 	"description": "Shared utilities for pi packages",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -38,7 +38,7 @@
 	},
 	"devDependencies": {
 		"@types/bun": "^1.3.13",
-		"@oh-my-pi/pi-natives": "14.9.9"
+		"@oh-my-pi/pi-natives": "15.0.0"
 	},
 	"engines": {
 		"bun": ">=1.3.7"

package/src/dirs.ts

@@ -324,6 +324,17 @@ export function getGpuCachePath(): string {
 	return dirs.rootSubdir("gpu_cache.json", "cache");
 }
 
+/**
+ * Get the GitHub view cache database path (~/.omp/cache/github-cache.db).
+ * Honors the `OMP_GITHUB_CACHE_DB` env var when set so tests can isolate the
+ * cache file without touching the rest of the config root.
+ */
+export function getGithubCacheDbPath(): string {
+	const override = process.env.OMP_GITHUB_CACHE_DB;
+	if (override) return override;
+	return dirs.rootSubdir(path.join("cache", "github-cache.db"), "cache");
+}
+
 /** Get the natives directory (~/.omp/natives). */
 export function getNativesDir(): string {
 	return dirs.rootSubdir("natives", "cache");

package/src/prompt.ts

@@ -8,7 +8,7 @@ export type PromptRenderPhase = "pre-render" | "post-render";
 export interface PromptFormatOptions {
 	renderPhase?: PromptRenderPhase;
 	replaceAsciiSymbols?: boolean;
-	boldRfc2119Keywords?: boolean;
+	normalizeRfc2119?: boolean;
 }
 
 // Opening XML tag (not self-closing, not closing)
@@ -22,21 +22,27 @@ const TABLE_ROW = /^\|.*\|$/;
 // Table separator (|---|---|)
 const TABLE_SEP = /^\|[-:\s|]+\|$/;
 
-/** RFC 2119 keywords used in prompts. */
-const RFC2119_KEYWORDS = /\b(?:MUST NOT|SHOULD NOT|SHALL NOT|RECOMMENDED|REQUIRED|OPTIONAL|SHOULD|SHALL|MUST|MAY)\b/g;
-
-function boldRfc2119Keywords(line: string): string {
-	return line.replace(RFC2119_KEYWORDS, (match, offset, source) => {
-		const isAlreadyBold =
-			source[offset - 2] === "*" &&
-			source[offset - 1] === "*" &&
-			source[offset + match.length] === "*" &&
-			source[offset + match.length + 1] === "*";
-		if (isAlreadyBold) {
-			return match;
-		}
-		return `**${match}**`;
-	});
+/**
+ * RFC 2119 keywords (plus project aliases NEVER/AVOID) wrapped in markdown bold
+ * — `**MUST**`, `**MUST NOT**`, `**NEVER**`, etc.
+ */
+const RFC2119_BOLD = /\*\*(MUST NOT|SHOULD NOT|RECOMMENDED|REQUIRED|OPTIONAL|SHOULD|MUST|MAY|NEVER|AVOID)\*\*/g;
+
+/**
+ * Normalize RFC 2119 markers per project convention:
+ *   - Strip `**KEYWORD**` bold (visual noise, no semantics).
+ *   - Alias `MUST NOT` → `NEVER` and `SHOULD NOT` → `AVOID` (single-token equivalents).
+ * Skips spans inside inline code (`` `…` ``) so alias definitions can be quoted literally.
+ */
+function normalizeRfc2119(line: string): string {
+	const segments = line.split("`");
+	for (let i = 0; i < segments.length; i += 2) {
+		segments[i] = segments[i]
+			.replace(RFC2119_BOLD, "$1")
+			.replace(/\bMUST NOT\b/g, "NEVER")
+			.replace(/\bSHOULD NOT\b/g, "AVOID");
+	}
+	return segments.join("`");
 }
 
 /** Compact a table row by trimming cell padding */
@@ -75,7 +81,7 @@ export function format(content: string, options: PromptFormatOptions = {}): stri
 	const {
 		renderPhase = "post-render",
 		replaceAsciiSymbols = false,
-		boldRfc2119Keywords: shouldBoldRfc2119 = false,
+		normalizeRfc2119: shouldNormalizeRfc2119 = false,
 	} = options;
 	const isPreRender = renderPhase === "pre-render";
 	const lines = content.split("\n");
@@ -125,8 +131,8 @@ export function format(content: string, options: PromptFormatOptions = {}): stri
 			line = `${leadingWhitespace}${compactTableRow(trimmedStart)}`;
 		}
 
-		if (shouldBoldRfc2119) {
-			line = boldRfc2119Keywords(line);
+		if (shouldNormalizeRfc2119) {
+			line = normalizeRfc2119(line);
 		}
 
 		if (trimmed === "") {