Skip to content

feat(hints): add hints field and rules to ResolveOutput#149

Merged
HumanBean17 merged 2 commits into
masterfrom
feat/hints-v2-resolve
May 16, 2026
Merged

feat(hints): add hints field and rules to ResolveOutput#149
HumanBean17 merged 2 commits into
masterfrom
feat/hints-v2-resolve

Conversation

@HumanBean17
Copy link
Copy Markdown
Owner

Summary

  • Extends ResolveOutput with hints and resolved_identifier (query-time only; no re-index).
  • Implements the four locked resolve hint templates in mcp_hints.py (120-char drop-on-overflow, wildcard/seed suppression).
  • Refactors resolve_v2 to a unified _resolve_finalize_success assembler so every success=True path echoes resolved_identifier and runs hint generation (wildcard → hints: [] via suppression).
  • Documents v2 resolve behavior in README and mentions advisory hints on the resolve MCP tool.

Plan: plans/PLAN-HINTS-V2.md § PR-A
Propose: propose/HINTS-V2-PROPOSE.md §3–§4, Appendix A

Test plan

  • .venv/bin/ruff check mcp_hints.py mcp_v2.py tests/test_mcp_hints.py
  • .venv/bin/python -m pytest tests/test_mcp_hints.py -v -k "resolve"
  • .venv/bin/ruff check .
  • PATH="$PWD/.venv/bin:$PATH" .venv/bin/python -m pytest tests -v (514 passed, 10 skipped)

Re-index: not required (ontology_version unchanged).

Made with Cursor

HumanBean17 and others added 2 commits May 16, 2026 15:16
Wire generate_hints resolve catalog through a unified success assembler so
wildcard and empty-match paths echo resolved_identifier and respect UC2c
suppression. No ontology or re-index change.

Co-authored-by: Cursor <cursoragent@cursor.com>
Document status-driven resolve hints; suppress when success is explicitly
false. Extend round-trip with status many and route/client none paths.
Assert resolve invariants on exception output.

Co-authored-by: Cursor <cursoragent@cursor.com>
@HumanBean17 HumanBean17 merged commit 9280d22 into master May 16, 2026
1 check passed
HumanBean17 added a commit that referenced this pull request May 16, 2026
Hints v2 shipped in #148/#149/#150. Per repo convention, propose and
plans move to completed/ after the code lands.
@HumanBean17 HumanBean17 deleted the feat/hints-v2-resolve branch May 23, 2026 16:21
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