Fix compiled_code, reimplement sql as wrapper

[jtcohen6]

resolves #9502

Problem

• Refactor deferral: always merge_from_artifact & support all commands #9040 added defer_relation to all nodes any time --defer was provided
• We had routing logic for the clone command that depended on the presence of defer_relation
• Refactor deferral: always merge_from_artifact & support all commands #9040 updated this logic for sql context member, but not for compiled_code

Solution

• Update the logic for compiled_code context member
• Reimplement sql context member as a wrapper around compiled_code, to prevent future divergence

Checklist

• I have read the contributing guide and understand what's expected of me
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
• This PR includes type annotations for new and modified functions

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2d59a51) 88.03% compared to head (c523369) 88.01%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9503      +/-   ##
==========================================
- Coverage   88.03%   88.01%   -0.03%     
==========================================
  Files         167      167              
  Lines       22130    22126       -4     
==========================================
- Hits        19482    19474       -8     
- Misses       2648     2652       +4     

Flag	Coverage Δ
integration	85.56% <100.00%> (-0.04%)	⬇️
unit	62.36% <100.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[mikealfare]

If I'm following the logic correctly, I think this previously resulted in self.model.compiled_code. I want to make sure this is intentional since both are syntactically correct.

[jtcohen6]

🎩 Confirmed this fix locally by running an incremental model on dbt-snowflake

Before:

On model.my_project.my_model: /* {"app": "dbt", "dbt_version": "1.8.0a1", "profile_name": "snowflake", "target_name": "dev", "node_id": "model.my_project.my_model"} */
create or replace transient table analytics_dev.dbt_jcohen.my_model
         as
        (select * from analytics.analytics.my_model
        );

After:

On model.my_project.my_model: /* {"app": "dbt", "dbt_version": "1.8.0a1", "profile_name": "snowflake", "target_name": "dev", "node_id": "model.my_project.my_model"} */
create or replace transient table analytics_dev.dbt_jcohen.my_model
         as
        (

... tons of SQL ...
);