René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"agent: when adding file manager reads to _deps.py, agent must also update all transitive callers in tests","articleBody":"## Lesson learned from PR #903 (infer dependencies from setup.cfg)\n\nWhen adding a new file manager (e.g. `SetupCFGManager`) read to `_deps.py`'s `get_project_deps()` or `get_dep_groups()`, the agent correctly updated `tests/usethis/test_deps.py` but missed `tests/usethis/test_init.py`, which calls `write_simple_requirements_txt()` — a function that transitively calls `get_project_deps()`.\n\nThis caused 5 CI test failures:\n- `tests/usethis/test_init.py::TestWriteSimpleRequirementsTxt::*` all raised `UnexpectedSetupCFGIOError` because `SetupCFGManager` was not opened in the test context.\n\n## Root cause\n\nThe agent only searched for direct callers of `get_project_deps` and `get_dep_groups` in tests, not for **transitive** callers (functions that call those functions). `write_simple_requirements_txt()` in `_init.py` calls `get_project_deps()` but is tested in a different test file.\n\n## Recommendation\n\nWhen modifying `_deps.py` to add new file manager reads, the agent should:\n1. Search for **all** functions that transitively call the modified functions (not just direct callers).\n2. Run a broader set of tests — at minimum `tests/usethis/test_init.py`, `tests/usethis/test_deps.py`, and `tests/usethis/_core/test_core_tool.py` — not just `test_deps.py`.\n\nSee also the stored memory: \"When adding new file manager reads to _deps.py, ALL existing tests calling those functions must also open the corresponding file manager context.\"","author":{"url":"https://github.com/nathanjmcdougall","@type":"Person","name":"nathanjmcdougall"},"datePublished":"2026-04-04T02:31:32.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/1808/usethis-python/issues/1808"}