feat: add support for OrExpression in JMESPathVisitor.kt (#1012)

dayaffe · web-flow · commit 66a35b2a7eb1 · 2025-12-26T13:01:23.000-05:00
diff --git a/smithy-swift-codegen/src/main/kotlin/software/amazon/smithy/swift/codegen/waiters/JMESPathVisitor.kt b/smithy-swift-codegen/src/main/kotlin/software/amazon/smithy/swift/codegen/waiters/JMESPathVisitor.kt
@@ -475,7 +475,12 @@ class JMESPathVisitor(
         }
     }
 
-    override fun visitOr(expression: OrExpression): JMESVariable = throw Exception("OrExpression is unsupported")
+    override fun visitOr(expression: OrExpression): JMESVariable {
+        val leftVar = expression.left!!.accept(this)
+        val rightVar = expression.right!!.accept(this)
+        val orResultVar = JMESVariable("orResult", false, boolShape)
+        return addTempVar(orResultVar, "\$L || \$L", leftVar.name, rightVar.name)
+    }
 
     // Maps a collection into a collection of a different type.
     override fun visitProjection(expression: ProjectionExpression): JMESVariable {

Original file line number	Diff line number	Diff line change
`@@ -475,7 +475,12 @@ class JMESPathVisitor(`
`475`	`475`	`}`
`476`	`476`	`}`
`477`	`477`
`478`		`- override fun visitOr(expression: OrExpression): JMESVariable = throw Exception("OrExpression is unsupported")`
	`478`	`+ override fun visitOr(expression: OrExpression): JMESVariable {`
	`479`	`+ val leftVar = expression.left!!.accept(this)`
	`480`	`+ val rightVar = expression.right!!.accept(this)`
	`481`	`+ val orResultVar = JMESVariable("orResult", false, boolShape)`
	`482`	`+ return addTempVar(orResultVar, "\$L \|\| \$L", leftVar.name, rightVar.name)`
	`483`	`+ }`
`479`	`484`
`480`	`485`	`// Maps a collection into a collection of a different type.`
`481`	`486`	`override fun visitProjection(expression: ProjectionExpression): JMESVariable {`