fix(task-review): accept org-native LAST_REVIEWED stamps, warn on bad ones

task-review-staleness.sh expected a bare 2026-07-09 and treated an org inactive timestamp ([2026-07-09 Thu], the form matching the CREATED:/CLOSED: cookies in the same drawer) as unparseable. The count branch folded that into the stale count, so a freshly-reviewed task reported as never-reviewed, and a full review pass never dropped the startup nudge. Both the bare and bracketed forms now normalize to the ISO date. A value that is neither warns loudly to stderr (file:line:value) and stays out of the count, since a data error shouldn't hide as "never reviewed." task-review.org documents the accepted format.

Tested red/green: 5 new bats cases (bracketed fresh and stale, list-mode sort by real date, malformed warns and is excluded). Full suite 273 ok.

1 files changed, 51 insertions, 2 deletions

diff --git a/.ai/scripts/tests/task-review-staleness.bats b/.ai/scripts/tests/task-review-staleness.bats
index 488b023..79aad79 100644
--- a/.ai/scripts/tests/task-review-staleness.bats
+++ b/.ai/scripts/tests/task-review-staleness.bats
@@ -49,6 +49,16 @@ task_unreviewed() {
   printf '** %s [#%s] %s\nBody.\n\n' "$keyword" "$prio" "$title" >> "$TODO"
 }
 
+# Emit a qualifying task whose LAST_REVIEWED is an org-native inactive
+# timestamp — [YYYY-MM-DD Day] — matching the CREATED:/CLOSED: cookies that
+# sit in the same drawer. The date is derived from an ISO date via `date`.
+task_reviewed_org() {
+  local keyword="$1" prio="$2" title="$3" isodate="$4"
+  local org="[$(date -d "$isodate" '+%F %a')]"
+  printf '** %s [#%s] %s\n:PROPERTIES:\n:LAST_REVIEWED: %s\n:END:\nBody.\n\n' \
+    "$keyword" "$prio" "$title" "$org" >> "$TODO"
+}
+
 # ---- Normal cases ----------------------------------------------------
 
 @test "staleness: empty file reports zero" {
@@ -85,6 +95,20 @@ task_unreviewed() {
   [ "$output" = "2" ]
 }
 
+@test "staleness: org-native bracketed LAST_REVIEWED parses — recent is fresh" {
+  task_reviewed_org TODO A "Reviewed five days ago, org stamp" "$D5"
+  run bash "$SCRIPT" "$TODO" 30
+  [ "$status" -eq 0 ]
+  [ "$output" = "0" ]
+}
+
+@test "staleness: org-native bracketed LAST_REVIEWED parses — old is stale" {
+  task_reviewed_org TODO A "Reviewed forty days ago, org stamp" "$D40"
+  run bash "$SCRIPT" "$TODO" 30
+  [ "$status" -eq 0 ]
+  [ "$output" = "1" ]
+}
+
 # ---- Boundary cases --------------------------------------------------
 
 @test "staleness: age exactly equal to threshold is fresh" {
@@ -136,9 +160,23 @@ task_unreviewed() {
   [ "$output" = "0" ]
 }
 
-@test "staleness: malformed LAST_REVIEWED is treated as stale" {
+@test "staleness: malformed LAST_REVIEWED warns to stderr and is not counted" {
   task_reviewed TODO A "Bad date" "not-a-date"
-  run bash "$SCRIPT" "$TODO" 30
+  # stdout carries only the count — the malformed stamp is not folded in.
+  run bash -c "bash '$SCRIPT' '$TODO' 30 2>/dev/null"
+  [ "$status" -eq 0 ]
+  [ "$output" = "0" ]
+  # stderr carries the loud warning naming the offending value.
+  run bash -c "bash '$SCRIPT' '$TODO' 30 2>&1 1>/dev/null"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"not-a-date"* ]]
+  [[ "$output" == *"LAST_REVIEWED"* ]]
+}
+
+@test "staleness: malformed stamp is excluded while real stale tasks still count" {
+  task_reviewed TODO A "Real stale" "$D40"
+  task_reviewed TODO B "Broken stamp" "garbage"
+  run bash -c "bash '$SCRIPT' '$TODO' 30 2>/dev/null"
   [ "$status" -eq 0 ]
   [ "$output" = "1" ]
 }
@@ -161,6 +199,17 @@ task_unreviewed() {
   [[ "${lines[2]}" == *"Reviewed recently"* ]]
 }
 
+@test "staleness --list: org-native bracketed stamp sorts by its real date" {
+  task_reviewed TODO A "Bare recent" "$D5"
+  task_reviewed_org TODO B "Org-stamped old" "$D40"
+  run bash "$SCRIPT" --list "$TODO" 10
+  [ "$status" -eq 0 ]
+  # The org-bracketed old stamp must sort ahead of the bare recent one —
+  # proof it parsed to a real date rather than falling to 0000-00-00.
+  [[ "${lines[0]}" == *"Org-stamped old"* ]]
+  [[ "${lines[1]}" == *"Bare recent"* ]]
+}
+
 @test "staleness --list: takes only the requested count" {
   task_unreviewed TODO A "First"
   task_reviewed TODO B "Second" "$D40"