Skip to content

Fix MCP indicator source resolution - #623

Merged
saleh-mir merged 1 commit into
masterfrom
fix/mcp-indicator-source-resolution
Aug 27, 2026
Merged

saleh-mir merged 1 commit into
masterfrom
fix/mcp-indicator-source-resolution

Conversation

@saleh-mir

Copy link
Copy Markdown
Member

Summary

  • resolve indicator details from exported functions instead of assuming function names match filenames
  • filter package-module attributes out of list_indicators
  • correct MCP stochastic examples to use the public stoch name

This fixes details lookup for:

  • stoch
  • support_resistance_with_breaks
  • waddah_attar_explosion

Verification

  • all 174 indicators returned by list_indicators resolve successfully
  • direct registered MCP wrapper calls pass for all three mismatched names
  • 33 MCP tests pass
  • 178 indicator tests pass
  • Python compilation and git diff --check pass

Note: pytest plugin auto-loading was disabled because this environment has an unrelated incompatible AnyIO pytest plugin (_pytest.scope import error). No packages were changed.

@saleh-mir
saleh-mir merged commit d53f6d1 into master Aug 27, 2026
15 checks passed
@saleh-mir
saleh-mir deleted the fix/mcp-indicator-source-resolution branch August 27, 2026 10:00
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