visionOS 版を .tortoise のビューワーにする #53

.claude/skills/release/SKILL.md

@@ -31,8 +31,18 @@ The policy page states what was *measured* — no accounts, no analytics, no
 advertising or third-party SDK, no tracking, and no networking code anywhere
 in the app or in TortoiseGraphics2 — which is also why no
 `PrivacyInfo.xcprivacy` is needed: nothing here touches a required-reason
-API, not even `UserDefaults`. Re-check that if a dependency or an
-`@AppStorage` ever arrives. **Both pages read in one language**, chosen by
+API, not even `UserDefaults`. That last one is **held on purpose and has
+already had to be defended once**: #53's viewer read a debug launch flag with
+`UserDefaults.standard.bool(forKey:)`, which is where a `-flag value` pair
+normally arrives, and one debug read of it obliges the whole app to ship a
+manifest declaring `CA92.1`. It now reads `ProcessInfo.processInfo.arguments`
+instead — same launch command, no manifest. So the thing to re-check is not
+only "a dependency or an `@AppStorage`": *any* `UserDefaults` call counts,
+including a read, including one that only fires in development, because it is
+compiled into the shipping binary. `NSWorldSensingUsageDescription` (#53) is
+**not** on that list — a usage description is a permission, not a
+required-reason API, and it changes the policy page rather than the manifest.
+**Both pages read in one language**, chosen by
 `?lang=` first, then `navigator.languages` in the reader's own order, falling
 back to English; adding a language is a code in `LANGS`, an `<option>`, and a
 translated `<section>`. Two rules hold it together: the site stores nothing (a
@@ -126,8 +136,29 @@ One more vocabulary mismatch to remember: deliver says `osx`, the Connect API
 says `MAC_OS`, and passing the former to spaceship reports a missing version
 that plainly exists.
 
+**visionOS is a third listing, spelled three different ways** (#11). It is a
+native app on the xrOS SDK, not "Designed for iPad", so App Store Connect gives
+it its own platform version — which the app record must carry *before* either
+lane will run (`get_edit_app_store_version` returns nil otherwise, and
+`metadata_diff` stops with "No editable xros version"). The text is shared with
+the other two, as it already was between iOS and macOS; only the screenshots
+are per-platform. Then the vocabulary: **`visionos` names the screenshots
+directory, `xros` goes to deliver, `VISION_OS` goes to spaceship**, and the
+first of those is not a style choice. A Vision Pro capture is 3840×2160, the
+same size as an Apple TV one, so deliver cannot resolve the display type from
+the size and falls back to asking whether the *path* contains `vision`
+(downcased) — name the directory after deliver's own platform value and every
+screenshot is filed as `APP_APPLE_TV` on an app with no tvOS listing.
+The captures need no staging: `xcrun simctl io <device> screenshot` on the
+visionOS simulator writes exactly 3840×2160, the simulated room and all, which
+is what visionOS screenshots look like anyway. They do carry an alpha channel,
+so `-alpha off` applies here like everywhere else. And no new identifier is
+needed — spaceship maps `xros` onto the **iOS** `BundleIdPlatform`, so the App
+IDs the iPhone/iPad build already registered are the ones visionOS signs
+against.
+
 **A release is a `v*` tag** (#4). Xcode Cloud runs one `Release` workflow off
-it — two Archive actions, iOS and macOS, each with a TestFlight internal
+it — an Archive action per platform, each with a TestFlight internal
 post-action bound to its own archive artifact. It carries no Build or Test
 action: GitHub Actions has already run the lint, the Kit tests and both
 platform builds on the way to main, and Xcode Cloud's 25 free compute

.github/workflows/appstore-metadata.yml

@@ -13,8 +13,8 @@ on:
       platform:
         description: Which listing
         type: choice
-        options: [both, ios, macos]
-        default: both
+        options: [all, ios, macos, visionos]
+        default: all
       apply:
         description: Upload (unticked, only the diff is shown)
         type: boolean
@@ -48,9 +48,10 @@ jobs:
         run: |
           lane='${{ inputs.apply && 'metadata_push' || 'metadata_diff' }}'
           case '${{ inputs.platform }}' in
-            ios)   platforms="ios" ;;
-            macos) platforms="mac" ;;
-            *)     platforms="ios mac" ;;
+            ios)      platforms="ios" ;;
+            macos)    platforms="mac" ;;
+            visionos) platforms="visionos" ;;
+            *)        platforms="ios mac visionos" ;;
           esac
           for p in $platforms; do
             echo "::group::$p $lane"

