skillnav 0.3.1__tar.gz → 0.4.2__tar.gz

{skillnav-0.3.1 → skillnav-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: skillnav
-Version: 0.3.1
+Version: 0.4.2
 Summary: CLI client for the Skill management platform (SkillNavigator)
 License: Apache-2.0
 Keywords: skill,cli,agent,registry
@@ -12,6 +12,8 @@ Description-Content-Type: text/markdown
 Requires-Dist: typer>=0.12
 Provides-Extra: dev
 Requires-Dist: pytest>=8.0; extra == "dev"
+Requires-Dist: ruff>=0.9; extra == "dev"
+Requires-Dist: mypy>=1.10; extra == "dev"
 
 # skillnav
 
@@ -36,7 +38,6 @@ skillnav config test
 skillnav login --api-key sk_...
 skillnav search demo
 skillnav info demo-skill
-skillnav review examples/demo-skill
 skillnav publish examples/demo-skill --dry-run
 skillnav download demo-skill -o /tmp/demo.zip
 skillnav update              # upgrade when PyPI has a newer release

{skillnav-0.3.1 → skillnav-0.4.2}/README.md

@@ -21,7 +21,6 @@ skillnav config test
 skillnav login --api-key sk_...
 skillnav search demo
 skillnav info demo-skill
-skillnav review examples/demo-skill
 skillnav publish examples/demo-skill --dry-run
 skillnav download demo-skill -o /tmp/demo.zip
 skillnav update              # upgrade when PyPI has a newer release

{skillnav-0.3.1 → skillnav-0.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "skillnav"
-version = "0.3.1"
+version = "0.4.2"
 description = "CLI client for the Skill management platform (SkillNavigator)"
 readme = "README.md"
 requires-python = ">=3.9"
@@ -20,10 +20,30 @@ dependencies = [
 ]
 
 [project.optional-dependencies]
-dev = ["pytest>=8.0"]
+dev = [
+    "pytest>=8.0",
+    "ruff>=0.9",
+    "mypy>=1.10",
+]
 
 [project.scripts]
 skillnav = "skillnav.cli:main"
 
+[tool.ruff]
+line-length = 100
+target-version = "py39"
+src = ["src"]
+
+[tool.ruff.lint]
+# 默认集（F 全部 + E4/E7/E9）；后续需要可扩展（I/UP/B 等）。
+select = ["E4", "E7", "E9", "F"]
+
+[tool.mypy]
+python_version = "3.9"
+check_untyped_defs = false
+warn_redundant_casts = true
+no_implicit_optional = true
+ignore_missing_imports = true
+
 [tool.setuptools.packages.find]
 where = ["src"]

{skillnav-0.3.1 → skillnav-0.4.2}/src/skillnav/__init__.py

@@ -1,3 +1,3 @@
 """skillnav — CLI client for the Skill management platform (SkillNavigator)."""
 
-__version__ = "0.3.1"
+__version__ = "0.4.2"

{skillnav-0.3.1 → skillnav-0.4.2}/src/skillnav/__main__.py

@@ -1,4 +1,4 @@
 from skillnav.cli import main
 
 if __name__ == "__main__":
-    raise SystemExit(main())
+    main()

{skillnav-0.3.1 → skillnav-0.4.2}/src/skillnav/api.py

@@ -37,6 +37,8 @@ def request_bytes(
     except urllib.error.URLError as exc:
         reason = getattr(exc, "reason", exc)
         raise NetworkError.from_hint(network_unreachable(str(reason), registry=url)) from exc
+    except OSError as exc:
+        raise NetworkError.from_hint(network_unreachable(str(exc), registry=url)) from exc
 
 
 def request_json(

{skillnav-0.3.1 → skillnav-0.4.2}/src/skillnav/cli.py

@@ -2,7 +2,6 @@
 
 from __future__ import annotations
 
-import sys
 from pathlib import Path
 from typing import Annotated, Any, Optional
 
@@ -16,7 +15,7 @@ from skillnav.api import (
     request_bytes,
     request_json,
 )
-from skillnav.config import get_profile, load_config, save_config
+from skillnav.config import get_profile, load_config, resolve_profile_api_key, save_config
 from skillnav.contributors import resolve_contributor_id
 from skillnav.context import CliContext
 from skillnav.error_hints import (
@@ -40,17 +39,16 @@ from skillnav.errors import (
 from skillnav.output import (
     emit_error,
     emit_json,
+    filter_skill_body_version,
     print_leaderboard,
     print_report_version,
-    print_review,
-    print_review_result,
+    print_inspection_result,
     print_search_results,
     print_skill_info,
     print_skill_status,
-    print_virustotal_summary,
     unwrap_resource_id,
 )
-from skillnav.packages import extract_zip_to_directory, package_to_base64, resolve_user_path
+from skillnav.packages import extract_zip_to_directory, package_to_base64
 from skillnav.publish_metadata import build_publish_metadata, read_frontmatter_hints, resolve_package_path
 from skillnav.urls import join_registry_url, slug_path
 
@@ -133,7 +131,7 @@ def _version_callback(value: bool) -> None:
 
 
 @app.callback()
-def main(
+def cli_root(
     ctx: typer.Context,
     registry: Annotated[
         Optional[str], typer.Option("--registry", help="API base URL (overrides profile)")
@@ -400,8 +398,14 @@ def info_cmd(slug: Annotated[str, typer.Argument(help="Skill slug")]) -> None:
 
 
 @app.command("status")
-def status_cmd(slug: Annotated[str, typer.Argument(help="Skill slug")]) -> None:
-    """Show publish status and version review summaries."""
+def status_cmd(
+    slug: Annotated[str, typer.Argument(help="Skill slug")],
+    version: Annotated[
+        Optional[str],
+        typer.Option("--version", help="Version to show (default: latest)"),
+    ] = None,
+) -> None:
+    """Show publish and inspection status for one skill version."""
     try:
         cli = _ctx()
         status, body = request_json(
@@ -410,10 +414,14 @@ def status_cmd(slug: Annotated[str, typer.Argument(help="Skill slug")]) -> None:
             token=cli.token,
         )
         raise_for_api_status(status, body)
+        target_version = version or body.get("latestVersion")
+        if not target_version:
+            raise SkillnavError(f"No versions found for skill '{slug}'")
+        body = filter_skill_body_version(body, str(target_version))
         if cli.json_output:
             emit_json(body)
         else:
-            print_skill_status(body)
+            print_skill_status(body, version=str(target_version))
     except Exception as exc:  # noqa: BLE001
         _handle_error(exc)
 
@@ -443,7 +451,28 @@ def report_cmd(
         _handle_error(exc)
 
 
-# --- publish / review ---
+# --- publish ---
+
+
+def _print_publish_response(status: int, payload: dict[str, Any], *, waited: bool) -> None:
+    if status == 202 or payload.get("inspectionStatus") == "inspecting":
+        typer.echo(
+            f"Uploaded {payload.get('name')} ({payload.get('slug')})@{payload.get('version')}"
+        )
+        if waited:
+            typer.echo("Inspection started (inspectionStatus: inspecting).")
+        else:
+            typer.echo("Inspection started in the background (inspectionStatus: inspecting).")
+        typer.echo(f"Check progress: skillnav status {payload.get('slug')}")
+        return
+
+    typer.echo(
+        f"Published {payload.get('name')} ({payload.get('slug')})@{payload.get('version')}"
+    )
+    typer.echo(f"Status: {payload.get('status')}")
+    typer.echo(f"Hash: {payload.get('contentHash')}")
+    if payload.get("inspection") or payload.get("review") or payload.get("evaluation"):
+        print_inspection_result(payload)
 
 
 @app.command("publish")
@@ -468,8 +497,15 @@ def publish_cmd(
     ] = None,
     changelog: Annotated[Optional[str], typer.Option("--changelog", help="Changelog text")] = None,
     dry_run: Annotated[bool, typer.Option("--dry-run", help="Preview without publishing")] = False,
+    wait: Annotated[
+        bool,
+        typer.Option(
+            "--wait",
+            help="Wait for the full inspection pipeline to finish before returning (default: upload only, inspection in background)",
+        ),
+    ] = False,
 ) -> None:
-    """Publish a skill package to the registry."""
+    """Upload a skill package to the registry; inspection runs in the background unless --wait."""
     try:
         cli = _ctx()
         token = cli.require_token()
@@ -491,6 +527,8 @@ def publish_cmd(
         body: dict[str, Any] = {"archiveBase64": archive_base64, "metadata": metadata}
         if changelog:
             body["changelog"] = changelog
+        if not dry_run:
+            body["async"] = not wait
         path = "/skills/publish/preview" if dry_run else "/skills/publish"
         status, payload = request_json(
             method="POST",
@@ -515,41 +553,39 @@ def publish_cmd(
             if payload.get("entryPath"):
                 typer.echo(f"Entry: {payload['entryPath']}")
             return
-        typer.echo(
-            f"Published {payload.get('name')} ({payload.get('slug')})@{payload.get('version')}"
-        )
-        typer.echo(f"Status: {payload.get('status')}")
-        typer.echo(f"Hash: {payload.get('contentHash')}")
-        if payload.get("review") or payload.get("evaluation"):
-            print_review_result(payload)
+        _print_publish_response(status, payload, waited=wait)
     except Exception as exc:  # noqa: BLE001
         _handle_error(exc)
 
 
-@app.command("review")
-def review_cmd(
-    package: Annotated[str, typer.Argument(help="Skill directory or .zip")],
-    version: Annotated[Optional[str], typer.Option("--version", help="Version label")] = None,
+@app.command("retry-publish")
+def retry_publish_cmd(
+    slug: Annotated[str, typer.Argument(help="Skill slug")],
+    wait: Annotated[
+        bool,
+        typer.Option(
+            "--wait",
+            help="Wait for the full inspection pipeline to finish before returning",
+        ),
+    ] = False,
 ) -> None:
-    """Run remote review without publishing."""
+    """Re-run inspection on the stored package after a failed or incomplete inspection."""
     try:
         cli = _ctx()
-        archive_base64 = package_to_base64(resolve_user_path(package))
-        body: dict[str, Any] = {"archiveBase64": archive_base64}
-        if version:
-            body["version"] = version
+        token = cli.require_token()
         status, payload = request_json(
-            "POST",
-            join_registry_url(cli.registry, "/reviews/run"),
-            body=body,
-            token=cli.require_token(),
+            method="POST",
+            url=join_registry_url(cli.registry, f"/skills/{slug_path(slug)}/retry-publish"),
+            body={"async": not wait},
+            token=token,
         )
-        raise_for_api_status(status, payload)
+        if status >= 400:
+            hint = enrich_api_error(api_error_message(payload), status=status, body=payload)
+            raise SkillnavError(hint.summary, hint=hint)
         if cli.json_output:
             emit_json(payload)
             return
-        if payload.get("review") or payload.get("evaluation"):
-            print_review_result(payload)
+        _print_publish_response(status, payload, waited=wait)
     except Exception as exc:  # noqa: BLE001
         _handle_error(exc)
 
@@ -786,6 +822,9 @@ def remove_contributor_cmd(
             raise_for_api_status(status, skill_body)
             contributor_id = resolve_contributor_id(skill_body, username)
 
+        if contributor_id is None:
+            raise UsageError.from_hint(enrich_usage_error("no contributor id available to remove"))
+
         status, payload = request_json(
             "DELETE",
             join_registry_url(

{skillnav-0.3.1 → skillnav-0.4.2}/src/skillnav/error_hints.py

@@ -25,7 +25,7 @@ def not_logged_in(*, profile: str | None = None) -> ErrorHint:
     return ErrorHint(
         summary=f"Not logged in (no API key on profile{profile_label})",
         detail=(
-            "Write commands (publish, review, download, install, rate, issue, contributors) "
+            "Write commands (publish, download, install, rate, issue, contributors) "
             "require a platform API key. None is configured for the active profile."
         ),
         next_steps=_steps(
@@ -55,8 +55,9 @@ def network_unreachable(reason: str, *, registry: str | None = None) -> ErrorHin
     return ErrorHint(
         summary=f"Cannot reach the Skill platform API ({reason})",
         detail=(
-            "The CLI could not open an HTTP connection to the registry."
-            f"{registry_line} The API may be stopped, the URL wrong, or a firewall blocking the port."
+            "The CLI could not complete an HTTP request to the registry."
+            f"{registry_line} The API may be stopped, crashed mid-response, the URL wrong,"
+            " or a firewall blocking the port."
         ),
         next_steps=_steps(
             "Start the API locally: npm run dev:api (or npm run dev for API + Web).",
@@ -115,13 +116,25 @@ def enrich_api_error(raw: str, *, status: int, body: Any = None) -> ErrorHint:
                 "Or publish under a new slug (first publish creates a new skill).",
             ),
         ),
-        "review_pipeline_incomplete": ErrorHint(
-            summary="Review pipeline incomplete (503)",
+        "inspection_pipeline_incomplete": ErrorHint(
+            summary="Inspection pipeline incomplete (503)",
             detail="SkillSpector, VirusTotal, or HaluCatch did not finish. The server marks this as retryable.",
             next_steps=_steps(
-                "Wait a few seconds and retry the same command.",
-                "Run a local dry-run first: skillnav publish <package> --dry-run",
-                "If it persists, check API logs and review-engine dependencies (Python, SkillSpector, HaluCatch).",
+                "The package may already be saved — retry inspection: skillnav retry-publish <slug>",
+                "Wait a few seconds and retry, or check progress: skillnav status <slug>",
+                "If it persists, check API logs and inspection-engine dependencies (Python, SkillSpector, HaluCatch).",
+            ),
+        ),
+        "pending_publish_use_retry": ErrorHint(
+            summary="Package already uploaded — use retry-publish",
+            detail=(
+                "This slug@version has a stored package from a previous upload. "
+                "Re-uploading is blocked; run inspection again on the saved package."
+            ),
+            next_steps=_steps(
+                "Retry inspection: skillnav retry-publish <slug>",
+                "Inspect state: skillnav status <slug>",
+                "To upload changed files, bump the version in SKILL.md or pass --version.",
             ),
         ),
         "publish_rate_limited": ErrorHint(
@@ -220,7 +233,7 @@ def enrich_api_error(raw: str, *, status: int, body: Any = None) -> ErrorHint:
             summary=f"Version {got} is not greater than latest {latest}",
             detail=f"New releases for '{slug}' must use semver strictly greater than {latest}.",
             next_steps=_steps(
-                f"Publish with a higher version: skillnav publish <package> --version <next-semver>",
+                "Publish with a higher version: skillnav publish <package> --version <next-semver>",
                 f"Inspect versions: skillnav status {slug}",
             ),
         )
@@ -272,7 +285,7 @@ def enrich_api_error(raw: str, *, status: int, body: Any = None) -> ErrorHint:
         next_steps=_steps(
             "Read the message above and fix the request input.",
             "Use --json for the raw response body when debugging.",
-            "See Web docs: /docs/cli-developer-guide",
+            "See Web docs: /docs/cli-guide",
         ),
     )
 

{skillnav-0.3.1 → skillnav-0.4.2}/src/skillnav/errors.py

@@ -40,10 +40,6 @@ class SkillnavError(Exception):
     def from_hint(cls, hint: ErrorHint) -> SkillnavError:
         return cls(hint.summary, hint=hint)
 
-    @classmethod
-    def from_hint(cls, hint: ErrorHint) -> SkillnavError:
-        return cls(hint.summary, hint=hint)
-
 
 class AuthError(SkillnavError):
     exit_code = EXIT_AUTH