unitup 0.0.6 → 0.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "unitup",
-  "version": "0.0.6",
+  "version": "0.0.7",
   "description": "Minimal systemd user service wrapper for Node.js scripts",
   "main": "src/index.js",
   "types": "./index.d.ts",

package/src/systemd.js

@@ -479,42 +479,60 @@ export async function listServices() {
 export async function runJournalctlLogs(name, opts = {}) {
   const safeName = sanitizeServiceName(name);
   const unitFilename = getUnitFilename(safeName);
-
-  const baseArgs = ['-u', unitFilename];
-
-  if (opts.cat || opts.output === 'cat') {
-    baseArgs.push('-o', 'cat');
-  } else if (opts.output) {
-    baseArgs.push('-o', opts.output);
-  }
+  const syslogId = `unitup-${safeName}`;
 
   if (opts.follow) {
-    baseArgs.push('-f');
+    const followArgs = ['--user', '-u', unitFilename, '-f'];
+    if (opts.cat || opts.output === 'cat') {
+      followArgs.push('-o', 'cat');
+    } else if (opts.output) {
+      followArgs.push('-o', opts.output);
+    }
     try {
-      const child = spawn('journalctl', ['--user', ...baseArgs], { stdio: 'inherit', shell: false });
+      const child = spawn('journalctl', followArgs, { stdio: 'inherit', shell: false });
       child.on('error', () => {});
       return child;
     } catch {
       return null;
     }
-  } else {
-    baseArgs.push('--no-pager');
+  }
+
+  const candidateArgSets = [
+    ['--user', '-u', unitFilename],
+    ['--user', '-u', `unitup-${safeName}`],
+    ['--user', '-t', syslogId],
+    ['-u', unitFilename],
+    ['-u', `unitup-${safeName}`],
+    ['-t', syslogId]
+  ];
+
+  for (const candidateArgs of candidateArgSets) {
+    const args = [...candidateArgs, '--no-pager'];
+    if (opts.cat || opts.output === 'cat') {
+      args.push('-o', 'cat');
+    } else if (opts.output) {
+      args.push('-o', opts.output);
+    }
     const lines = opts.lines ? String(opts.lines) : '100';
-    baseArgs.push('-n', lines);
-
-    // Try --user mode first
-    const userRes = await runCommand('journalctl', ['--user', ...baseArgs]);
-    const userOut = userRes.stdout || userRes.stderr || '';
-
-    // If user journal files do not exist or returned no entries, fallback to system journal
-    if (!userOut || userOut.includes('No journal files') || userOut.includes('No entries') || userRes.code !== 0) {
-      const sysRes = await runCommand('journalctl', baseArgs);
-      const sysOut = sysRes.stdout || sysRes.stderr || '';
-      if (sysOut && !sysOut.includes('No journal files')) {
-        return sysOut;
-      }
+    args.push('-n', lines);
+
+    const res = await runCommand('journalctl', args);
+    const out = (res.stdout || res.stderr || '').trim();
+
+    if (
+      out &&
+      !out.includes('No journal files') &&
+      !out.includes('No entries') &&
+      !out.includes('0 entries') &&
+      res.code === 0
+    ) {
+      return out;
     }
-
-    return userOut || 'No logs found.';
   }
+
+  // Fallback return
+  const fallbackArgs = ['--user', '-u', unitFilename, '--no-pager', '-n', String(opts.lines || 100)];
+  if (opts.cat || opts.output === 'cat') fallbackArgs.push('-o', 'cat');
+  const defaultRes = await runCommand('journalctl', fallbackArgs);
+  return defaultRes.stdout || defaultRes.stderr || 'No logs found.';
 }