Skip to content

Treat continue-mode hits as successful at iteration limit#16

Open
gorizontniy wants to merge 1 commit into
mainfrom
codex/web-run-unbounded-search
Open

Treat continue-mode hits as successful at iteration limit#16
gorizontniy wants to merge 1 commit into
mainfrom
codex/web-run-unbounded-search

Conversation

@gorizontniy

@gorizontniy gorizontniy commented May 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • return success when a bounded continue-after-success run already reserved at least one target IP
  • keep the existing error only for runs that reach max_iterations without any hit in the current run
  • add regression coverage for hybrid continue mode finding an IP and then ending at max_iterations=1

Why

A real run found and kept a target IP, sent Telegram, and protected the success cloud/folder. Because continue_after_success=true and the old runtime had max_iterations=1, the loop ended afterward and incorrectly returned code 2 with No requested IP reserved after max_iterations=1.

Tests

  • python -m unittest .\bin\test_yc_ip_hunter.py -v
  • python -m unittest discover -s .\bin -p "test_*.py" -v
  • git diff --check

Live-mode risk

This changes live-run exit status only after at least one target IP has already been reserved in the current run. It does not allocate extra resources by itself; it prevents successful bounded collection runs from being marked as failed.

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