@nanobpm/nano-workforce 0.186.4 → 0.187.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +6 -0
- package/SPEC.md +35 -0
- package/app/agentic/vocab/auto-subscribe.test.ts +261 -0
- package/app/agentic/vocab/job-types.ts +123 -3
- package/app/contracts.ts +8 -0
- package/package.json +1 -1
package/CHANGELOG.md
CHANGED
|
@@ -1,3 +1,9 @@
|
|
|
1
|
+
## [0.187.0](https://github.com/nanobpm/nano-workforce/compare/v0.186.4...v0.187.0) (2026-09-12)
|
|
2
|
+
|
|
3
|
+
### Features
|
|
4
|
+
|
|
5
|
+
* **agentic:** autoSubscribe opt-out marker + single agentDefinition-external convention ([#782](https://github.com/nanobpm/nano-workforce/issues/782)) ([6a0c616](https://github.com/nanobpm/nano-workforce/commit/6a0c616158db7e83ab2479d66677dd652c12f886)), closes [jwulf/c8ctl-plugin-nano#235](https://github.com/jwulf/c8ctl-plugin-nano/issues/235) [#779](https://github.com/nanobpm/nano-workforce/issues/779) [#745](https://github.com/nanobpm/nano-workforce/issues/745)
|
|
6
|
+
|
|
1
7
|
## [0.186.4](https://github.com/nanobpm/nano-workforce/compare/v0.186.3...v0.186.4) (2026-09-12)
|
|
2
8
|
|
|
3
9
|
### Bug Fixes
|
package/SPEC.md
CHANGED
|
@@ -357,6 +357,41 @@ the app, which deploys on boot), and the next agent job of that type picks it up
|
|
|
357
357
|
> (`$AGENT_RESULT_FILE` / `::nano:result::`), and no task may still carry the retired
|
|
358
358
|
> baked `io.nanobpm.agentTask.task.prompt` header.
|
|
359
359
|
|
|
360
|
+
> **`<zeebe:agentDefinition agentType="external" />` is the ONE agentic-task signal.**
|
|
361
|
+
> Every hand-authored `senior:*` agent service task in `resources/processes` carries this
|
|
362
|
+
> engine-native AgentTask marker
|
|
363
|
+
> (issue #745) alongside its `<zeebe:taskDefinition>`, and it is the **single
|
|
364
|
+
> convention** the worker harness `--auto` reconciliation scans to discover agentic
|
|
365
|
+
> tasks — replacing the legacy `linkName="prompt"` / header dual signal so the app and
|
|
366
|
+
> harness converge on one signal (issue #779, harness jwulf/c8ctl-plugin-nano#235). The
|
|
367
|
+
> marker is CI-enforced over the deployed `resources/` process models
|
|
368
|
+
> (`agentTaskTypesMissingExternalMarker`,
|
|
369
|
+
> `agent-marker.test.ts`), so no prompt-bearing agent task in those models relies on
|
|
370
|
+
> prompt-link-only discovery. (The delivery-graph compiler's GENERATED agent BPMN —
|
|
371
|
+
> deployed at run time by `runDeliveryGraph`, not authored under `resources/` — is a
|
|
372
|
+
> separate deployed path NOT covered by this static guard; whether its generated cells
|
|
373
|
+
> should also carry the marker/opt-out convention is tracked separately under issue #745,
|
|
374
|
+
> not #779.) To **exclude** a task from `--auto` — one that must be served only by a
|
|
375
|
+
> worker that explicitly subscribes (`--job-type <type>` / a profile capability) — add
|
|
376
|
+
> the inert opt-out property inside its `extensionElements`, nested in the
|
|
377
|
+
> `<zeebe:properties>` wrapper the models and engine expect (as
|
|
378
|
+
> `resources/processes/feature.bpmn:57-63` does — a bare `<zeebe:property>` placed
|
|
379
|
+
> directly under `<bpmn:extensionElements>` is NOT the accepted shape):
|
|
380
|
+
>
|
|
381
|
+
> ```xml
|
|
382
|
+
> <bpmn:extensionElements>
|
|
383
|
+
> <zeebe:properties>
|
|
384
|
+
> <zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
385
|
+
> </zeebe:properties>
|
|
386
|
+
> </bpmn:extensionElements>
|
|
387
|
+
> ```
|
|
388
|
+
>
|
|
389
|
+
> Absence (or any value other than `"false"`) auto-subscribes as normal — opt-out is
|
|
390
|
+
> explicit and fail-safe. The property is inert to the engine (no migration, no
|
|
391
|
+
> behaviour change). It is a registered contract (`agentTask.autoSubscribe` in
|
|
392
|
+
> `app/contracts.ts`), read by the ONE helper `agentTaskTypesOptedOutOfAuto`
|
|
393
|
+
> (`app/agentic/vocab/job-types.ts`) and guarded by `auto-subscribe.test.ts`.
|
|
394
|
+
|
|
360
395
|
Per-instance dynamic context still rides **`appendPrompt`** (unchanged): an ioMapping
|
|
361
396
|
sets a job-local `appendPrompt` string (a plan's rejection findings, a feature task's
|
|
362
397
|
brief, the failing-check list) which the agent harness concatenates **verbatim** onto
|
|
@@ -0,0 +1,261 @@
|
|
|
1
|
+
// Tests for the `--auto` opt-OUT marker (issue #779, harness jwulf/c8ctl-plugin-nano#235). The single
|
|
2
|
+
// agentic-task signal the harness `--auto` reconciliation scans is `<zeebe:agentDefinition
|
|
3
|
+
// agentType="external" />`; this marker — `<zeebe:property name="io.nanobpm.agentTask.autoSubscribe"
|
|
4
|
+
// value="false" />` inside an agent task's extensionElements — is the explicit escape hatch that
|
|
5
|
+
// EXCLUDES a task from `--auto` so it is served only by a worker that explicitly subscribes. Mirrors
|
|
6
|
+
// agent-marker.test.ts: assert the marker's shape/placement, and guard that any opted-out task in the
|
|
7
|
+
// deployed models is itself a real (externally-marked) agent task, so the opt-out can't drift onto a
|
|
8
|
+
// non-agent element.
|
|
9
|
+
import { readFileSync, readdirSync } from "node:fs";
|
|
10
|
+
import { test } from "node:test";
|
|
11
|
+
import { dirname, join, relative } from "node:path";
|
|
12
|
+
import { fileURLToPath } from "node:url";
|
|
13
|
+
import { assert, assertEquals } from "#test-assert";
|
|
14
|
+
import {
|
|
15
|
+
agentTaskTypesMissingExternalMarker,
|
|
16
|
+
agentTaskTypesOptedOutMissingExternalMarker,
|
|
17
|
+
agentTaskTypesOptedOutOfAuto,
|
|
18
|
+
MALFORMED_OPTOUT_LABEL,
|
|
19
|
+
} from "./job-types.ts";
|
|
20
|
+
|
|
21
|
+
const RESOURCES_DIR = join(dirname(fileURLToPath(import.meta.url)), "../../../resources");
|
|
22
|
+
|
|
23
|
+
// urban deploys `resources/` recursively (every file at ANY depth), so an opted-out task in a `.bpmn`
|
|
24
|
+
// added under any resources subdirectory — not just `resources/processes` — would still deploy. Root
|
|
25
|
+
// the walk at the `resources/` convention root (mirroring the deploy contract) so a deployed BPMN
|
|
26
|
+
// placed elsewhere under `resources/` cannot bypass this guard.
|
|
27
|
+
function bpmnFiles(): string[] {
|
|
28
|
+
const walk = (dir: string): string[] =>
|
|
29
|
+
readdirSync(dir, { withFileTypes: true }).flatMap((entry) => {
|
|
30
|
+
const full = join(dir, entry.name);
|
|
31
|
+
if (entry.isDirectory()) return walk(full);
|
|
32
|
+
return entry.name.endsWith(".bpmn") ? [relative(RESOURCES_DIR, full)] : [];
|
|
33
|
+
});
|
|
34
|
+
return walk(RESOURCES_DIR).sort();
|
|
35
|
+
}
|
|
36
|
+
|
|
37
|
+
test("agentTaskTypesOptedOutOfAuto flags a task carrying the value=\"false\" opt-out property", () => {
|
|
38
|
+
const xml = `
|
|
39
|
+
<bpmn:serviceTask id="agent">
|
|
40
|
+
<bpmn:extensionElements>
|
|
41
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
42
|
+
<zeebe:agentDefinition agentType="external" />
|
|
43
|
+
<zeebe:properties>
|
|
44
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
45
|
+
</zeebe:properties>
|
|
46
|
+
</bpmn:extensionElements>
|
|
47
|
+
</bpmn:serviceTask>`;
|
|
48
|
+
assertEquals(agentTaskTypesOptedOutOfAuto(xml), ["senior:special"]);
|
|
49
|
+
});
|
|
50
|
+
|
|
51
|
+
test("agentTaskTypesOptedOutOfAuto tolerates reversed attribute order (name/value swapped)", () => {
|
|
52
|
+
const xml = `
|
|
53
|
+
<bpmn:serviceTask id="agent">
|
|
54
|
+
<bpmn:extensionElements>
|
|
55
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
56
|
+
<zeebe:properties>
|
|
57
|
+
<zeebe:property value="false" name="io.nanobpm.agentTask.autoSubscribe" />
|
|
58
|
+
</zeebe:properties>
|
|
59
|
+
</bpmn:extensionElements>
|
|
60
|
+
</bpmn:serviceTask>`;
|
|
61
|
+
assertEquals(agentTaskTypesOptedOutOfAuto(xml), ["senior:special"]);
|
|
62
|
+
});
|
|
63
|
+
|
|
64
|
+
test("agentTaskTypesOptedOutOfAuto ignores a task without the marker and one with a non-false value", () => {
|
|
65
|
+
const xml = `
|
|
66
|
+
<bpmn:serviceTask id="plain">
|
|
67
|
+
<bpmn:extensionElements>
|
|
68
|
+
<zeebe:taskDefinition type="senior:feature" />
|
|
69
|
+
<zeebe:agentDefinition agentType="external" />
|
|
70
|
+
</bpmn:extensionElements>
|
|
71
|
+
</bpmn:serviceTask>
|
|
72
|
+
<bpmn:serviceTask id="explicitTrue">
|
|
73
|
+
<bpmn:extensionElements>
|
|
74
|
+
<zeebe:taskDefinition type="senior:retro" />
|
|
75
|
+
<zeebe:properties>
|
|
76
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="true" />
|
|
77
|
+
</zeebe:properties>
|
|
78
|
+
</bpmn:extensionElements>
|
|
79
|
+
</bpmn:serviceTask>`;
|
|
80
|
+
assertEquals(agentTaskTypesOptedOutOfAuto(xml), []);
|
|
81
|
+
});
|
|
82
|
+
|
|
83
|
+
test("agentTaskTypesOptedOutOfAuto ignores an unrelated property named the same-ish", () => {
|
|
84
|
+
const xml = `
|
|
85
|
+
<bpmn:serviceTask id="agent">
|
|
86
|
+
<bpmn:extensionElements>
|
|
87
|
+
<zeebe:taskDefinition type="senior:feature" />
|
|
88
|
+
<zeebe:properties>
|
|
89
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribeMode" value="false" />
|
|
90
|
+
</zeebe:properties>
|
|
91
|
+
</bpmn:extensionElements>
|
|
92
|
+
</bpmn:serviceTask>`;
|
|
93
|
+
assertEquals(agentTaskTypesOptedOutOfAuto(xml), []);
|
|
94
|
+
});
|
|
95
|
+
|
|
96
|
+
test("agentTaskTypesOptedOutOfAuto ignores a bare opt-out property OUTSIDE the <zeebe:properties> wrapper (placement contract)", () => {
|
|
97
|
+
// The property carries the exact name/value, but it sits directly under <bpmn:extensionElements>
|
|
98
|
+
// rather than inside the <zeebe:properties> wrapper the engine honours — so the engine ignores it
|
|
99
|
+
// and it is NOT an active opt-out. Both the reader and the drift guard must treat it as absent.
|
|
100
|
+
const xml = `
|
|
101
|
+
<bpmn:serviceTask id="bare">
|
|
102
|
+
<bpmn:extensionElements>
|
|
103
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
104
|
+
<zeebe:agentDefinition agentType="external" />
|
|
105
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
106
|
+
</bpmn:extensionElements>
|
|
107
|
+
</bpmn:serviceTask>`;
|
|
108
|
+
assertEquals(agentTaskTypesOptedOutOfAuto(xml), []);
|
|
109
|
+
assertEquals(agentTaskTypesOptedOutMissingExternalMarker(xml), []);
|
|
110
|
+
});
|
|
111
|
+
|
|
112
|
+
test("agentTaskTypesMissingExternalMarker flags a prompt-bearing task whose external marker sits OUTSIDE extensionElements (placement contract)", () => {
|
|
113
|
+
// The task is a real prompt-bearing agent task, but its <zeebe:agentDefinition> marker is a sibling
|
|
114
|
+
// of <serviceTask> rather than inside extensionElements, so the engine/harness ignores it — the
|
|
115
|
+
// task is effectively unmarked. A whole-block scan would see the marker "somewhere" and wrongly
|
|
116
|
+
// pass; the placement-scoped scan flags the drift.
|
|
117
|
+
const xml = `
|
|
118
|
+
<bpmn:serviceTask id="misplaced">
|
|
119
|
+
<zeebe:agentDefinition agentType="external" />
|
|
120
|
+
<bpmn:extensionElements>
|
|
121
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
122
|
+
<zeebe:linkedResource resourceId="prompts/x.md" linkName="prompt" />
|
|
123
|
+
</bpmn:extensionElements>
|
|
124
|
+
</bpmn:serviceTask>`;
|
|
125
|
+
assertEquals(agentTaskTypesMissingExternalMarker(xml), ["senior:special"]);
|
|
126
|
+
});
|
|
127
|
+
|
|
128
|
+
test("agentTaskTypesOptedOutMissingExternalMarker flags an opt-out on a block lacking the external marker", () => {
|
|
129
|
+
// A host task (no external marker, no prompt link) that carries the opt-out is authoring drift:
|
|
130
|
+
// the block-level check catches it even though `agentTaskTypesMissingExternalMarker` (prompt-only)
|
|
131
|
+
// never reports it.
|
|
132
|
+
const xml = `
|
|
133
|
+
<bpmn:serviceTask id="host">
|
|
134
|
+
<bpmn:extensionElements>
|
|
135
|
+
<zeebe:taskDefinition type="pr.finalize" />
|
|
136
|
+
<zeebe:properties>
|
|
137
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
138
|
+
</zeebe:properties>
|
|
139
|
+
</bpmn:extensionElements>
|
|
140
|
+
</bpmn:serviceTask>`;
|
|
141
|
+
assertEquals(agentTaskTypesMissingExternalMarker(xml), []);
|
|
142
|
+
assertEquals(agentTaskTypesOptedOutMissingExternalMarker(xml), ["pr.finalize"]);
|
|
143
|
+
});
|
|
144
|
+
|
|
145
|
+
test("agentTaskTypesOptedOutMissingExternalMarker passes an opt-out on an externally-marked agent task", () => {
|
|
146
|
+
const xml = `
|
|
147
|
+
<bpmn:serviceTask id="agent">
|
|
148
|
+
<bpmn:extensionElements>
|
|
149
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
150
|
+
<zeebe:agentDefinition agentType="external" />
|
|
151
|
+
<zeebe:properties>
|
|
152
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
153
|
+
</zeebe:properties>
|
|
154
|
+
</bpmn:extensionElements>
|
|
155
|
+
</bpmn:serviceTask>`;
|
|
156
|
+
assertEquals(agentTaskTypesOptedOutMissingExternalMarker(xml), []);
|
|
157
|
+
});
|
|
158
|
+
|
|
159
|
+
test("agentTaskTypesOptedOutMissingExternalMarker checks each block independently (a marked sibling does not cover a drifted opt-out)", () => {
|
|
160
|
+
// Two tasks share the `senior:special` type: one is a proper externally-marked agent task, the
|
|
161
|
+
// other opts out but lacks the marker. A deduplicated cross-task comparison would miss this; the
|
|
162
|
+
// per-block check flags the unmarked one.
|
|
163
|
+
const xml = `
|
|
164
|
+
<bpmn:serviceTask id="marked">
|
|
165
|
+
<bpmn:extensionElements>
|
|
166
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
167
|
+
<zeebe:agentDefinition agentType="external" />
|
|
168
|
+
</bpmn:extensionElements>
|
|
169
|
+
</bpmn:serviceTask>
|
|
170
|
+
<bpmn:serviceTask id="drifted">
|
|
171
|
+
<bpmn:extensionElements>
|
|
172
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
173
|
+
<zeebe:properties>
|
|
174
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
175
|
+
</zeebe:properties>
|
|
176
|
+
</bpmn:extensionElements>
|
|
177
|
+
</bpmn:serviceTask>`;
|
|
178
|
+
assertEquals(agentTaskTypesOptedOutMissingExternalMarker(xml), ["senior:special"]);
|
|
179
|
+
});
|
|
180
|
+
|
|
181
|
+
test("agentTaskTypesOptedOutMissingExternalMarker flags an opt-out whose external marker sits OUTSIDE extensionElements (placement contract)", () => {
|
|
182
|
+
// The opt-out is correctly placed inside extensionElements, but the external marker is out of place
|
|
183
|
+
// (a sibling of <serviceTask>, not inside extensionElements) so the engine ignores it — the block
|
|
184
|
+
// is therefore NOT a real marked agent task. A whole-block scan would see the marker "somewhere" and
|
|
185
|
+
// wrongly pass; the placement-scoped scan flags the drift.
|
|
186
|
+
const xml = `
|
|
187
|
+
<bpmn:serviceTask id="misplaced">
|
|
188
|
+
<zeebe:agentDefinition agentType="external" />
|
|
189
|
+
<bpmn:extensionElements>
|
|
190
|
+
<zeebe:taskDefinition type="senior:special" />
|
|
191
|
+
<zeebe:properties>
|
|
192
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
193
|
+
</zeebe:properties>
|
|
194
|
+
</bpmn:extensionElements>
|
|
195
|
+
</bpmn:serviceTask>`;
|
|
196
|
+
assertEquals(agentTaskTypesOptedOutMissingExternalMarker(xml), ["senior:special"]);
|
|
197
|
+
});
|
|
198
|
+
|
|
199
|
+
test("agentTaskTypesOptedOutMissingExternalMarker surfaces an opt-out on a block with a missing/empty taskDefinition type", () => {
|
|
200
|
+
// An opt-out on an unmarked block whose <zeebe:taskDefinition> type is empty cannot be a real agent
|
|
201
|
+
// task, so it is still drift — surfaced under the sentinel rather than silently skipped.
|
|
202
|
+
const xml = `
|
|
203
|
+
<bpmn:serviceTask id="typeless">
|
|
204
|
+
<bpmn:extensionElements>
|
|
205
|
+
<zeebe:taskDefinition type="" />
|
|
206
|
+
<zeebe:properties>
|
|
207
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
208
|
+
</zeebe:properties>
|
|
209
|
+
</bpmn:extensionElements>
|
|
210
|
+
</bpmn:serviceTask>`;
|
|
211
|
+
assertEquals(agentTaskTypesOptedOutMissingExternalMarker(xml), [MALFORMED_OPTOUT_LABEL]);
|
|
212
|
+
});
|
|
213
|
+
|
|
214
|
+
test("agentTaskTypesOptedOutMissingExternalMarker surfaces a MARKED opt-out block with a missing/empty taskDefinition type", () => {
|
|
215
|
+
// The block carries BOTH the external marker AND the opt-out, but its <zeebe:taskDefinition> type
|
|
216
|
+
// is empty — so it still cannot be a real agent task. The malformed check must run BEFORE the
|
|
217
|
+
// external-marker short-circuit, or the marker would wrongly let this typeless opt-out pass.
|
|
218
|
+
const xml = `
|
|
219
|
+
<bpmn:serviceTask id="markedTypeless">
|
|
220
|
+
<bpmn:extensionElements>
|
|
221
|
+
<zeebe:taskDefinition type="" />
|
|
222
|
+
<zeebe:agentDefinition agentType="external" />
|
|
223
|
+
<zeebe:properties>
|
|
224
|
+
<zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" />
|
|
225
|
+
</zeebe:properties>
|
|
226
|
+
</bpmn:extensionElements>
|
|
227
|
+
</bpmn:serviceTask>`;
|
|
228
|
+
assertEquals(agentTaskTypesOptedOutMissingExternalMarker(xml), [MALFORMED_OPTOUT_LABEL]);
|
|
229
|
+
});
|
|
230
|
+
|
|
231
|
+
test("GUARD: the deploy-scan walks from the resources/ convention root (not just resources/processes)", () => {
|
|
232
|
+
// Regression for the deploy-by-convention coverage gap: urban deploys `resources/` recursively, so
|
|
233
|
+
// the guard must root its BPMN walk at `resources/` — a `.bpmn` added under any other resources
|
|
234
|
+
// subdirectory must still be scanned. Assert the walk actually reaches the process models AND that
|
|
235
|
+
// every returned path is relative to the resources root (carries its subdirectory segment), so a
|
|
236
|
+
// future refactor that narrows the root back to `resources/processes` is caught here.
|
|
237
|
+
const files = bpmnFiles();
|
|
238
|
+
assert(files.length > 0, "expected the resources/ walk to discover deployed BPMN models");
|
|
239
|
+
assert(
|
|
240
|
+
files.some((f) => f.startsWith("processes/")),
|
|
241
|
+
`expected resources-root-relative paths (e.g. "processes/…"); got ${JSON.stringify(files.slice(0, 3))}`,
|
|
242
|
+
);
|
|
243
|
+
});
|
|
244
|
+
|
|
245
|
+
test("GUARD: every deployed opted-out task is itself an externally-marked agent task", () => {
|
|
246
|
+
for (const file of bpmnFiles()) {
|
|
247
|
+
const xml = readFileSync(join(RESOURCES_DIR, file), "utf8");
|
|
248
|
+
// Drive the guard DIRECTLY from the block-level, placement-scoped helper rather than gating on
|
|
249
|
+
// `agentTaskTypesOptedOutOfAuto` (which skips missing/empty task-definition types, so a typeless
|
|
250
|
+
// opt-out would never reach the check). The helper scans every service task itself, surfaces a
|
|
251
|
+
// malformed typeless opt-out under the sentinel, and checks the external marker on the SAME
|
|
252
|
+
// block's extensionElements — so a typeless opt-out, an out-of-place property, or a marker that
|
|
253
|
+
// drifted onto a non-agent element all fail CI here.
|
|
254
|
+
const drifted = agentTaskTypesOptedOutMissingExternalMarker(xml);
|
|
255
|
+
assertEquals(
|
|
256
|
+
drifted,
|
|
257
|
+
[],
|
|
258
|
+
`${file}: task(s) ${JSON.stringify(drifted)} opt out of --auto but lack <zeebe:agentDefinition agentType="external" /> on the same block — an opt-out belongs only on a real agent task`,
|
|
259
|
+
);
|
|
260
|
+
}
|
|
261
|
+
});
|
|
@@ -48,11 +48,53 @@ export function jobTypeToRoutingToken(jobType: string): string | undefined {
|
|
|
48
48
|
|
|
49
49
|
const SERVICE_TASK = /<(?:\w+:)?serviceTask\b[\s\S]*?<\/(?:\w+:)?serviceTask>/g;
|
|
50
50
|
const TASK_DEFINITION_TYPE = /<(?:\w+:)?taskDefinition\b[^>]*\btype="([^"]*)"/;
|
|
51
|
+
const EXTENSION_ELEMENTS = /<(?:\w+:)?extensionElements\b[\s\S]*?<\/(?:\w+:)?extensionElements>/;
|
|
52
|
+
// The engine only honours a `<zeebe:property>` nested inside a `<zeebe:properties>` wrapper (itself
|
|
53
|
+
// inside `<bpmn:extensionElements>`). A bare `<zeebe:property>` placed directly under
|
|
54
|
+
// `extensionElements` (or `serviceTask`) is ignored, so property-contract scans (the `--auto`
|
|
55
|
+
// opt-out) run against THIS wrapper — a misplaced bare property is treated as absent, exactly as the
|
|
56
|
+
// engine treats it.
|
|
57
|
+
const ZEEBE_PROPERTIES = /<(?:\w+:)?properties\b[\s\S]*?<\/(?:\w+:)?properties>/;
|
|
51
58
|
const PROMPT_LINK = /<(?:\w+:)?linkedResource\b[^>]*\blinkName="prompt"/;
|
|
52
59
|
// The engine-native AgentTask marker (issue #745): a `<zeebe:agentDefinition agentType="external" />`
|
|
53
60
|
// sibling of the `<zeebe:taskDefinition>` inside a `senior:*` agent task's extensionElements. It is
|
|
54
61
|
// what makes the element eligible for engine-native AgentInstance minting by the worker harness.
|
|
55
62
|
const EXTERNAL_AGENT_MARKER = /<(?:\w+:)?agentDefinition\b[^>]*\bagentType="external"/;
|
|
63
|
+
// The `--auto` opt-OUT marker (issue #779): a `<zeebe:property name="io.nanobpm.agentTask.
|
|
64
|
+
// autoSubscribe" value="false" />` sibling inside an agent task's extensionElements. It declares the
|
|
65
|
+
// task is EXCLUDED from the harness `--auto` reconciliation (which keys on EXTERNAL_AGENT_MARKER) and
|
|
66
|
+
// is served only by a worker that explicitly subscribes. The property is inert to the engine. Only
|
|
67
|
+
// the exact `value="false"` opts out — any other value auto-subscribes as normal (fail-safe).
|
|
68
|
+
const AUTO_SUBSCRIBE_OPTOUT =
|
|
69
|
+
/<(?:\w+:)?property\b[^>]*\bname="io\.nanobpm\.agentTask\.autoSubscribe"[^>]*\bvalue="false"|<(?:\w+:)?property\b[^>]*\bvalue="false"[^>]*\bname="io\.nanobpm\.agentTask\.autoSubscribe"/;
|
|
70
|
+
|
|
71
|
+
// The label surfaced for an opted-out, unmarked service task whose `<zeebe:taskDefinition>` is
|
|
72
|
+
// missing or has an empty `type` (issue #779 drift guard). Such a block cannot be a real
|
|
73
|
+
// externally-marked agent task, so the opt-out is drift regardless of the absent type — we surface it
|
|
74
|
+
// under a descriptive sentinel rather than letting the dedupe skip swallow it.
|
|
75
|
+
export const MALFORMED_OPTOUT_LABEL = "(opted-out task with missing/empty taskDefinition type)";
|
|
76
|
+
|
|
77
|
+
/**
|
|
78
|
+
* The `<bpmn:extensionElements>…</bpmn:extensionElements>` content of a service-task block, or the
|
|
79
|
+
* empty string when the block has none. Placement-contract scans (issue #745/#779: a marker is only
|
|
80
|
+
* honoured by the engine INSIDE `extensionElements`) run against THIS scope, so a property/marker
|
|
81
|
+
* sitting outside `extensionElements` is treated as absent — the engine ignores it, and so must the
|
|
82
|
+
* guard (an out-of-place external marker cannot "cover" an opt-out).
|
|
83
|
+
*/
|
|
84
|
+
function extensionElementsOf(block: string): string {
|
|
85
|
+
return block.match(EXTENSION_ELEMENTS)?.[0] ?? "";
|
|
86
|
+
}
|
|
87
|
+
|
|
88
|
+
/**
|
|
89
|
+
* The `<zeebe:properties>…</zeebe:properties>` content nested inside a block's `extensionElements`,
|
|
90
|
+
* or the empty string when absent. The engine only honours `<zeebe:property>` entries INSIDE this
|
|
91
|
+
* wrapper, so the `--auto` opt-out property scan runs against THIS scope — a bare `<zeebe:property>`
|
|
92
|
+
* sitting directly under `extensionElements` (or `serviceTask`) is treated as absent, exactly as the
|
|
93
|
+
* engine treats it, matching the registered/documented shape (`<zeebe:properties>`-nested).
|
|
94
|
+
*/
|
|
95
|
+
function optOutPropertiesOf(block: string): string {
|
|
96
|
+
return extensionElementsOf(block).match(ZEEBE_PROPERTIES)?.[0] ?? "";
|
|
97
|
+
}
|
|
56
98
|
|
|
57
99
|
/**
|
|
58
100
|
* Scan one BPMN document for the job types of its PROMPT-BEARING service tasks — the deployed fleet
|
|
@@ -78,15 +120,18 @@ export function promptBearingTaskTypes(xml: string): string[] {
|
|
|
78
120
|
* engine-native AgentTask marker `<zeebe:agentDefinition agentType="external" />` (issue #745). Every
|
|
79
121
|
* deployed `senior:*` agent task must carry the marker so the worker harness mints an AgentInstance
|
|
80
122
|
* for it; a newly-added agent task that forgets it is a silent drift surface (its run never persists
|
|
81
|
-
* durable AgentHistory), so the regression guard fails CI.
|
|
82
|
-
*
|
|
123
|
+
* durable AgentHistory), so the regression guard fails CI. The marker check is scoped to the block's
|
|
124
|
+
* `<bpmn:extensionElements>` (the engine-honoured PLACEMENT scope, via `extensionElementsOf`) — a
|
|
125
|
+
* marker sitting outside `extensionElements` is ignored by the engine, so it must not "cover" a task
|
|
126
|
+
* here either. Returns the offending task types in first-occurrence order (empty when every agent
|
|
127
|
+
* task is marked).
|
|
83
128
|
*/
|
|
84
129
|
export function agentTaskTypesMissingExternalMarker(xml: string): string[] {
|
|
85
130
|
const seen = new Set<string>();
|
|
86
131
|
const missing: string[] = [];
|
|
87
132
|
for (const [block] of xml.matchAll(SERVICE_TASK)) {
|
|
88
133
|
if (!PROMPT_LINK.test(block)) continue;
|
|
89
|
-
if (EXTERNAL_AGENT_MARKER.test(block)) continue;
|
|
134
|
+
if (EXTERNAL_AGENT_MARKER.test(extensionElementsOf(block))) continue;
|
|
90
135
|
const type = block.match(TASK_DEFINITION_TYPE)?.[1];
|
|
91
136
|
if (type === undefined || type.length === 0 || seen.has(type)) continue;
|
|
92
137
|
seen.add(type);
|
|
@@ -94,3 +139,78 @@ export function agentTaskTypesMissingExternalMarker(xml: string): string[] {
|
|
|
94
139
|
}
|
|
95
140
|
return missing;
|
|
96
141
|
}
|
|
142
|
+
|
|
143
|
+
/**
|
|
144
|
+
* Scan one BPMN document for the job types of agent service tasks that OPT OUT of the harness
|
|
145
|
+
* `--auto` reconciliation (issue #779) — those carrying `<zeebe:property
|
|
146
|
+
* name="io.nanobpm.agentTask.autoSubscribe" value="false" />` inside their extensionElements. An
|
|
147
|
+
* opted-out task is served ONLY by a worker that explicitly subscribes (`--job-type <type>` / a
|
|
148
|
+
* profile capability), never by `--auto` auto-discovery (which keys on the
|
|
149
|
+
* `<zeebe:agentDefinition agentType="external" />` marker). The property is inert to the engine;
|
|
150
|
+
* only the exact `value="false"` opts out (any other value auto-subscribes, fail-safe). The opt-out
|
|
151
|
+
* scan is scoped to the block's `<zeebe:properties>` wrapper inside `<bpmn:extensionElements>` (the
|
|
152
|
+
* engine-honoured PLACEMENT scope, via `optOutPropertiesOf`) — a bare `<zeebe:property>` sitting
|
|
153
|
+
* directly under `<bpmn:extensionElements>` or `<bpmn:serviceTask>` is ignored by the engine, so it
|
|
154
|
+
* must not be reported as an active opt-out here (this keeps the reader consistent with the
|
|
155
|
+
* placement-scoped drift guard `agentTaskTypesOptedOutMissingExternalMarker`). Returns the
|
|
156
|
+
* distinct opted-out task types in first-occurrence order (empty when no task opts out).
|
|
157
|
+
*/
|
|
158
|
+
export function agentTaskTypesOptedOutOfAuto(xml: string): string[] {
|
|
159
|
+
const seen = new Set<string>();
|
|
160
|
+
const optedOut: string[] = [];
|
|
161
|
+
for (const [block] of xml.matchAll(SERVICE_TASK)) {
|
|
162
|
+
if (!AUTO_SUBSCRIBE_OPTOUT.test(optOutPropertiesOf(block))) continue;
|
|
163
|
+
const type = block.match(TASK_DEFINITION_TYPE)?.[1];
|
|
164
|
+
if (type === undefined || type.length === 0 || seen.has(type)) continue;
|
|
165
|
+
seen.add(type);
|
|
166
|
+
optedOut.push(type);
|
|
167
|
+
}
|
|
168
|
+
return optedOut;
|
|
169
|
+
}
|
|
170
|
+
|
|
171
|
+
/**
|
|
172
|
+
* Scan one BPMN document for the job types of service tasks that OPT OUT of `--auto` (issue #779) yet
|
|
173
|
+
* are NOT themselves externally-marked agent tasks — i.e. an opt-out property on a block WITHOUT a
|
|
174
|
+
* sibling `<zeebe:agentDefinition agentType="external" />`. An opt-out only makes sense on a real
|
|
175
|
+
* agent task (one that WOULD otherwise be auto-discovered via its external marker); a marker that has
|
|
176
|
+
* drifted onto a host task (e.g. `pr.finalize`, which carries no external marker) or onto one task
|
|
177
|
+
* that merely shares a `taskDefinition` type with a properly-marked sibling is authoring drift. This
|
|
178
|
+
* checks both markers on the SAME service-task block — the opt-out property inside the block's
|
|
179
|
+
* `<zeebe:properties>` wrapper (via `optOutPropertiesOf`, so a bare misplaced `<zeebe:property>` is
|
|
180
|
+
* ignored just as the engine ignores it) and the external marker inside that block's
|
|
181
|
+
* `<bpmn:extensionElements>` (the engine-honoured PLACEMENT scope), so an out-of-place external
|
|
182
|
+
* marker sitting outside `extensionElements` cannot spuriously "cover" the opt-out — so, unlike
|
|
183
|
+
* comparing the deduplicated `agentTaskTypesOptedOutOfAuto` / `agentTaskTypesMissingExternalMarker`
|
|
184
|
+
* lists (the latter only reports PROMPT-BEARING tasks, so a non-prompt host task's opt-out is
|
|
185
|
+
* invisible to it), the drift cannot hide. An opt-out on a block with a missing/empty
|
|
186
|
+
* `<zeebe:taskDefinition>` type (which likewise cannot be a real agent task) is surfaced under a
|
|
187
|
+
* descriptive sentinel BEFORE the external-marker short-circuit — so even a typeless block that
|
|
188
|
+
* carries the external marker is still flagged as malformed drift. Returns the offending task types
|
|
189
|
+
* in first-occurrence order (empty when every opted-out task is a properly-typed, externally-marked
|
|
190
|
+
* agent task).
|
|
191
|
+
*/
|
|
192
|
+
export function agentTaskTypesOptedOutMissingExternalMarker(xml: string): string[] {
|
|
193
|
+
const seen = new Set<string>();
|
|
194
|
+
const offending: string[] = [];
|
|
195
|
+
for (const [block] of xml.matchAll(SERVICE_TASK)) {
|
|
196
|
+
const ext = extensionElementsOf(block);
|
|
197
|
+
if (!AUTO_SUBSCRIBE_OPTOUT.test(optOutPropertiesOf(block))) continue;
|
|
198
|
+
const type = block.match(TASK_DEFINITION_TYPE)?.[1];
|
|
199
|
+
// A missing/empty type cannot be a real agent task, so an opt-out here is malformed drift
|
|
200
|
+
// REGARDLESS of any external marker — surface it under the sentinel BEFORE the marker
|
|
201
|
+
// short-circuit, so a marked-yet-typeless opt-out is caught rather than passed by the marker.
|
|
202
|
+
if (type === undefined || type.length === 0) {
|
|
203
|
+
if (seen.has(MALFORMED_OPTOUT_LABEL)) continue;
|
|
204
|
+
seen.add(MALFORMED_OPTOUT_LABEL);
|
|
205
|
+
offending.push(MALFORMED_OPTOUT_LABEL);
|
|
206
|
+
continue;
|
|
207
|
+
}
|
|
208
|
+
// A properly-typed opt-out is fine only when the SAME block is externally marked inside its
|
|
209
|
+
// extensionElements (the engine-honoured placement scope).
|
|
210
|
+
if (EXTERNAL_AGENT_MARKER.test(ext)) continue;
|
|
211
|
+
if (seen.has(type)) continue;
|
|
212
|
+
seen.add(type);
|
|
213
|
+
offending.push(type);
|
|
214
|
+
}
|
|
215
|
+
return offending;
|
|
216
|
+
}
|
package/app/contracts.ts
CHANGED
|
@@ -494,6 +494,14 @@ export const WIRE_CONTRACTS = {
|
|
|
494
494
|
"The engine-native AgentTask marker (issue #745, umbrella #746 — Camunda 8.10 parity). Every `senior:*` agent service task carries `<zeebe:agentDefinition agentType=\"external\" />` INSIDE its `<bpmn:extensionElements>`, COEXISTING with the existing `<zeebe:taskDefinition type=\"senior:*\"/>` dispatch verb (the verb stays, per #464). The marker makes the element eligible for engine-native AgentInstance minting by the worker harness (jwulf/c8ctl-plugin-nano#194): the harness mints Create/Update/Complete AgentInstance/AgentHistory records against the pinned engine (`@nanobpm/engine-wasm` 0.8.6, broker REST, SDK) while the element still emits its NORMAL `senior:*` job. `agentType=\"external\"` means the agent runs OUTSIDE the engine (a remote fleet worker), not an engine-embedded model call. This is the PRODUCER half; the durable AgentInstance/AgentHistory it mints is read back by the Cockpit historical view via `searchAgentInstanceHistory` (the CONSUMER half — see the `agentTask.historyRead` contract; the read path landed on `@nanobpm/urban`'s EngineClient in urban 0.93 / nanobpm/nano-ide#563). It is authored in the hand-written BPMN semantic model, NOT the generated `<bpmndi:…>` DI, and survives `npm run layout` untouched. Add the marker to a NEW `senior:*` agent task — never a second/synonym marker element.",
|
|
495
495
|
shape: '<zeebe:agentDefinition agentType="external" /> (sibling of <zeebe:taskDefinition> in a senior:* service task\'s extensionElements)',
|
|
496
496
|
},
|
|
497
|
+
"agentTask.autoSubscribe": {
|
|
498
|
+
category: "wire",
|
|
499
|
+
name: "agentTask.autoSubscribe",
|
|
500
|
+
owner: "resources/processes/*.bpmn",
|
|
501
|
+
semantics:
|
|
502
|
+
"The `--auto` opt-OUT marker (issue #779, harness jwulf/c8ctl-plugin-nano#235). The ONE agentic-task signal the harness `--auto` reconciliation scans is `<zeebe:agentDefinition agentType=\"external\" />` (the `agentTask.agentDefinition` marker) — it replaces the legacy `linkName=\"prompt\"` / header dual signal so both sides converge on a single convention. This marker is the escape hatch: a `<zeebe:property name=\"io.nanobpm.agentTask.autoSubscribe\" value=\"false\" />` INSIDE an agent task's `<bpmn:extensionElements>` declares the task is EXCLUDED from `--auto` auto-discovery and is served ONLY by a worker that explicitly subscribes (`--job-type <type>` / a profile capability). Absence of the marker (or any value other than the literal string `\"false\"`) means the task auto-subscribes as normal — opt-out is explicit and fail-safe. The `zeebe:property` is INERT to the engine (no runtime/behaviour change, no migration). Authored in the hand-written BPMN semantic model, NOT the generated `<bpmndi:…>` DI, and survives `npm run layout` untouched. The scan helper `agentTaskTypesOptedOutOfAuto(xml)` in app/agentic/vocab/job-types.ts is the ONE reader of this marker (mirrors `agentTaskTypesMissingExternalMarker`); a CI guard asserts its shape/placement. Use THIS one marker to opt a task out — never a second/synonym opt-out property.",
|
|
503
|
+
shape: '<zeebe:properties><zeebe:property name="io.nanobpm.agentTask.autoSubscribe" value="false" /></zeebe:properties> (the <zeebe:property> nested in a <zeebe:properties> wrapper inside a senior:* agent service task\'s <bpmn:extensionElements> — a bare <zeebe:property> directly under extensionElements is NOT the accepted shape; see SPEC.md)',
|
|
504
|
+
},
|
|
497
505
|
"agentTask.historyRead": {
|
|
498
506
|
category: "wire",
|
|
499
507
|
name: "agentTask.historyRead",
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@nanobpm/nano-workforce",
|
|
3
|
-
"version": "0.
|
|
3
|
+
"version": "0.187.0",
|
|
4
4
|
"description": "Nano Workforce — an Agent Graph Orchestration application for Agentic SDLC: durable BPMN processes that coordinate a graph of AI agents across the software delivery lifecycle.",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"main": "main.ts",
|