Skip to content

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

Open
FaiziNerd wants to merge 1 commit into
grayhatdevelopers:mainfrom
FaiziNerd:feat/show-media-state
Open

feat(cli): show media state in list command #82#112
FaiziNerd wants to merge 1 commit into
grayhatdevelopers:mainfrom
FaiziNerd:feat/show-media-state

Conversation

@FaiziNerd

@FaiziNerd FaiziNerd commented Aug 7, 2026

Copy link
Copy Markdown

Summary

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

This allows users to distinguish ready, pending, and failed media 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.

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
Author

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

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