Preserved branch: queue/4541-3121-subtotal-match — decision needed (merge / cherry-pick / drop) #3547

Activity

1. dezren39

   added

   priority/lowNice to have; do when convenientNice to have; do when convenient

   on Nov 1, 2026
2. developing-today commented on Nov 1, 2026

   

   developing-todayBot

   on Nov 1, 2026 – with developing.today

   Contributor

   More actions

   CODE-1374
3. github-actions

   assigned

   dezren39

   on Nov 1, 2026
4. dezren39 commented on Nov 3, 2026

   

   dezren39

   on Nov 3, 2026

   MemberAuthor

   More actions

   🤖 Agent note · picked up

   • Agent: queue worker (repo-worker-code, OpenChamber scheduled lane 9aca43b4-5439-4939-9f71-323f5d32521b)
   • Task: queue task 7313 — triage Preserved branch: queue/4541-3121-subtotal-match — decision needed (merge / cherry-pick / drop) #3547
   • Disposition: verifying the preserved branch queue/4541-3121-subtotal-match against current main; no PR from that head yet, checking branch commits and any linked PR before deciding.
   • Links: Preserved branch: queue/4541-3121-subtotal-match — decision needed (merge / cherry-pick / drop) #3547
5. dezren39

   added

   queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

   status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

   work/assignedQueue worker lifecycle stateQueue worker lifecycle state

   on Nov 3, 2026
6. dezren39 commented on Nov 3, 2026

   

   dezren39

   on Nov 3, 2026

   MemberAuthor

   More actions

   Triage result: this preserved change has effectively landed via a different branch — no cherry-pick; the rescue must NOT be applied as-is.

   • Branch queue/4541-3121-subtotal-match exists on origin at 1bf0aaf (Nov 1 rescue of uncommitted work for fix(aggregator): charges producer body fallback matches 'total' inside 'Subtotal', recording the subtotal as the charge (PR #2149) #3121: amountFromBody with paid-labels-first/standalone-total fallback, import.meta.main guard, 5 tests). No PR from this head; the commit is not an ancestor of current main (8e76c6fa).
   • The same fix(aggregator): charges producer body fallback matches 'total' inside 'Subtotal', recording the subtotal as the charge (PR #2149) #3121 fix landed via PR #3164 (branch queue/4554-3121-subtotal-total, merged 2026-10-31 at faa3895c, ancestor of main), and main has since moved further: the test file is now 242 lines (amount-due exclusion fix(aggregator): charges producer turns declined purchase alerts and outstanding invoices into charge candidates (PR #2149) #3120, gmailMessageDay aggregator(charges-producer): one Gmail message with a missing/garbled date throws RangeError and discards the whole Gmail candidate batch #2448, staged mode Split charge-candidates producer: staged inputs staged on the sandbox, producer runs on the laptop #4641/aggregator(charges-producer): staged mode never validates the manifest period — a field-less manifest writes a candidates file with period {} and exits 0 #4663) and amountFromBody is the fuller variant.
   • Cherry-picking the rescue onto main would revert that later work, so this closes as superseded by fix(aggregator): body amount fallback — standalone total, paid labels win (#3121) #3164. Preserved branch left untouched.
7. dezren39

   closed this as completedon Nov 3, 2026
8. dezren39

   removed

   status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

   queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

   work/assignedQueue worker lifecycle stateQueue worker lifecycle state

   on Nov 3, 2026
9. dezren39 commented on Nov 3, 2026

   

   dezren39

   on Nov 3, 2026

   MemberAuthor

   More actions

   🤖 Agent note · done

   • Agent: queue worker (repo-worker-code, OpenChamber scheduled lane 9aca43b4-5439-4939-9f71-323f5d32521b)
   • Task: queue task 7313 — triage Preserved branch: queue/4541-3121-subtotal-match — decision needed (merge / cherry-pick / drop) #3547
   • Disposition: closed Preserved branch: queue/4541-3121-subtotal-match — decision needed (merge / cherry-pick / drop) #3547 as superseded by merged PR fix(aggregator): body amount fallback — standalone total, paid labels win (#3121) #3164 (same fix(aggregator): charges producer body fallback matches 'total' inside 'Subtotal', recording the subtotal as the charge (PR #2149) #3121 fix landed via a different branch; main now carries a superset). Cherry-pick deliberately not done — it would revert later main work. Preserved branch not deleted.
   • Evidence: fix(aggregator): body amount fallback — standalone total, paid labels win (#3121) #3164 · Preserved branch: queue/4541-3121-subtotal-match — decision needed (merge / cherry-pick / drop) #3547 (comment)