artifacty 0.6.0 → 0.7.0

package/README.md

@@ -46,7 +46,7 @@ artifacty stop
 ```
 
 `artifacty start` and `artifacty serve --detach` use the same lifecycle path as `artifacty serve`. Logs are written under `~/.artifacty/logs/`.
-These lifecycle commands use Node's detached process support and work on macOS, Linux, and Windows. `artifacty stop` uses Windows `taskkill` on Windows and process-group signals on macOS/Linux.
+These lifecycle commands use Node's detached process support and work on macOS, Linux, and Windows. `artifacty stop` uses process-group signals on macOS/Linux and Windows `taskkill`, falling back to `/F` when Windows requires forceful termination.
 
 For foreground debugging, keep the process attached:
 
@@ -199,6 +199,8 @@ artifacty start
 artifacty status
 artifacty stop
 artifacty service install --dry-run
+artifacty service unit --dry-run
+artifacty service task --dry-run
 ```
 
 When working from a source checkout without global installation, replace `artifacty` with `node src/cli.js` and `artifacty-mcp` with `node src/mcp-server.js`.

package/docs/integrations.md

@@ -34,10 +34,10 @@ node src/cli.js stop
 The lifecycle commands are intended to be cross-platform:
 
 - macOS and Linux: `stop` signals the detached process group first, then falls back to the server process id.
-- Windows: `start` hides the child console window, and `stop` uses `taskkill /PID <pid> /T`; `--force` adds `/F`.
+- Windows: `start` hides the child console window, and `stop` uses `taskkill /PID <pid> /T`, falling back to `/F` when Windows requires forceful termination. `--force` uses `/F` immediately.
 - All platforms: `status` combines the managed pid file with the HTTP `/health` endpoint, so a stale pid alone is not reported as healthy.
 
-For login/startup persistence, use the operating system's service manager. Artifacty's `service` command currently generates a macOS LaunchAgent plist; Linux systemd user units and Windows Task Scheduler/Service wrappers should be configured explicitly until first-class installers are added.
+For login/startup persistence, use the operating system's service manager. Artifacty's `service` command can generate macOS LaunchAgent, Linux systemd user-unit, and Windows Task Scheduler definitions.
 
 Create artifacts directly in the browser at `http://127.0.0.1:8787/new`.
 
@@ -299,17 +299,33 @@ Renderer notes:
 
 ## Background Service
 
-Generate or install a macOS LaunchAgent plist:
+Generate platform-specific service definitions:
+
+```bash
+node src/cli.js service plist --dry-run
+node src/cli.js service unit --dry-run
+node src/cli.js service task --dry-run
+```
+
+Install the definition for the current OS:
 
 ```bash
-node src/cli.js service plist
 node src/cli.js service install --dry-run
 node src/cli.js service install
+node src/cli.js service uninstall --dry-run
 ```
 
-The generated service runs `src/server.js` with explicit `--host` and `--home` arguments. It includes `--port` only when you configure a port, which keeps the default port fallback available. Load or unload it manually with the `launchctl` commands returned by `service install`.
+Use `--platform macos|linux|windows` when preparing a definition for another host. Use `--path` to choose the output path, or the platform-specific aliases `--plist`, `--unit`, and `--script`.
+
+The generated service runs `src/server.js` with explicit `--host` and `--home` arguments. It includes `--port` only when you configure a port, which keeps the default port fallback available. `--api-token`, `--share-mode`, and `--allow-secrets` are preserved in the generated definition when explicitly provided.
+
+Activation is still delegated to the OS service manager:
+
+- macOS: `launchctl load ~/Library/LaunchAgents/com.artifacty.server.plist`
+- Linux: `systemctl --user enable --now com.artifacty.server.service`
+- Windows: run the generated PowerShell script, which registers and starts the `ArtifactyServer` scheduled task.
 
