Skip to content

feat: Skills ecosystem - Hooks/Knowledge/Streaming (clean rebuild) - #146

Open
raymondginger2018-sudo wants to merge 1 commit into
HKUDS:mainfrom
raymondginger2018-sudo:feat/upstream-skills-v2
Open

feat: Skills ecosystem - Hooks/Knowledge/Streaming (clean rebuild)#146
raymondginger2018-sudo wants to merge 1 commit into
HKUDS:mainfrom
raymondginger2018-sudo:feat/upstream-skills-v2

Conversation

@raymondginger2018-sudo

Copy link
Copy Markdown
Contributor

Overview

Clean rebuild of the skills ecosystem PR. The previous PR (#142) accidentally included runtime data files (SQLite databases, JSONL logs, empty files). This version contains only the actual skill implementations.

Included skills (3)

deepcode-hooks

  • Generic Pre/Post hook framework (beforeWrite/afterWrite/beforeCommand/afterCommand/onError)
  • hooks.py (968 lines) + SKILL.md + plugin.json

deepcode-knowledge

  • Obsidian-style knowledge base + KV memory system
  • knowledge_server.py + memory_manager.py + rebuild_vault_index.py (1389 lines) + SKILL.md + plugin.json

deepcode-streaming

  • SSE streaming API client
  • streaming_api.py (514 lines) + SKILL.md + plugin.json

What's excluded

  • No .db SQLite data files
  • No .jsonl log files
  • No empty placeholder files
  • No nested duplicate directories
  • No __pycache__ / .pyc artifacts

Verification

  • All Python files pass py_compile
  • Clean tree: 11 files, +3306 lines

Happy to address any feedback! 🙏

重建 PR HKUDS#142:仅包含 3 个有实际代码的 skill,排除运行时数据文件:

- deepcode-hooks: 通用 Pre/Post 钩子框架 (hooks.py)
- deepcode-knowledge: Obsidian 风格笔记 + KV 记忆系统 (knowledge_server.py + memory_manager.py + rebuild_vault_index.py)
- deepcode-streaming: SSE 流式 API 客户端 (streaming_api.py)

不含: .db SQLite 数据文件 / .jsonl 日志 / 空文件 / 嵌套重复目录
@raymondginger2018-sudo

Copy link
Copy Markdown
Contributor Author

Hi maintainers, this is a clean rebuild of the previous #142 which accidentally contained runtime data files (SQLite .db, .jsonl logs, empty placeholders).This PR contains only the 3 real skill implementations:- deepcode-hooks — Pre/Post hook framework (hooks.py, 968 lines)- deepcode-knowledge — Obsidian-style KB + KV memory (3 modules, 1389 lines)- deepcode-streaming — SSE streaming API client (514 lines)All verified with py_compile. No data files, no binaries, no nested duplicates. Ready for review.

@raymondginger2018-sudo

Copy link
Copy Markdown
Contributor Author

@Zongwei9888 The CI workflow (Linting and Formatting) for this PR is stuck in action_required -- GitHub requires a maintainer to approve running Actions on fork pull requests.

All code has been verified locally (npm run check / py_compile all pass, no conflicts). Could you please Approve and run workflows on this PR, or adjust repo Actions settings to allow fork PRs without approval?

Thanks!

@Zongwei9888

Copy link
Copy Markdown
Collaborator

Thank you very much! I will check later!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants