@miphamai/cli 0.72.0 → 0.73.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miphamai/cli",
-  "version": "0.72.0",
+  "version": "0.73.0",
   "description": "Mipham Code — Multi-model open-core intelligent coding terminal by MiphamAI",
   "keywords": [
     "ai",

package/src/commands/project.ts

@@ -553,9 +553,7 @@ async function setupStep3(ctx: CommandContext): Promise<CommandResult> {
     lines.push(`  ${p.id}${p.id === ctx.providerId ? ' ← current' : ''}`)
     for (const m of p.models.filter((m) => m.status === 'active')) {
       const marker = m.id === ctx.modelId ? ' ★' : '  '
-      lines.push(
-        `${marker}  ${m.id.padEnd(30)} ${m.contextWindow.toLocaleString()} ctx  ${m.vision ? '🖼' : '📝'}`,
-      )
+      lines.push(`${marker}  ${m.id.padEnd(30)}  ${m.vision ? '🖼' : '📝'}`)
     }
     lines.push('')
   }

package/src/core/permission-rules.ts

@@ -251,8 +251,37 @@ export function wildcardMatch(pattern: string, input: string): boolean {
   return new RegExp(regexStr).test(input)
 }
 
+/**
+ * Validate a rule pattern string's structure, mirroring exactly what
+ * `matchBashRule` / `ruleMatches` will actually match. A pattern that is
+ * syntactically valid but never matches (e.g. `Bash(ls) x`, `Read(foo`,
+ * `Bash()`) is silently dead today — this returns a human-readable reason so
+ * the caller can report it as an invalid setting instead of ignoring it.
+ *
+ * Returns null when the pattern is valid, or a reason string when malformed.
+ */
+export function validateRulePattern(pattern: string): string | null {
+  if (!pattern.trim()) return 'rule pattern is empty'
+
+  if (pattern.includes('(')) {
+    if (!pattern.includes(')')) return 'unclosed parenthesis'
+    // Empty parameter: `Bash()` or `Bash( )`
+    if (/\(\s*\)$/.test(pattern)) return 'empty parameter'
+    // Must be exactly `ToolName(param)` with nothing before or after.
+    if (!/^(\w+)\((.+)\)$/.test(pattern)) {
+      return 'unexpected text after the closing parenthesis'
+    }
+    return null
+  }
+
+  // No parenthesis → must be a plain tool name (matched via `pattern === tool.name`).
+  if (!/^\w+$/.test(pattern)) return 'not a single tool name'
+  return null
+}
+
 /** Compile a rule pattern string into a PermissionRuleEntry. */
 export function compileRule(pattern: string, level: 'allow' | 'deny' | 'ask'): PermissionRuleEntry {
+  const invalid = validateRulePattern(pattern)
   const regexStr =
     '^' +
     pattern
@@ -261,5 +290,5 @@ export function compileRule(pattern: string, level: 'allow' | 'deny' | 'ask'): P
       .replace(/\\\*/g, '.*')
       .replace(/\\\?/g, '.') +
     '$'
-  return { pattern, level, compiled: new RegExp(regexStr) }
+  return { pattern, level, compiled: new RegExp(regexStr), ...(invalid ? { invalid } : {}) }
 }

package/src/core/permission.ts

@@ -489,6 +489,24 @@ export class PermissionSystem {
     this.invalidateCache()
   }
 
+  /**
+   * Report any rule whose pattern is structurally invalid and can therefore
+   * never match (e.g. `Bash(ls) x`, `Read(foo`, `Bash()`). These are silently
+   * dead today — callers should surface them as invalid settings rather than
+   * let a deny rule fail closed without the user noticing.
+   */
+  getInvalidRules(): string[] {
+    const invalid: string[] = []
+    for (const entry of [...this.allowRules, ...this.denyRules, ...this.askRules]) {
+      if (entry.invalid) {
+        invalid.push(
+          `permission rule "${entry.pattern}" is invalid (${entry.invalid}) and will never match`,
+        )
+      }
+    }
+    return invalid
+  }
+
   removeRule(toolName: string): void {
     this.legacyRules.delete(toolName)
     this.removeRuleFromArrays(toolName)

package/src/daemon/server.ts

@@ -95,6 +95,9 @@ export function buildDaemonPermission(
     for (const rule of rules.allow ?? []) permission.allow(rule)
     for (const rule of rules.deny ?? []) permission.deny(rule)
   }
+  for (const msg of permission.getInvalidRules()) {
+    process.stderr.write(`⚠ Mipham Code: ${msg}\n`)
+  }
   return permission
 }
 

package/src/index.tsx

@@ -559,6 +559,12 @@ export async function runApp(options: RunOptions): Promise<void> {
   for (const rule of settingsJson.permissions.allow) engine.getPermission().allow(rule)
   for (const rule of settingsJson.permissions.deny) engine.getPermission().deny(rule)
 
+  // Surface malformed permission rules as warnings instead of silently letting
+  // them never match (e.g. a deny rule typed `Bash(ls) x` would fail closed).
+  for (const msg of engine.getPermission().getInvalidRules()) {
+    process.stderr.write(`⚠ Mipham Code: ${msg}\n`)
+  }
+
   // Initialize agent registry and load plugin agents/skills/MCP/hooks
   const agentRegistry = new AgentRegistry()
   agentRegistry.loadUserAgents()

package/src/shared/format.ts

@@ -0,0 +1,15 @@
+/**
+ * Format a model's context window (token count) into a friendly unit string.
+ * Decimal K/M: 16384 → "16K", 131072 → "131K", 200000 → "200K", 1000000 → "1M".
+ * Used only on the developer-facing `/models` command — the first-run model
+ * picker deliberately omits context window (raw token counts make Mipham's own
+ * models look small next to 1M-context competitors).
+ */
+export function formatContextWindow(tokens: number): string {
+  if (tokens >= 1_000_000) {
+    const m = tokens / 1_000_000
+    return `${Number.isInteger(m) ? m : m.toFixed(1)}M`
+  }
+  if (tokens >= 1_000) return `${Math.round(tokens / 1_000)}K`
+  return `${tokens}`
+}

package/src/shared/mipham-models.json

@@ -11,34 +11,34 @@
   "models": [
     {
       "id": "om-v5-flash",
-      "name": "OM V5 Flash",
+      "name": "om-V5-Flash",
       "providerId": "mipham",
       "contextWindow": 16384,
       "maxOutput": 4096,
       "vision": false,
       "status": "active"
     },
-    {
-      "id": "om-v5-pro",
-      "name": "OM V5 Pro",
-      "providerId": "mipham",
-      "contextWindow": 131072,
-      "maxOutput": 16384,
-      "vision": false,
-      "status": "active"
-    },
     {
       "id": "om-v5-visual",
-      "name": "OM V5 Visual",
+      "name": "om-V5-Visual",
       "providerId": "mipham",
       "contextWindow": 65536,
       "maxOutput": 4096,
       "vision": true,
       "status": "active"
     },
+    {
+      "id": "om-v5-pro",
+      "name": "om-V5-Pro",
+      "providerId": "mipham",
+      "contextWindow": 131072,
+      "maxOutput": 16384,
+      "vision": false,
+      "status": "active"
+    },
     {
       "id": "om-v5-apex",
-      "name": "OM V5 Apex",
+      "name": "om-V5-Apex",
       "providerId": "mipham",
       "contextWindow": 200000,
       "maxOutput": 32768,
@@ -46,5 +46,5 @@
       "status": "active"
     }
   ],
-  "synced_at": "2026-08-04T00:00:00Z"
+  "synced_at": "2026-09-04T02:32:33.479Z"
 }

package/src/shared/package-info.ts

@@ -9,7 +9,7 @@
 export const PACKAGE_NAME = '@miphamai/cli' as const
 
 /** 当前发布版本 */
-export const PACKAGE_VERSION = '0.72.0' as const
+export const PACKAGE_VERSION = '0.73.0' as const
 
 /** npm install 全局安装命令 */
 export const NPM_INSTALL_COMMAND = `npm install -g ${PACKAGE_NAME}` as const

package/src/shared/types.ts

@@ -315,6 +315,8 @@ export interface PermissionRuleEntry {
   pattern: string // e.g., "Bash(git:*)"
   level: 'allow' | 'deny' | 'ask'
   compiled: RegExp
+  /** Set when the pattern is structurally invalid and can never match. */
+  invalid?: string
 }
 
 export interface PermissionRule {

package/src/ui/commands.ts

@@ -6,6 +6,7 @@
  */
 import type { QueryEngine } from '../core/engine'
 import type { MiphamConfig } from '../shared/index.ts'
+import { formatContextWindow } from '../shared/format'
 import type { SkillsLoader } from '../skills/loader'
 import type { PluginManager } from '../plugin/plugin-manager'
 import type { Message } from '../shared/types.js'
@@ -438,7 +439,7 @@ const modelsCmd: CommandHandler = (ctx) => {
         .filter((m) => m.status === 'active')
         .map(
           (m) =>
-            `  ${p.id.padEnd(12)} ${m.id.padEnd(30)} ${m.contextWindow.toLocaleString()} ctx  ${m.vision ? '🖼' : '📝'}`,
+            `  ${p.id.padEnd(12)} ${m.id.padEnd(30)} ${formatContextWindow(m.contextWindow)} ctx  ${m.vision ? '🖼' : '📝'}`,
         ),
     )
 

package/src/ui/picker.tsx

@@ -198,7 +198,7 @@ export function ModelPicker({
                 </Text>
                 <Text dimColor>
                   {'   '}
-                  {m.id} · {m.contextWindow.toLocaleString()} ctx
+                  {m.id}
                   {m.vision ? ' · 🖼 vision' : ''}
                 </Text>
               </Box>