[Repo Assist] Monthly Activity 2026-03

[github-actions]

🤖 Repo Assist here — I'm an automated AI assistant for this repository.

Activity for March 2026

Suggested Actions for Maintainer

Comprehensive list of all pending actions requiring maintainer attention (excludes items already actioned and checked off).

• Review PR [Repo Assist] Fix FormatException when #if directive appears between attribute lists in mutually recursive type #3290: Fix FormatException when #if directive appears between attribute lists in mutually recursive type — fixes #if in mutually recursive class definition throws #3174 — Review
• Review PR [Repo Assist] Fix record copy expression indentation when nested in parentheses #3297: Fix record copy expression indentation when nested in parentheses — fixes Indentation warnings in record expression #2529 — Review
• Review PR [Repo Assist] Fix paren closing alignment when triple-quoted string ends at column zero, 2816 #3298: Fix paren closing alignment when triple-quoted string ends at column zero — fixes Multiline paren expr in chained application leads to invalidate code #2816 — Review
• Review PR [Repo Assist] Add tests for begin..end expressions and auto-property edge cases #3299: Add tests for begin..end expressions and auto-property edge cases — needs rebase on main — Review
• Review PR (pending): Update NuGet dependencies (minor/patch, 10 packages) — branch repo-assist/eng-dependency-updates-2026-03-20
• Review PR Improve formatting of query expressions with join/on/into clauses #3223: Improve formatting of query expressions with join/on/into clauses — fixes Query Expresion Formatting - Joins #3156 — Review
• Close issue Fantomas inserts space that breaks index lookups (without a dot) #2519: Index space bug confirmed fixed in current main — View
• Check comment Invalid .editorconfig file should give an error instead of being ignored #1236: Repo Assist analysed root cause — silent fallback when .editorconfig value is invalid (e.g. indent_size=2,); suggested fix approach — View
• Check comment when clause in try/with malformatted #2526: Repo Assist confirmed when clause bug is fixed for indent_size=2 via doubleIndent mechanism — suggest closing — View
• Check comment Commented line inside of record should be indented at the same level as the fields #2482: Repo Assist submitted fix (PR [Repo Assist] Fix trailing comments in record types not indented to field level #3300) which was closed by nojaf — issue still needs attention — View
• Check comment Unicode leads to incorrect indentation #2945: Repo Assist identified root cause — String.length counts UTF-16 units not grapheme clusters in Context.fs column tracking — View
• Check comment Wrong change of comment's position #1959: PR [Repo Assist] Fix block comment after 'of' in union cases and exceptions #3286 closed — nojaf indicated the real fix requires of keyword as trivia on the parser side — View
• Check comment Comment after colon is restored after return type #2827: PR [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3285 closed by nojaf — alternative approach needed for comment after colon in property return type — View
• Check comment Mishandling of some complex collections #2779: Repo Assist provided current state analysis — soundness bug still present with embedded-newline string literals in list tuples — View
• Check comment List expression items don't respect fsharp_blank_lines_around_nested_multiline_expressions #2856: Repo Assist triaged fsharp_blank_lines_around_nested_multiline_expressions not applying to list items — good first issue — View
• Check comment Misplaced comment in "else if" #3011: Repo Assist explained trivia-attribution challenge for comments between else if — View
• Check comment Regression: Stroustrup bracket style in anonymous record type parameter is not respected #3069: Repo Assist investigated MultilineBracketStyle = Stroustrup regression for anon record type params — View
• Check comment Closing > in nested multiline generic type applications violates offside rule #3243: Repo Assist analysed root cause for offset/indentation soundness bug (Type.AppPrefix multiline) — View
• Check comment Formatting with comment in fluent call breaks code #2694: Repo Assist analysed root cause — comment trivia in dot-chain expression disrupts indentation — View
• Check comment Custom operator is applied to anonymous record after formatting #3274: Repo Assist investigated lambda LHS infix operator semantics bug — View
• Check comment Lambda in tuple in list on single line changes meaning of code #3278: Repo Assist investigated lambda-in-tuple-in-list soundness bug — View
• Check comment Allow "--check" to differ with line-endings style #3103: Repo Assist provided analysis of --check vs line endings (CRLF/LF) — View
• Check comment Query Expresion Formatting - Joins #3156: Repo Assist pointed to WIP PR Improve formatting of query expressions with join/on/into clauses #3223 for query expression join/on/into formatting — View
• Check comment #if in mutually recursive class definition throws #3174: Repo Assist investigated FormatException in mutually recursive type — draft PR [Repo Assist] Fix FormatException when #if directive appears between attribute lists in mutually recursive type #3290 submitted — View
• Check comment Idempotency problem: IndentSize = 2 broke offside rule #2740: Repo Assist analysed root cause — |> after multiline match in lambda body placed at wrong indentation column — related to RFC RFC: How should Fantomas handle lambda expressions that would change semantics on a single line? #3279 — View
• Check comment code using mix of #if and parentheses doesn't parse after formatting #1989: Repo Assist confirmed FormatException is fixed in current main; residual odd indentation in #if/paren nesting is non-breaking — View
• Check comment Indentation warnings in record expression #2529: Repo Assist submitted fix for record copy expression indentation when nested in parens — draft PR [Repo Assist] Fix record copy expression indentation when nested in parentheses #3297 — View
• Check comment Multiline paren expr in chained application leads to invalidate code #2816: Repo Assist submitted fix for triple-quoted string paren alignment — draft PR [Repo Assist] Fix paren closing alignment when triple-quoted string ends at column zero, 2816 #3298 — View

