Handle missing project.version automatically

[nathanjmcdougall]

No description provided.

[codspeed-hq]

CodSpeed Performance Report

Merging #301 will improve performances by 25.37%

_{Comparing 299-handle-missing-projectversion-field-in-pyprojecttoml (da29758) with main (0c6fb50)}

Summary

⚡ 1 improvements

Benchmarks breakdown

	Benchmark	BASE	HEAD	Change
⚡	test_several_tools_add_and_remove	776 ms	618.9 ms	+25.37%

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 95.16908% with 10 lines in your changes missing coverage. Please review.

Project coverage is 96.59%. Comparing base (0c6fb50) to head (da29758).

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
src/usethis/_interface/readme.py	71.42%	4 Missing ⚠️
src/usethis/_interface/version.py	66.66%	3 Missing ⚠️
src/usethis/_core/badge.py	50.00%	1 Missing ⚠️
src/usethis/_core/show.py	85.71%	1 Missing ⚠️
src/usethis/_interface/ci.py	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #301      +/-   ##
==========================================
+ Coverage   95.03%   96.59%   +1.56%     
==========================================
  Files          61       66       +5     
  Lines        2759     2878     +119     
==========================================
+ Hits         2622     2780     +158     
+ Misses        137       98      -39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nathanjmcdougall]

There is definitely a performance regression to address. One idea would be: only read the pyproject.toml once for the whole call. When we "write" we just save to global state until the very last moment and then on packdown we actually do the write. The idea being, we can avoid doing as much IO.

Also, need to add test coverage for the following cases:

• not project section at all
• invalid dtype for project
• invalid dtype for project.dynamic
• Not needed for test coverage, but probably good to have a test which checks that we don't try and add project.version when project.dynamic contains "version".