Skip to content

fix(migrate): print runnable ZIP import command - #1048

Merged
shiyiyue1102 merged 2 commits into
agentscope-ai:mainfrom
RerankerGuo:fix/migration-import-command
Jul 17, 2026
Merged

fix(migrate): print runnable ZIP import command#1048
shiyiyue1102 merged 2 commits into
agentscope-ai:mainfrom
RerankerGuo:fix/migration-import-command

Conversation

@RerankerGuo

@RerankerGuo RerankerGuo commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • include the required worker resource type and Worker name in generated ZIP import instructions
  • reject normalized Worker names that the hiclaw CLI cannot accept
  • add an end-to-end regression test for the import command and invalid names

Problem

generate-zip.sh currently tells users to run bash hiclaw-import.sh --zip .... The current import wrapper treats its first argument as a resource type, so that command fails with Unknown resource type: --zip.

The generator can also normalize inputs such as !!! to an empty name or preserve a leading hyphen, then build a package and print an import command that hiclaw apply worker rejects. The generated instructions should be directly runnable and follow the CLI's Worker-name contract.

Testing

  • baseline generator completes the ZIP but fails the new command assertion
  • bash migrate/skill/tests/test-generate-zip.sh
  • shellcheck migrate/skill/tests/test-generate-zip.sh
  • bash -n migrate/skill/scripts/generate-zip.sh migrate/skill/tests/test-generate-zip.sh
  • git diff --check

Full ShellCheck on the existing generator reports only its pre-existing SC2018/SC2019 tr warnings. This does not modify the import wrappers changed by #1044; it fixes their standalone migration-package caller.

@shiyiyue1102 shiyiyue1102 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. The generated migration instruction now matches the import wrapper contract by including the worker resource type and name, and the normalized name validation matches the CLI worker-name rules. The end-to-end regression passes and the required check is green.

@shiyiyue1102
shiyiyue1102 merged commit ced0f18 into agentscope-ai:main Jul 17, 2026
1 check passed
nattiini45 added a commit to nattiini45/AgentTeams that referenced this pull request Jul 23, 2026
* fix: align AgentTeams runtime naming defaults (agentscope-ai#1041)

* fix: prepare beta release workflow (agentscope-ai#1054)

* fix: harden beta release gates (agentscope-ai#1056)

* fix(cli): reject unsafe plugin archive links (agentscope-ai#1043)

* fix(scripts): redact complete Matrix events (agentscope-ai#1047)

* fix(migrate): analyze current cron job format (agentscope-ai#1049)

* fix(migrate): print runnable ZIP import command (agentscope-ai#1048)

* fix(replay): capture immediate manager replies (agentscope-ai#1045)

* fix(hack): persist containerized skopeo auth (agentscope-ai#1050)

* chore: archive changelog for v1.2.0-beta.1 (agentscope-ai#1058)

Co-authored-by: shiyiyue1102 <[email protected]>

* fix(copaw): route Team Leader assignments to Team Room (agentscope-ai#1060)

* fix(install): user can specify which docker.sock to mount when run install script (agentscope-ai#553)

* docs: clarify Element homeserver port (agentscope-ai#978)

Co-authored-by: Ziyang Guo <[email protected]>

* docs: clarify Higress AI route matching (agentscope-ai#980)

Co-authored-by: Ziyang Guo <[email protected]>

* docs: clarify OpenAI-compatible provider setup (agentscope-ai#1013)

* refactor: complete AgentTeams runtime rename (agentscope-ai#1063)

* refactor: complete the AgentTeams hard-cut rename (agentscope-ai#1065)

* docs: add v1.2.0-beta.1 release news (agentscope-ai#1066)

* feat(sync): restore fork-only product trees on agentteams paths

Replay dashboard Helm templates under helm/agentteams, dashboard values and
helpers, qwenpaw worker image defaults, and path-adapted remediation-gates
(agentteams-controller / helm/agentteams). Root dashboard/, shared/python/,
openwiki/, and Gastown skills already survived the merge baseline.

Co-authored-by: Cursor <[email protected]>

* feat(controller): port fork Project CRD, health, APIs, and status CLI

Replay fork-only agentteams-controller features onto the upstream rename
baseline: Project CRD/reconciler/Helm, health monitor, message and
manager-tasks APIs, worker health probes, managerstate CLI, enhanced agt
status overview, SoloOperator/QwenPaw/docker resource wiring, and related
tests.

Co-authored-by: Cursor <[email protected]>

* feat(runtimes): restore quiet-rooms and Higress extra-provider prefix strip

Keep upstream CoPaw Team Leader DM-to-Team-Room routing while replaying
fork AGENTTEAMS_QUIET_ROOMS bridge/channel wiring. Restore OPT-IN
AGENTTEAMS_EXTRA_LLM_PROVIDERS setup with modelMapping prefix strip in
setup-higress.sh (route names agentteams-*-route).

Co-authored-by: Cursor <[email protected]>

* docs(sync): truth-up migration doc, merge changelog, agt naming sweep

Rewrite upstream-integration-migration.md for rename acceptance + fork
overlay DoD (drop dual-CRD/HICLAW_* claims). Merge changelog entries from
both sides. Point pre-commit and openwiki paths at agentteams-controller /
helm/agentteams. Align agent-facing CLI examples to agt.

Co-authored-by: Cursor <[email protected]>

* fix(qwenpaw): resolve Windows file:// agent package refs

urlparse puts file://C:\path entirely in netloc with an empty path; Path('') became cwd and silently copied the repo instead of the package archive.

Co-authored-by: Cursor <[email protected]>

* fix(ci): purge brand leftovers and restore fork runtime bridge/sync

Clear remaining retired-brand strings/paths so helm-lint rename gate passes, restore CoPaw fork bridge/worker APIs and Hermes thin sync semantics, and harden installer prompt helpers with printf -v.

Co-authored-by: Cursor <[email protected]>

* fix(copaw): restore Matrix channel APIs and thin sync wrapper

Bring back fork Matrix channel behavior needed by worker tests, switch sync back to the agentteams_sync wrapper, and share filesync helpers via _toolhelpers.

Co-authored-by: Cursor <[email protected]>

* fix(sync): restore agentteams- bucket prefix stripping for team storage

The rename pass incorrectly shortened the hiclaw- bucket prefix check to agt-, breaking team path derivation for agentteams-* buckets.

Co-authored-by: Cursor <[email protected]>

* fix(security): remediate Kilo review findings on sync/upstream-main

Harden appservice token compare, proxy body limits, lifecycle error logging,
Helm image/RBAC/CRD alignment, OpenHuman TOML escaping, and storage alias safety.

Co-authored-by: Cursor <[email protected]>

* fix(helm): avoid retired brand in storage.bucket comment

Rename gate rejects hiclaw tokens even in comments; keep migration note brand-free.

Co-authored-by: Cursor <[email protected]>

---------

Co-authored-by: shiyiyue1102 <[email protected]>
Co-authored-by: Ziyang Guo <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: shiyiyue1102 <[email protected]>
Co-authored-by: Daniel Qian <[email protected]>
Co-authored-by: Ziyang Guo <[email protected]>
Co-authored-by: Cursor <[email protected]>
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