SED-4885 Creating plan samples for RPA by jeromecomte · Pull Request #32 · exense/step-samples

jeromecomte · 2026-08-21T14:09:36Z

No description provided.

…before

…bout throughput measurements

… sample

david-stephan

Overall good, but took me quite some time to go through it, added several comments. Note, I skipped all keywords and stopped checking the YAML at some points.

david-stephan · 2026-08-24T06:54:27Z

+| `Demo_Google-search.json` | A sequence calling an Echo keyword and asserting on its output |
+| `Demo_Data-driven.json` | A data-driven plan iterating over a data source |
+
+Plans built in the Step plan editor are stored serialized as JSON, and these two files are


I'm not sure how the plan are stored in DB are relevant, I would just say that this is the format used for import/export and not meant for authoring

Totally agree. Will be changed

david-stephan · 2026-08-24T07:06:43Z

@@ -0,0 +1,67 @@
+# A standalone YAML plan: the same tree you would write inside an automation


The wording is not very clear for me. Not perfect either, but another propostion:

==============================================================================

STANDALONE PLAN

This file represents a single, self-contained Step plan.

Usage:

• Upload directly: Import into the Step UI via "Add plan" > "Create from YAML".

• Convert/add to a package: Move the contents under a plans: list entry inside

an automation package manifest.

This file is a syntax illustration, not a runnable plan - the keywords it

calls do not exist. For plans you can execute, see ../rpa/

==============================================================================

Agree with your suggestion. Will be changed

david-stephan · 2026-08-24T07:17:25Z

+          keyword: "Submit Record"
+          inputs:
+            - literalInput: "abc"              # static
+            - numericInput: 777                # static number


This is still a string not sure the comment is correct/clear:

To be discussed

david-stephan · 2026-08-24T07:38:17Z

+
+## What these samples teach
+
+The controls are the vocabulary; these are the ideas that decide whether a plan is any good.


I would write "The controls provide the vocabulary; the following principles determine whether a plan is well-designed:"

david-stephan · 2026-08-24T07:42:13Z

+   is the reason to use a plan at all instead of one large script.
+
+6. **Values come from outside the plan.** The same bot serves a person on demand
+   (execution parameters) and a nightly `schedules` entry, with credentials held in


not sure it's clear what we means with (execution parameters) a concrete example would help; also not sure that the distinction between on-demand run and scehduled ones belongs to *Values come from outside the plan.

david-stephan · 2026-08-24T08:46:21Z

+It governs whether one failing cleanup step stops the **remaining** cleanup steps — the same
+"keep going" semantics as on a sequence, applied to the cleanup list. Without it, a failing


"same "keep going" semantics as on a sequence", I have the feeling we did not mention continueOnError for sequence before, so that's a bit confusing

david-stephan · 2026-08-24T08:53:14Z

+session.
+
+Put a `sequence` inside the session and use **its** `before` / `after` instead. Those steps
+run within the session's token, and `after` still runs when the body fails — so cleanup is


"token" is probably not required, it just runs outside of the session. I guess all this is (and some other sections of this PR) comes from claude trying something, figuring it out it's not working as expecting and documenting it as such. While this is good as other user will fall in the same traps, I feel the wording around this a bit unnatural.

This is exactly the case and I had to perform several iterations to remove such "logs". To be fixed

david-stephan · 2026-08-24T08:57:08Z

+| Mechanism | Returns outputs? | Use it when |
+|-----------|------------------|-------------|
+| **Composite keyword** | Yes, via `return` | The thing you are reusing is a *step* inside someone else's plan. **Reach for this first** |
+| `callPlan` | No — it produces its own report branch | The thing you are reusing is a whole bot in its own right, with its own schedule |


I don't get " own right, with its own schedule"

Don't get it neither ;) To be fixed

david-stephan · 2026-08-24T09:00:12Z

+
+## `forEach threads` vs `testSet threads`
+
+- `threads` on a **loop** parallelises **rows** of one bot.


would say "row processing" or something

david-stephan · 2026-08-24T09:02:47Z

+
+## A plan that is called is still a plan
+
+`Shared - Archive processed records` exists to be called by plan B, but nothing marks it as


this might also be confusing for readers. Are seems to be more an artefact of "this is a sample, it should not fail". In real life, if I define such a shared plan, nothing say it has to run standalone too and ahve default values.

Agree. We should clearly mark this as an requirement for this sample or mark this plan as expected to fail

… content

SED-4885 Creating plan samples for RPA

8fe5261

jeromecomte requested a review from david-stephan August 21, 2026 14:19

jeromecomte added 6 commits August 21, 2026 22:51

SED-4885 Creating plan samples for load testing (draft)

0eb0430

SED-4885 Creating plan samples for load testing - Precising iterations

1a9dbc1

SED-4885 Creating plan samples for load testing - Moving dataSets to …

29687c7

…before

SED-4885 Creating plan samples for load testing - Fixing register slips

829c5a1

SED-4885 Creating plan samples for load testing - Improving clarity a…

cd3b63c

…bout throughput measurements

SED-4885 Creating plan samples for load testing - Remove assertMetric…

74a2250

… sample

david-stephan requested changes Aug 24, 2026

View reviewed changes

jeromecomte added 2 commits August 24, 2026 11:19

SED-4885 Creating plan samples for load testing - Removing duplicated…

3c50835

… content

SED-4885 PR Feedbacks RPA Samples

b89c699

		@@ -0,0 +1,67 @@
		# A standalone YAML plan: the same tree you would write inside an automation


		## What these samples teach

		The controls are the vocabulary; these are the ideas that decide whether a plan is any good.

		It governs whether one failing cleanup step stops the remaining cleanup steps — the same
		"keep going" semantics as on a sequence, applied to the cleanup list. Without it, a failing


		## `forEach threads` vs `testSet threads`

		- `threads` on a loop parallelises rows of one bot.


		## A plan that is called is still a plan

		`Shared - Archive processed records` exists to be called by plan B, but nothing marks it as

Uh oh!

Conversation

jeromecomte commented Aug 21, 2026

Uh oh!

david-stephan left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jeromecomte Aug 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

==============================================================================

STANDALONE PLAN

This file represents a single, self-contained Step plan.

Usage:

• Upload directly: Import into the Step UI via "Add plan" > "Create from YAML".

• Convert/add to a package: Move the contents under a plans: list entry inside

an automation package manifest.

This file is a syntax illustration, not a runnable plan - the keywords it

calls do not exist. For plans you can execute, see ../rpa/

==============================================================================

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

jeromecomte Aug 24, 2026 •

edited

Loading

• Convert/add to a package: Move the contents under a `plans:` list entry inside