Improve llmdoc update workflow#28
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the llmdoc plugin to version 2.1.0, introducing a more flexible and efficient documentation update workflow. Key changes include the implementation of three update modes (fast, analysis, and full) to optimize the balance between speed and investigation depth, the introduction of a local temporary context cache in .llmdoc-tmp/ for investigator scratch reports, and an automated active-memory archiving process. Documentation across READMEs, agent prompts, and skills has been synchronized to reflect these new protocols. Review feedback identifies that the decisions/ directory should be excluded from the active memory count to ensure durable design decisions are not prematurely archived.
| - Reconcile `llmdoc/memory/doc-gaps.md`: close resolved gaps, mark stale gaps, and add only actionable new gaps with closure criteria. | ||
|
|
||
| 6. Run the active-memory archive check. | ||
| - After any new reflection is written, count active memory files under `llmdoc/memory/`, excluding `llmdoc/memory/lessons-learned.md`, `llmdoc/memory/doc-gaps.md`, and anything under `llmdoc/memory/archive/`. |
There was a problem hiding this comment.
The archiving logic currently counts all files under llmdoc/memory/ (except for a few exclusions) to determine if the threshold of 5 is met. However, llmdoc/memory/decisions/ is intended for durable design or process decisions, which should typically not be archived. Including them in the count might trigger the archiving process prematurely or lead to durable decisions being moved to the archive. Consider explicitly excluding the decisions/ directory from the active memory count.
| - After any new reflection is written, count active memory files under `llmdoc/memory/`, excluding `llmdoc/memory/lessons-learned.md`, `llmdoc/memory/doc-gaps.md`, and anything under `llmdoc/memory/archive/`. | |
| - After any new reflection is written, count active memory files under llmdoc/memory/, excluding llmdoc/memory/lessons-learned.md, llmdoc/memory/doc-gaps.md, llmdoc/memory/decisions/, and anything under llmdoc/memory/archive/. |
| - Keep `llmdoc/memory/doc-gaps.md` reconciled by closing resolved gaps and adding only actionable new gaps. | ||
|
|
||
| 6. Run the active-memory archive check. | ||
| - After any new reflection is written, count active memory files under `llmdoc/memory/`, excluding `llmdoc/memory/lessons-learned.md`, `llmdoc/memory/doc-gaps.md`, and anything under `llmdoc/memory/archive/`. |
There was a problem hiding this comment.
The archiving logic currently counts all files under llmdoc/memory/ (except for a few exclusions) to determine if the threshold of 5 is met. However, llmdoc/memory/decisions/ is intended for durable design or process decisions, which should typically not be archived. Including them in the count might trigger the archiving process prematurely or lead to durable decisions being moved to the archive. Consider explicitly excluding the decisions/ directory from the active memory count.
| - After any new reflection is written, count active memory files under `llmdoc/memory/`, excluding `llmdoc/memory/lessons-learned.md`, `llmdoc/memory/doc-gaps.md`, and anything under `llmdoc/memory/archive/`. | |
| - After any new reflection is written, count active memory files under llmdoc/memory/, excluding llmdoc/memory/lessons-learned.md, llmdoc/memory/doc-gaps.md, llmdoc/memory/decisions/, and anything under llmdoc/memory/archive/. |
1 participant
No description provided.