@mmerterden/multi-agent-pipeline 20.8.1 → 20.8.2

package/pipeline/skills/shared/external/localization-reuse-map/scripts/scan-screen-keys.py

@@ -13,7 +13,10 @@ import os
 import re
 import sys
 
-from _shared import IDENT, WEB_DOLLAR_T, WEB_T, WORD, is_interpolated, lsk_keys, source_files
+from _shared import (
+    ACCESSOR_EXTENSIONS, DEFAULT_KEY_TYPES, IDENT, WEB_DOLLAR_T, WEB_T, WORD, is_interpolated, key_type_alternation,
+    key_type_warning, key_types, lsk_keys, source_files,
+)
 
 ANDROID = {".kt", ".kts", ".xml"}
 WEB = {".ts", ".tsx", ".js", ".jsx", ".vue"}
@@ -26,7 +29,6 @@ R_STRING = re.compile(rf"\bR\.string\.([{WORD}]+)")
 XML_STRING = re.compile(rf"@string/([{WORD}.]+)")
 WEB_T_DYNAMIC = re.compile(r"(?<![\w$])t\(\s*[`$A-Za-z_]")
 WEB_DOLLAR_T_DYNAMIC = re.compile(r"\$t\(\s*[`$A-Za-z_]")
-LSK_DYNAMIC = re.compile(r"LocalizationStringKey\(")
 FORMAT_DYNAMIC = re.compile(r"String\(format:")
 GETSTRING_DYNAMIC = re.compile(r"\bgetString\(\s*[A-Za-z_]")
 INTERPOLATED_LOCALIZED = re.compile(r"\"([^\"\n]*(?:\\\(|\$\{|\$[A-Za-z_])[^\"\n]*)\"\.localized\b")
@@ -42,7 +44,11 @@ def platform_of(path):
     return "ios"
 
 
-def hits_in_line(line, ext):
+def dynamic_key_pattern(types):
+    return re.compile(rf"({key_type_alternation(types)})\(")
+
+
+def hits_in_line(line, ext, types=DEFAULT_KEY_TYPES):
     static, dynamic = [], []
     if ext == ".xml":
         static.extend(m.group(1) for m in XML_STRING.finditer(line))
@@ -55,12 +61,13 @@ def hits_in_line(line, ext):
         if WEB_DOLLAR_T_DYNAMIC.search(line):
             dynamic.append("(dynamic: $t(...))")
         return static, dynamic
-    static.extend(lsk_keys(line))
+    static.extend(lsk_keys(line, types))
     static.extend(m.group(1) for m in LITERAL_LOCALIZED.finditer(line) if not is_interpolated(m.group(1)))
     static.extend(m.group(1) for m in R_STRING.finditer(line))
-    if LSK_DYNAMIC.search(line):
-        dynamic.append("(dynamic: LocalizationStringKey(...))")
-    mentions_key = any(marker in line for marker in (".localized", "LocalizationStringKey"))
+    constructed = dynamic_key_pattern(types).search(line)
+    if constructed:
+        dynamic.append(f"(dynamic: {constructed.group(1)}(...))")
+    mentions_key = any(marker in line for marker in (".localized", *types))
     if mentions_key and FORMAT_DYNAMIC.search(line):
         dynamic.append("(dynamic: String(format:...))")
     if "R.string." not in line and GETSTRING_DYNAMIC.search(line):
@@ -69,26 +76,30 @@ def hits_in_line(line, ext):
         dynamic.append(f'(dynamic: "{match.group(1)}".localized)')
     for match in RECEIVER_LOCALIZED.finditer(line):
         receiver = match.group(1)
-        if not receiver.startswith("LocalizationStringKey"):
+        if not receiver.startswith(tuple(types)):
             dynamic.append(f"(dynamic: {receiver}.localized)")
     return static, dynamic
 
 
-def scan(root):
+def scan(root, types=DEFAULT_KEY_TYPES):
     keys = []
     seen = set()
+    accessor_texts = []
     scanned = 0
     for path in source_files(root):
         try:
             with open(path, encoding="utf-8") as stream:
-                lines = stream.read().splitlines()
+                text = stream.read()
         except (OSError, UnicodeDecodeError):
             continue
+        lines = text.splitlines()
         scanned += 1
         ext = os.path.splitext(path)[1]
+        if ext in ACCESSOR_EXTENSIONS:
+            accessor_texts.append(text)
         platform = platform_of(path)
         for number, line in enumerate(lines, 1):
