Skip to content

Fix failing e2e test: repomix_config not registered in create_all_assessors #400

@jwm4

Description

@jwm4

Bug

tests/e2e/test_critical_paths_simplified.py::TestCriticalConfigHandling::test_valid_config_application fails on main with:

Error: Invalid attribute ID(s) in config file: repomix_config

The test excludes repomix_config via a config file to verify that exclusion works. The assess command rejects it because repomix_config is not in the set of known attribute IDs.

Root cause

RepomixConfigAssessor is defined in src/agentready/assessors/repomix.py with attribute_id = "repomix_config" but is never registered in create_all_assessors() in src/agentready/assessors/__init__.py. The attribute validator only knows about registered assessors, so it treats repomix_config as an unknown ID.

Fix

Either register RepomixConfigAssessor in create_all_assessors() (the right fix if the assessor is intentionally part of the product), or update the test to exclude a valid attribute ID (if the assessor is intentionally unregistered and the test just needs a valid example).

Repro

pytest tests/e2e/test_critical_paths_simplified.py::TestCriticalConfigHandling::test_valid_config_application -v

Fails on main as of 4f99a69.


Opened by Claude Code under the supervision of Bill Murdock.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions