Skip to content

fix: concurrent single-page calibration scan and null-page planner JSON - #318

Merged
EricNGOntos merged 1 commit into
mainfrom
fix/wuchengke/calibration-scan-concurrent-pages
Aug 23, 2026
Merged

fix: concurrent single-page calibration scan and null-page planner JSON#318
EricNGOntos merged 1 commit into
mainfrom
fix/wuchengke/calibration-scan-concurrent-pages

Conversation

@EricNGOntos

Copy link
Copy Markdown
Contributor

Summary

  • Phase-1 title scan now inspects pages one-at-a-time concurrently within the existing (2,4,6,10) windows, with true/false + reason answers.
  • Continue to later windows when a page errors without a hit; earliest true page still wins within a round.
  • Tighten null-page ReAct planner prompt (prose fields, trailing-metadata strip rule) to stop DeepSeek emitting trailing JSON junk.

Test plan

  • make check (ruff + pyright)
  • pytest calibration scan / phase1 / structure anchoring contracts
  • CI green on PR

Made with Cursor

- Changed the calculation of inspect_calls to sum the number of scanned pages instead of rounds.
- Introduced concurrent page inspection to improve efficiency in the scan_title_forward function.
- Updated related functions and tests to reflect the new inspection logic, ensuring accurate results and error handling.
- Enhanced test cases to validate the new single-page inspection approach and its impact on scanning behavior.
@EricNGOntos
EricNGOntos merged commit 70d71ef into main Aug 23, 2026
6 checks passed
@EricNGOntos
EricNGOntos deleted the fix/wuchengke/calibration-scan-concurrent-pages branch August 23, 2026 11:59
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