github-actions[bot] commented on code in PR #66319:
URL: https://github.com/apache/doris/pull/66319#discussion_r3690135855
##########
.github/workflows/code-review-runner.yml:
##########
@@ -647,6 +690,27 @@ jobs:
if [ -z "$failure_reason" ]; then
failure_reason="Codex exited with status $status"
fi
+
+ usage_limit_retry_after_epoch=""
+ if grep -Fqi "You've hit your usage limit" <<<"$failure_reason";
then
+ now_epoch="$(date +%s)"
+ retry_at_text="$(sed -nE \
+ 's/.*[Tt]ry again at ([A-Za-z]{3} [0-9]{1,2}(st|nd|rd|th)?,
[0-9]{4} [0-9]{1,2}:[0-9]{2} (AM|PM)).*/\1/p' \
Review Comment:
[P2] Parse Codex's same-day retry timestamp
Codex formats a reset on the current local day as a time-only value such as
`Try again at 5:01 PM`; it includes the month, day, and year only when the
reset is on another date. This regex accepts only the dated form, so a common
same-day limit falls through to `now + 30 minutes`. If the real reset is still
several hours away, that marker expires early and the next `/review` selects
the same exhausted account again, defeating the cooldown. Please parse and test
the time-only form as well as the full-date form while still requiring a future
UTC epoch.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
