obsidian-mcp-server 1.2.1 → 1.2.3

package/README.md

@@ -2,7 +2,7 @@
 
 [![TypeScript](https://img.shields.io/badge/TypeScript-5.3-blue.svg)](https://www.typescriptlang.org/)
 [![Model Context Protocol](https://img.shields.io/badge/MCP-1.4.0-green.svg)](https://modelcontextprotocol.io/)
-[![Version](https://img.shields.io/badge/Version-1.2.1-blue.svg)]()
+[![Version](https://img.shields.io/badge/Version-1.2.3-blue.svg)]()
 [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
 [![Status](https://img.shields.io/badge/Status-Stable-blue.svg)]()
 [![GitHub](https://img.shields.io/github/stars/cyanheads/obsidian-mcp-server?style=social)](https://github.com/cyanheads/obsidian-mcp-server)
@@ -16,25 +16,21 @@ Requires the Local REST API plugin in Obsidian.
 ## Features
 
 ### File Operations
-- Path-based file/directory management with atomic updates
-- Content read/write operations with validation
+- Atomic file/directory operations with validation
 - Resource monitoring and cleanup
 
 ### Search System
-- Full-text and JsonLogic-based complex search
-- Configurable context boundaries and token limits
-- Optimized query processing
+- Full-text and JsonLogic search with context control
+- Optimized query processing with token limits
 
 ### Property Management
-- YAML frontmatter parsing and validation
-- Intelligent property merging and updates
-- Automatic timestamp management
+- YAML frontmatter parsing and intelligent merging
+- Automatic timestamps (created by Obsidian, modified by server)
 - Custom field support
 
 ### Security & Performance
-- API key authentication and rate limiting
-- SSL verification options
-- Resource management and health monitoring
+- API key auth with rate limiting and SSL options
+- Resource monitoring and health checks
 
 ## Installation
 
@@ -138,9 +134,8 @@ obsidian_update_properties: {
   filepath: string,  // Path relative to vault root
   properties: {
     title?: string,
-    created?: string,  // ISO date
-    modified?: string, // ISO date (auto-updated)
     author?: string,
+    // Note: created/modified timestamps are managed automatically
     type?: Array<"concept" | "architecture" | "specification" | 
       "protocol" | "api" | "research" | "implementation" | 
       "guide" | "reference">,
@@ -161,28 +156,21 @@ obsidian_update_properties: {
 ## Best Practices
 
 ### File Operations
-- Use atomic operations
-- Validate content before modifications
-- Implement proper error handling
-- Monitor operation performance
+- Use atomic operations with validation
+- Handle errors and monitor performance
 
 ### Search Implementation
-- Optimize query specificity
-- Control context boundaries
-- Handle large result sets
-- Consider token limits
+- Optimize queries and control context size
+- Handle large results within token limits
 
 ### Property Management
-- Validate property values before updates
-- Use appropriate property types
-- Handle array merging appropriately
-- Consider custom field implications
+- Use appropriate types and validate updates
+- Handle arrays and custom fields properly
+- Never set timestamps (managed automatically)
 
 ### Error Prevention
-- Validate inputs thoroughly
-- Handle API errors gracefully
-- Monitor error patterns
-- Check rate limits
+- Validate inputs and handle errors gracefully
+- Monitor patterns and respect rate limits
 
 ## Contributing
 

package/build/properties.js

@@ -1,5 +1,5 @@
 import { parse, stringify } from 'yaml';
-import { ObsidianPropertiesSchema } from './propertyTypes.js';
+import { ObsidianPropertiesSchema, PropertyUpdateSchema } from './propertyTypes.js';
 export class PropertyManager {
     client;
     constructor(client) {
@@ -51,7 +51,7 @@ export class PropertyManager {
      * Validate property values
      */
     validateProperties(properties) {
-        const result = ObsidianPropertiesSchema.safeParse(properties);
+        const result = PropertyUpdateSchema.safeParse(properties);
         if (result.success) {
             return { valid: true, errors: [] };
         }
@@ -66,7 +66,8 @@ export class PropertyManager {
     mergeProperties(existing, updates) {
         const merged = { ...existing };
         for (const [key, value] of Object.entries(updates)) {
-            if (value === undefined)
+            // Skip undefined values and timestamp fields
+            if (value === undefined || key === 'created' || key === 'modified')
                 continue;
             const currentValue = merged[key];
             // Special handling for arrays - merge rather than replace
@@ -87,7 +88,7 @@ export class PropertyManager {
                 merged[key] = value;
             }
         }
-        // Always update modified date
+        // Always update modified date (this is the only place we set it)
         merged.modified = new Date().toISOString();
         return merged;
     }

package/build/propertyTools.js

@@ -60,7 +60,7 @@ export class UpdatePropertiesToolHandler extends BaseToolHandler {
     getToolDescription() {
         return {
             name: this.name,
-            description: "Update properties in an Obsidian note's YAML frontmatter. Intelligently merges arrays (tags, type, status), handles custom fields, and automatically updates the modified timestamp. Existing properties not included in the update are preserved.",
+            description: "Update properties in an Obsidian note's YAML frontmatter. Intelligently merges arrays (tags, type, status), handles custom fields, and automatically manages timestamps (created by Obsidian, modified by MCP server). Existing properties not included in the update are preserved.",
             examples: [
                 {
                     description: "Update basic metadata",
@@ -111,9 +111,8 @@ export class UpdatePropertiesToolHandler extends BaseToolHandler {
                         description: "Properties to update",
                         properties: {
                             title: { type: "string" },
-                            created: { type: "string", format: "date-time" },
-                            modified: { type: "string", format: "date-time" },
                             author: { type: "string" },
+                            // Note: created and modified timestamps are managed automatically
                             type: {
                                 type: "array",
                                 items: {

package/build/propertyTypes.js

@@ -17,11 +17,34 @@ export const StatusEnum = z.enum([
     "review",
     "complete"
 ]);
+// Schema for reading properties (includes timestamps)
 export const ObsidianPropertiesSchema = z.object({
+    // Basic Metadata
+    // Note: Timestamps are managed automatically
+    title: z.string().optional(),
+    modified: z.string().datetime().optional(), // Read-only, managed by MCP server
+    author: z.string().optional(),
+    // Classification
+    type: z.array(PropertyTypeEnum).optional(),
+    // Organization
+    tags: z.array(z.string().startsWith("#")).optional(),
+    // Technical Metadata
+    status: z.array(StatusEnum).optional(),
+    version: z.string().optional(),
+    platform: z.string().optional(),
+    repository: z.string().url().optional(),
+    dependencies: z.array(z.string()).optional(),
+    // References
+    sources: z.array(z.string()).optional(),
+    urls: z.array(z.string().url()).optional(),
+    papers: z.array(z.string()).optional(),
+    // Custom Fields
+    custom: z.record(z.unknown()).optional()
+});
+// Schema for validating property updates (excludes timestamps)
+export const PropertyUpdateSchema = z.object({
     // Basic Metadata
     title: z.string().optional(),
-    created: z.string().datetime().optional(),
-    modified: z.string().datetime().optional(),
     author: z.string().optional(),
     // Classification
     type: z.array(PropertyTypeEnum).optional(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "obsidian-mcp-server",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "Model Context Protocol server for Obsidian integration with token-aware response handling",
   "main": "build/index.js",
   "type": "module",

package/src/properties.ts

@@ -1,10 +1,11 @@
 import { parse, stringify } from 'yaml';
 import { ObsidianClient } from './obsidian.js';
-import { 
-  ObsidianProperties, 
-  ObsidianPropertiesSchema, 
+import {
+  ObsidianProperties,
+  ObsidianPropertiesSchema,
+  PropertyUpdateSchema,
   PropertyManagerResult,
-  ValidationResult 
+  ValidationResult
 } from './propertyTypes.js';
 
 export class PropertyManager {
@@ -62,7 +63,7 @@ export class PropertyManager {
    * Validate property values
    */
   validateProperties(properties: Partial<ObsidianProperties>): ValidationResult {
-    const result = ObsidianPropertiesSchema.safeParse(properties);
+    const result = PropertyUpdateSchema.safeParse(properties);
     
     if (result.success) {
       return { valid: true, errors: [] };
@@ -70,7 +71,7 @@ export class PropertyManager {
 
     return {
       valid: false,
-      errors: result.error.errors.map(err => 
+      errors: result.error.errors.map(err =>
         `${err.path.join('.')}: ${err.message}`
       )
     };
@@ -86,7 +87,8 @@ export class PropertyManager {
     const merged = { ...existing };
 
     for (const [key, value] of Object.entries(updates)) {
-      if (value === undefined) continue;
+      // Skip undefined values and timestamp fields
+      if (value === undefined || key === 'created' || key === 'modified') continue;
 
       const currentValue = merged[key as keyof ObsidianProperties];
 
@@ -109,7 +111,7 @@ export class PropertyManager {
       }
     }
 
-    // Always update modified date
+    // Always update modified date (this is the only place we set it)
     merged.modified = new Date().toISOString();
 
     return merged;

package/src/propertyTools.ts

@@ -79,7 +79,7 @@ export class UpdatePropertiesToolHandler extends BaseToolHandler<UpdatePropertie
   getToolDescription(): Tool {
     return {
       name: this.name,
-      description: "Update properties in an Obsidian note's YAML frontmatter. Intelligently merges arrays (tags, type, status), handles custom fields, and automatically updates the modified timestamp. Existing properties not included in the update are preserved.",
+      description: "Update properties in an Obsidian note's YAML frontmatter. Intelligently merges arrays (tags, type, status), handles custom fields, and automatically manages timestamps (created by Obsidian, modified by MCP server). Existing properties not included in the update are preserved.",
       examples: [
         {
           description: "Update basic metadata",
@@ -130,9 +130,8 @@ export class UpdatePropertiesToolHandler extends BaseToolHandler<UpdatePropertie
             description: "Properties to update",
             properties: {
               title: { type: "string" },
-              created: { type: "string", format: "date-time" },
-              modified: { type: "string", format: "date-time" },
               author: { type: "string" },
+              // Note: created and modified timestamps are managed automatically
               type: {
                 type: "array",
                 items: {

package/src/propertyTypes.ts

@@ -20,11 +20,40 @@ export const StatusEnum = z.enum([
   "complete"
 ]);
 
+// Schema for reading properties (includes timestamps)
 export const ObsidianPropertiesSchema = z.object({
+  // Basic Metadata
+  // Note: Timestamps are managed automatically
+  title: z.string().optional(),
+  modified: z.string().datetime().optional(), // Read-only, managed by MCP server
+  author: z.string().optional(),
+
+  // Classification
+  type: z.array(PropertyTypeEnum).optional(),
+
+  // Organization
+  tags: z.array(z.string().startsWith("#")).optional(),
+
+  // Technical Metadata
+  status: z.array(StatusEnum).optional(),
+  version: z.string().optional(),
+  platform: z.string().optional(),
+  repository: z.string().url().optional(),
+  dependencies: z.array(z.string()).optional(),
+
+  // References
+  sources: z.array(z.string()).optional(),
+  urls: z.array(z.string().url()).optional(),
+  papers: z.array(z.string()).optional(),
+
+  // Custom Fields
+  custom: z.record(z.unknown()).optional()
+});
+
+// Schema for validating property updates (excludes timestamps)
+export const PropertyUpdateSchema = z.object({
   // Basic Metadata
   title: z.string().optional(),
-  created: z.string().datetime().optional(),
-  modified: z.string().datetime().optional(),
   author: z.string().optional(),
 
   // Classification
@@ -50,6 +79,7 @@ export const ObsidianPropertiesSchema = z.object({
 });
 
 export type ObsidianProperties = z.infer<typeof ObsidianPropertiesSchema>;
+export type PropertyUpdate = z.infer<typeof PropertyUpdateSchema>;
 
 export interface PropertyOperation {
   operation: 'get' | 'update' | 'patch';