Skip to content

chore: bump modular to v1.13.5#998

Merged
intel352 merged 3 commits into
mainfrom
chore/bump-modular-v1135
Jul 5, 2026
Merged

chore: bump modular to v1.13.5#998
intel352 merged 3 commits into
mainfrom
chore/bump-modular-v1135

Conversation

@intel352

@intel352 intel352 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • bump Workflow's root github.com/GoCodeAlone/modular dependency from v1.13.4 to v1.13.5
  • refresh go.sum for the new Modular release checksum

Why

Issue #133 follow-up found that workflow v0.85.1 was still built against modular v1.13.4. This patch lets the next Workflow release carry the Modular lock-across-I/O fixes into downstream plugin builds.

Validation

  • GOWORK=off go test ./module ./plugin ./plugin/external ./platform/state ./platform/providers/dockercompose -count=1
  • git diff --check

Copilot AI review requested due to automatic review settings July 5, 2026 04:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Bumps the root Workflow module’s github.com/GoCodeAlone/modular dependency from v1.13.4 to v1.13.5 so the next Workflow release can pick up Modular’s latest fixes (notably the lock-across-I/O changes referenced in the PR description).

Changes:

  • Update go.mod to require github.com/GoCodeAlone/modular v1.13.5
  • Refresh go.sum entries for the new Modular version and checksum

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
go.mod Updates the root module requirement to modular v1.13.5.
go.sum Updates checksums to match the modular v1.13.5 dependency resolution.

Comment thread go.mod
@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

⏱ Benchmark Results

No significant performance regressions detected.

benchstat comparison (baseline → PR)
## benchstat: baseline → PR
baseline-bench.txt:308: parsing iteration count: invalid syntax
baseline-bench.txt:419772: parsing iteration count: invalid syntax
baseline-bench.txt:803746: parsing iteration count: invalid syntax
baseline-bench.txt:1201349: parsing iteration count: invalid syntax
baseline-bench.txt:1535283: parsing iteration count: invalid syntax
baseline-bench.txt:1928722: parsing iteration count: invalid syntax
benchmark-results.txt:308: parsing iteration count: invalid syntax
benchmark-results.txt:256986: parsing iteration count: invalid syntax
benchmark-results.txt:577048: parsing iteration count: invalid syntax
benchmark-results.txt:882074: parsing iteration count: invalid syntax
benchmark-results.txt:1214769: parsing iteration count: invalid syntax
benchmark-results.txt:1520917: parsing iteration count: invalid syntax
goos: linux
goarch: amd64
pkg: github.com/GoCodeAlone/workflow/dynamic
cpu: AMD EPYC 7763 64-Core Processor                
                            │ benchmark-results.txt │
                            │        sec/op         │
InterpreterCreation-4                  7.884m ± 60%
ComponentLoad-4                        3.677m ±  6%
ComponentExecute-4                     1.978µ ±  2%
PoolContention/workers-1-4             1.095µ ±  3%
PoolContention/workers-2-4             1.095µ ±  4%
PoolContention/workers-4-4             1.092µ ±  1%
PoolContention/workers-8-4             1.095µ ±  1%
PoolContention/workers-16-4            1.098µ ±  1%
ComponentLifecycle-4                   3.644m ±  2%
SourceValidation-4                     2.365µ ±  1%
RegistryConcurrent-4                   810.8n ±  3%
LoaderLoadFromString-4                 3.705m ±  2%
geomean                                19.08µ

                            │ benchmark-results.txt │
                            │         B/op          │
InterpreterCreation-4                  2.027Mi ± 0%
ComponentLoad-4                        2.180Mi ± 0%
ComponentExecute-4                     1.203Ki ± 0%
PoolContention/workers-1-4             1.203Ki ± 0%
PoolContention/workers-2-4             1.203Ki ± 0%
PoolContention/workers-4-4             1.203Ki ± 0%
PoolContention/workers-8-4             1.203Ki ± 0%
PoolContention/workers-16-4            1.203Ki ± 0%
ComponentLifecycle-4                   2.183Mi ± 0%
SourceValidation-4                     1.984Ki ± 0%
RegistryConcurrent-4                   1.133Ki ± 0%
LoaderLoadFromString-4                 2.182Mi ± 0%
geomean                                15.25Ki

                            │ benchmark-results.txt │
                            │       allocs/op       │
