[Bug] state:modified should include check_modified_contract

[jtcohen6]

Is this a new bug in dbt-core?

• I believe this is a new bug in dbt-core
• I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior

Following the implementation in:

• [Fix] Renaming or removing a contracted model raises a BreakingChange warning/error #10221

If I delete/rename a contracted model, I expect to see a warning/error when I dbt <command> --select state:modified.

But I only see this warning/error with dbt <command> --select state:modified.contracts

Expected Behavior

state:modified should include check_modified_contract: 14453eb

@T-Dunlap:

It's going to be really difficult to educate customers that they have to explicitly call out state:modified.contract in their CI jobs. It's best if this is baked directly into state:modified

@joellabes:

my understanding was that state:modified contained everything the more granular state:modified.* methods offered (and then some) and would not at all expect that there was something only available with a submethod

Steps To Reproduce

1. I deleted the model and all references to the model in the project.
2. I deleted the contract, which contained two versions
3. My downstream project was pointed to v1, which was technically deprecated.
4. No downstream models were pointed to v2.
5. No error or warning.

Relevant log output

No response

Environment

dbt: 1.9 / Versionless

Which database adapter are you using with dbt?

No response

Additional Context

Would this require a behavior-change flag? I don't think so, because it feels in keeping with the original intent of the issue, with what we've documented, and with what users would expect to happen

It also generally crops up in CI jobs, so unlikely to suddenly break someone's prod runs unless they're actually proposing code changes. (I think they'd need to be doing something wacky with dynamically enabling/disabling contracted versioned models)