Revert "Removes treating first element as operator in SExp"

This reverts commit c489a4e.

src/IC-222/kotlin/com/amazon/ion/plugin/intellij/formatting/blocks/IonSExpressionBlock.kt

@@ -32,4 +32,37 @@ class IonSExpressionBlock(
         IonTypes.LPAREN,
         IonTypes.RPAREN
     )
+
+    override fun buildChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions =
+        buildSpecialCaseChildBlockFormatting(child) ?:
+            super.buildChildBlockFormatting(child)
+
+    private fun buildSpecialCaseChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions? {
+
+        // Lazy evaluate the previous sibling if needed.
+        val previous by lazy { child.previousSibling() }
+
+        /**
+         * Check if we are the first comment within the expression, there is a special comment
+         * case where we don't want to apply the child alignment to the comment. For example:
+         *
+         * (join // special case comment which is inline with operator
+         *   // child comments are inline with inner values
+         *   anotherValue
+         * )
+         */
+        if (child elementIsA IonTypes.COMMENT && previous?.elementType == IonTypes.SEXPRESSION_OPERATOR) {
+
+            logger.debug { "Formatting [${child.psi.getLineNumber()}] - Special case inline expression comment line" }
+
+            val comment = child.psi
+            val expressionOperator = previous!!
+
+            if (comment.isOneLiner() && child.sameLineAs(expressionOperator)) {
+                return IonBlockFormatting.sameAlignment(this)
+            }
+        }
+
+        return null
+    }
 }

src/IC-231/kotlin/com/amazon/ion/plugin/intellij/formatting/blocks/IonSExpressionBlock.kt

@@ -32,4 +32,37 @@ class IonSExpressionBlock(
         IonTypes.LPAREN,
         IonTypes.RPAREN
     )
+
+    override fun buildChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions =
+        buildSpecialCaseChildBlockFormatting(child) ?:
+            super.buildChildBlockFormatting(child)
+
+    private fun buildSpecialCaseChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions? {
+
+        // Lazy evaluate the previous sibling if needed.
+        val previous by lazy { child.previousSibling() }
+
+        /**
+         * Check if we are the first comment within the expression, there is a special comment
+         * case where we don't want to apply the child alignment to the comment. For example:
+         *
+         * (join // special case comment which is inline with operator
+         *   // child comments are inline with inner values
+         *   anotherValue
+         * )
+         */
+        if (child elementIsA IonTypes.COMMENT && previous?.elementType == IonTypes.SEXPRESSION_OPERATOR) {
+
+            logger.debug { "Formatting [${child.psi.getLineNumber()}] - Special case inline expression comment line" }
+
+            val comment = child.psi
+            val expressionOperator = previous!!
+
+            if (comment.isOneLiner() && child.sameLineAs(expressionOperator)) {
+                return IonBlockFormatting.sameAlignment(this)
+            }
+        }
+
+        return null
+    }
 }

src/IC-232/kotlin/com/amazon/ion/plugin/intellij/formatting/blocks/IonSExpressionBlock.kt

@@ -32,4 +32,37 @@ class IonSExpressionBlock(
         IonTypes.LPAREN,
         IonTypes.RPAREN
     )
+
+    override fun buildChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions =
+        buildSpecialCaseChildBlockFormatting(child) ?:
+            super.buildChildBlockFormatting(child)
+
+    private fun buildSpecialCaseChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions? {
+
+        // Lazy evaluate the previous sibling if needed.
+        val previous by lazy { child.previousSibling() }
+
+        /**
+         * Check if we are the first comment within the expression, there is a special comment
+         * case where we don't want to apply the child alignment to the comment. For example:
+         *
+         * (join // special case comment which is inline with operator
+         *   // child comments are inline with inner values
+         *   anotherValue
+         * )
+         */
+        if (child elementIsA IonTypes.COMMENT && previous?.elementType == IonTypes.SEXPRESSION_OPERATOR) {
+
+            logger.debug { "Formatting [${child.psi.getLineNumber()}] - Special case inline expression comment line" }
+
+            val comment = child.psi
+            val expressionOperator = previous!!
+
+            if (comment.isOneLiner() && child.sameLineAs(expressionOperator)) {
+                return IonBlockFormatting.sameAlignment(this)
+            }
+        }
+
+        return null
+    }
 }

src/IC-233/kotlin/com/amazon/ion/plugin/intellij/formatting/blocks/IonSExpressionBlock.kt

@@ -32,4 +32,37 @@ class IonSExpressionBlock(
         IonTypes.LPAREN,
         IonTypes.RPAREN
     )
+
+    override fun buildChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions =
+        buildSpecialCaseChildBlockFormatting(child) ?:
+            super.buildChildBlockFormatting(child)
+
+    private fun buildSpecialCaseChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions? {
+
+        // Lazy evaluate the previous sibling if needed.
+        val previous by lazy { child.previousSibling() }
+
+        /**
+         * Check if we are the first comment within the expression, there is a special comment
+         * case where we don't want to apply the child alignment to the comment. For example:
+         *
+         * (join // special case comment which is inline with operator
+         *   // child comments are inline with inner values
+         *   anotherValue
+         * )
+         */
+        if (child elementIsA IonTypes.COMMENT && previous?.elementType == IonTypes.SEXPRESSION_OPERATOR) {
+
+            logger.debug { "Formatting [${child.psi.getLineNumber()}] - Special case inline expression comment line" }
+
+            val comment = child.psi
+            val expressionOperator = previous!!
+
+            if (comment.isOneLiner() && child.sameLineAs(expressionOperator)) {
+                return IonBlockFormatting.sameAlignment(this)
+            }
+        }
+
+        return null
+    }
 }

src/IC-241/kotlin/com/amazon/ion/plugin/intellij/formatting/blocks/IonSExpressionBlock.kt

@@ -32,4 +32,37 @@ class IonSExpressionBlock(
         IonTypes.LPAREN,
         IonTypes.RPAREN
     )
+
+    override fun buildChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions =
+        buildSpecialCaseChildBlockFormatting(child) ?:
+            super.buildChildBlockFormatting(child)
+
+    private fun buildSpecialCaseChildBlockFormatting(child: ASTNode): IonBlockFormattingOptions? {
+
+        // Lazy evaluate the previous sibling if needed.
+        val previous by lazy { child.previousSibling() }
+
+        /**
+         * Check if we are the first comment within the expression, there is a special comment
+         * case where we don't want to apply the child alignment to the comment. For example:
+         *
+         * (join // special case comment which is inline with operator
+         *   // child comments are inline with inner values
+         *   anotherValue
+         * )
+         */
+        if (child elementIsA IonTypes.COMMENT && previous?.elementType == IonTypes.SEXPRESSION_OPERATOR) {
+
+            logger.debug { "Formatting [${child.psi.getLineNumber()}] - Special case inline expression comment line" }
+
+            val comment = child.psi
+            val expressionOperator = previous!!
+
+            if (comment.isOneLiner() && child.sameLineAs(expressionOperator)) {
+                return IonBlockFormatting.sameAlignment(this)
+            }
+        }
+
+        return null
+    }
 }