Open
Conversation
Bot proxy endpoints (/chat, /chat/stream) previously required auth unconditionally, while other endpoints like /find use get_request_context which allows dev mode without API keys. This caused inconsistent behavior where ov find worked but ov chat failed with 401. Now bot endpoints use the same get_request_context dependency for consistent auth behavior across all endpoints. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Update URI utilities for better path handling - Improve memory type registry and updater - Enhance session extraction context provider - Update memory prompt templates Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- bot/ov_server.py: pass message timestamp to session.add_message() - client/session.py: pass created_at to underlying client - sync_client.py: add created_at parameter to add_message() When created_at is not provided, the server will use current time. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ground tasks - Add end_time check in tracer.info/set/error to avoid "Tried calling _add_event on an ended span" - Add AsyncioInstrumentor to auto-create child spans for asyncio.create_task - Add @tracer decorator to _run_memory_extraction for background task tracing - Add opentelemetry-instrumentation-asyncio dependency Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Failed to generate code suggestions for PR |
The openviking/telemetry/langfuse.py was not being imported anywhere in the openviking package. Project uses bot/vikingbot/integrations/langfuse.py instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove unused sys import - Sort import block alphabetically - Remove unused span variable Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- 合并两个方法，统一模板渲染逻辑 - 添加extract_context参数支持模板渲染 - 添加[content_template]日志标签用于调试events渲染问题 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1. tools.py: 导入缺失的 truncate_content 函数和 ToolContext 类型 2. tools.py: 修复 f-string 无占位符的问题 3. test_memory_utils.py: 删除引用不存在函数的测试 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This reverts commit ede6192.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Related Issue
Type of Change
Changes Made
Testing
Checklist
Screenshots (if applicable)
Additional Notes