craft-native 0.0.185 → 0.0.187

package/dist/ios/src/index.js

@@ -156,7 +156,12 @@ function serializeSimulatorSteps(steps) {
 `;
 }
 function createNativeMutationValidator() {
-  const output = mkdtempSync(join2(tmpdir(), "craft-native-mutation-"));
+  const output = mkdtempSync(join2(tmpdir(), `craft-native-mutation-p${process.pid}-`));
+  const removeOutput = () => {
+    process.off("exit", removeOutput);
+    rmSync(output, { recursive: true, force: true });
+  };
+  process.on("exit", removeOutput);
   const binary = join2(output, "validate");
   try {
     writeFileSync(join2(output, "main.swift"), readFileSync(join2(templates, "MutationValidator.swift")));
@@ -164,7 +169,7 @@ function createNativeMutationValidator() {
     if (build.exitCode !== 0)
       throw new Error(build.stderr.toString());
   } catch (error) {
-    rmSync(output, { recursive: true, force: true });
+    removeOutput();
     throw error;
   }
   let disposed = false;
@@ -185,7 +190,7 @@ function createNativeMutationValidator() {
     },
     dispose() {
       disposed = true;
-      rmSync(output, { recursive: true, force: true });
+      removeOutput();
     }
   };
 }

package/dist/ios/templates/CraftNativeScreen.swift

@@ -577,6 +577,9 @@ private final class CraftNativeVideoView: UIView, AVPlayerViewControllerDelegate
         // A muted video makes no sound, so it leaves the session as it is.
         if muted { releaseSession() } else { claimSession() }
         playerController.showsPlaybackControls = controls
+        // The system's controls are for VoiceOver too; a bare video stays one element.
+        playerController.view.accessibilityElementsHidden = !controls
+        isUserInteractionEnabled = true
         playerController.videoGravity = resizeMode == "cover" ? .resizeAspectFill
             : resizeMode == "stretch" ? .resize
             : .resizeAspect
@@ -5687,6 +5690,13 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
         return handler
     }
 
+    /// Views that take their own touches whatever handlers the screen gives
+    /// them: a scroll view, a modal, and a video's system controls (with
+    /// interaction off, its play, scrubber and full screen never answered).
+    private func ownsTouches(_ view: UIView) -> Bool {
+        view is UIScrollView || view is CraftNativeModalView || view is CraftNativeVideoView
+    }
+
     private func updatePressHandler(_ handler: String?, for view: UIView) {
         let id = ObjectIdentifier(view)
         let handler = handler?.isEmpty == false ? handler : nil
@@ -5701,7 +5711,7 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
         }
         // A container keeps receiving touches without a handler of its own,
         // or every control inside it (and the screen root) would be dead.
-        if !(view is UIScrollView || view is CraftNativeModalView) { view.isUserInteractionEnabled = actionable || longPressHandlers[id] != nil || view is CraftNativeFlowView }
+        if !ownsTouches(view) { view.isUserInteractionEnabled = actionable || longPressHandlers[id] != nil || view is CraftNativeFlowView }
     }
 
     private func updatePressFeedback(_ pressIn: String?, _ pressOut: String?, for view: UIView) {
@@ -5710,7 +5720,7 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
         if let pressOut, !pressOut.isEmpty { pressOutHandlers[id] = pressOut } else { pressOutHandlers.removeValue(forKey: id) }
         guard pressInHandlers[id] != nil || pressOutHandlers[id] != nil else {
             if let recognizer = pressFeedbackRecognizers.removeValue(forKey: id) { view.removeGestureRecognizer(recognizer) }
-            if !(view is UIScrollView || view is CraftNativeModalView) { view.isUserInteractionEnabled = handlers[id] != nil || linkTargets[id] != nil || longPressHandlers[id] != nil || view is CraftNativeFlowView }
+            if !ownsTouches(view) { view.isUserInteractionEnabled = handlers[id] != nil || linkTargets[id] != nil || longPressHandlers[id] != nil || view is CraftNativeFlowView }
             return
         }
         if pressFeedbackRecognizers[id] == nil {
@@ -5722,14 +5732,14 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
             pressFeedbackRecognizers[id] = recognizer
             view.addGestureRecognizer(recognizer)
         }
-        if !(view is UIScrollView || view is CraftNativeModalView) { view.isUserInteractionEnabled = true }
+        if !ownsTouches(view) { view.isUserInteractionEnabled = true }
     }
 
     private func updateLongPressHandler(_ handler: String?, for view: UIView) {
         let id = ObjectIdentifier(view)
         if let handler, !handler.isEmpty {
             longPressHandlers[id] = handler
-            if !(view is UIScrollView || view is CraftNativeModalView) { view.isUserInteractionEnabled = true }
+            if !ownsTouches(view) { view.isUserInteractionEnabled = true }
             if longPressRecognizers[id] == nil {
                 let recognizer = UILongPressGestureRecognizer(target: self, action: #selector(viewLongPressed(_:)))
                 longPressRecognizers[id] = recognizer
@@ -5740,7 +5750,7 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
             if let recognizer = longPressRecognizers.removeValue(forKey: id) {
                 view.removeGestureRecognizer(recognizer)
             }
-            if !(view is UIScrollView || view is CraftNativeModalView) { view.isUserInteractionEnabled = handlers[id] != nil || linkTargets[id] != nil || view is CraftNativeFlowView }
+            if !ownsTouches(view) { view.isUserInteractionEnabled = handlers[id] != nil || linkTargets[id] != nil || view is CraftNativeFlowView }
         }
     }
 
@@ -6336,10 +6346,14 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
         // reads: its label names the group, as a tab strip or a pager's
         // name, and never stands in for its content. As an element of its
         // own, a labelled pager hid every page from VoiceOver.
+        // A video with its system controls is a group too: VoiceOver reaches
+        // play, the scrubber, skipping and full screen inside it.
+        let controlledVideo = type == "Video" && (props["controls"] as? Bool) == true
         let container = ["ScrollView", "FlatList", "SectionList"].contains(type)
             || ["tablist", "radiogroup", "list", "menu", "menubar", "toolbar"].contains(role ?? "")
+            || controlledVideo
         view.isAccessibilityElement = !container && role != "none" && (view.accessibilityLabel != nil || role != nil || ["Text", "Button", "Link", "Checkbox", "Image", "ImageBackground", "Video", "TextInput", "Picker", "DatePicker", "Switch", "Slider"].contains(type))
-        view.accessibilityContainerType = container && view.accessibilityLabel != nil ? .semanticGroup : .none
+        view.accessibilityContainerType = container && (view.accessibilityLabel != nil || controlledVideo) ? .semanticGroup : .none
         var traits: UIAccessibilityTraits = []
         switch role ?? type.lowercased() {
         case "tablist": traits.insert(.tabBar)

package/dist/temp-dir.d.ts

@@ -0,0 +1,65 @@
+/**
+ * Scratch directories that do not outlive the process that made them.
+ *
+ * Every Craft temp directory goes through here: packaging, signing, the
+ * updater's staging area, the sidebar config spill file, and the build and
+ * test harnesses. Issue #357: a bare `mkdtempSync(join(tmpdir(), 'craft-…'))`
+ * per run, with no cleanup or a `finally` that a killed run never reaches,
+ * left 1,983 directories (~13 GB) in the macOS per-user temp folder.
+ *
+ * Three layers, because each one misses something the next one catches:
+ *
+ * 1. The caller removes the directory when it is done (`withTempDir`, or
+ *    `removeTempDir` in a `finally`).
+ * 2. Every live directory is also removed on process exit and on SIGINT,
+ *    SIGTERM and SIGHUP, which is what a Ctrl-C, a CI timeout or a tool's
+ *    kill sends. Node skips `exit` listeners on a signal's default action,
+ *    so the signals need their own handlers.
+ * 3. SIGKILL cannot be caught, and a synchronous child (`Bun.spawnSync`
+ *    running xcodebuild) can hold the event loop until the parent is killed
+ *    for good. So each name carries the owning pid (`craft-ios-p4242-a1B2c3`),
+ *    and the first directory a later process creates sweeps up any whose
+ *    owner is gone.
+ */
+/** Remove every directory this process created and has not removed yet. */
+export declare function removeAllTempDirs(): void;
+export interface SweepOptions {
+    /** Directory to sweep. Defaults to the OS temp directory. */
+    root?: string;
+    /** Whether a pid still runs. Injected by tests. */
+    isAlive?: (pid: number) => boolean;
+    /** Minimum age before an orphan is removed. */
+    minAgeMs?: number;
+    now?: number;
+}
+/**
+ * Remove Craft temp directories whose owning process has exited.
+ *
+ * Only names this module produced are considered (`craft-…-p<pid>-XXXXXX`),
+ * so another tool's `craft-something` directory is never touched. Returns
+ * the paths it removed.
+ */
+export declare function sweepOrphanedTempDirs(options?: SweepOptions): string[];
+/**
+ * The mkdtemp prefix for a label: `ios-native` and `craft-ios-native-` both
+ * become `craft-ios-native-p<pid>-`.
+ */
+export declare function tempDirPrefix(label: string): string;
+/**
+ * Create a scratch directory in the OS temp folder that is removed when this
+ * process ends, however it ends. Remove it sooner with `removeTempDir`.
+ */
+export declare function createTempDir(label: string, root?: string): string;
+/** Remove a directory from `createTempDir` now rather than at exit. */
+export declare function removeTempDir(dir: string): void;
+/**
+ * Let a directory from `createTempDir` outlive the process, for a harness
+ * asked to keep its generated project for inspection. The caller now owns
+ * it, and its pid-stamped name means a later sweep removes it once this
+ * process is gone.
+ */
+export declare function keepTempDir(dir: string): void;
+/** Run `fn` with a fresh scratch directory that is removed when it settles. */
+export declare function withTempDir<T>(label: string, fn: (dir: string) => T | Promise<T>): Promise<T>;
+/** Directories created here and not yet removed. For tests and diagnostics. */
+export declare function liveTempDirs(): string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "craft-native",
-  "version": "0.0.185",
+  "version": "0.0.187",
   "type": "module",
   "description": "Build desktop apps with web languages - TypeScript SDK for Craft",
   "author": "Chris Breuer <chris@stacksjs.org>",