oasr 0.4.0__tar.gz → 0.4.2__tar.gz

{oasr-0.4.0 → oasr-0.4.2}/CHANGELOG.md

@@ -4,9 +4,22 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased]
+## [0.4.2] - 2026-02-01
 
-## [0.4.0] - TBD
+### Added
+- **registry prune subcommand**: Added `oasr registry prune` to align with registry command taxonomy
+
+### Changed
+- **clean command**: Deprecated `oasr clean` in favor of `oasr registry prune` (will be removed in v0.6.0)
+- **documentation**: Updated REGISTRY.md with prune subcommand documentation and usage examples
+
+## [0.4.1] - 2026-02-01
+
+### Fixed
+- **exec command**: Fixed `CompletedProcess` attribute error where code incorrectly referenced `.success`, `.output`, and `.error` attributes that don't exist. Now correctly uses `.returncode`
+- **clone documentation**: Removed non-existent `-r, --recursive` flag from CLONE.md documentation
+
+## [0.4.0] - 2026-01-31
 
 ### Added
 - **🚀 `oasr exec` command** — Execute skills as CLI tools from anywhere

{oasr-0.4.0 → oasr-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: oasr
-Version: 0.4.0
+Version: 0.4.2
 Summary: CLI for managing agent skills across IDE integrations
 Project-URL: Homepage, https://github.com/jgodau/asr
 Project-URL: Repository, https://github.com/jgodau/asr

{oasr-0.4.0 → oasr-0.4.2}/docs/commands/.INDEX.md

@@ -3,7 +3,7 @@
 ## Core Commands (v0.4.0)
 
 - [oasr](ROOT.md) - Global flags and options
-- [oasr registry](REGISTRY.md) - Manage skill registry (validate, add, remove, list, sync)
+- [oasr registry](REGISTRY.md) - Manage skill registry (validate, add, remove, list, sync, prune)
 - [oasr diff](DIFF.md) - Show status of tracked skills
 - [oasr sync](SYNC.md) - Refresh outdated tracked skills
 - [oasr config](CONFIG.md) - Manage configuration (NEW in v0.4.0)
@@ -11,7 +11,6 @@
 - [oasr exec](EXEC.md) - Execute skills as CLI tools (NEW in v0.4.0)
 - [oasr find](FIND.md) - Find/discover skills in your file system
 - [oasr validate](VALIDATE.md) - Validate a skill
-- [oasr clean](CLEAN.md) - Clean up the registry
 - [oasr adapter](ADAPTER.md) - Generate IDE/Tooling adapters
 - [oasr update](UPDATE.md) - Update the `oasr` CLI
 - [oasr info](INFO.md) - Show detailed information about a skill
@@ -19,6 +18,9 @@
 
 ## Deprecated Commands
 
+### v0.4.1 Deprecations
+- [oasr clean](CLEAN.md) - **Deprecated**, use `oasr registry prune` instead (will be removed in v0.6.0)
+
 ### v0.4.0 Deprecations
 - [oasr use](USE.md) - **Deprecated**, use `oasr clone` instead (will be removed in v0.5.0)
 

oasr-0.4.2/docs/commands/CLEAN.md

@@ -0,0 +1,13 @@
+# `oasr clean` (DEPRECATED)
+
+> **⚠️ Warning: This command is deprecated and will be removed in v0.6.0.**  
+> Use `oasr registry prune` instead.
+
+Remove orphaned manifests and entries for missing skills.
+
+```bash
+oasr clean              # Shows deprecation warning
+oasr registry prune     # New command (recommended)
+```
+
+This command now delegates to `oasr registry prune`. See [REGISTRY.md](./REGISTRY.md#oasr-registry-prune) for full documentation.

{oasr-0.4.0 → oasr-0.4.2}/docs/commands/CLONE.md

@@ -17,7 +17,6 @@ oasr clone skill-one skill-two        # Multiple skills
 ## Options
 
 - `-d, --dest DIR` — Destination directory (default: current directory)
-- `-r, --recursive` — Create destination directory if it doesn't exist
 - `--quiet` — Suppress informational output
 - `--json` — Output results in JSON format
 

{oasr-0.4.0 → oasr-0.4.2}/docs/commands/REGISTRY.md

@@ -105,6 +105,40 @@ Syncing remote skills...
 Synced: 2 skills
 ```
 
+### `oasr registry prune`
+
+Clean up corrupted/missing skills and orphaned artifacts:
+
+```bash
+oasr registry prune              # Interactive cleanup
+oasr registry prune -y           # Skip confirmation
+oasr registry prune --dry-run    # Show what would be cleaned
+oasr registry prune --json       # JSON output
+```
+
+This command:
+- Removes skills whose source files/URLs are no longer accessible
+- Removes orphaned manifest files not in the registry
+- Requires confirmation unless `-y` flag is used
+
+**Example output:**
+```bash
+$ oasr registry prune
+Checking 3 remote skill(s)...
+  ↓ python-analyzer (checking GitHub...)
+  ✓ python-analyzer (checked)
+
+The following will be cleaned:
+
+Skills with missing sources:
+  ✗ old-skill (/path/to/missing)
+
+Proceed with cleanup? [y/N] y
+Removed skill: old-skill
+
+Cleaned 1 skill(s), 0 manifest(s)
+```
+
 ## Migration from v0.2.0
 
 The v0.3.0 CLI taxonomy reorganizes commands under the `registry` subcommand:
@@ -117,3 +151,5 @@ The v0.3.0 CLI taxonomy reorganizes commands under the `registry` subcommand:
 | `oasr status` | `oasr registry -v` |
 | `oasr sync` (manifest validation) | `oasr registry` |
 | `oasr sync --update` (remote sync) | `oasr registry sync` |
+
+**v0.4.1 update:** `oasr clean` → `oasr registry prune`

{oasr-0.4.0 → oasr-0.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "oasr"
-version = "0.4.0"
+version = "0.4.2"
 description = "CLI for managing agent skills across IDE integrations"
 readme = "README.md"
 requires-python = ">=3.10"

{oasr-0.4.0 → oasr-0.4.2}/src/cli.py

@@ -13,7 +13,7 @@ from pathlib import Path
 from commands import adapter, clean, clone, config, diff, exec, find, registry, sync, update, use, validate
 from commands import help as help_cmd
 
-__version__ = "0.4.0"
+__version__ = "0.4.2"
 
 
 def main(argv: list[str] | None = None) -> int:

oasr-0.4.2/src/commands/clean.py

@@ -0,0 +1,30 @@
+"""`asr clean` command - DEPRECATED, use `oasr registry prune` instead."""
+
+from __future__ import annotations
+
+import argparse
+import sys
+
+
+def register(subparsers) -> None:
+    p = subparsers.add_parser(
+        "clean",
+        help="(Deprecated) Clean up corrupted/missing skills - use 'oasr registry prune'",
+    )
+    p.add_argument("-y", "--yes", action="store_true", help="Skip confirmation prompt")
+    p.add_argument("--json", action="store_true", help="Output in JSON format")
+    p.add_argument("--dry-run", action="store_true", help="Show what would be cleaned without doing it")
+    p.set_defaults(func=run)
+
+
+def run(args: argparse.Namespace) -> int:
+    """Delegate to registry prune with deprecation warning."""
+    # Show deprecation warning unless --json or --quiet
+    if not args.json:
+        print("⚠ Warning: 'oasr clean' is deprecated. Use 'oasr registry prune' instead.", file=sys.stderr)
+        print("   This command will be removed in v0.6.0.\n", file=sys.stderr)
+
+    # Delegate to registry prune
+    from commands.registry import run_prune
+
+    return run_prune(args)

{oasr-0.4.0 → oasr-0.4.2}/src/commands/exec.py

@@ -104,14 +104,9 @@ def run(args: argparse.Namespace) -> int:
 
     try:
         result = driver.execute(skill_content, user_prompt)
-        if result.success:
-            # Print agent output to stdout (this is the actual result)
-            print(result.output)
-            return 0
-        else:
-            print("\nError executing skill:", file=sys.stderr)
-            print(result.error, file=sys.stderr)
-            return 1
+        # CompletedProcess has returncode attribute (0 = success)
+        # Output was already streamed to stdout since capture_output=False
+        return result.returncode
     except Exception as e:
         print(f"\nUnexpected error: {e}", file=sys.stderr)
         return 1

{oasr-0.4.0 → oasr-0.4.2}/src/commands/registry.py

@@ -64,6 +64,13 @@ def register(subparsers) -> None:
     sync_p.add_argument("--config", type=Path, help="Override config file path")
     sync_p.set_defaults(func=run_sync)
 
+    # registry prune
+    prune_p = registry_subparsers.add_parser("prune", help="Clean up corrupted/missing skills and orphaned artifacts")
+    prune_p.add_argument("-y", "--yes", action="store_true", help="Skip confirmation prompt")
+    prune_p.add_argument("--json", action="store_true", help="Output in JSON format")
+    prune_p.add_argument("--dry-run", action="store_true", help="Show what would be cleaned without doing it")
+    prune_p.set_defaults(func=run_prune)
+
 
 def run_validate(args: argparse.Namespace) -> int:
     """Validate registry manifests (default oasr registry behavior)."""
@@ -301,3 +308,140 @@ def run_sync(args: argparse.Namespace) -> int:
             print(f"{len(pruned)} skill(s) pruned")
 
     return 1 if missing_count > 0 else 0
+
+
+def run_prune(args: argparse.Namespace) -> int:
+    """Clean up corrupted/missing skills and orphaned artifacts (oasr registry prune)."""
+    from manifest import delete_manifest, list_manifests
+
+    entries = load_registry()
+    registered_names = {e.name for e in entries}
+    manifest_names = set(list_manifests())
+
+    to_remove_skills = []
+    to_remove_manifests = []
+
+    # Check for remote skills and show progress header
+    remote_count = 0
+    for entry in entries:
+        manifest = load_manifest(entry.name)
+        if manifest and is_remote_source(manifest.source_path):
+            remote_count += 1
+
+    if remote_count > 0 and not args.json:
+        print(f"Checking {remote_count} remote skill(s)...", file=sys.stderr)
+
+    for entry in entries:
+        manifest = load_manifest(entry.name)
+        if manifest:
+            # Show progress for remote skills
+            is_remote = is_remote_source(manifest.source_path)
+            if is_remote and not args.json:
+                platform = (
+                    "GitHub"
+                    if "github.com" in manifest.source_path
+                    else "GitLab"
+                    if "gitlab.com" in manifest.source_path
+                    else "remote"
+                )
+                print(f"  ↓ {entry.name} (checking {platform}...)", file=sys.stderr, flush=True)
+
+            status = check_manifest(manifest)
+
+            if is_remote and not args.json:
+                print(f"  ✓ {entry.name} (checked)", file=sys.stderr)
+
+            if status.status == "missing":
+                to_remove_skills.append(
+                    {
+                        "name": entry.name,
+                        "reason": "source missing",
+                        "path": entry.path,
+                    }
+                )
+
+    orphaned = manifest_names - registered_names
+    for name in orphaned:
+        to_remove_manifests.append(
+            {
+                "name": name,
+                "reason": "orphaned manifest (not in registry)",
+            }
+        )
+
+    if not to_remove_skills and not to_remove_manifests:
+        if args.json:
+            print(json.dumps({"cleaned": 0, "message": "nothing to clean"}))
+        else:
+            print("Nothing to clean.")
+        return 0
+
+    if args.json:
+        result = {
+            "skills_to_remove": to_remove_skills,
+            "manifests_to_remove": to_remove_manifests,
+            "dry_run": args.dry_run,
+        }
+        if not args.dry_run and not args.yes:
+            result["requires_confirmation"] = True
+        print(json.dumps(result, indent=2))
+        if args.dry_run:
+            return 0
+    else:
+        print("The following will be cleaned:\n")
+
+        if to_remove_skills:
+            print("Skills with missing sources:")
+            for s in to_remove_skills:
+                print(f"  ✗ {s['name']} ({s['path']})")
+
+        if to_remove_manifests:
+            print("\nOrphaned manifests:")
+            for m in to_remove_manifests:
+                print(f"  ✗ {m['name']}")
+
+        print()
+
+        if args.dry_run:
+            print("(dry run - no changes made)")
+            return 0
+
+    if not args.yes and not args.json:
+        try:
+            response = input("Proceed with cleanup? [y/N] ").strip().lower()
+            if response not in ("y", "yes"):
+                print("Aborted.")
+                return 1
+        except (EOFError, KeyboardInterrupt):
+            print("\nAborted.")
+            return 1
+
+    removed_skills = []
+    removed_manifests = []
+
+    for s in to_remove_skills:
+        remove_skill(s["name"])
+        removed_skills.append(s["name"])
+
+    for m in to_remove_manifests:
+        delete_manifest(m["name"])
+        removed_manifests.append(m["name"])
+
+    if args.json:
+        print(
+            json.dumps(
+                {
+                    "removed_skills": removed_skills,
+                    "removed_manifests": removed_manifests,
+                },
+                indent=2,
+            )
+        )
+    else:
+        for name in removed_skills:
+            print(f"Removed skill: {name}")
+        for name in removed_manifests:
+            print(f"Removed manifest: {name}")
+        print(f"\nCleaned {len(removed_skills)} skill(s), {len(removed_manifests)} manifest(s)")
+
+    return 0

{oasr-0.4.0 → oasr-0.4.2}/tests/test_exec.py

@@ -107,8 +107,7 @@ class TestExecCommand:
         )
 
         mock_result = mock.Mock()
-        mock_result.success = True
-        mock_result.output = "Agent output here"
+        mock_result.returncode = 0  # Success
 
         mock_driver = mock.Mock()
         mock_driver.execute.return_value = mock_result
@@ -122,7 +121,6 @@ class TestExecCommand:
 
         assert result == 0
         captured = capsys.readouterr()
-        assert "Agent output here" in captured.out
         assert "Executing skill 'test-skill' with codex" in captured.err
 
         # Verify driver was called with skill content and prompt
@@ -143,8 +141,7 @@ class TestExecCommand:
         )
 
         mock_result = mock.Mock()
-        mock_result.success = False
-        mock_result.error = "Agent failed with error"
+        mock_result.returncode = 1  # Failure
 
         mock_driver = mock.Mock()
         mock_driver.execute.return_value = mock_result
@@ -157,8 +154,6 @@ class TestExecCommand:
             result = exec_cmd.run(args)
 
         assert result == 1
-        captured = capsys.readouterr()
-        assert "Agent failed with error" in captured.err
 
     def test_exec_with_explicit_agent(self, capsys, mock_registry):
         """Test exec with explicit agent flag."""
@@ -172,8 +167,7 @@ class TestExecCommand:
         )
 
         mock_result = mock.Mock()
-        mock_result.success = True
-        mock_result.output = "Success"
+        mock_result.returncode = 0  # Success
 
         mock_driver = mock.Mock()
         mock_driver.execute.return_value = mock_result

{oasr-0.4.0 → oasr-0.4.2}/uv.lock

@@ -138,7 +138,7 @@ wheels = [
 
 [[package]]
 name = "oasr"
-version = "0.4.0"
+version = "0.4.2"
 source = { editable = "." }
 dependencies = [
     { name = "pyyaml" },

oasr-0.4.0/docs/commands/CLEAN.md

@@ -1,11 +0,0 @@
-# `oasr clean`
-
-Remove orphaned manifests and entries for missing skills.
-
-```bash
-oasr clean
-oasr clean --dry-run
-```
-
-> **note**  
-> The same behaviour can be achieved using `oasr sync --prune`

oasr-0.4.0/doctor/SKILL.md

@@ -1,35 +0,0 @@
----
-name: doctor
-license: MIT
-description: 'Diagnoses software failures by combining deterministic evidence gathering
-  with agent judgment. Models failures as medical cases. Idempotent — run repeatedly
-  until confident diagnosis, then generate schema-based treatment.
-
-  '
-metadata:
-  author: Jordan Godau
-  version: 0.2.0
-  references:
-  - 00_ROUTER.md
-  - 01_SUMMARY.md
-  - 02_TRIGGERS.md
-  - 03_ALWAYS.md
-  - 04_NEVER.md
-  - 05_PROCEDURE.md
-  - 06_FAILURES.md
-  keywords:
-  - diagnose
-  - debug
-  - investigate
-  - evidence
-  - hypothesis
-  - treatment
-  oasr:
-    hash: sha256:f9fda45b7797923e9379e36c1b519c40b09f05f78bfa172a9d006c14b7283535
-    source: /home/jgodau/work/personal/skills/doctor/doctor
-    synced: '2026-02-01T06:35:54.685306Z'
----
-
-# INSTRUCTIONS
-
-1. Refer to `metadata.references`.

oasr-0.4.0/doctor/assets/schemas/session.schema.yaml

@@ -1,87 +0,0 @@
-# Doctor Session Schema
-# Artifact: .doctor/session.yaml
-
-type: object
-required:
-  - status
-  - created_at
-
-properties:
-  status:
-    type: string
-    enum: [investigating, diagnosed, treated, abandoned]
-    description: Current session status
-
-  created_at:
-    type: string
-    format: date-time
-    description: RFC3339 UTC timestamp
-
-  updated_at:
-    type: string
-    format: date-time
-    description: RFC3339 UTC timestamp of last update
-
-  patient:
-    type: string
-    description: Repository or system name
-
-  symptoms:
-    type: array
-    items:
-      type: object
-      properties:
-        description:
-          type: string
-        category:
-          type: string
-          enum: [error, timeout, crash, wrong_output, performance, unknown]
-        evidence:
-          type: string
-    description: Observed symptoms
-
-  hypotheses:
-    type: array
-    items:
-      type: object
-      properties:
-        description:
-          type: string
-        confidence:
-          type: integer
-          minimum: 0
-          maximum: 100
-        evidence_for:
-          type: array
-          items:
-            type: string
-        evidence_against:
-          type: array
-          items:
-            type: string
-        falsifiable_by:
-          type: string
-    description: Working hypotheses with confidence
-
-  diagnosis:
-    type: object
-    properties:
-      summary:
-        type: string
-      confidence:
-        type: integer
-        minimum: 0
-        maximum: 100
-      root_cause:
-        type: string
-      contributing_factors:
-        type: array
-        items:
-          type: string
-    description: Final diagnosis (when confident)
-
-  evidence_files:
-    type: array
-    items:
-      type: string
-    description: Paths to evidence snapshots in .doctor/evidence/

oasr-0.4.0/doctor/assets/schemas/treatment.schema.yaml

@@ -1,70 +0,0 @@
-# Treatment Plan Schema
-# Artifact: .doctor/treatment.md (generated from this schema)
-
-type: object
-required:
-  - diagnosis_summary
-  - confidence
-  - options
-
-properties:
-  diagnosis_summary:
-    type: string
-    description: One-line diagnosis
-
-  confidence:
-    type: integer
-    minimum: 0
-    maximum: 100
-    description: Diagnosis confidence percentage
-
-  root_cause:
-    type: string
-    description: Identified root cause
-
-  options:
-    type: array
-    minItems: 1
-    items:
-      type: object
-      required:
-        - name
-        - description
-        - risk
-      properties:
-        name:
-          type: string
-          description: Short option name
-        description:
-          type: string
-          description: What this option does
-        risk:
-          type: string
-          enum: [low, medium, high]
-        effort:
-          type: string
-          enum: [trivial, small, medium, large]
-        reversible:
-          type: boolean
-        steps:
-          type: array
-          items:
-            type: string
-          description: Ordered implementation steps
-    description: Treatment options (at least one required)
-
-  recommended:
-    type: string
-    description: Name of recommended option
-
-  caveats:
-    type: array
-    items:
-      type: string
-    description: Warnings or considerations
-
-  follow_up:
-    type: array
-    items:
-      type: string
-    description: Suggested follow-up actions