-
Notifications
You must be signed in to change notification settings - Fork 749
FIX: fixes multiple issues in the OpenViking chat functionality and unifies session ID generation logic between Python and Rust CLI implementations. #446
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 22 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
6a2e059
refactor(sandbox): remove docker/aiosandbox backends, simplify SRT co…
chenjw 9e7d877
refactor(sandbox): remove docker/aiosandbox backends, simplify SRT co…
chenjw e8aae41
fix: remove unused handle_chat_direct function and fix unused logs va…
chenjw 775ebd0
Fix UTF-8 issues in chat command
chenjw cda6b55
Add tab indentation to Think, Calling, and Result lines in CLI output
chenjw 92a1876
Add first release workflow
chenjw 8b4dee0
Update release workflow with correct working directory
chenjw 6049dc0
修改 SessionKey 构建逻辑:统一使用 type="cli",channel_id 默认 "default",chat_id 作为…
chenjw 0557391
Implement machine unique ID as default session ID for ov chat
chenjw 6eebae3
Remove unsupported --logs parameter from chat command
chenjw 21f236e
统一 Python 和 Rust CLI 的默认 session ID 生成逻辑
chenjw e2fc9ef
修改日志
chenjw 1d44c06
rebase main
chenjw b92e928
去掉log依赖
chenjw 7ceb2d2
docs: add VikingBot quick start section to READMEs
chenjw 59f4e87
fix: use vikingbot chat instead of ov chat in READMEs
chenjw 8fb714a
Revert "fix: use vikingbot chat instead of ov chat in READMEs"
chenjw fe82005
fix: use UUID v4 for machine ID generation in both Rust and Python
chenjw 86ed2e1
refactor: move truncate_utf8 to utils, fix chat history path, and use…
chenjw 824d15d
refactor: update machine ID generation and remove unused chat_v2
chenjw c1e9398
优化格式
chenjw 0f35f67
rebase
chenjw fa86efc
ruff format .
chenjw File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| name: First Release to PyPI | ||
|
|
||
| on: | ||
| workflow_dispatch: # 手动触发 | ||
|
|
||
| jobs: | ||
| release: | ||
| runs-on: ubuntu-latest | ||
| defaults: | ||
| run: | ||
| working-directory: bot | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: '3.11' | ||
|
|
||
| - name: Install build dependencies | ||
| run: pip install build | ||
|
|
||
| - name: Build package | ||
| run: python -m build | ||
|
|
||
| - name: Publish to PyPI | ||
| uses: pypa/gh-action-pypi-publish@release/v1 | ||
| with: | ||
| password: ${{ secrets.VIKINGBOT_PYPI_API_TOKEN }} | ||
| packages-dir: bot/dist/ |
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| name: Release to PyPI | ||
chenjw marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| on: | ||
| push: | ||
| tags: | ||
| - 'v*' | ||
|
|
||
| jobs: | ||
| release: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| permissions: | ||
| id-token: write | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Set up Python | ||
| uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: '3.11' | ||
|
|
||
| - name: Install build dependencies | ||
| run: | | ||
| pip install build | ||
|
|
||
| - name: Build package | ||
| run: | | ||
| python -m build | ||
|
|
||
| - name: Publish to PyPI | ||
| uses: pypa/gh-action-pypi-publish@release/v1 | ||
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
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
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
This file was deleted.
Oops, something went wrong.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| #!/bin/bash | ||
| BOT_DIR="$HOME/.openviking/data/bot" | ||
|
|
||
| echo "🧹 Cleaning VikingBot data directory..." | ||
| echo "📂 Cleaning contents of: $BOT_DIR" | ||
|
|
||
| if [ -d "$BOT_DIR" ]; then | ||
| echo "🗑️ Deleting items:" | ||
| for item in "$BOT_DIR"/*; do | ||
| if [ -e "$item" ]; then | ||
| echo " - $(basename "$item")" | ||
| rm -rf "$item" | ||
| fi | ||
| done | ||
| echo "✅ Done!" | ||
| else | ||
| echo "⚠️ Directory does not exist: $BOT_DIR" | ||
| fi | ||
|
|
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.[bot] 这种会不会好点
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
现在vikingbot还是源码安装,下个pr改成.[bot] 这样