Remove env_state from planning request

[Levi-Armstrong]

Having the env_state in the request introduces confusion on if the actual environment objects state matches this. This is removed and you should get the state from the environment object if needed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.71%. Comparing base (ad73309) to head (dfaeddb).
Report is 2 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #547      +/-   ##
==========================================
- Coverage   78.84%   78.71%   -0.13%     
==========================================
  Files         251      251              
  Lines       14833    14738      -95     
==========================================
- Hits        11695    11601      -94     
+ Misses       3138     3137       -1     

see 13 files with indirect coverage changes