Skip to content

Fix doc review RLM inputs (2.3.1) - #55

Merged
khezen merged 1 commit into
mainfrom
fix/doc-rlm-inputs
Oct 20, 2026
Merged

khezen merged 1 commit into
mainfrom
fix/doc-rlm-inputs

Conversation

@khezen

@khezen khezen commented Oct 20, 2026

Copy link
Copy Markdown
Owner

Problem

Since 2.3.0, doc review failed on every scope with Unexpected inputs not declared in the signature: ['scope']. The RLM call passed scope= instead of the signature's patches / documentation inputs. The error was caught per scope, so doc review silently reported no issues.

Fix

  • doc/agent.py: pass patches, documentation, categories (+ prefrontal_memory when set) to run_rlm.
  • tests/test_doc_reviewer.py: regression test asserting run_rlm kwargs equal the signature's input fields (with and without prefrontal memory). Fails on the old call, passes with the fix.
  • Bump to 2.3.1 + CHANGELOG ### Fixed entry.

Validation

  • poetry run pytest tests/test_doc_reviewer.py tests/test_prefrontal_signature.py tests/test_rlm_tools.py: 22 passed, 1 skipped.
  • Full suite: 20 failures + 3 errors in cerebral / config_memory / doc_extractor / review_memories tests, none touched by this change.

@khezen
khezen merged commit 99099aa into main Oct 20, 2026
6 checks passed
@khezen
khezen deleted the fix/doc-rlm-inputs branch October 20, 2026 21:07
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