@danielblomma/cortex-mcp 2.0.15 → 2.0.18
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/README.md +5 -3
- package/package.json +1 -1
- package/scaffold/mcp/package-lock.json +59 -60
- package/scaffold/mcp/package.json +2 -2
- package/scaffold/mcp/src/cli/stage.ts +15 -18
- package/scaffold/mcp/src/cli/telemetry-test.ts +2 -1
- package/scaffold/mcp/src/core/license.ts +3 -2
- package/scaffold/mcp/src/core/telemetry/collector.ts +5 -4
- package/scaffold/mcp/src/core/telemetry/state-dir.ts +40 -0
- package/scaffold/mcp/src/core/validators/builtins.ts +24 -4
- package/scaffold/mcp/src/core/workflow/index.ts +1 -0
- package/scaffold/mcp/src/core/workflow/mcp-tools.ts +15 -25
- package/scaffold/mcp/src/core/workflow/resolution.ts +113 -0
- package/scaffold/mcp/src/daemon/main.ts +4 -3
- package/scaffold/mcp/src/enterprise/index.ts +16 -1
- package/scaffold/mcp/src/enterprise/reviews/policy-selection.ts +65 -0
- package/scaffold/mcp/src/enterprise/reviews/push.ts +55 -4
- package/scaffold/mcp/src/enterprise/reviews/trust-state.ts +305 -0
- package/scaffold/mcp/src/enterprise/tools/enterprise.ts +40 -47
- package/scaffold/mcp/src/enterprise/tools/harness.ts +2 -0
- package/scaffold/mcp/src/enterprise/workflow/state.ts +21 -0
- package/scaffold/mcp/tests/enterprise-review-trust.test.mjs +318 -0
- package/scaffold/mcp/tests/fixtures/org-skillz-contract.json +59 -0
- package/scaffold/mcp/tests/fixtures/review-trust-contract.json +111 -0
- package/scaffold/mcp/tests/review-policy-selection.test.mjs +85 -0
- package/scaffold/mcp/tests/review-trust-contract.test.mjs +167 -0
- package/scaffold/mcp/tests/review-validator-source.test.mjs +52 -0
- package/scaffold/mcp/tests/skill-sync-checker.test.mjs +277 -0
- package/scaffold/mcp/tests/telemetry-collector.test.mjs +31 -1
- package/scaffold/mcp/tests/workflow-cli.test.mjs +102 -1
- package/scaffold/mcp/tests/workflow-mcp-tools.test.mjs +22 -0
- package/scaffold/mcp/tests/workflow-synced-registry.test.mjs +56 -0
- package/scaffold/scripts/parsers/node_modules/.package-lock.json +0 -1
|
@@ -6,6 +6,32 @@ import path from "node:path";
|
|
|
6
6
|
|
|
7
7
|
import { runStageCommand } from "../dist/cli/stage.js";
|
|
8
8
|
|
|
9
|
+
const TINY_WORKFLOW = {
|
|
10
|
+
id: "tiny",
|
|
11
|
+
description: "Two-stage tests workflow",
|
|
12
|
+
version: 1,
|
|
13
|
+
stages: [
|
|
14
|
+
{
|
|
15
|
+
name: "plan",
|
|
16
|
+
artifact: "plan.md",
|
|
17
|
+
reads: [],
|
|
18
|
+
required_fields: [],
|
|
19
|
+
validators: [],
|
|
20
|
+
capability: "planner",
|
|
21
|
+
description: "Produce a plan.",
|
|
22
|
+
},
|
|
23
|
+
{
|
|
24
|
+
name: "review",
|
|
25
|
+
artifact: "review.md",
|
|
26
|
+
reads: ["plan"],
|
|
27
|
+
required_fields: ["approved"],
|
|
28
|
+
validators: [],
|
|
29
|
+
capability: "reviewer",
|
|
30
|
+
description: "Review the plan.",
|
|
31
|
+
},
|
|
32
|
+
],
|
|
33
|
+
};
|
|
34
|
+
|
|
9
35
|
function makeWorkspace() {
|
|
10
36
|
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "cortex-stage-cli-"));
|
|
11
37
|
process.env.CORTEX_PROJECT_ROOT = dir;
|
|
@@ -21,6 +47,37 @@ function makeWorkspace() {
|
|
|
21
47
|
return dir;
|
|
22
48
|
}
|
|
23
49
|
|
|
50
|
+
function writeSyncedWorkflow(homeDir, workflowId, definition) {
|
|
51
|
+
const cortexDir = path.join(homeDir, ".cortex");
|
|
52
|
+
fs.mkdirSync(cortexDir, { recursive: true });
|
|
53
|
+
fs.writeFileSync(
|
|
54
|
+
path.join(cortexDir, "workflows.local.json"),
|
|
55
|
+
JSON.stringify({
|
|
56
|
+
workflows: {
|
|
57
|
+
[workflowId]: {
|
|
58
|
+
workflow_id: workflowId,
|
|
59
|
+
version: definition.version,
|
|
60
|
+
updated_at: "2026-05-17T10:00:00.000Z",
|
|
61
|
+
definition,
|
|
62
|
+
},
|
|
63
|
+
},
|
|
64
|
+
}),
|
|
65
|
+
"utf8",
|
|
66
|
+
);
|
|
67
|
+
}
|
|
68
|
+
|
|
69
|
+
function setEnforcedMode(projectRoot) {
|
|
70
|
+
fs.writeFileSync(
|
|
71
|
+
path.join(projectRoot, ".context", "govern.local.json"),
|
|
72
|
+
JSON.stringify({
|
|
73
|
+
installs: {
|
|
74
|
+
codex: { mode: "enforced" },
|
|
75
|
+
},
|
|
76
|
+
}),
|
|
77
|
+
"utf8",
|
|
78
|
+
);
|
|
79
|
+
}
|
|
80
|
+
|
|
24
81
|
function captureStdout(run) {
|
|
25
82
|
const original = process.stdout.write.bind(process.stdout);
|
|
26
83
|
let captured = "";
|
|
@@ -37,6 +94,7 @@ function captureStdout(run) {
|
|
|
37
94
|
|
|
38
95
|
test.afterEach(() => {
|
|
39
96
|
delete process.env.CORTEX_PROJECT_ROOT;
|
|
97
|
+
delete process.env.HOME;
|
|
40
98
|
});
|
|
41
99
|
|
|
42
100
|
test("stage start: creates run + returns first envelope as JSON on stdout", async () => {
|
|
@@ -56,6 +114,8 @@ test("stage start: creates run + returns first envelope as JSON on stdout", asyn
|
|
|
56
114
|
assert.equal(parsed.state.task_id, "task-1");
|
|
57
115
|
assert.equal(parsed.state.current_stage, "plan");
|
|
58
116
|
assert.equal(parsed.envelope.expectedArtifact, "plan.md");
|
|
117
|
+
assert.equal(parsed.workflow_source, "bundled");
|
|
118
|
+
assert.equal(parsed.warnings.length, 1);
|
|
59
119
|
assert.ok(fs.existsSync(path.join(cwd, ".agents", "task-1", "state.json")));
|
|
60
120
|
});
|
|
61
121
|
|
|
@@ -71,7 +131,48 @@ test("stage start: rejects unknown workflow", async () => {
|
|
|
71
131
|
"--workflow",
|
|
72
132
|
"no-such",
|
|
73
133
|
]),
|
|
74
|
-
/
|
|
134
|
+
/Available bundled: secure-build/,
|
|
135
|
+
);
|
|
136
|
+
});
|
|
137
|
+
|
|
138
|
+
test("stage start: accepts synced workflow ids from the local cache", async () => {
|
|
139
|
+
makeWorkspace();
|
|
140
|
+
const fakeHome = fs.mkdtempSync(path.join(os.tmpdir(), "cortex-stage-home-"));
|
|
141
|
+
process.env.HOME = fakeHome;
|
|
142
|
+
writeSyncedWorkflow(fakeHome, "tiny", TINY_WORKFLOW);
|
|
143
|
+
|
|
144
|
+
const { captured } = await captureStdout(() =>
|
|
145
|
+
runStageCommand([
|
|
146
|
+
"start",
|
|
147
|
+
"--task-id",
|
|
148
|
+
"task-synced",
|
|
149
|
+
"--description",
|
|
150
|
+
"Use synced workflow",
|
|
151
|
+
"--workflow",
|
|
152
|
+
"tiny",
|
|
153
|
+
]),
|
|
154
|
+
);
|
|
155
|
+
const parsed = JSON.parse(captured);
|
|
156
|
+
assert.equal(parsed.state.workflow_id, "tiny");
|
|
157
|
+
assert.equal(parsed.workflow_source, "synced");
|
|
158
|
+
assert.deepEqual(parsed.warnings, []);
|
|
159
|
+
});
|
|
160
|
+
|
|
161
|
+
test("stage start: blocks bundled workflow fallback in enforced mode", async () => {
|
|
162
|
+
const cwd = makeWorkspace();
|
|
163
|
+
setEnforcedMode(cwd);
|
|
164
|
+
|
|
165
|
+
await assert.rejects(
|
|
166
|
+
runStageCommand([
|
|
167
|
+
"start",
|
|
168
|
+
"--task-id",
|
|
169
|
+
"task-enforced",
|
|
170
|
+
"--description",
|
|
171
|
+
"Use bundled fallback",
|
|
172
|
+
"--workflow",
|
|
173
|
+
"secure-build",
|
|
174
|
+
]),
|
|
175
|
+
/requires a synced org workflow/,
|
|
75
176
|
);
|
|
76
177
|
});
|
|
77
178
|
|
|
@@ -86,6 +86,7 @@ test("runWorkflowStart: creates run and returns the first envelope", () => {
|
|
|
86
86
|
assert.equal(result.state.current_stage, "plan");
|
|
87
87
|
assert.equal(result.state.outcome, "in_progress");
|
|
88
88
|
assert.equal(result.envelope.expectedArtifact, "plan.md");
|
|
89
|
+
assert.equal(result.workflow_source, "injected");
|
|
89
90
|
assert.match(result.envelope.prompt, /# STAGE: plan/);
|
|
90
91
|
|
|
91
92
|
// state.json persisted under .agents/<task-id>/
|
|
@@ -117,6 +118,27 @@ test("runWorkflowStart: defaults to bundled DEFAULT_WORKFLOWS when no registry g
|
|
|
117
118
|
);
|
|
118
119
|
assert.equal(result.state.workflow_id, "secure-build");
|
|
119
120
|
assert.equal(result.envelope.expectedArtifact, "plan.md");
|
|
121
|
+
assert.equal(result.workflow_source, "bundled");
|
|
122
|
+
assert.equal(result.warnings.length, 0);
|
|
123
|
+
});
|
|
124
|
+
|
|
125
|
+
test("runWorkflowStart: blocks bundled fallback when the caller requires synced workflows", () => {
|
|
126
|
+
const cwd = makeWorkspace();
|
|
127
|
+
assert.throws(
|
|
128
|
+
() =>
|
|
129
|
+
runWorkflowStart(
|
|
130
|
+
{
|
|
131
|
+
task_id: "task-enforced",
|
|
132
|
+
task_description: "Use the default workflow",
|
|
133
|
+
workflow_id: SECURE_BUILD_WORKFLOW.id,
|
|
134
|
+
},
|
|
135
|
+
{
|
|
136
|
+
cwd,
|
|
137
|
+
bundledFallbackPolicy: "block",
|
|
138
|
+
},
|
|
139
|
+
),
|
|
140
|
+
/requires a synced org workflow/,
|
|
141
|
+
);
|
|
120
142
|
});
|
|
121
143
|
|
|
122
144
|
test("runWorkflowAdvance: writes artifact + state, returns next envelope while in_progress", () => {
|
|
@@ -8,6 +8,7 @@ import {
|
|
|
8
8
|
loadSyncedWorkflows,
|
|
9
9
|
syncedWorkflowsCachePath,
|
|
10
10
|
} from "../dist/core/workflow/synced-registry.js";
|
|
11
|
+
import { resolveWorkflowDefinition } from "../dist/core/workflow/resolution.js";
|
|
11
12
|
import { runWorkflowStart } from "../dist/core/workflow/mcp-tools.js";
|
|
12
13
|
import { SECURE_BUILD_WORKFLOW } from "../dist/core/workflow/default-workflows.js";
|
|
13
14
|
|
|
@@ -173,6 +174,61 @@ test("resolveWorkflow integration: synced cache adds new workflow_ids beyond def
|
|
|
173
174
|
);
|
|
174
175
|
assert.equal(result.state.workflow_id, "tiny");
|
|
175
176
|
assert.equal(result.state.current_stage, "plan");
|
|
177
|
+
assert.equal(result.workflow_source, "synced");
|
|
178
|
+
} finally {
|
|
179
|
+
delete process.env.HOME;
|
|
180
|
+
}
|
|
181
|
+
});
|
|
182
|
+
|
|
183
|
+
test("resolveWorkflowDefinition: includes bundled fallback warning when synced cache lacks the requested id", () => {
|
|
184
|
+
const fakeHome = makeWorkspace();
|
|
185
|
+
process.env.HOME = fakeHome;
|
|
186
|
+
try {
|
|
187
|
+
const resolved = resolveWorkflowDefinition("secure-build", {
|
|
188
|
+
emitBundledFallbackWarning: true,
|
|
189
|
+
});
|
|
190
|
+
assert.equal(resolved.source, "bundled");
|
|
191
|
+
assert.equal(resolved.warnings.length, 1);
|
|
192
|
+
assert.match(resolved.warnings[0].message, /bundled registry/);
|
|
193
|
+
} finally {
|
|
194
|
+
delete process.env.HOME;
|
|
195
|
+
}
|
|
196
|
+
});
|
|
197
|
+
|
|
198
|
+
test("resolveWorkflowDefinition: can block bundled fallback when synced workflows are required", () => {
|
|
199
|
+
const fakeHome = makeWorkspace();
|
|
200
|
+
process.env.HOME = fakeHome;
|
|
201
|
+
try {
|
|
202
|
+
assert.throws(
|
|
203
|
+
() =>
|
|
204
|
+
resolveWorkflowDefinition("secure-build", {
|
|
205
|
+
bundledFallbackPolicy: "block",
|
|
206
|
+
}),
|
|
207
|
+
/requires a synced org workflow/,
|
|
208
|
+
);
|
|
209
|
+
} finally {
|
|
210
|
+
delete process.env.HOME;
|
|
211
|
+
}
|
|
212
|
+
});
|
|
213
|
+
|
|
214
|
+
test("resolveWorkflowDefinition: unknown workflow errors list bundled and synced ids", () => {
|
|
215
|
+
const fakeHome = makeWorkspace();
|
|
216
|
+
process.env.HOME = fakeHome;
|
|
217
|
+
try {
|
|
218
|
+
writeCache(path.join(fakeHome, ".cortex"), {
|
|
219
|
+
workflows: {
|
|
220
|
+
tiny: {
|
|
221
|
+
workflow_id: "tiny",
|
|
222
|
+
version: 1,
|
|
223
|
+
updated_at: "2026-05-06T12:00:00.000Z",
|
|
224
|
+
definition: TINY_WORKFLOW,
|
|
225
|
+
},
|
|
226
|
+
},
|
|
227
|
+
});
|
|
228
|
+
assert.throws(
|
|
229
|
+
() => resolveWorkflowDefinition("missing-workflow"),
|
|
230
|
+
/Available bundled: secure-build\. Available synced: tiny\./,
|
|
231
|
+
);
|
|
176
232
|
} finally {
|
|
177
233
|
delete process.env.HOME;
|
|
178
234
|
}
|