diff options
Diffstat (limited to 'modules/core/module-aiassist.js')
| -rw-r--r-- | modules/core/module-aiassist.js | 47 |
1 files changed, 31 insertions, 16 deletions
diff --git a/modules/core/module-aiassist.js b/modules/core/module-aiassist.js index 9965e6a..be5ebce 100644 --- a/modules/core/module-aiassist.js +++ b/modules/core/module-aiassist.js @@ -1530,13 +1530,21 @@ buildSystemPrompt() { try { var fs = this.terminal.fs; if (fs) { - var index = fs.fsReadJSON('/skills/index.json') || {}; - var lessonKeys = Object.keys(index).filter(function(k) { return k.indexOf('loop-lesson-') === 0; }); + // MAINT-020: lessons live in /lessons, not in /skills — writing them into + // the skills directory made auto-generated lessons show up as skills. + var index = fs.fsReadJSON('/lessons/index.json') || {}; + var lessonKeys = Object.keys(index); + if (lessonKeys.length === 0) { + // one-release fallback for lessons written by older builds + var legacy = fs.fsReadJSON('/skills/index.json') || {}; + lessonKeys = Object.keys(legacy).filter(function(k) { return k.indexOf('loop-lesson-') === 0; }); + } if (lessonKeys.length > 0) { var recentLessons = lessonKeys.sort().slice(-3); var lessonTexts = []; for (var li = 0; li < recentLessons.length; li++) { - var lc = fs.readFile('/skills/' + recentLessons[li] + '.md'); + var lc = fs.readFile('/lessons/' + recentLessons[li] + '.md') + || fs.readFile('/skills/' + recentLessons[li] + '.md'); if (lc) { var excerpt = lc.substring(0, 300); lessonTexts.push('[Lesson: ' + recentLessons[li] + '] ' + excerpt); @@ -4672,23 +4680,30 @@ buildSystemPrompt() { var suggestion = suggestions.join('\n'); try { var fs = this.terminal.fs; - var index = fs.fsReadJSON('/skills/index.json') || {}; - var autoSkills = Object.entries(index).filter(function(e) { return (e[1].tags || []).indexOf('auto-improvement') !== -1; }); - // One-time cleanup: remove all existing loop-lesson files on first run + // MAINT-020: lessons are kept in /lessons with their own index; /skills is + // reserved for real skills the user or the agent creates deliberately. + try { fs.mkdirRecursive('/lessons'); } catch (ex) {} + var index = fs.fsReadJSON('/lessons/index.json') || {}; + var autoSkills = Object.entries(index); + // One-time migration: earlier builds wrote loop-lesson-* into /skills. if (!this._loopLessonCleaned) { this._loopLessonCleaned = true; - autoSkills.forEach(function(e) { - try { fs.unlink('/skills/' + e[0] + '.md'); } catch (ex) {} - delete index[e[0]]; - }); - autoSkills = []; + try { + var sIdx = fs.fsReadJSON('/skills/index.json') || {}; + Object.keys(sIdx).filter(function(k) { return k.indexOf('loop-lesson-') === 0; }) + .forEach(function(k) { + try { fs.unlink('/skills/' + k + '.md'); } catch (ex) {} + delete sIdx[k]; + }); + fs.fsWriteJSON('/skills/index.json', sIdx); + } catch (ex) {} } // Age limit: remove lessons older than 24h var now = Date.now(); autoSkills = autoSkills.filter(function(e) { var created = new Date(e[1].created).getTime(); if (now - created > 86400000) { - try { fs.unlink('/skills/' + e[0] + '.md'); } catch (ex) {} + try { fs.unlink('/lessons/' + e[0] + '.md'); } catch (ex) {} delete index[e[0]]; return false; } @@ -4700,17 +4715,17 @@ buildSystemPrompt() { // FIFO rotation: if at cap, delete oldest before creating if (!isDuplicate && autoSkills.length >= 3) { var oldest = autoSkills[0]; - try { fs.unlink('/skills/' + oldest[0] + '.md'); } catch (ex) {} + try { fs.unlink('/lessons/' + oldest[0] + '.md'); } catch (ex) {} delete index[oldest[0]]; autoSkills.shift(); } if (!isDuplicate) { var skillName = 'loop-lesson-' + Date.now(); var content = '---\nname: ' + skillName + '\ndescription: Auto-saved from agent loop anomaly detection\ntags: [auto-improvement, agent-loop, auto]\ncategory: system\n---\n## Anomalies Detected\n\n' + JSON.stringify(anomalies, null, 2) + '\n\n## Suggestions\n\n' + suggestion + '\n\n## Context\n- Profile: ' + (this._activeLoopProfile ? this._activeLoopProfile.name : 'none') + '\n- Turns: ' + result.toolTurns + '\n- Tools used: ' + toolLog.map(function(t) { return t.name; }).join(', '); - fs.mkdirRecursive('/skills'); - fs.writeFile('/skills/' + skillName + '.md', content); + fs.mkdirRecursive('/lessons'); + fs.writeFile('/lessons/' + skillName + '.md', content); index[skillName] = { name: skillName, description: 'Auto-saved loop lesson', tags: ['auto-improvement', 'agent-loop', 'auto'], category: 'system', created: new Date().toISOString(), updated: new Date().toISOString(), anomalyTypes: anomalyTypes }; - fs.fsWriteJSON('/skills/index.json', index); + fs.fsWriteJSON('/lessons/index.json', index); } } catch (e) { /* VFS not available */ } var msg = '💡 Self-improvement: ' + anomalies.length + ' anomaly(ies)'; |