Merge PR #913: respect CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED setting

This commit is contained in:
Alex Newman
2026-02-06 01:36:58 -05:00
3 changed files with 29 additions and 15 deletions
+24 -14
View File
@@ -16,6 +16,8 @@ import { parseObservations, parseSummary, type ParsedObservation, type ParsedSum
import { updateCursorContextForProject } from '../../integrations/CursorHooksInstaller.js'; import { updateCursorContextForProject } from '../../integrations/CursorHooksInstaller.js';
import { updateFolderClaudeMdFiles } from '../../../utils/claude-md-utils.js'; import { updateFolderClaudeMdFiles } from '../../../utils/claude-md-utils.js';
import { getWorkerPort } from '../../../shared/worker-utils.js'; import { getWorkerPort } from '../../../shared/worker-utils.js';
import { SettingsDefaultsManager } from '../../../shared/SettingsDefaultsManager.js';
import { USER_SETTINGS_PATH } from '../../../shared/paths.js';
import type { ActiveSession } from '../../worker-types.js'; import type { ActiveSession } from '../../worker-types.js';
import type { DatabaseManager } from '../DatabaseManager.js'; import type { DatabaseManager } from '../DatabaseManager.js';
import type { SessionManager } from '../SessionManager.js'; import type { SessionManager } from '../SessionManager.js';
@@ -215,21 +217,29 @@ async function syncAndBroadcastObservations(
// Update folder CLAUDE.md files for touched folders (fire-and-forget) // Update folder CLAUDE.md files for touched folders (fire-and-forget)
// This runs per-observation batch to ensure folders are updated as work happens // This runs per-observation batch to ensure folders are updated as work happens
const allFilePaths: string[] = []; // Only runs if CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED is true (default: false)
for (const obs of observations) { const settings = SettingsDefaultsManager.loadFromFile(USER_SETTINGS_PATH);
allFilePaths.push(...(obs.files_modified || [])); // Handle both string 'true' and boolean true from JSON settings
allFilePaths.push(...(obs.files_read || [])); const settingValue = settings.CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED;
} const folderClaudeMdEnabled = settingValue === 'true' || settingValue === true;
if (allFilePaths.length > 0) { if (folderClaudeMdEnabled) {
updateFolderClaudeMdFiles( const allFilePaths: string[] = [];
allFilePaths, for (const obs of observations) {
session.project, allFilePaths.push(...(obs.files_modified || []));
getWorkerPort(), allFilePaths.push(...(obs.files_read || []));
projectRoot }
).catch(error => {
logger.warn('FOLDER_INDEX', 'CLAUDE.md update failed (non-critical)', { project: session.project }, error as Error); if (allFilePaths.length > 0) {
}); updateFolderClaudeMdFiles(
allFilePaths,
session.project,
getWorkerPort(),
projectRoot
).catch(error => {
logger.warn('FOLDER_INDEX', 'CLAUDE.md update failed (non-critical)', { project: session.project }, error as Error);
});
}
} }
} }
@@ -121,6 +121,7 @@ export class SettingsRoutes extends BaseRouteHandler {
// Feature Toggles // Feature Toggles
'CLAUDE_MEM_CONTEXT_SHOW_LAST_SUMMARY', 'CLAUDE_MEM_CONTEXT_SHOW_LAST_SUMMARY',
'CLAUDE_MEM_CONTEXT_SHOW_LAST_MESSAGE', 'CLAUDE_MEM_CONTEXT_SHOW_LAST_MESSAGE',
'CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED',
]; ];
for (const key of settingKeys) { for (const key of settingKeys) {
+4 -1
View File
@@ -51,6 +51,7 @@ export interface SettingsDefaults {
// Feature Toggles // Feature Toggles
CLAUDE_MEM_CONTEXT_SHOW_LAST_SUMMARY: string; CLAUDE_MEM_CONTEXT_SHOW_LAST_SUMMARY: string;
CLAUDE_MEM_CONTEXT_SHOW_LAST_MESSAGE: string; CLAUDE_MEM_CONTEXT_SHOW_LAST_MESSAGE: string;
CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED: string;
} }
export class SettingsDefaultsManager { export class SettingsDefaultsManager {
@@ -96,6 +97,7 @@ export class SettingsDefaultsManager {
// Feature Toggles // Feature Toggles
CLAUDE_MEM_CONTEXT_SHOW_LAST_SUMMARY: 'true', CLAUDE_MEM_CONTEXT_SHOW_LAST_SUMMARY: 'true',
CLAUDE_MEM_CONTEXT_SHOW_LAST_MESSAGE: 'false', CLAUDE_MEM_CONTEXT_SHOW_LAST_MESSAGE: 'false',
CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED: 'false',
}; };
/** /**
@@ -122,10 +124,11 @@ export class SettingsDefaultsManager {
/** /**
* Get a boolean default value * Get a boolean default value
* Handles both string 'true' and boolean true from JSON
*/ */
static getBool(key: keyof SettingsDefaults): boolean { static getBool(key: keyof SettingsDefaults): boolean {
const value = this.get(key); const value = this.get(key);
return value === 'true'; return value === 'true' || value === true;
} }
/** /**