Skip to content

fix(prd): document a fallback for memlog.py write failures - #2900

Draft
gmoncor wants to merge 1 commit into
bmad-code-org:devfrom
gmoncor:fix/s01-15-document-memlog-fallback-prd
Draft

gmoncor wants to merge 1 commit into
bmad-code-org:devfrom
gmoncor:fix/s01-15-document-memlog-fallback-prd

Conversation

@gmoncor

@gmoncor gmoncor commented Sep 17, 2026

Copy link
Copy Markdown

What: Add one fallback sentence to the File-roles line and to the memlog.py call sites stating that on failure, the entry should be appended inline tagged [UNLOGGED: <type> <text>], to be reconciled into .memlog.md during the existing Memlog audit Finalize step.

Why: Every memlog.py call site in this skill has no documented fallback, unlike the resolve_customization.py call in the same file which already documents one; an unhandled failure silently drops a decision-trail entry.

How:

  • Add the fallback clause to the File-roles line and to the Update/headless memlog.py call sites.
  • Specify where the inline note is persisted so it survives session close.

Testing: Read-through diff confirming the fallback location is durable and that the existing Memlog audit Finalize step is referenced as the reconciliation point.

Related to #2621 (concurrent appends can lose entries) — not the same defect: #2621 is about inter-process lock loss on concurrent writes, this is about the missing documented fallback in the consumer skill when memlog.py fails or is unavailable. Both touch memlog.py reliability as the decision-trail record.

🤖 Generated with Claude Code

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