Skip to content

fix(recall): ignore archived page titles#494

Open
Steve-too wants to merge 1 commit into
vouchdev:testfrom
Steve-too:fix/recall-ignore-archived-pages
Open

fix(recall): ignore archived page titles#494
Steve-too wants to merge 1 commit into
vouchdev:testfrom
Steve-too:fix/recall-ignore-archived-pages

Conversation

@Steve-too

Copy link
Copy Markdown
Contributor

Summary

  • exclude archived pages from the session-start recall digest
  • add a regression test covering archived page titles while keeping live pages visible

Closes #490

Tests

  • PYTHONPATH=src python3 -m pytest tests/test_recall.py
  • uv run --extra dev python -m pytest tests/test_recall.py
  • uv run --extra dev python -m ruff check src/vouch/recall.py tests/test_recall.py
  • git diff --check

@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e070bccc-91c1-401d-9158-63fec068268f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added tests tests and fixtures size: XS less than 50 changed non-doc lines labels Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

1 participant