Skip to content

docs(memory): record the staging-discipline trap and that @rules references rot unchecked - #786

Merged
pekral merged 1 commit into
masterfrom
memory-778
Aug 25, 2026
Merged

pekral merged 1 commit into
masterfrom
memory-778

Conversation

@pekral

@pekral pekral commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Summary

Compound-memory zápis po #785 podle @rules/compound-engineering/general.mdc Write protocol. Dva poznatky, oba prošly promotion barem.

Nový záznam — stage-by-path-when-the-tree-holds-unrelated-work. Workflow pro pre-existující nálezy chce každou opravu ve vlastním commitu ordered před feature commity — jenže ten nález obvykle najdeš, když už je feature rozpracovaná ve stejném pracovním stromě. git add -A v tu chvíli zabalí obojí a vznikne commit, jehož subject popisuje malou opravu, zatímco diff veze celou feature. Porušuje to logický rozklad dvakrát: jeden commit nese dva koncepty a feature skončí rozdělená horizontálně po vrstvách. Řešení: stage podle cesty, a když se oprava s feature potkají v jednom souboru, rekonstruovat přes reset --soft + checkout <base> -- <file>. A hlavně — partition ověřovat git show --stat per commit, ne ze subjectů; mispopsaný commit vypadá v git log --oneline správně.

Zesílený záznam — verify-a-referenced-skill-exists-before-copying-a-lens-list. Dosud mluvil o dangling jménu, které někdo zkopíruje dál. #785 ukázal ostřejší zobecnění: nic v composer build neověřuje, že @rules/* nebo @skills/* odkaz míří na existující soubor, takže odkaz shnije na místě ve chvíli, kdy se pravidlo přejmenuje nebo přesune — bez jediného červeného testu. security-review takhle tři ze čtyř deklarovaných bezpečnostních rule setů vůbec nenačítal.

Kurátorský průchod: první je nový, druhý podle Dedup zesiluje existující záznam místo zakládání druhého.

Jak otestovat

composer build — zeleně (394 testů, 2799 assertions, 100 % coverage). Soubor je projektová data, žádný pin test se ho nedotýká.

@pekral

pekral commented Aug 25, 2026

Copy link
Copy Markdown
Owner Author

Status: converged
Counts: Critical 0 · Moderate 0 · Minor 0 · Refactoring 0
Last updated: head of memory-778

Summary: assignment conformance: conformant — the compound-memory write mandated by @rules/compound-engineering/general.mdc Write protocol after PR #785 · reviewer comments: none · no linked issue — issue summary skipped · the diff touches docs/memory/PROJECT_MEMORY.md only: project data the installer never syncs, no executable line, no src/ change, so the coverage gate reports nothing and no Architecture or Database Analysis section applies. Curation pass honoured — the staging-discipline lesson is new, and the dangling-reference lesson strengthened the existing verify-a-referenced-skill-exists-before-copying-a-lens-list entry rather than adding a second per Dedup. composer build green — 394 tests, 2799 assertions, 100% coverage.

@pekral
pekral merged commit fff0519 into master Aug 25, 2026
1 check passed
@pekral
pekral deleted the memory-778 branch August 25, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant