filekeeper-cli 0.1.7__tar.gz → 0.1.8__tar.gz

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: filekeeper-cli
-Version: 0.1.7
+Version: 0.1.8
 Summary: Safe, predictable filesystem operations for humans and AI agents, with atomic writes and workspace confinement.
 Keywords: filesystem,atomic-write,cli,agents,mcp
 Author: sangingeum
@@ -160,6 +160,16 @@ If the workspace is a git repository, Filekeeper adds `.filekeeper/` to
 `.gitignore` when it creates the state directory. Outside a repository it writes
 nothing.
 
+**Reported paths when the root is `/`.** Result paths are normally
+workspace-relative, so they are the same whichever tree holds them. With a root
+of `/` that form is indistinguishable from a repository-relative path, so every
+reported path keeps its leading `/` and is exactly the absolute path: read
+results, `file_info`, `file_read_many` entries, search results, directory
+listings, copy/move result paths, and history target and item paths. Every other
+root is unchanged, byte for byte. History rows are stored relative and rendered
+when read, so existing records display correctly without a migration. Error
+envelopes do not change.
+
 ## Durability guarantees, in plain words
 
 - **One file, replaced at a time, is atomic.** A whole-file write goes to a

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/README.md

@@ -139,6 +139,16 @@ If the workspace is a git repository, Filekeeper adds `.filekeeper/` to
 `.gitignore` when it creates the state directory. Outside a repository it writes
 nothing.
 
+**Reported paths when the root is `/`.** Result paths are normally
+workspace-relative, so they are the same whichever tree holds them. With a root
+of `/` that form is indistinguishable from a repository-relative path, so every
+reported path keeps its leading `/` and is exactly the absolute path: read
+results, `file_info`, `file_read_many` entries, search results, directory
+listings, copy/move result paths, and history target and item paths. Every other
+root is unchanged, byte for byte. History rows are stored relative and rendered
+when read, so existing records display correctly without a migration. Error
+envelopes do not change.
+
 ## Durability guarantees, in plain words
 
 - **One file, replaced at a time, is atomic.** A whole-file write goes to a

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "filekeeper-cli"
-version = "0.1.7"
+version = "0.1.8"
 description = "Safe, predictable filesystem operations for humans and AI agents, with atomic writes and workspace confinement."
 readme = "README.md"
 requires-python = ">=3.12,<3.13"

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/pyproject.toml.orig

@@ -1,6 +1,6 @@
 [project]
 name = "filekeeper-cli"
-version = "0.1.7"
+version = "0.1.8"
 description = "Safe, predictable filesystem operations for humans and AI agents, with atomic writes and workspace confinement."
 readme = "README.md"
 requires-python = ">=3.12,<3.13"

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/__init__.py

@@ -23,7 +23,7 @@ from .core import (
     Workspace,
 )
 
-__version__ = "0.1.7"
+__version__ = "0.1.8"
 
 __all__ = [
     "CallerSurface",

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/cli/app.py

@@ -482,14 +482,19 @@ def _handle_workspace_list(context: _Context) -> int:
         output.workspace_summary(workspace, is_default=workspace.name == default_name)
         for workspace in workspaces
     ]
-    context.emit(
-        {
-            "ok": True,
-            "default_workspace": default_name,
-            "workspaces": output.jsonable(workspaces),
-        },
-        "\n".join(lines) if lines else "no workspaces registered",
-    )
+    payload: dict[str, object] = {
+        "ok": True,
+        "default_workspace": default_name,
+        "workspaces": output.jsonable(workspaces),
+    }
+    warnings = [
+        f"workspace '{workspace.name}' is registered at a root that no longer exists"
+        for workspace in workspaces
+        if not workspace.root_exists
+    ]
+    if warnings:
+        payload["warnings"] = warnings
+    context.emit(payload, "\n".join(lines) if lines else "no workspaces registered")
     return exit_codes.SUCCESS
 
 

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/cli/output.py

@@ -163,9 +163,7 @@ def mutation(result: MutationResult) -> str:
             "(recoverable through history, not atomic across files)"
         )
         for file in result.patch_files:
-            hunks = ", ".join(
-                f"hunk {hunk.old_start}: {hunk.status}" for hunk in file.hunks
-            )
+            hunks = ", ".join(f"hunk {hunk.old_start}: {hunk.status}" for hunk in file.hunks)
             lines.append(f"  {file.change:<9} {file.path}  {hunks}")
     if result.preview_diff:
         lines.append("diff:")
@@ -240,6 +238,8 @@ def history_diff(diff: OperationDiff) -> str:
 
 def workspace_summary(workspace: Workspace, *, is_default: bool = False) -> str:
     suffix = "  (default)" if is_default else ""
