Skip to content

[codex] fix focus list array response#21

Merged
JeelyWu merged 1 commit into
masterfrom
codex/fix-focus-list-array-response
May 16, 2026
Merged

[codex] fix focus list array response#21
JeelyWu merged 1 commit into
masterfrom
codex/fix-focus-list-array-response

Conversation

@JeelyWu
Copy link
Copy Markdown
Owner

@JeelyWu JeelyWu commented May 16, 2026

Summary

  • Support both wrapped {focuses: [...]} and top-level array responses from the focus list endpoint.
  • Add a regression test for the array response shape observed against dida365.

Root Cause

focus ls decoded the focus list response only as an object with a focuses field, but the dida365 API can return a top-level JSON array.

Validation

  • make test
  • make smoke
  • Manual end-to-end CLI coverage for focus and the broader command set before this commit.

@JeelyWu JeelyWu marked this pull request as ready for review May 16, 2026 07:06
@JeelyWu JeelyWu merged commit 019d5a6 into master May 16, 2026
1 check passed
@JeelyWu JeelyWu deleted the codex/fix-focus-list-array-response branch May 16, 2026 14:24
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.

1 participant