obsidian-plugin-config 1.6.17 → 1.7.0

package/.editorconfig

@@ -1,14 +1,9 @@
-# top-most EditorConfig file
-root = true
-
-[*]
-charset = utf-8
-end_of_line = lf
-insert_final_newline = true
-indent_style = tab
-indent_size = 4
-tab_width = 4
-
-[*.json]
-indent_style = space
-indent_size = 2
+# top-most EditorConfig file
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+insert_final_newline = true
+indent_style = space
+indent_size = 2

package/.prettierrc

@@ -1,19 +1,10 @@
 {
-	"useTabs": true,
-	"tabWidth": 4,
+	"useTabs": false,
+	"tabWidth": 2,
 	"printWidth": 90,
 	"endOfLine": "lf",
 	"trailingComma": "none",
 	"semi": true,
 	"singleQuote": true,
-	"arrowParens": "always",
-	"overrides": [
-		{
-			"files": ["*.json"],
-			"options": {
-				"useTabs": false,
-				"tabWidth": 2
-			}
-		}
-	]
+	"arrowParens": "always"
 }

package/bin/obsidian-inject.js

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

package/docs/INTERACTIVE_INJECTION.md

@@ -105,11 +105,11 @@ Inject .env (template)? [Y/n]: y
 Proceed with these options? [Y/n]: y
 ```
 
-## Fichiers Modifiés
+## Fichiers Concernés
 
-1. **scripts/inject-options.ts** (nouveau) - Gestion des options
-2. **scripts/inject-core.ts** - Ajout paramètre `options` aux fonctions
-3. **scripts/inject-path.ts** - Ajout flags `--interactive` et `--preset`
+1. **scripts/inject-core.ts** - Logique d'injection principale (options support)
+2. **scripts/inject-prompt.ts** - Entrée interactive
+3. **scripts/inject-path.ts** - Entrée CLI avec flags `--interactive` et `--preset`
 
 ## Cas d'Usage
 

package/docs/LLM-GUIDE.md

@@ -66,11 +66,18 @@ Creates `.injection-info.json` in target with version and date.
 
 Scripts in `templates/scripts/` that get copied to target plugins:
 
-- `esbuild.config.ts` — build configuration (handles SASS automatically)
+**Build system (modular):**
+- `esbuild.config.ts` — build entry point, orchestrates all build modules
+- `constants.ts` — shared constants (external deps list, banner, REST port)
+- `env.ts` — environment validation, manifest check, build path resolution
+- `reload.ts` — Obsidian hot-reload via Local REST API
+- `typingsPlugin.ts` — esbuild plugin for obsidian-typings resolution
+- `utils.ts` — shared utilities (readline, file ops, git, vault path helpers)
+
+**Workflow scripts:**
 - `acp.ts` — add, commit, push workflow
 - `update-version.ts` — version bump utility
 - `release.ts` — GitHub release automation
-- `utils.ts` — shared utilities
 - `help.ts` — help documentation
 
 ---
@@ -83,6 +90,8 @@ When `--sass` flag is used:
 2. Adds `esbuild-sass-plugin` dependency
 3. The injected `esbuild.config.ts` automatically detects `.scss` files and uses the plugin
 
+Without `--sass`, the build still works — the SASS import in `esbuild.config.ts` is conditional and only activates if `.scss` files exist.
+
 ---
 
 ## What NOT to do
@@ -90,6 +99,7 @@ When `--sass` flag is used:
 - ❌ Do not hardcode deps/scripts in `inject-core.ts` — they must come from `templates/package.json`
 - ❌ 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 inject `esbuild.config.ts` without its dependencies (`constants.ts`, `env.ts`, `reload.ts`, `typingsPlugin.ts`, `utils.ts`) — they are all required
 
 ---
 

package/eslint.config.mts

@@ -1,64 +1,63 @@
-import * as typescriptEslintParser from "@typescript-eslint/parser";
-import typescriptEslintPlugin from "@typescript-eslint/eslint-plugin";
-import "eslint-import-resolver-typescript";
-import type {
-  Linter
-} from "eslint";
-
-const configs: Linter.Config[] = [
-  {
-    ignores: [
-      "eslint.config.mts",
-      "templates/**"
-    ]
-  },
-  {
-    files: ["**/*.ts"],
-    ignores: [
-      "dist/**",
-      "node_modules/**",
-      "main.js"
-    ],
-    languageOptions: {
-      parser: typescriptEslintParser,
-      sourceType: "module",
-      parserOptions: {
-        project: "./tsconfig.json",
-        ecmaVersion: 2023
-      }
-    },
-    plugins: {
-      "@typescript-eslint": typescriptEslintPlugin as any // Type assertion to bypass type checking
-    },
-    rules: {
-      // Base rules
-      "no-unused-vars": "off",
-      "@typescript-eslint/no-unused-vars": ["error", { "args": "none", "varsIgnorePattern": "^_" }],
-      "@typescript-eslint/ban-ts-comment": "warn",
-      "no-prototype-builtins": "off",
-      "@typescript-eslint/no-empty-function": "off",
-
-      // Useful rules but not too strict
-      "semi": "error",
-      "eqeqeq": ["error", "always"],
-      "prefer-const": "error",
-      "@typescript-eslint/explicit-function-return-type": ["warn", { "allowExpressions": true }],
-      "@typescript-eslint/no-explicit-any": "warn",
-      "@typescript-eslint/consistent-type-imports": ["warn", { "prefer": "type-imports" }],
-
-      // Disable overly strict rules
-      "@typescript-eslint/no-unsafe-assignment": "off",
-      "@typescript-eslint/no-unsafe-call": "off",
-      "@typescript-eslint/no-unsafe-member-access": "off",
-      "@typescript-eslint/no-unsafe-argument": "off"
-    }
-  },
-  {
-    files: ["src/**/*.ts"],
-    rules: {
-      "no-console": ["warn", { "allow": ["warn", "error", "debug"] }]
-    }
-  }
-];
-
-export default configs;
+import * as typescriptEslintParser from "@typescript-eslint/parser";
+import typescriptEslintPlugin from "@typescript-eslint/eslint-plugin";
+import "eslint-import-resolver-typescript";
+import type {
+  Linter
+} from "eslint";
+
+const configs: Linter.Config[] = [
+  {
+    ignores: [
+      "eslint.config.mts"
+    ]
+  },
+  {
+    files: ["**/*.ts"],
+    ignores: [
+      "dist/**",
+      "node_modules/**",
+      "main.js"
+    ],
+    languageOptions: {
+      parser: typescriptEslintParser,
+      sourceType: "module",
+      parserOptions: {
+        project: "./tsconfig.json",
+        ecmaVersion: 2023
+      }
+    },
+    plugins: {
+      "@typescript-eslint": typescriptEslintPlugin as any // Type assertion to bypass type checking
+    },
+    rules: {
+      // Base rules
+      "no-unused-vars": "off",
+      "@typescript-eslint/no-unused-vars": ["error", { "args": "none", "varsIgnorePattern": "^_" }],
+      "@typescript-eslint/ban-ts-comment": "warn",
+      "no-prototype-builtins": "off",
+      "@typescript-eslint/no-empty-function": "off",
+
+      // Useful rules but not too strict
+      "semi": "error",
+      "eqeqeq": ["error", "always"],
+      "prefer-const": "error",
+      "@typescript-eslint/explicit-function-return-type": ["warn", { "allowExpressions": true }],
+      "@typescript-eslint/no-explicit-any": "warn",
+      "@typescript-eslint/consistent-type-imports": ["warn", { "prefer": "type-imports" }],
+
+      // Disable overly strict rules
+      "@typescript-eslint/no-unsafe-assignment": "off",
+      "@typescript-eslint/no-unsafe-call": "off",
+      "@typescript-eslint/no-unsafe-member-access": "off",
+      "@typescript-eslint/no-unsafe-argument": "off"
+    }
+  },
+  {
+    files: ["src/**/*.ts"],
+    rules: {
+      "no-console": ["warn", { "allow": ["warn", "error", "debug"] }]
+    }
+  }
+];
+
+export default configs;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "obsidian-plugin-config",
-  "version": "1.6.17",
+  "version": "1.7.0",
   "description": "Global CLI injection tool for Obsidian plugins",
   "type": "module",
   "bin": {
@@ -20,6 +20,8 @@
     "v": "tsx scripts/update-version-config.ts",
     "lint": "eslint . --ext .ts",
     "lint:fix": "eslint . --ext .ts --fix",
+    "prettier": "prettier --check '**/*.{ts,json}'",
+    "prettier:fix": "prettier --write '**/*.{ts,json}'",
     "inject-path": "tsx scripts/inject-path.ts",
     "inject-sass": "tsx scripts/inject-path.ts --sass",
     "inject-prompt": "tsx scripts/inject-prompt.ts",
@@ -31,7 +33,6 @@
   "devDependencies": {
     "@types/eslint": "latest",
     "@types/node": "^22.15.26",
-    "@types/semver": "^7.7.0",
     "@typescript-eslint/eslint-plugin": "^8.58.0",
     "@typescript-eslint/parser": "^8.58.0",
     "dedent": "^1.6.0",
@@ -39,7 +40,6 @@
     "eslint-import-resolver-typescript": "latest",
     "jiti": "latest",
     "prettier": "^3.4.0",
-    "semver": "^7.7.2",
     "tsx": "^4.21.0",
     "typescript": "^5.8.2"
   },

package/scripts/acp.ts

@@ -2,78 +2,74 @@ import { execSync } from 'child_process';
 import fs from 'fs';
 import path from 'path';
 import {
-	askQuestion,
-	cleanInput,
-	createReadlineInterface,
-	gitExec,
-	ensureGitSync
+  askQuestion,
+  cleanInput,
+  createReadlineInterface,
+  gitExec,
+  ensureGitSync
 } from './utils.js';
 
 const rl = createReadlineInterface();
 
 // Check if we're in the centralized config repo
 function isInCentralizedRepo(): boolean {
-	const packageJsonPath = path.join(process.cwd(), 'package.json');
-	if (!fs.existsSync(packageJsonPath)) return false;
+  const packageJsonPath = path.join(process.cwd(), 'package.json');
+  if (!fs.existsSync(packageJsonPath)) return false;
 
-	const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	return packageJson.name === 'obsidian-plugin-config';
+  const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+  return packageJson.name === 'obsidian-plugin-config';
 }
 
 async function main(): Promise<void> {
-	try {
-		if (process.argv.includes('-b')) {
-			console.log('Building...');
-			// For obsidian-plugin-config, just run TypeScript check
-			if (isInCentralizedRepo()) {
-				gitExec('npx tsc --noEmit');
-			} else {
-				gitExec('yarn build');
-			}
-			console.log('Build successful.');
-		}
+  try {
+    if (process.argv.includes('-b')) {
+      console.log('Building...');
+      // For obsidian-plugin-config, just run TypeScript check
+      if (isInCentralizedRepo()) {
+        gitExec('npx tsc --noEmit');
+      } else {
+        gitExec('yarn build');
+      }
+      console.log('Build successful.');
+    }
 
+    const input: string = await askQuestion('Enter commit message: ', rl);
 
+    const cleanedInput = cleanInput(input);
 
-		const input: string = await askQuestion('Enter commit message: ', rl);
+    try {
+      gitExec('git add -A');
+      gitExec(`git commit -m "${cleanedInput}"`);
+    } catch {
+      console.log('Commit already exists or failed.');
+      return;
+    }
 
-		const cleanedInput = cleanInput(input);
+    // get current branch name
+    const currentBranch = execSync('git rev-parse --abbrev-ref HEAD').toString().trim();
 
-		try {
-			gitExec('git add -A');
-			gitExec(`git commit -m "${cleanedInput}"`);
-		} catch {
-			console.log('Commit already exists or failed.');
-			return;
-		}
+    // Ensure Git is synchronized before pushing
+    await ensureGitSync();
 
-		// get current branch name
-		const currentBranch = execSync('git rev-parse --abbrev-ref HEAD')
-			.toString()
-			.trim();
-
-		// Ensure Git is synchronized before pushing
-		await ensureGitSync();
-
-		try {
-			gitExec(`git push origin ${currentBranch}`);
-			console.log('Commit and push successful.');
-		} catch {
-			// new branch
-			console.log(`New branch detected. Setting upstream for ${currentBranch}...`);
-			gitExec(`git push --set-upstream origin ${currentBranch}`);
-			console.log('Upstream branch set and push successful.');
-		}
-	} catch (error) {
-		console.error('Error:', error instanceof Error ? error.message : String(error));
-	} finally {
-		rl.close();
-	}
+    try {
+      gitExec(`git push origin ${currentBranch}`);
+      console.log(`Commit and push successful to origin/${currentBranch}`);
+    } catch {
+      // new branch
+      console.log(`New branch detected. Setting upstream for ${currentBranch}...`);
+      gitExec(`git push --set-upstream origin ${currentBranch}`);
+      console.log('Upstream branch set and push successful.');
+    }
+  } catch (error) {
+    console.error('Error:', error instanceof Error ? error.message : String(error));
+  } finally {
+    rl.close();
+  }
 }
 
 main()
-	.catch(console.error)
-	.finally(() => {
-		console.log('Exiting...');
-		process.exit();
-	});
+  .catch(console.error)
+  .finally(() => {
+    console.log('Exiting...');
+    process.exit();
+  });