InterpreterCreation-4                   15.68k ± 0%
ComponentLoad-4                         18.02k ± 0%
ComponentExecute-4                       25.00 ± 0%
PoolContention/workers-1-4               25.00 ± 0%
PoolContention/workers-2-4               25.00 ± 0%
PoolContention/workers-4-4               25.00 ± 0%
PoolContention/workers-8-4               25.00 ± 0%
PoolContention/workers-16-4              25.00 ± 0%
ComponentLifecycle-4                    18.07k ± 0%
SourceValidation-4                       32.00 ± 0%
RegistryConcurrent-4                     2.000 ± 0%
LoaderLoadFromString-4                  18.06k ± 0%
geomean                                  183.3

cpu: AMD EPYC 9V74 80-Core Processor                
                            │ baseline-bench.txt │
                            │       sec/op       │
InterpreterCreation-4              4.870m ± 100%
ComponentLoad-4                    2.838m ±   9%
ComponentExecute-4                 1.450µ ±   3%
PoolContention/workers-1-4         819.5n ±   3%
PoolContention/workers-2-4         832.4n ±   1%
PoolContention/workers-4-4         826.6n ±   1%
PoolContention/workers-8-4         844.5n ±   3%
PoolContention/workers-16-4        855.2n ±   4%
ComponentLifecycle-4               3.037m ±   1%
SourceValidation-4                 1.697µ ±   1%
RegistryConcurrent-4               614.6n ±   3%
LoaderLoadFromString-4             3.098m ±   1%
geomean                            14.41µ

                            │ baseline-bench.txt │
                            │        B/op        │
InterpreterCreation-4               2.027Mi ± 0%
ComponentLoad-4                     2.180Mi ± 0%
ComponentExecute-4                  1.203Ki ± 0%
PoolContention/workers-1-4          1.203Ki ± 0%
PoolContention/workers-2-4          1.203Ki ± 0%
PoolContention/workers-4-4          1.203Ki ± 0%
PoolContention/workers-8-4          1.203Ki ± 0%
PoolContention/workers-16-4         1.203Ki ± 0%
ComponentLifecycle-4                2.183Mi ± 0%
SourceValidation-4                  1.984Ki ± 0%
RegistryConcurrent-4                1.133Ki ± 0%
LoaderLoadFromString-4              2.182Mi ± 0%
geomean                             15.25Ki

                            │ baseline-bench.txt │
                            │     allocs/op      │
InterpreterCreation-4                15.68k ± 0%
ComponentLoad-4                      18.02k ± 0%
ComponentExecute-4                    25.00 ± 0%
PoolContention/workers-1-4            25.00 ± 0%
PoolContention/workers-2-4            25.00 ± 0%
PoolContention/workers-4-4            25.00 ± 0%
PoolContention/workers-8-4            25.00 ± 0%
PoolContention/workers-16-4           25.00 ± 0%
ComponentLifecycle-4                 18.07k ± 0%
SourceValidation-4                    32.00 ± 0%
RegistryConcurrent-4                  2.000 ± 0%
LoaderLoadFromString-4               18.06k ± 0%
geomean                               183.3

pkg: github.com/GoCodeAlone/workflow/middleware
cpu: AMD EPYC 7763 64-Core Processor                
                                  │ benchmark-results.txt │
                                  │        sec/op         │
CircuitBreakerDetection-4                     290.3n ± 6%
CircuitBreakerExecution_Success-4             21.57n ± 0%
CircuitBreakerExecution_Failure-4             66.31n ± 0%
geomean                                       74.61n

                                  │ benchmark-results.txt │
                                  │         B/op          │
CircuitBreakerDetection-4                    144.0 ± 0%
CircuitBreakerExecution_Success-4            0.000 ± 0%
CircuitBreakerExecution_Failure-4            0.000 ± 0%
geomean                                                 ¹
¹ summaries must be >0 to compute geomean

                                  │ benchmark-results.txt │
                                  │       allocs/op       │
CircuitBreakerDetection-4                    1.000 ± 0%
CircuitBreakerExecution_Success-4            0.000 ± 0%
CircuitBreakerExecution_Failure-4            0.000 ± 0%
geomean                                                 ¹
¹ summaries must be >0 to compute geomean

cpu: AMD EPYC 9V74 80-Core Processor                
                                  │ baseline-bench.txt │
                                  │       sec/op       │
CircuitBreakerDetection-4                 232.5n ± 19%
CircuitBreakerExecution_Success-4         17.57n ±  1%
CircuitBreakerExecution_Failure-4         55.18n ±  0%
geomean                                   60.87n

                                  │ baseline-bench.txt │
                                  │        B/op        │
