Add an early throw for the this.querier and remove the non-null asserts in the cli codebase

[canova]

Main | Deploy preview

This is only a refactor without changing the behavior. I wanted to remove these TS non-null asserts for a while. Now this function checks the this.querier at the start of the function and early throws, which allows TS to reduce the type to non-null, so we don't have to include ! every time we access it.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.43%. Comparing base (a812a34) to head (77da7e9).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6145   +/-   ##
=======================================
  Coverage   83.43%   83.43%           
=======================================
  Files         342      342           
  Lines       36356    36356           
  Branches    10099    10195   +96     
=======================================
  Hits        30333    30333           
  Misses       5595     5595           
  Partials      428      428           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[fatadel]

Thanks for easing the devs' lives! 😁