-            static, dynamic = hits_in_line(line, ext)
+            static, dynamic = hits_in_line(line, ext, types)
             in_error = bool(ERROR_CONTEXT.search(line))
             found = [(k, "error" if in_error else "static") for k in static] + [(k, "dynamic") for k in dynamic]
             for key, category in found:
@@ -104,7 +115,7 @@ def scan(root):
                     "line": number,
                     "snippet": line.strip()[:200],
                 })
-    return scanned, keys
+    return scanned, keys, key_type_warning(accessor_texts, types)
 
 
 CATEGORIES = ("static", "error", "dynamic")
@@ -115,22 +126,31 @@ def main():
     cli.add_argument("--screen-path", required=True, help="folder or single file of the built screen")
     cli.add_argument("--out", help="JSON destination; stdout when omitted")
     cli.add_argument("--category", default="all", choices=["all", "error", "dynamic", "static"])
+    cli.add_argument(
+        "--key-type", action="append", metavar="NAME",
+        help="type whose member chains are keys, e.g. the project's generated key enum; repeatable or comma list "
+             f"(default {', '.join(DEFAULT_KEY_TYPES)})",
+    )
     opts = cli.parse_args()
 
     if not os.path.exists(opts.screen_path):
         cli.error(f"--screen-path not found: {opts.screen_path}")
+    try:
+        types = key_types(opts.key_type)
+    except ValueError as exc:
+        cli.error(str(exc))
 
-    scanned, keys = scan(opts.screen_path)
+    scanned, keys, type_warning = scan(opts.screen_path, types)
     if opts.category != "all":
         keys = [entry for entry in keys if entry["category"] == opts.category]
     keys.sort(key=lambda entry: (entry["category"], entry["file"], entry["line"]))
     per_category = [entry["category"] for entry in keys]
     counts = {name: per_category.count(name) for name in CATEGORIES}
 
-    text = json.dumps(
-        {"screenPath": opts.screen_path, "filesScanned": scanned, "counts": counts, "keys": keys},
-        indent=2, ensure_ascii=False,
-    ) + "\n"
+    report = {"screenPath": opts.screen_path, "filesScanned": scanned, "counts": counts, "keys": keys}
+    if type_warning:
+        report["keyTypeWarning"] = type_warning
+    text = json.dumps(report, indent=2, ensure_ascii=False) + "\n"
     if opts.out:
         with open(opts.out, "w", encoding="utf-8") as sink:
             sink.write(text)
@@ -138,6 +158,8 @@ def main():
         sys.stdout.write(text)
     summary = ", ".join(f"{counts[name]} {name}" for name in CATEGORIES)
     print(f"# {scanned} file(s) read: {summary}", file=sys.stderr)
+    if type_warning:
+        print(f"# {type_warning}", file=sys.stderr)
 
 
 if __name__ == "__main__":

package/pipeline/skills/shared/external/localization-reuse-map/scripts/verify-map.py

@@ -12,14 +12,15 @@ import re
 import sys
 
 from _shared import (
-    LITERAL_KEY, WEB_DOLLAR_T, WEB_T, clean_key, is_component, is_dynamic, lsk_keys, normalize_key, raw_key,
-    source_files,
+    ACCESSOR_EXTENSIONS, DEFAULT_KEY_TYPES, LITERAL_KEY, WEB_DOLLAR_T, WEB_T, clean_key, is_component, is_dynamic,
+    key_type_warning, key_types, lsk_keys, normalize_key, raw_key, source_files,
 )
 
 
-def scan_code(root):
+def scan_code(root, types=DEFAULT_KEY_TYPES):
     keys = set()
     corpus = []
+    accessor_texts = []
     scanned = 0
     for path in source_files(root):
         try:
@@ -29,10 +30,12 @@ def scan_code(root):
             continue
         scanned += 1
         corpus.append(text.lower())
-        keys.update(key.lower() for key in lsk_keys(text))
+        if os.path.splitext(path)[1] in ACCESSOR_EXTENSIONS:
+            accessor_texts.append(text)
+        keys.update(key.lower() for key in lsk_keys(text, types))
         for pattern, group in ((LITERAL_KEY, 1), (WEB_T, 2), (WEB_DOLLAR_T, 2)):
             keys.update(hit.group(group).lower() for hit in pattern.finditer(text))
-    return keys, "\n".join(corpus), scanned
+    return keys, "\n".join(corpus), scanned, key_type_warning(accessor_texts, types, mapping_hint=True)
 
 
 def filled(value):
