@stylusnexus/work-plan 2026.7.13 → 2026.7.15

package/skills/work-plan/tests/test_github_state.py

@@ -12,7 +12,8 @@ from lib.github_state import (
     fetch_issues, fetch_issue, fetch_issues_concurrent,
     fetch_repo_issues_graphql, fetch_export_issues, _normalize_gql_node,
     extract_priority, fetch_recent_issues, short_milestone,
-    repo_visibility, _VIS_CACHE, fetch_open_issues,
+    repo_visibility, _VIS_CACHE, fetch_open_issues, repo_full_name,
+    gh_auth_status,
     _GQL_FIELDS_LEAN, _GQL_FIELDS_FULL,
     _gql_query, _GQL_ISSUE_DEPS,
 )
@@ -608,5 +609,44 @@ class NormalizeDepsTest(unittest.TestCase):
         self.assertFalse(out["deps_truncated"])
 
 
+class TestRepoFullName(unittest.TestCase):
+    def test_returns_full_name_on_success(self):
+        fake = MagicMock(returncode=0, stdout="org/repo-renamed\n", stderr="")
+        with patch("lib.github_state.subprocess.run", return_value=fake) as m:
+            result = repo_full_name("org/repo-old")
+        self.assertEqual(result, "org/repo-renamed")
+        args = m.call_args[0][0]
+        self.assertEqual(args, ["gh", "api", "repos/org/repo-old", "--jq", ".full_name"])
+
+    def test_none_on_nonzero_exit(self):
+        fake = MagicMock(returncode=1, stdout="", stderr="not found")
+        with patch("lib.github_state.subprocess.run", return_value=fake):
+            self.assertIsNone(repo_full_name("org/gone"))
+
+    def test_none_on_invalid_slug(self):
+        self.assertIsNone(repo_full_name("not-a-slug"))
+
+    def test_none_on_subprocess_exception(self):
+        with patch("lib.github_state.subprocess.run", side_effect=OSError("boom")):
+            self.assertIsNone(repo_full_name("org/repo"))
+
+
+def _gh_authenticated():
+    return gh_auth_status()["authenticated"]
+
+
+class TestRepoFullNameLiveRedirect(unittest.TestCase):
+    def test_musical_family_trees_redirect_is_still_live(self):
+        # The actual repo this whole design's incident was about. If this ever
+        # stops returning the renamed slug, the redirect has lapsed and the
+        # design's core "gh api repos/<slug> follows GitHub's own rename
+        # redirect" assumption needs re-verifying before trusting doctor's
+        # output on real data.
+        if not _gh_authenticated():
+            self.skipTest("gh not authenticated — skipping live network check")
+        result = repo_full_name("evemcgivern/musical-family-trees")
+        self.assertEqual(result, "evemcgivern/soundstellation")
+
+
 if __name__ == "__main__":
     unittest.main()

package/skills/work-plan/tests/test_notes_vcs.py

@@ -188,6 +188,36 @@ class RemoteAndOwnershipTest(unittest.TestCase):
                 self.assertTrue(notes_vcs.is_owned(Path(d)))
 
 
+class DirtyPathsCheckedTest(unittest.TestCase):
+    def test_ok_true_on_clean_tree(self):
+        with tempfile.TemporaryDirectory() as d:
+            with patch.object(notes_vcs, "_git", _FakeGit(porcelain="")):
+                ok, paths = notes_vcs.dirty_paths_checked(Path(d))
+        self.assertTrue(ok)
+        self.assertEqual(paths, set())
+
+    def test_ok_true_with_dirty_paths(self):
+        with tempfile.TemporaryDirectory() as d:
+            with patch.object(notes_vcs, "_git",
+                                _FakeGit(porcelain=" M foo.md\n?? bar.md\n")):
+                ok, paths = notes_vcs.dirty_paths_checked(Path(d))
+        self.assertTrue(ok)
+        self.assertEqual(paths, {"foo.md", "bar.md"})
+
+    def test_ok_false_on_git_call_failure(self):
+        with tempfile.TemporaryDirectory() as d:
+            with patch.object(notes_vcs, "_git", _FakeGit(missing=True)):
+                ok, paths = notes_vcs.dirty_paths_checked(Path(d))
+        self.assertFalse(ok)
+        self.assertEqual(paths, set())
+
+    def test_dirty_paths_thin_wrapper_matches_checked(self):
+        with tempfile.TemporaryDirectory() as d:
+            with patch.object(notes_vcs, "_git",
+                                _FakeGit(porcelain=" M foo.md\n")):
+                self.assertEqual(notes_vcs.dirty_paths(Path(d)), {"foo.md"})
+
+
 class DirtyPathsTest(unittest.TestCase):
     def test_parses_porcelain_into_path_set(self):
         body = " M alpha.md\n?? beta.md\n D gone.md\n"

package/skills/work-plan/tests/test_tracks.py

