Skip to content

set DEFAULT_COMMIT_TOKEN_THRESHOLD to 20000 for openclaw-plugin#1052

Merged
Mijamind719 merged 1 commit intovolcengine:mainfrom
wlff123:oc_ov_fix
Mar 28, 2026
Merged

set DEFAULT_COMMIT_TOKEN_THRESHOLD to 20000 for openclaw-plugin#1052
Mijamind719 merged 1 commit intovolcengine:mainfrom
wlff123:oc_ov_fix

Conversation

@wlff123
Copy link
Copy Markdown
Contributor

@wlff123 wlff123 commented Mar 28, 2026 •
edited
Loading

Description

Related Issue

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test update

Changes Made

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this on the following platforms:
    • Linux
    • macOS
    • Windows

Checklist

  • My code follows the project's coding style
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

Screenshots (if applicable)

Additional Notes

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 28, 2026

Failed to generate code suggestions for PR

@Mijamind719 Mijamind719 merged commit 8c57069 into volcengine:main Mar 28, 2026
7 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenViking project Mar 28, 2026
zeattacker pushed a commit to zeattacker/OpenViking that referenced this pull request Apr 1, 2026
@wlff123
set DEFAULT_COMMIT_TOKEN_THRESHOLD to 20000 for openclaw-plugin (volc…
b11a9e8 
…engine#1052)
zeattacker pushed a commit to zeattacker/OpenViking that referenced this pull request Apr 1, 2026
@claude
fix: resolve merge conflicts from upstream cherry-pick (volcengine#1052…
041f321 
…, volcengine#1053, volcengine#1054)

- Restore missing DEFAULT_EMIT_STANDARD_DIAGNOSTICS and DEFAULT_COMMIT_TOKEN_THRESHOLD constants
- Re-add alignment, profileInjection, recallFormat config fields lost in --strategy-option theirs
- Fix duplicate variable declarations (sessionId, localCacheKey) in index.ts
- Fix corrupted normalizeAssistantContent/tryLegacyCompact functions in context-engine.ts
- Fix orphaned try/catch blocks from interleaved afterTurn function
- Re-export roughEstimate, msgTokenEstimate, convertToAgentMessages, formatMessageFaithful, buildSystemPromptAddition
- Remove stale duplicate-registration guard code (superseded by upstream structure)
- Update commitTokenThreshold test default from 2000 to 20000

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mijamind719 Mijamind719 Mijamind719 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

OpenViking project
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wlff123 @Mijamind719