fix: align refractor status command with API response schema #122

cal · 2026-03-25T21:18:21Z

cal commented

2026-03-25 21:18:21 +00:00

Summary

The /refractor status command was returning "No refractor data found" because the cog used field names that didn't match the actual API response from GET /api/v2/refractor/cards.

data.get("cards") → data.get("items") — API returns items key
formula_value → current_value — correct API field name
card_type → track.card_type — nested inside track object
Added limit=500 param so the API returns all cards (was defaulting to 10)

Also fixes the apply_close_filter function which read the same wrong field name.

Test plan

/refractor status shows cards with correct values and progress bars
/refractor status card_type:batter filters correctly
/refractor status progress:close uses correct value for threshold comparison

## Summary The `/refractor status` command was returning "No refractor data found" because the cog used field names that didn't match the actual API response from `GET /api/v2/refractor/cards`. - `data.get("cards")` → `data.get("items")` — API returns `items` key - `formula_value` → `current_value` — correct API field name - `card_type` → `track.card_type` — nested inside `track` object - Added `limit=500` param so the API returns all cards (was defaulting to 10) Also fixes the `apply_close_filter` function which read the same wrong field name. ## Test plan - [ ] `/refractor status` shows cards with correct values and progress bars - [ ] `/refractor status card_type:batter` filters correctly - [ ] `/refractor status progress:close` uses correct value for threshold comparison

cal added 1 commit 2026-03-25 21:18:22 +00:00

fix: align refractor status command with API response schema

Ruff Lint / lint (pull_request) Successful in 21s

Details

7a3c21f6bd

- 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>

Claude approved these changes 2026-03-25 21:19:15 +00:00

Claude left a comment

Approved via pd-pr

Claude merged commit c285e9e12d into main

2026-03-25 21:20:21 +00:00

Claude deleted branch fix/refractor-status-field-names

2026-03-25 21:20:22 +00:00

Claude referenced this issue from a commit

2026-03-25 21:20:23 +00:00

Merge pull request 'fix: align refractor status command with API response schema' (#122) from fix/refractor-status-field-names into main

Sign in to join this conversation.

No reviewers