@@ -54,12 +57,26 @@ def main():
     parser.add_argument("--screen-path", required=True, help="folder or single file of the built screen")
     parser.add_argument("--resources-root", help="resources checkout or snapshot; turns on the Suggested-file check")
     parser.add_argument("--out", help="JSON report destination; stdout when omitted")
+    parser.add_argument(
+        "--key-type", action="append", metavar="NAME",
+        help="type whose member chains are keys; repeatable or comma list. Falls back to the mapping's "
+             f"keyType, then {', '.join(DEFAULT_KEY_TYPES)}",
+    )
     args = parser.parse_args()
 
     with open(args.mapping, encoding="utf-8") as stream:
         mapping = json.load(stream)
     rows = mapping.get("rows") or []
-    code_keys, corpus, scanned = scan_code(args.screen_path)
+    from_mapping = mapping.get("keyType") or []
+    if isinstance(from_mapping, str):
+        from_mapping = [from_mapping]
+    elif not (isinstance(from_mapping, list) and all(isinstance(name, str) for name in from_mapping)):
+        parser.error(f"keyType in the mapping must be a string or a list of strings, got {from_mapping!r}")
+    try:
+        types = key_types(args.key_type or from_mapping)
+    except ValueError as exc:
+        parser.error(str(exc))
+    code_keys, corpus, scanned, type_warning = scan_code(args.screen_path, types)
 
     verified = 0
     exempt = 0
@@ -113,6 +130,8 @@ def main():
         "valueGaps": gaps,
         "untranslated": untranslated,
     }
+    if type_warning:
+        report["keyTypeWarning"] = type_warning
     text = json.dumps(report, indent=2, ensure_ascii=False) + "\n"
     if args.out:
         with open(args.out, "w", encoding="utf-8") as sink:
@@ -128,6 +147,8 @@ def main():
         f"value gaps {len(gaps)} ({len(no_legacy)} hard), untranslated {len(untranslated)}",
         file=sys.stderr,
     )
+    if type_warning:
+        print(f"# {type_warning}", file=sys.stderr)
     for item in unverified[:12]:
         print(f"#   UNVERIFIED {item['key']}  ({item['element']})", file=sys.stderr)
     for key in not_in_map[:12]:
@@ -135,7 +156,7 @@ def main():
     if hard:
         print(f"# FAIL: {hard} hard issue(s); fix them or record a conscious waiver before publishing", file=sys.stderr)
         sys.exit(2)
-    print("# PASS", file=sys.stderr)
+    print("# PASS, but see the key type WARNING above" if type_warning else "# PASS", file=sys.stderr)
 
 
 if __name__ == "__main__":

package/pipeline/skills/shared/external/passkit-wallet/SKILL.md

@@ -14,10 +14,11 @@ charges.
 Examples assume Swift 6.3 and iOS 26.
 
 One rule shapes advanced checkouts: a single `PKPaymentRequest` may carry at
-most one optional advanced type. The candidates are a recurring request, an
-automatic-reload request, a deferred request, the Apple Pay Later setting, and
-multi-token contexts. When a flow
-needs two of them, build two requests.
+most one of `multiTokenContexts`, `recurringPaymentRequest`,
+`automaticReloadPaymentRequest` and `deferredPaymentRequest`. When a flow
+needs two of them, build two requests. `applePayLaterAvailability` (iOS 17+)
+is not part of that rule; it is a separate opt-out of Apple Pay Later that
+defaults to enabled.
 
 ## Setup
 

package/pipeline/skills/shared/external/passkit-wallet/references/wallet-passes.md

@@ -10,10 +10,11 @@ Each `PKPaymentRequest` may set only one of these:
 - `recurringPaymentRequest`
 - `automaticReloadPaymentRequest`
 - `deferredPaymentRequest`
-- the Apple Pay Later availability setting
 - `multiTokenContexts`
 
-If a checkout needs two, split it into two requests.
+If a checkout needs two, split it into two requests. `applePayLaterAvailability`
+(iOS 17+) sits outside this rule: it opts the request out of Apple Pay Later,
+and defaults to enabled; the exclusion covers only the four above.
 
 ## Recurring payments
 

package/pipeline/skills/shared/external/pencilkit-drawing/SKILL.md

@@ -133,8 +133,8 @@ drawing throws there. `PKContentVersion` names each step:
 |---|---|
 | `.version1` | Original inks: pen, pencil, marker (iPadOS 14 era) |
 | `.version2` | Inks added in iPadOS 17: `.monoline`, `.fountainPen`, `.watercolor`, `.crayon` |
