bigpowers 2.77.0 → 2.77.2

package/skills/using-bigpowers/SKILL.md

@@ -78,7 +78,7 @@ UTILITY     terse-mode / craft-skill / edit-document (any phase)
 
 If you work alone and do not want PR ceremony every task:
 
-1. Read [profiles/solo-git.md](../profiles/solo-git.md).
+1. Read [profiles/solo-git.md](../../profiles/solo-git.md).
 2. Register with `compose-workflow` → `specs/WORKFLOW-solo-git.md`.
 3. Ship with `release-branch` in **solo-local** mode (`land-branch.sh`), not `gh pr create`.
 

package/skills/write-document/REFERENCE.md

@@ -141,7 +141,7 @@ Auto-fill from CLAUDE.md `Test` and `Lint` commands.
 ```markdown
 ## Changelog
 
-See [CHANGELOG.md](CHANGELOG.md) or [Releases](https://github.com/user/repo/releases).
+See [CHANGELOG.md](../../CHANGELOG.md) or [Releases](https://github.com/user/repo/releases).
 ```
 
 ### 13. Links
@@ -158,7 +158,7 @@ See [CHANGELOG.md](CHANGELOG.md) or [Releases](https://github.com/user/repo/rele
 ```markdown
 ## License
 
-MIT — see [LICENSE](LICENSE) for details.
+MIT — see [LICENSE](../../LICENSE) for details.
 ```
 
 Detect from CLAUDE.md or project LICENSE file.

package/skills-lock.json

@@ -13,7 +13,7 @@
     },
     "audit-code": {
       "description": "Self-review checklist for the coding agent to run before dispatching a reviewer. Checks CONVENTIONS.md compliance, Boy Scout Rule, test coverage, types, and SOLID. Produces a pass/fail checklist. Use before request-review, before committing, or when user asks for a code quality check.",
-      "sha256": "bbb128254464f8f4",
+      "sha256": "0636f1d9d6c591a3",
       "path": "skills/audit-codeSKILL.md"
     },
     "audit-plan": {
@@ -48,7 +48,7 @@
     },
     "craft-skill": {
       "description": "Create new bigpowers skills with proper structure, progressive disclosure, and bundled resources. Use when user wants to create, write, or build a new skill for the bigpowers lifecycle.",
-      "sha256": "cbf8ab939cfbab87",
+      "sha256": "2caa9d8ae9298b4b",
       "path": "skills/craft-skillSKILL.md"
     },
     "deepen-architecture": {
@@ -108,7 +108,7 @@
     },
     "elaborate-spec": {
       "description": "Refine a rough idea into a clear, detailed specification through dialogue. Does not produce code. Use when user has a vague idea, wants to think through a feature before planning, or needs to turn \"I want X\" into a concrete spec.",
-      "sha256": "62ebfca87e6d0162",
+      "sha256": "9307ab61c789a0a0",
       "path": "skills/elaborate-specSKILL.md"
     },
     "enforce-first": {
@@ -158,7 +158,7 @@
     },
     "guard-git": {
       "description": "Block dangerous git commands (push, force push, reset --hard, clean, branch -D, checkout/restore .) and enforce Conventional Commits & Branch Protection before an AI agent runs them. Installs hook scripts for Claude Code, Cursor, Cursor CLI, and Gemini CLI; documents Google Antigravity Terminal deny lists. Use when the user wants git safety hooks, to block git push or destructive git in agents, or to mirror the same policy across AI coding tools.",
-      "sha256": "9ee881df89273b00",
+      "sha256": "e7083cba6d95a15f",
       "path": "skills/guard-gitSKILL.md"
     },
     "harden-vps": {
@@ -178,7 +178,7 @@
     },
     "investigate-bug": {
       "description": "Investigate a bug or issue by exploring the codebase to find root cause, then write a TDD-based fix plan to specs/bugs/BUG-*.md. Use when user reports a bug, wants to investigate a problem, mentions \"triage\", or wants to plan a fix.",
-      "sha256": "b842fd329d1f7741",
+      "sha256": "54f8c5417be34ef0",
       "path": "skills/investigate-bugSKILL.md"
     },
     "kickoff-branch": {
@@ -218,12 +218,12 @@
     },
     "plan-refactor": {
       "description": "Create a detailed refactor plan with tiny commits via user interview, then save it as specs/REFACTOR_LATEST.md. Use when user wants to plan a refactor, create a refactoring RFC, or break a refactor into safe incremental steps.",
-      "sha256": "c418c850806e2f16",
+      "sha256": "2d18809da7cc9de8",
       "path": "skills/plan-refactorSKILL.md"
     },
     "plan-release": {
       "description": "\"RELEASE-INDEX BUILDER — Sequence elaborated epics into specs/release-plan.yaml with WSJF ordering and BCP baselines. NOT a planning-spine substitute: it does not scope work (scope-work) or write story tasks (plan-work). Use after elaborate-spec when the user wants a versioned release index of epics.\"",
-      "sha256": "b56b800817cd383e",
+      "sha256": "9334ff48e5b1fab3",
       "path": "skills/plan-releaseSKILL.md"
     },
     "plan-tests": {
@@ -358,7 +358,7 @@
     },
     "using-bigpowers": {
       "description": "One-time bootstrap that introduces the bigpowers skills system, the PMBOK lifecycle arc, and tells you which skill to call first for your situation. Use when starting with bigpowers for the first time, when user asks \"where do I start?\", or when the skills system needs to be explained.",
-      "sha256": "332d6468868b759b",
+      "sha256": "bad9ebe9bf303b36",
       "path": "skills/using-bigpowersSKILL.md"
     },
     "validate-contracts": {

package/tests/test_srp_engine.py

@@ -0,0 +1,148 @@
+#!/usr/bin/env python3
+"""tests/test_srp_engine.py — unit tests for rewrite_links_for_pi and link_utils.
+
+Run: python3 -m pytest tests/test_srp_engine.py -v
+  or: python3 tests/test_srp_engine.py
+"""
+import os
+import sys
+import unittest
+
+# Make scripts/lib importable without installation.
+_REPO_ROOT = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
+_LIB = os.path.join(_REPO_ROOT, "scripts", "lib")
+sys.path.insert(0, _LIB)
+sys.path.insert(0, os.path.join(_REPO_ROOT, "scripts"))
+
+# srp-engine.py has a hyphen so we load it via importlib.
+import importlib.util as _ilu
+_spec = _ilu.spec_from_file_location("srp_engine", os.path.join(_LIB, "srp-engine.py"))
+_srp = _ilu.module_from_spec(_spec)
+sys.modules["srp_engine"] = _srp
+_spec.loader.exec_module(_srp)
+
+from srp_engine import rewrite_links_for_pi  # type: ignore[import]
+from link_utils import strip_code_spans, LINK_RE, MACHINE_PATH_RE
+
+
+class TestStripCodeSpans(unittest.TestCase):
+    def test_fenced_block_blanked(self):
+        md = "Before.\n```\n[fake](fake.md)\n```\nAfter."
+        shadow = strip_code_spans(md)
+        self.assertNotIn("[fake](fake.md)", shadow)
+        self.assertEqual(len(shadow), len(md))  # positions preserved
+
+    def test_inline_code_blanked(self):
+        md = "Use `[x](y.md)` as an example."
+        shadow = strip_code_spans(md)
+        self.assertNotIn("[x](y.md)", shadow)
+        self.assertEqual(len(shadow), len(md))
+
+    def test_real_link_survives(self):
+        md = "See [docs](guide.md) for details."
+        shadow = strip_code_spans(md)
+        self.assertIn("[docs](guide.md)", shadow)
+
+
+class TestLinkRE(unittest.TestCase):
+    def test_plain_link(self):
+        m = LINK_RE.search("[text](target.md)")
+        self.assertIsNotNone(m)
+        self.assertEqual(m.group(2), "target.md")
+
+    def test_link_with_title(self):
+        m = LINK_RE.search('[text](target.md "My Title")')
+        self.assertIsNotNone(m)
+        self.assertEqual(m.group(2), "target.md")
+
+    def test_link_with_fragment(self):
+        m = LINK_RE.search("[text](doc.md#section)")
+        self.assertIsNotNone(m)
+        self.assertEqual(m.group(2), "doc.md#section")
+
+    def test_no_match_on_image(self):
+        # Image links (![...](url)) contain the same syntax — LINK_RE will match
+        # the inner part; they should be treated as pass-through by callers.
+        m = LINK_RE.search("![alt](image.png)")
+        # The regex matches the inner ](url) portion — callers must handle images.
+        self.assertIsNotNone(m)
+
+
+class TestMachinePathRE(unittest.TestCase):
+    def test_file_uri(self):
+        self.assertTrue(MACHINE_PATH_RE.search("file:///Users/alice/project"))
+
+    def test_macos_path(self):
+        self.assertTrue(MACHINE_PATH_RE.search("/Users/alice/project"))
+
+    def test_linux_path(self):
+        self.assertTrue(MACHINE_PATH_RE.search("/home/alice/project"))
+
+    def test_windows_path_single_backslash(self):
+        self.assertTrue(MACHINE_PATH_RE.search("C:\\Users\\alice"))
+
+    def test_github_url_not_flagged(self):
+        self.assertFalse(
+            MACHINE_PATH_RE.search("https://github.com/danielvm-git/bigpowers")
+        )
+
+
+class TestRewriteLinksForPi(unittest.TestCase):
+    """Tests run without a real repo tree — path existence is irrelevant here;
+    we only verify that the URL construction logic is correct."""
+
+    NAME = "my-skill"
+
+    def _rw(self, body):
+        return rewrite_links_for_pi(body, self.NAME)
+
+    def test_external_link_untouched(self):
+        body = "[docs](https://example.com)"
+        self.assertEqual(self._rw(body), body)
+
+    def test_anchor_only_untouched(self):
+        body = "[section](#heading)"
+        self.assertEqual(self._rw(body), body)
+
+    def test_absolute_path_untouched(self):
+        body = "[root](/absolute/path.md)"
+        self.assertEqual(self._rw(body), body)
+
+    def test_relative_link_repointed(self):
+        body = "[ref](REFERENCE.md)"
+        result = self._rw(body)
+        # Must not contain the original bare target
+        self.assertNotIn("](REFERENCE.md)", result)
+        # Must be a relative path targeting the skills source tree
+        self.assertIn("REFERENCE.md", result)
+
+    def test_fragment_preserved(self):
+        body = "[section](REFERENCE.md#anchor)"
+        result = self._rw(body)
+        self.assertIn("#anchor", result)
+        # Fragment must not be mangled into a path component
+        self.assertNotIn("anchor" + os.sep, result)
+
+    def test_link_with_title_attribute(self):
+        body = '[ref](REFERENCE.md "My Title")'
+        result = self._rw(body)
+        # Must not be left unrewritten
+        self.assertNotIn("](REFERENCE.md", result)
+
+    def test_code_block_links_not_rewritten(self):
+        body = "Real: [ref](REFERENCE.md)\n\n```\n[fake](nowhere.md)\n```\n"
+        result = self._rw(body)
+        # The real link should be rewritten
+        self.assertNotIn("](REFERENCE.md)", result)
+        # The code-block link must be untouched
+        self.assertIn("[fake](nowhere.md)", result)
+
+    def test_inline_code_link_not_rewritten(self):
+        body = "Example: `[x](bad.md)` — real [ref](REFERENCE.md)."
+        result = self._rw(body)
+        self.assertIn("[x](bad.md)", result)  # inline code preserved
+        self.assertNotIn("](REFERENCE.md)", result)  # real link rewritten
+
+
+if __name__ == "__main__":
+    unittest.main()