CircuitBreakerDetection-4                 144.0 ± 0%
CircuitBreakerExecution_Success-4         0.000 ± 0%
CircuitBreakerExecution_Failure-4         0.000 ± 0%
geomean                                              ¹
¹ summaries must be >0 to compute geomean

                                  │ baseline-bench.txt │
                                  │     allocs/op      │
CircuitBreakerDetection-4                 1.000 ± 0%
CircuitBreakerExecution_Success-4         0.000 ± 0%
CircuitBreakerExecution_Failure-4         0.000 ± 0%
geomean                                              ¹
¹ summaries must be >0 to compute geomean

pkg: github.com/GoCodeAlone/workflow/module
cpu: AMD EPYC 7763 64-Core Processor                
                                 │ benchmark-results.txt │
                                 │        sec/op         │
IaCStateBackend_InProcess-4                 318.1n ± 35%
IaCStateBackend_GRPC-4                      9.699m ±  2%
JQTransform_Simple-4                        725.2n ± 32%
JQTransform_ObjectConstruction-4            1.577µ ±  1%
JQTransform_ArraySelect-4                   3.580µ ±  1%
JQTransform_Complex-4                       41.16µ ±  1%
JQTransform_Throughput-4                    1.903µ ±  2%
SSEPublishDelivery-4                        66.38n ±  1%
geomean                                     4.001µ

                                 │ benchmark-results.txt │
                                 │         B/op          │
IaCStateBackend_InProcess-4                416.0 ±  0%
IaCStateBackend_GRPC-4                   5.849Mi ± 13%
JQTransform_Simple-4                     1.273Ki ±  0%
JQTransform_ObjectConstruction-4         1.773Ki ±  0%
JQTransform_ArraySelect-4                2.625Ki ±  0%
JQTransform_Complex-4                    16.31Ki ±  0%
JQTransform_Throughput-4                 1.984Ki ±  0%
SSEPublishDelivery-4                       0.000 ±  0%
geomean                                                ¹
¹ summaries must be >0 to compute geomean

                                 │ benchmark-results.txt │
                                 │       allocs/op       │
IaCStateBackend_InProcess-4                 2.000 ± 0%
IaCStateBackend_GRPC-4                     6.835k ± 0%
JQTransform_Simple-4                        10.00 ± 0%
JQTransform_ObjectConstruction-4            15.00 ± 0%
JQTransform_ArraySelect-4                   30.00 ± 0%
JQTransform_Complex-4                       328.0 ± 0%
JQTransform_Throughput-4                    17.00 ± 0%
SSEPublishDelivery-4                        0.000 ± 0%
geomean                                                ¹
¹ summaries must be >0 to compute geomean

cpu: AMD EPYC 9V74 80-Core Processor                
                                 │ baseline-bench.txt │
                                 │       sec/op       │
IaCStateBackend_InProcess-4              246.8n ± 16%
IaCStateBackend_GRPC-4                   8.212m ±  2%
JQTransform_Simple-4                     563.5n ± 19%
JQTransform_ObjectConstruction-4         1.185µ ±  2%
JQTransform_ArraySelect-4                2.915µ ±  1%
JQTransform_Complex-4                    34.54µ ±  1%
JQTransform_Throughput-4                 1.457µ ±  3%
SSEPublishDelivery-4                     50.22n ±  1%
geomean                                  3.161µ

                                 │ baseline-bench.txt │
                                 │        B/op        │
IaCStateBackend_InProcess-4             416.0 ±  0%
IaCStateBackend_GRPC-4                5.654Mi ± 14%
JQTransform_Simple-4                  1.273Ki ±  0%
JQTransform_ObjectConstruction-4      1.773Ki ±  0%
JQTransform_ArraySelect-4             2.625Ki ±  0%
JQTransform_Complex-4                 16.31Ki ±  0%
JQTransform_Throughput-4              1.984Ki ±  0%
SSEPublishDelivery-4                    0.000 ±  0%
geomean                                             ¹
¹ summaries must be >0 to compute geomean

                                 │ baseline-bench.txt │
                                 │     allocs/op      │
IaCStateBackend_InProcess-4              2.000 ± 0%
IaCStateBackend_GRPC-4                  6.863k ± 0%
JQTransform_Simple-4                     10.00 ± 0%
JQTransform_ObjectConstruction-4         15.00 ± 0%
JQTransform_ArraySelect-4                30.00 ± 0%
JQTransform_Complex-4                    328.0 ± 0%
JQTransform_Throughput-4                 17.00 ± 0%
SSEPublishDelivery-4                     0.000 ± 0%
geomean                                             ¹
¹ summaries must be >0 to compute geomean

