arisa 5.1.65 → 5.1.66

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arisa",
-  "version": "5.1.65",
+  "version": "5.1.66",
   "description": "Telegram + Pi Agent modular assistant",
   "type": "module",
   "main": "src/index.js",

package/src/core/tools/official-tool-installer.js

@@ -11,7 +11,6 @@ import {
   rename,
   rm
 } from "node:fs/promises";
-import os from "node:os";
 import path from "node:path";
 import { getToolDir } from "../../runtime/paths.js";
 import { normalizeToolDependencies, resolveToolDependencyPlan, satisfiesToolVersion } from "./tool-dependencies.js";
@@ -181,7 +180,7 @@ export async function installLockedOfficialTool({
   toolName,
   lock,
   destination,
-  scratchRoot = os.tmpdir(),
+  scratchRoot,
   checkout = checkoutRepository,
   installDependencies = installPackageDependencies,
   validate = validateEntrypoint
@@ -190,7 +189,9 @@ export async function installLockedOfficialTool({
   if (await exists(destination)) {
     throw new Error(`Refusing to overwrite installed tool: ${destination}`);
   }
-  const scratch = await mkdtemp(path.join(scratchRoot, `arisa-${toolName}-`));
+  const stagingRoot = scratchRoot || path.dirname(destination);
+  await mkdir(stagingRoot, { recursive: true });
+  const scratch = await mkdtemp(path.join(stagingRoot, `.arisa-${toolName}-`));
   const checkoutDir = path.join(scratch, "repository");
   const stageDir = path.join(scratch, "stage");
   try {

package/test/official-tool-installer.test.js

@@ -84,6 +84,7 @@ test("installs a verified staged tree without overwriting an existing tool", asy
   const { root, source, files } = await fixture(t);
   const destination = path.join(root, "installed", "master-slave");
   const checkout = async ({ checkoutDir }) => {
+    assert.equal(path.dirname(path.dirname(checkoutDir)), path.dirname(destination));
     await mkdir(path.join(checkoutDir, "tools"), { recursive: true });
     await cp(source, path.join(checkoutDir, "tools", "master-slave"), { recursive: true });
   };
@@ -92,7 +93,6 @@ test("installs a verified staged tree without overwriting an existing tool", asy
     toolName: "master-slave",
     lock: lock(files),
     destination,
-    scratchRoot: root,
     checkout,
     installDependencies: async () => { lifecycle.push("dependencies"); },
     validate: async () => { lifecycle.push("validate"); }