Skip to content

Commit 264c1d9

Browse files
authored
Merge pull request #412 from badaitech:fix/flows-cache
fix: ensure flow clone is awaited to prevent potential race conditions
2 parents a477b66 + aa9b383 commit 264c1d9

2 files changed

Lines changed: 55 additions & 12 deletions

File tree

.vscode/settings.json

Lines changed: 54 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,16 +6,56 @@
66
"source.organizeImports": "never"
77
},
88
"eslint.rules.customizations": [
9-
{ "rule": "style/*", "severity": "off", "fixable": true },
10-
{ "rule": "format/*", "severity": "off", "fixable": true },
11-
{ "rule": "*-indent", "severity": "off", "fixable": true },
12-
{ "rule": "*-spacing", "severity": "off", "fixable": true },
13-
{ "rule": "*-spaces", "severity": "off", "fixable": true },
14-
{ "rule": "*-order", "severity": "off", "fixable": true },
15-
{ "rule": "*-dangle", "severity": "off", "fixable": true },
16-
{ "rule": "*-newline", "severity": "off", "fixable": true },
17-
{ "rule": "*quotes", "severity": "off", "fixable": true },
18-
{ "rule": "*semi", "severity": "off", "fixable": true }
9+
{
10+
"rule": "style/*",
11+
"severity": "off",
12+
"fixable": true
13+
},
14+
{
15+
"rule": "format/*",
16+
"severity": "off",
17+
"fixable": true
18+
},
19+
{
20+
"rule": "*-indent",
21+
"severity": "off",
22+
"fixable": true
23+
},
24+
{
25+
"rule": "*-spacing",
26+
"severity": "off",
27+
"fixable": true
28+
},
29+
{
30+
"rule": "*-spaces",
31+
"severity": "off",
32+
"fixable": true
33+
},
34+
{
35+
"rule": "*-order",
36+
"severity": "off",
37+
"fixable": true
38+
},
39+
{
40+
"rule": "*-dangle",
41+
"severity": "off",
42+
"fixable": true
43+
},
44+
{
45+
"rule": "*-newline",
46+
"severity": "off",
47+
"fixable": true
48+
},
49+
{
50+
"rule": "*quotes",
51+
"severity": "off",
52+
"fixable": true
53+
},
54+
{
55+
"rule": "*semi",
56+
"severity": "off",
57+
"fixable": true
58+
}
1959
],
2060
"eslint.validate": [
2161
"javascript",
@@ -38,5 +78,8 @@
3878
"scss",
3979
"pcss",
4080
"postcss"
81+
],
82+
"github-actions.workflows.pinned.workflows": [
83+
".github/workflows/release.yml"
4184
]
42-
}
85+
}

packages/chaingraph-executor/server/dbos/steps/ExecuteFlowAtomicStep.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ class FlowCachedLoader {
212212
// Check if version matches
213213
if (cached.flowVersion === flowVersion) {
214214
DBOS.logger.debug(`Flow cache hit: ${flowId} (version: ${cached.flowVersion})`)
215-
return cached.flow.clone()
215+
return await cached.flow.clone() as Flow
216216
}
217217
} else {
218218
// Cache expired

0 commit comments

Comments
 (0)