Add --output-prefix flag to con-duct ls

[asmacdo]

Internal changes:

• Move DUCT_OUTPUT_PREFIX and EXECUTION_SUMMARY_FORMAT from duct_main.py to cli.py, renamed to DEFAULT_OUTPUT_PREFIX and DEFAULT_SUMMARY_FORMAT
• Break circular import by adding --output-prefix / -p to con-duct ls, so ls.py receives the prefix via args instead of importing it from duct_main

Fix latent bug:

• the output prefix default was evaluated at import time (before .env files were loaded); now os.getenv() runs at parser creation time inside main()

Closes #386

🤖 Generated with Claude Code

[codecov]

Codecov Report

❌ Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.56%. Comparing base (9b7a33a) to head (c814301).

Files with missing lines	Patch %	Lines
src/con_duct/ls.py	66.66%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #393      +/-   ##
==========================================
- Coverage   91.69%   90.56%   -1.14%     
==========================================
  Files          15       15              
  Lines        1120     1123       +3     
  Branches      139      139              
==========================================
- Hits         1027     1017      -10     
- Misses         70       84      +14     
+ Partials       23       22       -1     

☔ 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

This PR adds an --output-prefix (-p) option to con-duct ls and relocates CLI default values into cli.py to remove a circular import and ensure defaults are read after .env files are loaded.

Changes:

• Move CLI default constants from duct_main.py to cli.py (DEFAULT_OUTPUT_PREFIX, DEFAULT_SUMMARY_FORMAT).
• Add --output-prefix/-p to con-duct ls and refactor ls.py to derive the glob search prefix from args instead of importing from duct_main.
• Update tests to use the new constants and cover custom ls --output-prefix behavior.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/con_duct/cli.py	Introduces new default constants and uses env-evaluated defaults at parser creation; adds ls --output-prefix.
src/con_duct/duct_main.py	Removes CLI-default constants from the execution module.
src/con_duct/ls.py	Stops importing defaults and uses args.output_prefix to build the glob pattern when no paths are provided.
test/utils.py	Updates test helper to use new default constants from cli.py.
test/test_ls.py	Adds coverage for ls with default and custom output-prefix globbing.
test/duct_main/test_report.py	Updates imports to new summary format constant location.
test/duct_main/test_aggregation.py	Updates imports to new summary format constant location.

Comments suppressed due to low confidence (1)

src/con_duct/cli.py:287

• The help text for --output-prefix lists {datetime} as a supported format variable, but LogPaths.create() only provides pid and datetime_filesafe when calling output_prefix.format(...). Using {datetime} will raise a KeyError at runtime. Either add a datetime value to LogPaths.create() formatting, or remove {datetime} from the help/ docs to match actual behavior.

        help="File string format to be used as a prefix for the files -- the captured "
        "stdout and stderr and the resource usage logs. The understood variables are "
        "{datetime}, {datetime_filesafe}, and {pid}. "
        "Leading directories will be created if they do not exist. "

---

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

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (1)

src/con_duct/cli.py:287

• The help text says the understood variables are {datetime} and {pid}, but LogPaths.create() still supports {datetime_filesafe} for backwards compatibility (marked deprecated). Consider mentioning {datetime_filesafe} is supported-but-deprecated to avoid confusing users migrating older configs.

        default=os.getenv("DUCT_OUTPUT_PREFIX", DEFAULT_OUTPUT_PREFIX),
        help="File string format to be used as a prefix for the files -- the captured "
        "stdout and stderr and the resource usage logs. The understood variables are "
        "{datetime} and {pid}. "
        "Leading directories will be created if they do not exist. "

---

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