@stylusnexus/work-plan 2026.6.13-2 → 2026.6.14

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

@@ -106,5 +106,53 @@ class RenderTrackRowTest(unittest.TestCase):
         self.assertNotIn("(P3, open, ", row)
 
 
+class RenderInProgressTest(unittest.TestCase):
+    def _block(self, next_up):
+        return {
+            "name": "alpha", "operational_status": "active", "launch_priority": "P2",
+            "milestone_alignment": "—", "last_touched_label": "1d ago",
+            "last_handoff_label": "1d ago", "next_up": next_up,
+            "next_up_stale_closed_count": 0, "track_slug": "alpha",
+            "active_branches": [], "new_issues": [], "blockers": [],
+            "drift_items": [], "closure_ready": False, "closure_signals_summary": None,
+        }
+
+    def test_in_progress_item_marked(self):
+        row = render_track_row(self._block([
+            {"number": 271, "title": "x", "priority": "P1", "state": "open",
+             "milestone": None, "in_progress": True},
+        ]))
+        self.assertIn("in-progress", row)
+        self.assertIn("#271", row)
+
+    def test_non_in_progress_item_not_marked(self):
+        row = render_track_row(self._block([
+            {"number": 9, "title": "y", "priority": "P2", "state": "open",
+             "milestone": None, "in_progress": False},
+        ]))
+        self.assertNotIn("in-progress", row)
+
+
+class RenderBlockedByTest(unittest.TestCase):
+    def _block(self, item):
+        return {"name": "alpha", "operational_status": "active", "launch_priority": "P2",
+                "milestone_alignment": "—", "last_touched_label": "1d", "last_handoff_label": "1d",
+                "next_up": [item], "next_up_stale_closed_count": 0, "track_slug": "alpha",
+                "active_branches": [], "new_issues": [], "blockers": [],
+                "drift_items": [], "closure_ready": False, "closure_signals_summary": None}
+
+    def _item(self, blocked):
+        return {"number": 5, "title": "x", "priority": "P1", "state": "open",
+                "milestone": None, "in_progress": False, "blocked_by_display": blocked}
+
+    def test_blocked_by_annotation_rendered(self):
+        row = render_track_row(self._block(self._item(["#9"])))
+        self.assertIn("blocked by #9", row)
+
+    def test_no_annotation_when_empty(self):
+        row = render_track_row(self._block(self._item([])))
+        self.assertNotIn("blocked by", row)
+
+
 if __name__ == "__main__":
     unittest.main()

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

@@ -513,5 +513,85 @@ class OrientRepoFlagTest(unittest.TestCase):
         self.assertIn("ambiguous", out.lower())
 
 
+class OrientInProgressTest(unittest.TestCase):
+    def test_next_pick_marked_in_progress(self):
+        from types import SimpleNamespace
+        track = SimpleNamespace(
+            name="alpha", repo="o/r", local_path=Path("/repo"), path=Path("/n/alpha.md"),
+            body="", meta={"track": "alpha", "launch_priority": "P1",
+                           "milestone_alignment": "—",
+                           "github": {"issues": [271]}, "next_up": [271]})
+        issue = {"number": 271, "title": "x", "state": "open", "labels": [], "milestone": None}
+        with mock.patch("commands.where_was_i.fetch_issues", return_value=[issue]), \
+             mock.patch("commands.where_was_i.hot_issue_numbers", return_value={271}), \
+             mock.patch("commands.where_was_i.current_branch", return_value=None), \
+             mock.patch("commands.where_was_i.find_new_issues_for_tracks", return_value={}):
+            out = io.StringIO()
+            with redirect_stdout(out):
+                where_was_i._orient_track(track)
+        self.assertIn("in-progress", out.getvalue())
+
+
+class OrientBlockedByTest(unittest.TestCase):
+    def test_next_pick_shows_blocked_by(self):
+        from types import SimpleNamespace
+        track = SimpleNamespace(name="alpha", repo="o/r", local_path=None,
+                                path=Path("/n/alpha.md"), body="",
+                                meta={"track": "alpha", "launch_priority": "P1",
+                                      "milestone_alignment": "—",
+                                      "github": {"issues": [5]}, "next_up": [5], "blockers": []})
+        issue = {"number": 5, "title": "x", "state": "open", "labels": [],
+                 "blocked_by": [{"number": 9, "repo": "o/r", "title": "dep"}], "blocking": []}
+        with mock.patch("commands.where_was_i.fetch_issues", return_value=[issue]), \
+             mock.patch("commands.where_was_i.hot_issue_numbers", return_value=set()), \
+             mock.patch("commands.where_was_i.current_branch", return_value=None), \
+             mock.patch("commands.where_was_i.find_new_issues_for_tracks", return_value={}):
+            out = io.StringIO()
+            with redirect_stdout(out):
+                from commands import where_was_i
+                where_was_i._orient_track(track)
+        self.assertIn("blocked by #9", out.getvalue())
+
+    def _orient_with_blocked_by(self, blocked_by, blockers):
+        from types import SimpleNamespace
+        track = SimpleNamespace(name="alpha", repo="o/r", local_path=None,
+                                path=Path("/n/alpha.md"), body="",
+                                meta={"track": "alpha", "launch_priority": "P1",
+                                      "milestone_alignment": "—",
+                                      "github": {"issues": [5]}, "next_up": [5],
+                                      "blockers": blockers})
+        issue = {"number": 5, "title": "x", "state": "open", "labels": [],
+                 "blocked_by": blocked_by, "blocking": []}
+        with mock.patch("commands.where_was_i.fetch_issues", return_value=[issue]), \
+             mock.patch("commands.where_was_i.hot_issue_numbers", return_value=set()), \
+             mock.patch("commands.where_was_i.current_branch", return_value=None), \
+             mock.patch("commands.where_was_i.find_new_issues_for_tracks", return_value={}):
+            out = io.StringIO()
+            with redirect_stdout(out):
+                from commands import where_was_i
+                where_was_i._orient_track(track)
+        return out.getvalue()
+
+    def test_cross_repo_blocked_by_shows_qualified_ref(self):
+        # A cross-repo edge renders owner/repo#N, not a bare #N.
+        out = self._orient_with_blocked_by(
+            [{"number": 9, "repo": "other/repo", "title": "dep"}], [])
+        self.assertIn("blocked by other/repo#9", out)
+
+    def test_same_repo_edge_in_manual_blockers_is_suppressed(self):
+        # A same-repo edge whose number is a manual blocker is owned by the
+        # "Blocker:" line, so it is not re-annotated.
+        out = self._orient_with_blocked_by(
+            [{"number": 9, "repo": "o/r", "title": "dep"}], [9])
+        self.assertNotIn("blocked by", out)
+
+    def test_cross_repo_edge_not_suppressed_by_same_number_blocker(self):
+        # A manual blocker #9 means o/r#9; a cross-repo other/repo#9 is a
+        # different issue and must still be annotated.
+        out = self._orient_with_blocked_by(
+            [{"number": 9, "repo": "other/repo", "title": "dep"}], [9])
+        self.assertIn("blocked by other/repo#9", out)
+
+
 if __name__ == "__main__":
     unittest.main()

