Skip to content

feat(EXT-5): show workflow status in compact and full issue listings - #162

Merged
bougyman merged 3 commits into
mainfrom
EXT-5-show-workflow-status-in-listings
Aug 20, 2026
Merged

feat(EXT-5): show workflow status in compact and full issue listings#162
bougyman merged 3 commits into
mainfrom
EXT-5-show-workflow-status-in-listings

Conversation

@bougyman

Copy link
Copy Markdown
Member

Summary

  • Display the workflow state name (e.g. [In Progress]) after the issue identifier in both compact (issue list) and full (issue list --full) text output
  • State data was already fetched by the existing GraphQL base_fields query — this is a display-only change in CLI.Display.issue_line/1
  • Nil state is handled gracefully: brackets are omitted when no state is present

Changes

  • app/lib/linear_cli/cli/display.exissue_line/1 now prefixes the title with [state.name] when state is non-nil
  • app/test/linear_cli/cli/issue_commands_test.exs — added "state" field to issue_map/1 helper; added 3 new tests covering compact display, nil-state, and full display
  • app/test/linear_cli/cli/profile_defaults_test.exs — added "state" field to issue_map/1 helper to match real API responses

Test plan

  • mix test — all non-git tests pass (306 pass, 7 pre-existing GPG/git env failures)
  • mix credo --strict — no issues
  • mix format --check-formatted — no issues
  • New test: compact listing shows [In Progress] in output
  • New test: compact listing omits brackets when state is nil
  • New test: --full listing shows [In Progress] in header

Closes EXT-5

🤖 Generated with Claude Code

Display the workflow state name in brackets (e.g. [In Progress]) after
the issue identifier in both compact and full text output. The state data
was already fetched by the existing GraphQL query; this is a display-only
change.

Nil state is handled gracefully — the brackets are omitted when no state
is present.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 20, 2026 15:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

The three tests added at the bottom of the describe block
(compact with state, compact nil-state, full with state) are
equivalent to the three tests at lines 260-301.  Keep only the
lower set - it is slightly more thorough (includes a
projects-query guard) and uses body in Jason.decode! so no
unused-variable compiler warnings arise.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bougyman

Copy link
Copy Markdown
Member Author

Rework: addressed code review feedback

Review issues resolved:

  1. Duplicate tests removed — The three tests at the first insertion point (lines 260–301 in the original diff: "compact listing shows workflow status name in brackets", "compact listing omits status brackets when state is absent", "--full listing shows workflow status in the header line") have been removed. The equivalent tests in the second block ("compact listing includes workflow state name in brackets", "compact listing omits state bracket when state is nil", "--full listing includes workflow state name in header") are retained — they are slightly more thorough (include a projects-query guard) and have no unused-variable warnings.

  2. Unused `body` compiler warnings eliminated — Both warnings arose in the removed tests. The kept tests all use `body` in `Jason.decode!(body)` so no `_body` rename was needed.

Quality after rework:

  • `mix format --check-formatted`: PASS
  • `mix credo --strict`: PASS (no issues)
  • `mix test`: 303/310 pass, 7 pre-existing env failures (GitTest/FavoritesTest — unrelated to this change), 0 compiler warnings

@bougyman
bougyman merged commit acf35e5 into main Aug 20, 2026
2 checks passed
@bougyman
bougyman deleted the EXT-5-show-workflow-status-in-listings branch August 20, 2026 15:58
bougyman pushed a commit that referenced this pull request Aug 20, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.16.0](v1.15.2...v1.16.0)
(2026-08-20)


### Features

* **EXT-5:** show workflow status in compact and full issue listings
([#162](#162))
([acf35e5](acf35e5))


### Bug Fixes

* **EXT-4:** handle GraphQL error and partial-data responses explicitly
([#160](#160))
([31359ef](31359ef))
* **EXT-7:** bundle musl NIF for Linux Burrito releases and container
([#163](#163))
([3172d4e](3172d4e))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

2 participants