Skip to content

feat: add DeepSeek Harness agent integration - #30

Merged
lijicode merged 2 commits into
MemTensor:mainfrom
leslie1992-dqp:dqp-memos-cloud-cli-nohook
Aug 17, 2026
Merged

feat: add DeepSeek Harness agent integration#30
lijicode merged 2 commits into
MemTensor:mainfrom
leslie1992-dqp:dqp-memos-cloud-cli-nohook

Conversation

@leslie1992-dqp

Copy link
Copy Markdown
Contributor

No description provided.

@Memtensor-AI Memtensor-AI added area:docs 文档、示例 status:in-progress Someone or AI is working on it | 人工或 AI 正在处理 labels Aug 17, 2026
@Memtensor-AI
Memtensor-AI requested a review from lijicode August 17, 2026 06:22
@Memtensor-AI

Copy link
Copy Markdown
Collaborator

🤖 Open Code Review

Target: PR #30
Task: a92a9dcf9969ee19
Base: main
Head: dqp-memos-cloud-cli-nohook

🔍 OpenCodeReview found 1 issue(s) in this PR.

⚠️ 4 warning(s) occurred during review.


1. src/memos_cli/commands/init.py (L120)

The expression Path(os.path.abspath(Path(dsh_home).expanduser())) mixes pathlib and os.path APIs unnecessarily: Path.expanduser() already returns a Path, which is then serialised to a string by os.path.abspath and wrapped in Path again. The idiomatic, equivalent replacement is Path(dsh_home).expanduser().resolve(), which avoids the round-trip and is consistent with how the codex branch resolves its path (Path(codex_home).expanduser()).

💡 Suggested Change

Before:

            return Path(os.path.abspath(Path(dsh_home).expanduser())) / "skills"

After:

            return Path(dsh_home).expanduser().resolve() / "skills"

Generated by cloud-assistant via Open Code Review.

@Memtensor-AI

Copy link
Copy Markdown
Collaborator

✅ Automated Test Results: PASSED

All tests passed (32/32 executed). memos_github_open_source/smoke: 1/1, memos_python_core/changed-repo-python: 31/31. Duration: 3s

Branch: dqp-memos-cloud-cli-nohook

@Memtensor-AI Memtensor-AI added status:ready Ready for implementation; waiting for assignee or AI dispatch | 可进入实现,等待认领或派发 and removed status:in-progress Someone or AI is working on it | 人工或 AI 正在处理 labels Aug 17, 2026
@lijicode
lijicode merged commit a6199ed into MemTensor:main Aug 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs 文档、示例 status:ready Ready for implementation; waiting for assignee or AI dispatch | 可进入实现,等待认领或派发

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants