fix(package): Do not trust bash with anything important!

[skip ci]

.github/workflows/package.yaml

@@ -28,8 +28,12 @@ jobs:
       - name: Verify Ref.
         id: release-ref-verify
         run: |
-          github_ref_name='${{ github.event.inputs.release_version }}' || '${{ github.ref_name }}'
-          python3 ./scripts/validate_ref_name.py $github_ref_name
+          release_name=$(
+            python3 ./scripts/validate_ref_name.py \
+              '${{ github.event.inputs.release_version }}' \
+              '${{ github.ref_name }}'
+          )
+          echo "CAPTURA_RELEASE_NAME=$release_name" >> $GITHUB_ENV
 
       - name: Get Body.
         id: release-get-body
@@ -85,7 +89,7 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # This token is provided by Actions, you do not need to create your own token
         with:
           tag_name: ${{ github.ref_name }}
-          release_name: v${{ github.ref_name }}
+          release_name: v${{ github.CAPTURA_RELEASE_NAME }}
           body: ${{ env.CAPTURA_RELEASE_NOTES }} 
           draft: false
           prerelease: false

scripts/validate_ref_name.py

@@ -1,3 +1,9 @@
+"""Yes, this could be done in bash. But it has been inconsistent.
+
+- First argument should be the override value.
+- Second should be the actual value.
+"""
+
 # =========================================================================== #
 import re
 import sys
@@ -12,9 +18,15 @@
 
 if __name__ == "__main__":
 
-    matched = semver.match(github_ref_name := sys.argv[1])
+    if len(sys.argv) != 3:
+        print("Not enough arguments.")
+
+    _, tag_override, tag = sys.argv
+    matched = semver.match(github_ref_name := tag_override or tag)
+
     if matched is None:
-        print(f"Illegal git ref name `{ github_ref_name }`. Exitting.")
         sys.exit(1)
     else:
-        print(f"Valid git ref name `{ github_ref_name }`.")
+        # NOTE: Should only print this since it is used to determine the release
+        #       name.
+        print(github_ref_name)