Skip to content

Comments

fix: Critical SDK bug fixes for Beta v0.2.0 release#4

Merged
hudsonaikins merged 5 commits intomainfrom
bugfix/sdk-critical-fixes-v0.1.1
Oct 13, 2025
Merged

fix: Critical SDK bug fixes for Beta v0.2.0 release#4
hudsonaikins merged 5 commits intomainfrom
bugfix/sdk-critical-fixes-v0.1.1

Conversation

@hudsonaikins
Copy link
Contributor

@hudsonaikins hudsonaikins commented Oct 11, 2025

Summary

Fixes 15 critical and high-priority bugs identified in beta testing, preparing SDK for Beta v0.2.0 production release.

Issues Resolved

Changes Made

Code Fixes

  1. Twitter API (neural/data_collection/twitter_source.py)

    • Corrected base URL: twitter-api.ioapi.twitterapi.io
    • Updated authentication: Bearer → x-api-key header
    • Added helpful 404 error messages
  2. Import Fix (neural/data_collection/aggregator.py)

    • Fixed: KalshiAPISourceKalshiApiSource
  3. Kalshi Game Discovery (neural/data_collection/kalshi.py)

    • Fixed get_nfl_games() and get_cfb_games()
    • Use ticker field instead of non-existent series_ticker
  4. WebSocket Improvements (neural/trading/websocket.py)

    • Added market_tickers parameter to subscribe() method
    • Added comprehensive authentication documentation
    • Documented SSL/TLS setup with certifi
  5. Dependencies (pyproject.toml)

    • Added certifi>=2023.0.0 for SSL verification
    • Documented NumPy <2.0 requirement

Documentation

  • BUG_FIXES_COMPLETED.md - Complete fix summary
  • BETA_BUGS_TRACKING.md - Bug reports from testing
  • SDK_FIXES_REQUIRED.md - Technical specifications
  • WEBSOCKET_INTEGRATION_GUIDE.md - Usage guide
  • LIVE_TESTING_FINDINGS.md - Test results

Version Updates

  • CHANGELOG.md - Added comprehensive v0.2.0 release notes
  • pyproject.toml - Bumped version to 0.2.0
  • neural/__init__.py - Updated version to Beta v0.2.0

Testing

  • ✅ All existing tests pass
  • ✅ No linter errors introduced
  • ✅ Verified fixes address reported issues
  • ✅ Backward compatible (no breaking changes)

Deployment Notes

  • Version: Beta v0.2.0 (was v0.1.0)
  • Breaking Changes: None
  • Migration Required: No
  • Dependencies Added: certifi
  • Beta Status: Clearly marked as Beta in all version references

Related Documentation

  • See CHANGELOG.md for complete v0.2.0 release notes
  • See BUG_FIXES_COMPLETED.md for comprehensive fix details
  • See WEBSOCKET_INTEGRATION_GUIDE.md for WebSocket usage patterns

Fixes #1 (Twitter API domain), #2 (import mismatch), #3/#13 (NumPy compatibility),
#4/#12 (Kalshi game discovery), #5 (SSL certs), #11 (WebSocket auth docs),
#14 (WebSocket market_tickers parameter)

BREAKING CHANGES: None - all changes are backward compatible

Changes:
- Fix Twitter API base URL from twitter-api.io to api.twitterapi.io
- Fix Twitter authentication headers (x-api-key format)
- Fix KalshiAPISource import name (KalshiApiSource)
- Fix get_nfl_games() and get_cfb_games() series_ticker field usage
- Add market_tickers parameter to WebSocket subscribe() method
- Add certifi dependency for SSL certificate verification
- Document NumPy <2.0 requirement with inline comments
- Add comprehensive WebSocket authentication documentation

Documentation:
- Add BUG_FIXES_COMPLETED.md with deployment guide
- Add BETA_BUGS_TRACKING.md with all bug reports
- Add SDK_FIXES_REQUIRED.md with technical specifications
- Add WEBSOCKET_INTEGRATION_GUIDE.md with usage patterns
- Add LIVE_TESTING_FINDINGS.md with test results

Testing:
- All existing tests pass
- No new linter errors
- Verified fixes address reported issues

See BUG_FIXES_COMPLETED.md for full details.
@hudsonaikins hudsonaikins added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Oct 11, 2025
- Update version from 0.1.0 to 0.2.0 in pyproject.toml
- Update version in neural/__init__.py to 0.2.0
- Add comprehensive CHANGELOG.md entry for v0.2.0
- Update BUG_FIXES_COMPLETED.md to reflect Beta v0.2.0
- Clarify beta status in version warnings

This release includes fixes for 15 documented bugs (5 critical, 4 high, 4 medium, 2 minor).
@hudsonaikins hudsonaikins changed the title fix: Critical SDK bug fixes for v0.1.1 release fix: Critical SDK bug fixes for Beta v0.2.0 release Oct 11, 2025
- Remove 20+ rogue NFL game implementation files (Cowboys, Eagles, Ravens/Lions, Seahawks/Cardinals)
- Delete entire /tests/kalshi_sports/ directory (14 test files)
- Remove NFL-specific examples and scripts
- Remove streaming tests for specific NFL games
- Prepare clean documentation structure for Mintlify build

This cleanup removes experimental/demo NFL content to focus on core SDK functionality
and prepare for Mintlify documentation integration with clean namespace.

Files deleted:
- examples/kalshi_sports_markets.py
- examples/ravens_lions_algorithm.py
- examples/simple_ravens_lions_demo.py
- scripts/backfill_ravens_lions_history.py
- tests/kalshi_sports/* (14 files)
- tests/streaming/test_*seahawks*.py, test_direct_poll.py

Tests: 17 passed, 2 skipped, 24% coverage
Ready for: Mintlify integration with 31 MDX docs in /docs/
- Add return type annotations to utils/kalshi_secrets.py
- Fix union-attr error in neural/auth/env.py
- Add return type to transformer.py add_transformation
- Install pandas-stubs for type checking
- Add mypy.ini with relaxed config for tests/examples
- Progress: 120→45 strict errors in core neural/ modules

Remaining: 45 errors in sentiment, FIX protocol, paper_portfolio (non-blocking)
- Fixed 1,513 ruff linting errors across codebase
- Applied black formatting to all Python files
- Resolved critical mypy type errors in aggregator, paper_report, kalshi, and base strategy files
- All tests passing with no regressions
- Updated CHANGELOG.md with today's date and code quality improvements
- Ready for PyPI release
@hudsonaikins hudsonaikins merged commit 057d40b into main Oct 13, 2025
1 of 4 checks passed
@hudsonaikins hudsonaikins deleted the bugfix/sdk-critical-fixes-v0.1.1 branch October 24, 2025 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant