Skip to content

fix(session-split): exclude archived pages from taken topics - #713

Merged
plind-junior merged 4 commits into
vouchdev:testfrom
kurosawareiji7007-hub:fix/session-split-exclude-archived
Jul 31, 2026
Merged

fix(session-split): exclude archived pages from taken topics#713
plind-junior merged 4 commits into
vouchdev:testfrom
kurosawareiji7007-hub:fix/session-split-exclude-archived

Conversation

@kurosawareiji7007-hub

Copy link
Copy Markdown
Contributor

Summary

  • session-split TAKEN TOPICS prompts and _file_drafts collision ignore ARCHIVED pages.
  • Reuses compile._live_pages so archive is not a one-way trap for session titles.

Fixes #712

Test plan

  • pytest tests/test_session_split_renarrate.py::test_renarrate_prompt_and_file_drafts_ignore_archived_pages
  • Confirm archived title absent from renarrate TAKEN TOPICS
  • Confirm _file_drafts can reuse an archived title

Prompts and draft collision treated archived session pages as taken,
same one-way trap compile had before vouchdev#700. Reuse compile._live_pages.

Fixes vouchdev#712
@github-actions github-actions Bot added docs documentation, specs, examples, and repo guidance tests tests and fixtures size: XS less than 50 changed non-doc lines labels Jul 31, 2026
Upstream imported coerce_numeric but still used bare int()/float(),
which left an unused-import ruff F401 that failed CI on vouchdev#713.
@github-actions github-actions Bot added size: S 50-199 changed non-doc lines and removed size: XS less than 50 changed non-doc lines labels Jul 31, 2026
kurosawareiji7007-hub added a commit to kurosawareiji7007-hub/vouch that referenced this pull request Jul 31, 2026
Upstream imported coerce_numeric but still used bare int()/float(),
which left an unused-import ruff F401 that failed CI on vouchdev#713.
kurosawareiji7007-hub added a commit to kurosawareiji7007-hub/vouch that referenced this pull request Jul 31, 2026
Upstream imported coerce_numeric but still used bare int()/float(),
which left an unused-import ruff F401 that failed CI on vouchdev#713.
changelog union only; no source conflict.
@github-actions github-actions Bot added size: XS less than 50 changed non-doc lines and removed size: S 50-199 changed non-doc lines labels Jul 31, 2026
changelog union only; no source conflict.
@plind-junior
plind-junior merged commit 48af1fd into vouchdev:test Jul 31, 2026
11 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

diff coverage: n/a — this PR changes no python under src/vouch/, so there is nothing for the gate to measure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: passing ci is green docs documentation, specs, examples, and repo guidance size: XS less than 50 changed non-doc lines tests tests and fixtures

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(session-split): archived pages block TAKEN TOPICS and draft collision

2 participants