pkg: github.com/GoCodeAlone/workflow/schema
cpu: AMD EPYC 7763 64-Core Processor                
                                    │ benchmark-results.txt │
                                    │        sec/op         │
SchemaValidation_Simple-4                       1.098µ ± 5%
SchemaValidation_AllFields-4                    1.654µ ± 2%
SchemaValidation_FormatValidation-4             1.594µ ± 2%
SchemaValidation_ManySchemas-4                  1.835µ ± 2%
geomean                                         1.518µ

                                    │ benchmark-results.txt │
                                    │         B/op          │
SchemaValidation_Simple-4                      0.000 ± 0%
SchemaValidation_AllFields-4                   0.000 ± 0%
SchemaValidation_FormatValidation-4            0.000 ± 0%
SchemaValidation_ManySchemas-4                 0.000 ± 0%
geomean                                                   ¹
¹ summaries must be >0 to compute geomean

                                    │ benchmark-results.txt │
                                    │       allocs/op       │
SchemaValidation_Simple-4                      0.000 ± 0%
SchemaValidation_AllFields-4                   0.000 ± 0%
SchemaValidation_FormatValidation-4            0.000 ± 0%
SchemaValidation_ManySchemas-4                 0.000 ± 0%
geomean                                                   ¹
¹ summaries must be >0 to compute geomean

cpu: AMD EPYC 9V74 80-Core Processor                
                                    │ baseline-bench.txt │
                                    │       sec/op       │
SchemaValidation_Simple-4                    838.3n ± 2%
SchemaValidation_AllFields-4                 1.260µ ± 1%
SchemaValidation_FormatValidation-4          1.240µ ± 2%
SchemaValidation_ManySchemas-4               1.247µ ± 2%
geomean                                      1.131µ

                                    │ baseline-bench.txt │
                                    │        B/op        │
SchemaValidation_Simple-4                   0.000 ± 0%
SchemaValidation_AllFields-4                0.000 ± 0%
SchemaValidation_FormatValidation-4         0.000 ± 0%
SchemaValidation_ManySchemas-4              0.000 ± 0%
geomean                                                ¹
¹ summaries must be >0 to compute geomean

                                    │ baseline-bench.txt │
                                    │     allocs/op      │
SchemaValidation_Simple-4                   0.000 ± 0%
SchemaValidation_AllFields-4                0.000 ± 0%
SchemaValidation_FormatValidation-4         0.000 ± 0%
SchemaValidation_ManySchemas-4              0.000 ± 0%
geomean                                                ¹
¹ summaries must be >0 to compute geomean

pkg: github.com/GoCodeAlone/workflow/store
cpu: AMD EPYC 7763 64-Core Processor                
                                   │ benchmark-results.txt │
                                   │        sec/op         │
EventStoreAppend_InMemory-4                   1.183µ ± 18%
EventStoreAppend_SQLite-4                     1.488m ±  5%
GetTimeline_InMemory/events-10-4              14.30µ ±  6%
GetTimeline_InMemory/events-50-4              71.69µ ± 14%
GetTimeline_InMemory/events-100-4             127.2µ ±  1%
GetTimeline_InMemory/events-500-4             652.8µ ±  2%
GetTimeline_InMemory/events-1000-4            1.334m ±  1%
GetTimeline_SQLite/events-10-4                71.14µ ±  2%
GetTimeline_SQLite/events-50-4                206.1µ ±  1%
GetTimeline_SQLite/events-100-4               368.7µ ±  2%
GetTimeline_SQLite/events-500-4               1.664m ±  1%
GetTimeline_SQLite/events-1000-4              3.287m ±  0%
geomean                                       206.2µ

                                   │ benchmark-results.txt │
                                   │         B/op          │
EventStoreAppend_InMemory-4                     795.5 ± 9%
EventStoreAppend_SQLite-4                     2.054Ki ± 2%
GetTimeline_InMemory/events-10-4              7.953Ki ± 0%
GetTimeline_InMemory/events-50-4              46.62Ki ± 0%
GetTimeline_InMemory/events-100-4             94.48Ki ± 0%
GetTimeline_InMemory/events-500-4             472.8Ki ± 0%
GetTimeline_InMemory/events-1000-4            944.3Ki ± 0%
GetTimeline_SQLite/events-10-4                15.59Ki ± 0%
GetTimeline_SQLite/events-50-4                80.56Ki ± 0%
GetTimeline_SQLite/events-100-4               162.1Ki ± 0%
GetTimeline_SQLite/events-500-4               777.4Ki ± 0%
GetTimeline_SQLite/events-1000-4              1.503Mi ± 0%
geomean                                       65.36Ki

                                   │ benchmark-results.txt │
                                   │       allocs/op       │
