MAESTRO: Merge PR #913 - respect CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED setting

Folder CLAUDE.md generation is now gated behind the CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED setting (default: false). The setting is exposed via the settings API and handles both string and boolean JSON values. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-06 01:38:01 -05:00
parent 117710d449
commit 9c68a3ba3d
4 changed files with 62 additions and 61 deletions
@@ -36,4 +36,5 @@ These all modify `src/utils/claude-md-utils.ts` — review together.

 ## Folder CLAUDE.md Setting (winner from Phase 02 dedup)

- [ ] Review and merge PR #913 (`fix: respect CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED setting` by @superbiche). Files: `src/services/worker/agents/ResponseProcessor.ts`, `src/services/worker/http/routes/SettingsRoutes.ts`, `src/shared/SettingsDefaultsManager.ts`. This is the chosen PR from the 4-way duplicate group. Steps: (1) `gh pr checkout 913` (2) Verify the setting check is in the right place (before generating, not after) (3) Run `npm run build` (4) If clean: `gh pr merge 913 --rebase --delete-branch`
+- [x] Review and merge PR #913 (`fix: respect CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED setting` by @superbiche). Files: `src/services/worker/agents/ResponseProcessor.ts`, `src/services/worker/http/routes/SettingsRoutes.ts`, `src/shared/SettingsDefaultsManager.ts`. This is the chosen PR from the 4-way duplicate group. Steps: (1) `gh pr checkout 913` (2) Verify the setting check is in the right place (before generating, not after) (3) Run `npm run build` (4) If clean: `gh pr merge 913 --rebase --delete-branch`
+  - **MERGED** (2026-02-06): Clean rebase onto main, no conflicts. Three commits: (1) adds `CLAUDE_MEM_FOLDER_CLAUDEMD_ENABLED` to SettingsDefaults interface and DEFAULTS with value `'false'`, (2) adds the key to SettingsRoutes `settingKeys` array so it's exposed via GET/POST `/api/settings`, (3) wraps the `updateFolderClaudeMdFiles` call in ResponseProcessor with a settings check that handles both string `'true'` and boolean `true` from JSON. The setting defaults to `false`, meaning folder CLAUDE.md generation is now opt-in rather than always-on. Build passes. Credit to @superbiche for the clean implementation.