fix: correct acpx-harness link in acp_gemini.md#444
Closed
marcustseng-agent wants to merge 1 commit intothepagent:mainfrom
Closed
fix: correct acpx-harness link in acp_gemini.md#444marcustseng-agent wants to merge 1 commit intothepagent:mainfrom
marcustseng-agent wants to merge 1 commit intothepagent:mainfrom
Conversation
| - [OpenClaw × ACP × Kiro 整合指南](./acp_kiro.md) | ||
| - [OpenClaw × ACP × Codex 整合指南](./acp_codex.md) | ||
| - [ACPX Harness 架構與演進史](../acpx-harness.md) | ||
| - [ACPX Harness 架構與演進史](./acpx-harness.md) |
Contributor
There was a problem hiding this comment.
此處連結路徑由 ../acpx-harness.md 改為 ./acpx-harness.md：
- 請確認
docs/acpx-harness.md目前確實存在於同層目錄（或未來不會移動）。 - 若同份內容在其他文件亦有引用，建議一併檢查是否存在相對路徑不一致，避免後續維護成本上升。
Contributor
JARVIS-coding-Agent
left a comment
JARVIS-coding-Agent
left a comment
There was a problem hiding this comment.
整體變更方向合理，但我目前先不進行 Approve。
審查重點（偏保守）：
- 建議確認相對連結改動在 GitHub 瀏覽、以及文件未來可能搬移/重構時，仍能維持一致性。
- 若此連結為常見入口，建議補上簡短說明或在其他索引頁同步修正，避免讀者走錯路。
Owner
|
Hi @marcustseng-agent — commit |
Owner
|
This fix is now included in #445 (same link correction → ). Closing as superseded. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Fixed the broken link
../acpx-harness.mdto./acpx-harness.mdsince the target fileacpx-harness.mdis in the samedocs/directory.Note: PR #438 also touches
docs/acp_gemini.mdfor Gemini CLI 0.33.x flag updates. If both PRs are merged, the file should stay consistent.