EventStoreAppend_InMemory-4                     7.000 ± 0%
EventStoreAppend_SQLite-4                       55.00 ± 0%
GetTimeline_InMemory/events-10-4                125.0 ± 0%
GetTimeline_InMemory/events-50-4                653.0 ± 0%
GetTimeline_InMemory/events-100-4              1.306k ± 0%
GetTimeline_InMemory/events-500-4              6.514k ± 0%
GetTimeline_InMemory/events-1000-4             13.02k ± 0%
GetTimeline_SQLite/events-10-4                  316.0 ± 0%
GetTimeline_SQLite/events-50-4                 1.486k ± 0%
GetTimeline_SQLite/events-100-4                2.940k ± 0%
GetTimeline_SQLite/events-500-4                14.79k ± 0%
GetTimeline_SQLite/events-1000-4               29.80k ± 0%
geomean                                        1.065k

cpu: AMD EPYC 9V74 80-Core Processor                
                                   │ baseline-bench.txt │
                                   │       sec/op       │
EventStoreAppend_InMemory-4                777.1n ±  4%
EventStoreAppend_SQLite-4                  10.68m ± 84%
GetTimeline_InMemory/events-10-4           8.173µ ±  3%
GetTimeline_InMemory/events-50-4           44.78µ ±  2%
GetTimeline_InMemory/events-100-4          90.46µ ±  3%
GetTimeline_InMemory/events-500-4          450.8µ ±  1%
GetTimeline_InMemory/events-1000-4         918.5µ ±  1%
GetTimeline_SQLite/events-10-4             44.81µ ±  2%
GetTimeline_SQLite/events-50-4             144.7µ ±  1%
GetTimeline_SQLite/events-100-4            266.9µ ±  1%
GetTimeline_SQLite/events-500-4            1.232m ±  1%
GetTimeline_SQLite/events-1000-4           2.449m ±  4%
geomean                                    170.3µ

                                   │ baseline-bench.txt │
                                   │        B/op        │
EventStoreAppend_InMemory-4                 751.5 ±  3%
EventStoreAppend_SQLite-4                 2.059Ki ± 21%
GetTimeline_InMemory/events-10-4          7.953Ki ±  0%
GetTimeline_InMemory/events-50-4          46.62Ki ±  0%
GetTimeline_InMemory/events-100-4         94.48Ki ±  0%
GetTimeline_InMemory/events-500-4         472.8Ki ±  0%
GetTimeline_InMemory/events-1000-4        944.3Ki ±  0%
GetTimeline_SQLite/events-10-4            15.59Ki ±  0%
GetTimeline_SQLite/events-50-4            80.56Ki ±  0%
GetTimeline_SQLite/events-100-4           162.1Ki ±  0%
GetTimeline_SQLite/events-500-4           777.4Ki ±  0%
GetTimeline_SQLite/events-1000-4          1.503Mi ±  0%
geomean                                   65.06Ki

                                   │ baseline-bench.txt │
                                   │     allocs/op      │
EventStoreAppend_InMemory-4                  7.000 ± 0%
EventStoreAppend_SQLite-4                    55.00 ± 2%
GetTimeline_InMemory/events-10-4             125.0 ± 0%
GetTimeline_InMemory/events-50-4             653.0 ± 0%
GetTimeline_InMemory/events-100-4           1.306k ± 0%
GetTimeline_InMemory/events-500-4           6.514k ± 0%
GetTimeline_InMemory/events-1000-4          13.02k ± 0%
GetTimeline_SQLite/events-10-4               316.0 ± 0%
GetTimeline_SQLite/events-50-4              1.486k ± 0%
GetTimeline_SQLite/events-100-4             2.940k ± 0%
GetTimeline_SQLite/events-500-4             14.79k ± 0%
GetTimeline_SQLite/events-1000-4            29.80k ± 0%
geomean                                     1.065k

Benchmarks run with go test -bench=. -benchmem -count=6.
Regressions ≥ 20% are flagged. Results compared via benchstat.

@codecov

codecov Bot commented Jul 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copilot AI review requested due to automatic review settings July 5, 2026 04:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 12 out of 24 changed files in this pull request and generated no new comments.

@intel352
intel352 merged commit 60b8acf into main Jul 5, 2026
29 checks passed
@intel352
intel352 deleted the chore/bump-modular-v1135 branch July 5, 2026 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants