Commit Graph

568 Commits

Author SHA1 Message Date
Cal Corum
8d2cdc81fe fix: round refractor values to integers in display
All checks were successful
Ruff Lint / lint (pull_request) Successful in 19s
Cast current_value and next_threshold to int to avoid ugly floating
point numbers like 53.0/149.0 in the progress display.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 16:55:03 -05:00
cal
27ce8b3617 Merge pull request 'fix: add debug logging for successful refractor API responses' (#124) from fix/refractor-debug-logging into main
All checks were successful
Build Docker Image / build (push) Successful in 2m47s
2026-03-25 21:47:41 +00:00
Cal Corum
17d124feb4 fix: add debug logging for successful refractor API responses
All checks were successful
Ruff Lint / lint (pull_request) Successful in 9s
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 16:46:01 -05:00
Cal Corum
1c21f674c2 fix: add error logging and user-facing message for API failures
All checks were successful
Ruff Lint / lint (pull_request) Successful in 20s
- Log API error detail when refractor endpoint returns an error
- Show "Something went wrong" instead of misleading "No refractor data"
- Log empty response case for debugging

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 16:44:20 -05:00
Cal Corum
a9ef04d102 fix: use server-side pagination and fix limit=500 exceeding API max
- Switch from client-side pagination (fetch all, slice) to server-side
  (pass limit/offset/progress params to API)
- Fixes limit=500 rejection (API max is 100)
- Footer now shows total_count from API response
- progress=close filter delegated to API instead of client-side

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 16:43:58 -05:00
c285e9e12d Merge pull request 'fix: align refractor status command with API response schema' (#122) from fix/refractor-status-field-names into main
All checks were successful
Build Docker Image / build (push) Successful in 2m50s
2026-03-25 21:20:20 +00:00
Cal Corum
7a3c21f6bd fix: align refractor status command with API response schema
All checks were successful
Ruff Lint / lint (pull_request) Successful in 21s
- data.get("cards") → data.get("items") to match list endpoint response
- formula_value → current_value (API field name)
- card_type from top-level → track.card_type (nested in track object)
- Add limit=500 param so API returns all cards instead of default 10

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 16:18:07 -05:00
cal
b185874607 Merge pull request 'docs: update refractor test plan — resolve list endpoint gap, add API tests' (#121) from docs/update-refractor-test-plan into main
All checks were successful
Build Docker Image / build (push) Successful in 3m2s
2026-03-25 16:25:39 +00:00
Cal Corum
7191400b5b docs: update refractor test plan — resolve list endpoint gap, add API tests
All checks were successful
Ruff Lint / lint (pull_request) Successful in 17s
- Mark CRITICAL missing endpoint gap as resolved (database PR #173 merged)
- Add REF-API-06 through REF-API-10 covering the new GET /refractor/cards
  list endpoint (team filter, card_type, tier, progress, pagination)
- Update prerequisites, execution checklist, and time estimates
- Total test cases: 87 (was 82)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 11:19:40 -05:00
cal
fd89349f29 Merge pull request 'docs: note manual game testing in refractor test plan' (#120) from docs/test-plan-note into main 2026-03-25 06:28:55 +00:00
Cal Corum
3b36dc33ee docs: note manual game testing in refractor test plan
All checks were successful
Ruff Lint / lint (pull_request) Successful in 10s
Sections 7-8 (game simulation, tier-up notifications) are marked as
manual testing by Cal — impractical to automate via Playwright.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 01:28:21 -05:00
cal
cf1d9c372d Merge pull request 'test: add refractor integration test plan and preflight script' (#119) from test/refractor-integration-plan into main 2026-03-25 06:20:06 +00:00
Cal Corum
190aa88d43 test: add refractor integration test plan and preflight script
All checks were successful
Ruff Lint / lint (pull_request) Successful in 21s
82-case in-app test plan for Playwright automation covering /refractor status,
tier badges, post-game hooks, tier-up notifications, and edge cases.
Preflight script for quick deploy verification via curl.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-25 01:16:21 -05:00
cal
24a59e9670 Merge pull request 'fix: wire WP-14 tier-up notification embeds into post-game hook' (#118) from fix/wp14-wire-notifications into main
All checks were successful
Build Docker Image / build (push) Successful in 2m44s
2026-03-25 04:45:14 +00:00
Cal Corum
571a86fe7e fix: wire WP-14 tier-up notification embeds into post-game hook
All checks were successful
Ruff Lint / lint (pull_request) Successful in 25s
Replace the logging-only stub in logic_gameplay.py with the real
notify_tier_completion from helpers/refractor_notifs.py. Tier-up
events now send Discord embeds instead of just logging.

- Import notify_tier_completion from helpers.refractor_notifs
- Remove 16-line stub function and redundant inline logging
- Update tests: verify real embed-sending behavior, replace
  bug-documenting T1-5 diagnostic with shape validation guards

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-24 23:43:05 -05:00
cal
8c0c2eb21a test: refractor system comprehensive test coverage (#117)
test: add comprehensive refractor system test coverage (23 tests)

Covers TIER_NAMES/TIER_BADGES cross-module consistency, WP-14 tier_up dict
shape mismatch (latent KeyError documented), None channel handling, filter
combinations, progress bar boundaries, and malformed API response handling.
2026-03-24 21:06:13 +00:00
cal
55efdb39de Merge pull request 'fix: update evolution/cards endpoint to refractor/cards (#113)' (#114) from issue/113-update-evolution-cards-endpoint-to-refractor-cards into main
All checks were successful
Build Docker Image / build (push) Successful in 2m31s
2026-03-24 05:20:10 +00:00
Cal Corum
fddac59f7e fix: update test mock endpoint strings to refractor/cards/ (#114)
All checks were successful
Ruff Lint / lint (pull_request) Successful in 20s
Mock routing in _patch_db_get and _failing_db_get still checked for
"evolution/cards/" after the production endpoint was renamed, causing
all badge-presence assertions to pass vacuously (evo_state=None).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-24 00:19:23 -05:00
Cal Corum
45894c72ee fix: update evolution/cards endpoint to refractor/cards (#113)
Closes #113

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-24 00:19:23 -05:00
cal
8940965ff8 Merge pull request 'ci: switch buildx cache from registry to local volume' (#116) from ci/local-buildx-cache into main
All checks were successful
Build Docker Image / build (push) Successful in 2m51s
2026-03-24 02:17:17 +00:00
Cal Corum
6e45686457 ci: switch buildx cache from registry to local volume
All checks were successful
Ruff Lint / lint (pull_request) Successful in 1m20s
Replaces type=registry cache (which causes 400 errors from Docker Hub
due to stale buildx builders) with type=local backed by a named Docker
volume on the runner. Adds cache rotation step to prevent unbounded growth.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 21:16:08 -05:00
cal
f85ead1f60 Merge pull request 'ci: add dev tag trigger to Docker build workflow' (#115) from ci/dev-tag-workflow into main
Some checks failed
Build Docker Image / build (push) Has been cancelled
2026-03-23 21:10:24 +00:00
Cal Corum
9bbd5305ef ci: add dev tag trigger to Docker build workflow
All checks were successful
Ruff Lint / lint (pull_request) Successful in 15s
Allows deploying to dev environment by pushing a "dev" tag.
Dev tags build with :dev Docker tag instead of :production.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 16:10:08 -05:00
cal
187ae854ca Merge pull request 'feat: WP-14 tier completion notification embeds' (#112) from feature/wp14-tier-notifications into main 2026-03-23 20:26:22 +00:00
cal
dc128ad995 Merge branch 'main' into feature/wp14-tier-notifications
All checks were successful
Ruff Lint / lint (pull_request) Successful in 14s
2026-03-23 20:26:04 +00:00
cal
aa2fce94b8 Merge pull request 'feat: WP-13 post-game evolution callback hook' (#111) from feature/wp13-postgame-hook into main 2026-03-23 20:25:43 +00:00
cal
80344fe473 Merge branch 'main' into feature/wp13-postgame-hook
All checks were successful
Ruff Lint / lint (pull_request) Successful in 14s
2026-03-23 20:25:31 +00:00
Cal Corum
29f2a8683f fix: rename evolution/ to refractor/ endpoint and remove misplaced notifs module
All checks were successful
Ruff Lint / lint (pull_request) Successful in 14s
- Change `evolution/evaluate-game/` API call to `refractor/evaluate-game/` in
  complete_game() hook (was calling the wrong endpoint path)
- Update all test assertions in test_complete_game_hook.py to match the
  corrected endpoint path and update docstrings to "refractor" naming
- Remove helpers/evolution_notifs.py and tests/test_evolution_notifications.py
  from this PR — they belong to PR #112 (WP-14 tier notifications). The
  notify_tier_completion stub in logic_gameplay.py remains as the WP-14
  integration target.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 15:22:25 -05:00
Cal Corum
9940b160db fix: rename evolution to refractor terminology, fix tier names
All checks were successful
Ruff Lint / lint (pull_request) Successful in 12s
- Rename helpers/evolution_notifs.py -> helpers/refractor_notifs.py
- Rename tests/test_evolution_notifications.py -> tests/test_refractor_notifs.py
- Delete utilities/evolution_notifications.py (replaced by helpers/refractor_notifs.py)
- Update TIER_NAMES to canonical names: Base Card, Base Chrome, Refractor, Gold Refractor, Superfractor
- Update T4 embed title from "FULLY EVOLVED!" to "SUPERFRACTOR!"
- Update FOOTER_TEXT from "Paper Dynasty Evolution" to "Paper Dynasty Refractor"
- Update non-max tier embed title from "Evolution Tier Up!" to "Refractor Tier Up!"
- Add discord.abc.Messageable type annotation to notify_tier_completion channel param
- Update all test assertions to match new tier names and strings

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 15:12:59 -05:00
cal
d2410ab374 Merge pull request 'feat: /refractor status slash command (WP-11)' (#87) from ai/paper-dynasty-database#76 into main 2026-03-23 20:11:32 +00:00
cal
c85359ca5d Merge branch 'main' into ai/paper-dynasty-database#76
All checks were successful
Ruff Lint / lint (pull_request) Successful in 21s
2026-03-23 20:11:17 +00:00
Cal Corum
45d71c61e3 fix: address reviewer issues — rename evolution endpoints, add TIER_BADGES
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m32s
- Update module docstring: replace evolution/cards with refractor/cards,
  drop old tier names (Unranked/Initiate/Rising/Ascendant/Evolved), add
  correct tier names (Base Card/Base Chrome/Refractor/Gold Refractor/
  Superfractor)
- Fix API call: db_get("evolution/cards") → db_get("refractor/cards")
- Add TIER_BADGES dict {1:"[BC]", 2:"[R]", 3:"[GR]", 4:"[SF]"}
- Update format_refractor_entry to prepend badge label for T1-T4 (T0 has
  no badge)
- Add TestTierBadges test class (11 tests) asserting badge values and
  presence in formatted output
- Update test_player_name_in_output to accommodate badge-prefixed bold name

Dead utilities/evolution_notifications.py has no source file on this branch
(WP-14/PR #112 already delivered the replacement).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 15:08:39 -05:00
cal
7a50ab0bce Merge pull request 'feat: Refractor tier badge prefix in card embed title (WP-12) (#77)' (#88) from ai/paper-dynasty-database#77 into main 2026-03-23 20:05:44 +00:00
cal
3ce5aebc57 Merge branch 'main' into ai/paper-dynasty-database#77
All checks were successful
Ruff Lint / lint (pull_request) Successful in 16s
2026-03-23 20:05:29 +00:00
Cal Corum
3a85564a6d fix: remove unused Optional import
All checks were successful
Ruff Lint / lint (pull_request) Successful in 18s
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 14:55:03 -05:00
Cal Corum
911c6842e4 feat: WP-14 tier completion notification embeds
Adds helpers/evolution_notifs.py with build_tier_up_embed() and
notify_tier_completion(). Each tier-up gets its own embed with
tier-specific colors (T1 green, T2 gold, T3 purple, T4 teal).
Tier 4 uses a special 'FULLY EVOLVED!' title with a future rating
boosts note. Notification failure is non-fatal (try/except). 23
unit tests cover all tiers, empty list, and failure path.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 14:55:03 -05:00
Cal Corum
2c57fbcdf5 fix: remove dead real_notify import in test
All checks were successful
Ruff Lint / lint (pull_request) Successful in 23s
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 14:52:49 -05:00
Cal Corum
b04219d208 feat: WP-13 post-game callback hook for season stats and evolution
After complete_game() saves the game result and posts rewards, fire two
non-blocking API calls in order:
  1. POST season-stats/update-game/{game_id}
  2. POST evolution/evaluate-game/{game_id}

Any failure in the evolution block is caught and logged as a warning —
the game is already persisted so evolution will self-heal on the next
evaluate pass. A notify_tier_completion stub is added as a WP-14 target.

Closes #78 on cal/paper-dynasty-database

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 14:52:36 -05:00
Cal Corum
687b91a009 fix: rename test file and docstring to Refractor terminology
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m8s
Renames tests/test_card_embed_evolution.py to tests/test_card_embed_refractor.py
and updates the module-level docstring to use "Refractor tier progression" /
"Refractor API" instead of "evolution progress" / "evolution API".

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 12:31:38 -05:00
f4a57879ab Merge pull request 'ci: switch to tag-based Docker builds' (#110) from ci/tag-based-docker-builds into main 2026-03-23 17:21:57 +00:00
Cal Corum
f09470b1f1 ci: switch to tag-based Docker builds (already committed)
All checks were successful
Ruff Lint / lint (pull_request) Successful in 17s
2026-03-23 10:49:27 -05:00
Cal Corum
fcd2e33916 ci: switch to tag-based Docker builds
Replace branch/PR-triggered Docker builds with tag-only triggers.
Images are now built only when a CalVer tag is pushed
(git tag YYYY.M.BUILD && git push origin YYYY.M.BUILD).

- Remove calver, docker-tags, and gitea-tag reusable actions
- Add inline version extraction from tag ref
- Keep existing build cache config
- Update CLAUDE.md versioning and CI/CD sections

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 10:46:55 -05:00
Cal Corum
1f26020bd7 fix: move TIER_BADGES to module level and fix unknown tier fallback
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m25s
- TIER_BADGES dict moved from inside get_card_embeds() to module level
- Unknown tiers now show no badge instead of silently promoting to [SF]

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 10:39:56 -05:00
Cal Corum
cc02d6db1e fix: align badge labels with updated tier names
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m34s
Tier badges shifted to match updated spec:
  T1=[BC] Base Chrome, T2=[R] Refractor, T3=[GR] Gold Refractor, T4=[SF] Superfractor
T0 (Base Card) shows no badge. All 11 tests pass.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 10:32:10 -05:00
Cal Corum
5670cd6e88 fix: correct tier names and group variable convention
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m45s
Tier names updated per Cal's spec:
  T0=Base Card, T1=Base Chrome, T2=Refractor, T3=Gold Refractor, T4=Superfractor

Also renames refractor_group → group_refractor per project convention.
All 39 tests pass.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 10:30:47 -05:00
Cal Corum
fc8508fbd5 refactor: rename Evolution badges to Refractor tier names
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m24s
- Badge labels: [R] Refractor, [GR] Gold Refractor, [SF] Superfractor, [SF★] fully evolved
- Fix broken {e} log format strings (restore `as e` + add f-string prefix)
- Restore ruff.toml from main (branch had stripped global config)
- Update all test assertions for new badge names (11/11 pass)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 08:50:11 -05:00
Cal Corum
6b4957ec70 refactor: rename Evolution to Refractor system
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m34s
- cogs/evolution.py → cogs/refractor.py (class, group, command names)
- Tier names: Base Chrome, Refractor, Gold Refractor, Superfractor
- Fix import: helpers.main.get_team_by_owner
- Fix shadowed builtin: type → card_type parameter
- Tests renamed and updated (39/39 pass)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-23 08:48:31 -05:00
cal
e2ddaf75b7 Merge pull request 'fix: tighten ruff.toml + add CI lint step (#108)' (#109) from ai/paper-dynasty-discord#108 into main
All checks were successful
Build Docker Image / build (push) Successful in 1m24s
2026-03-23 12:55:30 +00:00
Cal Corum
bf7a8f8394 fix: tighten ruff.toml + add CI lint step (#108)
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m39s
Ruff Lint / lint (pull_request) Successful in 1m42s
- Remove global F841/F401 suppression; scope to legacy directories via
  per-file-ignores so new files outside those paths get full enforcement
- Add per-file-ignores covering all 26 pre-existing violations that
  currently block the pre-commit hook (E711/E713/E721/E722/F811/F821)
- Keep global ignores only for genuine project patterns:
  F403/F405 (star imports in __init__.py), E712 (SQLModel ORM ==),
  F541 (1000+ legacy f-strings, cosmetic, deferred cleanup)
- Add .gitea/workflows/ruff-lint.yml — ruff check on every PR to main,
  so violations are caught before merge even if hook was bypassed

Closes #108

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 01:37:24 -05:00
9167bc2f1c Merge pull request 'fix: daily check-in interaction migration + paperdex dupe detection' (#107) from fix/checkin-and-paperdex-fixes into main
All checks were successful
Build Docker Image / build (push) Successful in 58s
2026-03-23 04:43:04 +00:00