feat(metadata): enhance filesystem loader and update schemas with new properties

Author: hotlong

packages/metadata/src/loaders/filesystem-loader.ts

@@ -22,6 +22,13 @@ import type { MetadataSerializer } from '../serializers/serializer-interface.js'
 export class FilesystemLoader implements MetadataLoader {
   readonly contract: MetadataLoaderContract = {
     name: 'filesystem',
+    protocol: 'file',
+    capabilities: {
+      read: true,
+      write: true,
+      watch: true,
+      list: true,
+    },
     supportedFormats: ['json', 'yaml', 'typescript', 'javascript'],
     supportsWatch: true,
     supportsWrite: true,
@@ -99,7 +106,7 @@ export class FilesystemLoader implements MetadataLoader {
 
       // Load and deserialize
       const content = await fs.readFile(filePath, 'utf-8');
-      const serializer = this.getSerializer(stats.format);
+      const serializer = this.getSerializer(stats.format!);
 
       if (!serializer) {
         throw new Error(`No serializer found for format: ${stats.format}`);
@@ -111,7 +118,7 @@ export class FilesystemLoader implements MetadataLoader {
       if (useCache) {
         this.cache.set(cacheKey, {
           data,
-          etag: stats.etag,
+          etag: stats.etag || '',
           timestamp: Date.now(),
         });
       }

packages/spec/json-schema/system/MetadataLoadResult.json

@@ -66,6 +66,9 @@
         },
         "notModified": {
           "type": "boolean"
+        },
+        "loadTime": {
+          "type": "number"
         }
       },
       "additionalProperties": false

packages/spec/json-schema/system/MetadataLoaderContract.json

@@ -22,6 +22,12 @@
         "supportsWatch": {
           "type": "boolean"
         },
+        "supportsWrite": {
+          "type": "boolean"
+        },
+        "supportsCache": {
+          "type": "boolean"
+        },
         "capabilities": {
           "type": "object",
           "properties": {

packages/spec/json-schema/system/MetadataManagerConfig.json

@@ -8,12 +8,10 @@
           "type": "array"
         },
         "watch": {
-          "type": "boolean",
-          "default": false
+          "type": "boolean"
         },
         "cache": {
-          "type": "boolean",
-          "default": true
+          "type": "boolean"
         },
         "basePath": {
           "type": "string"
@@ -38,9 +36,6 @@
         },
         "watchOptions": {}
       },
-      "required": [
-        "loaders"
-      ],
       "additionalProperties": false
     }
   },

packages/spec/json-schema/system/MetadataSaveResult.json

@@ -53,6 +53,12 @@
         },
         "size": {
           "type": "number"
+        },
+        "saveTime": {
+          "type": "number"
+        },
+        "backupPath": {
+          "type": "string"
         }
       },
       "required": [

packages/spec/json-schema/system/MetadataWatchEvent.json

@@ -18,6 +18,9 @@
         "path": {
           "type": "string"
         },
+        "name": {
+          "type": "string"
+        },
         "stats": {
           "type": "object",
           "properties": {
@@ -58,6 +61,11 @@
         },
         "metadataType": {
           "type": "string"
+        },
+        "data": {},
+        "timestamp": {
+          "type": "string",
+          "format": "date-time"
         }
       },
       "required": [

packages/spec/src/system/metadata-persistence.zod.ts

@@ -103,9 +103,9 @@ export const MetadataStatsSchema = z.object({
   size: z.number().optional(),
   mtime: z.date().optional(),
   hash: z.string().optional(),
-  etag: z.string().optional(),
+  etag: z.string().optional(), // Required by local cache
   modifiedAt: z.date().optional(), // Alias for mtime
-  format: MetadataFormatSchema.optional(),
+  format: MetadataFormatSchema.optional(), // Required for serialization
 });
 
 /**
@@ -118,6 +118,8 @@ export const MetadataLoaderContractSchema = z.object({
   description: z.string().optional(),
   supportedFormats: z.array(z.string()).optional(),
   supportsWatch: z.boolean().optional(),
+  supportsWrite: z.boolean().optional(),
+  supportsCache: z.boolean().optional(),
   capabilities: z.object({
     read: z.boolean().default(true),
     write: z.boolean().default(false),
@@ -153,6 +155,7 @@ export const MetadataLoadResultSchema = z.object({
   fromCache: z.boolean().optional(),
   etag: z.string().optional(),
   notModified: z.boolean().optional(),
+  loadTime: z.number().optional(),
 });
 
 /**
@@ -179,6 +182,8 @@ export const MetadataSaveResultSchema = z.object({
   stats: MetadataStatsSchema.optional(),
   etag: z.string().optional(),
   size: z.number().optional(),
+  saveTime: z.number().optional(),
+  backupPath: z.string().optional(),
 });
 
 /**
@@ -187,8 +192,11 @@ export const MetadataSaveResultSchema = z.object({
 export const MetadataWatchEventSchema = z.object({
   type: z.enum(['add', 'change', 'unlink', 'added', 'changed', 'deleted']),
   path: z.string(),
+  name: z.string().optional(),
   stats: MetadataStatsSchema.optional(),
   metadataType: z.string().optional(),
+  data: z.any().optional(),
+  timestamp: z.date().optional(),
 });
 
 /**
@@ -220,9 +228,9 @@ export const MetadataImportOptionsSchema = z.object({
  * Metadata Manager Config
  */
 export const MetadataManagerConfigSchema = z.object({
-  loaders: z.array(z.any()),
-  watch: z.boolean().default(false),
-  cache: z.boolean().default(true),
+  loaders: z.array(z.any()).optional(),
+  watch: z.boolean().optional(),
+  cache: z.boolean().optional(),
   basePath: z.string().optional(),
   rootDir: z.string().optional(),
   formats: z.array(MetadataFormatSchema).optional(),