Skip to content

Conversation

@usama-ayaz
Copy link
Contributor

We have implemented the Girudo integration for Spellbot.

This includes the implementation of a Girudo service for LFG action. This enables the user to select Girudo as a platform for the "lfg" command and can play with other users in their Discord servers.

We are using multiple environment variables, which are listed as follows:

  • GIRUDO_BASE_URL
  • GIRUDO_AUTH_URL
  • GIRUDO_CREATE_URL
  • GIRUDO_LOBBY_URL
  • GIRUDO_STORE_DATA_URL
  • GIRUDO_EMAILS
  • GIRUDO_PASSWORDS
  • GIRUDO_DEFAULT_FORMAT_UUID
  • GIRUDO_DEFAULT_FORMAT_NAME
  • GIRUDO_DEFAULT_TCG_UUID
  • GIRUDO_DEFAULT_TCG_NAME
  • GIRUDO_DEFAULT_TCG_MAGIC_UUID
  • GIRUDO_RETRY_ATTEMPTS
  • GIRUDO_TIMEOUT_S

Note: We have reviewed our test coverage, and it satisfies the threshold that has been mentioned.

engineerraohammadali and others added 3 commits January 6, 2026 16:08
…tegration

- Add None checks and URL validation in girudo.py to ensure type safety.
- Improve type annotations in test_girudo.py and mocks/girudo.py.
- Align # type: ignore comments in tests to satisfy pyright.
- Fix ruff linting errors and format code.
- Create tests/integration/__init__.py to fix implicit namespace package warning.
- Sort exports in tests/mocks/__init__.py.
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

❌ Patch coverage is 87.88344% with 79 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.10%. Comparing base (28955a1) to head (cb7eeab).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
src/spellbot/integrations/girudo.py 72.72% 55 Missing and 17 partials ⚠️
tests/mocks/girudo.py 94.73% 4 Missing ⚠️
src/spellbot/client.py 25.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2138      +/-   ##
==========================================
- Coverage   95.56%   95.10%   -0.46%     
==========================================
  Files         198      202       +4     
  Lines       10330    10981     +651     
  Branches      430      470      +40     
==========================================
+ Hits         9872    10444     +572     
- Misses        407      468      +61     
- Partials       51       69      +18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@usama-ayaz usama-ayaz requested a review from lexicalunit January 7, 2026 13:08
@lexicalunit lexicalunit merged commit 6efac34 into lexicalunit:main Jan 8, 2026
5 of 7 checks passed
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.

3 participants