chore: deprecate injectionContainer on agent

[TimoGlastra]

Marks the agent.injectionContainer as deprecated. We now have the dependencyManager that wraps the injection container from tsyringe.

This doesn't remove the property yet, but updates all usage of it in the codebase and marks it as deprecated so we can remove it in 0.4.0.

[jleach]

I didn't notice the deprecated API in 0.3.3. If you add the following VSCode will annotate it as deprecated. In any case I've updated Bifold to use dependencyManager and it seems to be working fine. Nice work!

/**
 * @deprecated The function should not be used
 */

[karimStekelenburg]

Strange... I'm also not seeing VSCode warnings although the @deprecated tag seems to have been added in packages/core/src/agent/BaseAgent.ts...

[karimStekelenburg]

LGTM

[TimoGlastra]

I didn't notice the deprecated API in 0.3.3.

Version 0.3.3 didn't have the @deprecated tag yet! I noticed in your issue that it was still available, and it should have been removed already in 0.3.0. This will give everyone that updates to 0.3.4 (when we release it) a warning to upgrade before 0.4.0

[codecov-commenter]

Codecov Report

Merging #1241 (2d2fbf4) into main (0f6d231) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1241      +/-   ##
==========================================
- Coverage   87.66%   87.64%   -0.02%     
==========================================
  Files         739      739              
  Lines       17619    17619              
  Branches     2992     2992              
==========================================
- Hits        15445    15443       -2     
- Misses       2167     2169       +2     
  Partials        7        7              

Impacted Files	Coverage Δ
packages/core/src/agent/BaseAgent.ts	92.94% <ø> (-2.36%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.