Fix mypy errors caught in 1.11.2

[eapolinario]

Why are the changes needed?

mypy 1.11.2 catches new errors in flytekit:

❯ make lint
mypy flytekit/core
flytekit/core/type_engine.py:439: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
flytekit/core/type_engine.py:942: error: Argument 1 to "append" of "list" has incompatible type "tuple[Any, type[Any]]"; expected "tuple[Any,
GenericAlias]"  [arg-type]
flytekit/core/type_engine.py:951: error: Argument 1 to "append" of "list" has incompatible type "tuple[Any, type[Any]]"; expected "tuple[Any, GenericAlias]"  [arg-type]
flytekit/core/type_engine.py:2166: error: Argument 1 to "append" of "list" has incompatible type "tuple[Any, type[Any]]"; expected "tuple[Any, GenericAlias]"  [arg-type]
flytekit/core/python_auto_container.py:211: error: Need type annotation for "env" (hint: "env: dict[<type>, <type>] = ...")  [var-annotated]
flytekit/core/array_node_map_task.py:83: error: Need type annotation for "transformer"  [var-annotated]
Found 5 errors in 3 files (checked 48 source files)
make: *** [lint] Error 1

What changes were proposed in this pull request?

This PR fixes the new errors and opportunistically updates the version of ruff in precommit.

Notice that these types added to handle dataclasses only paper over the problem. We have some code that doesn't make any sense in the python type system, for example

flytekit/flytekit/core/type_engine.py

Line 2197 in d22e8a8

) -> Type[dataclasses.dataclass()]: # type: ignore

. Ideally we should rip this all of this dataclass code out of flytekit.

How was this patch tested?

Setup process

Screenshots

Check all the applicable boxes

• I updated the documentation accordingly.
• All new and existing tests passed.
• All commits are signed-off.

Related PRs

Docs link

[eapolinario]

python/cpython#102699 is a discussion about having a baseclass for dataclasses that could be used here instead of Type[dataclasses.dataclass()] which is invalid. Unfortunately that is just a proposal for now.

[codecov]

Codecov Report

Attention: Patch coverage is 61.53846% with 5 lines in your changes missing coverage. Please review.

Project coverage is 51.15%. Comparing base (cd8216a) to head (ef4e713).
Report is 8 commits behind head on master.

Files with missing lines	Patch %	Lines
flytekit/core/type_engine.py	57.14%	3 Missing ⚠️
flytekit/core/array_node_map_task.py	50.00%	1 Missing ⚠️
flytekit/core/python_auto_container.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #2808       +/-   ##
===========================================
- Coverage   76.82%   51.15%   -25.67%     
===========================================
  Files         196      302      +106     
  Lines       20301    25341     +5040     
  Branches     2610     2613        +3     
===========================================
- Hits        15596    12964     -2632     
- Misses       4004    12270     +8266     
+ Partials      701      107      -594     

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