package/skills/work-plan/work_plan.py

@@ -55,6 +55,7 @@ SUBCOMMANDS = {
     "plan-ack": "commands.plan_ack",
     "plan-baseline": "commands.plan_baseline",
     "close-issue": "commands.close_issue",
+    "in-progress": "commands.in_progress",
     "export": "commands.export",
     "auth-status": "commands.auth_status",
     "list-open-issues": "commands.list_open_issues",
@@ -190,9 +191,13 @@ DESCRIPTIONS = [
      "When you want a tripwire on a plan you believe is done: stamp its baseline, and get alerted if it later regresses.",
      "/work-plan plan-baseline --repo=myproject -- docs/superpowers/plans/2026-03-16-idea-mode-ui.md"),
     ("close-issue", "--repo=<key|slug> [--reason=completed|not_planned] [--comment=<text>] -- <number>",
-     "⚠️ The toolkit's ONLY GitHub-mutating command — closes a GitHub issue via `gh issue close` (everything else is read-only on GitHub). PRs merged to `dev` don't auto-close issues (GitHub auto-closes only from the default branch), so done-but-OPEN issues pile up; this closes one. `--reason` maps to GitHub's completed/not-planned; `--comment` posts a closing note. `--repo` takes a config key or an org/repo slug. The VS Code viewer gates this behind a mandatory 'Close on GitHub?' modal on every close.",
+     "⚠️ A GitHub-mutating command (others: `in-progress`, `plan-status --issues`) — closes a GitHub issue via `gh issue close` (most of the toolkit is read-only on GitHub). PRs merged to `dev` don't auto-close issues (GitHub auto-closes only from the default branch), so done-but-OPEN issues pile up; this closes one. `--reason` maps to GitHub's completed/not-planned; `--comment` posts a closing note. `--repo` takes a config key or an org/repo slug. The VS Code viewer gates this behind a mandatory 'Close on GitHub?' modal on every close.",
      "When an issue is actually done but stayed open because its PR merged to dev, not main — close it without leaving the editor.",
      "/work-plan close-issue --repo=stylusnexus/work-plan-toolkit --reason=completed --comment='Closed via dev merge' -- 287"),
+    ("in-progress", "<n> [--clear] [--repo=<key|slug>] [--confirm=<token>]",
+     "Mark a tracked GitHub issue as in-progress by adding the `work-plan:in-progress` label (or remove it with --clear). Repo-scoped: resolves <n> to the one tracked repo that lists it, or pass --repo to disambiguate. The label is auto-created. Writes into a PUBLIC repo only with a confirm token (prints {needs_confirm, token} otherwise — the VS Code viewer surfaces it as a modal). brief/orient/the viewer also derive in-progress for free from a hot feat/<n>- or fix/<n>- branch.",
+     "When you start actively working an issue that has no hot branch yet (a hot branch is detected automatically), or to clear the flag when you stop.",
+     "/work-plan in-progress 271"),
     ("set-notes-root", "<path>",
      "Update notes_root in ~/.claude/work-plan/config.yml to an absolute path. Creates the target directory if absent. Prints a WARN if existing frontmatter'd tracks live at the old location (they won't be moved — manual migration required). Non-interactive: safe to call from a GUI or script.",
      "VS Code viewer cold-start: user has picked a folder for their private track notes and the extension invokes this to persist the choice. Also useful on the CLI to relocate notes without hand-editing config.yml.",