TYP: fix typing error in hookspecs, refactor NebariStage

[fangchenli]

Reference Issues or PRs

xref #1800 ##1894

What does this implement/fix?

Put a x in the boxes that apply

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds a feature)
• Breaking change (fix or feature that would cause existing features not to work as expected)
• Documentation Update
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Build related changes
• Other (please describe):

Testing

• Did you test the pull request locally?
• Did you add new tests?

Any other comments?

[iameskild]

Thanks @fangchenli! I left a few comments that I think we should discuss a bit further.

[iameskild]

@costrouc could you have a look at this PR?

[dcmcand]

@viniciusdc will check

[viniciusdc]

This has some big refactorings in the Nebari Stage which needs to be retested now that we changed a bit the behavior for the default schema dumps -- I will follow up on this, next week

[dcmcand]

@viniciusdc should we close this one or is there still value here?

[viniciusdc]

Hi @dcmcand, There is a lot of value here. This is good work that we will need to encompass sooner or later (related to that schema refactoring I was talking about recently). I was waiting on this due to the exposing default config values Adam was working on.

My opinion on this would be to close, and we will incorporate and cite those changes in a follow up PR