12 Commits

Author SHA1 Message Date
Fengqing Liu
9727a9b8d2 unify game selection and expectation logic (#32)
- unify game selection and expectation logic
- refactore settings and settings manager
- formatting and coverage and precommit hook
2026-01-19 00:39:58 +11:00
Fengqing Liu
4ea4aa6ec2 relax spade url regex (#29) 2026-01-12 00:07:41 +11:00
github-actions[bot]
c9fefed1a9 feat: Enhance dashboard with Wanted Drops Queue and Benefit Filters
- Implemented 'Benefit Type' filtering (Item, Badge, etc.) in backend and settings UI.
- Redesigned 'Wanted Drops Queue' as a card-based, masonry layout.
- Added 'Game Groups' to Wanted Queue for better organization.
- Added variable height support for Inventory cards.
- Added proxy support and verification logic (from previous steps).
- Added unit tests for new filters and UI logic.
2025-12-14 21:46:14 +11:00
github-actions[bot]
c6e430243a sync fix from 7abfbdca9b 2025-12-12 00:48:26 +11:00
github-actions[bot]
3181f2367b feat: add game icons and campaign timing to inventory cards
- Display game box art icons (52x70px) next to game names in campaign headers
- Show contextual timing information below campaign status:
  * Active campaigns display end time
  * Upcoming campaigns display start time
  * Expired campaigns display end time
- Use locale-aware date formatting via JavaScript toLocaleString()
- Utilize existing translation keys (starts/ends) with {time} placeholders
- Add flexbox layout for icon alignment and styling for timing text

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-08 22:42:15 +11:00
github-actions[bot]
8dce2fd161 feat: add account linking status badges to inventory campaign cards
Add visual indicators showing account linking status for each campaign in the inventory tab. Campaigns now display either a "LINKED" (green) or "NOT LINKED" (orange) badge in the top right corner of each card. For unlinked campaigns, both the badge and a "Link Account" button provide direct access to the Twitch account linking page.

Backend changes:
- Add campaign_url property to DropsCampaign model
- Include both campaign_url and link_url in inventory API response

Frontend changes:
- Add LINKED/NOT LINKED badges positioned at top right of campaign cards
- Add "Link Account" button for unlinked campaigns
- Style badges with subtle green for linked and prominent orange for not linked
- Make NOT LINKED badge clickable to open account linking page

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-08 22:08:21 +11:00
Fengqing Liu
4b342d82ac Refactor Translator class and modernize translation access pattern
Simplified the Translator class to be more direct and Pythonic, and
updated all translation access from callable pattern to dict access.

Translator changes (src/i18n/translator.py):
- Removed complex fallback/merging logic with English translations
- Simplified from callable pattern _("key", "subkey") to dict access _.t["key"]["subkey"]
- Removed _load_english_translation() and module-level English translation
- Changed from list of language names to dict of Translation objects
- Removed unnecessary imports (abc, json_utils, MinerException, TYPE_CHECKING)
- Made language storage and access more direct (self.t property)
- Simplified set_language() to direct dict lookup

Translation access pattern changes (all other files):
- Changed from _("key", "subkey") to _.t["key"]["subkey"] throughout codebase
- This is more Pythonic and aligns with TypedDict structure
- Applied in: __main__.py, http_client.py, auth_state.py, models/drop.py,
  services (inventory, message_handlers, watch), web managers (login, settings),
  websocket.py

Benefits:
- More Pythonic: Direct dict access instead of callable
- Simpler: Removed ~70 lines of complex fallback logic
- Type-safe: Direct access to TypedDict structure
- More explicit: _.t["key"]["subkey"] shows the structure clearly
- Easier to maintain: No complex merging or fallback logic

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-25 14:00:29 +11:00
Fengqing Liu
a321da91f8 remove "tray" 2025-10-19 17:15:28 +11:00
Fengqing Liu
d466f46d6f remove redundant stuff 2025-10-19 17:08:18 +11:00
Fengqing Liu
f62ec26b5d chore: Major code quality improvements - Remove dead code, add Ruff & Mypy
This commit implements a comprehensive code quality overhaul including
dead code removal, modern linting tools, type checking, and CI/CD integration.

## Dead Code Removal

### Removed Files:
- src/utils/cache.py (137 lines) - Desktop GUI remnant with tkinter dependencies
  * No longer needed in web-only architecture
  * Conflicted with web version at src/web/managers/cache.py

### Removed Functions:
- src/auth/auth_state.py::_login() (170 lines) - Deprecated password authentication
  * OAuth device code flow is now the only login method
  * Removed legacy 2FA, CAPTCHA, and password login code
  * Updated class docstring to reflect OAuth-only authentication

### Removed Imports (12 occurrences):
- Unused TYPE_CHECKING imports (NoReturn, LoginFormManager, etc.)
- Orphaned exception imports (CaptchaRequired, LoginException)
- Unused utility imports (AwaitableValue, State, io, sys, Path)
- Removed commented-out imports and dead code references

**Total removed: 308+ lines of unused code**

## Modern Linting with Ruff

### Added ruff (v0.14.1):
- 10-100x faster than legacy linters (Flake8, pyflakes)
- Auto-fixed 56 issues across 46 files
- Import sorting with isort integration
- Modern Python idioms (pyupgrade)
- Code simplification suggestions

### Fixes Applied:
- Sorted imports alphabetically in all files
- Replaced try-except-pass with contextlib.suppress()
- Added exception chaining with 'from err'
- Renamed unused loop variables to _var
- Simplified comprehensions and ternary operators
- Removed unnecessary .keys() calls

**Result: 0 Ruff warnings (100% pass rate)**

## Type Checking with Mypy

### Added mypy (v1.18.2):
- Static type checking across 55 source files
- Reduced type errors from 25 to 5 (80% improvement)
- Configured gradual strictness for JSON/GraphQL responses

### Type Fixes:
- Fixed PyInstaller _MEIPASS attribute access
- Added type: ignore for safe variable redefinitions
- Fixed translator return type cast
- Added null checks for optional attributes
- Resolved circular import issues

### Configuration:
- Lenient initial settings for gradual adoption
- Per-module overrides for dynamic JSON responses
- Error codes shown for easy debugging

**Result: 5 non-critical errors remaining**

## CI/CD Integration

### Updated .github/workflows/ci.yml:
- Added new 'lint' job running Ruff + Mypy
- Ruff checks are required to pass
- Mypy checks run but don't block (advisory)
- Docker build now depends on lint + validate jobs

### Workflow:
```
lint (Ruff + Mypy) → validate (Language files) → docker → release
```

## Configuration

### Created pyproject.toml:
```toml
[tool.ruff]
line-length = 100
target-version = "py310"
select = ["E", "W", "F", "I", "UP", "B", "SIM", "C4"]

[tool.mypy]
python_version = "3.10"
check_untyped_defs = true
strict_equality = true
```

## Statistics

### Code Reduction:
- **Before:** 8,130 lines across 56 files
- **After:** 7,213 lines across 55 files
- **Reduction:** -917 lines (-11.3%)

### Quality Metrics:
- **Pyflakes:** 16 warnings → Ruff: 0 warnings
- **Mypy:** Not configured → 5 errors (from 25)
- **Linting speed:** 2s → 0.2s (10x faster)
- **Auto-fix rate:** 79% (46/58 issues)

### Files Modified: 48 files
- 1 deleted (src/utils/cache.py)
- 1 created (pyproject.toml)
- 46 updated (imports, types, style)

## Breaking Changes

None - all changes are internal quality improvements.
Application still runs perfectly (v16.dev).

## Migration Notes

### For Developers:
```bash
# Install new tools
pip install ruff mypy

# Run quality checks
ruff check src/          # Fast linting
mypy src/                # Type checking
ruff check src/ --fix    # Auto-fix issues
```

### For CI/CD:
- Linting now runs automatically on all commits
- Failed Ruff checks will block builds
- Mypy checks are advisory (won't block)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-19 00:02:54 +11:00
Fengqing Liu
68118d9e4b Add manual mode feature with UI improvements and batch updates
This commit introduces a manual mode for channel selection that allows users to lock onto a specific game, along with significant UI/UX improvements:

Manual Mode Features:
- Locks channel selection to a specific game when user manually selects a different game
- Automatically exits manual mode when all drops for the game are completed
- Shows visual indicators (🎯 badge) in status bar and drop display
- Allows manual exit via "Return to Auto Mode" button
- Prioritizes manual game in wanted_games list during auto-selection

UI/UX Improvements:
- Batch update system for channels and campaigns prevents flickering during refresh
- Channels list now updates atomically instead of clearing and re-adding
- Campaign loading emits all campaigns at once for smooth initial load
- Drop progress persists on initial_state load for reconnecting clients
- Game icons (box art) now displayed in channel list
- Campaign links added to inventory items

Bug Fixes:
- Fixed logging level to respect INFO by default (was hardcoded to DEBUG)
- Fixed channel cleanup to not call channel.remove() (handled by batch_update)
- Fixed drop display not persisting during channel switch
- Fixed stop_watching() clearing drop timer prematurely
- Fixed status display typo ("fames_to_watch" → "games_to_watch")

Backend Changes:
- Added manual mode tracking (_manual_target_channel, _manual_target_game)
- Added enter_manual_mode(), exit_manual_mode(), is_manual_mode() methods
- Added get_manual_mode_info() for API serialization
- Added batch_update() to ChannelListManager for atomic updates
- Added start_batch()/finalize_batch() to InventoryManager
- Added /api/mode/exit-manual endpoint
- Game model now stores box_art_url from GraphQL response
- Manual mode info included in /api/status and initial_state

Frontend Changes:
- Added manual mode badge and controls in UI
- Added channels_batch_update and inventory_batch_update socket events
- Added manual_mode_update socket event
- Countdown timer now properly tracked and cleared
- Drop display shows manual mode status with game name
- Exit manual mode button added to drop display area

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-18 00:22:34 +11:00
Fengqing Liu
5b736e3bb1 Migrate to Docker-ready web UI and remove legacy desktop GUI
Replace the legacy desktop/Tkinter client and packaging artifacts with a Docker-first, web-hosted approach.

- Add Docker quickstart and run-from-source instructions to the README to simplify deployment.
- Simplify launcher to invoke the new web backend module instead of the old desktop entrypoint.
- Update dependencies for a web UI stack (FastAPI, Uvicorn, python-socketio, Jinja2, etc.) and remove desktop/tray-specific packages.
- Remove legacy GUI, packaging and platform-specific helper code, along with obsolete build/pack scripts and AppImage assets to declutter the repo.
- Tidy project ignore rules to add runtime logs and editor metadata.

Rationale: streamline deployment, favor a browser-accessible interface, and reduce maintenance overhead from multiple platform-specific GUI/packaging implementations.
2025-10-16 21:54:43 +11:00