[leowerneck]

Change Summary

Closes #355.

Overview

This PR introduces the following syntax:

imap-data-access query FILENAME

in addition to the standard (and still supported)

imap-data-access query [options]

Note that passing any querying options after (or before) FILENAME will result in an error, since all query parameters are extracted from FILENAME.

File changes

• imap_data_access/cli.py: changed how the CLI parses command line arguments.
• tests/test_cli.py: added tests for the new CLI syntax.
• examples/api_demo.ipynb: added new syntax example

Testing

Running:

uv sync --extra test
uv run pytest tests

results in 148 tests passing.

[Copilot]

🟡 Changes recommended

Legacy science filenames can return no results, and explicit default-valued query options bypass conflict validation.

2 open findings

• Legacy science filenames omit records when querying minor versions · New
• Reject filename queries with explicitly supplied default-valued options · New

What changed in this PR

Adds positional filename querying while retaining option-based queries.

Changes:

• Derives query parameters from science, ancillary, and SPICE filenames.
• Rejects filenames combined with query options.
• Adds tests and updates the API demo.

File	Description
imap_data_access/​cli.py	Implements filename-based query parsing and filtering.
tests/​test_cli.py	Tests supported filename types and conflicting options.
examples/​api_demo.ipynb	Updates the filename-query example.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟢 Approval recommended

The implementation addresses the reported query failures and is adequately covered by focused tests.

0 open findings

2 resolved since last review

• Reject filename queries with explicitly supplied default-valued options
• Legacy science filenames omit records when querying minor versions

🧠 Review effort: Balanced

[bryan-harter]

Looks good to me!