ci: count the Claude marketplace as shipped, and fail when the change probe errors

build() ships .claude-plugin/marketplace.json with dist-relative sources,
but SHIPPED (--list-paths) and release_notes.py only knew the copied FILES,
so a marketplace-only edit never cut the patch release that republishes
plugin-dist. A new REWRITTEN tuple joins SHIPPED and the release-notes
classifier, and a test builds the dist and fails on any file it ships that
SHIPPED does not list (falsified: dropping REWRITTEN fails it).

release.yml also read plugin-content-changed.sh's exit 2 (path list
unreadable) as 'no change' inside an elif; it now captures the code and
fails the job.

Closes #535

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LpkeXRvikhJZSVGBFM91Dv
Signed-off-by: TM Hospitality Strategies <TMHospitalityStrategies@gmail.com>

Author: TMHSDigital

.github/scripts/release_notes.py

@@ -28,7 +28,7 @@
 from pathlib import Path
 
 sys.path.insert(0, str(Path(__file__).resolve().parent.parent.parent / "scripts"))
-from build_plugin_dist import DIRS, FILES  # noqa: E402  (the one list of shipped paths)
+from build_plugin_dist import DIRS, FILES, REWRITTEN  # noqa: E402  (the one list of shipped paths)
 
 REPO_URL = "https://github.com/TMHSDigital/Blender-Developer-Tools"
 GROUPS = (
@@ -40,7 +40,7 @@
 # change here is a change a plugin user receives, including the manifests and
 # the opt-in live-session plugin.
 USER_DIRS = tuple(d + "/" for d in DIRS)
-USER_FILES = frozenset(FILES)
+USER_FILES = frozenset(FILES + REWRITTEN)
 NO_USER_CHANGES = ("No changes to the plugin content (skills, rules, snippets, templates, "
                    "manifests, MCP config) in this release.")
 

.github/workflows/release.yml

@@ -76,14 +76,28 @@ jobs:
           # .github/scripts/bump-kind.sh owns the patterns (pages.yml shares it).
           # [skip ci] (guarded on the job) remains an optional belt-and-suspenders override.
           kind=$(git log "$range" --format='%B%x00' | bash .github/scripts/bump-kind.sh)
+          # 0 = plugin content changed, 1 = it did not, anything else = the
+          # script could not read the shipped-path list. Inside an `elif` the
+          # error would read as "no change" and skip the release silently (#535).
+          plugin_changed=1
+          if [ -n "$last_tag" ]; then
+            set +e
+            bash .github/scripts/plugin-content-changed.sh "$last_tag"
+            plugin_changed=$?
+            set -e
+            if [ "$plugin_changed" -gt 1 ]; then
+              echo "::error::plugin-content-changed.sh failed (exit $plugin_changed); cannot decide whether to release"
+              exit 1
+            fi
+          fi
           bump="patch"
           release="false"
           if [ -z "$last_tag" ]; then
             # Initial release: no prior tag -> cut the first release at the current VERSION.
             release="true"
           elif [ "$kind" != "none" ]; then
             bump="$kind"; release="true"
-          elif bash .github/scripts/plugin-content-changed.sh "$last_tag"; then
+          elif [ "$plugin_changed" -eq 0 ]; then
             # No feat/fix subject, but skills/rules/snippets/templates/claude
             # changed: ship a patch so a docs: correction still reaches plugin
             # users instead of waiting for the next feat/fix (#350).

scripts/build_plugin_dist.py

@@ -45,10 +45,16 @@
          ".codex-plugin/plugin.json", "assets/favicon.svg")
 # marketplace.json entry -> source inside the dist tree.
 DIST_SOURCES = {"blender-developer-tools": "./", "blender-live-session": "./live-session"}
+# Repo files the dist ships after rewriting rather than copying: build() writes
+# the Claude marketplace with dist-relative sources. They are not in FILES (a
+# plain copy would ship the repo's sources) but they are still shipped (#535).
+REWRITTEN = (".claude-plugin/marketplace.json",)
 # Every repo path the plugin ships. release.yml's plugin-content-changed.sh and
 # release_notes.py read this (--list-paths), so a change to any shipped file,
 # manifests and mcp.json included, counts as a plugin change (#494).
-SHIPPED = DIRS + FILES
+SHIPPED = DIRS + FILES + REWRITTEN
+# Files build() generates with no repo source (the dist README).
+GENERATED = ("README.md",)
 MAX_BYTES = 2_000_000
 IGNORE = shutil.ignore_patterns("__pycache__", "*.pyc")
 

tests/test_build_plugin_dist.py

@@ -53,6 +53,23 @@ def test_refuses_git_checkout_even_with_force(self):
             self.assertIn(".git", err)
             self.assertTrue((Path(td) / ".git").is_dir())
 
+    def test_every_built_file_is_listed_as_shipped(self):
+        # --list-paths decides whether a release ships plugin changes; a file
+        # the dist carries but SHIPPED omits never triggers one (#535).
+        with tempfile.TemporaryDirectory() as td:
+            out = Path(td) / "dist"
+            code, _ = run(["--out", str(out)])
+            self.assertEqual(code, 0)
+            dirs = tuple(d + "/" for d in b.DIRS)
+            unlisted = [
+                rel for rel in (p.relative_to(out).as_posix()
+                                for p in out.rglob("*") if p.is_file())
+                if not rel.startswith(dirs) and rel not in b.SHIPPED
+                and rel not in b.GENERATED
+            ]
+            self.assertEqual(unlisted, [])
+            self.assertIn(".claude-plugin/marketplace.json", b.SHIPPED)
+
     def test_builds_into_missing_dir(self):
         with tempfile.TemporaryDirectory() as td:
             out = Path(td) / "dist"