add single_line_body opt-in format rule; bridge global format config to rules via FormatAwareRule protocol (#188)

Author: Jason-Abbott

.issues/8/8ak-rh2--standardize-rule-names-for-consistency.md

@@ -0,0 +1,71 @@
+---
+# 8ak-rh2
+title: Standardize rule names for consistency
+status: in-progress
+type: task
+priority: normal
+created_at: 2026-04-11T19:12:38Z
+updated_at: 2026-04-11T19:12:38Z
+sync:
+    github:
+        issue_number: "189"
+        synced_at: "2026-04-11T19:12:48Z"
+---
+
+Rename ~25 rules for naming consistency across three merged rule sources.
+
+## Changes
+
+### 1. Synonym Standardization (redundant/unneeded/unnecessary/superfluous → redundant)
+- [ ] UnneededOverrideRule → RedundantOverrideRule
+- [ ] UnneededEscapingRule → RedundantEscapingRule
+- [ ] UnneededSynthesizedInitializerRule → RedundantSynthesizedInitializerRule
+- [ ] UnneededBreakInSwitchRule → RedundantBreakInSwitchRule
+- [ ] UnneededThrowsRule → RedundantThrowsRule (id: redundant_throws)
+- [ ] UnneededParenthesesInClosureArgumentRule → RedundantClosureArgumentParensRule
+- [ ] UnnecessaryFileprivateRule → RedundantFileprivateRule
+- [ ] SuperfluousElseRule → RedundantElseRule
+- [ ] SuperfluousDisableCommandRule → RedundantDisableCommandRule
+
+### 2. Verb Tense
+- [ ] SortedEnumCasesRule → SortEnumCasesRule
+- [ ] SortedFirstLastRule → MinMaxOverSortedRule
+
+### 3. Gerund vs Noun
+- [ ] ForceUnwrappingRule → ForceUnwrapRule
+
+### 4. Prefer prefix
+- [ ] PreferSelfTypeOverTypeOfSelfRule → SelfTypeOverTypeOfSelfRule
+- [ ] PreferZeroOverExplicitInitRule → ZeroOverExplicitInitRule
+
+### 5. Type/ID mismatch
+- [ ] EnumCaseAssociatedValuesLengthRule → EnumCaseAssociatedValuesCountRule
+
+### 6. Long names
+- [ ] VerticalParameterAlignmentOnCallRule → CallParameterAlignmentRule
+- [ ] AnonymousArgumentInMultilineClosureRule → MultilineClosureAnonymousArgumentRule
+- [ ] EmptyParenthesesWithTrailingClosureRule → TrailingClosureEmptyParensRule
+- [ ] MultipleClosuresWithTrailingClosureRule → MultipleTrailingClosuresRule
+- [ ] RawValueForCamelCasedCodableEnumRule → CodableEnumRawValueRule
+- [ ] ContainsOverRangeNilComparisonRule → ContainsOverRangeCheckRule
+- [ ] NonOptionalStringDataConversionRule → StringDataConversionRule
+- [ ] BlankLinesBetweenChainedFunctionsRule → NoBlankLineInChainRule
+
+### 7. Vague names
+- [ ] CaptureVariableRule → ImplicitSelfCaptureRule
+- [ ] ControlStatementRule → ControlStatementParensRule
+- [ ] LetVarWhitespaceRule → DeclarationWhitespaceRule
+- [ ] AttributesRule → AttributePlacementRule
+
+### 8. Spacing vs Whitespace
+- [ ] ReturnArrowWhitespaceRule → ReturnArrowSpacingRule
+- [ ] FunctionNameWhitespaceRule → FunctionNameSpacingRule
+- [ ] OperatorUsageWhitespaceRule → OperatorUsageSpacingRule
+
+### 9. Plural
+- [ ] BlankLinesAfterGuardStatementsRule → BlankLineAfterGuardRule
+
+### Final
+- [ ] Run GeneratePipeline
+- [ ] Build succeeds
+- [ ] Tests pass

.issues/9/9qw-72x--improve-rule-summaries-fill-empty-static-let-summa.md

@@ -1,15 +1,15 @@
 ---
 # 9qw-72x
 title: 'Improve rule summaries: fill empty static let summary fields'
-status: ready
+status: completed
 type: task
 priority: normal
 created_at: 2026-04-11T18:22:20Z
-updated_at: 2026-04-11T18:22:20Z
+updated_at: 2026-04-11T18:55:20Z
 sync:
     github:
         issue_number: "181"
-        synced_at: "2026-04-11T18:44:01Z"
+        synced_at: "2026-04-11T19:12:32Z"
 ---
 
 6+ rules have `static let summary = ""`. Fill them in with concise, useful descriptions.
@@ -25,6 +25,42 @@ Identified rules with empty summaries:
 
 ## Tasks
 
-- [ ] Find all rules with empty summary fields
-- [ ] Write concise summaries for each
-- [ ] Verify build passes
+- [x] Find all rules with empty summary fields (found 16)
+- [x] Write concise summaries for each
+- [x] Verify build passes
+
+
+## Summary of Changes
+
+Filled in 16 empty rule summaries and improved 9 existing ones that were vague or awkwardly phrased.
+
+### 16 new summaries
+- `number_separator` — Underscores should be used as thousand separators in large decimal numbers
+- `unhandled_throwing_task` — Errors thrown inside a Task are silently lost unless the result is checked
+- `non_overridable_class_declaration` — Use `static` or `final` instead of `class` for non-overridable members
+- `superfluous_disable_command` — Disable commands for rules that didn't trigger a violation are superfluous
+- `blanket_disable_command` — Disable commands should be scoped with `next`, `this`, or `previous` instead of the whole file
+- `static_over_final_class` — Prefer `static` over `final class` or `class` in a final class
+- `shorthand_argument` — Shorthand closure arguments (`$0`, `$1`) should be used sparingly and close to the closure opening
+- `inclusive_language` — Identifiers should use inclusive language that avoids discrimination
+- `unused_parameter` — Unused function parameters should be removed or replaced with `_`
+- `type_name` — Type names should be alphanumeric, start uppercase, and have a reasonable length
+- `contrasted_opening_brace` — Opening braces should be on a separate line from the preceding declaration
+- `opening_brace` — Opening braces should be preceded by a single space and on the same line as the declaration
+- `switch_case_alignment` — Case statements should vertically align with their enclosing switch's closing brace
+- `colon` — Colons should have no space before and one space after
+- `attribute_name_spacing` — There should be no space between an attribute or modifier and its arguments
+- `attributes` — Attributes should be on their own line for functions and types, same line for variables and imports
+
+### 9 improved summaries
+- `invalid_command` — was "sm: command is invalid"
+- `unused_setter_value` — was "Setter value is not used"
+- `joined_default_parameter` — was "Discouraged explicit usage of the default separator"
+- `file_types_order` — was "Specifies how the types within a file should be ordered."
+- `type_contents_order` — was "Specifies the order of subtypes, properties, methods & more within a type."
+- `xct_specific_matcher` — was "Prefer specific XCTest matchers."
+- `discouraged_direct_init` — was "Discouraged direct initialization of types that can be harmful"
+- `syntactic_sugar` — was "Shorthand syntactic sugar should be used, i.e. [Int] instead of Array<Int>."
+- `self_binding` — was "Re-bind `self` to a consistent identifier name."
+
+Confirmed: `summary` is the only description field on the `Rule` protocol — there is no separate `description` property. They are the same thing.

.issues/y/ypa-vrg--single-line-body-format-rule.md

@@ -0,0 +1,53 @@
+---
+# ypa-vrg
+title: Single-line body format rule
+status: completed
+type: feature
+priority: normal
+created_at: 2026-04-11T19:01:44Z
+updated_at: 2026-04-11T19:11:37Z
+sync:
+    github:
+        issue_number: "188"
+        synced_at: "2026-04-11T19:12:32Z"
+---
+
+Add a `single_line_body` format rule that condenses any braced block with a single statement onto one line when it fits within the print width.
+
+## Requirements
+
+- [x] Add `FormatAwareRule` protocol to bridge global `format.max_width` to rules
+- [x] Update `RuleResolver.loadRules` with `formatDefaults` injection
+- [x] Update call sites to pass format defaults
+- [x] Create `SingleLineBodyOptions` with `max_width`
+- [x] Create `SingleLineBodyRule` with Visitor + Rewriter
+- [x] Run `swift run GeneratePipeline`
+- [x] Create tests (20 passing)
+- [x] Build and test (full suite — 474 pass, 3 pre-existing failures in unrelated rule)
+
+## Constructs handled
+
+- guard, if, for, while, do, defer, catch (`CodeBlockSyntax`)
+- closures (`ClosureExprSyntax`)
+- getter-only computed properties (`AccessorBlockSyntax`)
+- willSet/didSet/get/set (`AccessorDeclSyntax`)
+
+## Width calculation
+
+Visual start column (tabs × indent width + spaces) + flattened single-line length ≤ max_width
+
+
+
+## Summary of Changes
+
+Added `single_line_body` opt-in format rule that condenses any single-statement braced block to one line when it fits within `format.max_width`.
+
+**New files:**
+- `SingleLineBodyRule.swift` — Rule + Visitor (4 node types) + Rewriter
+- `SingleLineBodyOptions.swift` — `max_width` option
+- `SingleLineBodyRuleTests.swift` — 20 tests
+
+**Infrastructure:**
+- `FormatAwareRule` protocol in `Rule.swift` — lets rules declare which global format keys they need
+- `RuleResolver.loadRules` gains `formatDefaults` parameter — injects global format config into FormatAwareRule conformers
+- Call sites (`SwiftiomaticCLI`, `FormatCommand`) pass `formatDefaults`

CHANGELOG.md

@@ -9,6 +9,7 @@
 - Nested per-directory configuration ([#169](https://github.com/toba/swiftiomatic/issues/169))
 - Add `dump-config` CLI subcommand to show resolved configuration
 - Support file-level `sm:disable:file` scope
+- `single_line_body` format rule; condense single-statement blocks to one line ([#188](https://github.com/toba/swiftiomatic/issues/188))
 
 ### 🐛 Fixes
 

Sources/SwiftiomaticCLI/FormatCommand.swift

@@ -133,6 +133,7 @@ struct FormatCommand: ParsableCommand {
       enabled: cfg.enabledLintRules.isEmpty ? nil : Set(cfg.enabledLintRules),
       disabled: Set(cfg.disabledLintRules),
       ruleConfigs: cfg.lintRuleConfigs,
+      formatDefaults: ["max_width": cfg.formatMaxWidth],
     )
     let correctableRules = allRules.filter { type(of: $0).isCorrectable }
     guard !correctableRules.isEmpty else { return 0 }

Sources/SwiftiomaticCLI/SwiftiomaticCLI.swift

@@ -108,6 +108,7 @@ struct Analyze: AsyncParsableCommand {
       disabled: mergedDisabled,
       onlyRules: Set(onlyRule),
       ruleConfigs: cfg.lintRuleConfigs,
+      formatDefaults: ["max_width": cfg.formatMaxWidth],
     )
 
     let analyzer = Analyzer(

Sources/SwiftiomaticKit/Rules/AccessControl/Modifiers/NonOverridableClassDeclarationRule.swift

@@ -3,7 +3,7 @@ import SwiftSyntax
 struct NonOverridableClassDeclarationRule {
   static let id = "non_overridable_class_declaration"
   static let name = "Class Declaration in Final Class"
-  static let summary = ""
+  static let summary = "Use `static` or `final` instead of `class` for non-overridable members"
   static let isCorrectable = true
   static let isOptIn = true
   static var nonTriggeringExamples: [Example] {

Sources/SwiftiomaticKit/Rules/AccessControl/Modifiers/StaticOverFinalClassRule.swift

@@ -3,7 +3,7 @@ import SwiftSyntax
 struct StaticOverFinalClassRule {
   static let id = "static_over_final_class"
   static let name = "Static Over Final Class"
-  static let summary = ""
+  static let summary = "Prefer `static` over `final class` or `class` in a final class"
   static var nonTriggeringExamples: [Example] {
     [
       Example(

Sources/SwiftiomaticKit/Rules/ControlFlow/Closures/SelfBindingRule.swift

@@ -5,7 +5,7 @@ import SwiftSyntax
 struct SelfBindingRule {
   static let id = "self_binding"
   static let name = "Self Binding"
-  static let summary = "Re-bind `self` to a consistent identifier name."
+  static let summary = "Rebind `self` to a consistent identifier name in closures"
   static let isCorrectable = true
   static let isOptIn = true
   static var nonTriggeringExamples: [Example] {

Sources/SwiftiomaticKit/Rules/ControlFlow/Closures/ShorthandArgumentRule.swift

@@ -3,7 +3,7 @@ import SwiftSyntax
 struct ShorthandArgumentRule {
   static let id = "shorthand_argument"
   static let name = "Shorthand Argument"
-  static let summary = ""
+  static let summary = "Shorthand closure arguments (`$0`, `$1`) should be used sparingly and close to the closure opening"
   static let isOptIn = true
   static var nonTriggeringExamples: [Example] {
     [