hermes - 💡(How to fix) Fix Context compaction issues on minimax [1 participants]

TL;DR

The provided commit appears to address several issues with minimax 2.7, including context size handling, duplicate tool calls, and large tool call summaries, which could potentially resolve compaction issues.

Guidance

• Review the changes in the provided commit to understand the specific fixes applied, such as proper context size handling and removal of duplicate tool calls.
• Verify if these fixes are already tracked or addressed in other issues or pull requests to avoid duplication of effort.
• Consider testing the changes locally to confirm they resolve the compaction issues with minimax 2.7.
• Evaluate the potential impact of these fixes on other parts of the system to ensure no regressions are introduced.

Notes

The provided commit includes multiple fixes, and it's unclear which specific change resolves the compaction issue. Further testing and review are necessary to confirm the effectiveness of these changes.

Recommendation

Apply workaround: The changes in the provided commit seem to address the reported issues, but since they are not formally tested or reviewed, applying them as a workaround could potentially resolve the compaction problems with minimax 2.7, pending further verification and potential upstreaming.