Skip to content

Deepen repo graph view interface#67

Merged
jwalin-shah merged 1 commit into
mainfrom
codex/WP-085-shallow-module-deepening
May 14, 2026
Merged

Deepen repo graph view interface#67
jwalin-shah merged 1 commit into
mainfrom
codex/WP-085-shallow-module-deepening

Conversation

@jwalin-shah

Copy link
Copy Markdown
Owner

Summary

  • Add a concrete RepoGraphView facade that owns repo graph loading, adjacency, search, direct-import formatting, dependency checks, and report rendering.
  • Route dependency_report() and repo_graph_repl() through the facade so callers no longer duplicate adjacency construction or direct-import formatting.
  • Update the focused repo graph test to exercise the facade instead of helper assembly trivia.

Validation

  • PASS: python3 -m pytest tests/test_tensor_logic_core.py::TensorLogicCoreTest::test_repo_graph_helpers_and_report -q
  • PASS: git diff --check
  • FAIL: python3 -m pytest -q currently has 2 failures in Program.query arity-message assertions unrelated to this repo_graph_view change: tests expect "edge expects 2 symbols..." but current code raises "relation 'edge' expects 2 args...".

Residual Risk

  • Full-suite validation is blocked by the existing Program.query assertion/message mismatch outside the owned repo-graph surface.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@jwalin-shah
jwalin-shah merged commit 5bab9c9 into main May 14, 2026
1 check failed
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.

2 participants