@vtstech/pi-diag 1.0.8 → 1.1.0

package/diag.js

@@ -12,18 +12,19 @@ import {
   msHuman,
   pct
 } from "@vtstech/pi-shared/format";
-import { MODELS_JSON_PATH, getOllamaBaseUrl, BUILTIN_PROVIDERS } from "@vtstech/pi-shared/ollama";
+import { MODELS_JSON_PATH, getOllamaBaseUrl, BUILTIN_PROVIDERS, readModelsJson, EXTENSION_VERSION } from "@vtstech/pi-shared/ollama";
 import {
   BLOCKED_COMMANDS,
   BLOCKED_URL_PATTERNS,
   validatePath,
   isSafeUrl,
   sanitizeCommand,
-  readRecentAuditEntries
+  readRecentAuditEntries,
+  AUDIT_LOG_PATH
 } from "@vtstech/pi-shared/security";
 function diag_temp_default(pi) {
   const branding = [
-    `  \u26A1 Pi Diagnostics v1.0.8`,
+    `  \u26A1 Pi Diagnostics v${EXTENSION_VERSION}`,
     `  Written by VTSTech`,
     `  GitHub: https://github.com/VTSTech`,
     `  Website: www.vts-tech.org`
@@ -182,12 +183,10 @@ function diag_temp_default(pi) {
     }
     lines.push(section("MODELS.JSON"));
     const agentDir = path.join(os.homedir(), ".pi", "agent");
-    const modelsJsonPath = MODELS_JSON_PATH;
     let configuredModels = [];
-    let modelsJson = null;
-    if (fs.existsSync(modelsJsonPath)) {
+    const modelsJson = readModelsJson();
+    if (modelsJson && Object.keys(modelsJson.providers || {}).length > 0) {
       try {
-        modelsJson = JSON.parse(fs.readFileSync(modelsJsonPath, "utf-8"));
         const providers = modelsJson.providers || {};
         lines.push(info(`Providers configured: ${Object.keys(providers).length}`));
         for (const [providerName, providerConfig] of Object.entries(providers)) {
@@ -225,7 +224,7 @@ function diag_temp_default(pi) {
         lines.push(fail(`models.json parse error: ${e.message}`));
       }
     } else {
-      lines.push(fail(`models.json not found at ${modelsJsonPath}`));
+      lines.push(fail(`models.json not found at ${MODELS_JSON_PATH}`));
       lines.push(info("  \u2192 Run /ollama-sync to create it"));
     }
     lines.push(section("SETTINGS"));
@@ -370,9 +369,8 @@ function diag_temp_default(pi) {
     }
     lines.push(info("Audit log status:"));
     const auditEntries = readRecentAuditEntries(50);
-    const auditLogPath = path.join(os.homedir(), ".pi", "agent", "audit.log");
-    if (fs.existsSync(auditLogPath)) {
-      lines.push(ok(`Audit log exists: ${auditLogPath}`));
+    if (fs.existsSync(AUDIT_LOG_PATH)) {
+      lines.push(ok(`Audit log exists: ${AUDIT_LOG_PATH}`));
       if (auditEntries.length > 0) {
         lines.push(info(`  Recent entries: ${auditEntries.length} (last 50)`));
         const recentSample = auditEntries.slice(-3);
@@ -385,7 +383,7 @@ function diag_temp_default(pi) {
         lines.push(info("  No audit entries found (log is empty or unparseable)"));
       }
     } else {
-      lines.push(warn(`Audit log not found at ${auditLogPath}`));
+      lines.push(warn(`Audit log not found at ${AUDIT_LOG_PATH}`));
       lines.push(info("  \u2192 Audit logging will begin when security events occur"));
     }
     lines.push(section("CURRENT SESSION"));
@@ -464,7 +462,10 @@ function diag_temp_default(pi) {
     promptGuidelines: [
       "When the user asks for a diagnostic, health check, or system test, call self_diagnostic."
     ],
-    parameters: {},
+    parameters: {
+      type: "object",
+      properties: {}
+    },
     execute: async (_toolCallId, _params, _signal, _onUpdate, ctx) => {
       try {
         const report = await runDiagnostics(ctx);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vtstech/pi-diag",
-  "version": "1.0.8",
+  "version": "1.1.0",
   "description": "Diagnostics extension for Pi Coding Agent",
   "main": "diag.js",
   "keywords": ["pi-extensions"],
@@ -14,7 +14,7 @@
     "url": "https://github.com/VTSTech/pi-coding-agent"
   },
   "dependencies": {
-    "@vtstech/pi-shared": "1.0.8"
+    "@vtstech/pi-shared": "1.1.0"
   },
   "peerDependencies": {
     "@mariozechner/pi-coding-agent": ">=0.66"