Skip to content

fix: make Wordstat view content gate soft - #14

Merged
axisrow merged 2 commits into
mainfrom
ao/wordstat-8/issue-13
Aug 21, 2026
Merged

axisrow merged 2 commits into
mainfrom
ao/wordstat-8/issue-13

Conversation

@axisrow

@axisrow axisrow commented Aug 21, 2026 •

Copy link
Copy Markdown
Owner

Closes #13

Summary

  • make table-text comparison in _select_view a separate required=False corroboration after the sole hard readiness gate (checked, download button, and table rows where applicable)
  • remove _is_view_active and retain the two-attempt timeout budget
  • preserve a table snapshot before the final map view so the next batch phrase gets its soft phrase-change nudge
  • document the soft-signal invariant and add regressions for both behaviors

Verification

  • uv run pytest — 112 passed
  • uv run ruff check . — passed
  • CDP endpoint http://127.0.0.1:9223: one-phrase collection and close-pair batch (ремонт квартир, ремонт квартир цена) completed after the rebase.

Live comparison with main

No row-count regression observed. This branch produced top_popular: 0, top_related: 0, dynamics: 24, and regions: 933 for ремонт квартир; the close-pair batch's second phrase produced 0, 0, 24, 692. The zero-row top reports reproduce on main before this change (for example, prior PR #7-era runs: купить телефон = 0/0/24/950, новогодние подарки = 0/0/24/765) and are documented as Wordstat behavior outside issue #13.

@axisrow
axisrow force-pushed the ao/wordstat-8/issue-13 branch from 48211f6 to bb11cae Compare August 21, 2026 02:50
@axisrow
axisrow marked this pull request as ready for review August 21, 2026 02:50
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.

Content-delta gate in _select_view should be soft (required=False), not hard

1 participant