Rename all_pb_names to all_product_block_names and refactor related logic

[saket-geant]

currently draft

[codspeed-hq]

CodSpeed Performance Report

Merging #968 will not alter performance

_{Comparing bug/914-allPbNames-issue (86a0293) with main (fc4c57e)}

Summary

✅ 12 untouched benchmarks

[codecov]

Codecov Report

Attention: Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 83.85%. Comparing base (fc4c57e) to head (86a0293).

Files with missing lines	Patch %	Lines
orchestrator/graphql/schemas/product.py	75.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #968      +/-   ##
==========================================
- Coverage   83.87%   83.85%   -0.02%     
==========================================
  Files         210      210              
  Lines       10186    10188       +2     
  Branches     1003     1004       +1     
==========================================
  Hits         8543     8543              
- Misses       1376     1377       +1     
- Partials      267      268       +1     

☔ View full report in Codecov by Sentry.
📢 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.

[Mark90]

Looking good!

There is a fixture test/unit_tests/fixtures/products/product_blocks/product_block_one_nested.py which has recursive product block relations.

Could you add a testcase using that fixture to test the recursion case?