[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:

• Review PR [Repo Assist] fix: write back output parameters in AsyncExecuteNonQuery and AsyncExecuteSeq #479: fix: write back output parameters in AsyncExecuteNonQuery and AsyncExecuteSeq — Review
• Review PR [Repo Assist] perf: pre-compute hasOutputParameters flag at construction time #478: perf: pre-compute hasOutputParameters flag at construction time — Review
• Review PR [Repo Assist] perf: pre-compute getReaderBehavior flags to avoid seq allocation per execution #474: perf: pre-compute getReaderBehavior flags — Review
• Review PR [Repo Assist] fix: auto-open connection in LoadTempTables (Closes #306) #473: fix: auto-open connection in LoadTempTables (Closes ExecuteNonQuery requires an open and available connection #306) — Review
• Review PR [Repo Assist] fix: exclude computed columns from BulkCopy column mappings (Closes #272) #472: fix: exclude computed columns from BulkCopy column mappings (Closes SqlProgrammability provider fails to update/bulk copy into tables with computed persistent columns #272) — Review
• Review PR [Repo Assist] fix: resolve FSharp.Core version mismatch in item type resolution (Closes #433) #471: fix: resolve FSharp.Core version mismatch in item type resolution (Closes dotnet SDK projects using FSharp.Core version 7.0.0 breaks at runtime #433) — Review
• Review PR [Repo Assist] improve: replace silent assert with descriptive error in QuotationsFactory.GetBody; remove dead assert in MapArrayNullableItems #470: improve: replace silent assert with descriptive error in QuotationsFactory.GetBody — Review
• Review PR [Repo Assist] feat: add appsettings.json support to DesignTimeConnectionString (Closes #370) #468: feat: add appsettings.json support to DesignTimeConnectionString (Closes Add support working with appsettings.json in Asp.Net Core 2.x and above #370) — Review
• Review PR [Repo Assist] fix: activate empty-string guard in DesignTimeConnectionString.Parse; add parse edge-case tests #466: fix: activate empty-string guard in DesignTimeConnectionString.Parse — Review
• Review PR use appveyor vs2026 image. #465: use appveyor vs2026 image — Review
• Review PR [Repo Assist] fix: replace assertions with descriptive exceptions in findTypeInfoBySqlEngineTypeId #463: fix: replace assertions with descriptive exceptions in findTypeInfoBySqlEngineTypeId (Closes Assertion in findTypeInfoBySqlEngineTypeId #354) — Review
• Review PR [Repo Assist] fix: only exclude nullable columns from DataTable PK when they have a default constraint #462: fix: only exclude nullable columns from DataTable PK with default constraint (Closes Multi column PK may result in System.Data.ConstraintException when PK is widened  #376 and Wrong inferred constraint on composite primary key with a default #283) — Review
• Review PR [Repo Assist] chore: fix ToCommantText typo, simplify getSqlDbType, fix TempTableLoader.Close #459: chore: fix ToCommantText typo, simplify getSqlDbType, fix TempTableLoader.Close — Review
• Review PR [Repo Assist] test: expand DesignTimeConnectionStringTests and add net8.0 multi-targeting #455: test: expand DesignTimeConnectionStringTests and add net8.0 multi-targeting — Review
• Review PR [Repo Assist] perf: cache Type.GetType calls in VerifyResultsetColumns #454: perf: cache Type.GetType calls in VerifyResultsetColumns — Review
• Review PR [Repo Assist] fix: make connection parameter optional when literal connection string is configured (Closes #245) #452: fix: make connection optional when literal connection string configured (Closes SqlProgrammabilityProvider requires open SqlConnection #245) — Review
• Review PR [Repo Assist] improve: cache TVP reflection lookups in SetParameters #448: improve: cache TVP reflection lookups in SetParameters — Review
• Review PR 🔧 Engineering overhaul: drop .NET Framework, migrate to .NET 9, add Fantomas #444: Engineering overhaul — drop .NET Framework, migrate to .NET 9, add Fantomas — Review
• Action needed [Repo Assist] ci: upgrade setup-dotnet to v4 and use 8.0.x version specifier #467: CI workflow patch (setup-dotnet v3→v4, 8.0.x) needs workflows write permission — human must apply manually — View
• Close issue [aw] Repo Assist failed #447: Workflow failure issue (expired Mar 15, 2026) — View
• Close issue create FAQ help page #134: FAQ page now exists at (fsprojects.github.io/redacted) — View
• Check comment Remove "Provider" post-fix. #159: Repo Assist noted PR 🔧 Engineering overhaul: drop .NET Framework, migrate to .NET 9, add Fantomas #444 addresses Provider postfix renaming — View
• Check comment enable natural setters on data tables #378: Repo Assist noted that natural setters are already generated for non-ReadOnly columns — View
• Check comment Migrate from FileSystemWatcher to HostFileChangeMonitor  #83: Repo Assist explained why FileSystemWatcher is the right cross-platform approach; suggested closing — View
• Check comment NullReferenceException in SqlConnection.UseLocally #428: Repo Assist traced NullReferenceException root cause in UseLocally (cmd.Connection null after transaction/timeout) — View
• Check comment Wrong inferred constraint on composite primary key with a default #283: Repo Assist linked PR [Repo Assist] fix: only exclude nullable columns from DataTable PK when they have a default constraint #462 as the fix (same root cause as Multi column PK may result in System.Data.ConstraintException when PK is widened  #376) — View
• Check comment ExecuteNonQuery requires an open and available connection #306: Repo Assist traced root cause of LoadTempTables connection error; fix PR [Repo Assist] fix: auto-open connection in LoadTempTables (Closes #306) #473 submitted — View
• Check comment SqlProgrammability provider fails to update/bulk copy into tables with computed persistent columns #272: Repo Assist traced root cause of BulkCopy computed column failure; fix PR [Repo Assist] fix: exclude computed columns from BulkCopy column mappings (Closes #272) #472 submitted — View
• Check comment SQL LIKE operation doesn't recognize variable #437: Repo Assist provided concrete LIKE parameter workaround code example — View
• Check comment SqlProgrammabilityProvider requires open SqlConnection #245: Repo Assist posted fix PR link (PR [Repo Assist] fix: make connection parameter optional when literal connection string is configured (Closes #245) #452) — View
• Check comment Multi column PK may result in System.Data.ConstraintException when PK is widened  #376: Repo Assist traced root cause; PR [Repo Assist] fix: only exclude nullable columns from DataTable PK when they have a default constraint #462 created — View
• Check comment Assertion in findTypeInfoBySqlEngineTypeId #354: Repo Assist explained cache invalidation race; PR [Repo Assist] fix: replace assertions with descriptive exceptions in findTypeInfoBySqlEngineTypeId #463 created — View
• Check comment dotnet SDK projects using FSharp.Core version 7.0.0 breaks at runtime #433: Repo Assist noted FSharp.Core version mismatch root cause; PR [Repo Assist] fix: resolve FSharp.Core version mismatch in item type resolution (Closes #433) #471 created — View
• Check comment Microsoft.Data.SqlClient #441: Repo Assist technical analysis of TVP/UDTT SqlDataRecord fix — View
• Check comment Add support working with appsettings.json in Asp.Net Core 2.x and above #370: Repo Assist linked new appsettings.json PR ([Repo Assist] feat: add appsettings.json support to DesignTimeConnectionString (Closes #370) #468) — View
• Check comment Not platform detection Fable #411: Repo Assist explained Fable detection error is expected — View
• Check comment SqlProgrammabilityProvider design-time: "The given key was not present in the dictionary" and then "No constructors are available" the first time an sproc is used until project reload #379: Repo Assist explained design-time race condition root cause — View
• Check comment Option not inferred for optional sproc parameters of type binary and nvarchar #377: Repo Assist explained Optional wrapping is value-type only — View
• Check comment Could not load file or assembly System.Data.SqlClient if building using dotnet build #373: Repo Assist noted PR 🔧 Engineering overhaul: drop .NET Framework, migrate to .NET 9, add Fantomas #444 as fix path + current workaround — View
• Check comment Build is broken without full VS installation #412: Repo Assist explained System.Runtime.Caching GAC issue — View
• Check comment Conversion failed when using User Defined Table Types #424: Repo Assist identified duplicate [C2] column in INSERT list as likely root cause — View
• Check comment The azure database used on CI / unit tests is not there anymore #387: Repo Assist confirmed CI failures caused by missing AdventureWorks Person schema — View
• Check comment Must System.Configuration.ConfigurationManager be manually installed? #358: Repo Assist explained System.Configuration.ConfigurationManager NuGet requirement — View
• Check comment Question: For what reason would a SPROC not have an available constructor? #368: Repo Assist explained AddMembersDelayed failure scenarios — View
• Check comment cannot build netstandard2.0 project with the dotnet SDK build #366: Repo Assist noted workaround and long-term fix via PR 🔧 Engineering overhaul: drop .NET Framework, migrate to .NET 9, add Fantomas #444 — View
• Check comment Add support for netcore/netstandard based tooling (for example FAKE) #337: Repo Assist noted PR 🔧 Engineering overhaul: drop .NET Framework, migrate to .NET 9, add Fantomas #444 as path forward + DACFx standalone option — View
• Check comment Add SqlCommand.Table type for ResultType.DataReader and ExecuteReader for ResultType.DataTable #336: Repo Assist explained implementation approach for Table factory feature — View
• Check comment Allow use pre-existing type for SqlCommandProvider #334: Repo Assist noted fslang-suggestions#212 status + unbox workaround — View
• Check comment Extremely long intellisense on errors #328: Repo Assist noted issue is stale; suggested closing — View
• Check comment Add AllParametersOption as safer alternative to AllParametersOptional #327: Repo Assist explained AllParametersOptional already gives required-but-option-typed behavior — View
• Check comment Is it possible to turn off concurrency violation checking? #311: Repo Assist explained SqlCommandBuilder ConflictOption root cause — View
• Check comment Sql Errors not bubbling up - System.TypeInitializationException #268: Repo Assist explained TypeInitializationException wrapping root cause — View
• Check comment Needs an AllParametersOptional on SqlProgrammabilityProvider or? #265: Repo Assist explained AllParametersOptional is available on CreateCommand — View
• Check comment SqlProgrammabilityProvider generates wrong Execute method if SP has default to null output parameter #246: Repo Assist explained F# TP limitation for non-primitive output parameters — View
• Check comment Provide types with measures #65: Repo Assist explained existing units-of-measure support via (unitname) SQL type naming convention — View
• Check comment Any 2023 tricks for running on Windows? #427: Repo Assist explained Windows sni.dll probing failure — View
• Check comment Support for 'Active Directory Interactive' authentication #429: Repo Assist noted Microsoft.Data.SqlClient migration resolves AD Interactive auth — View
• Check comment Question: Creating a provider without an access to DB #409: Repo Assist summarised offline-schema options — View
• Check comment Switch to Microsoft.Data.SqlClient  #374: Repo Assist noted active Microsoft.Data.SqlClient migration progress — View
• Check comment Include Task-based execution on SqlCommandProvider #371: Repo Assist noted FSharp.Core task CE approach — View
• Check comment Add support for NetTopologySuite #369: Repo Assist explained NetTopologySuite extension-point design needed — View
• Check comment SqlEnumProvider breaks VS 2019 #410: Repo Assist explained generative TP limitation and DbProviderFactories flag — View
• Check comment support asyncSeq or IAsyncEnumerator #15: Repo Assist noted IAsyncEnumerable(T) as the modern BCL approach for async streaming — View
• Check comment .NET 4.5 optimized version #16: Repo Assist noted .NET 4.5 async is now the baseline; issue effectively resolved — View

Future Work for Repo Assist

• Note: CI workflow upgrades require workflows write permission — Repo Assist cannot push these directly (see [Repo Assist] ci: upgrade setup-dotnet to v4 and use 8.0.x version specifier #467)

Run History

2026-03-16 13:22 UTC — Run

• 🔧 Created PR [Repo Assist] fix: write back output parameters in AsyncExecuteNonQuery and AsyncExecuteSeq #479: fix: write back output parameters in AsyncExecuteNonQuery and AsyncExecuteSeq
• 💬 Commented on create FAQ help page #134: noted FAQ page exists at fsprojects.github.io/FSharp.Data.SqlClient/faq.html, suggested closing
• 💬 Commented on Remove "Provider" post-fix. #159: noted PR 🔧 Engineering overhaul: drop .NET Framework, migrate to .NET 9, add Fantomas #444 addresses Provider postfix renaming as part of Microsoft.Data.SqlClient overhaul

2026-03-16 07:22 UTC — Run

• 🔧 Created PR [Repo Assist] perf: pre-compute hasOutputParameters flag at construction time #478: perf: pre-compute hasOutputParameters flag at construction time
• 💬 Commented on enable natural setters on data tables #378: noted natural setters already generated for non-ReadOnly columns
• 💬 Commented on Migrate from FileSystemWatcher to HostFileChangeMonitor  #83: explained FileSystemWatcher is correct cross-platform approach; suggested closing

2026-03-16 01:25 UTC — Run

• 🔧 Created PR [Repo Assist] perf: pre-compute quoted column names for DataTable.Update RowUpdating handler #477: perf: pre-compute quoted column names for DataTable.Update RowUpdating handler (merged ✅)
• 💬 Commented on NullReferenceException in SqlConnection.UseLocally #428: traced NullReferenceException root cause in UseLocally (cmd.Connection null after transaction/timeout)

2026-03-15 19:07 UTC — Run

• 🔧 Created PR [Repo Assist] chore: remove EOL netcoreapp3.1 test invocation from build script #476: chore: remove EOL netcoreapp3.1 test invocation from build script (merged ✅)
• 💬 Commented on Wrong inferred constraint on composite primary key with a default #283: linked to PR [Repo Assist] fix: only exclude nullable columns from DataTable PK when they have a default constraint #462 as the fix (same root cause as Multi column PK may result in System.Data.ConstraintException when PK is widened  #376)

2026-03-15 13:17 UTC — Run

• 🔧 Created PR [Repo Assist] fix: auto-open connection in LoadTempTables (Closes #306) #473: fix: auto-open connection in LoadTempTables (Closes ExecuteNonQuery requires an open and available connection #306)
• 🔧 Created PR [Repo Assist] perf: pre-compute getReaderBehavior flags to avoid seq allocation per execution #474: perf: pre-compute getReaderBehavior flags to avoid seq allocation per execution
• 💬 Commented on ExecuteNonQuery requires an open and available connection #306: traced root cause (LoadTempTables not calling UseLocally), linked fix PR

2026-03-15 07:15 UTC — Run

• 🔧 Created PR [Repo Assist] fix: exclude computed columns from BulkCopy column mappings (Closes #272) #472: fix: exclude computed columns from BulkCopy column mappings (Closes SqlProgrammability provider fails to update/bulk copy into tables with computed persistent columns #272)
• 💬 Commented on SqlProgrammability provider fails to update/bulk copy into tables with computed persistent columns #272: traced root cause, linked fix PR
• 💬 Commented on SQL LIKE operation doesn't recognize variable #437: provided concrete code example for the LIKE parameter workaround

2026-03-15 01:24 UTC — Run

• 🔧 Created PR [Repo Assist] fix: resolve FSharp.Core version mismatch in item type resolution (Closes #433) #471: fix: resolve FSharp.Core version mismatch in item type resolution (Closes dotnet SDK projects using FSharp.Core version 7.0.0 breaks at runtime #433)
• 💬 Commented on support asyncSeq or IAsyncEnumerator #15, .NET 4.5 optimized version #16

2026-03-14 19:06 UTC — Run

• 🔧 Created PR [Repo Assist] improve: replace silent assert with descriptive error in QuotationsFactory.GetBody; remove dead assert in MapArrayNullableItems #470: replace silent assert with descriptive error in QuotationsFactory.GetBody
• 💬 Commented on Add AllParametersOption as safer alternative to AllParametersOptional #327, Provide types with measures #65

2026-03-14 13:16 UTC — Run

• 💬 Commented on Switch to Microsoft.Data.SqlClient  #374
• 🔧 Attempted CI upgrade PR: setup-dotnet v3→v4 and 8.0.x patch channel (requires human action, see [Repo Assist] ci: upgrade setup-dotnet to v4 and use 8.0.x version specifier #467)

2026-03-14 07:10 UTC — Run

• 🔧 Created PR [Repo Assist] feat: add appsettings.json support to DesignTimeConnectionString (Closes #370) #468: feat: add appsettings.json support to DesignTimeConnectionString (Closes #370)
• 💬 Commented on Add support working with appsettings.json in Asp.Net Core 2.x and above #370, Question: Creating a provider without an access to DB #409

2026-03-14 01:24 UTC — Run

• 🔧 Attempted CI upgrade PR → failed due to workflows permission; issue [Repo Assist] ci: upgrade setup-dotnet to v4 and use 8.0.x version specifier #467 created
• 💬 Commented on Add support for NetTopologySuite #369, Not platform detection Fable #411

2026-03-13 19:08 UTC — Run

• 🔧 Created PR [Repo Assist] fix: activate empty-string guard in DesignTimeConnectionString.Parse; add parse edge-case tests #466: fix: activate empty-string guard in DesignTimeConnectionString.Parse

2026-03-13 07:14 UTC — Run

• 🔧 Pushed net8.0 CI fix to PR [Repo Assist] fix: only exclude nullable columns from DataTable PK when they have a default constraint #462 and PR [Repo Assist] fix: replace assertions with descriptive exceptions in findTypeInfoBySqlEngineTypeId #463
• 🔧 Created PR [Repo Assist] fix: correct SqlMetaData constructor selection for datetime2/datetimeoffset/time TVP columns #464: fix: correct SqlMetaData constructor for datetime2/datetimeoffset/time TVP columns (Closes #393)
• 💬 Commented on Lots of ArgumentException in 2.0.7: "Metadata for field 'FieldName' of record '2' did not match the original record's metadata" #393

2026-03-13 01:24 UTC — Run

• 💬 Commented on Conversion failed when using User Defined Table Types #424, SqlEnumProvider breaks VS 2019 #410
• 🔧 Created PR [Repo Assist] fix: replace assertions with descriptive exceptions in findTypeInfoBySqlEngineTypeId #463: fix: replace assertions with descriptive exceptions in findTypeInfoBySqlEngineTypeId

2026-03-12 and earlier

• Multiple PRs created ([Repo Assist] improve: cache TVP reflection lookups in SetParameters #448–[Repo Assist] fix: only exclude nullable columns from DataTable PK when they have a default constraint #462) and many issues commented on
• 🏷️ Labelled 29 previously-unlabelled issues

Generated by Repo Assist · ◷

To install this agentic workflow, run

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