Skip to content

fix(cli): Pass include_archived param through model layer for list command#399

Merged
arockwell merged 2 commits intomainfrom
fix/list-command-include-archived
Feb 6, 2026
Merged

fix(cli): Pass include_archived param through model layer for list command#399
arockwell merged 2 commits intomainfrom
fix/list-command-include-archived

Conversation

@arockwell
Copy link
Owner

Summary

  • emdx list was broken with list_documents() got an unexpected keyword argument 'include_archived'
  • The model layer's list_documents() was missing the include_archived parameter that the CLI command passes
  • Added the parameter as a passthrough to db.list_documents() which already supported it

Test plan

  • emdx list works without --archived flag
  • emdx list --archived works and shows archived status column

🤖 Generated with Claude Code

arockwell and others added 2 commits February 5, 2026 22:41
…mmand

The `emdx list` command was broken because `models/documents.py:list_documents()`
did not accept the `include_archived` parameter that `commands/browse.py` was passing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@arockwell arockwell merged commit a180bbb into main Feb 6, 2026
1 check failed
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