@lorekit/cli 1.6.0 → 1.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lorekit/cli",
-  "version": "1.6.0",
+  "version": "1.7.0",
   "description": "Install the LoreKit shared-memory skill and run health checks for the LoreKit MCP server.",
   "license": "MIT",
   "repository": {

package/src/doctor.mjs

@@ -24,8 +24,9 @@ export async function doctor(args) {
   const root = resolveProjectRoot(args.dir);
   let failures = 0;
   let warnings = 0;
+  const failedChecks = [];
   const record = (kind, label, detail) => {
-    if (kind === 'fail') failures++;
+    if (kind === 'fail') { failures++; failedChecks.push(label); }
     if (kind === 'warn') warnings++;
     status(kind, label, detail);
   };
@@ -91,7 +92,8 @@ export async function doctor(args) {
       }.`,
     );
   }
-  return failures === 0 ? 0 : 1;
+  const exitCode = failures === 0 ? 0 : 1;
+  return { exitCode, 'lorekit.cli.doctor.failed_checks': failedChecks };
 }
 
 // Merge doctor's --endpoint / --token flags into the env the resolver reads, so

package/src/telemetry.mjs

@@ -135,12 +135,13 @@ function resourceAttributes(version) {
  * Collect the bounded, non-PII attributes for a command invocation. Only the
  * command name, allow-listed boolean flags, the outcome and the exit code.
  */
-export function commandAttributes({ command, args = {}, outcome, exitCode }) {
+export function commandAttributes({ command, args = {}, outcome, exitCode, extraAttrs = {} }) {
   const attrs = { 'lorekit.cli.command': command, 'lorekit.cli.outcome': outcome };
   if (typeof exitCode === 'number') attrs['lorekit.cli.exit_code'] = exitCode;
   for (const flag of FLAG_ATTRS) {
     if (args[flag]) attrs[`lorekit.cli.flag.${flag}`] = true;
   }
+  Object.assign(attrs, extraAttrs);
   return attrs;
 }
 
@@ -280,8 +281,23 @@ export async function traceCommand(command, args, version, run) {
   let exitCode = 0;
   let status = 'ok';
   let statusMessage;
+  let extraAttrs = {};
   try {
-    exitCode = await run();
+    const result = await run();
+    // Commands may return either a plain exit code (number) or an object with
+    // { exitCode, ...extra } — the latter lets commands surface bounded,
+    // non-PII diagnostic fields (e.g. which checks failed in `doctor`).
+    if (result !== null && typeof result === 'object') {
+      exitCode = result.exitCode ?? 0;
+      const { exitCode: _ec, ...rest } = result;
+      // Flatten any array-valued extras to a comma-joined string so they fit
+      // the flat attribute bag shape (OTLP stringValue).
+      for (const [k, v] of Object.entries(rest)) {
+        extraAttrs[k] = Array.isArray(v) ? v.join(',') : v;
+      }
+    } else {
+      exitCode = result ?? 0;
+    }
     if (typeof exitCode === 'number' && exitCode !== 0) {
       status = 'error';
       statusMessage = `exit ${exitCode}`;
@@ -308,6 +324,7 @@ export async function traceCommand(command, args, version, run) {
         args,
         outcome: status === 'error' ? 'error' : 'ok',
         exitCode: typeof exitCode === 'number' ? exitCode : undefined,
+        extraAttrs,
       });
       await exportInvocation(config, {
         version,