aboutsummaryrefslogtreecommitdiffstats
path: root/docs/superpowers/plans/2026-09-24-calendar.md
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-10-31 12:43:21 +0100
committerDanilo M. <danix@danix.xyz>2026-10-31 12:43:21 +0100
commit2b6ecd2a1f624c12c071320ff1ed96404d75f839 (patch)
tree1d2eb2a875098e97558b4cbcff07fd32e841b7e9 /docs/superpowers/plans/2026-09-24-calendar.md
parentc9882b609ade04a8b17a89f6b05d51bf939e330c (diff)
downloadqtmaildir-2b6ecd2a1f624c12c071320ff1ed96404d75f839.tar.gz
qtmaildir-2b6ecd2a1f624c12c071320ff1ed96404d75f839.zip
fix: handle every rule answer, and keep its failure out of errorOccurred
A second Apply overwrote the pending token, so the first rule's answer was dropped as stale although its writes had landed: no undo, no pending count, no accounts. A write's answer cannot be superseded, so the token is gone and every ruleApplied() is handled. Pending counts are keyed by generation for the same reason, since a second Apply before the first count returned dropped the first Apply entirely. applyRule() no longer emits errorOccurred(). That path reverted an unrelated in-flight user write and its message was then overwritten by "nothing to change". The failure travels in the reply instead: ruleApplied(description, changes, error). openReadOnly() and writeTags() take an optional error out-parameter so the rule's path stays quiet. The window shows the error, still records and offers undo for whatever landed first, and names the rule from the reply's description. Item 212. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Diffstat (limited to 'docs/superpowers/plans/2026-09-24-calendar.md')
0 files changed, 0 insertions, 0 deletions