+    if not workspace.root_exists:
+        suffix += "  (root missing)"
     return f"{workspace.name}\t{workspace.root_path}\t{workspace.id}{suffix}"
 
 
@@ -249,6 +249,7 @@ def workspace_detail(workspace: Workspace, *, is_default: bool = False) -> str:
             f"name: {workspace.name}",
             f"id: {workspace.id}",
             f"root: {workspace.root_path}",
+            f"root_exists: {str(workspace.root_exists).lower()}",
             f"created_at: {workspace.created_at}",
             f"updated_at: {workspace.updated_at}",
             f"default: {is_default}",

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/config.py

@@ -91,8 +91,7 @@ def load_config(home: Path) -> Config:
             raw = tomllib.load(handle)
     except (OSError, tomllib.TOMLDecodeError) as exc:
         raise StorageError(
-            f"cannot read configuration file: "
-            f"{getattr(exc, 'strerror', None) or exc}",
+            f"cannot read configuration file: {getattr(exc, 'strerror', None) or exc}",
             details={"path": str(path)},
         ) from exc
 

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/errors.py

@@ -76,9 +76,7 @@ class OverrideHint:
 def override_hint(*, core: str, cli: str, mcp: str) -> OverrideHint:
     """Build a hint whose sentence names each surface's own argument."""
 
-    return OverrideHint(
-        {CallerSurface.CORE: core, CallerSurface.CLI: cli, CallerSurface.MCP: mcp}
-    )
+    return OverrideHint({CallerSurface.CORE: core, CallerSurface.CLI: cli, CallerSurface.MCP: mcp})
 
 
 class FilekeeperError(Exception):

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/history/blobs.py

@@ -68,9 +68,7 @@ class BlobStore:
                 "blob is missing from the store", details={"hash": normalize_hash(digest)}
             ) from exc
         except OSError as exc:  # pragma: no cover - filesystem failure
