@stylusnexus/work-plan 2026.6.15 → 2026.6.16

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

@@ -197,5 +197,100 @@ class SetNextUpTest(unittest.TestCase):
         self.assertIn("--order is ignored", err.getvalue())
 
 
+class SetNextUpAutoFlagTest(unittest.TestCase):
+    """Tests for --auto=on|off flag on set-next-up."""
+
+    def _drive_with_stderr(self, args, vis="PRIVATE", cfg=None, track=None):
+        """Like _drive but also captures stderr."""
+        base_cfg = {"notes_root": "/tmp"}
+        if cfg is not None:
+            base_cfg.update(cfg)
+        t = track if track is not None else _t()
+        with patch("commands.set_next_up.load_config", return_value=base_cfg), \
+             patch("commands.set_next_up.discover_tracks", return_value=[t]), \
+             patch("lib.write_guard.repo_visibility", return_value=vis), \
+             patch("commands.set_next_up.write_file") as mw:
+            out, err = io.StringIO(), io.StringIO()
+            with redirect_stdout(out), redirect_stderr(err):
+                rc = set_next_up.run(args)
+        return rc, mw, out.getvalue(), err.getvalue()
+
+    def test_auto_on_writes_next_up_auto_true(self):
+        """--auto=on sets next_up_auto: True in track meta."""
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=on"])
+        self.assertEqual(rc, 0)
+        mw.assert_called_once()
+        meta = mw.call_args[0][1]
+        self.assertTrue(meta.get("next_up_auto"))
+
+    def test_auto_off_removes_next_up_auto(self):
+        """--auto=off removes next_up_auto from track meta."""
+        t = _t(meta={"status": "active", "next_up_auto": True})
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=off"], track=t)
+        self.assertEqual(rc, 0)
+        mw.assert_called_once()
+        meta = mw.call_args[0][1]
+        self.assertNotIn("next_up_auto", meta)
+
+    def test_auto_off_on_track_without_key_still_succeeds(self):
+        """--auto=off on a track with no next_up_auto key still writes ok (no KeyError)."""
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=off"])
+        self.assertEqual(rc, 0)
+        mw.assert_called_once()
+        meta = mw.call_args[0][1]
+        self.assertNotIn("next_up_auto", meta)
+
+    def test_auto_bogus_returns_rc2_no_write(self):
+        """--auto=bogus → rc=2 and no write."""
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=bogus"])
+        self.assertEqual(rc, 2)
+        mw.assert_not_called()
+
+    def test_auto_standalone_private_writes(self):
+        """--auto=on alone (no --preset/--order/--clear) is accepted on private repo."""
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=on"])
+        self.assertEqual(rc, 0)
+        mw.assert_called_once()
+
+    def test_auto_standalone_public_needs_confirm(self):
+        """--auto=on alone on a PUBLIC repo → needs_confirm, no write."""
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=on"], vis="PUBLIC")
+        self.assertEqual(rc, 0)
+        mw.assert_not_called()
+        self.assertIn("needs_confirm", out)
+
+    def test_auto_standalone_public_with_valid_confirm_writes(self):
+        """--auto=on alone on PUBLIC repo with valid --confirm token proceeds to write."""
+        tok = make_token("o/r", "ph")
+        rc, mw, out, err = self._drive_with_stderr(
+            ["ph", "--auto=on", f"--confirm={tok}"], vis="PUBLIC"
+        )
+        self.assertEqual(rc, 0)
+        mw.assert_called_once()
+        meta = mw.call_args[0][1]
+        self.assertTrue(meta.get("next_up_auto"))
+
+    def test_auto_on_combined_with_preset_writes_both(self):
+        """--auto=on --preset=backlog sets BOTH next_up_auto AND next_up_order in one write."""
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=on", "--preset=backlog"])
+        self.assertEqual(rc, 0)
+        mw.assert_called_once()
+        meta = mw.call_args[0][1]
+        self.assertTrue(meta.get("next_up_auto"))
+        self.assertEqual(meta.get("next_up_order"), {"preset": "backlog"})
+
+    def test_auto_on_prints_success_message(self):
+        """--auto=on prints a clear success line."""
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=on"])
+        self.assertIn("next_up_auto", out)
+        self.assertIn("true", out.lower())
+
+    def test_auto_off_prints_success_message(self):
+        """--auto=off prints a clear success line."""
+        t = _t(meta={"status": "active", "next_up_auto": True})
+        rc, mw, out, err = self._drive_with_stderr(["ph", "--auto=off"], track=t)
+        self.assertIn("next_up_auto", out)
+
+
 if __name__ == "__main__":
     unittest.main()

package/skills/work-plan/work_plan.py

