obsidian-plugin-config 1.7.2 → 1.7.3

package/bin/obsidian-inject.js

@@ -3,7 +3,7 @@
 /**
  * Obsidian Plugin Config - CLI Entry Point
  * Global command: obsidian-inject
- * Version: 1.7.2
+ * Version: 1.7.3
  */
 
 import { execSync } from 'child_process';

package/docs/INTERACTIVE_INJECTION.md

@@ -58,7 +58,7 @@ Tous les fichiers du template sont pris en compte à chaque injection (pas de
 sélection par composant) :
 
 - `templates/scripts/*` → `<cible>/scripts/`
-- `templates/tsconfig.json`, `eslint.config.mts`, `.editorconfig`,
+- `templates/tsconfig.json.template`, `eslint.config.mts`, `.editorconfig`,
   `.prettierrc`, `.prettierignore`, `.npmrc`, `.env`
 - `templates/.vscode/*`
 - `templates/.github/workflows/*`

package/docs/LLM-GUIDE.md

@@ -13,8 +13,8 @@
 `templates/` contains everything that gets injected into target plugins:
 
 - `templates/scripts/` — scripts copied into `<target>/scripts/`
-- `templates/package.json` — base deps/scripts merged into `<target>/package.json`
-- `templates/tsconfig.json` — TypeScript config injected as `<target>/tsconfig.json`
+- `templates/package.json.template` — base deps/scripts merged into `<target>/package.json`
+- `templates/tsconfig.json.template` — TypeScript config injected as `<target>/tsconfig.json`
 - `templates/eslint.config.mts` — ESLint config injected into target
 - `templates/.editorconfig`, `templates/.prettierrc`, etc. — config files injected into target
 - `templates/.github/workflows/` — GitHub Actions workflows
@@ -37,7 +37,7 @@
 
 ### 1. Package.json merge
 
-`inject-core.ts → updatePackageJson()` reads `templates/package.json` and merges into the target plugin's `package.json`:
+`inject-core.ts → updatePackageJson()` reads `templates/package.json.template` and merges into the target plugin's `package.json`:
 
 - All `scripts` are overwritten with template values
 - All `devDependencies` from template are added/updated
@@ -51,7 +51,8 @@
 - `templates/scripts/*` → `<target>/scripts/`
 - `templates/tsconfig.json` → `<target>/tsconfig.json`
 - `templates/eslint.config.mts` → `<target>/eslint.config.mts`
-- `templates/.editorconfig`, `.prettierrc`, `.npmrc`, `.env` → `<target>/`
+- `templates/.editorconfig`, `.prettierrc`, `.npmrc`, `.env`, `.gitattributes` → `<target>/`
+- `templates/.vscode/*` → `<target>/.vscode/`
 - `templates/.github/workflows/*` → `<target>/.github/workflows/`
 - `templates/gitignore.template` → `<target>/.gitignore`
 
@@ -95,9 +96,9 @@ The `esbuild-sass-plugin` dependency is **not** injected automatically. If a plu
 
 ## What NOT to do
 
-- ❌ Do not hardcode deps/scripts in `inject-core.ts` — they must come from `templates/package.json`
+- ❌ Do not hardcode deps/scripts in `inject-core.ts` — they must come from `templates/package.json.template`
 - ❌ Do not modify root config files thinking it will affect injected plugins — always modify `templates/`
-- ❌ Do not add `obsidian-plugin-config` as a dependency in `templates/package.json` — injected plugins are standalone
+- ❌ Do not add `obsidian-plugin-config` as a dependency in `templates/package.json.template` — injected plugins are standalone
 - ❌ Do not inject `esbuild.config.ts` without its dependencies (`constants.ts`, `env.ts`, `reload.ts`, `typingsPlugin.ts`, `utils.ts`) — they are all required
 
 ---
@@ -115,7 +116,7 @@ To change what gets injected:
 
 ## obsidian-typings paths
 
