Skip to content

fix: align legacy "oh" naming with clawteam branding#128

Open
shadowinlife wants to merge 4 commits intoHKUDS:mainfrom
shadowinlife:fix/rename-oh-to-clawteam
Open

fix: align legacy "oh" naming with clawteam branding#128
shadowinlife wants to merge 4 commits intoHKUDS:mainfrom
shadowinlife:fix/rename-oh-to-clawteam

Conversation

@shadowinlife
Copy link
Copy Markdown

概述

项目从 oh / OpenHarness 更名为 ClawTeam 后,代码库中仍有大量残留的旧名称出现在 CLI 帮助文本、错误提示、文档和环境变量中。本 PR 统一清理这些遗留命名,同时保持对旧环境变量的向后兼容。

变更内容

代码修复(8 文件):

  • cli/commands.py — CLI docstring、用户提示中的 ohclawteam
  • harness/phases.py — 审批门控提示 oh harness approveclawteam harness approve
  • spawn/prompt.py — docstring 更新
  • spawn/tmux_backend.py — docstring 会话命名 + 错误信息
  • spawn/wsh_backend.py — 错误信息
  • identity.pyOH_PLAN_MODE_REQUIRED 迁移为 CLAWTEAM_PLAN_MODE_REQUIRED(主)+ OH_PLAN_MODE_REQUIRED(回退);_env() 参数名泛化
  • events/hooks.py — shell handler 同时导出 CLAWTEAM_{key}OH_{key} 环境变量

文档修复(6 文件):

  • README.md / README_CN.md / README_KR.md — 所有命令示例 ohclawteam
  • skills/clawteam/SKILL.md(×2)— 触发关键词标注为 legacy alias
  • skills/clawteam/references/cli-reference.md(×2)— 默认路径 ~/.oh~/.clawteam

向后兼容

环境变量 策略
OH_PLAN_MODE_REQUIRED 保留读取(作为 fallback)+ 继续导出
OH_{event_field} (hooks) 继续导出,同时新增 CLAWTEAM_ 前缀版本
OH_MSG_* (watcher) 未改动 — 内部消息传递合约,不属于本次范围

测试

92 tests passed,无回归。

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR cleans up remaining legacy oh / OpenHarness naming across the ClawTeam repo (CLI help text, user-facing messages, and documentation) while preserving backward compatibility via legacy env-var aliases.

Changes:

  • Updated CLI help text, prompts, and error messages to reference clawteam instead of oh.
  • Migrated env var usage to CLAWTEAM_* while keeping legacy OH_* exports/fallbacks in key places.
  • Updated docs/skill references and default paths from ~/.oh to ~/.clawteam.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
skills/clawteam/SKILL.md Clarifies oh as a legacy alias in skill trigger description.
skills/clawteam/references/cli-reference.md Updates default data dir examples to ~/.clawteam and /tmp/clawteam-data.
docs/skills/clawteam/SKILL.md Mirrors skill trigger description update (legacy alias note).
docs/skills/clawteam/references/cli-reference.md Mirrors CLI reference default path updates.
README.md Rewrites command examples and guidance from oh to clawteam.
README_CN.md Same as README.md, Chinese version.
README_KR.md Same as README.md, Korean version.
clawteam/spawn/wsh_backend.py Updates user-facing error message to reference clawteam spawn.
clawteam/spawn/tmux_backend.py Updates session naming docs and user-facing error message to clawteam spawn.
clawteam/spawn/prompt.py Updates module docstring to reference “clawteam CLI”.
clawteam/identity.py Introduces CLAWTEAM_PLAN_MODE_REQUIRED and keeps legacy OH_PLAN_MODE_REQUIRED export; generalizes env helper naming.
clawteam/harness/phases.py Updates approval instruction to clawteam harness approve.
clawteam/events/hooks.py Exports both CLAWTEAM_{FIELD} and legacy OH_{FIELD} env vars to shell hooks.
clawteam/cli/commands.py Updates CLI docstrings/help output and various user-facing command hints to clawteam.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@shadowinlife
Copy link
Copy Markdown
Author

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@shadowinlife
Copy link
Copy Markdown
Author

@tjb-tech 请review这个patch, 我在使用中遇到了doc文档和安装后行为不一样的问题. 当前patch主要是针对从open harness到独立命令工具进行修正

@ruttybob
Copy link
Copy Markdown

@tjb-tech Yes, please! :) This is a critical PR, I want to "pull" it, but I can't due to the fear that everything might break.

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