fix: require script/Build.sol, no legacy name, no skip

Accepting the legacy name kept the hardcoded-name hazard alive and let repos
defer indefinitely. script/Build.sol is the name; a repo that commits
src/generated/ without it goes red. Repos that don't want to upgrade pin rainix.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: thedavidmeister

.github/workflows/rainix-copy-artifacts.yaml

@@ -29,32 +29,19 @@ jobs:
       - name: Regenerate meta artifacts
         if: hashFiles('script/build-meta.sh') != ''
         run: nix develop github:rainlanguage/rainix/${{ env.RAINIX_SHA }}#sol-shell -c ./script/build-meta.sh
-      # A repo that commits generated sources but exposes no regen step passes
-      # every check below without anything being regenerated, which is
-      # indistinguishable from a real pass. Fail loudly instead.
-      - name: Assert committed generated sources have a regen step
+      # Committed generated sources must be regenerable here, or the currency
+      # check below passes without checking anything. The codegen script is
+      # `script/Build.sol`, matched exactly: a repo that renames or drops it
+      # goes red rather than skipping regeneration and reporting green.
+      - name: Regenerate generated sources
         run: |
-          if [ -d src/generated ] \
-            && [ ! -f script/Build.sol ] \
-            && [ ! -f script/BuildPointers.sol ] \
-            && [ ! -f script/build.sh ]; then
-            echo "::error::src/generated/ is committed but no regen step was found (script/Build.sol, script/BuildPointers.sol or script/build.sh), so the currency check below would pass without regenerating anything."
+          if [ -d src/generated ] && [ ! -f script/Build.sol ]; then
+            echo "::error::src/generated/ is committed but script/Build.sol was not found, so the committed sources cannot be currency checked here. The codegen script must be script/Build.sol."
             exit 1
           fi
-      # The codegen script is consumer named: `script/Build.sol` is the
-      # convention, `script/BuildPointers.sol` the legacy name. Both are matched
-      # because a repo that renames its script must not silently stop being
-      # regenerated, and so must not be forced to rename in lockstep with this
-      # workflow, which consumers track at @main.
-      - name: Regenerate generated sources
-        if: hashFiles('script/Build.sol', 'script/BuildPointers.sol') != ''
-        run: |
-          for candidate in script/Build.sol script/BuildPointers.sol; do
-            if [ -f "$candidate" ]; then
-              nix develop github:rainlanguage/rainix/${{ env.RAINIX_SHA }}#sol-shell -c forge script "./$candidate"
-              break
-            fi
-          done
+          if [ -f script/Build.sol ]; then
+            nix develop github:rainlanguage/rainix/${{ env.RAINIX_SHA }}#sol-shell -c forge script ./script/Build.sol
+          fi
       - name: Build Solidity
         run: nix develop github:rainlanguage/rainix/${{ env.RAINIX_SHA }}#sol-shell -c forge build
       - name: Copy forge artifacts into committed location
@@ -71,6 +58,6 @@ jobs:
       - name: Assert committed artifacts match freshly built
         run: |
           if ! git diff --exit-code; then
-            echo "::error::Committed artifacts are stale. Regenerate (script/build-meta.sh, your codegen script, script/CopyArtifacts.sol, script/build.sh, forge fmt) and commit."
+            echo "::error::Committed artifacts are stale. Regenerate (script/build-meta.sh, script/Build.sol, script/CopyArtifacts.sol, script/build.sh, forge fmt) and commit."
             exit 1
           fi