-The correct paths for `obsidian-typings` in `templates/tsconfig.json`:
+The correct paths for `obsidian-typings` in `templates/tsconfig.json.template`:
 
 ```json
 "types": ["obsidian-typings"],

package/docs/SCSS-FLOW.md

@@ -0,0 +1,273 @@
+# Comment fonctionne SCSS dans cette config
+
+Ce document explique de bout en bout comment un fichier `.scss` est détecté, compilé puis copié dans le dossier `plugins` d'Obsidian, en s'appuyant sur le code du dépôt.
+
+> Remarque : la configuration SCSS n'est pas activée dans `obsidian-plugin-config` lui-même. Elle est définie dans les **templates** qui sont copiés dans chaque plugin Obsidian lors de l'injection (`yarn inject` / `obsidian-inject`). Tout le code cité ci-dessous vit donc dans `templates/scripts/`.
+
+---
+
+## 1. Vue d'ensemble du flux
+
+```
+src/styles.scss
+      │
+      │  détection (esbuild.config.ts → main())
+      ▼
+entryPoints = [src/main.ts, src/styles.scss]
+      │
+      │  esbuild.context() + sassPlugin() (esbuild.config.ts → createBuildContext)
+      ▼
+buildPath/<id>/
+  ├── main.js
+  └── main.css        ← produit par esbuild-sass-plugin
+      │
+      │  plugin "remove-main-css" (esbuild.config.ts) → utils.removeMainCss
+      │  + plugin "copy-to-plugins-folder" (esbuild.config.ts) → utils.copyFilesToTargetDir
+      ▼
+buildPath/<id>/
+  ├── manifest.json   (copié)
+  ├── main.js
+  └── styles.css      ← renommé depuis main.css ? non : voir point 3
+```
+
+Les deux moments clés sont :
+
+1. **Détection et compilation** dans `templates/scripts/esbuild.config.ts`.
+2. **Nettoyage et copie vers le dossier plugins** via les fonctions utilitaires de `templates/scripts/utils.ts`.
+
+---
+
+## 2. Où passe le fichier SCSS
+
+### 2.1 Détection du fichier SCSS
+
+Dans `templates/scripts/esbuild.config.ts`, fonction `main()` (lignes 115-140) :
+
+```ts
+// Check for SCSS first, then CSS in src, then in root
+const srcStylesScssPath = path.join(pluginDir, 'src/styles.scss');
+const srcStylesPath    = path.join(pluginDir, 'src/styles.css');
+const rootStylesPath   = path.join(pluginDir, 'styles.css');
+
+const scssExists = await isValidPath(srcStylesScssPath);
+const stylePath = scssExists
+  ? srcStylesScssPath
+  : (await isValidPath(srcStylesPath))
+    ? srcStylesPath
+    : (await isValidPath(rootStylesPath))
+      ? rootStylesPath
+      : '';
+
+const mainTsPath = path.join(pluginDir, 'src/main.ts');
+const entryPoints = stylePath ? [mainTsPath, stylePath] : [mainTsPath];
+const context = await createBuildContext(buildPath, isProd, entryPoints, scssExists);
+```
+
+- Le fichier SCSS est donc attendu à `src/styles.scss` (priorité maximale).
+- Sinon, fallback sur `src/styles.css` puis `styles.css` à la racine.
+- `scssExists` (booléen) sert ensuite à activer ou non le plugin SASS dans esbuild.
+
+### 2.2 Compilation via esbuild + sassPlugin
+
+Toujours dans `templates/scripts/esbuild.config.ts`, fonction `createBuildContext()` (lignes 33-69) :
+
+```ts
+const plugins = [
+  // Add SASS plugin if SCSS files are detected
+  ...(hasSass
+    ? [
+        await (async () => {
+          try {
+            // @ts-expect-error - esbuild-sass-plugin is installed during injection
+            const { sassPlugin } = await import('esbuild-sass-plugin');
+            return sassPlugin({
+              syntax: 'scss',
+              style: 'expanded'
+            });
+          } catch (error) {
+            console.warn(
+              '⚠️  esbuild-sass-plugin not found. Install it with: yarn add -D esbuild-sass-plugin'
+            );
+            throw error;
+          }
+        })(),
+        {
+          name: 'remove-main-css',
+          setup(build: esbuild.PluginBuild): void {
+            build.onEnd(async (result) => {
+              if (result.errors.length === 0) {
+                await removeMainCss(buildPath);
+              }
+            });
+          }
+        }
+      ]
+    : []),
+```
+
+- Le `sassPlugin` n'est importé que si `hasSass` est `true` (donc si `src/styles.scss` existe).
+- Il est configuré en syntaxe `scss` et style `expanded` (non minifié, lisible).
+- esbuild produit un fichier `main.css` à côté de `main.js` dans `outdir` (= `buildPath`).
+- Un second plugin esbuild interne, `remove-main-css`, supprime ce `main.css` à la fin du build (voir 3.1).
+
+### 2.3 Dépendance : `esbuild-sass-plugin`
+
+- Import dynamique, donc l'installation de la dépendance n'est pas obligatoire pour les plugins qui n'utilisent pas SCSS.
+- Le code affiche un warning et échoue si SCSS est détecté mais que le plugin n'est pas installé :
+
+  > `⚠️  esbuild-sass-plugin not found. Install it with: yarn add -D esbuild-sass-plugin`
+
+- Côté utilisateur, la doc le précise : `README.md`, section **SASS Support** (lignes 85-96).
+
+---
+
+## 3. Comment c'est copié dans le dossier plugins
+
+### 3.1 Nettoyage du `main.css` parasite
+
+Dans `templates/scripts/utils.ts`, fonction `removeMainCss()` (lignes 169-180) :
+
+```ts
+export async function removeMainCss(outdir: string): Promise<void> {
+  const mainCssPath = path.join(outdir, 'main.css');
+  try {
+    await rm(mainCssPath);
+  } catch (error: unknown) {
+    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
+      console.warn(
+        `Warning: Could not remove main.css: ${error instanceof Error ? error.message : String(error)}`
+      );
+    }
+  }
+}
+```
+
+Pourquoi ? Le `sassPlugin` d'esbuild génère un fichier `main.css` parce qu'il hérite du `name` (`main`) de l'entry point. Or Obsidian attend `styles.css` dans le dossier du plugin. `removeMainCss` supprime donc ce fichier temporaire immédiatement après chaque build (uniquement si le build n'a pas d'erreur, cf. `if (result.errors.length === 0)` dans `esbuild.config.ts`).
+
+### 3.2 Copie vers le dossier plugins
+
+Dans `templates/scripts/utils.ts`, fonction `copyFilesToTargetDir()` (lignes 74-123) :
+
+```ts
+export async function copyFilesToTargetDir(buildPath: string): Promise<void> {
+  const pluginDir = process.cwd();
+  const manifestSrc  = path.join(pluginDir, 'manifest.json');
+  const manifestDest = path.join(buildPath, 'manifest.json');
+  const cssDest      = path.join(buildPath, 'styles.css');
+  const folderToRemove = path.join(buildPath, '_.._');
+  ...
+  // Copy CSS
+  try {
+    const srcStylesPath  = path.join(pluginDir, 'src/styles.css');
+    const rootStylesPath = path.join(pluginDir, 'styles.css');
+
+    if (await isValidPath(srcStylesPath)) {
+      await copyFile(srcStylesPath, cssDest);
+    }
+    else if (await isValidPath(rootStylesPath)) {
+      await copyFile(rootStylesPath, cssDest);
+      if (await isValidPath(folderToRemove)) {
+        await rm(folderToRemove, { recursive: true });
+      }
+    } else {
+      return;
+    }
+  } ...
+}
+```
+
+> Note importante : `copyFilesToTargetDir` ne copie **que** `manifest.json` et un éventuel `styles.css` / `styles.css` racine. Le `main.css` généré par esbuild-sass-plugin est supprimé à l'étape précédente (3.1).
+
+### 3.3 Branchement sur le cycle de vie esbuild
+
+Toujours dans `templates/scripts/esbuild.config.ts`, le plugin `copy-to-plugins-folder` (lignes 70-94) déclenche la copie après chaque build :
+
+```ts
+{
+  name: 'copy-to-plugins-folder',
+  setup: (build: esbuild.PluginBuild): void => {
+    build.onEnd(async () => {
+      if (isProd) {
+        if (process.argv.includes('-r') || process.argv.includes('real')) {
+          await copyFilesToTargetDir(buildPath);
+          console.log(`Successfully installed in ${buildPath}`);
+          await reloadObsidian();
+        } else {
+          const folderToRemove = path.join(buildPath, '_.._');
+          if (await isValidPath(folderToRemove)) {
+            await rm(folderToRemove, { recursive: true });
+          }
+          console.log('Build done in initial folder');
+        }
+      } else {
+        // watch (dev)
+        await copyFilesToTargetDir(buildPath);
+      }
+    });
+  }
+}
+```
+
+- Mode **dev (`yarn dev`)** : après chaque rebuild (watch), `copyFilesToTargetDir` recopie manifest + CSS vers le vault de test.
+- Mode **build prod (`yarn build`)** : rien n'est copié par défaut, sauf si on passe `-r` / `real` (=> copie vers le vault de prod + reload Obsidian).
+
+### 3.4 Cible : `buildPath`
+
+`buildPath` est calculé par `env.ts` (`getBuildPath`) à partir de `.env` (`TEST_VAULT` / `REAL_VAULT`) ou du dossier courant (développement in-place). Il pointe vers :
+
+```
+<vault>/.obsidian/plugins/<pluginId>/
+```
+
+Donc concrètement, le résultat final pour le SCSS est :
+
+1. `src/styles.scss` → compilé par esbuild-sass-plugin → `buildPath/main.css` (transitoire)
+2. `main.css` renommé en `styles.css` par `renameMainCss`
+3. Le CSS est directement disponible et référencé via `manifest.json` (`"css": "styles.css"`).
+
+> 💡 Le `styles.css` final est produit automatiquement par ce pipeline en renommant le fichier `main.css` issu de la compilation SCSS. Aucun fichier source `styles.css` n'est nécessaire à la racine si vous utilisez exclusivement du SCSS.
+
+---
+
+## 4. Où l'injecteur installe tout ça dans le plugin cible
+
+L'injection des templates se fait dans `scripts/inject-core.ts`, fonction `injectScripts()` (lignes 455-612). Le tableau `scriptFiles` (lignes 469-480) liste explicitement les fichiers copiés dans le dossier `scripts/` du plugin cible :
+
+```ts
+const scriptFiles = [
+  'templates/scripts/utils.ts',
+  'templates/scripts/esbuild.config.ts',
+  'templates/scripts/acp.ts',
+  'templates/scripts/update-version.ts',
+  'templates/scripts/release.ts',
+  'templates/scripts/help.ts',
+  'templates/scripts/constants.ts',
+  'templates/scripts/env.ts',
+  'templates/scripts/reload.ts',
+  'templates/scripts/typingsPlugin.ts'
+];
+```
+
+Donc après injection, le plugin Obsidian cible contient `scripts/esbuild.config.ts` (avec la logique SCSS) et `scripts/utils.ts` (avec `removeMainCss` et `copyFilesToTargetDir`).
+
+Le `.gitignore` injecté (`templates/gitignore.template`, ligne 25-26) ignore explicitement le fichier intermédiaire :
+
+```
+# scss result
+main.css
+```
+
+---
+
+## 5. Récapitulatif des fichiers clés
+
+| Rôle | Fichier | Fonctions / lignes |
+|------|---------|--------------------|
+| Détection SCSS + build esbuild | `templates/scripts/esbuild.config.ts` | `main()` ~l. 115-140, `createBuildContext()` ~l. 33-69 |
+| Plugin esbuild pour compiler SCSS | `esbuild-sass-plugin` (importé dynamiquement) | `esbuild.config.ts` ~l. 41-58 |
+| Suppression du `main.css` transitoire | `templates/scripts/utils.ts` | `removeMainCss()` ~l. 169-180 |
+| Copie finale vers le dossier plugins | `templates/scripts/utils.ts` | `copyFilesToTargetDir()` ~l. 74-123 |
+| Branchement sur `onEnd` esbuild | `templates/scripts/esbuild.config.ts` | plugin `copy-to-plugins-folder` ~l. 70-94 |
+| Copie des templates vers le plugin cible | `scripts/inject-core.ts` | `injectScripts()` ~l. 455-612, `buildFileList()` ~l. 284-353 |
+| Dépendance optionnelle | `esbuild-sass-plugin` (ajoutée par l'utilisateur si SCSS) | `README.md` § SASS Support, ~l. 85-96 |
+| Ignore du fichier transitoire | `templates/gitignore.template` | `main.css` ligne 25-26 |

package/docs/audit.md

@@ -0,0 +1,39 @@
+# Audit — obsidian-plugin-config
+
+Audit mis à jour du repo d'injection de configuration pour plugins Obsidian.
+
+---
+
+## 1. Bugs & problèmes fonctionnels restants ou nouveaux
+
+### 🔴 BUG — Erreurs de compilation TypeScript dans les templates & conflits d'IDE
+
+Le fait d'avoir un fichier `package.json` et `tsconfig.json` directement sous `templates/` perturbe les IDE et cause des erreurs de types car `node_modules` n'y est plus présent.
+De plus, la racine du projet (`tsconfig.json` de la racine) inclut `templates/**/*.ts` mais son `package.json` ne contient pas les dépendances de développement utilisées par les templates (`dotenv`, `builtin-modules`, `esbuild`, `obsidian`, `obsidian-typings`, `tslib`). Cela produit des erreurs `Cannot find module 'dotenv'` lors de l'exécution de `tsc` à la racine.
+
+**Actions recommandées :**
+1. Renommer `templates/package.json` en `templates/package.json.template`.
+2. Renommer `templates/tsconfig.json` en `templates/tsconfig.json.template`.
+3. Installer les dépendances requises par les templates en tant que `devDependencies` dans le `package.json` racine pour que le compilateur TS de la racine résolve correctement les types sans nécessiter de dossier `node_modules` dans `templates/`.
+4. Mettre à jour le code d'injection dans [inject-core.ts](file:///c:/Users/dd200/Documents/Mes_projets/Mes%20repo%20obsidian%20new/obsidian-plugin-config/scripts/inject-core.ts) pour copier à partir des fichiers `.template` et les restaurer sans suffixe dans la cible.
+5. Mettre à jour la documentation et les scripts d'aide pour mentionner les nouveaux noms de templates.
+
+---
+
+## 2. Incohérences & améliorations de code restantes
+
+### 🟡 INCOHÉRENCE — Flags CLI inversés dans la doc
+
+[INTERACTIVE_INJECTION.md:42-53](file:///c:/Users/dd200/Documents/Mes_projets/Mes%20repo%20obsidian%20new/obsidian-plugin-config/docs/INTERACTIVE_INJECTION.md#L42-L53)
+
+Le tableau indique `--no, -n` pour la CLI globale comme « auto-confirme tous les remplacements ». C'est correct dans le code (`bin/obsidian-inject.js` L100 : `--no` / `-n`). Mais le nommage est contre-intuitif : `--no` signifie « pas de confirmation » (= auto-confirme tout), ce qui peut prêter à confusion car `--no` semble dire « non ».
+
+---
+
+### 🟡 Améliorations possibles (non-bloquantes)
+
+| # | Suggestion | Impact |
+|---|-----------|--------|
+| 1 | Unifier les flags `--yes`/`--no` entre CLI globale et scripts locaux | UX |
+| 2 | Remplacer le mix fs sync/async dans `inject-core.ts` par une approche uniforme | Propreté |
+| 3 | Nettoyer les numéros de lignes obsolètes dans `docs/SCSS-FLOW.md` | Doc maintenance |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "obsidian-plugin-config",
-  "version": "1.7.2",
+  "version": "1.7.3",
   "description": "Global CLI injection tool for Obsidian plugins",
   "type": "module",
   "bin": {
@@ -34,11 +34,17 @@
     "@types/node": "^22.15.26",
     "@typescript-eslint/eslint-plugin": "^8.58.0",
     "@typescript-eslint/parser": "^8.58.0",
+    "builtin-modules": "latest",
     "dedent": "^1.6.0",
+    "dotenv": "latest",
+    "esbuild": "latest",
     "eslint": "latest",
     "eslint-import-resolver-typescript": "latest",
     "jiti": "latest",
+    "obsidian": "*",
+    "obsidian-typings": "latest",
     "prettier": "^3.4.0",
+    "tslib": "2.4.0",
     "tsx": "^4.21.0",
     "typescript": "^5.8.2"
   },

package/scripts/acp.ts

@@ -1,4 +1,4 @@
-import { execSync } from 'child_process';
+import { execSync, spawnSync } from 'child_process';
 import fs from 'fs';
 import path from 'path';
 import {
@@ -39,7 +39,10 @@ async function main(): Promise<void> {
 
     try {
       gitExec('git add -A');
-      gitExec(`git commit -m "${cleanedInput}"`);
+      const result = spawnSync('git', ['commit', '-m', cleanedInput], { stdio: 'inherit' });
+      if (result.status !== 0) {
+        throw new Error('Commit failed');
+      }
     } catch {
       console.log('Commit already exists or failed.');
       return;

package/scripts/help.ts

@@ -41,8 +41,8 @@ PURE INJECTION TOOL:
   bin/obsidian-inject.js          # Global CLI entry point
 
 templates/ → what gets injected:
-  templates/package.json          # Base deps/scripts for target plugins
-  templates/tsconfig.json         # TypeScript config
+  templates/package.json.template # Base deps/scripts for target plugins
+  templates/tsconfig.json.template # TypeScript config
   templates/scripts/*             # Scripts copied to target
   templates/eslint.config.mts     # ESLint config
   templates/.github/workflows/*   # GitHub Actions

package/scripts/inject-core.ts

@@ -106,11 +106,8 @@ export async function ensurePluginConfigClean(): Promise<void> {
     return;
   }
 
-  const originalCwd = process.cwd();
-
   try {
-    process.chdir(configRoot);
-    const gitStatus = execSync('git status --porcelain', { encoding: 'utf8' }).trim();
+    const gitStatus = execSync('git status --porcelain', { cwd: configRoot, encoding: 'utf8' }).trim();
 
     if (gitStatus) {
       console.log(`\n⚠️  Plugin-config has uncommitted changes:`);
@@ -118,21 +115,23 @@ export async function ensurePluginConfigClean(): Promise<void> {
       console.log(`\n🔧 Auto-committing changes...`);
 
       const msg = '🔧 Update plugin-config templates';
-      gitExec('git add -A');
-      gitExec(`git commit -m "${msg}"`);
+      gitExec('git add -A', configRoot);
+      gitExec(`git commit -m "${msg}"`, configRoot);
 
       try {
         const branch = execSync('git rev-parse --abbrev-ref HEAD', {
+          cwd: configRoot,
           encoding: 'utf8'
         }).trim();
-        gitExec(`git push origin ${branch}`);
+        gitExec(`git push origin ${branch}`, configRoot);
         console.log(`✅ Changes committed and pushed`);
       } catch {
         try {
           const branch = execSync('git rev-parse --abbrev-ref HEAD', {
+            cwd: configRoot,
             encoding: 'utf8'
           }).trim();
-          gitExec(`git push --set-upstream origin ${branch}`);
+          gitExec(`git push --set-upstream origin ${branch}`, configRoot);
           console.log(`✅ New branch pushed with upstream`);
         } catch {
           console.log(`⚠️  Committed locally, push failed`);
@@ -141,8 +140,8 @@ export async function ensurePluginConfigClean(): Promise<void> {
     } else {
       console.log(`✅ Plugin-config repo is clean`);
     }
-  } finally {
-    process.chdir(originalCwd);
+  } catch (error) {
+    console.error(`⚠️  Failed to check or commit plugin-config: ${error}`);
   }
 }
 
@@ -255,8 +254,7 @@ export async function cleanOldLintFiles(targetPath: string): Promise<void> {
     if (await isValidPath(filePath)) {
       fs.unlinkSync(filePath);
       console.log(
-        `🗑️  Removed old ESLint file: ${fileName} (replaced by 1
-				fig.ts)`
+        `🗑️  Removed old ESLint file: ${fileName} (replaced by eslint.config.mts)`
       );
     }
   }
@@ -307,13 +305,14 @@ function buildFileList(targetPath: string): FileEntry[] {
 
   // Root config files
   const configFileMap: Array<[string, string, boolean?]> = [
-    ['templates/tsconfig.json', 'tsconfig.json'],
+    ['templates/tsconfig.json.template', 'tsconfig.json'],
     ['templates/gitignore.template', '.gitignore'],
     ['templates/eslint.config.mts', 'eslint.config.mts'],
     ['templates/.editorconfig', '.editorconfig'],
     ['templates/.prettierrc', '.prettierrc'],
     ['templates/.prettierignore', '.prettierignore'],
     ['templates/npmrc.template', '.npmrc'],
+    ['templates/.gitattributes', '.gitattributes'],
     ['templates/env.template', '.env', true]
   ];
   for (const [src, destName, mergeEnv] of configFileMap) {
@@ -486,13 +485,14 @@ export async function injectScripts(
   // Files with .template suffix (NPM excludes dotfiles)
   // Map: { source: targetName }
   const configFileMap: Record<string, string> = {
-    'templates/tsconfig.json': 'tsconfig.json',
+    'templates/tsconfig.json.template': 'tsconfig.json',
     'templates/gitignore.template': '.gitignore',
     'templates/eslint.config.mts': 'eslint.config.mts',
     'templates/.editorconfig': '.editorconfig',
     'templates/.prettierrc': '.prettierrc',
     'templates/.prettierignore': '.prettierignore',
     'templates/npmrc.template': '.npmrc',
+    'templates/.gitattributes': '.gitattributes',
     'templates/env.template': '.env'
   };
 
@@ -629,7 +629,7 @@ export async function updatePackageJson(
 
     const configRoot = findPluginConfigRoot();
     const templatePkg = JSON.parse(
-      fs.readFileSync(path.join(configRoot, 'templates/package.json'), 'utf8')
+      fs.readFileSync(path.join(configRoot, 'templates/package.json.template'), 'utf8')
     );
 
     const obsoleteScripts = ['version'];
@@ -750,10 +750,11 @@ export async function cleanNpmArtifactsIfNeeded(targetPath: string): Promise<voi
       if (fs.existsSync(nodeModulesPath)) {
         console.log(`   ⏳ Removing node_modules (this may take a moment)...`);
 
-        execSync(`rmdir /s /q "${nodeModulesPath}"`, {
-          stdio: 'pipe',
-          windowsHide: true
-        });
+        try {
+          fs.rmSync(nodeModulesPath, { recursive: true, force: true });
+        } catch {
+          // Ignore initial error, lock detection checks if it still exists below
+        }
 
         if (fs.existsSync(nodeModulesPath)) {
           // rmdir failed silently (locked .exe files) - rename instead

package/scripts/utils.ts

@@ -115,9 +115,9 @@ export async function copyFilesToTargetDir(buildPath: string): Promise<void> {
   }
 }
 
-export function gitExec(command: string): void {
+export function gitExec(command: string, cwd?: string): void {
   try {
-    execSync(command, { stdio: 'inherit' });
+    execSync(command, { stdio: 'inherit', ...(cwd ? { cwd } : {}) });
   } catch (error: unknown) {
     console.error(`Error executing '${command}':`, (error as Error).message);
     throw error;

package/templates/.github/workflows/release.yml

@@ -12,9 +12,9 @@ jobs:
       contents: write
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Use Node.js
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: "22.x"
 

package/templates/{package.json → package.json.template}

@@ -25,7 +25,7 @@
   },
   "devDependencies": {
     "@types/eslint": "latest",
-    "@types/node": "latest",
+    "@types/node": "^25.9.2",
     "@typescript-eslint/eslint-plugin": "latest",
     "@typescript-eslint/parser": "latest",
     "builtin-modules": "latest",

package/templates/scripts/acp.ts

@@ -1,4 +1,4 @@
-import { execSync } from 'child_process';
+import { execSync, spawnSync } from 'child_process';
 import {
   askQuestion,
   cleanInput,
@@ -23,7 +23,10 @@ async function main(): Promise<void> {
 
     try {
       gitExec('git add -A');
-      gitExec(`git commit -m "${cleanedInput}"`);
+      const result = spawnSync('git', ['commit', '-m', cleanedInput], { stdio: 'inherit' });
+      if (result.status !== 0) {
+        throw new Error('Commit failed');
+      }
     } catch {
       console.log('Commit already exists or failed.');
       return;

package/templates/scripts/esbuild.config.ts

@@ -16,7 +16,7 @@ import {
   copyFilesToTargetDir,
   createReadlineInterface,
   isValidPath,
-  removeMainCss
+  renameMainCss
 } from './utils.ts';
 import { reloadObsidian } from './reload.ts';
 
@@ -56,11 +56,11 @@ async function createBuildContext(
             }
           })(),
           {
-            name: 'remove-main-css',
+            name: 'rename-main-css',
             setup(build: esbuild.PluginBuild): void {
               build.onEnd(async (result) => {
                 if (result.errors.length === 0) {
-                  await removeMainCss(buildPath);
+                  await renameMainCss(buildPath);
                 }
               });
             }

package/templates/scripts/utils.ts

@@ -1,4 +1,4 @@
-import { access, mkdir, copyFile, rm, writeFile } from 'fs/promises';
+import { access, mkdir, copyFile, rm, writeFile, rename } from 'fs/promises';
 import { existsSync, readFileSync } from 'fs';
 import path from 'path';
 import * as readline from 'readline';
@@ -122,9 +122,9 @@ export async function copyFilesToTargetDir(buildPath: string): Promise<void> {
   }
 }
 
-export function gitExec(command: string): void {
+export function gitExec(command: string, cwd?: string): void {
   try {
-    execSync(command, { stdio: 'inherit' });
+    execSync(command, { stdio: 'inherit', ...(cwd ? { cwd } : {}) });
   } catch (error: unknown) {
     console.error(
       `Error executing '${command}':`,
@@ -163,19 +163,20 @@ export async function ensureGitSync(): Promise<void> {
 }
 
 /**
- * Remove main.css file generated by esbuild when compiling SCSS
- * This prevents the unwanted main.css from being included in the plugin
+ * Rename main.css file generated by esbuild when compiling SCSS to styles.css
+ * This ensures the compiled CSS is correctly recognized by Obsidian
  */
-export async function removeMainCss(outdir: string): Promise<void> {
+export async function renameMainCss(outdir: string): Promise<void> {
   const mainCssPath = path.join(outdir, 'main.css');
+  const stylesCssPath = path.join(outdir, 'styles.css');
   try {
-    await rm(mainCssPath);
-  } catch (error: unknown) {
-    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
-      console.warn(
-        `Warning: Could not remove main.css: ${error instanceof Error ? error.message : String(error)}`
-      );
+    if (existsSync(mainCssPath)) {
+      await rename(mainCssPath, stylesCssPath);
     }
+  } catch (error: unknown) {
+    console.warn(
+      `Warning: Could not rename main.css to styles.css: ${error instanceof Error ? error.message : String(error)}`
+    );
   }
 }
 

package/tsconfig.json

@@ -19,6 +19,6 @@
     "resolveJsonModule": true,
     "lib": ["DOM", "ES2024"]
   },
-  "include": ["./scripts/**/*.ts"],
-  "exclude": ["node_modules", "eslint.config.ts", "templates"]
+  "include": ["./scripts/**/*.ts", "templates/**/*.ts"],
+  "exclude": ["node_modules", "eslint.config.ts"]
 }