.github/workflows/ci.yml

@@ -99,6 +99,13 @@ jobs:
             destination: generic/platform=iOS Simulator
           - name: macOS
             destination: platform=macOS
+          # visionOS (#11). It shares every source file with iPadOS, so what
+          # this catches is the `#if` that quietly stops applying: SwiftUI
+          # marks a handful of iOS API unavailable there (`ToolbarSpacer`),
+          # and an `#if os(iOS)` compiles clean on all three platforms while
+          # doing nothing on this one.
+          - name: visionOS
+            destination: generic/platform=visionOS Simulator
     steps:
       - uses: actions/checkout@v7
 

App/Document/BlocksDocument.swift

@@ -25,14 +25,23 @@ struct BlocksDocument: FileDocument {
         guard let data = configuration.file.regularFileContents else {
             throw CocoaError(.fileReadCorruptFile)
         }
+        self.project = try Self.project(from: data)
+    }
+
+    /// Decoding, apart from `FileDocument`, because the visionOS viewer (#53)
+    /// reads `.tortoise` files through a `fileImporter` and has no
+    /// `DocumentGroup` to hand it a `ReadConfiguration`. The version gate has
+    /// to be the same one, or the two ways in would disagree about which files
+    /// are from the future.
+    static func project(from data: Data) throws -> BlocksProject {
         // Probe just the version before the full decode: a newer file's
         // unknown block shapes would fail the full decode with a generic
         // "corrupt" error before the version gate could explain it.
         let probe = try JSONDecoder().decode(SchemaVersionProbe.self, from: data)
         guard probe.schemaVersion <= BlocksProject.currentSchemaVersion else {
             throw DocumentError.newerSchema
         }
-        self.project = try JSONDecoder().decode(BlocksProject.self, from: data)
+        return try JSONDecoder().decode(BlocksProject.self, from: data)
     }
 
     func fileWrapper(configuration: WriteConfiguration) throws -> FileWrapper {

App/InfoPlist.xcstrings

@@ -30,6 +30,18 @@
         }
       }
     },
+    "NSWorldSensingUsageDescription" : {
+      "comment" : "Privacy - World Sensing Usage Description",
+      "extractionState" : "extracted_with_value",
+      "localizations" : {
+        "en" : {
+          "stringUnit" : {
+            "state" : "new",
+            "value" : "Used to place your drawing on a real table."
+          }
+        }
+      }
+    },
     "Tortoise Blocks Block" : {
 
     },

App/Localizable.xcstrings

@@ -14,6 +14,56 @@
         }
       }
     },
+    "Away" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "ださない"
+          }
+        }
+      }
+    },
+    "Blocks" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "ブロック"
+          }
+        }
+      }
+    },
+    "In Front of You" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "めのまえ"
+          }
+        }
+      }
+    },
+    "On a Table" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "つくえ"
+          }
+        }
+      }
+    },
+    "Where should the drawing go?" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "えを どこに だす？"
+          }
+        }
+      }
+    },
     "🎲" : {
       "localizations" : {
         "ja" : {
@@ -184,6 +234,16 @@
         }
       }
     },
+    "Can't Open This" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "これは ひらけません"
+          }
+        }
+      }
+    },
     "Cancel" : {
       "localizations" : {
         "ja" : {
@@ -494,6 +554,16 @@
         }
       }
     },
+    "Looking for a table…" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "つくえを さがしています…"
+          }
+        }
+      }
+    },
     "item %lld" : {
       "localizations" : {
         "ja" : {
@@ -634,6 +704,26 @@
         }
       }
     },
+    "No Drawing" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "えが ありません"
+          }
+        }
+      }
+    },
+    "No table found, so it is floating in front of you." : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "つくえが みつからないので、めのまえに だしているよ。"
+          }
+        }
+      }
+    },
     "Nothing to play" : {
       "localizations" : {
         "ja" : {
@@ -674,6 +764,26 @@
         }
       }
     },
+    "Open" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "ひらく"
+          }
+        }
+      }
+    },
+    "Open a drawing, or start from a sample." : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "えを ひらくか、サンプルから はじめてね。"
+          }
+        }
+      }
+    },
     "or less" : {
       "localizations" : {
         "ja" : {
@@ -784,6 +894,16 @@
         }
       }
     },
