Remove invalid required=True from click.argument with nargs=-1

[Copilot]

The @click.argument decorator was using required=True with nargs=-1, creating a semantic conflict—nargs=-1 means "0 or more" while required=True implies "at least 1". Click's argument class accepts the required parameter, but using it with variadic arguments is non-idiomatic.

Changes:

• Removed required=True from @click.argument decorator
• Added explicit validation in function body using click.UsageError
• Added required=True to --output option to prevent None value errors

Before:

@click.command()
@click.argument("files", nargs=-1, type=click.Path(exists=True), required=True)
@click.option("--output", "-o", help="Output markdown file")
def main(files, output):
    output_md = Path(output)  # output can be None
    # ...

After:

@click.command()
@click.argument("files", nargs=-1, type=click.Path(exists=True))
@click.option("--output", "-o", required=True, help="Output markdown file")
def main(files, output):
    if not files:
        raise click.UsageError("At least one file is required")
    output_md = Path(output)  # output is guaranteed non-None
    # ...

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.