Skip to content

Cli parse refactor - #143

Merged
CodingInAVan merged 2 commits into
mainfrom
cli_parse_refactor
Jul 31, 2026
Merged

Cli parse refactor#143
CodingInAVan merged 2 commits into
mainfrom
cli_parse_refactor

Conversation

@CodingInAVan

Copy link
Copy Markdown
Contributor

Description

  • Split launcher CLI parsing into focused modules:
    • cli_help.cpp for help text
    • cli_parse_internal.* for shared flag/value and numeric parsing
    • cli_trace_options.* for trace option registration and generated help
    • cli_subcommand_options.* for upload, monitor, and info registries
  • Added reusable CliOptionManager<Args> with alias registration and duplicate-alias protection.
  • Migrated upload, monitor, and info option dispatch to typed registries.
  • Kept positional arguments, removed-option errors, and trace policy-heavy flags explicit.
  • Generated trace’s simple option help from the same registry used for parsing.
  • Hardened integer parsing against overflow while retaining supported leading-space and + forms.
  • Added coverage for integer overflow, inline boolean values, and generated trace help.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring

Testing

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

Planning and contract documents live in gpufl-manual
(client/session_segmentation/), not in the public client repo. The
compatibility matrix under docs/compatibility stays: it is reference
material for users of this client, not internal planning.

No code referenced the moved path.
@CodingInAVan
CodingInAVan merged commit e236790 into main Jul 31, 2026
4 checks passed
@CodingInAVan
CodingInAVan deleted the cli_parse_refactor branch July 31, 2026 23:45
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