Skip to content

point agent docs at resolve for identifier lookups#140

Merged
HumanBean17 merged 1 commit into
masterfrom
feat/resolve-tool-docs
May 15, 2026
Merged

point agent docs at resolve for identifier lookups#140
HumanBean17 merged 1 commit into
masterfrom
feat/resolve-tool-docs

Conversation

@HumanBean17
Copy link
Copy Markdown
Owner

Summary

  • PR-RESOLVE-2 (see plans/PLAN-RESOLVE-TOOL.md): agent-facing prose sweep so identifier workflows use resolve first; removes pre-resolve search + describe-per-candidate wording from MCP tool descriptions and docs.
  • describe_v2 FQN-collision hint now recommends resolve(identifier=…, hint_kind='symbol') instead of find / search chains.
  • Adds test_server_tool_descriptions_no_pre_resolve_fallback and renames the duplicate-FQN hint test.
  • No ontology bump; no graph schema changes.

Merge order: resolve (PR-RESOLVE-1) is already on master; this PR stacks on that.

Manual evidence

Sentinel grep (expect no output):

grep -En 'per\.candidate|until.*resolve|promising candidates|search\(query=.*\).*describe' \
  server.py mcp_v2.py docs/AGENT-GUIDE.md README.md AGENTS.md \
  docs/JAVA-CODEBASE-RAG-CLI.md docs/MANUAL-VERIFICATION-CHECKLIST.md \
  docs/skills/java-codebase-explore.md .cursor/rules/project-overview.mdc || true

Pytest:

.venv/bin/ruff check .
.venv/bin/python -m pytest tests/test_mcp_v2.py -v -k 'resolve or describe_by_fqn_duplicate_hint_points_to_resolve or tool_descriptions_no_pre_resolve'
PATH="$(pwd)/.venv/bin:$PATH" .venv/bin/python -m pytest tests -v

Test plan

  • Ruff clean
  • Sentinel grep reviewed (empty)
  • Focused + full tests (full suite: ensure .venv/bin is on PATH for CLI tests)

Made with Cursor

PR-RESOLVE-2: five-tool prose sweep, describe FQN hint, sentinel test.
Co-authored-by: Cursor <cursoragent@cursor.com>
@HumanBean17 HumanBean17 merged commit f3d3a56 into master May 15, 2026
1 check passed
HumanBean17 added a commit that referenced this pull request May 15, 2026
Track PR #135 and #140 as landed; fix cross-links for new paths.

Co-authored-by: Cursor <cursoragent@cursor.com>
@HumanBean17 HumanBean17 deleted the feat/resolve-tool-docs branch May 23, 2026 16:22
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