-For background services, prefer a stable `ARTIFACTY_API_TOKEN` in the service environment. `serve --generate-token` is intended for temporary interactive sessions; the parent CLI returns the generated token in JSON.
+For background services, prefer a stable `ARTIFACTY_API_TOKEN` or `--api-token` value. `serve --generate-token` is intended for temporary interactive sessions; the parent CLI returns the generated token in JSON.
 
 ## Backup and Audit
 

package/docs/release-checklist.md

@@ -25,6 +25,7 @@ This runs syntax checks, the full Node test suite, and a local smoke test that s
 - Require `ARTIFACTY_API_TOKEN` and `ARTIFACTY_SHARE_MODE=lan` or `team` before binding to `0.0.0.0`.
 - Confirm non-loopback startup output includes the LAN/team warning.
 - Confirm `artifacty serve` starts a managed background server and returns prompt-friendly JSON; use `artifacty serve --foreground` for attached log checks.
+- Confirm `artifacty service plist|unit|task --dry-run` renders macOS, Linux, and Windows service definitions.
 - Prefer `x-artifacty-token` or `Authorization: Bearer <token>` over query tokens in scripts.
 - Review secret-scan bypasses. `--allow-secrets` and `ARTIFACTY_ALLOW_SECRETS=true` should be deliberate and temporary.
 - Review [../SECURITY.md](../SECURITY.md) and [threat-model.md](threat-model.md) when changing auth, rendering, MCP, or network-sharing behavior.
@@ -46,4 +47,4 @@ This runs syntax checks, the full Node test suite, and a local smoke test that s
 
 - Export a backup before upgrades: `artifacty backup`.
 - Confirm `artifacty audit --limit 20` shows recent create/update/read/archive events.
-- For macOS background service installs, dry-run first: `artifacty service install --dry-run`.
+- For background service installs, dry-run first: `artifacty service install --dry-run`. Use `--platform macos|linux|windows` to review another OS definition.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "artifacty",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Local artifact exchange for heterogeneous LLM agents via HTTP and MCP.",
   "type": "module",
   "keywords": [
@@ -28,7 +28,7 @@
     "start": "node src/server.js",
     "mcp": "node src/mcp-server.js",
     "test": "node --test",
-    "lint": "node --check src/*.js src/lib/*.js src/client/*.js test/*.test.js",
+    "lint": "node scripts/lint.mjs",
     "smoke": "bash scripts/smoke.sh",
     "release:check": "npm run lint && npm test && npm run smoke"
   },
@@ -42,6 +42,7 @@
     "docs/release-checklist.md",
     "docs/sarif-csv-artifact-plan.md",
     "docs/threat-model.md",
+    "scripts/lint.mjs",
     "scripts/smoke.sh",
     "README.md",
     "LICENSE",

package/scripts/lint.mjs

@@ -0,0 +1,33 @@
+#!/usr/bin/env node
+import { spawnSync } from "node:child_process";
+import { readdir } from "node:fs/promises";
+import path from "node:path";
+
+const targets = [
+  { dir: "src", pattern: /\.js$/ },
+  { dir: path.join("src", "lib"), pattern: /\.js$/ },
+  { dir: path.join("src", "client"), pattern: /\.js$/ },
+  { dir: "test", pattern: /\.test\.js$/ }
+];
+
+const files = [];
+
+for (const target of targets) {
+  const entries = await readdir(target.dir, { withFileTypes: true });
+  for (const entry of entries) {
+    if (entry.isFile() && target.pattern.test(entry.name)) {
+      files.push(path.join(target.dir, entry.name));
+    }
+  }
+}
+
+files.sort();
+
+for (const file of files) {
+  const result = spawnSync(process.execPath, ["--check", file], {
+    stdio: "inherit"
+  });
+  if (result.status !== 0) {
+    process.exit(result.status || 1);
+  }
+}

package/src/cli.js