-| `.version3` | Barrel-roll angle data on strokes |
-| `.version4` | Reed pen |
+| `.version3` | The new fountain pen ink (iOS 17.5, macOS 14.5) |
+| `.version4` | The new reed pen ink (iOS 26, macOS 26, visionOS 26) |
 
 When reviewing compatibility, lay out this whole table before recommending a
 cap, and if the fix is a curated picker, say that `PKToolPicker(toolItems:)`

package/pipeline/skills/shared/external/pencilkit-drawing/evals/evals.json

@@ -19,7 +19,7 @@
       "id": 1,
       "name": "older-ipad-sync",
       "prompt": "Teachers annotate worksheets on new iPads and students open them on iPads still running iPadOS 16. Students report that some worksheets fail to open. Our plan is to set maximumSupportedContentVersion to .version2 on the canvas. Review the plan.",
-      "expected_output": "Explains that inks newer than the recipient supports make loading fail, gives the full content version map (version1 pen/pencil/marker, version2 the iPadOS 17 inks, version3 barrel-roll data, version4 reed pen), and says .version2 does not help iPadOS 16, which needs .version1 or a verified fallback drawing. The cap must be set on both canvas and tool picker. Notes .reed needs 26 and curated picker items need iOS 18 or visionOS 2.",
+      "expected_output": "Explains that inks newer than the recipient supports make loading fail, gives the full content version map (version1 pen/pencil/marker, version2 the iPadOS 17 inks, version3 the new fountain pen from iOS 17.5, version4 the new reed pen), and says .version2 does not help iPadOS 16, which needs .version1 or a verified fallback drawing. The cap must be set on both canvas and tool picker. Notes .reed needs 26 and curated picker items need iOS 18 or visionOS 2.",
       "files": [],
       "expectations": [
         "States that unsupported inks make older systems fail to load the drawing",

package/pipeline/skills/shared/external/pencilkit-drawing/references/pencilkit-patterns.md

@@ -168,7 +168,7 @@ func describe(_ sketch: PKDrawing) -> String {
     switch sketch.requiredContentVersion {
     case .version1: "pen, pencil, marker"
     case .version2: "adds monoline, fountain pen, watercolor, crayon"
-    case .version3: "adds barrel-roll angles"
+    case .version3: "adds the new fountain pen (iOS 17.5)"
     case .version4: "adds reed pen"
     @unknown default: "unknown future content"
     }
@@ -215,9 +215,9 @@ func downgradedCopy(of sketch: PKDrawing) -> PKDrawing {
 }
 ```
 
-Swapping the ink is not always enough. The copied paths can still carry newer
-data such as barrel-roll angles, which is why the result is checked again
-before upload.
+Swapping the ink is not always enough. The copies keep every other stroke
+property, and only `requiredContentVersion` says what the result needs, which
+is why the result is checked again before upload.
 
 ## A configurable SwiftUI wrapper
 

package/pipeline/skills/shared/external/permissionkit/SKILL.md

@@ -32,10 +32,14 @@ against the Xcode 26 SDK you ship with before relying on it.
 Add `@preconcurrency import PermissionKit`. The SwiftUI and UIKit/AppKit
 pieces (`PermissionButton`, `AskCenter.ask(_:in:)`) come in through
 cross-import overlays, so importing SwiftUI or UIKit next to PermissionKit is
-enough. The framework's types carry no `Sendable` annotations, so under Swift 6
-a plain import turns every `ask(_:in:)` call from the main actor into a
-"sending 'question' risks causing data races" error; `@preconcurrency` is what
-lets the examples below compile.
+enough. Most of the framework's types are not `Sendable`: `PermissionQuestion`
+(a class), `PermissionResponse`, `PermissionChoice`, `CommunicationTopic`,
+`AskCenter` and `CommunicationLimits` declare no conformance. The exceptions
+are `CommunicationHandle`, which is `Sendable`, and the SwiftUI views
+`PermissionButton` and `CommunicationLimitsButton`. Because the question is not
+`Sendable`, under Swift 6 a plain import turns every `ask(_:in:)` call from the
+main actor into a "sending 'question' risks causing data races" error;
+`@preconcurrency` is what lets the examples below compile.
 
 Do not add made-up entitlement keys for PermissionKit. If a signing
 requirement exists, it will be in Apple's documentation and in the Xcode
@@ -177,7 +181,11 @@ those.
 in which the child sends the question; it does not return the parent's
 answer. Decisions arrive later on `responses(for:)`. If the child backs out of
 the send sheet, no `PermissionResponse` ever arrives, so a pending request
-needs its own expiry or retry.
+needs its own expiry or retry. `PermissionQuestion` has a settable
+`expirationDate: Date?` (nil by default); once that date passes, the parent
+can no longer respond. Set it before asking to bound the window, then move the
+local entry to expired at the same date, since the SDK delivers nothing when a
+question lapses.
 
 ```swift
 @available(iOS 26.2, *)
@@ -233,7 +241,8 @@ examples are in the
 
 Track each question by `question.id` (a `UUID`). Since a cancelled send
 produces nothing, give every pending entry a way out: a retry, an expiry, or
-both. When a flow combines known-handle checks with responses, keep the
+both. When the question carries an `expirationDate`, use that date for the
+local expiry so the app and the parent's window close together. When a flow combines known-handle checks with responses, keep the
 bundle-identifier guard from the section above.
 
 ```swift

package/pipeline/skills/shared/external/permissionkit/references/permissionkit-patterns.md

@@ -186,7 +186,8 @@ print(q.expirationDate as Any)           // nil unless set
 ```
 
 A question exposes `id`, `choices` (each with a `title`), `defaultChoice` and
-an optional `expirationDate`.
+an optional, settable `expirationDate`; after that date the parent can no
+longer respond.
 
 ## Limits-check view model
 

package/pipeline/skills/shared/external/push-notifications/SKILL.md

@@ -425,9 +425,13 @@ func handleChatAction(_ action: String, threadID: String?, typed: String?) async
 ## Grouping
 
 Notifications with the same `threadIdentifier` (local) or `thread-id` (APNs)
-stack into one group; each distinct value is its own group. `summaryArgument`
-and `summaryArgumentCount` on the content feed the group's summary line, and a
-category can shape that line:
+stack into one group; each distinct value is its own group. Grouping comes
+only from that identifier. `summaryArgument` and `summaryArgumentCount` on
+`UNMutableNotificationContent` are deprecated as ignored on iOS 15 and later
+(watchOS 8, tvOS 15); do not set them. A category can still supply a summary
+format through `categorySummaryFormat` (iOS 12+, not deprecated). The argument
+list in that format is optional, and with the per-notification argument ignored
+a count-only format is the one to rely on:
 
 ```swift
 let chat = UNNotificationCategory(
@@ -435,7 +439,7 @@ let chat = UNNotificationCategory(
     actions: [],
     intentIdentifiers: [],
     hiddenPreviewsBodyPlaceholder: nil,
-    categorySummaryFormat: "%u new messages from %@",
+    categorySummaryFormat: "%u new messages",
     options: []
 )
 ```

package/pipeline/skills/shared/external/push-notifications/references/notification-patterns.md

@@ -385,7 +385,7 @@ enum NotificationCategories {
             actions: [reply, markRead],
             intentIdentifiers: [],
             hiddenPreviewsBodyPlaceholder: nil,
-            categorySummaryFormat: "%u more from %@",
+            categorySummaryFormat: "%u more messages",
             options: [.customDismissAction]
         )
 

package/pipeline/skills/shared/external/realitykit-ar/SKILL.md

@@ -147,16 +147,35 @@ apart.
   entities that carry `CollisionComponent` shapes. Use it to pick entities and
   target gestures, never to find planes.
 - For simple placement on a detected plane, use `AnchorEntity(.plane(...))`.
-- To intersect a real surface once, build an ARKit `ARRaycastQuery`, run it
-  with `ARSession.raycast(_:)`, and anchor with `AnchorEntity(raycastResult:)`.
+  This is the placement path inside `RealityView`: `RealityViewCameraContent`
+  exposes no `ARSession`, so there is nothing to raycast against there.
+- To intersect a real surface at a tapped point, host an `ARView` (through
+  `UIViewRepresentable`), call its `raycast(from:allowing:alignment:)`, and
+  anchor with `AnchorEntity(raycastResult:)`. The same view offers
+  `makeRaycastQuery(from:allowing:alignment:)` when an `ARRaycastQuery` is
+  needed for `arView.session.raycast(_:)`.
 
 ```swift
+import RealityKit
+import SwiftUI
+
+struct PlaneDropView: View {
+    var body: some View {
+        RealityView { content in
+            content.camera = .spatialTracking
+            let floor = AnchorEntity(.plane(.horizontal, classification: .any, minimumBounds: [0.3, 0.3]))
+            floor.addChild(ModelEntity(mesh: .generateBox(size: 0.1)))
+            content.add(floor)
+        }
+    }
+}
+
 @MainActor
-func placeOnSurface(_ item: Entity, query: ARRaycastQuery, session: ARSession, into content: RealityViewCameraContent) {
-    guard let hit = session.raycast(query).first else { return }
+func placeOnSurface(_ item: Entity, at point: CGPoint, in arView: ARView) {
+    guard let hit = arView.raycast(from: point, allowing: .estimatedPlane, alignment: .any).first else { return }
     let anchor = AnchorEntity(raycastResult: hit)
     anchor.addChild(item)
-    content.add(anchor)
+    arView.scene.addAnchor(anchor)
 }
 ```
 
@@ -283,7 +302,7 @@ store it when you plan to cancel it later, or drop it with `_ =` as here.
 - [ ] Interaction notes explain that `CollisionComponent` provides the hittable shape, paired with `InputTargetComponent`
 - [ ] A Handoffs line mentions `AccessibilityComponent` and sends detailed SwiftUI and accessibility policy to the sibling skills
 - [ ] Entity hit tests, `ray(...)` and ARKit surface raycasts are kept distinct
-- [ ] Surface placement uses `ARSession.raycast(_:)` with `AnchorEntity(raycastResult:)`
+- [ ] Surface placement inside `RealityView` uses `AnchorEntity(.plane(...))`; a tap raycast uses `ARView.raycast(from:allowing:alignment:)` with `AnchorEntity(raycastResult:)`
 - [ ] Per-frame logic uses `SceneEvents.Update`, not SwiftUI timers
 - [ ] Large scenes load with async `ModelEntity(named:)`, not `Entity.load(named:)`
 - [ ] Anchor targets match the kind of surface the feature needs

package/pipeline/skills/shared/external/realitykit-ar/evals/evals.json

@@ -23,7 +23,7 @@
         "States that hitTest finds collision shapes on virtual entities, not physical surfaces",
         "States that ray(through:in:to:) is not evidence of a real surface",
         "Suggests AnchorEntity(.plane(...)) for simple plane placement",
-        "Uses ARRaycastQuery with ARSession.raycast(_:) and AnchorEntity(raycastResult:) for one-shot placement",
+        "Notes that RealityViewCameraContent exposes no ARSession and uses ARView.raycast(from:allowing:alignment:) with AnchorEntity(raycastResult:) for one-shot tap placement",
         "Keeps collision shapes for entity interaction only"
       ]
     },

