• Joined on 2026-02-03
cal pushed to main at cal/major-domo-v2 2026-03-30 21:46:21 +00:00
8e02889fd4 Merge pull request 'feat: enforce trade deadline in /trade commands' (#121) from feature/trade-deadline-enforcement into main
b872a05397 feat: enforce trade deadline in /trade commands
Compare 2 commits »
cal merged pull request cal/major-domo-v2#121 2026-03-30 21:46:19 +00:00
feat: enforce trade deadline in /trade commands
cal created pull request cal/major-domo-v2#121 2026-03-30 21:44:06 +00:00
feat: enforce trade deadline in /trade commands
cal pushed to feature/trade-deadline-enforcement at cal/major-domo-v2 2026-03-30 21:39:15 +00:00
b872a05397 feat: enforce trade deadline in /trade commands
cal created branch feature/trade-deadline-enforcement in cal/major-domo-v2 2026-03-30 21:39:14 +00:00
cal commented on pull request cal/paper-dynasty-database#177 2026-03-30 18:20:24 +00:00
feat: Refractor Phase 2 integration — wire boost into evaluate-game

Re-review — commit 7f17c9b — APPROVED

cal pushed to feature/refractor-phase2-integration at cal/paper-dynasty-database 2026-03-30 18:18:53 +00:00
cal created branch feature/refractor-phase2-integration in cal/paper-dynasty-database 2026-03-30 18:18:52 +00:00
cal deleted branch feature/refractor-phase2-integration from cal/paper-dynasty-database 2026-03-30 18:17:32 +00:00
cal pushed to main at cal/paper-dynasty-database 2026-03-30 18:17:31 +00:00
31c86525de Merge pull request 'feat: Refractor Phase 2 integration — wire boost into evaluate-game' (#177) from feature/refractor-phase2-integration into main
7f17c9b9f2 fix: address PR #177 review — move import os to top-level, add audit idempotency guard
6a176af7da feat: Refractor Phase 2 integration — wire boost into evaluate-game
Compare 3 commits »
cal merged pull request cal/paper-dynasty-database#177 2026-03-30 18:17:30 +00:00
feat: Refractor Phase 2 integration — wire boost into evaluate-game
cal pushed to feature/refractor-phase2-integration at cal/paper-dynasty-database 2026-03-30 18:16:34 +00:00
7f17c9b9f2 fix: address PR #177 review — move import os to top-level, add audit idempotency guard
cal commented on pull request cal/paper-dynasty-database#177 2026-03-30 18:13:47 +00:00
feat: Refractor Phase 2 integration — wire boost into evaluate-game

Review — feat: Refractor Phase 2 integration

cal commented on pull request cal/paper-dynasty-database#177 2026-03-30 18:13:47 +00:00
feat: Refractor Phase 2 integration — wire boost into evaluate-game

import os here is unnecessary inside the function body — os is a stdlib module with no circular-import risk. Move to top-level imports. CLAUDE.md: "Never add lazy imports to middle of file."

cal commented on pull request cal/paper-dynasty-database#177 2026-03-30 18:13:47 +00:00
feat: Refractor Phase 2 integration — wire boost into evaluate-game

No idempotency guard on the audit write. If apply_tier_boost is called twice for the same (card_state_id, tier), _audit_model.create() will raise an IntegrityError in PostgreSQL (UNIQUE constraint from migration). Cards and ratings have guards above but audit does not. Add existing_audit = _audit_model.get_or_none(...) check before create. Tests pass today only because SQLite in-memory doesn't enforce the migration-only UNIQUE constraint.

cal created pull request cal/paper-dynasty-database#177 2026-03-30 18:06:34 +00:00
feat: Refractor Phase 2 integration — wire boost into evaluate-game
cal pushed to feature/refractor-phase2-integration at cal/paper-dynasty-database 2026-03-30 18:05:55 +00:00
6a176af7da feat: Refractor Phase 2 integration — wire boost into evaluate-game
cal created branch feature/refractor-phase2-integration in cal/paper-dynasty-database 2026-03-30 18:05:55 +00:00
cal deleted branch feature/refractor-phase2-foundation from cal/paper-dynasty-database 2026-03-30 16:11:10 +00:00
cal pushed to main at cal/paper-dynasty-database 2026-03-30 16:11:09 +00:00
70f984392d Merge pull request 'feat: Refractor Phase 2 foundation — boost functions, schema, tests' (#176) from feature/refractor-phase2-foundation into main
a7d02aeb10 style: remove redundant parentheses on boost_delta_json declaration
776f1a5302 fix: address PR review findings — rename evolution_tier to refractor_tier
4a1251a734 feat: add Refractor Phase 2 foundation — boost functions, schema, tests
Compare 4 commits »