Skip to content

Release 0.8#4

Open
dawenrenhub wants to merge 3 commits into
mainfrom
0.8
Open

Release 0.8#4
dawenrenhub wants to merge 3 commits into
mainfrom
0.8

Conversation

@dawenrenhub
Copy link
Copy Markdown
Owner

Merge v0.7 changes into 0.8 and prepare for main.

主要更改:
- install.sh: 添加更多交互式确认提示(uninstall/删除重下载/git pull)
- install.sh: Step 3 项目配置可选跳过
- ralph_reset.sh: 增加删除 root my_requirements.md、logs、src、tests、test-debug、.git 的选项
- planning.sh: 注释掉 ralph-enable 块
- 删除空的 init.sh
- 修复 ralph-claude-code/install.sh 使用完整版 setup.sh
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 87bf891b7a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread planning.sh
Comment on lines +189 to 190
if ! (cd "$PROJECT_DIR" && claude "$(cat <<'EOF'
项目路径:$PROJECT_DIR
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Allow PROJECT_DIR to expand in brainstorm prompt

The heredoc for the Step 1 prompt is now single‑quoted, so $PROJECT_DIR is passed literally to Claude rather than the actual path. That removes the only explicit project location from the prompt, which can cause the CLI to misinterpret which directory it should inspect when asked to “判断这是不是一个新项目.” If the path is needed for correct context, this regression will lead to incorrect project detection.

Useful? React with 👍 / 👎.

Comment thread planning.sh
Comment on lines +264 to 266
if ! (cd "$PROJECT_DIR" && claude "$(cat <<'EOF'
请使用 superpowers 的 /write-plan 功能调取最新生成的design.md生成待办清单。
设计文档路径:$LATEST_DESIGN_FILE
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Expand LATEST_DESIGN_FILE in /write-plan prompt

The Step 2 prompt is also using a single‑quoted heredoc, so $LATEST_DESIGN_FILE is not expanded. This means Claude receives a literal placeholder instead of the actual design.md path, which defeats the instruction to read the latest design file and can cause /write-plan to use the wrong (or no) input.

Useful? React with 👍 / 👎.

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.

2 participants