Future Work for Repo Assist

• PR [Repo Assist] Fix trailing comments in record types not indented to field level #3300 (fix Commented line inside of record should be indented at the same level as the fields #2482 trailing comments in record types) was closed by nojaf — investigate why before re-attempting.
• PR [Repo Assist] Release 8.0.0-alpha-008 #3301 (alpha-008 release) was closed by nojaf — check release process.
• nojaf closed PR [Repo Assist] Fix block comment after 'of' in union cases and exceptions #3286 (fix Wrong change of comment's position #1959 block comment after of) — real fix requires of as trivia on parser side; do not re-attempt without parser changes.
• nojaf closed PR [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3285 (fix Comment after colon is restored after return type #2827 line comment in property accessor) — reason unknown; investigate alternative approach.
• nojaf is doing a larger refactor for consistent lambda-in-single-line semantics (Option B, per RFC RFC: How should Fantomas handle lambda expressions that would change semantics on a single line? #3279). Once merged, Lambda in tuple in list on single line changes meaning of code #3278 and Custom operator is applied to anonymous record after formatting #3274 can be closed.
• Issue Mishandling of some complex collections #2779 multiline-string-in-list soundness bug: still open; fix requires detecting embedded newlines in string literals and adding explicit semicolons.
• Issue Space should not be added before parentheses following ? operator #3159 (? operator space): 3 Repo Assist fix attempts were closed — do not re-attempt without maintainer guidance.
• Issue Idempotency problem: IndentSize = 2 broke offside rule #2740 (|> after match in lambda body at wrong column): root cause in genMultilineInfixExpr atCurrentColumn capturing lambda body column; fix requires careful change to indentation logic.
• Issue Unicode leads to incorrect indentation #2945 (Unicode combining characters widen column counter): root cause confirmed — String.length in Context.fs:103; fix needs grapheme-cluster counting throughout.

Run History

2026-03-20 UTC — Run

• 🔧 Submitted draft PR: Update NuGet dependencies (minor/patch) — 10 packages bumped; branch repo-assist/eng-dependency-updates-2026-03-20, PR pending
• 💬 Commented on Invalid .editorconfig file should give an error instead of being ignored #1236: root cause analysis for silent editorconfig fallback when value is invalid (e.g. indent_size=2,)

2026-03-20 UTC — Run

• 🔧 Created draft PR [Repo Assist] Fix trailing comments in record types not indented to field level #3300: Fix trailing comments in record types not indented to field level — fixes Commented line inside of record should be indented at the same level as the fields #2482 — CLOSED by nojaf
• 💬 Commented on when clause in try/with malformatted #2526: confirmed when clause bug is fixed for indent_size=2 via doubleIndent mechanism
• 💬 Commented on Commented line inside of record should be indented at the same level as the fields #2482: root cause analysis + fix description

2026-03-20 UTC — Run

• 🔧 Created draft PR: Fix trailing comments in record types not indented to field level — fixes Commented line inside of record should be indented at the same level as the fields #2482 — 2741 tests pass
• 💬 Commented on when clause in try/with malformatted #2526: confirmed when clause bug is fixed for indent_size=2 via doubleIndent mechanism
• 💬 Commented on Commented line inside of record should be indented at the same level as the fields #2482: root cause analysis + fix description

2026-03-19 UTC — Run

• 🔧 Created draft PR: Add 6 tests for begin..end expressions and auto-property edge cases (BeginEndTests.fs + AutoPropertiesTests.fs) — all 2745 tests pass
• 💬 Commented on Unicode leads to incorrect indentation #2945: root cause analysis — String.length in Context.fs:103 counts UTF-16 units, not grapheme clusters; fix suggestion

2026-03-18 UTC — Run

• 🔧 Created draft PR: Fix paren closing alignment when triple-quoted string ends at column zero — fixes Multiline paren expr in chained application leads to invalidate code #2816 — 2740 tests pass
• 💬 Commented on Multiline paren expr in chained application leads to invalidate code #2816: root cause analysis + fix description

2026-03-18 UTC — Run

• 🔧 Created draft PR [Repo Assist] Fix record copy expression indentation when nested in parentheses #3297: Fix record copy expression indentation when nested in parentheses — fixes #2529 — 2739 tests pass
• 💬 Commented on Indentation warnings in record expression #2529: root cause analysis + fix description

2026-03-17 13:33 UTC — Run

• 💬 Commented on Idempotency problem: IndentSize = 2 broke offside rule #2740: root cause analysis — |> after multiline match in lambda body placed at lambda body column instead of lambda column; affects all indent sizes; related to RFC RFC: How should Fantomas handle lambda expressions that would change semantics on a single line? #3279
• 💬 Commented on code using mix of #if and parentheses doesn't parse after formatting #1989: confirmed FormatException no longer occurs in current main; residual indentation quirk with #if/paren nesting is non-breaking

2026-03-17 01:35 UTC — Run

• 🔧 Created draft PR [Repo Assist] Add regression test for match on long anonymous record discriminant, 1903 #3296: Add regression test for match on long anonymous record type discriminant — closes #1903 — MERGED by nojaf — all 2739 tests pass
• 💬 Commented on Indentation warning for long match expression with anonymous record #1903: confirmed issue is fixed in current main, PR [Repo Assist] Add regression test for match on long anonymous record discriminant, 1903 #3296 submitted with regression test

2026-03-16 02:28 UTC — Run

• 🔧 Created draft PR [Repo Assist] Fix FormatException when #if directive appears between attribute lists in mutually recursive type #3290: Fix FormatException when #if directive appears between attribute lists in mutually recursive and type — fixes #3174 — root cause in genOnelinerAttributes dropping per-list ContentBefore directive trivia
• 💬 Commented on #if in mutually recursive class definition throws #3174: root cause analysis and fix description

2026-03-15 13:22 UTC — Run

• 🔧 Created draft PR [Repo Assist] Fix soundness bug: %% (double-percent) infix operator moves to new line, breaking code #3287: Fix soundness bug — %% (double-percent) infix operator incorrectly moved to new line — MERGED by nojaf
• 💬 Commented on Incorrect line split when using a custom operator #2107: root cause analysis for %% operator soundness bug

2026-03-15 UTC — Run

• 🔧 Submitted draft PR [Repo Assist] Fix block comment after 'of' in union cases and exceptions #3286: Fix block comment misplaced before of in union cases/exceptions — closed by nojaf (wants parser-side of trivia)
• 💬 Commented on Wrong change of comment's position #1959: root cause analysis and fix description

2026-03-14 13:37 UTC — Run

• 🔧 Revised fix for Comment after colon is restored after return type #2827: new PR [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3285 on branch repo-assist/fix-issue-2827-v2 — closed by nojaf
• 🔍 Scanned issues for Task 2 comment opportunities

2026-03-14 UTC — Run

• 💬 Commented on Query Expresion Formatting - Joins #3156: noted WIP PR Improve formatting of query expressions with join/on/into clauses #3223 addresses query expression join/on/into formatting
• 💬 Commented on Fantomas inserts space that breaks index lookups (without a dot) #2519: confirmed index-space soundness bug is fixed in current main — suggest closing

2026-03-13 13:48 UTC — Run

• 🔧 Created PR [Repo Assist] Extract isFirstCharUppercase helper in CodePrinter #3282: Extract isFirstCharUppercase helper in CodePrinter (removed duplicate inline logic) — closed by nojaf
• 💬 Commented on Mishandling of some complex collections #2779: current state analysis — soundness bug still reproducible with embedded-newline strings in list tuple elements

2026-03-13 01:35 UTC — Run

• 🔧 Rebased PR [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3270 on latest main — superseded by [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3285 → [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3285 now also closed

2026-03-12 13:29 UTC — Run

• 🔧 Created PR [Repo Assist] Add doc comments to 20 undocumented top-level Oak DU types and interfaces #3281: Add doc comments to 20 undocumented top-level Oak DU types and interfaces — MERGED by nojaf
• 💬 Commented on Allow "--check" to differ with line-endings style #3103: Technical analysis of --check line-endings issue

2026-03-12 01:35 UTC — Run

• 🔧 Created PR [Repo Assist] Stay multiline when list element is a tuple with lambda as last field (#3278) #3280: Stay multiline when list element is a tuple with lambda — fixes #3278 — CLOSED by nojaf (will solve in larger refactor)
• 💬 Commented on Lambda in tuple in list on single line changes meaning of code #3278: root cause analysis

2026-03-11 13:30 UTC — Run

• 🔧 Created PR [Repo Assist] Parenthesise lambda LHS of infix operator in single-line form to preserve semantics (#3274) #3276: Parenthesise lambda LHS of infix operator — fixes #3274 — CLOSED by nojaf (larger refactor)
• 💬 Commented on NamedIndexedPropertySet with two SynLongIdent remove space #3273, Custom operator is applied to anonymous record after formatting #3274: root cause analyses

2026-03-11 — Run

• 🔧 Created PR [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3270: Fix line comment between colon and return type in property accessor — fixes #2827 — superseded by [Repo Assist] Fix line comment between colon and return type in property accessor (#2827) #3285

2026-03-10 — Run

• 💬 Commented on Fantomas duplicates commented line at the end of the file #2499: root cause analysis
• 🔧 Created PR [Repo Assist] Fix doc comment (///) at end of file being duplicated #3266: Fix doc comment (///) at end of file being duplicated — fixes #2499 — MERGED

Generated by Repo Assist · ◷

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@d1d884596e62351dd652ae78465885dd32f0dd7d

[github-actions]

Run update — 2026-03-09 (run 22855741003)

Bug fixes

• #nowarn in arbitrary places produces a parsing failure #3258 — #nowarn/#warnon idempotency: Confirmed the root cause and submitted a fix PR. The FCS parser records WarnDirectiveTrivia ranges starting at column 0 regardless of actual indentation, so GetSubTextFromRange returns the full line including leading whitespace. Calling .TrimEnd() instead of .Trim() left that whitespace in the stored trivia, causing progressive indentation drift on each format pass. Fix: change .TrimEnd() → .Trim() in collectTriviaFromDirectiveRanges (Trivia.fs). Also added a regression test. PR staged for creation.

Coding improvements

• SyntaxOak.fs doc comments (part 3): Added /// documentation comments to ~27 more previously undocumented concrete node types, including InterfaceImplNode, ElseIfNode, IfKeywordNode, FillExprNode, ExternBindingNode, ModuleAbbrevNode, NestedModuleNode, OpenModuleOrNamespaceNode, OpenTargetNode, ExceptionDefnNode, MemberDefnInterfaceNode, MemberDefnAutoPropertyNode, MemberDefnAbstractSlotNode, ValNode, PropertyGetSetBindingNode, MemberDefnPropertyGetSetNode, MemberDefnSigMemberNode, LinkSingleAppParen, LinkSingleAppUnit, RationalNode, NegateRationalNode, UnitNode, ConstantMeasureNode, and more. PR staged for creation.

Labels applied

• Issue #nowarn in arbitrary places produces a parsing failure #3258: bug (soundness)

Prior RA PR status

• [Repo Assist] Add doc comments to remaining Expr*Node types in SyntaxOak.fs #3252 ✅ merged (Expr*Node doc comments)
• [Repo Assist] Fix: multiline CE builder name with fsharp_newline_before_multiline_computation_expression=false produces invalid code (#3155) #3255 ✅ merged (CE multiline fix for fsharp_newline_before_multiline_computation_expression = false may cause parse error if CE is called from an expression #3155)
• [Repo Assist] Fix: always add space before parameters in operator member definitions (#2864) #3253 ❌ closed without merge (operator member spacing)
• [Repo Assist] Fix: no space before paren args in ? operator chains (#3159) #3245 ❌ closed without merge (? operator chains)

Generated by Repo Assist · ◷

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@346204513ecfa08b81566450d7d599556807389f