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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: maxc-cli
-Version: 0.3.2
+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.2 → maxc_cli-0.4.0}/setup.py

@@ -9,7 +9,7 @@ README = ROOT / "README.md"
 
 setup(
     name="maxc-cli",
-    version="0.3.2",
+    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.2 → maxc_cli-0.4.0}/src/maxc_cli/__init__.py

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

{maxc_cli-0.3.2 → 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.2 → maxc_cli-0.4.0}/src/maxc_cli/app.py

@@ -2161,7 +2161,13 @@ class MaxCApp:
         self.log("meta.list-schemas", envelope.status, envelope.metadata)
         return envelope
 
-    def session_set(self, project: 'str | None' = None, schema: 'str | None' = None) -> 'Envelope':
+    def session_set(
+        self,
+        project: 'str | None' = None,
+        schema: 'str | None' = None,
+        *,
+        target_config_path: 'Path | None' = None,
+    ) -> 'Envelope':
         """Set default project and/or schema by writing to ~/.maxc/config.yaml.
 
         Mirrors `gcloud config set project` / `kubectl config use-context`: the
@@ -2169,8 +2175,12 @@ class MaxCApp:
         (e.g., ./.maxc/config.yaml) shadows the value, a warning is emitted but
         the write still happens — the in-memory value is updated for the current
         invocation.
+
+        When ``target_config_path`` is given (i.e. the user passed ``--config``),
+        the write goes to that file instead, so a subsequent ``session show
+        --config <same>`` round-trips correctly.
         """
-        target_path = default_global_config_path()
+        target_path = target_config_path or default_global_config_path()
         config_payload = load_config_mapping(target_path) if target_path.exists() else {}
 
         changes: list[str] = []
@@ -2246,13 +2256,18 @@ class MaxCApp:
         self.log("session.set", envelope.status, {"changes": changes})
         return envelope
     
-    def session_unset(self) -> 'Envelope':
+    def session_unset(
+        self, *, target_config_path: 'Path | None' = None
+    ) -> 'Envelope':
         """Remove default_project / default_schema from ~/.maxc/config.yaml.
 
         Project-level config files in the working directory are NOT modified, since
         they may be checked into version control. Edit those by hand if needed.
+
+        When ``target_config_path`` is given (``--config``), unset operates on
+        that file instead of the global one.
         """
-        target_path = default_global_config_path()
+        target_path = target_config_path or default_global_config_path()
         cleared: list[str] = []
 
         if target_path.exists():
@@ -2284,9 +2299,16 @@ class MaxCApp:
         self.log("session.unset", envelope.status, {})
         return envelope
     
-    def session_show(self) -> 'Envelope':
-        """Show current session settings with source information."""
-        config_path = default_global_config_path()
+    def session_show(
+        self, *, target_config_path: 'Path | None' = None
+    ) -> 'Envelope':
+        """Show current session settings with source information.
+
+        When ``target_config_path`` is given (``--config``), the reported
+        ``config_path`` reflects that file, matching what ``session set
+        --config <same>`` would write to.
+        """
+        config_path = target_config_path or default_global_config_path()
 
         env_project = os.environ.get("MAXCOMPUTE_PROJECT") or os.environ.get("ODPS_PROJECT")
         has_explicit_auth_provider = bool(self.config.auth.provider)
@@ -3469,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,
         *,
@@ -3478,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.2 → maxc_cli-0.4.0}/src/maxc_cli/backend/query.py

@@ -356,13 +356,14 @@ class QueryMixin:
 
         actual_sql, hints, priority = _parse_sql_with_hints(sql, force=force)
         priority_kwargs = {"priority": priority} if priority is not None else {}
+        idem_kwargs = {"unique_identifier_id": idempotency_key} if idempotency_key is not None else {}
 
         try:
             instance = self.client.run_sql(
                 actual_sql,
                 project=project,
                 hints=hints,
-                unique_identifier_id=idempotency_key,
+                **idem_kwargs,
                 **priority_kwargs,
             )
         except Exception as exc:

{maxc_cli-0.3.2 → 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  ")
 
 
@@ -82,6 +86,10 @@ def _epilog_for(command_path: str) -> str | None:
 # (e.g., --project, --limit) are NOT hoisted — they belong to specific
 # subparsers. arity = number of subsequent argv tokens consumed as a value
 # when given as `--flag value`; `--flag=value` is always a single token.
+#
+# -h/--help is deliberately NOT hoisted: argparse auto-adds it to every
+# subparser, and hoisting would turn `maxc query -h` into top-level help
+# instead of the query subcommand's help.
 _GLOBAL_FLAG_ARITY: dict[str, int] = {
     "--format":  1,
     "-f":        1,
@@ -92,8 +100,6 @@ _GLOBAL_FLAG_ARITY: dict[str, int] = {
     "--debug":   0,
     "-v":        0,
     "--version": 0,
-    "--help":    0,
-    "-h":        0,
 }
 
 
@@ -134,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)
 
 
@@ -159,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",
@@ -187,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,
@@ -197,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,
     )
@@ -1297,23 +1301,27 @@ def _handle_session_set(app: MaxCApp, args: argparse.Namespace, stdout: TextIO)
     """Set current project and/or schema for the session."""
     project = args.project
     schema = args.schema
-    
+
     if not project and not schema:
         raise ValidationError("At least one of `--project` or `--schema` must be specified.")
-    
-    envelope = app.session_set(project=project, schema=schema)
+
+    envelope = app.session_set(
+        project=project,
+        schema=schema,
+        target_config_path=args.requested_config_path,
+    )
     _emit_envelope(envelope, args=args, stdout=stdout, default_format="json")
 
 
 def _handle_session_show(app: MaxCApp, args: argparse.Namespace, stdout: TextIO) -> None:
     """Show current session settings."""
-    envelope = app.session_show()
+    envelope = app.session_show(target_config_path=args.requested_config_path)
     _emit_envelope(envelope, args=args, stdout=stdout, default_format="json")
 
 
 def _handle_session_unset(app: MaxCApp, args: argparse.Namespace, stdout: TextIO) -> None:
     """Clear session override."""
-    envelope = app.session_unset()
+    envelope = app.session_unset(target_config_path=args.requested_config_path)
     _emit_envelope(envelope, args=args, stdout=stdout, default_format="json")
 
 
@@ -1447,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:
@@ -1472,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,
@@ -1483,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.2 → 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.2 → 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.2 → 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.2
+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.2 → maxc_cli-0.4.0}/src/maxc_cli.egg-info/SOURCES.txt

@@ -76,6 +76,7 @@ tests/test_exit_codes.py
 tests/test_external_auth.py
 tests/test_flag_hoist.py
 tests/test_help_format.py
+tests/test_help_version_e2e.py
 tests/test_helpers.py
 tests/test_helpers_csv.py
 tests/test_integration.py

{maxc_cli-0.3.2 → 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.2 → 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):