Skip to content

fix: repair application registration contracts - #77

Merged
codeforester merged 1 commit into
mainfrom
bug/55-20260802-application-registration
Aug 2, 2026
Merged

codeforester merged 1 commit into
mainfrom
bug/55-20260802-application-registration

Conversation

@codeforester

Copy link
Copy Markdown
Contributor

Summary

  • make App.name authoritative across Click materialization, direct invocation, run_app, help, version, and runtime identity
  • freeze registration transactionally and reject duplicates, late mutation, reentrant builds, and custom command-class name overrides
  • make module-level command decorators independently retrievable and runnable with stable cross-Click name inference
  • document the registration lifecycle and exported get_command_app contract

Validation

  • full 244-test suite on Click 8.4.2
  • full 244-test suite on Click 8.1.8
  • 147 downstream Base engine tests plus 10 subtests
  • isolated wheel and sdist build and artifact validation
  • adversarial concurrency, naming, decorator, and public API review

Closes #55

@codeforester
codeforester marked this pull request as ready for review August 2, 2026 20:38
@codeforester
codeforester merged commit 6cf7b09 into main Aug 2, 2026
17 checks passed
@codeforester
codeforester deleted the bug/55-20260802-application-registration branch August 2, 2026 20:39
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.

Repair application naming and registration contracts

1 participant