+    "Pinch to resize, twist to turn, drag to move." : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "つまむと おおきさ、ひねると むき、うごかすと ばしょが かわるよ。"
+          }
+        }
+      }
+    },
     "Play" : {
       "localizations" : {
         "ja" : {
@@ -884,6 +1004,16 @@
         }
       }
     },
+    "Reset Position" : {
+      "localizations" : {
+        "ja" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "もとの ばしょに もどす"
+          }
+        }
+      }
+    },
     "Resume" : {
       "localizations" : {
         "ja" : {

App/Models/RunnerModel.swift

@@ -22,6 +22,16 @@ final class RunnerModel {
     /// exports render exactly what is on screen.
     private(set) var lastRunCommands: [TortoiseCommand] = []
 
+    /// Bounding box of the last run's visible output, or nil when it drew
+    /// nothing. `nil` on an empty stream, exactly as `DrawingBounds` reports it.
+    ///
+    /// Computed once per run rather than per use, because both users want it
+    /// repeatedly and neither can afford the replay: the export sizes its
+    /// frame from it (once per export, previously replaying the stream again),
+    /// and the visionOS table maps the 3-D tortoise's position through the
+    /// same `autoFit` transform the canvas uses — *every display frame*.
+    private(set) var drawingBounds: DrawingBounds?
+
     /// Set when expansion fails; drives a kid-friendly alert.
     var showsExpansionError = false
 
@@ -114,6 +124,8 @@ final class RunnerModel {
             let expanded = try BlockExpander.expand(blocks)
             expandedBlockIDs = expanded.map(\.blockID)
             lastRunCommands = expanded.map(\.command)
+            drawingBounds = DrawingBounds.compute(
+                from: CommandPlayer.play(commands: lastRunCommands))
             lastRunTreeHash = blocks.hashValue
             player.isPaused = startPaused
             tortoise.reset()
@@ -182,7 +194,8 @@ final class RunnerModel {
     func pngData(scale: CGFloat = 2) -> Data? {
         if let cached = pngDataCache[scale] { return cached }
         guard canExport else { return nil }
-        let data = Self.renderPNG(lastRunCommands, longSide: 512, scale: scale)
+        let data = Self.renderPNG(
+            lastRunCommands, bounds: drawingBounds, longSide: 512, scale: scale)
         pngDataCache[scale] = data
         return data
     }
@@ -209,7 +222,8 @@ final class RunnerModel {
     /// putting the picture on the file in the first place.
     func thumbnailData() -> Data? {
         guard canExport else { return nil }
-        return Self.renderPNG(lastRunCommands, longSide: 256, scale: 1, onWhite: true)
+        return Self.renderPNG(
+            lastRunCommands, bounds: drawingBounds, longSide: 256, scale: 1, onWhite: true)
     }
 
     /// Renders a command stream to PNG, cropped tight to the drawing and
@@ -228,14 +242,15 @@ final class RunnerModel {
     /// exports transparent, because an export is a picture you place somewhere
     /// yourself, and the thumbnail white, because Finder places that one.
     private static func renderPNG(
-        _ commands: [TortoiseCommand], longSide: Double, scale: CGFloat, onWhite: Bool = false
+        _ commands: [TortoiseCommand], bounds: DrawingBounds?, longSide: Double, scale: CGFloat,
+        onWhite: Bool = false
     ) -> Data? {
         let export = Tortoise()
         export.speed = 0
         export.backgroundColor = onWhite ? .white : .clear
         export.apply(commands)
         export.hideTortoise()
-        let size = exportFrameSize(for: commands, longSide: longSide)
+        let size = exportFrameSize(for: bounds, longSide: longSide)
         let renderer = ImageRenderer(
             content: TortoiseCanvas(export)
                 .frame(width: size.width, height: size.height)
@@ -255,11 +270,10 @@ final class RunnerModel {
     /// an unusable sliver; an empty drawing (no bounds) falls back to a square,
     /// mirroring SVG's own "no visible output" fallback.
     private static func exportFrameSize(
-        for commands: [TortoiseCommand], longSide long: Double
+        for bounds: DrawingBounds?, longSide long: Double
     ) -> CGSize {
         let square = CGSize(width: long, height: long)
-        guard let bounds = DrawingBounds.compute(from: CommandPlayer.play(commands: commands))
-        else { return square }
+        guard let bounds else { return square }
         // Mirrors TortoiseUI's autoFit inset — the tortoise sprite's
         // half-extent × tortoiseScaleMax — so the drawing fills the frame with
         // a uniform margin instead of a lopsided one. 20 is the *triangle's*