Adding a `make_dict_unstructure_fn` breaks `omit_if_default=True`

• cattrs version: 1.10.0
• attrs version: 21.4.0
• Python version: 3.8.12
• Operating System: Manjaro Linux

Description

I'm yet to get a minimal working example, but it's something like so:

@attrs.define
class Whatever:
    foo: str = "foo"
    bar: str = "bar"
    thing: str = "thing"
    u_thing: str = "_thing"

whatever = Whatever(bar="baz")


c = cattrs.GenConverter(omit_if_default=True)

c.unstructure(whatever)  # Does omit if default!

unstruct_hook = cattr.gen.make_dict_unstructure_fn(
    Whatever,
    c,
    **{"u_thing": cattr.override(rename="_thing")}
)
c.register_unstructure_hook(Whatever, unstruct_hook)

c.unstructure(whatever)  # Does NOT omit if default :c

I do a lot of other things to the converters, but from my testing, this is what's breaking it.

I need those extra hooks because I need both thing and _thing to show up on the output, and attrs doesn't seem to allow that, so I need to rename it on the fly.

Start by reproducing the shown example with attrs, cattrs.GenConverter, and make_dict_unstructure_fn, then compare the generated hook with the converter's default behavior. Trace how the renamed u_thing field is handled and verify that omit_if_default=True still omits unchanged defaults while preserving the rename.