obsidian-plugin-config 1.3.6 → 1.3.8

package/bin/obsidian-inject.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "obsidian-plugin-config",
-  "version": "1.3.6",
+  "version": "1.3.8",
   "description": "Système d'injection pour plugins Obsidian autonomes",
   "type": "module",
   "main": "src/index.ts",

package/scripts/build-npm.ts

@@ -199,44 +199,63 @@ main();
  */
 function buildAndPublishNpm(): void {
   console.log(`🚀 Obsidian Plugin Config - Complete NPM Workflow`);
-  console.log(`Full automation: version → commit → push → exports → bin → publish\n`);
+  console.log(
+    `Full automation: version → exports → bin → commit → publish\n`
+  );
 
   try {
-    // Step 1: Update version and push to GitHub
+    // Step 1: Update version in package.json only
+    // (no commit yet - we'll do one big commit after)
     console.log(`📋 Step 1/6: Updating version...`);
-    execSync('tsx scripts/update-version-config.ts', { stdio: 'inherit' });
-
-    // Step 2: Commit and push any remaining changes
-    console.log(`\n📤 Step 2/6: Committing and pushing changes...`);
-    try {
-      execSync('echo "Prepare NPM package publication" | tsx scripts/acp.ts -b', { stdio: 'inherit' });
-    } catch {
-      console.log(`   ℹ️  No additional changes to commit`);
-    }
+    execSync('tsx scripts/update-version-config.ts', {
+      stdio: 'inherit'
+    });
 
-    // Step 3: Update exports automatically
-    console.log(`\n📦 Step 3/6: Updating exports...`);
+    // Step 2: Update exports automatically
+    console.log(`\n📦 Step 2/6: Updating exports...`);
     execSync('yarn update-exports', { stdio: 'inherit' });
 
-    // Step 4: Generate bin file
-    console.log(`\n🔧 Step 4/6: Generating bin/obsidian-inject.js...`);
+    // Step 3: Generate bin file (uses updated version)
+    console.log(`\n🔧 Step 3/6: Generating bin/obsidian-inject.js...`);
     generateBinFile();
 
-    // Step 5: Verify package is ready
-    console.log(`\n📋 Step 5/6: Verifying package...`);
+    // Step 4: Verify package and sync versions.json
+    // (must happen before commit so versions.json is included)
+    console.log(`\n📋 Step 4/6: Verifying package...`);
     verifyPackage();
 
+    // Step 5: Commit and push ALL changes together
+    // (package.json version, bin/, versions.json, exports)
+    console.log(`\n📤 Step 5/6: Committing and pushing changes...`);
+    try {
+      execSync(
+        'echo "Publish NPM package" | tsx scripts/acp.ts -b',
+        { stdio: 'inherit' }
+      );
+    } catch {
+      console.log(`   ℹ️  No additional changes to commit`);
+    }
+
     // Step 6: Publish to NPM
     console.log(`\n📤 Step 6/6: Publishing to NPM...`);
-    execSync('npm publish --registry https://registry.npmjs.org/', { stdio: 'inherit' });
+    execSync(
+      'npm publish --registry https://registry.npmjs.org/',
+      { stdio: 'inherit' }
+    );
 
     console.log(`\n🎉 Complete workflow successful!`);
     console.log(`\n📋 Next steps:`);
     console.log(`   1. npm install -g obsidian-plugin-config`);
-    console.log(`   2. Test injection: cd any-plugin && obsidian-inject`);
+    console.log(
+      `   2. Test injection: cd any-plugin && obsidian-inject`
+    );
 
   } catch (error) {
-    console.error(`\n❌ Workflow failed: ${error instanceof Error ? error.message : String(error)}`);
+    console.error(
+      `\n❌ Workflow failed: ${
+        error instanceof Error ? error.message : String(error)
+      }`
+    );
     process.exit(1);
   }
 }

package/scripts/inject-core.ts

@@ -293,8 +293,11 @@ export async function injectScripts(targetPath: string, useSass: boolean = false
     "templates/scripts/help.ts"
   ];
 
-  // Files with .template suffix are renamed by NPM
-  // exclusion rules (.gitignore, .npmrc, .env)
+  // Files that should NOT be overwritten if they
+  // already exist (contain user-specific config)
+  const skipIfExists = new Set([".env"]);
+
+  // Files with .template suffix (NPM excludes dotfiles)
   // Map: { source: targetName }
   const configFileMap: Record<string, string> = {
     "templates/tsconfig.json": "tsconfig.json",
@@ -339,10 +342,20 @@ export async function injectScripts(targetPath: string, useSass: boolean = false
     configFileMap
   )) {
     try {
-      const content = copyFromLocal(src);
       const targetFile = path.join(
         targetPath, destName
       );
+      // Skip if file exists and is user-specific
+      if (
+        skipIfExists.has(destName) &&
+        fs.existsSync(targetFile)
+      ) {
+        console.log(
+          `   ⏭️  ${destName} (kept existing)`
+        );
+        continue;
+      }
+      const content = copyFromLocal(src);
       fs.writeFileSync(targetFile, content, "utf8");
       console.log(`   ✅ ${destName}`);
     } catch (error) {

package/templates/tsconfig.json

@@ -5,8 +5,8 @@
     ],
     "paths": {
       "obsidian-typings/implementations": [
-        "./node_modules/obsidian-typings/dist/cjs/implementations.d.cts",
-        "./node_modules/obsidian-typings/dist/esm/implementations.mjs"
+        "./node_modules/obsidian-typings/dist/implementations.d.ts",
+        "./node_modules/obsidian-typings/dist/implementations.cjs"
       ]
     },
     "inlineSourceMap": true,

package/versions.json

@@ -32,5 +32,7 @@
 	"1.3.3": "1.8.9",
 	"1.3.4": "1.8.9",
 	"1.3.5": "1.8.9",
-	"1.3.6": "1.8.9"
+	"1.3.6": "1.8.9",
+	"1.3.7": "1.8.9",
+	"1.3.8": "1.8.9"
 }