make pk optional

[emmyoop]

Resolves #

Problem

Solution

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 or 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 ✅

Project coverage is 86.37%. Comparing base (bdf28d7) to head (50815a4).

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10973      +/-   ##
==========================================
- Coverage   89.14%   86.37%   -2.78%     
==========================================
  Files         183      183              
  Lines       23553    23553              
==========================================
- Hits        20996    20343     -653     
- Misses       2557     3210     +653     

Flag	Coverage Δ
integration	86.37% <100.00%> (-0.15%)	⬇️
unit	?

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

Components	Coverage Δ
Unit Tests	86.37% <100.00%> (-2.78%)	⬇️
Integration Tests	86.37% <100.00%> (-0.15%)	⬇️