@@ -280,6 +280,13 @@ async function main() {
       projectDir: options.projectDir || PACKAGE_ROOT,
       serverPath: options.serverPath,
       plistPath: options.plist,
+      unitPath: options.unit,
+      scriptPath: options.script,
+      servicePath: options.path,
+      platform: options.platform,
+      apiToken: options.apiToken,
+      shareMode: options.shareMode,
+      allowSecrets: options.allowSecrets,
       host: options.host,
       port: options.port,
       home: options.home,
@@ -418,7 +425,7 @@ Usage:
   artifacty export --file <path>
   artifacty backup [--file <path>]
   artifacty import-store --file <path>
-  artifacty service plist|install|uninstall [--dry-run] [--plist <path>]
+  artifacty service plist|unit|task|install|uninstall [--platform macos|linux|windows] [--dry-run] [--path <path>]
   artifacty list [--query text] [--tag tag] [--source agent] [--limit 50] [--offset 0] [--include-archived]
   artifacty show <id> [--version n] [--raw]
 

package/src/lib/background.js

@@ -248,10 +248,20 @@ function buildServerEnv(options) {
 async function terminatePid(pid, options = {}) {
   const command = stopCommandForPlatform(pid, options);
   if (command) {
-    await execFileQuiet(command.command, command.args).catch((error) => {
+    await execFileQuiet(command.command, command.args).catch(async (error) => {
       if (!isPidRunning(pid)) {
         return;
       }
+      if (!options.force) {
+        const forced = stopCommandForPlatform(pid, { force: true });
+        await execFileQuiet(forced.command, forced.args).catch((forcedError) => {
+          if (!isPidRunning(pid)) {
+            return;
+          }
+          throw new Error(`Failed to stop Windows process ${pid}: ${forcedError.stderr || forcedError.message}`);
+        });
+        return;
+      }
       throw new Error(`Failed to stop Windows process ${pid}: ${error.stderr || error.message}`);
     });
     return;

package/src/lib/service.js

@@ -1,67 +1,70 @@
 import { mkdir, readFile, rm, writeFile } from "node:fs/promises";
 import { existsSync } from "node:fs";
-import { homedir } from "node:os";
+import { homedir, platform as osPlatform } from "node:os";
 import path from "node:path";
 
 const DEFAULT_LABEL = "com.artifacty.server";
+const DEFAULT_TASK_NAME = "ArtifactyServer";
 
 export async function serviceCommand(action, options = {}) {
-  const plistPath = path.resolve(options.plistPath || path.join(homedir(), "Library", "LaunchAgents", `${DEFAULT_LABEL}.plist`));
-  const plist = createLaunchAgentPlist(options);
+  const definition = serviceDefinition(action, options);
 
-  if (action === "plist") {
-    return { action, path: plistPath, content: plist, dryRun: true };
+  if (definition.renderOnly) {
+    return {
+      action,
+      platform: definition.platform,
+      path: definition.path,
+      content: definition.content,
+      dryRun: true,
+      nextSteps: definition.nextSteps
+    };
   }
 
   if (action === "install") {
-    const existing = await readFile(plistPath, "utf8").catch(() => "");
-    const changed = existing !== plist;
+    const existing = await readFile(definition.path, "utf8").catch(() => "");
+    const changed = existing !== definition.content;
     if (!options.dryRun) {
-      await mkdir(path.dirname(plistPath), { recursive: true });
-      await writeFile(plistPath, plist, "utf8");
+      await mkdir(path.dirname(definition.path), { recursive: true });
+      await writeFile(definition.path, definition.content, "utf8");
     }
     return {
       action,
-      path: plistPath,
+      platform: definition.platform,
+      path: definition.path,
       changed,
       dryRun: Boolean(options.dryRun),
-      content: options.dryRun ? plist : undefined,
-      nextSteps: [
-        `launchctl load ${plistPath}`,
-        `launchctl unload ${plistPath}`
-      ]
+      content: options.dryRun ? definition.content : undefined,
+      nextSteps: definition.nextSteps
     };
   }
 
   if (action === "uninstall") {
-    const existed = existsSync(plistPath);
+    const existed = existsSync(definition.path);
     if (!options.dryRun) {
-      await rm(plistPath, { force: true });
+      await rm(definition.path, { force: true });
     }
     return {
       action,
-      path: plistPath,
+      platform: definition.platform,
+      path: definition.path,
       changed: existed,
-      dryRun: Boolean(options.dryRun)
+      dryRun: Boolean(options.dryRun),
+      nextSteps: definition.uninstallSteps
     };
   }
 
-  throw new Error("service requires action: plist, install, or uninstall");
+  throw new Error("service requires action: plist, unit, task, install, or uninstall");
 }
 
 export function createLaunchAgentPlist(options = {}) {
-  const projectDir = path.resolve(options.projectDir || process.cwd());
-  const nodePath = process.execPath;
-  const serverPath = path.resolve(options.serverPath || path.join(projectDir, "src", "server.js"));
-  const host = options.host || process.env.ARTIFACTY_HOST || "127.0.0.1";
-  const port = options.port || process.env.ARTIFACTY_PORT;
-  const home = path.resolve(options.home || process.env.ARTIFACTY_HOME || path.join(homedir(), ".artifacty"));
-  const logDir = path.join(home, "logs");
-  const portArguments = port
-    ? `    <string>--port</string>
-    <string>${escapeXml(port)}</string>
-`
-    : "";
+  const config = serviceConfig(options);
+  const programArguments = [config.nodePath, ...serverArgs(config)]
+    .map((argument) => `    <string>${escapeXml(argument)}</string>`)
+    .join("\n");
+  const environment = environmentEntries(config)
+    .map(([key, value]) => `    <key>${escapeXml(key)}</key>
+    <string>${escapeXml(value)}</string>`)
+    .join("\n");
 
   return `<?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
@@ -71,31 +74,227 @@ export function createLaunchAgentPlist(options = {}) {
   <string>${DEFAULT_LABEL}</string>
   <key>ProgramArguments</key>
   <array>
-    <string>${escapeXml(nodePath)}</string>
-    <string>${escapeXml(serverPath)}</string>
-    <string>--host</string>
-    <string>${escapeXml(host)}</string>
-${portArguments}    <string>--home</string>
-    <string>${escapeXml(home)}</string>
+${programArguments}
   </array>
   <key>EnvironmentVariables</key>
   <dict>
-    <key>ARTIFACTY_HOME</key>
-    <string>${escapeXml(home)}</string>
+${environment}
   </dict>
   <key>RunAtLoad</key>
   <true/>
   <key>KeepAlive</key>
   <true/>
   <key>StandardOutPath</key>
-  <string>${escapeXml(path.join(logDir, "server.out.log"))}</string>
+  <string>${escapeXml(path.join(config.logDir, "server.out.log"))}</string>
   <key>StandardErrorPath</key>
-  <string>${escapeXml(path.join(logDir, "server.err.log"))}</string>
+  <string>${escapeXml(path.join(config.logDir, "server.err.log"))}</string>
 </dict>
 </plist>
 `;
 }
 
+export function createSystemdUserUnit(options = {}) {
+  const config = serviceConfig(options);
+  const environment = environmentEntries(config)
+    .map(([key, value]) => `Environment=${systemdQuote(`${key}=${value}`)}`)
+    .join("\n");
+  const execStart = [config.nodePath, ...serverArgs(config)]
+    .map(systemdQuote)
+    .join(" ");
+
+  return `[Unit]
+Description=Artifacty local artifact server
+After=network.target
+
+[Service]
+Type=simple
+${environment}
+ExecStart=${execStart}
+Restart=on-failure
+RestartSec=2
+
+[Install]
+WantedBy=default.target
+`;
+}
+
+export function createWindowsTaskScript(options = {}) {
+  const config = serviceConfig(options);
+  const argumentsLine = serverArgs(config, { includeApiTokenArg: true })
+    .map(quoteWindowsArg)
+    .join(" ");
+
+  return `$ErrorActionPreference = 'Stop'
+
+$taskName = ${powershellString(config.taskName)}
+$node = ${powershellString(config.nodePath)}
+$arguments = ${powershellString(argumentsLine)}
+$description = 'Artifacty local artifact server'
+
+$action = New-ScheduledTaskAction -Execute $node -Argument $arguments
+$trigger = New-ScheduledTaskTrigger -AtLogOn
+$settings = New-ScheduledTaskSettingsSet -RestartCount 3 -RestartInterval (New-TimeSpan -Minutes 1) -AllowStartIfOnBatteries -DontStopIfGoingOnBatteries
+
+Register-ScheduledTask -TaskName $taskName -Action $action -Trigger $trigger -Settings $settings -Description $description -Force | Out-Null
+Start-ScheduledTask -TaskName $taskName
+
+Write-Host "Installed and started scheduled task '$taskName'."
+Write-Host "Stop: schtasks /End /TN $taskName"
+Write-Host "Remove: schtasks /Delete /TN $taskName /F"
+`;
+}
+
+function serviceDefinition(action, options = {}) {
+  if (action === "plist") {
+    return definitionForPlatform("macos", options, true);
+  }
+  if (action === "unit") {
+    return definitionForPlatform("linux", options, true);
+  }
+  if (action === "task") {
+    return definitionForPlatform("windows", options, true);
+  }
+  if (action === "install" || action === "uninstall") {
+    return definitionForPlatform(normalizePlatform(options.platform), options, false);
+  }
+  throw new Error("service requires action: plist, unit, task, install, or uninstall");
+}
+
+function definitionForPlatform(platform, options = {}, renderOnly) {
+  const config = serviceConfig({ ...options, platform });
+  if (platform === "macos") {
+    return {
+      platform,
+      renderOnly,
+      path: servicePath(platform, options),
+      content: createLaunchAgentPlist(options),
+      nextSteps: [
+        `launchctl load ${shellQuote(servicePath(platform, options))}`,
+        `launchctl unload ${shellQuote(servicePath(platform, options))}`
+      ],
+      uninstallSteps: [
+        `launchctl unload ${shellQuote(servicePath(platform, options))}`
+      ]
+    };
+  }
+  if (platform === "linux") {
+    return {
+      platform,
+      renderOnly,
+      path: servicePath(platform, options),
+      content: createSystemdUserUnit(options),
+      nextSteps: [
+        "systemctl --user daemon-reload",
+        `systemctl --user enable --now ${path.basename(servicePath(platform, options))}`,
+        `systemctl --user status ${path.basename(servicePath(platform, options))}`,
+        `journalctl --user -u ${path.basename(servicePath(platform, options))} -f`,
+        `systemctl --user disable --now ${path.basename(servicePath(platform, options))}`
+      ],
+      uninstallSteps: [
+        "systemctl --user daemon-reload",
+        `systemctl --user disable --now ${path.basename(servicePath(platform, options))}`
+      ]
+    };
+  }
+  if (platform === "windows") {
+    return {
+      platform,
+      renderOnly,
+      path: servicePath(platform, options),
+      content: createWindowsTaskScript(options),
+      nextSteps: [
+        `powershell -ExecutionPolicy Bypass -File ${quoteWindowsArg(servicePath(platform, options))}`,
+        `schtasks /Query /TN ${config.taskName}`,
+        `schtasks /End /TN ${config.taskName}`,
+        `schtasks /Delete /TN ${config.taskName} /F`
+      ],
+      uninstallSteps: [
+        `schtasks /Delete /TN ${config.taskName} /F`
+      ]
+    };
+  }
+  throw new Error(`Unsupported service platform: ${platform}`);
+}
+
+function serviceConfig(options = {}) {
+  const projectDir = path.resolve(options.projectDir || process.cwd());
+  const home = path.resolve(options.home || process.env.ARTIFACTY_HOME || path.join(homedir(), ".artifacty"));
+  return {
+    projectDir,
+    nodePath: path.resolve(options.nodePath || process.execPath),
+    serverPath: path.resolve(options.serverPath || path.join(projectDir, "src", "server.js")),
+    host: options.host || process.env.ARTIFACTY_HOST || "127.0.0.1",
+    port: options.port !== undefined && options.port !== null ? String(options.port) : process.env.ARTIFACTY_PORT || "",
+    home,
+    logDir: path.join(home, "logs"),
+    apiToken: options.apiToken || "",
+    shareMode: options.shareMode || "",
+    allowSecrets: Boolean(options.allowSecrets),
+    taskName: options.taskName || DEFAULT_TASK_NAME
+  };
+}
+
+function serverArgs(config, options = {}) {
+  const args = [
+    config.serverPath,
+    "--host",
+    config.host,
+    "--home",
+    config.home
+  ];
+  if (config.port) {
+    args.push("--port", String(config.port));
+  }
+  if (config.shareMode) {
+    args.push("--share-mode", config.shareMode);
+  }
+  if (config.allowSecrets) {
+    args.push("--allow-secrets");
+  }
+  if (options.includeApiTokenArg && config.apiToken) {
+    args.push("--api-token", config.apiToken);
+  }
+  return args;
+}
+
+function environmentEntries(config) {
+  const entries = [["ARTIFACTY_HOME", config.home]];
+  if (config.apiToken) {
+    entries.push(["ARTIFACTY_API_TOKEN", config.apiToken]);
+  }
+  return entries;
+}
+
+function servicePath(platform, options = {}) {
+  if (options.servicePath || options.path) {
+    return path.resolve(options.servicePath || options.path);
+  }
+  if (platform === "macos") {
+    return path.resolve(options.plistPath || path.join(homedir(), "Library", "LaunchAgents", `${DEFAULT_LABEL}.plist`));
+  }
+  if (platform === "linux") {
+    return path.resolve(options.unitPath || path.join(homedir(), ".config", "systemd", "user", `${DEFAULT_LABEL}.service`));
+  }
+  if (platform === "windows") {
+    const localAppData = process.env.LOCALAPPDATA || path.join(homedir(), "AppData", "Local");
+    return path.resolve(options.scriptPath || path.join(localAppData, "Artifacty", "install-artifacty-server-task.ps1"));
+  }
+  throw new Error(`Unsupported service platform: ${platform}`);
+}
+
+function normalizePlatform(value = osPlatform()) {
+  if (value === "darwin" || value === "mac" || value === "macos") {
+    return "macos";
+  }
+  if (value === "win32" || value === "windows" || value === "win") {
+    return "windows";
+  }
+  if (value === "linux") {
+    return "linux";
+  }
+  throw new Error("service platform must be macos, linux, or windows");
+}
+
 function escapeXml(value) {
   return String(value)
     .replaceAll("&", "&amp;")
@@ -103,3 +302,23 @@ function escapeXml(value) {
     .replaceAll(">", "&gt;")
     .replaceAll('"', "&quot;");
 }
+
+function systemdQuote(value) {
+  return `"${String(value).replaceAll("\\", "\\\\").replaceAll('"', '\\"')}"`;
+}
+
+function powershellString(value) {
+  return `'${String(value).replaceAll("'", "''")}'`;
+}
+
+function quoteWindowsArg(value) {
+  const text = String(value);
+  if (!/[\s"]/.test(text)) {
+    return text;
+  }
+  return `"${text.replaceAll('"', '\\"')}"`;
+}
+
+function shellQuote(value) {
+  return `'${String(value).replaceAll("'", "'\\''")}'`;
+}