@mmerterden/multi-agent-pipeline 13.2.0 → 13.3.0

package/pipeline/skills/shared/external/ios-coding-standard/references/swiftlint.draft.yml

@@ -1,4 +1,4 @@
-# SwiftLint configuration generated from rules.yml — LOCAL USE ONLY.
+# SwiftLint configuration generated from rules.yml  -  LOCAL USE ONLY.
 #
 # By decision this is NOT part of the project: no committed .swiftlint.yml, no build phase, no
 # CI job. Run it through ./lint-local.sh, which resolves the placeholders below and keeps the
@@ -8,7 +8,7 @@
 #   ./lint-local.sh Domains/<Module> --strict           # then only new violations surface
 #
 # Because there is no CI gate, the ratchet is a habit rather than a wall: run it before opening
-# a PR. That is weaker than a gate and the audit says so — adopting it project-wide stays an
+# a PR. That is weaker than a gate and the audit says so  -  adopting it project-wide stays an
 # open decision, not a forgotten one.
 #
 # Every custom rule identifier IS its rules.yml ID, lowercased, so a CI failure points
@@ -52,13 +52,13 @@ opt_in_rules:
   - prohibited_super_call
 
 disabled_rules:
-  # WARN-02 owns TODO policy (untagged only); the built-in flags every TODO — pure noise.
+  # WARN-02 owns TODO policy (untagged only); the built-in flags every TODO  -  pure noise.
   - todo
   - line_length
   - trailing_comma
   - opening_brace
   - statement_position
-  # SwiftUI's own presentation API is `sheet(isPresented:onDismiss:) { content }` — a closure
+  # SwiftUI's own presentation API is `sheet(isPresented:onDismiss:) { content }`  -  a closure
   # argument plus a trailing view builder. The rule fires on Apple's documented shape, so
   # obeying it would make every sheet read worse than the framework intends.
   - multiple_closures_with_trailing_closure
@@ -74,14 +74,14 @@ function_body_length:
   warning: 40
   error: 100
 
-# READ-07 — the component names are bound to an external design mapping and cannot be
+# READ-07  -  the component names are bound to an external design mapping and cannot be
 # shortened, so the ceiling reflects the codebase rather than SwiftLint's 40-char default.
 type_name:
   max_length:
     warning: 62
     error: 75
 
-# STRUCT-01 — a pure constants namespace groups by nesting (`AppConstant.DeepLink.Path`);
+# STRUCT-01  -  a pure constants namespace groups by nesting (`AppConstant.DeepLink.Path`);
 # two levels is the depth that grouping needs, and nothing deeper is allowed.
 nesting:
   type_level: 2
@@ -92,14 +92,14 @@ identifier_name:
 cyclomatic_complexity:
   warning: 10
 
-# READ-09 — the built-in takes no per-rule `excluded`; Package.swift is excluded globally below
+# READ-09  -  the built-in takes no per-rule `excluded`; Package.swift is excluded globally below
 file_header:
-  # Author is the FULL git identity, department included — "NAME - <Department> Mudurlugu".
+  # Author is the FULL git identity, department included  -  "NAME - <Department> Mudurlugu".
   # A bare name is not enough: it does not say which team owns the file.
   required_pattern: '\A//\n//  .*\.swift\n//  .*\n//(?:\n//.*)*?\n//  Created by .+ - .+ on \d{2}\.\d{2}\.\d{4}\.\n//\n'
 
-# VIS-04 — only the module's public surface must be documented
-# An extension is a grouping, not an API — its members carry the docs. Requiring a comment on
+# VIS-04  -  only the module's public surface must be documented
+# An extension is a grouping, not an API  -  its members carry the docs. Requiring a comment on
 # the `extension` line itself only produces restatements of the MARK above it. A memberwise
 # init is likewise self-describing.
 missing_docs:
@@ -120,7 +120,7 @@ custom_rules:
   sec_01_no_over_persistence:
     name: "SEC-01 transient value being persisted"
     regex: '(credentialStore|keychain|UserDefaults|FileManager)[^\n]*\b((TRANSIENT))\b'
-    message: "This value is required-transient. Persisting it — Keychain included — creates a cleanup obligation with no owner. Hold it in the flow's model and drop it."
+    message: "This value is required-transient. Persisting it  -  Keychain included  -  creates a cleanup obligation with no owner. Hold it in the flow's model and drop it."
     severity: error
 
   sec_01_keychain_sync:
@@ -132,7 +132,7 @@ custom_rules:
   sec_01_keychain_accessibility_missing:
     name: "SEC-01 keychain write without an accessibility class"
     regex: 'kSecClassGenericPassword(?:(?!kSecAttrAccessible)[\s\S]){0,400}SecItemAdd'
-    message: "Declare kSecAttrAccessible explicitly — device-only unless there is a documented reason."
+    message: "Declare kSecAttrAccessible explicitly  -  device-only unless there is a documented reason."
     severity: error
 
   sec_02_hardcoded_secret:
@@ -174,10 +174,10 @@ custom_rules:
 
   sec_09_commented_out_compilation_condition:
     name: "SEC-09 compilation condition commented out"
-    # [ \t]* not \s* — \s matches newlines, so "//" ending a file header plus a real
+    # [ \t]* not \s*  -  \s matches newlines, so "//" ending a file header plus a real
     # "#if" two lines below would match as one commented-out directive.
     regex: '^[ \t]*//[ \t]*#(if|endif|else)\b'
