Update pre-commit hooks

[WilliamJamieson]

I noticed that several of the pre-commit hook versions were extremely out of date this PR updates them and fixes all the issues those updates uncovered.

I suggest that JWST use the pre-commit.ci bot like romancal does. It provides the style checks like the current workflow provides. In addition it provides in PR fixing tools and automatic updates to the pre-commit.ci config. If we don't want to go that root, something like https://github.com/marketplace/actions/pre-commit-autoupdate should be used.

Tasks

• If you have a specific reviewer in mind, tag them.
• add a build milestone, i.e. Build 12.0 (use the latest build if not sure)
• Does this PR change user-facing code / API? (if not, label with no-changelog-entry-needed)
  • write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see changelog readme for instructions)
  • update or add relevant tests
  • update relevant docstrings and / or docs/ page
  • start a regression test and include a link to the running job (click here for instructions) https://github.com/spacetelescope/RegressionTests/actions/runs/16198308588
    • Do truth files need to be updated ("okified")?
      • after the reviewer has approved these changes, run okify_regtests to update the truth files
• if a JIRA ticket exists, make sure it is resolved properly

[codecov]

Codecov Report

❌ Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (d875f84) to head (a1763ec).
⚠️ Report is 391 commits behind head on main.

Files with missing lines	Patch %	Lines
jwst/scripts/stfitsdiff.py	0.00%	6 Missing ⚠️
jwst/scripts/collect_pipeline_cfgs.py	0.00%	3 Missing ⚠️
jwst/scripts/schemadoc.py	0.00%	3 Missing ⚠️
jwst/scripts/world_coords.py	0.00%	2 Missing ⚠️
jwst/associations/lib/constraint.py	0.00%	1 Missing ⚠️
jwst/associations/mkpool.py	0.00%	1 Missing ⚠️
jwst/lib/set_telescope_pointing.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #9623   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files        367     367           
  Lines      37365   37313   -52     
=====================================
+ Misses     37365   37313   -52     

☔ 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.

[emolter]

This looks fine to me overall. It's a bit unfortunate that none of the lines that were touched have unit test coverage... it would be good to run regression tests just to be sure.

With regards to the pre-commit bot, it does sound good but TBH I'm probably the wrong person to ask

[pllim]

pre-commit bot

We have the bot on astropy and seems to work fine. Can get annoying if upstream keeps changing but also depends on your setup (opt-in vs opt-out). I think we should try it here.

[WilliamJamieson]

RT failed but I don't see how it is related? spacetelescope/RegressionTests/actions/runs/16180831251/job/45676952501

Those look like the result of a crds update?

[pllim]

RT try 2: https://github.com/spacetelescope/RegressionTests/actions/runs/16198308588

[pllim]

RT passed.

[melanieclarke]

LGTM, thanks.

I'll leave the bot conversation for later, if someone wants to pursue it (@tapastro @zacharyburnett ?)

[pllim]

Follow-up issue for bot at #9645