@@ -9,7 +9,7 @@ from unittest.mock import patch
 SKILL_ROOT = Path(__file__).resolve().parents[1]
 sys.path.insert(0, str(SKILL_ROOT))
 
-from lib.tracks import discover_tracks, discover_archived_tracks
+from lib.tracks import discover_tracks, discover_archived_tracks, iter_private_track_paths
 
 FIXTURES = Path(__file__).parent / "fixtures" / "notes_root"
 
@@ -463,5 +463,42 @@ class DiscoverArchivedSharedTest(unittest.TestCase):
             self.assertNotIn("WARN:", stderr_buf.getvalue())
 
 
+class TestIterPrivateTrackPaths(unittest.TestCase):
+    def _make_tree(self, root: Path):
+        (root / "proj-a").mkdir(parents=True)
+        (root / "proj-a" / "track-one.md").write_text("---\n---\nbody")
+        (root / "proj-a" / "_draft.md").write_text("---\n---\nbody")
+        (root / "proj-a" / "archive").mkdir()
+        (root / "proj-a" / "archive" / "old-track.md").write_text("---\n---\nbody")
+        (root / "proj-a" / ".hidden.md").write_text("---\n---\nbody")
+
+    def test_excludes_archive_by_default(self):
+        with tempfile.TemporaryDirectory() as tmp:
+            root = Path(tmp)
+            self._make_tree(root)
+            paths = iter_private_track_paths(root, include_archive=False)
+            names = {p.name for p in paths}
+            self.assertEqual(names, {"track-one.md"})
+
+    def test_includes_archive_when_requested(self):
+        with tempfile.TemporaryDirectory() as tmp:
+            root = Path(tmp)
+            self._make_tree(root)
+            paths = iter_private_track_paths(root, include_archive=True)
+            names = {p.name for p in paths}
+            self.assertEqual(names, {"track-one.md", "old-track.md"})
+
+    def test_skips_dotfile_underscore_dash_prefixed_names_in_both_modes(self):
+        with tempfile.TemporaryDirectory() as tmp:
+            root = Path(tmp)
+            self._make_tree(root)
+            (root / "proj-a" / "-dash.md").write_text("---\n---\nbody")
+            for include_archive in (False, True):
+                names = {p.name for p in iter_private_track_paths(root, include_archive)}
+                self.assertNotIn("_draft.md", names)
+                self.assertNotIn(".hidden.md", names)
+                self.assertNotIn("-dash.md", names)
+
+
 if __name__ == "__main__":
     unittest.main()

package/skills/work-plan/work_plan.py

@@ -61,6 +61,7 @@ SUBCOMMANDS = {
     "in-progress": "commands.in_progress",
     "export": "commands.export",
     "which-repo": "commands.which_repo",
+    "doctor": "commands.doctor",
     "auth-status": "commands.auth_status",
     "list-open-issues": "commands.list_open_issues",
     "set": "commands.set_field",
@@ -259,6 +260,17 @@ DESCRIPTIONS = [
      "Resolve the current directory to one configured repo — by local clone path first, then the git `origin` remote. Prints the matched config key + GitHub slug, or reports no match. Read-only. Underlies `brief` cwd auto-scope and the VS Code viewer's repo auto-focus.",
      "Rarely run by hand — it's the shared resolver the viewer and `brief` call. Useful to confirm which repo a checkout maps to.",
      "/work-plan which-repo --json"),
+    ("doctor", "[--json] [--fix]",
+     "Detect drift between config.yml, local git clones, GitHub, and notes_root track "
+     "frontmatter — a renamed local folder or GitHub repo that config.yml no longer "
+     "matches, a non-git local path, duplicate entries, an invalid/missing notes_root, "
+     "an orphaned notes folder, or a stale per-track github.repo. --fix corrects only "
+     "the two mechanically-safe cases (a GitHub-confirmed rename, a stale track slug) "
+     "and always re-scans afterward before deciding success.",
+     "Run right after renaming a project's local folder or its GitHub repo — this is "
+     "exactly the class of bug that silently breaks the VS Code viewer's Auto Focus "
+     "Repo setting with zero visible signal.",
+     "/work-plan doctor --fix"),
 ]
 
 
@@ -352,6 +364,14 @@ _READONLY_SUBCOMMANDS = frozenset({
     # plan-branch manages its OWN commits on the plan branch (init seeds +
     # commits the skeleton itself); the auto-commit hooks must not also fire.
     "plan-branch",
+    # doctor writes config.yml (unversioned, outside any repo) and private
+    # notes_root/ frontmatter under --fix, but never a repo's shared
+    # .work-plan/ tier — so it's read-only from the dispatcher's perspective
+    # too, sidestepping _shared_precommit_state's plan_worktree.ensure_worktree
+    # side effect. doctor --fix manages its own notes-vcs commit directly
+    # (see commands/doctor.py's dirty-file policy), same self-managed-commit
+    # precedent as plan-branch.
+    "doctor",
 })