core-cli 1.0.1__tar.gz → 1.0.2__tar.gz

{core_cli-1.0.1 → core_cli-1.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: core-cli
-Version: 1.0.1
+Version: 1.0.2
 Summary: Consumer governance CLI for CORE — pure HTTP client for governing repositories against CORE constitutional rules.
 Project-URL: Homepage, https://github.com/DariuszNewecki/CORE
 Project-URL: Repository, https://github.com/DariuszNewecki/core-cli
@@ -61,7 +61,7 @@ export CORE_API_URL=http://localhost:8000
 | `code` | `actions`, `audit_duplicates`, `bridges`, `check_imports`, `check_ui`, `docstrings`, `fix_atomic`, `format`, `integrity`, `lint`, `logging`, `test` |
 | `symbols` | `audit`, `fix_ids`, `resolve_duplicates`, `sync` |
 | `vectors` | `query`, `rebuild`, `status`, `sync`, `sync_code` |
-| `project` | `docs`, `onboard`, `scout` |
+| `project` | `docs`, `onboard`, `promote`, `scout` |
 
 ## Architecture
 

{core_cli-1.0.1 → core_cli-1.0.2}/README.md

@@ -35,7 +35,7 @@ export CORE_API_URL=http://localhost:8000
 | `code` | `actions`, `audit_duplicates`, `bridges`, `check_imports`, `check_ui`, `docstrings`, `fix_atomic`, `format`, `integrity`, `lint`, `logging`, `test` |
 | `symbols` | `audit`, `fix_ids`, `resolve_duplicates`, `sync` |
 | `vectors` | `query`, `rebuild`, `status`, `sync`, `sync_code` |
-| `project` | `docs`, `onboard`, `scout` |
+| `project` | `docs`, `onboard`, `promote`, `scout` |
 
 ## Architecture
 

{core_cli-1.0.1 → core_cli-1.0.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "core-cli"
-version = "1.0.1"
+version = "1.0.2"
 description = "Consumer governance CLI for CORE — pure HTTP client for governing repositories against CORE constitutional rules."
 readme = "README.md"
 license = {text = "MIT"}

{core_cli-1.0.1 → core_cli-1.0.2}/src/core_cli/resources/project/onboard.py

@@ -46,6 +46,12 @@ async def onboard_project(
     Pass --write --stage to write to work/staged/<name>/ first, inspect the
     files, then run `project onboard promote <path>` to deliver to the target.
     """
+    # `path` is sent as a plain string over HTTP and resolved by the CORE API
+    # process, not this CLI — a relative path (e.g. ".") would silently
+    # resolve against the *server's* cwd, not the caller's. Resolve to
+    # absolute here so it means the same thing on both sides.
+    path = path.resolve()
+
     if stage and not write:
         console.print(
             "[yellow]--stage has no effect without --write "
@@ -77,7 +83,7 @@ async def onboard_project(
         console.print(
             f"\n[bold green]Staged to[/bold green] {stage_dir}/.intent\n"
             f"[dim]Inspect, then run:[/dim]  "
-            f"[bold]core-admin project onboard promote {path}[/bold]"
+            f"[bold]core project promote {path}[/bold]"
         )
     elif mode == "dry-run":
         console.print("[yellow]DRY-RUN complete. Pass --write to apply.[/yellow]")
@@ -100,6 +106,9 @@ async def promote_onboard(
     Reads from work/staged/<name>/.intent/ (within CORE's repo) and writes to
     <path>/.intent/. Run `project onboard promote --help` for full lifecycle details.
     """
+    # Same reasoning as onboard_project: resolve before sending over HTTP.
+    path = path.resolve()
+
     console.print(f"[bold cyan]Promoting staged onboard to:[/bold cyan] {path}")
     client = CoreApiClient()
     try:
@@ -107,4 +116,4 @@ async def promote_onboard(
     except Exception as exc:
         console.print(f"[red]Promote failed: {exc}[/red]")
         raise typer.Exit(1) from exc
-    console.print(f"[bold green]Promoted to[/bold green] {Path(path).resolve() / '.intent'}")
+    console.print(f"[bold green]Promoted to[/bold green] {path / '.intent'}")

{core_cli-1.0.1 → core_cli-1.0.2}/src/core_cli/resources/project/scout.py

@@ -62,6 +62,11 @@ async def scout_project(
     (ADR-119 D5 — no --accept-all). Requires Phase A first:
     run `project onboard <target> --write` before this command.
     """
+    # path is sent as a plain string over HTTP for server-side signal
+    # extraction — a relative path would resolve against the CORE API
+    # process's cwd, not the caller's, and silently analyze the wrong repo.
+    path = path.resolve()
+
     console.print(Rule("[bold cyan]Scout — Phase B: Rule Induction[/bold cyan]"))
     console.print(f"[bold cyan]Target:[/bold cyan] {path}")
 
@@ -101,7 +106,7 @@ async def scout_project(
         return
 
     # ── Write ─────────────────────────────────────────────────────────────────
-    target_intent = Path(path).resolve() / ".intent"
+    target_intent = path / ".intent"
     rules_json = _build_rules_document(confirmed)
     mappings_yaml = _build_mappings_document(confirmed)
 

core_cli-1.0.2/tests/test_project.py

@@ -0,0 +1,98 @@
+# tests/test_project.py
+
+"""Unit tests for project onboard/promote/scout — stub the HTTP client layer.
+
+Covers the relative-path resolution fix: `path` is sent as a plain string
+over HTTP and resolved by the CORE API process, not this CLI. A relative
+path must be resolved to absolute here before it's ever sent, or it would
+silently mean a different location depending on which process resolves it.
+"""
+
+from __future__ import annotations
+
+from pathlib import Path
+from unittest.mock import AsyncMock, MagicMock, patch
+
+from core_cli.resources.project.onboard import onboard_project, promote_onboard
+from core_cli.resources.project.scout import scout_project
+
+
+_ONBOARD_CLIENT = "core_cli.resources.project.onboard.CoreApiClient"
+_SCOUT_CLIENT = "core_cli.resources.project.scout.CoreApiClient"
+
+
+def _make_project_client(**overrides) -> AsyncMock:
+    project = AsyncMock()
+    project.onboard.return_value = {"mode": "write", "stage_dir": None}
+    project.promote.return_value = {"promoted": True}
+    project.scout.return_value = {"candidates": [], "candidate_count": 0, "matched": 0}
+    for key, value in overrides.items():
+        setattr(project, key, value)
+
+    client = AsyncMock()
+    client.project = project
+    return client
+
+
+# ---------------------------------------------------------------------------
+# onboard_project
+# ---------------------------------------------------------------------------
+
+
+async def test_onboard_resolves_relative_path(tmp_path: Path, monkeypatch) -> None:
+    monkeypatch.chdir(tmp_path)
+    mock_client = _make_project_client()
+
+    with patch(_ONBOARD_CLIENT, return_value=mock_client):
+        await onboard_project.__wrapped__(
+            ctx=MagicMock(), path=Path("."), write=True, stage=False
+        )
+
+    mock_client.project.onboard.assert_called_once_with(
+        str(tmp_path), write=True, stage=False
+    )
+
+
+async def test_onboard_absolute_path_unchanged(tmp_path: Path) -> None:
+    mock_client = _make_project_client()
+
+    with patch(_ONBOARD_CLIENT, return_value=mock_client):
+        await onboard_project.__wrapped__(
+            ctx=MagicMock(), path=tmp_path, write=True, stage=False
+        )
+
+    mock_client.project.onboard.assert_called_once_with(
+        str(tmp_path), write=True, stage=False
+    )
+
+
+# ---------------------------------------------------------------------------
+# promote_onboard
+# ---------------------------------------------------------------------------
+
+
+async def test_promote_resolves_relative_path(tmp_path: Path, monkeypatch) -> None:
+    monkeypatch.chdir(tmp_path)
+    mock_client = _make_project_client()
+
+    with patch(_ONBOARD_CLIENT, return_value=mock_client):
+        await promote_onboard.__wrapped__(ctx=MagicMock(), path=Path("."))
+
+    mock_client.project.promote.assert_called_once_with(str(tmp_path))
+
+
+# ---------------------------------------------------------------------------
+# scout_project
+# ---------------------------------------------------------------------------
+
+
+async def test_scout_resolves_relative_path(tmp_path: Path, monkeypatch) -> None:
+    monkeypatch.chdir(tmp_path)
+    mock_client = _make_project_client()
+
+    with patch(_SCOUT_CLIENT, return_value=mock_client):
+        await scout_project.__wrapped__(
+            ctx=MagicMock(), path=Path("."), write=False, reset=False
+        )
+
+    mock_client.project.scout.assert_called_once_with(str(tmp_path), reset=False)