devlaunch 0.0.21__tar.gz → 0.0.22__tar.gz

{devlaunch-0.0.21 → devlaunch-0.0.22}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devlaunch
-Version: 0.0.21
+Version: 0.0.22
 Summary: DevLaunch - A streamlined CLI for devpod workspaces
 Project-URL: Source, https://github.com/blooop/devlaunch
 Project-URL: Home, https://github.com/blooop/devlaunch
@@ -427,13 +427,19 @@ cleanup tool must not ignore — `unsaved`:
   "repo": "blooop/devlaunch",
   "branch": "wayfinder/devlaunch-80",
   "state": "Stopped",
-  "unsaved": "2 uncommitted change(s) and 1 unpushed commit(s)"
+  "unsaved": "2 uncommitted change(s) (pixi.lock, notes.md) and 1 unpushed commit(s)"
 }
 ```
 
 `unsaved` is a description of what deleting would destroy, or `null` when the
 clone holds nothing that does not also exist on a remote — uncommitted changes
-(untracked files included) and commits no remote has. A workspace `dl` did not
+(untracked files included) and commits no remote has. The changed paths are
+named, not just counted, and that matters more than it looks: a devcontainer
+that runs a package install in its `postCreateCommand` can leave a tracked
+lockfile modified in *every* workspace it builds — this repo's own does — and as
+a bare count that is indistinguishable from an hour of unsaved work. A cleanup
+tool believing the count would then never clean anything. Named, it is
+judgeable. A workspace `dl` did not
 create reports `devlaunch: false` with no repo, branch or `unsaved`: there is no
 clone of `dl`'s to protect, and it has no business inspecting your checkout.
 

{devlaunch-0.0.21 → devlaunch-0.0.22}/README.md

@@ -405,13 +405,19 @@ cleanup tool must not ignore — `unsaved`:
   "repo": "blooop/devlaunch",
   "branch": "wayfinder/devlaunch-80",
   "state": "Stopped",
-  "unsaved": "2 uncommitted change(s) and 1 unpushed commit(s)"
+  "unsaved": "2 uncommitted change(s) (pixi.lock, notes.md) and 1 unpushed commit(s)"
 }
 ```
 
 `unsaved` is a description of what deleting would destroy, or `null` when the
 clone holds nothing that does not also exist on a remote — uncommitted changes
-(untracked files included) and commits no remote has. A workspace `dl` did not
+(untracked files included) and commits no remote has. The changed paths are
+named, not just counted, and that matters more than it looks: a devcontainer
+that runs a package install in its `postCreateCommand` can leave a tracked
+lockfile modified in *every* workspace it builds — this repo's own does — and as
+a bare count that is indistinguishable from an hour of unsaved work. A cleanup
+tool believing the count would then never clean anything. Named, it is
+judgeable. A workspace `dl` did not
 create reports `devlaunch: false` with no repo, branch or `unsaved`: there is no
 clone of `dl`'s to protect, and it has no business inspecting your checkout.
 

{devlaunch-0.0.21 → devlaunch-0.0.22}/devlaunch/workspace_state.py

@@ -56,6 +56,13 @@ def _git(repo: Path, *args: str) -> Optional[str]:
     never be reported as *safe to delete* on the strength of a failed command —
     every caller here treats ``None`` as "no information", and the one place
     that matters (:func:`holds_unsaved_work`) fails safe explicitly.
+
+    Only trailing newlines are trimmed, never leading whitespace. A full
+    ``strip()`` here was wrong in a way that took real use to notice: the first
+    line of ``git status --porcelain`` for a *modified tracked* file begins with
+    a space (`` M pixi.lock``), so stripping ate the status column and
+    :func:`_name_a_few` then reported ``ixi.lock``. Untracked entries start
+    ``??`` and were unharmed, which is exactly why the tests missed it.
     """
     try:
         result = subprocess.run(
@@ -67,7 +74,7 @@ def _git(repo: Path, *args: str) -> Optional[str]:
     if result.returncode != 0:
         logger.debug(f"git {' '.join(args)} in {repo}: {result.stderr.strip()}")
         return None
-    return result.stdout.strip()
+    return result.stdout.rstrip("\n")
 
 
 def read_clone(clone: Path) -> CloneState:
@@ -84,6 +91,26 @@ def read_clone(clone: Path) -> CloneState:
     return CloneState(branch=branch or None, unsaved=_unsaved(clone, branch))
 
 
+def _name_a_few(changed: List[str], limit: int = 3) -> str:
+    """Name the first few changed paths from `git status --porcelain` lines.
+
+    A count alone is not enough to decide anything with. A devcontainer that
+    runs `pixi install` in its `postCreateCommand` leaves the tracked lockfile
+    modified in *every* workspace it builds, so "1 uncommitted change(s)" is the
+    permanent state of an otherwise untouched clone — and a person told only the
+    count has no way to tell that from an hour of unsaved work. Told the name,
+    they can.
+
+    The porcelain format is two status characters, a space, then the path, so
+    the path starts at offset 3; a rename reads `old -> new`, and the whole
+    field is kept rather than split, because both halves are the news.
+    """
+    names = [line[3:].strip() for line in changed[:limit] if len(line) > 3]
+    if len(changed) > limit:
+        names.append("…")
+    return ", ".join(names)
+
+
 def _unsaved(clone: Path, branch: Optional[str]) -> Optional[str]:
     """What deleting *clone* would destroy, in words, or ``None`` if nothing.
 
@@ -91,7 +118,8 @@ def _unsaved(clone: Path, branch: Optional[str]) -> Optional[str]:
     force a delete wants both:
 
     - a dirty tree, **untracked files included** — an agent's scratch notes are
-      not less lost for never having been added;
+      not less lost for never having been added — with the first few paths
+      named, because a count alone cannot be judged (see :func:`_name_a_few`);
     - commits no remote-tracking ref contains. ``--not --remotes`` asks about
       *any* remote ref rather than this branch's upstream, so work that was
       pushed under another name, or merged and fetched back, is correctly not
@@ -100,7 +128,8 @@ def _unsaved(clone: Path, branch: Optional[str]) -> Optional[str]:
     losses: List[str] = []
     status = _git(clone, "status", "--porcelain")
     if status:
-        losses.append(f"{len(status.splitlines())} uncommitted change(s)")
+        changed = status.splitlines()
+        losses.append(f"{len(changed)} uncommitted change(s) ({_name_a_few(changed)})")
     if branch:
         # Argument order is load-bearing: `--not` flips the sense of every ref
         # *after* it, so the branch has to be named before it. `log --not

{devlaunch-0.0.21 → devlaunch-0.0.22}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "devlaunch"
-version = "0.0.21"
+version = "0.0.22"
 authors = [{ name = "Austin Gregg-Smith", email = "blooop@gmail.com" }]
 description = "DevLaunch - A streamlined CLI for devpod workspaces"
 readme = "README.md"