updating summarize cli for consistency with other subcmds.

[keighrim]

fixes#359 .

[marcverhagen]

Out of curiosity: what was wrong with the type hint in summarize.main?

[keighrim]

Nothing, I guess...? Probably due to copy-paste I did from other CLI modules. I reverted the change in the new force-push.

[Copilot]

Pull request overview

Aligns the mmif summarize subcommand’s CLI interface with other mmif CLI tools by standardizing input/output handling (positional input w/ STDIN support, optional output defaulting to STDOUT, and pretty printing), and centralizing stream opening/closing to avoid resource leaks.

Changes:

• Refactors summarize to accept positional input (or STDIN), optional -o/--output (or STDOUT), and -p/--pretty.
• Introduces open_cli_io_arg helper and migrates other CLI subcommands (describe, rewind, source) to use it.
• Adds/updates tests and documents the standard CLI I/O pattern in CONTRIBUTING.md.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
mmif/utils/cli/__init__.py	Adds open_cli_io_arg context manager for consistent stdin/stdout/file handling across CLI modules.
mmif/utils/cli/summarize.py	Updates summarize CLI to match conventions: positional input, optional output, pretty flag, stdin/stdout support.
mmif/utils/cli/describe.py	Switches output arg to path-like + uses open_cli_io_arg for input/output handling.
mmif/utils/cli/rewind.py	Switches MMIF_FILE/output to path-like + uses open_cli_io_arg for input/output handling.
mmif/utils/cli/source.py	Switches output to a path-like + uses open_cli_io_arg for output handling.
mmif/utils/summarizer/summary.py	Adds to_dict(), improves report() outfile flexibility, replaces debug prints with logging.
tests/test_utils_cli.py	Refactors MMIF fixtures and adds a new test suite for the updated summarize CLI behavior.
tests/mmif_examples.py	Makes sample loading path-safe via Path(__file__) instead of hard-coded relative paths.
CONTRIBUTING.md	Documents the standardized CLI I/O argument pattern using open_cli_io_arg.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

❌ Patch coverage is 72.64151% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.66%. Comparing base (503abe2) to head (c5c3d29).
⚠️ Report is 8 commits behind head on develop.

Files with missing lines	Patch %	Lines
mmif/utils/cli/__init__.py	78.94%	8 Missing ⚠️
mmif/utils/summarizer/summary.py	61.90%	8 Missing ⚠️
mmif/utils/cli/describe.py	25.00%	6 Missing ⚠️
mmif/utils/cli/rewind.py	20.00%	4 Missing ⚠️
mmif/utils/cli/summarize.py	90.32%	3 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## develop #360 +/- ##
===========================================
+ Coverage 66.58% 71.66% +5.07% 
===========================================
Files 22 22 Lines 2846 2915 +69 ===========================================
+ Hits 1895 2089 +194 + Misses 951 826 -125 

Flag	Coverage Δ
unittests	71.66% <72.64%> (+5.07%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.