@karmaniverous/jeeves-watcher 0.8.1 → 0.8.3

package/dist/cli/jeeves-watcher/index.js

@@ -2546,6 +2546,8 @@ function createConfigReindexHandler(deps) {
             processor: deps.processor,
             logger: deps.logger,
             reindexTracker: deps.reindexTracker,
+            valuesManager: deps.valuesManager,
+            issuesManager: deps.issuesManager,
         }, scope);
         return await reply.status(200).send({ status: 'started', scope });
     }, deps.logger, 'Config reindex request');
@@ -3529,7 +3531,14 @@ function createApiServer(options) {
         vectorStore,
         logger,
     }));
-    app.post('/config-reindex', createConfigReindexHandler({ config, processor, logger, reindexTracker }));
+    app.post('/config-reindex', createConfigReindexHandler({
+        config,
+        processor,
+        logger,
+        reindexTracker,
+        valuesManager,
+        issuesManager,
+    }));
     app.get('/issues', withCache(cacheTtlMs, createIssuesHandler({ issuesManager })));
     app.get('/config/schema', withCache(cacheTtlMs, createConfigSchemaHandler()));
     app.post('/config/match', createConfigMatchHandler({ config, logger }));
@@ -4389,6 +4398,13 @@ class DocumentProcessor {
         await this.withFileErrorHandling(filePath, 'Failed to process file', async () => {
             const ext = extname(filePath);
             const { metadata, extracted, renderedContent, matchedRules, metadataWithRules, } = await this.buildMetadataWithRules(filePath);
+            // Update values index before any skip conditions — values must
+            // reflect current rule matches even when content is unchanged.
+            if (this.valuesManager) {
+                for (const ruleName of matchedRules) {
+                    this.valuesManager.update(ruleName, metadata);
+                }
+            }
             const textToEmbed = renderedContent ?? extracted.text;
             if (!textToEmbed.trim()) {
                 this.logger.debug({ filePath }, 'Skipping empty file');
@@ -4416,11 +4432,6 @@ class DocumentProcessor {
                 logger: this.logger,
             }, textToEmbed, filePath, metadataWithRules, existingPayload, fileDates);
             this.issuesManager?.clear(filePath);
-            if (this.valuesManager) {
-                for (const ruleName of matchedRules) {
-                    this.valuesManager.update(ruleName, metadata);
-                }
-            }
         }, undefined);
     }
     /**

package/dist/index.js

@@ -2237,6 +2237,8 @@ function createConfigReindexHandler(deps) {
             processor: deps.processor,
             logger: deps.logger,
             reindexTracker: deps.reindexTracker,
+            valuesManager: deps.valuesManager,
+            issuesManager: deps.issuesManager,
         }, scope);
         return await reply.status(200).send({ status: 'started', scope });
     }, deps.logger, 'Config reindex request');
@@ -3220,7 +3222,14 @@ function createApiServer(options) {
         vectorStore,
         logger,
     }));
-    app.post('/config-reindex', createConfigReindexHandler({ config, processor, logger, reindexTracker }));
+    app.post('/config-reindex', createConfigReindexHandler({
+        config,
+        processor,
+        logger,
+        reindexTracker,
+        valuesManager,
+        issuesManager,
+    }));
     app.get('/issues', withCache(cacheTtlMs, createIssuesHandler({ issuesManager })));
     app.get('/config/schema', withCache(cacheTtlMs, createConfigSchemaHandler()));
     app.post('/config/match', createConfigMatchHandler({ config, logger }));
@@ -4367,6 +4376,13 @@ class DocumentProcessor {
         await this.withFileErrorHandling(filePath, 'Failed to process file', async () => {
             const ext = extname(filePath);
             const { metadata, extracted, renderedContent, matchedRules, metadataWithRules, } = await this.buildMetadataWithRules(filePath);
+            // Update values index before any skip conditions — values must
+            // reflect current rule matches even when content is unchanged.
+            if (this.valuesManager) {
+                for (const ruleName of matchedRules) {
+                    this.valuesManager.update(ruleName, metadata);
+                }
+            }
             const textToEmbed = renderedContent ?? extracted.text;
             if (!textToEmbed.trim()) {
                 this.logger.debug({ filePath }, 'Skipping empty file');
@@ -4394,11 +4410,6 @@ class DocumentProcessor {
                 logger: this.logger,
             }, textToEmbed, filePath, metadataWithRules, existingPayload, fileDates);
             this.issuesManager?.clear(filePath);
-            if (this.valuesManager) {
-                for (const ruleName of matchedRules) {
-                    this.valuesManager.update(ruleName, metadata);
-                }
-            }
         }, undefined);
     }
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@karmaniverous/jeeves-watcher",
-  "version": "0.8.1",
+  "version": "0.8.3",
   "author": "Jason Williscroft",
   "description": "Filesystem watcher that keeps a Qdrant vector store in sync with document changes",
   "license": "BSD-3-Clause",