tsoft-cli 3.12.3 → 3.12.4

package/CHANGELOG.md

@@ -5,6 +5,11 @@ All notable changes to T-Soft CLI will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.12.4] - 2026-08-11
+
+### Fixed
+- `theme pull` no longer fails on Windows with `EBUSY: resource busy or locked`. Putting a finished download in place moved the existing theme directory aside first, and Windows refuses to rename a directory that a terminal is sitting in or that has a file open in an editor — so the command failed for anyone whose editor had the theme open, which is the normal case. The theme directory now stays where it is: fresh files are copied in and files no longer present in the archive are removed afterwards, so nothing has to be renamed. macOS and Linux were unaffected because they allow renaming a directory that is in use.
+
 ## [3.12.3] - 2026-08-11
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tsoft-cli",
-  "version": "3.12.3",
+  "version": "3.12.4",
   "description": "Command-line tool for tsoft360 theme development and management",
   "type": "module",
   "main": "src/index.js",

package/src/theme-download.js

@@ -21,79 +21,85 @@ async function stageArchive(stream, stagingPath) {
     await fs.rm(zipPath, { force: true });
 }
 
-/**
- * Tema arşivini indirir, doğrular ve ancak ondan sonra hedefin üzerine açar.
- *
- * Hedefteki dosyalar silinmez; arşivdekiler üzerine yazılır. Bozuk veya yarım
- * inen bir arşiv hedefe hiç dokunmaz.
- *
- * @param {import('stream').Readable} stream - Arşiv veri akışı
- * @param {string} themePath - Arşivin açılacağı dizin
- * @returns {Promise<void>}
- */
-export async function mergeThemeArchive(stream, themePath) {
-    await fs.mkdir(themePath, { recursive: true });
+async function withStaging(themePath, apply) {
+    const parent = path.dirname(themePath);
+    await fs.mkdir(parent, { recursive: true });
 
-    const staging = await fs.mkdtemp(path.join(themePath, '.tsoft-update-'));
+    const staging = await fs.mkdtemp(path.join(parent, `.${path.basename(themePath)}-`));
 
     try {
-        const zipPath = path.join(staging, 'theme.zip');
+        return await apply(staging);
+    } finally {
+        await fs.rm(staging, { recursive: true, force: true });
+    }
+}
 
-        await pipeline(stream, createWriteStream(zipPath));
+async function copyInto(sourceDir, targetDir) {
+    for (const entry of await fs.readdir(sourceDir)) {
+        await fs.cp(path.join(sourceDir, entry), path.join(targetDir, entry), {
+            recursive: true,
+            force: true,
+        });
+    }
+}
 
-        const { size } = await fs.stat(zipPath);
+/**
+ * Arşivde bulunmayan girdileri hedeften siler.
+ *
+ * Dizini yeniden adlandırmak Windows'ta içine girilmişse veya içindeki bir
+ * dosya açıksa EBUSY verir; bu yüzden hedef dizin yerinde bırakılır ve yalnızca
+ * fazlalık girdiler kaldırılır. Kalıcı bir kilit tek bir dosyayı geride
+ * bırakabilir, ama temanın geri kalanı yine güncellenmiş olur.
+ */
+async function removeStaleEntries(themePath, stagingPath) {
+    const fresh = new Set(await fs.readdir(stagingPath));
 
-        if (size === 0) {
-            throw new Error('EMPTY_ARCHIVE');
+    for (const entry of await fs.readdir(themePath)) {
+        if (!fresh.has(entry)) {
+            await fs.rm(path.join(themePath, entry), { recursive: true, force: true });
         }
-
-        const zip = new AdmZip(zipPath);
-        zip.extractAllTo(themePath, true);
-    } finally {
-        await fs.rm(staging, { recursive: true, force: true });
     }
 }
 
 /**
- * Tema arşivini indirir, doğrular ve ancak ondan sonra hedefin üzerine koyar.
+ * Tema arşivini indirir, doğrular ve ancak ondan sonra hedefe yerleştirir.
  *
  * Arşiv hedefin yanındaki geçici dizine açılır; indirme yarıda koparsa veya
- * arşiv bozuksa hedef dizine dokunulmaz. Böylece başarısız bir indirme
- * kullanıcının mevcut tema dosyalarını silmez.
+ * arşiv bozuksa hedef dizine dokunulmaz. Hedef dizinin kendisi taşınmaz,
+ * yalnızca içeriği değiştirilir — dizin bir editörde veya kabukta açıkken de
+ * çalışır.
  *
  * @param {import('stream').Readable} stream - Arşiv veri akışı
  * @param {string} themePath - Temanın açılacağı hedef dizin
  * @returns {Promise<void>}
  */
 export async function extractThemeArchive(stream, themePath) {
-    const parent = path.dirname(themePath);
-    await fs.mkdir(parent, { recursive: true });
-
-    const staging = await fs.mkdtemp(path.join(parent, `.${path.basename(themePath)}-`));
-    const backup = `${staging}-previous`;
-
-    try {
+    await withStaging(themePath, async (staging) => {
         await stageArchive(stream, staging);
-    } catch (error) {
-        await fs.rm(staging, { recursive: true, force: true });
-        throw error;
-    }
 
-    const hadPrevious = await fs.access(themePath).then(() => true).catch(() => false);
+        await fs.mkdir(themePath, { recursive: true });
 
-    if (hadPrevious) {
-        await fs.rename(themePath, backup);
-    }
+        await copyInto(staging, themePath);
+        await removeStaleEntries(themePath, staging);
+    });
+}
 
-    try {
-        await fs.rename(staging, themePath);
-    } catch (error) {
-        if (hadPrevious) {
-            await fs.rename(backup, themePath);
-        }
-        await fs.rm(staging, { recursive: true, force: true });
-        throw error;
-    }
+/**
+ * Tema arşivini indirir, doğrular ve ancak ondan sonra hedefin üzerine açar.
+ *
+ * Hedefteki dosyalar silinmez; arşivdekiler üzerine yazılır. Bozuk veya yarım
+ * inen bir arşiv hedefe hiç dokunmaz.
+ *
+ * @param {import('stream').Readable} stream - Arşiv veri akışı
+ * @param {string} themePath - Arşivin açılacağı dizin
+ * @returns {Promise<void>}
+ */
+export async function mergeThemeArchive(stream, themePath) {
+    await withStaging(themePath, async (staging) => {
+        await stageArchive(stream, staging);
+
+        await fs.mkdir(themePath, { recursive: true });
 
-    await fs.rm(backup, { recursive: true, force: true });
+        await copyInto(staging, themePath);
+    });
 }