@@ -47,6 +47,7 @@ SUBCOMMANDS = {
     "duplicates": "commands.duplicates",
     "coverage": "commands.coverage",
     "canonicalize": "commands.canonicalize",
+    "dedupe-tiers": "commands.dedupe_tiers",
     "hygiene": "commands.hygiene",
     "--hygiene": "commands.hygiene",      # flag-style alias
     "plan-status": "commands.plan_status",
@@ -57,6 +58,7 @@ SUBCOMMANDS = {
     "close-issue": "commands.close_issue",
     "in-progress": "commands.in_progress",
     "export": "commands.export",
+    "which-repo": "commands.which_repo",
     "auth-status": "commands.auth_status",
     "list-open-issues": "commands.list_open_issues",
     "set": "commands.set_field",
@@ -147,8 +149,12 @@ DESCRIPTIONS = [
      "Insert a canonical master issue table at the top of a track. The table has a Milestone column and is ordered active-milestone-first (the track's milestone_alignment milestone, then other milestones grouped with a blank divider row, then no-milestone last) so near-term work sits above someday work (#101). Refresh-md then targets ONLY this table, re-deriving it (so the order self-heals) and leaving narrative tables alone. Use --repo=<key> or track@repo to disambiguate; with --all, --repo=<key> scopes to one repo.",
      "ONE-TIME for hand-written tracks with multiple narrative tables, OR after restructuring a track.",
      "/work-plan canonicalize ux-redesign"),
+    ("dedupe-tiers", "[--repo=<key>] [--apply]",
+     "Remove private track copies that a shared twin in a repo's .work-plan/ supersedes (#359). When a track is promoted to the shared tier, the private original under notes_root is sometimes left behind (bulk/manual promotion, or a failed unlink) — discover_tracks then warns 'exists in both shared and private' on every run with no cleanup path. This removes the safe orphans and REFUSES any whose private copy references issue numbers the shared one lacks (no silent data loss; the invariant is issue_refs(private) ⊆ issue_refs(shared)). Covers active and archived tiers. Default is a dry-run report; --apply deletes (auto-committed to notes_root, so undoable).",
+     "When `exists in both shared and private` warnings appear, or after a bulk promote that left private originals behind.",
+     "/work-plan dedupe-tiers --repo=critforge --apply"),
     ("hygiene", "[--yes] [--no-duplicates] [--repo=<key>] [--timeout=N]",
-     "Weekly cleanup wrapper: refresh-md + reconcile + duplicates. With --repo=<key>, steps 1 and 2 scope to that repo; the duplicates step (a global similarity scan) is skipped. --timeout=N sets the gh subprocess timeout for the duplicates step (default 30s).",
+     "Weekly cleanup wrapper: refresh-md + reconcile + dedupe-tiers (report-only) + duplicates. With --repo=<key>, steps 1–3 scope to that repo; the duplicates step (a global similarity scan) is skipped. --timeout=N sets the gh subprocess timeout for the duplicates step (default 30s).",
      "WEEKLY — runs all three hygiene commands in sequence so you don't have to remember each. Use --repo=<key> to clean up one project without touching the others.",
      "/work-plan hygiene --repo=myproject"),
     ("export", "--json",
@@ -167,10 +173,10 @@ DESCRIPTIONS = [
      "Guarded edit of a track's frontmatter fields (status, launch_priority, milestone_alignment, blockers, next_up). Validates field names + status values; blockers/next_up take comma-separated issue numbers. Setting `next_up` here writes ONLY the frontmatter field — for next_up plus a session-log entry (and a body refresh), use `handoff --set-next` instead. Writes into a PUBLIC repo only with a confirm token: without one it prints {needs_confirm, reason, token} and makes no change (the VS Code viewer surfaces that as a modal, then re-invokes with --confirm=<token>).",
      "Programmatic/GUI edits that have no dedicated verb — e.g. the VS Code extension changing a status or blockers list. On the terminal you'll usually use the named verbs instead.",
      "/work-plan set ux-redesign status=parked"),
-    ("set-next-up", "<track | track@repo> (--preset=<name> | --order=a,b,c | --clear) [--repo=<key>] [--confirm=<token>]",
-     "Configure the ranking preset for a track's auto next_up suggestion. --preset sets one of the named presets (flow, priority-driven, backlog) or 'custom' (which requires --order). --order=a,b,c sets a custom comma-separated criterion list (milestone, dependency, priority, recency, aging). --clear reverts to the global or default preset. Writes next_up_order into the track's frontmatter (does NOT touch the next_up issue list). Public-repo gated: without --confirm it prints {needs_confirm, reason, token} and makes no change.",
-     "When you want a track to use a different ranking order than the default (flow). Use priority-driven for pure backlog work with no milestones, backlog to surface oldest stalled issues first.",
-     "/work-plan set-next-up my-track --preset=priority-driven"),
+    ("set-next-up", "<track | track@repo> (--preset=<name> | --order=a,b,c | --clear | --auto=on|off) [--repo=<key>] [--confirm=<token>]",
+     "Configure the ranking preset and/or auto-derivation flag for a track's next_up list. --preset sets one of the named presets (flow, priority-driven, backlog) or 'custom' (which requires --order). --order=a,b,c sets a custom comma-separated criterion list (milestone, dependency, priority, recency, aging). --clear reverts to the global or default preset. --auto=on activates auto-derivation (brief/orient/export derive next-up live via the ranking preset instead of the curated list); --auto=off reverts to the curated list. --auto can be used standalone or combined with --preset/--order/--clear. Writes next_up_order and/or next_up_auto into the track's frontmatter (does NOT touch the next_up issue list). Public-repo gated: without --confirm it prints {needs_confirm, reason, token} and makes no change.",
+     "When you want a track to use a different ranking order than the default (flow), or to activate auto-derivation so the viewer always shows a freshly-ranked list. Use priority-driven for pure backlog work with no milestones, backlog to surface oldest stalled issues first.",
+     "/work-plan set-next-up my-track --preset=priority-driven --auto=on"),
     ("new-track", "<repo> <slug> [--priority=P0..P3] [--milestone=<m>] [--private] [--confirm=<token>]",
      "Create a brand-new track file under notes_root in one headless call. <repo> is either a configured key (e.g. 'myproject') or a bare org/repo slug (e.g. 'your-org/myproject'). Writes frontmatter with status=active and optional priority/milestone. Gates on public repos — prints {needs_confirm, token} and exits cleanly; re-run with --confirm=<token> to proceed.",
      "When a new feature branch or initiative starts and you want the track file created immediately — especially from a non-terminal caller like the VS Code extension that can't interactively run init.",
@@ -219,6 +225,10 @@ DESCRIPTIONS = [
      "Promote a PRIVATE track (local-only, in notes_root) to the repo's SHARED tier and publish it (#306). Moves the track's `.md` into the repo's `.work-plan/` (on its `plan_branch`, via a worktree), removes the private copy so it isn't duplicated, commits to the plan branch, and pushes — unless `--no-push` (keeps it local). The tier is derived from location, so this is a file move, not a frontmatter edit. Requires the repo to have a local clone + a `plan_branch` (else hints `plan-branch init`). Pushing to a PUBLIC repo makes the track world-visible, so the push is confirm-token gated (prints `needs_confirm` + token; re-run with `--confirm=<token>`).",
      "When a private track is ready to share with teammates — promote it to the shared plan branch in one step instead of hand-moving the file.",
      "/work-plan push-track my-feature --repo=myproject"),
+    ("which-repo", "[--json]",
+     "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"),
 ]
 
 

package/scripts/npm-check-deps.js

@@ -1,44 +0,0 @@
-#!/usr/bin/env node
-// Postinstall check for the @stylusnexus/work-plan npm package.
-//
-// The work-plan CLI is pure Python and shells out to three external tools at
-// runtime. npm can't install them (they're not npm packages), so we just check
-// they're on PATH and print a friendly heads-up if any are missing. This NEVER
-// fails the install — a missing tool is the user's to fix, and the CLI prints
-// its own clear error if one is absent when actually run.
-
-const { execSync } = require("node:child_process");
-
-const TOOLS = [
-  { cmd: "python3", why: "runs the CLI", hint: "https://www.python.org/downloads/ (or `brew install python`)" },
-  { cmd: "yq", why: "parses config + frontmatter (mikefarah/yq, the Go one — NOT the python yq)", hint: "brew install yq  ·  https://github.com/mikefarah/yq" },
-  { cmd: "gh", why: "reads GitHub issue state", hint: "brew install gh  ·  https://cli.github.com" },
-];
-
-function have(cmd) {
-  try {
-    const probe = process.platform === "win32" ? `where ${cmd}` : `command -v ${cmd}`;
-    execSync(probe, { stdio: "ignore", shell: true });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-try {
-  const missing = TOOLS.filter((t) => !have(t.cmd));
-  if (missing.length) {
-    const lines = [
-      "",
-      "  work-plan installed. It needs a few tools on your PATH that npm can't install:",
-      ...missing.map((t) => `    • ${t.cmd} — ${t.why}\n        ${t.hint}`),
-      "",
-      "  (The CLI will tell you specifically if one is missing when you run it.)",
-      "",
-    ];
-    console.warn(lines.join("\n"));
-  }
-} catch {
-  // Never let the check itself break the install.
-}
-process.exit(0);