Skip to content

Commit f769a61

Browse files
authored
Preliminary support for tracing workflow activity (#97)
Signed-off-by: Michael X. Grey <mxgrey@intrinsic.ai>
1 parent 7c7aec9 commit f769a61

26 files changed

+2092
-1216
lines changed

.github/workflows/ci_linux.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,9 @@ jobs:
3939
- name: Test diagram
4040
run: cargo test --workspace -F=diagram
4141

42+
- name: Test trace
43+
run: cargo test --workspace -F=trace
44+
4245
- name: Build single_threaded_async
4346
run: cargo build --features single_threaded_async
4447
- name: Test single_threaded_async

Cargo.toml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ async-task = { version = "4.7.1", optional = true }
3131
bevy_tasks = { version = "0.12", features = ["multi-threaded"] }
3232

3333
itertools = "0.13"
34-
smallvec = "1.13"
34+
smallvec = { version = "1.13", features = ["serde"] }
3535
tokio = { version = "1.39", features = ["sync"] }
3636
futures = "0.3"
3737
backtrace = "0.3"
@@ -68,6 +68,7 @@ diagram = [
6868
"dep:serde_json",
6969
"dep:strum",
7070
]
71+
trace = ["diagram"]
7172

7273
[dev-dependencies]
7374
async-std = { version = "1.12" }

diagram.schema.json

Lines changed: 199 additions & 15 deletions
Large diffs are not rendered by default.

examples/diagram/calculator/src/main.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,19 +40,19 @@ fn main() -> Result<(), Box<dyn Error>> {
4040

4141
let mut registry = DiagramElementRegistry::new();
4242
registry.register_node_builder(
43-
NodeBuilderOptions::new("add").with_name("Add"),
43+
NodeBuilderOptions::new("add").with_default_display_text("Add"),
4444
|builder, config: f64| builder.create_map_block(move |req: f64| req + config),
4545
);
4646
registry.register_node_builder(
47-
NodeBuilderOptions::new("sub").with_name("Subtract"),
47+
NodeBuilderOptions::new("sub").with_default_display_text("Subtract"),
4848
|builder, config: f64| builder.create_map_block(move |req: f64| req - config),
4949
);
5050
registry.register_node_builder(
51-
NodeBuilderOptions::new("mul").with_name("Multiply"),
51+
NodeBuilderOptions::new("mul").with_default_display_text("Multiply"),
5252
|builder, config: f64| builder.create_map_block(move |req: f64| req * config),
5353
);
5454
registry.register_node_builder(
55-
NodeBuilderOptions::new("div").with_name("Divide"),
55+
NodeBuilderOptions::new("div").with_default_display_text("Divide"),
5656
|builder, config: f64| builder.create_map_block(move |req: f64| req / config),
5757
);
5858

registry.schema.json

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,15 @@
2424
"additionalProperties": {
2525
"$ref": "#/$defs/SectionRegistration"
2626
}
27+
},
28+
"trace_supported": {
29+
"type": "boolean"
2730
}
2831
},
2932
"required": [
3033
"nodes",
3134
"sections",
35+
"trace_supported",
3236
"messages",
3337
"schemas"
3438
],
@@ -114,7 +118,8 @@
114118
"config_schema": {
115119
"$ref": "#/$defs/Schema"
116120
},
117-
"name": {
121+
"default_display_text": {
122+
"description": "If the user does not specify a default display text, the node ID will\n be used here.",
118123
"type": "string"
119124
},
120125
"request": {
@@ -125,7 +130,7 @@
125130
}
126131
},
127132
"required": [
128-
"name",
133+
"default_display_text",
129134
"request",
130135
"response",
131136
"config_schema"
@@ -204,15 +209,15 @@
204209
"config_schema": {
205210
"$ref": "#/$defs/Schema"
206211
},
212+
"default_display_text": {
213+
"type": "string"
214+
},
207215
"metadata": {
208216
"$ref": "#/$defs/SectionMetadata"
209-
},
210-
"name": {
211-
"type": "string"
212217
}
213218
},
214219
"required": [
215-
"name",
220+
"default_display_text",
216221
"metadata",
217222
"config_schema"
218223
]

0 commit comments

Comments
 (0)