package/pipeline/skills/shared/external/skill-creator/template.md

@@ -1,6 +1,6 @@
 # Skill templates
 
-Two skeletons. Copy the one that matches the layer, fill it, then delete every line that fails the cut test. The toolkits use these frontmatter keys: `name`, `description`, `allowed-tools`, `user-invocable`, plus an optional `argument-hint` and an optional `metadata` block with `version` (semver of the skill text) and `source` (`multi-agent-pipeline` for a first-party skill, otherwise the upstream it derives from, which a `NOTICE-*.md` beside it must list). Drop the block rather than leave a placeholder in it.
+Two skeletons. Copy the one that matches the layer, fill it, then delete every line that fails the cut test. The toolkits use these frontmatter keys: `name`, `description`, `allowed-tools`, `user-invocable`, plus an optional `argument-hint` and a `metadata` block with `version` (semver of the skill text) and `source` (`multi-agent-pipeline` for a first-party skill, otherwise `<upstream> (<license>)` for the upstream it derives from, which a `NOTICE-*.md` beside it must list). For a shared knowledge skill in the pipeline source the block with `source` is required: the pipeline's skill lint fails one that has none. Elsewhere the block is optional; drop it rather than leave a placeholder in it.
 
 ## Reference skill
 

package/pipeline/skills/shared/external/vision-framework/SKILL.md

@@ -44,7 +44,9 @@ optional.
 
 Most requests are value types. Requests that must remember earlier frames are
 `final class` types conforming to `StatefulRequest`: `TrackObjectRequest`,
-`TrackRectangleRequest`, `TrackOpticalFlowRequest`, `DetectTrajectoriesRequest`,
+`TrackRectangleRequest`, `TrackOpticalFlowRequest`,
+`TrackTranslationalImageRegistrationRequest`,
+`TrackHomographicImageRegistrationRequest`, `DetectTrajectoriesRequest`,
 `DetectHumanBodyPose3DRequest` and `GeneratePersonSegmentationRequest`. Keep one
 instance alive for the whole sequence.