maxc-cli 0.3.3__tar.gz → 0.4.0__tar.gz

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: maxc-cli
-Version: 0.3.3
+Version: 0.4.0
 Summary: Agent-native MaxCompute CLI for external coding agents
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.9

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/setup.py

@@ -9,7 +9,7 @@ README = ROOT / "README.md"
 
 setup(
     name="maxc-cli",
-    version="0.3.3",
+    version="0.4.0",
     description="Agent-native MaxCompute CLI for external coding agents",
     long_description=README.read_text(encoding="utf-8"),
     long_description_content_type="text/markdown",

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/src/maxc_cli/__init__.py

@@ -2,4 +2,4 @@
 
 __all__ = ["__version__"]
 
-__version__ = "0.3.3"
+__version__ = "0.4.0"

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/src/maxc_cli/agent_platforms.py

@@ -74,7 +74,7 @@ INVOCATIONS: dict[str, dict[str, str]] = {
 
 
 def _claude_root() -> Path:
-    return Path.home() / ".claude" / "plugins" / "maxc-cli"
+    return Path.home() / ".claude" / "skills" / "maxc-cli"
 
 
 def _codex_root() -> Path:
@@ -83,12 +83,12 @@ def _codex_root() -> Path:
     return (
         Path(os.environ.get("CODEX_HOME", str(Path.home() / ".codex")))
         / "skills"
-        / "maxcompute-cli-guidance"
+        / "maxc-cli"
     )
 
 
 def _simple_root(dotdir: str) -> Path:
-    return Path.home() / dotdir / "skills" / "maxcompute-cli-guidance"
+    return Path.home() / dotdir / "skills" / "maxc-cli"
 
 
 def render_claude_plugin(install_dir: Path, cli: str, cli_module: str) -> None:
@@ -117,8 +117,8 @@ def _build_registry() -> tuple[Platform, ...]:
             name="claude-code",
             install_root=_claude_root(),
             skill_subpath=None,
-            extra_files=(ExtraFile(".claude-plugin/plugin.json", "render_claude_plugin"),),
-            next_step_hint="Run /reload-plugins in Claude Code to activate",
+            extra_files=(),
+            next_step_hint="Restart Claude Code or run /reload-plugins to activate",
         ),
         Platform(name="cursor",    install_root=_simple_root(".cursor"),
                  next_step_hint="Restart Cursor to activate"),

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/src/maxc_cli/app.py

@@ -3491,6 +3491,16 @@ class MaxCApp:
 
         return sorted(files_copied)
 
+    _LEGACY_SKILL_DIRS = ("maxcompute-cli-guidance", "use-maxc-cli")
+
+    def _cleanup_legacy_skill_dir(self, target: 'Path') -> None:
+        """Remove legacy skill directories that have been superseded by the new path."""
+        import shutil
+        for old_name in self._LEGACY_SKILL_DIRS:
+            old_dir = target.parent / old_name
+            if old_dir.is_dir() and (old_dir / ".maxc-skill-version").is_file():
+                shutil.rmtree(str(old_dir))
+
     def skill_install(
         self,
         *,
@@ -3500,14 +3510,16 @@ class MaxCApp:
         force: 'bool' = False,
     ) -> 'Envelope':
         from . import agent_platforms
-        if invocation not in agent_platforms.INVOCATIONS:
-            raise ValidationError(
-                f"Unsupported invocation: {invocation}. "
-                f"Supported: {', '.join(agent_platforms.INVOCATIONS)}"
-            )
+        # invocation is now the literal cli name (e.g. "maxc", "aliyun maxc")
+        # For backwards compat, also accept legacy key "aliyun-maxc"
+        if invocation in agent_platforms.INVOCATIONS:
+            invocation_map = agent_platforms.INVOCATIONS[invocation]
+        else:
+            invocation_map = {"cli": invocation, "cli_module": invocation}
         platform_spec, target = self._resolve_skill_target(platform, dir_override)
-        invocation_map = agent_platforms.INVOCATIONS[invocation]
         skills_src = self._locate_skills_source()
+        if dir_override is None:
+            self._cleanup_legacy_skill_dir(target)
         version_marker = f"{__version__}+{invocation}"
         marker_path = target / ".maxc-skill-version"
         if not force and marker_path.is_file() and marker_path.read_text().strip() == version_marker:

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/src/maxc_cli/cli.py

@@ -3,6 +3,7 @@ from __future__ import annotations
 
 import argparse
 import difflib
+import os
 import sys
 from pathlib import Path
 from typing import Any, Sequence, TextIO
@@ -75,6 +76,9 @@ def _epilog_for(command_path: str) -> str | None:
     sample = SAMPLES.get(command_path)
     if sample is None:
         return None
+    cli_name = os.environ.get("MAXC_CLI_NAME", "").strip() or "maxc"
+    if cli_name != "maxc":
+        sample = sample.replace("maxc ", f"{cli_name} ")
     return "Sample:\n  " + sample.replace("\n", "\n  ")
 
 
@@ -136,11 +140,8 @@ def _hoist_global_flags(argv: list[str]) -> list[str]:
 
 
 def _make_parser(parent_subparsers, name, command_path, **kw):
-    """Wrap add_parser so every parser gets aliyun-style formatting + Sample epilog."""
-    epilog = _epilog_for(command_path)
+    """Wrap add_parser so every parser gets aliyun-style formatting."""
     kw.setdefault("formatter_class", AliyunStyleFormatter)
-    if epilog is not None and "epilog" not in kw:
-        kw["epilog"] = epilog
     return parent_subparsers.add_parser(name, **kw)
 
 
@@ -161,13 +162,12 @@ def build_parser() -> argparse.ArgumentParser:
         cli_version = get_version("maxc-cli")
     except Exception:
         from maxc_cli import __version__ as cli_version
+    cli_name = os.environ.get("MAXC_CLI_NAME", "").strip() or "maxc"
     parser = argparse.ArgumentParser(
-        prog="maxc",
-        description="MaxCompute CLI — 给 Agent 调用的结构化工具层",
+        prog=cli_name,
         formatter_class=AliyunStyleFormatter,
-        epilog=_epilog_for("__top__"),
     )
-    parser.add_argument("-v", "--version", action="version", version=f"maxc {cli_version}")
+    parser.add_argument("-v", "--version", action="version", version=f"{cli_name} {cli_version}")
     parser.add_argument("--config", help="Explicit path to a config file")
     parser.add_argument(
         "--format",
@@ -189,7 +189,9 @@ def build_parser() -> argparse.ArgumentParser:
 
     # Top-level subparsers are NOT required: bare `maxc` is handled in run()
     # (prints help when auth is configured, redirects to `auth login` otherwise).
-    subparsers = parser.add_subparsers(dest="command_group")
+    # Explicit prog= prevents argparse from calling format_help() to derive it
+    # (which would bake our version header into child parser prog strings).
+    subparsers = parser.add_subparsers(dest="command_group", prog=cli_name)
 
     query_parser = _make_parser(
         subparsers,
@@ -199,13 +201,13 @@ def build_parser() -> argparse.ArgumentParser:
         description=(
             "Run a SQL query.\n"
             "Usage:\n"
-            "  maxc query \"SELECT 1\"             # default: run\n"
-            "  maxc query run \"SELECT 1\"         # explicit run\n"
-            "  maxc query cost \"SELECT 1\"        # estimate cost\n"
-            "  maxc query explain \"SELECT 1\"     # show plan\n"
+            f"  {cli_name} query \"SELECT 1\"             # default: run\n"
+            f"  {cli_name} query run \"SELECT 1\"         # explicit run\n"
+            f"  {cli_name} query cost \"SELECT 1\"        # estimate cost\n"
+            f"  {cli_name} query explain \"SELECT 1\"     # show plan\n"
             "\n"
             "Legacy usage (--mode is deprecated):\n"
-            "  maxc query \"SELECT 1\" --mode cost"
+            f"  {cli_name} query \"SELECT 1\" --mode cost"
         ),
         formatter_class=AliyunRawTextFormatter,
     )
@@ -1453,23 +1455,13 @@ def _handle_agent_skill(app: MaxCApp, args: argparse.Namespace, stdout: TextIO)
     _emit_envelope(envelope, args=args, stdout=stdout, default_format="json")
 
 
-def _detect_invocation() -> str:
-    """Best-effort detection of the invocation form.
+def _detect_cli_name() -> str:
+    """Read MAXC_CLI_NAME env var. Defaults to 'maxc'.
 
-    When the CLI is delegated to from `aliyun maxc ...` (via the Aliyun CLI
-    plugin bridge), `MAXC_INVOCATION` or `ALIYUN_CLI_NAME` is typically set;
-    fall back to scanning argv[0] for `aliyun`. Defaults to `maxc`.
+    The value is used directly as {{cli}} in SKILL templates.
+    Example: MAXC_CLI_NAME='aliyun maxc' renders all commands as `aliyun maxc ...`.
     """
-    import os
-    explicit = os.environ.get("MAXC_INVOCATION", "").strip().lower()
-    if explicit in ("maxc", "aliyun-maxc"):
-        return explicit
-    if os.environ.get("ALIYUN_CLI_NAME") or os.environ.get("ALIYUN_CLI_VERSION"):
-        return "aliyun-maxc"
-    argv0 = (sys.argv[0] if sys.argv else "").lower()
-    if "aliyun" in os.path.basename(argv0):
-        return "aliyun-maxc"
-    return "maxc"
+    return os.environ.get("MAXC_CLI_NAME", "").strip() or "maxc"
 
 
 def _resolve_dir_override(args: argparse.Namespace) -> Path | None:
@@ -1478,7 +1470,7 @@ def _resolve_dir_override(args: argparse.Namespace) -> Path | None:
 
 
 def _handle_agent_skill_install(app: MaxCApp, args: argparse.Namespace, stdout: TextIO) -> None:
-    invocation = args.invocation or _detect_invocation()
+    invocation = args.invocation or _detect_cli_name()
     envelope = app.skill_install(
         platform=args.platform,
         invocation=invocation,
@@ -1489,7 +1481,7 @@ def _handle_agent_skill_install(app: MaxCApp, args: argparse.Namespace, stdout:
 
 
 def _handle_agent_skill_update(app: MaxCApp, args: argparse.Namespace, stdout: TextIO) -> None:
-    invocation = args.invocation or _detect_invocation()
+    invocation = args.invocation or _detect_cli_name()
     # skill_update doesn't accept dir_override — `--all` iterates registry by
     # default install_root, single-platform mode falls through to skill_install
     # which would accept dir_override if we plumbed it in. Surface a validation

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/src/maxc_cli/help_format.py

@@ -38,13 +38,11 @@ def strip_ansi(text: str) -> str:
 
 
 class AliyunStyleFormatter(argparse.HelpFormatter):
-    """Aliyun-style help: ``Usage:`` / ``Flags:`` / ``Commands:`` headings.
+    """Aliyun CLI style: version header, Commands, Flags (long first, no-space comma)."""
 
-    Compact synopsis and Sample-epilog handling are added in Tasks 3-4.
-    """
+    _has_subparsers = False
 
     _SECTION_REMAP = {
-        "positional arguments": "Commands",
         "options": "Flags",              # Python 3.10+
         "optional arguments": "Flags",   # Python 3.9
     }
@@ -52,11 +50,18 @@ class AliyunStyleFormatter(argparse.HelpFormatter):
     def start_section(self, heading):
         if heading in self._SECTION_REMAP:
             heading = self._SECTION_REMAP[heading]
+        # Rename "positional arguments" dynamically: "Commands" when a
+        # subparsers action exists, "Arguments" otherwise.
+        if heading == "positional arguments":
+            heading = "Commands" if self._has_subparsers else "Arguments"
         super().start_section(heading)
 
     def add_usage(self, usage, actions, groups, prefix=None):
         if prefix is None:
             prefix = "Usage:\n  "
+        self._has_subparsers = any(
+            isinstance(a, argparse._SubParsersAction) for a in (actions or [])
+        )
         super().add_usage(usage, actions, groups, prefix)
 
     def _format_usage(self, usage, actions, groups, prefix):
@@ -83,49 +88,50 @@ class AliyunStyleFormatter(argparse.HelpFormatter):
         line = " ".join(parts)
         return f"{prefix}{line}\n\n"
 
-    def _fill_text(self, text, width, indent):
-        # Preserve sample/epilog formatting (newlines, leading indent) only for
-        # pre-formatted multi-line text. Single-line strings fall through to
-        # argparse's default ``textwrap.fill`` so long descriptions still wrap.
-        if "\n" in text:
-            return "".join(indent + line for line in text.splitlines(keepends=True))
-        return super()._fill_text(text, width, indent)
-
-    def format_help(self):
-        text = super().format_help()
-        if not text.endswith("\n"):
-            text += "\n"
-        # Skip the footer on the top-level parser (matches aliyun behavior).
-        # Subparsers always render as ``"<prog> <group>..."`` with spaces; the
-        # root prog is bare (no spaces), so this check is durable across
-        # entry-point renames.
-        if " " not in self._prog:
-            return text
-        # Skip the footer when argparse calls format_help() internally just to
-        # derive a subparser's prog (``add_subparsers`` adds only the usage
-        # section then calls format_help().strip()). The root section contains
-        # exactly one usage item in that case; for a real --help invocation it
-        # contains usage + the action groups (Flags, Commands, ...).
-        if len(self._root_section.items) <= 1:
-            return text
-        text += f"\nUse `{self._prog} --help` for more information.\n"
-        return text
+    def _format_action_invocation(self, action):
+        if not action.option_strings:
+            # Positional with choices: show dest name, not {choice1,choice2,...}
+            if action.choices and not isinstance(action, argparse._SubParsersAction):
+                return action.dest
+            return super()._format_action_invocation(action)
+        # Long option first, short after, comma with no space
+        opts = sorted(action.option_strings, key=lambda s: (not s.startswith('--'), s))
+        return ','.join(opts)
 
     def _format_action(self, action):
-        text = super()._format_action(action)
         if isinstance(action, argparse._SubParsersAction):
+            # Skip the "{choices}" summary line, render only the sub-commands
+            parts = []
+            for sub_action in action._get_subactions():
+                parts.append(super()._format_action(sub_action))
+            text = ''.join(parts)
+            # Colorize command names
             for choice in action.choices:
-                # Anchor to line start: argparse renders subcommand rows as
-                # ``"    <choice>  <help>"`` (leading whitespace varies with
-                # nesting). Plain ``str.replace`` would match the first
-                # occurrence anywhere in the rendered block (e.g. inside
-                # ``{login}`` brace lists or inside a help string).
                 text = re.sub(
                     rf"(?m)^(\s+){re.escape(choice)}(?=\s)",
                     rf"\1{cyan(choice)}",
                     text,
                     count=1,
                 )
+            return text
+        return super()._format_action(action)
+
+    def _fill_text(self, text, width, indent):
+        if "\n" in text:
+            return "".join(indent + line for line in text.splitlines(keepends=True))
+        return super()._fill_text(text, width, indent)
+
+    def format_help(self):
+        text = super().format_help()
+        if not text.endswith("\n"):
+            text += "\n"
+        # Inject version header at top for root parser's full help only.
+        # Guard on "Usage:" to skip injection when argparse uses format_help()
+        # for non-help purposes (e.g. --version action formatting).
+        if " " not in self._prog and "Usage:" in text:
+            from maxc_cli import __version__
+            header = f"MaxCompute CLI {__version__}\n\n"
+            text = header + text
         return text
 
 

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/src/maxc_cli/skills/SKILL.md

@@ -1,13 +1,6 @@
 ---
-name: maxcompute-cli-guidance
+name: maxc-cli
 description: Use when the task involves MaxCompute, ODPS, or {{cli}} — querying tables, viewing table schema, listing tables, searching metadata, executing SQL, checking partitions, sampling data, uploading or downloading CSV data, managing jobs, or generating MaxCompute SQL.
-description_zh: 当用户需要查询 MaxCompute/ODPS 中的表、查看表结构、列出项目中的表、搜索元数据、执行 SQL、查看分区、预览数据、上传或下载 CSV 数据、跟踪任务或生成 MaxCompute SQL 时使用。适用于提到 ODPS、MaxCompute、maxc、数据仓库表查询、text2sql 等场景。
-name_zh: MaxCompute数据查询
-category: database
-keywords: [MaxCompute, ODPS, maxc, 表, 查表, 查数据, SQL, 数据仓库, 元数据, 分区, odps sql, 阿里云, 上传, 下载, CSV, tunnel, text2sql, sql generation, SQL 生成]
-requires: MaxCompute account with AK/SK or environment variables
-entry_point: maxc
-min_cli_version: "0.2.4"
 ---
 
 # Use MaxC CLI

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/src/maxc_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: maxc-cli
-Version: 0.3.3
+Version: 0.4.0
 Summary: Agent-native MaxCompute CLI for external coding agents
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.9

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/tests/test_agent_platforms.py

@@ -28,10 +28,9 @@ def test_resolve_unknown_raises():
         ap.resolve("nonexistent")
 
 
-def test_claude_code_has_plugin_json_extra_file():
+def test_claude_code_has_no_extra_files():
     claude = ap.resolve("claude-code")
-    assert any(ef.relative_path == ".claude-plugin/plugin.json"
-               for ef in claude.extra_files)
+    assert claude.extra_files == ()
 
 
 def test_install_root_matches_legacy_paths():
@@ -40,12 +39,12 @@ def test_install_root_matches_legacy_paths():
     # and the new value must be justified in the PR description (it would break
     # already-installed users).
     expected = {
-        "claude-code": Path.home() / ".claude" / "plugins" / "maxc-cli",
-        "cursor":      Path.home() / ".cursor"    / "skills" / "maxcompute-cli-guidance",
-        "windsurf":    Path.home() / ".windsurf"  / "skills" / "maxcompute-cli-guidance",
-        "qwen":        Path.home() / ".qwen"      / "skills" / "maxcompute-cli-guidance",
-        "qoder":       Path.home() / ".qoder"     / "skills" / "maxcompute-cli-guidance",
-        "qoderwork":   Path.home() / ".qoderwork" / "skills" / "maxcompute-cli-guidance",
+        "claude-code": Path.home() / ".claude" / "skills" / "maxc-cli",
+        "cursor":      Path.home() / ".cursor"    / "skills" / "maxc-cli",
+        "windsurf":    Path.home() / ".windsurf"  / "skills" / "maxc-cli",
+        "qwen":        Path.home() / ".qwen"      / "skills" / "maxc-cli",
+        "qoder":       Path.home() / ".qoder"     / "skills" / "maxc-cli",
+        "qoderwork":   Path.home() / ".qoderwork" / "skills" / "maxc-cli",
     }
     for name, expected_path in expected.items():
         assert ap.resolve(name).install_root == expected_path, name
@@ -61,7 +60,7 @@ def test_codex_install_root_respects_CODEX_HOME(monkeypatch, tmp_path):
     importlib.reload(ap)
     try:
         assert ap.resolve("codex").install_root == (
-            tmp_path / "my-codex" / "skills" / "maxcompute-cli-guidance"
+            tmp_path / "my-codex" / "skills" / "maxc-cli"
         )
     finally:
         # Reload again after monkeypatch rollback so REGISTRY doesn't keep the

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/tests/test_agent_skill_commands.py

@@ -51,10 +51,10 @@ def test_skill_install_with_dir_override(app, tmp_path):
     assert (custom / "SKILL.md").is_file()
 
 
-def test_skill_install_writes_extra_files_for_claude_code(app, tmp_path):
+def test_skill_install_no_extra_files_for_claude_code(app, tmp_path):
     env = app.skill_install(platform="claude-code", invocation="maxc")
     install = Path(env.data["install_path"])
-    assert (install / ".claude-plugin" / "plugin.json").is_file()
+    assert not (install / ".claude-plugin").exists()
 
 
 def test_skill_install_force_overwrites(app, tmp_path):

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/tests/test_agent_skill_commands_context.py

@@ -222,25 +222,25 @@ class TestAgentInstallSkill:
         """Remove skill install dirs before each test to avoid stale version files."""
         import shutil
         for d in [
-            Path.home() / ".claude" / "plugins" / "maxc-cli",
-            Path.home() / ".cursor" / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".windsurf" / "skills" / "maxcompute-cli-guidance",
-            Path(os.environ.get("CODEX_HOME", str(Path.home() / ".codex"))) / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".qwen" / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".qoder" / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".qoderwork" / "skills" / "maxcompute-cli-guidance",
+            Path.home() / ".claude" / "skills" / "maxc-cli",
+            Path.home() / ".cursor" / "skills" / "maxc-cli",
+            Path.home() / ".windsurf" / "skills" / "maxc-cli",
+            Path(os.environ.get("CODEX_HOME", str(Path.home() / ".codex"))) / "skills" / "maxc-cli",
+            Path.home() / ".qwen" / "skills" / "maxc-cli",
+            Path.home() / ".qoder" / "skills" / "maxc-cli",
+            Path.home() / ".qoderwork" / "skills" / "maxc-cli",
         ]:
             if d.exists():
                 shutil.rmtree(str(d))
         yield
         for d in [
-            Path.home() / ".claude" / "plugins" / "maxc-cli",
-            Path.home() / ".cursor" / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".windsurf" / "skills" / "maxcompute-cli-guidance",
-            Path(os.environ.get("CODEX_HOME", str(Path.home() / ".codex"))) / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".qwen" / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".qoder" / "skills" / "maxcompute-cli-guidance",
-            Path.home() / ".qoderwork" / "skills" / "maxcompute-cli-guidance",
+            Path.home() / ".claude" / "skills" / "maxc-cli",
+            Path.home() / ".cursor" / "skills" / "maxc-cli",
+            Path.home() / ".windsurf" / "skills" / "maxc-cli",
+            Path(os.environ.get("CODEX_HOME", str(Path.home() / ".codex"))) / "skills" / "maxc-cli",
+            Path.home() / ".qwen" / "skills" / "maxc-cli",
+            Path.home() / ".qoder" / "skills" / "maxc-cli",
+            Path.home() / ".qoderwork" / "skills" / "maxc-cli",
         ]:
             if d.exists():
                 shutil.rmtree(str(d))
@@ -253,9 +253,9 @@ class TestAgentInstallSkill:
         assert data["platform"] == "claude-code"
         assert data["upgraded"] is True
         install_path = Path(data["install_path"])
-        assert (install_path / ".claude-plugin" / "plugin.json").is_file()
         assert (install_path / "SKILL.md").is_file()
         assert (install_path / "references").is_dir()
+        assert not (install_path / ".claude-plugin").exists()
 
     def test_install_skill_cursor(self, tmp_path):
         config = _make_config(tmp_path)
@@ -265,7 +265,7 @@ class TestAgentInstallSkill:
         assert data["platform"] == "cursor"
         assert data["upgraded"] is True
         install_path = Path(data["install_path"])
-        assert "maxcompute-cli-guidance" in str(install_path)
+        assert "maxc-cli" in str(install_path)
         assert (install_path / "SKILL.md").is_file()
         assert not (install_path / ".claude-plugin").exists()
 
@@ -351,7 +351,7 @@ class TestAgentInstallSkill:
         """If version marker differs, files should be overwritten."""
         config = _make_config(tmp_path)
         _run_cmd(config, ["agent", "skill", "install", "claude-code", "--json"])
-        install_path = Path.home() / ".claude" / "plugins" / "maxc-cli"
+        install_path = Path.home() / ".claude" / "skills" / "maxc-cli"
         (install_path / ".maxc-skill-version").write_text("0.0.0")
         _, payload, _ = _run_cmd(config, ["agent", "skill", "install", "claude-code", "--json"])
         assert payload["data"]["upgraded"] is True
@@ -360,7 +360,7 @@ class TestAgentInstallSkill:
     def test_install_skill_version_file_created(self, tmp_path):
         config = _make_config(tmp_path)
         _run_cmd(config, ["agent", "skill", "install", "claude-code", "--json"])
-        install_path = Path.home() / ".claude" / "plugins" / "maxc-cli"
+        install_path = Path.home() / ".claude" / "skills" / "maxc-cli"
         version_file = install_path / ".maxc-skill-version"
         assert version_file.is_file()
         from maxc_cli import __version__

{maxc_cli-0.3.3 → maxc_cli-0.4.0}/tests/test_help_format.py

@@ -35,7 +35,7 @@ def test_section_headings_remapped():
     assert "Flags:" in text
     assert "options:" not in text
     assert "optional arguments:" not in text
-    assert "Commands:" in text
+    assert "Arguments:" in text
     assert "positional arguments:" not in text
 
 
@@ -81,11 +81,12 @@ def test_epilog_rendered_as_sample_section():
     assert '  maxc query "SELECT 1"' in text
 
 
-def test_footer_hint_present_on_subparser(monkeypatch):
+def test_no_footer_on_subparser(monkeypatch):
+    """Aliyun style: no footer hint text."""
     monkeypatch.setattr("sys.stdout.isatty", lambda: False)
     p = argparse.ArgumentParser(prog="maxc auth", formatter_class=AliyunStyleFormatter)
     text = p.format_help()
-    assert "Use `maxc auth --help` for more information." in text
+    assert "for more information." not in text
 
 
 def test_subcommand_names_colored_when_tty(monkeypatch):
@@ -97,8 +98,8 @@ def test_subcommand_names_colored_when_tty(monkeypatch):
     assert "\033[36mlogin\033[0m" in text
 
 
-def test_footer_hint_absent_on_toplevel(monkeypatch):
-    """The root `maxc` parser must not show a redundant 'Use `maxc --help`' footer."""
+def test_no_footer_on_toplevel(monkeypatch):
+    """The root `maxc` parser must not show a footer hint."""
     monkeypatch.setattr("sys.stdout.isatty", lambda: False)
     p = argparse.ArgumentParser(prog="maxc", formatter_class=AliyunStyleFormatter)
     text = p.format_help()
@@ -144,10 +145,10 @@ def test_top_level_maxc_help_uses_aliyun_style(monkeypatch):
     monkeypatch.setattr("sys.stdout.isatty", lambda: False)
     from maxc_cli.cli import build_parser
     text = build_parser().format_help()
-    assert text.startswith("Usage:\n  maxc")
+    assert "MaxCompute CLI" in text
+    assert "Usage:\n  maxc" in text
     assert "Flags:" in text
     assert "Commands:" in text
-    assert "Sample:" in text
 
 
 def test_auth_login_help_uses_aliyun_style(monkeypatch):
@@ -162,5 +163,4 @@ def test_auth_login_help_uses_aliyun_style(monkeypatch):
     text = login.format_help()
     assert "Usage:" in text
     assert "Flags:" in text
-    assert "Sample:" in text
-    assert "Use `maxc auth login --help`" in text
+    assert "for more information." not in text