refactor: reorganize directory structure for PostgreSQL Chapter 9 alignment

Cleaned up directory structure to eliminate empty placeholder directories
and align aggregate functions with PostgreSQL documentation organization.

Changes:
- Moved StandardAggregates.swift to Functions/Aggregate/ directory
- Removed empty Aggregates/ directory
- Deleted 7 empty Statements/ subdirectories (Common, Delete, Insert, Select, Update, Views, CommonTableExpressions)
- Deleted 5 empty Types/ subdirectories (Boolean, DateTime, Numeric, String, UUID)
- Deleted empty Functions/UUID/ directory

Benefits:
- Clear PostgreSQL Chapter 9 structure - all aggregates now in Functions/Aggregate/
- No ambiguity about where code belongs
- Directory structure matches official PostgreSQL documentation
- Zero empty directories creating confusion

Build status: ✅ All builds pass
Test status: ✅ All 280+ tests pass

WIP

WIP

fix .desc syntax for windows

Author: coenttb

Architecture.md

@@ -1424,7 +1424,7 @@ Conducted comprehensive audit against [PostgreSQL SELECT Documentation](https://
          .window("dept_salary") {
              WindowSpec()
                  .partition(by: $0.department)
-                 .order(by: $0.salary, .desc)
+                 .order(by: $0.salary.desc())
          }
          .select {
              (
@@ -1448,10 +1448,10 @@ Conducted comprehensive audit against [PostgreSQL SELECT Documentation](https://
          .window("list_order") {
              WindowSpec()
                  .partition(by: $0.remindersListID)
-                 .order(by: $0.title, .desc)
+                 .order(by: $0.title.desc())
          }
          .window("overall_order") {
-             WindowSpec().order(by: $0.title, .desc)
+             WindowSpec().order(by: $0.title.desc())
          }
          .select {
              (
@@ -1473,7 +1473,7 @@ Conducted comprehensive audit against [PostgreSQL SELECT Documentation](https://
          .window("list_window") {
              WindowSpec()
                  .partition(by: $0.remindersListID)
-                 .order(by: $0.title, .desc)
+                 .order(by: $0.title.desc())
          }
          .select {
              let id = $0.id

Package.swift

@@ -49,9 +49,13 @@ let package = Package(
             description:
                 "Enable SQL syntax validation against PostgreSQL using postgres-nio. Heavy dependency - only enable for validation testing."
         ),
-        .default(enabledTraits: [
-            "StructuredQueriesPostgresCasePaths", "StructuredQueriesPostgresSQLValidation",
-        ]),
+        .default(
+            enabledTraits: [
+                "StructuredQueriesPostgresCasePaths",
+//                "StructuredQueriesPostgresTagged",
+                "StructuredQueriesPostgresSQLValidation",
+            ]
+        ),
     ],
     dependencies: [
         .package(url: "https://github.com/pointfreeco/swift-case-paths", from: "1.0.0"),

README.md

@@ -27,11 +27,12 @@ struct User {
     let id: Int
     var name: String
     var email: String
+    var isActive: Bool = true
 }
 
 // Build query (this package)
 let statement = User
-    .where { $0.email.hasSuffix("@example.com") }
+    .where { $0.isActive && $0.email.hasSuffix("@example.com") }
     .order(by: \.name)
     .limit(10)
 
@@ -89,7 +90,11 @@ User
     .where { $0.isActive }
     .order(by: \.name)
     .limit(10)
-// SQL: SELECT * FROM "users" WHERE "users"."isActive" ORDER BY "users"."name" LIMIT 10
+// SQL: SELECT "users"."id", "users"."name", "users"."email", "users"."isActive"
+//      FROM "users"
+//      WHERE "users"."isActive"
+//      ORDER BY "users"."name"
+//      LIMIT 10
 ```
 
 **INSERT with Draft types** (PostgreSQL NULL PRIMARY KEY handling):
@@ -99,14 +104,14 @@ User
 User.insert {
     User.Draft(name: "Alice", email: "alice@example.com")
 }
-// SQL: INSERT INTO "users" ("name", "email") VALUES ('Alice', 'alice@example.com')
+// SQL: INSERT INTO "users" ("id", "name", "email", "isActive") VALUES (DEFAULT, 'Alice', 'alice@example.com', true)
 
 // Mixed records - uses DEFAULT for NULL PKs
 User.insert {
     User(id: 1, name: "Alice", email: "alice@example.com")
     User.Draft(name: "Bob", email: "bob@example.com")
 }
-// SQL: INSERT INTO "users" ("id", "name", "email") VALUES (1, 'Alice', 'alice@example.com'), (DEFAULT, 'Bob', 'bob@example.com')
+// SQL: INSERT INTO "users" ("id", "name", "email", "isActive") VALUES (1, 'Alice', 'alice@example.com', true), (DEFAULT, 'Bob', 'bob@example.com', true)
 ```
 
 **UPDATE with RETURNING:**
@@ -131,37 +136,35 @@ User
 
 ### PostgreSQL JSONB
 
-**Containment operators (@>, <@):**
+**Containment operator (@>):**
 
 ```swift
-struct UserSettings: Codable {
-    var theme: String
-    var notifications: Bool
-}
-
 @Table
 struct User {
     let id: Int
-    var settings: UserSettings
+    var name: String
+
+    @Column(as: Data.self)
+    var settings: Data
 }
 
 // Find users with dark theme
-User.where { $0.settings.contains(UserSettings(theme: "dark", notifications: true)) }
-// SQL: WHERE "users"."settings" @> '{"theme":"dark","notifications":true}'
+User.where { $0.settings.contains(["theme": "dark"]) }
+// SQL: WHERE "users"."settings" @> '{"theme":"dark"}'::jsonb
 ```
 
-**JSON path operators (->, ->>):**
+**JSON path operator (->>):**
 
 ```swift
-User.where { $0.settings.getText("theme") == "dark" }
+User.where { $0.settings.fieldAsText("theme") == "dark" }
 // SQL: WHERE ("users"."settings" ->> 'theme') = 'dark'
 ```
 
-**GIN indexing for fast JSONB queries:**
+**Key existence operator (?):**
 
 ```swift
-User.createGINIndex(on: \.settings, operatorClass: .jsonb_path_ops)
-// SQL: CREATE INDEX "users_settings_gin_idx" ON "users" USING GIN ("settings" jsonb_path_ops)
+User.where { $0.settings.hasKey("notifications") }
+// SQL: WHERE "users"."settings" ? 'notifications'
 ```
 
 ### Window Functions
@@ -172,32 +175,35 @@ User.createGINIndex(on: \.settings, operatorClass: .jsonb_path_ops)
 @Table
 struct Employee {
     let id: Int
+    var name: String
     var department: String
     var salary: Double
 }
 
 Employee
     .select {
-        (
+        let department = $0.department
+        let salary = $0.salary
+        return (
             $0.name,
-            $0.salary,
+            salary,
             rank().over {
-                $0.partition(by: $1.department)
-                    .order(by: $1.salary, .desc)
+                $0.partition(by: department)
+                    .order(by: salary.desc())
             }
         )
     }
-// SQL: SELECT "name", "salary", RANK() OVER (PARTITION BY "department" ORDER BY "salary" DESC)
+// SQL: SELECT "employees"."name", "employees"."salary", RANK() OVER (PARTITION BY "employees"."department" ORDER BY "employees"."salary" DESC)
+//      FROM "employees"
 ```
 
 **Named windows (WINDOW clause):**
 
 ```swift
 Employee
     .window("dept_salary") {
-        WindowSpec()
-            .partition(by: $0.department)
-            .order(by: $0.salary, .desc)
+        $0.partition(by: $1.department)
+            .order(by: $1.salary.desc())
     }
     .select {
         (
@@ -207,23 +213,9 @@ Employee
             rowNumber().over("dept_salary")
         )
     }
-// SQL: SELECT "name", RANK() OVER dept_salary, DENSE_RANK() OVER dept_salary, ROW_NUMBER() OVER dept_salary
+// SQL: SELECT "employees"."name", RANK() OVER dept_salary, DENSE_RANK() OVER dept_salary, ROW_NUMBER() OVER dept_salary
 //      FROM "employees"
-//      WINDOW dept_salary AS (PARTITION BY "department" ORDER BY "salary" DESC)
-```
-
-**Running totals with SUM() window function:**
-
-```swift
-Sale
-    .select {
-        (
-            $0.date,
-            $0.amount,
-            sum($0.amount).over { $0.order(by: $1.date) }
-        )
-    }
-// SQL: SELECT "date", "amount", SUM("amount") OVER (ORDER BY "date")
+//      WINDOW dept_salary AS (PARTITION BY "employees"."department" ORDER BY "employees"."salary" DESC)
 ```
 
 ### Triggers
@@ -235,6 +227,8 @@ Sale
 struct Product {
     let id: Int
     var name: String
+    var price: Double
+    var stock: Int
     var updatedAt: Date?
 }
 

Sources/StructuredQueriesCore/FrameClause.swift

@@ -209,7 +209,7 @@ extension WindowSpec {
     /// ```swift
     /// Leaderboard.select {
     ///     ($0.player, $0.score, $0.score.count().over {
-    ///         $0.order(by: $0.score, .desc)
+    ///         $0.order(by: $0.score.desc())
     ///           .groups(between: .currentRow, and: .unboundedFollowing)
     ///     })
     /// }

Sources/StructuredQueriesCore/Statements/Select/Select+Window.swift

@@ -11,7 +11,7 @@ extension Select {
     /// ```swift
     /// Employee
     ///     .window("dept_window") { spec, cols in
-    ///         spec.partition(by: cols.department).order(by: cols.salary, .desc)
+    ///         spec.partition(by: cols.department).order(by: cols.salary.desc())
     ///     }
     ///     .select {
     ///         ($0.name, rank().over("dept_window"))
@@ -23,7 +23,7 @@ extension Select {
     /// ```swift
     /// Employee
     ///     .window("dept_window") {
-    ///         $0.partition(by: $1.department).order(by: $1.salary, .desc)
+    ///         $0.partition(by: $1.department).order(by: $1.salary.desc())
     ///     }
     ///     .select {
     ///         ($0.name, rank().over("dept_window"))

Sources/StructuredQueriesCore/Statements/Update/Update+Returning.swift

@@ -16,7 +16,7 @@ extension Update {
     ) -> Update<From, (repeat each QueryValue)> {
         var returning: [QueryFragment] = []
         for resultColumn in repeat each selection(From.columns) {
-            returning.append("\(quote: resultColumn.name)")
+            returning.append(resultColumn.queryFragment)
         }
         return Update<From, (repeat each QueryValue)>(
             isEmpty: false,
@@ -63,7 +63,7 @@ extension Update {
     ) -> Update<From, From> {
         var returning: [QueryFragment] = []
         for resultColumn in From.TableColumns.allColumns {
-            returning.append("\(quote: resultColumn.name)")
+            returning.append(resultColumn.queryFragment)
         }
         return Update<From, From>(
             isEmpty: isEmpty,

Sources/StructuredQueriesCore/WindowSpec.swift

@@ -8,7 +8,7 @@
 /// ```swift
 /// WindowSpec()
 ///     .partition(by: columns.category)
-///     .order(by: columns.price, .desc)
+///     .order(by: columns.price.desc())
 /// ```
 public struct WindowSpec: Sendable {
     public var partitions: [QueryFragment] = []
@@ -43,17 +43,17 @@ public struct WindowSpec: Sendable {
         return copy
     }
 
-    /// Add an ordering expression with direction
+    /// Add an ordering expression
+    ///
+    /// Use `.asc()` or `.desc()` on the expression to specify direction:
+    /// ```swift
+    /// WindowSpec().order(by: columns.price.desc())
+    /// ```
     public func order(
-        by expression: some QueryExpression,
-        _ direction: OrderDirection = .asc
+        by expression: some QueryExpression
     ) -> WindowSpec {
         var copy = self
-        var fragment: QueryFragment = expression.queryFragment
-        if direction == .desc {
-            fragment.append(" DESC")
-        }
-        copy.orderings.append(fragment)
+        copy.orderings.append(expression.queryFragment)
         return copy
     }
 

Sources/StructuredQueriesPostgres/Aggregates/StandardAggregates.swift renamed to Sources/StructuredQueriesPostgres/Functions/Aggregate/StandardAggregates.swift

@@ -11,7 +11,7 @@ extension TableDefinition where QueryValue: FullTextSearchable {
     /// ```swift
     /// Article.where { $0.match("swift") }
     ///   .select { ($0, $0.rank(by: "swift")) }
-    ///   .order(by: \.1, .desc)
+    ///   .order(by: \.1.desc())
     /// // SELECT *, ts_rank("searchVector", to_tsquery('english', 'swift'))
     /// // FROM "articles"
     /// // WHERE "articles"."searchVector" @@ to_tsquery('english', 'swift')
@@ -56,7 +56,7 @@ extension TableDefinition where QueryValue: FullTextSearchable {
     /// ```swift
     /// Article.where { $0.match("quick <-> brown <-> fox") }
     ///   .select { ($0, $0.rank(byCoverage: "quick <-> brown <-> fox")) }
-    ///   .order(by: \.1, .desc)
+    ///   .order(by: \.1.desc())
     /// ```
     ///
     /// - Parameters:
@@ -89,7 +89,7 @@ extension TableDefinition where QueryValue: FullTextSearchable {
     /// // Weight A positions 10x more than D positions
     /// Article.where { $0.match("swift") }
     ///   .select { ($0, $0.rank(by: "swift", weights: [0.1, 0.2, 0.4, 1.0])) }
-    ///   .order(by: \.1, .desc)
+    ///   .order(by: \.1.desc())
     /// // SELECT *, ts_rank('{0.1, 0.2, 0.4, 1.0}', "searchVector", to_tsquery('swift'))
     /// ```
     ///
@@ -198,7 +198,7 @@ extension TableColumnExpression where Value == TextSearch.Vector {
     /// ```swift
     /// Article.where { $0.searchVector.match("swift") }
     ///   .select { ($0, $0.searchVector.rank(by: "swift")) }
-    ///   .order(by: \.1, .desc)
+    ///   .order(by: \.1.desc())
     /// ```
     ///
     /// - Parameters: