obsidian-dev-skills 1.1.0 → 1.1.2

package/obsidian-ops/references/build-workflow.md

@@ -10,7 +10,7 @@ Update frequency: Update as build process evolves
 
 After making any changes to theme code:
 
-### Simple CSS Themes
+## Simple CSS Themes
 
 If your theme is simple with just `theme.css` in the root and no build tools:
 
@@ -20,7 +20,7 @@ If your theme is simple with just `theme.css` in the root and no build tools:
 
 **How to detect**: If you have `theme.css` in root and no `src/scss/` directory, you have a simple CSS theme.
 
-### Complex Themes (SCSS + Build Tools)
+## Complex Themes (SCSS + Build Tools)
 
 If your theme uses build tools (Grunt, npm scripts, SCSS compiler, etc.) and has `src/scss/` directory:
 

package/obsidian-ops/references/security-privacy.md

@@ -15,7 +15,7 @@ Follow Obsidian's **Developer Policies** (https://docs.obsidian.md/Developer+pol
 - **Code obfuscation**: CSS must be readable and not minified/obfuscated
 - **Dynamic ads**: No dynamic advertising
 - **Client-side telemetry**: No hidden telemetry. If you collect optional analytics, require explicit opt-in and document clearly in `README.md`
-- **Self-updating mechanisms**: No automatic code updates outside of normal releases. Never execute remote code, fetch and eval scripts, or auto-update code
+- **Self-updating mechanisms**: No automatic code updates outside normal releases. Never execute remote code, fetch and eval scripts, or auto-update code
 
 ### Mandatory Disclosures
 

package/obsidian-ref/references/file-conventions.md

@@ -15,7 +15,7 @@ Update frequency: Check Obsidian Sample Theme repo for updates
 ## Simple CSS Theme Structure
 
 **Recommended for simple themes** (like the sample theme template):
-```
+```text
 theme.css          # ✅ Source CSS file (edit directly)
 manifest.json
 package.json
@@ -28,7 +28,7 @@ package.json
 ## Complex Theme Structure (SCSS + Build Tools)
 
 **For themes using SCSS, Grunt, npm scripts, or other build tools**:
-```
+```text
 src/
   scss/            # ✅ SCSS source files
     index.scss
@@ -49,7 +49,7 @@ package.json
 
 ## Wrong Structure (Common Mistakes)
 
-```
+```text
 theme.css         # ❌ DON'T have both
 src/
   scss/          # ❌ This causes confusion about which is the source

package/obsidian-ref/references/manifest.md

@@ -6,7 +6,7 @@ Update frequency: Check Obsidian releases repo for validation requirements
 
 # Manifest rules (`manifest.json`)
 
-### Required Fields
+## Required Fields
 
 All themes must include these fields in `manifest.json`:
 
@@ -15,12 +15,12 @@ All themes must include these fields in `manifest.json`:
 - **`minAppVersion`** (string, required) - Minimum Obsidian version required
 - **`author`** (string, required) - Author name (required for themes)
 
-### Optional Fields
+## Optional Fields
 
 - **`authorUrl`** (string, optional) - URL to author's website or profile
 - **`fundingUrl`** (string, optional) - URL for funding/support
 
-### Important Notes
+## Important Notes
 
 - Themes do **not** use `id` or `isDesktopOnly` fields.
 - Canonical requirements: https://github.com/obsidianmd/obsidian-releases/blob/master/.github/workflows/validate-theme-entry.yml

package/obsidian-ref/references/obsidian-file-formats.md

@@ -451,7 +451,7 @@ ___
 
 ### Properties (YAML metadata)
 
-**Important**: Obsidian uses the term "properties" (not "frontmatter" or "front-matter") when referring to YAML metadata at the top of Markdown files. All documentation, code comments, and UI text should use "properties" to align with Obsidian's official terminology.
+**Important**: Obsidian uses the term "properties" (the editable UI/metadata interface) and "frontmatter" (the YAML block serialization). While both terms are valid and complementary, this guide prefers "properties" for consistency with Obsidian's modern UI terminology.
 
 Properties use YAML metadata (properties) at the start of a note:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "obsidian-dev-skills",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "Agent skills for Obsidian plugin and theme development",
   "keywords": [
     "obsidian",

package/scripts/init.mjs

@@ -11,16 +11,29 @@ const __dirname = path.dirname(__filename);
 // The package root is one level up from scripts/
 const packageRoot = path.join(__dirname, '..');
 
+/**
+ * Compares two paths for equality, handling platform-specific differences.
+ */
+function arePathsEqual(path1, path2) {
+  if (!path1 || !path2) return false;
+  const norm1 = path.normalize(path1).replace(/[\\/]$/, '');
+  const norm2 = path.normalize(path2).replace(/[\\/]$/, '');
+
+  if (process.platform === 'win32') {
+    return norm1.toLowerCase() === norm2.toLowerCase();
+  }
+  return norm1 === norm2;
+}
+
+// Find the real project root where the package is being installed
 // Find the real project root where the package is being installed
 function getProjectRoot() {
-  // INIT_CWD is set by npm/pnpm/yarn to the directory where the command was run
-  if (process.env.INIT_CWD) {
-    return process.env.INIT_CWD;
-  }
+  // We want to find the root of the project, which is the directory containing 
+  // the first package.json that isn't for 'obsidian-dev-skills' (unless it's the dev repo).
+  // We start from INIT_CWD (where the command was run) or process.cwd() as fallback.
+  const initial = process.env.INIT_CWD || process.cwd();
+  let current = initial;
 
-  // Fallback: traverse up from process.cwd() to find the first package.json 
-  // that isn't the one in this package
-  let current = process.cwd();
   while (current !== path.parse(current).root) {
     const pkgPath = path.join(current, 'package.json');
     if (fs.existsSync(pkgPath)) {
@@ -29,11 +42,20 @@ function getProjectRoot() {
         if (pkg.name !== 'obsidian-dev-skills') {
           return current;
         }
+        // If we found 'obsidian-dev-skills', check if we are in node_modules.
+        // If we are NOT in node_modules, this is likely the development repository.
+        if (!current.toLowerCase().includes('node_modules')) {
+          return current;
+        }
       } catch (e) { }
     }
-    current = path.dirname(current);
+    const parent = path.dirname(current);
+    if (parent === current) break;
+    current = parent;
   }
-  return process.cwd();
+
+  // Fallback to the initial directory if no project root found
+  return initial;
 }
 
 const projectRoot = getProjectRoot();
@@ -303,10 +325,13 @@ Example:
 
 async function init() {
   // Determine if we are running in the package's own directory (development)
-  const isDevelopment = projectRoot === packageRoot;
+  const isDevelopment = arePathsEqual(projectRoot, packageRoot);
 
   if (isDevelopment && !process.env.FORCE_INIT) {
-    console.log('🛠️ Development mode detected (or forced skip), skipping initialization.');
+    console.log('🛠️ Development mode detected: skipping initialization in the skills repository.');
+    console.log('💡 To force initialization (e.g., for testing), run:');
+    console.log('   $env:FORCE_INIT=1; pnpm obsidian-dev-skills  (PowerShell)');
+    console.log('   FORCE_INIT=1 pnpm obsidian-dev-skills       (Bash/Zsh)');
     return;
   }