-    message: "A commented-out #if DEBUG is not a guard — the code inside it ships. Restore the directive or delete the code."
+    message: "A commented-out #if DEBUG is not a guard  -  the code inside it ships. Restore the directive or delete the code."
     severity: error
 
   # ── TEST ────────────────────────────────────────────────────────────────
@@ -238,16 +238,16 @@ custom_rules:
 
   # ── CONC ────────────────────────────────────────────────────────────────
   # Production code only. A test double is constructed inside one test and mutated by that
-  # test's single task, so the hatch carries no concurrency risk there — flagging every spy
+  # test's single task, so the hatch carries no concurrency risk there  -  flagging every spy
   # would bury the production hits that do matter.
   conc_03_escape_hatch:
     name: "CONC-03 concurrency escape hatch"
     regex: '(@preconcurrency|nonisolated\(unsafe\)|@unchecked\s+Sendable)'
     included: ".*/Sources/.*"
-    message: "Migration tool, not a design tool. Add a standard:exception marker with a removal condition — these are counted."
+    message: "Migration tool, not a design tool. Add a standard:exception marker with a removal condition  -  these are counted."
     severity: warning
 
-  # SVC-07 — a repository protocol method names the service it calls. The legitimate
+  # SVC-07  -  a repository protocol method names the service it calls. The legitimate
   # exceptions (variants over one service, no endpoint yet) are meant to surface here and be
   # answered with a standard:exception marker, not to be silently allowed by the pattern.
   svc_07_service_named_method:
@@ -257,7 +257,7 @@ custom_rules:
     message: "Name it `send<ServiceName>` (check-open-status -> sendCheckOpenStatus), or mark it standard:exception(SVC-07) with the reason."
     severity: warning
 
-  # SVC-08 — arithmetic or a policy default inside a mapper. Optional lowering (`?? ""`,
+  # SVC-08  -  arithmetic or a policy default inside a mapper. Optional lowering (`?? ""`,
   # `?? false`, `?? []`, `?? nil`) is deliberately not matched; a literal number or a division is.
   svc_08_rule_in_mapper:
     name: "SVC-08 calculation or policy default inside a mapper"
@@ -266,24 +266,24 @@ custom_rules:
     message: "A mapper moves values. Carry the wire value and do the arithmetic, clamp or default in the view model."
     severity: warning
 
-  # READ-04b — a component file under Presentation/Components must carry a preview. Reported
+  # READ-04b  -  a component file under Presentation/Components must carry a preview. Reported
   # per file that declares a View and has no #Preview.
   # READ-04b is NOT a swiftlint rule: a regex matches a LINE, and "this file has no #Preview"
   # is a statement about the whole file. Matching `struct X: View {` under Components/ flags
-  # every component, previewed or not — 25 false positives on first run. The check lives in the
+  # every component, previewed or not  -  25 false positives on first run. The check lives in the
   # standard's structural pass instead (component files vs files containing #Preview).
 
-  # PLAT-01 — raw SwiftUI presentation in a feature module. The app ships an OS-independent
+  # PLAT-01  -  raw SwiftUI presentation in a feature module. The app ships an OS-independent
   # wrapper; the system API drifts with every iOS release (the iOS 26 partial-detent inset).
   plat_01_raw_presentation:
     name: "PLAT-01 raw system presentation instead of the app's sheet"
     regex: '(?<![\w.])\.sheet\('
     included: ".*/Sources/.*"
     excluded_match_kinds: [comment, doccomment]
-    message: "Use the app's own sheet wrapper. System .sheet only for OS-owned surfaces (share, Siri, photo picker) — mark those standard:exception(PLAT-01)."
+    message: "Use the app's own sheet wrapper. System .sheet only for OS-owned surfaces (share, Siri, photo picker)  -  mark those standard:exception(PLAT-01)."
     severity: warning
 
-  # PLAT-03 — an asset URL on someone else's host.
+  # PLAT-03  -  an asset URL on someone else's host.
   plat_03_third_party_asset_host:
     name: "PLAT-03 asset URL on a third-party host"
     regex: 'https?://(?!([\w.-]*\.)?(example\.com|cdn\.example\.com))[\w.-]+/[^"]*\.(png|jpg|jpeg|svg|webp|gif)'
@@ -341,7 +341,7 @@ custom_rules:
   perf_04_formatter_per_render:
     name: "PERF-04 formatter constructed per render"
     regex: '(DateFormatter|NumberFormatter|RelativeDateTimeFormatter)\(\)'
-    message: "Hoist it out of the render path — construction is expensive and repeats every update."
+    message: "Hoist it out of the render path  -  construction is expensive and repeats every update."
     severity: warning
     included: ".*(View|Scene)\\.swift"
 
@@ -361,7 +361,7 @@ custom_rules:
   warn_02_untagged_todo:
     name: "WARN-02 TODO without an owner or ticket"
     regex: '//\s*TODO(?!\[)'
-    message: "Tag it — // TODO[TICKET-123]: … — or it is a note to someone who has left."
+    message: "Tag it  -  // TODO[TICKET-123]: ...  -  or it is a note to someone who has left."
     severity: warning
 
   warn_02_fixme: