com.robotsquid.squidhub 0.5.8 → 0.5.9

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 0.5.9
+
+**Build and Upload just uploads.** The "Upload N files (X MB)?" confirmation is
+gone. Choosing the action IS the decision, and a modal that appears after a long
+WebGL build is worse than redundant — it stops an unattended build from ever
+finishing.
+
+Nothing it said is lost, only moved. The file count, the size, the endpoint, the
+skipped OS metadata, the missing-manifest warning and the build warnings all go
+to the Console, where they can be read *during* the upload and are still there
+afterwards. A dialog is gone the moment it is dismissed, which is exactly when a
+warning stops being read.
+
+⚠️ **Refusals still stop the run.** A precompressed build or one with no
+`index.html` at its top level is a failure, not a confirmation, and the server
+would reject it anyway.
+
 ## 0.5.8
 
 **The game fills the frame, and Unity's chrome is gone.**

package/Editor/SquidHubUploader.cs

@@ -85,32 +85,49 @@ namespace SquidHub.Editor
 
             var publish = SquidHubUploadSettings.PublishAfterUpload;
 
-            if (!EditorUtility.DisplayDialog("SquidHub",
-                    $"Upload {scan.Files.Count} files ({scan.TotalBytes / 1024f / 1024f:F1} MB)?" +
-                    (scan.Skipped.Count > 0
-                        ? $"\n\n{scan.Skipped.Count} OS metadata file(s) will be skipped."
-                        : "") +
-                    // ⚠️ Said before the upload, not after. A build folder with no
-                    // manifest.json declares nothing — save schema 1, no display
-                    // constraints — and the publisher cannot tell from the result.
-                    (scan.Files.Any(f => f.Path == "manifest.json")
-                        ? ""
-                        : "\n\nThere is no manifest.json in this folder, so the build declares " +
-                          "nothing: save schema 1 and no display constraints. Set one up in " +
-                          "SquidHub → Game Manifest… and rebuild.") +
-                    // ⚠️ These describe a build that uploads fine and PLAYS badly,
-                    // which is the kind nobody catches until a player does.
-                    (scan.Warnings.Count > 0 ? "\n\n" + string.Join("\n\n", scan.Warnings) : "") +
-                    (publish
-                        ? "\n\nIt will be published to players if the server allows it — that is " +
-                          "dev only. On prod, promote it in the console."
-                        : ""),
-                    "Upload", "Cancel"))
+            // ⚠️ NO CONFIRMATION. Choosing "Build and Upload" IS the decision;
+            // asking again is a modal between the publisher and the thing they
+            // just asked for, and one that appears after a long WebGL build is
+            // worse than useless — it stops an unattended build from finishing.
+            //
+            // ⚠️ Nothing is lost, only moved. Everything the dialog used to say
+            // goes to the Console, where it can be read during the upload and is
+            // still there afterwards — a dialog is gone the moment it is
+            // dismissed, which is exactly when a warning stops being read.
+            // Refusals above still stop the run; those are failures, not
+            // confirmations.
+            Report(scan, publish);
+
+            Run(Upload(scan, publish));
+        }
+
+        /// <summary>
+        /// What the dialog used to ask, said to the Console instead.
+        ///
+        /// ⚠️ Warnings are LogWarning, not Log. They describe a build that
+        /// uploads perfectly and plays badly, so they have to survive a Console
+        /// that is already full of build spam.
+        /// </summary>
+        private static void Report(BuildScan scan, bool publish)
+        {
+            Debug.Log($"[SquidHub] uploading {scan.Files.Count} file(s), " +
+                      $"{scan.TotalBytes / 1024f / 1024f:F1} MB, to {SquidHubUploadSettings.Endpoint}" +
+                      (scan.Skipped.Count > 0
+                          ? $" ({scan.Skipped.Count} OS metadata file(s) skipped)"
+                          : "") +
+                      (publish ? " — will publish on success where the server allows it (dev only)." : "."));
+
+            // ⚠️ A build folder with no manifest.json declares nothing — save
+            // schema 1, no display constraints — and nothing in the upload
+            // result would ever tell the publisher that.
+            if (!scan.Files.Any(f => f.Path == "manifest.json"))
             {
-                return;
+                Debug.LogWarning(
+                    "[SquidHub] no manifest.json in this build, so it declares nothing: save schema 1 " +
+                    "and no display constraints. Set one up in SquidHub → Game Manifest… and rebuild.");
             }
 
-            Run(Upload(scan, publish));
+            foreach (var warning in scan.Warnings) Debug.LogWarning("[SquidHub] " + warning);
         }
 
         // ⚠️ Driven by EditorApplication.update rather than blocking. Sleeping

package/README.md

@@ -144,6 +144,11 @@ an upload key is not a person — it sits in CI configs and Editor preferences.
 That asymmetry is deliberate: it is what keeps a leaked key from replacing the
 live build for everyone.
 
+⚠️ **It does not ask.** Choosing the action is the decision, so there is no
+"upload N files?" confirmation to dismiss — the file count, size, destination and
+any warnings go to the Console instead, where they survive being read. A build
+that *cannot* upload still stops with a dialog naming the fix.
+
 The upload is refused before a byte moves if the build is precompressed or has
 no `index.html` at its top level, and OS metadata is dropped rather than sent —
 which is also what makes a build pushed from Unity hash identically to the same

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "com.robotsquid.squidhub",
-  "version": "0.5.8",
+  "version": "0.5.9",
   "displayName": "SquidHub",
   "description": "Cloud saves, analytics and the coin economy for Unity WebGL games published on SquidHub. Installs the C# wrapper, the .jslib half and the page bridge, and rewrites the exported index.html so it survives the platform's CSP.",
   "unity": "2021.3",