-            raise StorageError(
-                f"cannot read blob: {exc.strerror or type(exc).__name__}"
-            ) from exc
+            raise StorageError(f"cannot read blob: {exc.strerror or type(exc).__name__}") from exc
         actual = hash_bytes(data)
         if actual != normalize_hash(digest):
             raise StorageError(

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/history/journal.py

@@ -321,9 +321,7 @@ def _load_metadata(raw: str | None) -> dict[str, Any]:
 
 
 def _current_state(root: Path, item: sqlite3.Row | OperationItemRecord) -> tuple[bool, str | None]:
-    relative = (
-        str(item["relative_path"]) if isinstance(item, sqlite3.Row) else item.relative_path
-    )
+    relative = str(item["relative_path"]) if isinstance(item, sqlite3.Row) else item.relative_path
     path = root / relative
     if str(item["item_type"] if isinstance(item, sqlite3.Row) else item.item_type) == "directory":
         return path.is_dir(), None
@@ -351,9 +349,7 @@ def _matches(
     return current_hash == digest
 
 
-def _classify_pending(
-    root: Path, items: list[sqlite3.Row]
-) -> tuple[str, list[str]]:
+def _classify_pending(root: Path, items: list[sqlite3.Row]) -> tuple[str, list[str]]:
     if not items:
         return FAILED, []
     all_after = True

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/history/repository.py

@@ -3,21 +3,45 @@
 from __future__ import annotations
 
 import sqlite3
+from dataclasses import replace
+from pathlib import Path
 from typing import Any
 
 from ..diffing import unified_diff_text
 from ..errors import HistoryUnavailableError, OperationNotFoundError, StorageError
 from ..models import DiffEntry, OperationDiff, OperationItemRecord, OperationRecord
+from ..paths import reported_path
 from .journal import OperationJournal, _row_to_operation
 
 
+def _reported_record(root: Path, record: OperationRecord) -> OperationRecord:
+    """Render a stored record's paths for a caller, without migrating the row.
+
+    History stores workspace-relative paths; a workspace rooted at ``/`` reports
+    them with their leading separator so the string is the absolute path. Every
+    other root renders byte-identically to what was stored.
+    """
+
+    return replace(
+        record,
+        target_path=(
+            None if record.target_path is None else reported_path(root, record.target_path)
+        ),
+        items=tuple(
+            replace(item, relative_path=reported_path(root, item.relative_path))
+            for item in record.items
+        ),
+        history_skipped=tuple(reported_path(root, path) for path in record.history_skipped),
+    )
+
+
 def get_operation(journal: OperationJournal, operation_id: str) -> OperationRecord:
     record = journal.get(operation_id)
     if record is None:
         raise OperationNotFoundError(
             "no operation with that id in this workspace", details={"operation_id": operation_id}
         )
-    return record
+    return _reported_record(journal.root, record)
 
 
 def _matching_operations(journal: OperationJournal, path: str | None) -> list[str]:
@@ -80,7 +104,7 @@ def list_history(
                 "SELECT * FROM operation_items WHERE operation_id = ? ORDER BY id",
                 (row["id"],),
             ).fetchall()
-            records.append(_row_to_operation(row, items))
+            records.append(_reported_record(journal.root, _row_to_operation(row, items)))
     except sqlite3.Error as exc:
         raise StorageError(f"cannot read operation history: {exc}") from exc
     finally:
@@ -116,7 +140,7 @@ def _diff_entry(journal: OperationJournal, item: OperationItemRecord) -> DiffEnt
     after_text = _decode(after)
     if before_text is None or after_text is None:
         return DiffEntry(
-            path=item.relative_path,
+            path=reported_path(journal.root, item.relative_path),
             item_type=item.item_type,
             before_exists=item.before_exists,
             after_exists=item.after_exists,
@@ -130,7 +154,7 @@ def _diff_entry(journal: OperationJournal, item: OperationItemRecord) -> DiffEnt
         tofile=f"b/{item.relative_path}" if item.after_exists else "/dev/null",
     )
     return DiffEntry(
-        path=item.relative_path,
+        path=reported_path(journal.root, item.relative_path),
         item_type=item.item_type,
         before_exists=item.before_exists,
         after_exists=item.after_exists,
@@ -140,7 +164,11 @@ def _diff_entry(journal: OperationJournal, item: OperationItemRecord) -> DiffEnt
 
 
 def diff_operation(journal: OperationJournal, operation_id: str) -> OperationDiff:
-    record = get_operation(journal, operation_id)
+    record = journal.get(operation_id)
+    if record is None:
+        raise OperationNotFoundError(
+            "no operation with that id in this workspace", details={"operation_id": operation_id}
+        )
     return OperationDiff(
         operation_id=record.operation_id,
         items=tuple(

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/history/undo.py

@@ -26,6 +26,7 @@ from ..errors import (
     storage_failure,
 )
 from ..models import MutationResult, OperationItemRecord, OperationItemResult, OperationRecord
+from ..paths import reported_path
 from ..safety.atomic import atomic_write_bytes, fsync_directory
 from ..safety.hashing import hash_file
 from .journal import (
@@ -225,9 +226,7 @@ def _apply(journal: OperationJournal, steps: list[_Step]) -> None:
             path.unlink()
             fsync_directory(path.parent)
         except OSError as exc:
-            raise storage_failure(
-                "remove file during undo", step.item.relative_path, exc
-            ) from exc
+            raise storage_failure("remove file during undo", step.item.relative_path, exc) from exc
 
     directories = [step for step in removable if step.action == "remove-directory"]
     for step in sorted(
@@ -271,7 +270,7 @@ def _result(
 ) -> MutationResult:
     items = tuple(
         OperationItemResult(
-            path=step.item.relative_path,
+            path=reported_path(journal.root, step.item.relative_path),
             change=(
                 "deleted"
                 if not step.item.before_exists
@@ -287,7 +286,7 @@ def _result(
     )
     return MutationResult(
         workspace=workspace_name,
-        path=record.target_path or "",
+        path=(reported_path(journal.root, record.target_path) if record.target_path else ""),
         changed=True,
         before_hash=record.after_hash,
         after_hash=record.before_hash,
@@ -393,9 +392,7 @@ def undo_operation(
         raise
     journal.commit(begun.operation_id)
     journal._set_status(record.operation_id, UNDONE)
-    return _result(
-        journal, record, begun.operation_id, workspace_name=workspace_name, steps=steps
-    )
+    return _result(journal, record, begun.operation_id, workspace_name=workspace_name, steps=steps)
 
 
 def latest_undoable(journal: OperationJournal, *, actor: str | None = None) -> OperationRecord:

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/models.py

@@ -24,6 +24,7 @@ class Workspace:
     root_path: str
     created_at: str
     updated_at: str
+    root_exists: bool = True
 
 
 @dataclass(frozen=True, slots=True)

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/operations/directory.py

@@ -72,9 +72,7 @@ def _walk_tree(root: Path) -> list[Path]:
     return found
 
 
-def _file_bytes(
-    path: Path, cap: int, *, display_path: str
-) -> tuple[str | None, bytes | None]:
+def _file_bytes(path: Path, cap: int, *, display_path: str) -> tuple[str | None, bytes | None]:
     try:
         size = path.stat().st_size
     except OSError:
@@ -175,7 +173,7 @@ def create_directory(
 
     result = MutationResult(
         workspace=handle.name,
-        path=resolved.relative,
+        path=resolved.reported,
         changed=True,
         before_hash=None,
         after_hash=None,
@@ -183,7 +181,7 @@ def create_directory(
         operation_id=operation_id,
         undo_available=not dry_run and operation_id is not None,
         dry_run=dry_run,
-        items=(OperationItemResult(path=resolved.relative, change="created"),),
+        items=(OperationItemResult(path=resolved.reported, change="created"),),
         affected_count=1,
     )
     if dry_run:
@@ -306,8 +304,7 @@ def move_directory(
             operation_type="move_directory",
             target_path=destination_resolved.relative,
             summary=(
-                f"Moved directory {source_resolved.relative} "
-                f"to {destination_resolved.relative}."
+                f"Moved directory {source_resolved.relative} to {destination_resolved.relative}."
             ),
             items=items,
             actor=actor,
@@ -337,7 +334,7 @@ def move_directory(
 
     result = MutationResult(
         workspace=handle.name,
-        path=destination_resolved.relative,
+        path=destination_resolved.reported,
         changed=True,
         before_hash=None,
         after_hash=None,
@@ -349,8 +346,8 @@ def move_directory(
         undo_available=not dry_run and operation_id is not None and not skipped,
         dry_run=dry_run,
         items=(
-            OperationItemResult(path=source_resolved.relative, change="moved"),
-            OperationItemResult(path=destination_resolved.relative, change="created"),
+            OperationItemResult(path=source_resolved.reported, change="moved"),
+            OperationItemResult(path=destination_resolved.reported, change="created"),
         ),
         affected_count=1,
         history_skipped=tuple(sorted(set(skipped))),
@@ -398,8 +395,7 @@ def delete_directory(
     entries = _count_entries(target)
     if entries > 0 and not recursive:
         raise ValidationError(
-            "directory is not empty; "
-            f"{DIRECTORY_DELETE_HINT.sentence(CallerSurface.CORE)}",
+            f"directory is not empty; {DIRECTORY_DELETE_HINT.sentence(CallerSurface.CORE)}",
             details={"path": resolved.relative, "entries": entries},
             override_hint=DIRECTORY_DELETE_HINT,
         )
@@ -480,7 +476,7 @@ def delete_directory(
 
     result = MutationResult(
         workspace=handle.name,
-        path=resolved.relative,
+        path=resolved.reported,
         changed=True,
         before_hash=None,
         after_hash=None,
@@ -491,7 +487,7 @@ def delete_directory(
         operation_id=operation_id,
         undo_available=not dry_run and operation_id is not None and not skipped,
         dry_run=dry_run,
-        items=(OperationItemResult(path=resolved.relative, change="deleted"),),
+        items=(OperationItemResult(path=resolved.reported, change="deleted"),),
         affected_count=entries,
         warnings=(
             *warnings,

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/operations/edit.py

@@ -51,7 +51,7 @@ from ..models import (
     PatchHunkResult,
     as_dry_run,
 )
-from ..paths import ResolvedPath, ensure_parent_directory, resolve_path
+from ..paths import ResolvedPath, ensure_parent_directory, reported_path, resolve_path
 from ..safety.atomic import atomic_write_bytes, fsync_directory
 from ..safety.hashing import hash_bytes, hashes_match, normalize_hash
 from ..safety.textdata import TextProfile, decode_bytes, encode_text
@@ -106,9 +106,7 @@ def _load(
         expect="file",
         state_dir=handle.state_dir,
     )
-    warnings = _guard_sensitive(
-        handle, resolved, allow_sensitive_write=allow_sensitive_write
-    )
+    warnings = _guard_sensitive(handle, resolved, allow_sensitive_write=allow_sensitive_write)
     try:
         raw = resolved.absolute.read_bytes()
     except OSError as exc:
@@ -220,10 +218,7 @@ def _candidates(text: str, old: str) -> list[dict[str, Any]]:
         ratio = difflib.SequenceMatcher(None, probe, line.text.strip()).ratio()
         scored.append((ratio, index, line.text))
     scored.sort(key=lambda item: item[0], reverse=True)
-    return [
-        {"line": number, "text": content}
-        for _, number, content in scored[:3]
-    ]
+    return [{"line": number, "text": content} for _, number, content in scored[:3]]
 
 
 def _finish(
@@ -241,9 +236,7 @@ def _finish(
     actor: str | None,
     task_id: str | None,
 ) -> MutationResult:
-    data = encode_text(
-        new_text, encoding=loaded.profile.encoding, has_bom=loaded.profile.has_bom
-    )
+    data = encode_text(new_text, encoding=loaded.profile.encoding, has_bom=loaded.profile.has_bom)
     _check_size(handle, data, loaded.resolved.relative)
 
     preview: str | None = None
@@ -337,9 +330,7 @@ def replace_text(
     """
 
     if match not in _MATCH_POLICIES:
-        raise ValidationError(
-            "match policy must be 'first' or 'all'", details={"match": match}
-        )
+        raise ValidationError("match policy must be 'first' or 'all'", details={"match": match})
     if old == "":
         raise ValidationError("replacement target must not be empty")
     if expected_match_count is not None and expected_match_count < 0:
@@ -489,17 +480,11 @@ def replace_lines(
     newline = _newline_for(loaded.profile)
     replacement_lines = _replacement_lines(replacement, newline)
     if replacement_lines and not replacement.endswith(("\n", "\r")):
-        replacement_lines[-1] = DiffLine(
-            replacement_lines[-1].text, lines[end_line - 1].ending
-        )
+        replacement_lines[-1] = DiffLine(replacement_lines[-1].text, lines[end_line - 1].ending)
     new_lines = lines[: start_line - 1] + replacement_lines + lines[end_line:]
     new_text = join_lines(new_lines)
 
-    span = (
-        f"line {start_line}"
-        if start_line == end_line
-        else f"lines {start_line}-{end_line}"
-    )
+    span = f"line {start_line}" if start_line == end_line else f"lines {start_line}-{end_line}"
     return _finish(
         handle,
         loaded,
@@ -620,11 +605,7 @@ def delete_lines(
 
     new_lines = lines[: start_line - 1] + lines[end_line:]
     new_text = join_lines(new_lines)
-    span = (
-        f"line {start_line}"
-        if start_line == end_line
-        else f"lines {start_line}-{end_line}"
-    )
+    span = f"line {start_line}" if start_line == end_line else f"lines {start_line}-{end_line}"
     return _finish(
         handle,
         loaded,
@@ -779,9 +760,7 @@ def _parse_patch(patch_text: str) -> tuple[_FilePatch, ...]:
                             raise _syntax("newline marker has no preceding line", index + 1)
                         body[-1] = dataclasses.replace(body[-1], terminated=False)
                         index += 1
-                    hunks.append(
-                        _Hunk(old_start, old_count, new_start, new_count, tuple(body))
-                    )
+                    hunks.append(_Hunk(old_start, old_count, new_start, new_count, tuple(body)))
                     continue
                 if current.strip() == "" or current.startswith(_METADATA_PREFIXES):
                     index += 1
@@ -922,9 +901,7 @@ def _plan_file(
         for_write=True,
         state_dir=handle.state_dir,
     )
-    warnings = _guard_sensitive(
-        handle, resolved, allow_sensitive_write=allow_sensitive_write
-    )
+    warnings = _guard_sensitive(handle, resolved, allow_sensitive_write=allow_sensitive_write)
 
     newline = ""
     if create:
@@ -979,9 +956,7 @@ def _plan_file(
         after_exists = False
         change: Literal["created", "modified", "deleted", "unchanged"] = "deleted"
     else:
-        after_bytes = encode_text(
-            after_text, encoding=profile_encoding, has_bom=has_bom
-        )
+        after_bytes = encode_text(after_text, encoding=profile_encoding, has_bom=has_bom)
         _check_size(handle, after_bytes, relative)
         after_hash = hash_bytes(after_bytes)
         after_exists = True
@@ -1081,18 +1056,14 @@ def apply_patch(
     """
 
     if not patch_text.strip():
-        raise PatchFailedError(
-            "invalid patch: the patch is empty", details={"reason": "syntax"}
-        )
+        raise PatchFailedError("invalid patch: the patch is empty", details={"reason": "syntax"})
 
     patches = _parse_patch(patch_text)
     seen: set[str] = set()
     plans: list[_Plan] = []
     warnings: list[str] = []
     for patch in patches:
-        plan, plan_warnings = _plan_file(
-            handle, patch, allow_sensitive_write=allow_sensitive_write
-        )
+        plan, plan_warnings = _plan_file(handle, patch, allow_sensitive_write=allow_sensitive_write)
         if plan.relative in seen:
             raise PatchFailedError(
                 f"the patch names {plan.relative} more than once",
@@ -1106,10 +1077,7 @@ def apply_patch(
         plans = [
             dataclasses.replace(
                 plan,
-                hunks=tuple(
-                    dataclasses.replace(hunk, status="would_apply")
-                    for hunk in plan.hunks
-                ),
+                hunks=tuple(dataclasses.replace(hunk, status="would_apply") for hunk in plan.hunks),
             )
             for plan in plans
         ]
@@ -1123,7 +1091,7 @@ def apply_patch(
 
     file_results = tuple(
         PatchFileResult(
-            path=plan.relative,
+            path=reported_path(handle.root, plan.relative),
             change=plan.change,
             applied=True,
             before_hash=plan.before_hash,
@@ -1134,7 +1102,7 @@ def apply_patch(
     )
     item_results = tuple(
         OperationItemResult(
-            path=plan.relative,
+            path=reported_path(handle.root, plan.relative),
             change=plan.change,
             before_hash=plan.before_hash,
             after_hash=plan.after_hash,
@@ -1147,7 +1115,7 @@ def apply_patch(
         return as_dry_run(
             MutationResult(
                 workspace=handle.name,
-                path=primary.relative,
+                path=reported_path(handle.root, primary.relative),
                 changed=True,
                 before_hash=primary.before_hash,
                 after_hash=primary.after_hash,
@@ -1240,7 +1208,7 @@ def apply_patch(
 
     return MutationResult(
         workspace=handle.name,
-        path=primary.relative,
+        path=reported_path(handle.root, primary.relative),
         changed=True,
         before_hash=primary.before_hash,
         after_hash=primary.after_hash,

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/operations/read.py

@@ -28,7 +28,7 @@ from ..models import (
     PathInfo,
     ReadManyEntry,
 )
-from ..paths import resolve_path
+from ..paths import reported_path, resolve_path
 from ..safety.hashing import hash_file
 from ..safety.textdata import (
     BINARY_SAMPLE_SIZE,
@@ -136,7 +136,7 @@ def read_file(
     next_start_line = last + 1 if truncated and last == loaded_line_count else None
 
     return FileReadResult(
-        path=resolved.relative,
+        path=resolved.reported,
         content=content,
         start_line=first,
         end_line=last,
@@ -265,7 +265,7 @@ def list_directory(
                     raise storage_failure("inspect", relative, exc) from exc
                 entries.append(
                     DirectoryEntry(
-                        path=relative,
+                        path=reported_path(handle.root, relative),
                         name=child.name,
                         kind=kind,
                         size_bytes=size_bytes,
@@ -277,7 +277,7 @@ def list_directory(
 
     add(base, 0)
     return DirectoryListing(
-        path=resolved.relative or ".",
+        path=resolved.reported,
         entries=tuple(entries),
         truncated=truncated,
         max_entries=limit,
@@ -305,7 +305,7 @@ def get_file_info(handle: WorkspaceHandle, path: str) -> PathInfo:
         kind = "other"
 
     base = PathInfo(
-        path=resolved.relative,
+        path=resolved.reported,
         kind=kind,
         size_bytes=info.st_size,
         mode=info.st_mode & 0o7777,
@@ -366,5 +366,5 @@ def get_file_hash(handle: WorkspaceHandle, path: str) -> str:
 def file_exists(handle: WorkspaceHandle, path: str) -> ExistsResult:
     resolved = resolve_path(handle.root, path, state_dir=handle.state_dir)
     if not resolved.exists:
-        return ExistsResult(path=resolved.relative, exists=False)
-    return ExistsResult(path=resolved.relative, exists=True, kind=resolved.kind)
+        return ExistsResult(path=resolved.reported, exists=False)
+    return ExistsResult(path=resolved.reported, exists=True, kind=resolved.kind)

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/operations/search.py

@@ -15,7 +15,7 @@ from pathlib import Path
 
 from ..errors import FilekeeperError, InvalidPatternError, ValidationError, storage_failure
 from ..models import ContentMatch, ContentSearchResult, PathSearchResult
-from ..paths import resolve_path
+from ..paths import reported_path, resolve_path
 from ..safety.textdata import decode_bytes, looks_binary
 from ..workspace import WorkspaceHandle
 
@@ -94,10 +94,10 @@ def search_files(
             break
         relative = _relative(handle, candidate)
         if _matches_glob(relative, candidate.name, glob):
-            matches.append(relative)
+            matches.append(reported_path(handle.root, relative))
 
     return PathSearchResult(
-        path=resolved.relative or ".",
+        path=resolved.reported,
         glob=glob,
         paths=tuple(matches),
         truncated=truncated,
@@ -179,7 +179,7 @@ def search_content(
                 column = index + 1
             matches.append(
                 ContentMatch(
-                    path=relative,
+                    path=reported_path(handle.root, relative),
                     line_number=number,
                     line=line,
                     column=column,
@@ -189,7 +189,7 @@ def search_content(
             break
 
     return ContentSearchResult(
-        path=resolved.relative or ".",
+        path=resolved.reported,
         query=query,
         regex=regex,
         matches=tuple(matches),

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/operations/write.py

@@ -39,7 +39,7 @@ from ..errors import (
 )
 from ..history.journal import OperationJournal, PendingItem
 from ..models import MutationResult, OperationItemResult, as_dry_run
-from ..paths import ResolvedPath, ensure_parent_directory, resolve_path
+from ..paths import ResolvedPath, ensure_parent_directory, reported_path, resolve_path
 from ..safety.atomic import atomic_write_bytes, fsync_directory
 from ..safety.hashing import format_digest, hash_bytes, hash_file, hashes_match, normalize_hash
 from ..safety.sensitive import sensitive_reason
@@ -81,9 +81,7 @@ def _default_file_mode() -> int:
     return 0o666 & ~current
 
 
-def _snapshot(
-    path: Path, cap: int, *, display_path: str
-) -> tuple[str | None, bytes | None, bool]:
+def _snapshot(path: Path, cap: int, *, display_path: str) -> tuple[str | None, bytes | None, bool]:
     """Return ``(hash, bytes, exists)`` for a file, without reading oversize content.
 
     ``bytes`` is ``None`` for an oversize file: the hash is still recorded, the
@@ -237,7 +235,7 @@ def _after_write(
 
     result = MutationResult(
         workspace=handle.name,
-        path=resolved.relative,
+        path=resolved.reported,
         changed=True,
         before_hash=before_hash,
         after_hash=after_hash,
@@ -250,7 +248,7 @@ def _after_write(
         warnings=warnings,
         items=(
             OperationItemResult(
-                path=resolved.relative,
+                path=resolved.reported,
                 change=change,  # type: ignore[arg-type]
                 before_hash=before_hash,
                 after_hash=after_hash,
@@ -289,8 +287,7 @@ def create_file(
         raise NotAFileError("path is a directory", details={"path": resolved.relative})
     if resolved.exists and expected_absent:
         raise AlreadyExistsError(
-            "file already exists; "
-            f"{CREATE_OVERWRITE_HINT.sentence(CallerSurface.CORE)}",
+            f"file already exists; {CREATE_OVERWRITE_HINT.sentence(CallerSurface.CORE)}",
             details={"path": resolved.relative},
             override_hint=CREATE_OVERWRITE_HINT,
         )
@@ -448,8 +445,7 @@ def copy_file(
         )
         if expected_destination_hash is None:
             raise AlreadyExistsError(
-                "destination already exists; "
-                f"{COPY_OVERWRITE_HINT.sentence(CallerSurface.CORE)}",
+                f"destination already exists; {COPY_OVERWRITE_HINT.sentence(CallerSurface.CORE)}",
                 details={"path": destination_resolved.relative},
                 override_hint=COPY_OVERWRITE_HINT,
             )
@@ -495,12 +491,10 @@ def copy_file(
         actor=actor,
         task_id=task_id,
         metadata={"source_path": source_resolved.relative},
-        journal_summary=(
-            f"Copied {source_resolved.relative} to {destination_resolved.relative}."
-        ),
+        journal_summary=(f"Copied {source_resolved.relative} to {destination_resolved.relative}."),
     )
     item = OperationItemResult(
-        path=source_resolved.relative,
+        path=reported_path(handle.root, source_resolved.relative),
         change="unchanged",
         before_hash=hash_bytes(data),
         after_hash=hash_bytes(data),
@@ -584,8 +578,7 @@ def move_file(
         )
     if destination_resolved.exists and not overwrite:
         raise AlreadyExistsError(
-            "destination already exists; "
-            f"{MOVE_OVERWRITE_HINT.sentence(CallerSurface.CORE)}",
+            f"destination already exists; {MOVE_OVERWRITE_HINT.sentence(CallerSurface.CORE)}",
             details={"path": destination_resolved.relative},
             override_hint=MOVE_OVERWRITE_HINT,
         )
@@ -608,9 +601,7 @@ def move_file(
             begun = journal.begin(
                 operation_type="move_file",
                 target_path=destination_resolved.relative,
-                summary=(
-                    f"Moved {source_resolved.relative} to {destination_resolved.relative}."
-                ),
+                summary=(f"Moved {source_resolved.relative} to {destination_resolved.relative}."),
                 items=[
                     PendingItem(
                         relative_path=source_resolved.relative,
@@ -677,7 +668,7 @@ def move_file(
 
     result = MutationResult(
         workspace=handle.name,
-        path=destination_resolved.relative,
+        path=destination_resolved.reported,
         changed=True,
         before_hash=source_hash,
         after_hash=after_hash,
@@ -691,13 +682,13 @@ def move_file(
         warnings=warnings,
         items=(
             OperationItemResult(
-                path=source_resolved.relative,
+                path=source_resolved.reported,
                 change="moved",
                 before_hash=source_hash,
                 after_hash=None,
             ),
             OperationItemResult(
-                path=destination_resolved.relative,
+                path=destination_resolved.reported,
                 change="created",
                 before_hash=destination_before,
                 after_hash=after_hash,
@@ -787,8 +778,7 @@ def delete_file(
             if journal is not None and operation_id is not None:
                 journal.abort(operation_id)
             raise StorageError(
-                f"cannot delete file {resolved.relative}: "
-                f"{exc.strerror or type(exc).__name__}",
+                f"cannot delete file {resolved.relative}: {exc.strerror or type(exc).__name__}",
                 details={"path": resolved.relative},
             ) from exc
         fsync_directory(resolved.absolute.parent)
@@ -797,7 +787,7 @@ def delete_file(
 
     result = MutationResult(
         workspace=handle.name,
-        path=resolved.relative,
+        path=resolved.reported,
         changed=True,
         before_hash=before_hash,
         after_hash=None,
@@ -807,7 +797,7 @@ def delete_file(
         dry_run=dry_run,
         items=(
             OperationItemResult(
-                path=resolved.relative,
+                path=resolved.reported,
                 change="deleted",
                 before_hash=before_hash,
                 after_hash=None,

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/paths.py

@@ -47,6 +47,21 @@ from .errors import (
 PathKind = Literal["file", "directory", "symlink", "other"]
 
 
+def reported_path(root: Path, relative: str) -> str:
+    """Render a workspace-relative path for a caller.
+
+    A relative path is the normal form and is returned byte-for-byte. The one
+    exception is a workspace whose root is the filesystem root: there the
+    relative form cannot be told apart from a repository-relative path, so the
+    leading separator is restored and the reported string is the absolute path.
+    The empty relative path is rendered as the root itself.
+    """
+
+    if Path(root) == Path("/"):
+        return "/" + relative
+    return relative or "."
+
+
 @dataclass(frozen=True, slots=True)
 class ResolvedPath:
     """A path proven to be inside the workspace.
@@ -55,14 +70,22 @@ class ResolvedPath:
     operations act on. ``link_path`` is the path as located in the workspace,
     which differs only when the final component is a symlink; metadata calls need
     it to stat the link itself instead of its target.
+
+    ``reported`` is the string a caller sees; ``relative`` stays the form
+    operations and the journal use.
     """
 
+    root: Path
     absolute: Path
     relative: str
     exists: bool
     is_symlink: bool
     link_path: Path
 
+    @property
+    def reported(self) -> str:
+        return reported_path(self.root, self.relative)
+
     @property
     def kind(self) -> PathKind:
         if self.is_symlink:
@@ -202,6 +225,7 @@ def resolve_path(
     reported = link_location if is_link else canonical
     relative = reported.relative_to(root_canonical).as_posix()
     resolved = ResolvedPath(
+        root=root_canonical,
         absolute=canonical,
         relative=relative if relative != "." else "",
         exists=exists and canonical.exists(),

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/service.py

@@ -231,9 +231,7 @@ class FilekeeperService:
         if self.registered and self.handle.workspace_id is not None:
             workspace = self.owner.registry.get(self.handle.name)
             return self.owner.registry.ensure_state_dir(workspace)
-        return ensure_state_directory(
-            self.handle.state_dir, root=self.root, home=self.owner.home
-        )
+        return ensure_state_directory(self.handle.state_dir, root=self.root, home=self.owner.home)
 
     def blob_store(self) -> BlobStore:
         return BlobStore(self.state_database_path().parent / "blobs", fsync=self.config.fsync)

{filekeeper_cli-0.1.7 → filekeeper_cli-0.1.8}/src/filekeeper/core/workspace.py

@@ -69,9 +69,7 @@ def global_workspace_state_dir(workspace_id: str, home: Path) -> Path:
     return Path(home) / "workspaces" / workspace_id
 
 
-def display_state_location(
-    state_dir: Path, *, root: Path | None, home: Path | None
-) -> str:
+def display_state_location(state_dir: Path, *, root: Path | None, home: Path | None) -> str:
     """Render a state directory without naming an absolute host path.
 
     A public error envelope is read by a caller confined to the workspace, so a
@@ -123,9 +121,7 @@ def resolve_state_location(workspace: Workspace, config: Config, home: Path) ->
     if config.state_location_explicit:
         # The operator asked for project-local state; do not redirect it.
         return StateLocation(project_local)
-    return StateLocation(
-        global_workspace_state_dir(workspace.id, home), GLOBAL_FALLBACK_WARNING
-    )
+    return StateLocation(global_workspace_state_dir(workspace.id, home), GLOBAL_FALLBACK_WARNING)
 
 
 def workspace_state_dir(workspace: Workspace, config: Config, home: Path) -> Path:
@@ -434,10 +430,12 @@ class WorkspaceRegistry:
 
 
 def _row_to_workspace(row: sqlite3.Row) -> Workspace:
+    root_path = str(row["root_path"])
     return Workspace(
         id=str(row["id"]),
         name=str(row["name"]),
-        root_path=str(row["root_path"]),
+        root_path=root_path,
         created_at=str(row["created_at"]),
         updated_at=str(row["updated_at"]),
+        root_exists=Path(root_path).is_dir(),
     )