-
Notifications
You must be signed in to change notification settings - Fork 68
Open
Description
π€ 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 (new, branch
repo-assist/fix-issue-272-bulkcopy-computed-columns): fix: exclude computed columns from BulkCopy column mappings (Closes SqlProgrammability provider fails to update/bulk copy into tables with computed persistent columnsΒ #272) β Branch - 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; add parse edge-case tests β 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 when they have a default constraint (Closes Multi column PK may result in System.Data.ConstraintException when PK is widened Β #376) β 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
connectionparameter optional when literal connection string is configured (Closes #245)Β #452: fix: makeconnectionoptional when literal connection string configured (Closes SqlProgrammabilityProvider requires open SqlConnectionΒ #245) β Review - Review PR [Repo Assist] improve: cache TVP reflection lookups in SetParametersΒ #448: 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
workflowswrite permission β human must apply manually β View - Close issue [aw] Repo Assist failedΒ #447: Workflow failure issue (expired Mar 15, 2026) β 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 submitted β View
- Check comment SQL LIKE operation doesn't recognize variableΒ #437: Repo Assist provided concrete LIKE parameter workaround code example β 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
- Check comment Add AllParametersOption as safer alternative to AllParametersOptionalΒ #327: Repo Assist explained AllParametersOptional already gives safer required-but-option-typed behavior in current code β 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 Switch to
Microsoft.Data.SqlClientΒ #374: Repo Assist noted active Microsoft.Data.SqlClient migration progress (fork ready, π§ Engineering overhaul: drop .NET Framework, migrate to .NET 9, add FantomasΒ #444/Requirements going forward?Β #445 discussion) β 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
Personschema; docker CI from Requirements going forward?Β #445 is the fix β 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 SqlEnumProvider breaks VS 2019Β #410: Repo Assist explained generative TP limitation and DbProviderFactories flag β 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 Extremely long intellisense on errorsΒ #328: Repo Assist noted issue is stale; suggested closing β View
- Check comment Support for 'Active Directory Interactive' authenticationΒ #429: Repo Assist noted Microsoft.Data.SqlClient migration resolves AD Interactive auth β 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 Allow use pre-existing type for SqlCommandProviderΒ #334: Repo Assist noted fslang-suggestions#212 status + unbox workaround β 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 Needs an AllParametersOptional on SqlProgrammabilityProvider or?Β #265: Repo Assist explained AllParametersOptional is available on CreateCommand β 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 Question: For what reason would a SPROC not have an available constructor?Β #368: Repo Assist explained AddMembersDelayed failure scenarios β View
- Check comment Include Task-based execution on SqlCommandProviderΒ #371: Repo Assist noted FSharp.Core task CE approach β View
- Check comment Is it possible to turn off concurrency violation checking?Β #311: Repo Assist explained SqlCommandBuilder ConflictOption root cause β View
- Check comment SqlProgrammabilityProvider requires open SqlConnectionΒ #245: Repo Assist posted fix PR link (PR [Repo Assist] fix: make
connectionparameter optional when literal connection string is configured (Closes #245)Β #452) β View - Check comment SqlProgrammabilityProvider generates wrong
Executemethod if SP has default to null output parameterΒ #246: Repo Assist explained F# TP limitation for non-primitive output parameters β View - Check comment Sql Errors not bubbling up - System.TypeInitializationExceptionΒ #268: Repo Assist explained TypeInitializationException wrapping root cause β View
- Check comment Must System.Configuration.ConfigurationManager be manually installed?Β #358: Repo Assist explained System.Configuration.ConfigurationManager NuGet requirement β View
- Check comment Could not load file or assembly
System.Data.SqlClientif building usingdotnet 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 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 Build is broken without full VS installationΒ #412: Repo Assist explained System.Runtime.Caching GAC issue β View
- Check comment Any 2023 tricks for running on Windows?Β #427: Repo Assist explained Windows sni.dll probing failure β View
- Check comment NullReferenceException in SqlConnection.UseLocallyΒ #428: Repo Assist noted NullReferenceException root cause in UseLocally β 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 created β View
- Check comment Wrong inferred constraint on composite primary key with a defaultΒ #283: Repo Assist noted wrong PK constraint for DEFAULT columns β View
- Check comment Microsoft.Data.SqlClientΒ #441: Repo Assist technical analysis of TVP/UDTT SqlDataRecord fix β View
- Check comment Add support for NetTopologySuiteΒ #369: Repo Assist explained NetTopologySuite extension-point design needed β View
- Check comment Not platform detection FableΒ #411: Repo Assist explained Fable detection error is expected (not a SqlClient bug) β 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 Question: Creating a provider without an access to DBΒ #409: Repo Assist summarised offline-schema options (LocalDB, DACPAC, generated types) β View
- Check comment ExecuteNonQuery requires an open and available connectionΒ #306: Repo Assist noted LoadTempTables connection issue root cause β View
- Check comment Fixed length binary columns in user defined table types fail when constructedΒ #413: Repo Assist noted fix was shipped in 2.1.1, suggested closing β 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 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
Future Work for Repo Assist
- Continue commenting on older unaddressed issues (Generated types SqlCommand type provider. Records only.Β #162, Metadata from SqlCommandProviderΒ #188 and others)
- Note: CI workflow upgrades require
workflowswrite 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-15 07:15 UTC β Run
- π§ Created PR (branch
repo-assist/fix-issue-272-bulkcopy-computed-columns): 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 (ReadOnly DataColumn flag already set from is_computed), 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 (branch
repo-assist/fix-issue-433-fsharpcore-version): 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: noted IAsyncEnumerable(T) as the modern BCL approach for async streaming
- π¬ Commented on .NET 4.5 optimized versionΒ #16: noted .NET 4.5 async is now the baseline; issue effectively resolved
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 (branch
repo-assist/improve-quotations-factory-errors-20260314): replace silent assert with descriptive error in QuotationsFactory.GetBody; remove dead assert in MapArrayNullableItems - π¬ Commented on Add AllParametersOption as safer alternative to AllParametersOptionalΒ #327: explained AllParametersOptional already gives required-but-option-typed (safe) behavior in current code
- π¬ Commented on Provide types with measuresΒ #65: explained existing units-of-measure support via
(unitname)SQL type naming convention
2026-03-14 13:16 UTC β Run
- π¬ Commented on Switch to
Microsoft.Data.SqlClientΒ #374: noted active Microsoft.Data.SqlClient migration progress (fork FSharp.Data.MicrosoftSqlClient ready, π§ Engineering overhaul: drop .NET Framework, migrate to .NET 9, add FantomasΒ #444/Requirements going forward?Β #445 discussion) - π§ Attempted CI upgrade PR: setup-dotnet v3βv4 and 8.0.x patch channel (may require human action due to
workflowspermission)
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 Add support working with appsettings.json in Asp.Net Core 2.x and aboveΒ #370)
- π¬ Commented on Add support working with appsettings.json in Asp.Net Core 2.x and aboveΒ #370: linked new appsettings.json PR and explained design-time-only limitation
- π¬ Commented on Question: Creating a provider without an access to DBΒ #409: summarised offline-schema options (LocalDB SSDT, DACPAC, generated types, metadata caching)
2026-03-14 01:24 UTC β Run
- π§ Attempted CI upgrade PR β failed due to
workflowspermission; issue [Repo Assist] ci: upgrade setup-dotnet to v4 and use 8.0.x version specifierΒ #467 created with manual patch instructions - π¬ Commented on Add support for NetTopologySuiteΒ #369: explained NetTopologySuite integration requires custom CLR-type mapping extension point
- π¬ Commented on Not platform detection FableΒ #411: explained Fable detection error is expected β SqlClient is server-side only
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; add parse edge-case tests
2026-03-13 13:21 UTC β Run
- π¬ Commented on The azure database used on CI / unit tests is not there anymoreΒ #387: confirmed current CI failures are caused by missing AdventureWorks
Personschema; docker-based CI from Requirements going forward?Β #445 is the fix path - π Task 6: all Repo Assist PRs have infrastructure-only CI failures (no code changes needed)
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 selection for datetime2/datetimeoffset/time TVP columns (Closes Lots of ArgumentException in 2.0.7: "Metadata for field 'FieldName' of record '2' did not match the original record's metadata"Β #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 (
Closes #354)
2026-03-12 19:14 UTC β Run
- π§ Created 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 when they have a default constraint (
Closes #376) - π¬ Commented on Multi column PK may result in System.Data.ConstraintException when PK is widened Β #376, Extremely long intellisense on errorsΒ #328
2026-03-12 13:21 UTC β Run
- π¬ Commented on Support for 'Active Directory Interactive' authenticationΒ #429
- π§ Created PR [Repo Assist] ci: update FAKE build script target from net6.0 to net8.0Β #461: ci: update FAKE build script from net6.0βnet8.0
2026-03-12 07:16 UTC β Run
- π¬ Commented on Question: For what reason would a SPROC not have an available constructor?Β #368, Any 2023 tricks for running on Windows?Β #427
2026-03-12 01:24 UTC β Run
- π§ Created PR [Repo Assist] fix: improve error when Microsoft.SqlServer.Types assembly is missingΒ #460: fix: improve error when Microsoft.SqlServer.Types assembly is missing (
Closes #264) - π¬ Commented on Improve error if SQL Types dependency is missingΒ #264, Add support for netcore/netstandard based tooling (for example FAKE)Β #337, Allow use pre-existing type for SqlCommandProviderΒ #334
2026-03-11 19:14 UTC β Run
- π§ Created PR [Repo Assist] chore: fix ToCommantText typo, simplify getSqlDbType, fix TempTableLoader.CloseΒ #459: chore: fix ToCommantText typo, simplify getSqlDbType, fix TempTableLoader.Close
- π¬ Commented on Needs an AllParametersOptional on SqlProgrammabilityProvider or?Β #265
2026-03-11 13:22 UTC β Run
- π§ Extended PR [Repo Assist] perf: cache Type.GetType calls in VerifyResultsetColumnsΒ #454: added TypeInfo.ClrType cache
- π¬ Commented on cannot build netstandard2.0 project with the dotnet SDK buildΒ #366, Include Task-based execution on SqlCommandProviderΒ #371
2026-03-11 01:24 UTC β Run
- π¬ Commented on Is it possible to turn off concurrency violation checking?Β #311
2026-03-10 and earlier
- π¬ Commented on many issues including ExecuteNonQuery requires an open and available connectionΒ #306, Option not inferred for optional sproc parameters of type binary and nvarcharΒ #377, 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, Must System.Configuration.ConfigurationManager be manually installed?Β #358, Could not load file or assembly
System.Data.SqlClientif building usingdotnet buildΒ #373, Add SqlCommand.Table type for ResultType.DataReader and ExecuteReader for ResultType.DataTableΒ #336, Build is broken without full VS installationΒ #412, Microsoft.Data.SqlClientΒ #441, NullReferenceException in SqlConnection.UseLocallyΒ #428, dotnet SDK projects using FSharp.Core version 7.0.0 breaks at runtimeΒ #433, Wrong inferred constraint on composite primary key with a defaultΒ #283, SqlProgrammability provider fails to update/bulk copy into tables with computed persistent columnsΒ #272, SqlProgrammabilityProvider requires open SqlConnectionΒ #245, SqlProgrammabilityProvider generates wrongExecutemethod if SP has default to null output parameterΒ #246, Sql Errors not bubbling up - System.TypeInitializationExceptionΒ #268, Assertion in findTypeInfoBySqlEngineTypeIdΒ #354, docs of SqlCommandProvider is missing info on TempTableDefinitions and TableVarMappingΒ #350, Provide types with measuresΒ #65, Add AllParametersOption as safer alternative to AllParametersOptionalΒ #327 and others - π§ Created PRs [Repo Assist] improve: cache TVP reflection lookups in SetParametersΒ #448, [Repo Assist] docs: document TempTableDefinitions and TableVarMapping parametersΒ #450, [Repo Assist] fix: make
connectionparameter optional when literal connection string is configured (Closes #245)Β #452, [Repo Assist] perf: cache Type.GetType calls in VerifyResultsetColumnsΒ #454, [Repo Assist] test: expand DesignTimeConnectionStringTests and add net8.0 multi-targetingΒ #455, [Repo Assist] chore: minor code clarity improvementsΒ #456, [Repo Assist] docs: fix broken tool links in website navigation templateΒ #457, [Repo Assist] ci: update setup-dotnet v3βv4 and use 8.0.x patch channelΒ #458, [Repo Assist] chore: fix ToCommantText typo, simplify getSqlDbType, fix TempTableLoader.CloseΒ #459, [Repo Assist] fix: improve error when Microsoft.SqlServer.Types assembly is missingΒ #460 - π·οΈ 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
Reactions are currently unavailable