Skip to content

feat(cli): show media state in list command #82 - #112

Merged
tulayha merged 2 commits into
grayhatdevelopers:mainfrom
FaiziNerd:feat/show-media-state
Aug 12, 2026
Merged

feat(cli): show media state in list command #82#112
tulayha merged 2 commits into
grayhatdevelopers:mainfrom
FaiziNerd:feat/show-media-state

Conversation

@FaiziNerd

@FaiziNerd FaiziNerd commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

vidxp media list now displays the existing MediaAsset.state value in a State column in the human-readable table (#82).

This allows users to see the currently supported media state without opening each media item.

The change is needed because the media state was already available to the CLI but was omitted from the human-readable table.

Only the existing ready state is currently supported. Additional media lifecycle states will be wired in a follow-up PR.

The existing JSON output and pagination contract are unchanged. No new lifecycle states, state-computation logic, dependencies, environment variables, downloads, or migrations are introduced.

This is a public-facing CLI output change.

Validation

  • Focused regression test:
    py -m uv run pytest -q tests/test_cli.py -k media_list_shows_media_state

    • Passed.
  • Manual CLI verification:
    py -m uv run vidxp media list

    • Confirmed the human-readable table includes the State column.
    • Confirmed the command runs successfully with the existing media list behavior.
  • Full test suite:
    py -m uv run pytest -q

    • 673 passed
    • 1 skipped
    • 1 failed: tests/test_frontend_app.py::FrontendAppTests::test_ready_page_renders_once_and_submits_first_search
    • The failure was an AppTest script run timed out after 10(s) timeout.

@tulayha

tulayha commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Please update PR title/description according to contributions guidelines and clarify any public facing change.
Is this against any of the open issues?

@FaiziNerd FaiziNerd changed the title Show media state in list command Show media state in the CLI media list #82 Aug 7, 2026
@FaiziNerd FaiziNerd changed the title Show media state in the CLI media list #82 feat(cli): show media state in list command Aug 7, 2026
@FaiziNerd FaiziNerd changed the title feat(cli): show media state in list command feat(cli): show media state in list command #82 Aug 7, 2026
@FaiziNerd

Copy link
Copy Markdown
Contributor Author

This is issue number 82. Improved the Title and the description of the PR. Do tell me if anything can be improved

@tulayha

tulayha commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

This allows users to distinguish ready, pending, and failed media without opening each media item.

Please adjust the wording. Currently, the enum only contains the ready state and the remaining states will need to be wired.
PR can be merged once wording is adjusted but will need another pr for the states. @FaiziNerd

@tulayha
tulayha merged commit e699e06 into grayhatdevelopers:main Aug 12, 2026
12 checks passed
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