Skip to content

Conversation

matdev83
Copy link
Owner

@matdev83 matdev83 commented Oct 9, 2025

Summary

  • ensure HybridLoopDetector classifies detection_method using the configured short detector chunk size
  • add a regression test that exercises short pattern detection and verifies the reported method

Testing

  • python -m pytest --override-ini addopts="" tests/unit/loop_detection/test_hybrid_loop_result_details.py
  • python -m pytest --override-ini addopts=""

https://chatgpt.com/codex/tasks/task_e_68e6e2f655fc83339e3e4b89a878f07f

@matdev83
Copy link
Owner Author

APPROVED - This PR correctly fixes the HybridLoopDetector bug where detection_method was using a hardcoded threshold (500) instead of the configured short_detector.content_chunk_size. The fix is clean, well-tested with a comprehensive regression test, and all relevant tests pass. Safe to merge.

@matdev83 matdev83 merged commit 5b255da into dev Oct 10, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant