fix(import): align wrappers with current CLI flags#1044
Conversation
|
CI attribution for run #1669: translation, change detection, the controller/base build, and all six downstream image builds passed. I audited the raw logs for all 10 failed integration jobs. Each job has an empty The focused local contract regression remains green ( |
|
The wrapper and CLI contract alignment is useful, and the focused Bash and PowerShell regression passes locally. The remaining blocker is the current required CI result: ten integration jobs are still recorded as failed. Please update or rebase the branch onto the latest main branch and rerun the required checks. If the failures still occur before tests start because of CI credentials, link the new run and its evidence; otherwise address the failing paths before final review. |
Summary
hiclawCLI does not implementmcpServersconfiguration through YAML and align English/Chinese examplesRoot cause
The wrappers lagged behind the current Go CLI contract after its apply/import behavior changed. They still accepted
--mcp-servers,--dry-run, and--prune, so users only saw a failure after the arguments reached the container CLI.Validation
PWSH_BIN=/tmp/powershell-7.6.3/pwsh install/tests/test-import-wrapper-contract.sh(14 passed, 0 failed; baseline was2 passed, 8 failed)6 passed, 0 failed)bash -n install/hiclaw-import.sh install/hiclaw-apply.sh install/tests/test-import-wrapper-contract.shshellcheck -S warning install/hiclaw-import.sh install/hiclaw-apply.sh install/tests/test-import-wrapper-contract.shgo run ./cmd/hiclaw apply worker --helpconfirms the removed flags are absentgit diff --check