@tea-agent/loop-agent 0.13.0-alpha.0 → 0.13.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/AGENTS.md +4 -0
- package/CHANGELOG.md +57 -53
- package/README.md +12 -3
- package/dist/application/dag/generate-task-dag.js +28 -58
- package/dist/application/evaluation/candidate-hash.js +75 -0
- package/dist/application/evaluation/candidate.js +52 -0
- package/dist/application/evaluation/replay.js +289 -0
- package/dist/application/evaluation/types.js +130 -0
- package/dist/cli/command-definitions.js +17 -4
- package/dist/cli/program.js +8 -4
- package/dist/commands/eval.js +235 -0
- package/dist/commands/init.js +131 -24
- package/dist/executors/pi-sdk-executor.js +38 -24
- package/dist/executors/shell-executor.js +226 -15
- package/dist/executors/shell-presets.js +20 -0
- package/dist/executors/shell-verification.js +7 -0
- package/dist/governance/manifest-types.js +1 -0
- package/dist/infrastructure/evaluation/candidate-store.js +439 -0
- package/dist/infrastructure/evaluation/store.js +40 -0
- package/dist/task/config-types.js +23 -0
- package/dist/worker/observe/routes.js +18 -3
- package/dist/worker/observe/spec-evidence.js +1 -1
- package/dist/worker/observe/static/dom.js +160 -1
- package/dist/worker/observe/static/state.js +14 -0
- package/dist/worker/observe/static/views/dag-inspector.js +35 -4
- package/dist/worker/observe/static/views/dag.js +9 -0
- package/dist/worker/observe/static/views/dashboard.js +702 -445
- package/dist/worker/observe/static/views/session-timeline.js +15 -1
- package/dist/workflows/dag/backend-test-analysis-contract.js +120 -0
- package/dist/workflows/dag/backend-test-case-manifest.js +503 -0
- package/dist/workflows/dag/backend-test-execution-contract.js +353 -0
- package/dist/workflows/dag/backend-test-result-contract.js +568 -0
- package/dist/workflows/dag/decision-envelope.js +57 -2
- package/dist/workflows/dag/dynamic-runtime/map.js +90 -2
- package/dist/workflows/dag/frontend-implementation-contract.js +240 -0
- package/dist/workflows/dag/frontend-project-capability.js +309 -0
- package/dist/workflows/dag/frontend-repair.js +341 -0
- package/dist/workflows/dag/frontend-risk.js +161 -0
- package/dist/workflows/dag/frontend-verification-trace.js +190 -0
- package/dist/workflows/dag/init-hybrid.js +2407 -297
- package/dist/workflows/dag/node-execution.js +9 -0
- package/dist/workflows/dag/prompt.js +9 -0
- package/dist/workflows/dag/repair-artifact.js +43 -3
- package/dist/workflows/dag/report.js +35 -1
- package/dist/workflows/dag/runner.js +28 -2
- package/dist/workflows/dag/skill-instructions.js +4 -2
- package/dist/workflows/dag/task-demand-routing.js +383 -0
- package/dist/workflows/dag/types.js +71 -13
- package/dist/workflows/dag/upstream-artifacts.js +1 -0
- package/dist/workflows/dag/validate.js +59 -1
- package/docs/README.md +6 -3
- package/docs/agent-dag-recovery-playbook.md +5 -3
- package/docs/agent-dag-runner.md +3 -3
- package/docs/architecture/README.md +3 -3
- package/docs/architecture/dag-execution.md +1 -1
- package/docs/architecture/evolution.md +13 -13
- package/docs/architecture/facts-and-state.md +1 -1
- package/docs/architecture/runtime-boundaries.md +7 -7
- package/docs/architecture/system-overview.md +3 -3
- package/docs/architecture/worker-and-feature.md +3 -3
- package/docs/design/README.md +124 -42
- package/docs/development-principles.md +4 -4
- package/docs/exec-plans/active/README.md +12 -11
- package/docs/exec-plans/completed/README.md +33 -0
- package/docs/feature-workflow.md +114 -39
- package/docs/init-surface.manifest.json +30 -3
- package/docs/loop-agent-harness.md +9 -8
- package/docs/production-readiness.md +1 -1
- package/docs/progress/README.md +23 -1
- package/docs/reports/README.md +65 -6
- package/docs/skills/vetted-skill-registry.md +2 -0
- package/docs/templates/agent-dag.schema.json +29 -1
- package/docs/templates/agent-dag.supervised-implementation.json +127 -8
- package/docs/templates/backend-test-analysis.schema.json +44 -0
- package/docs/templates/backend-test-case-manifest.schema.json +190 -0
- package/docs/templates/backend-test-dag.classify.prompt.md +75 -0
- package/docs/templates/backend-test-dag.generate-pytest.prompt.md +81 -16
- package/docs/templates/backend-test-dag.json +311 -40
- package/docs/templates/backend-test-dag.retrospect.prompt.md +44 -30
- package/docs/templates/backend-test-dag.review-cases.prompt.md +10 -8
- package/docs/templates/backend-test-execution.schema.json +133 -0
- package/docs/templates/backend-test-result.schema.json +99 -0
- package/docs/templates/branch-merge-report.md +93 -0
- package/docs/templates/frontend-design-contract.md +9 -0
- package/docs/templates/frontend-eval/fixtures/failures/01-type-build-error.md +17 -0
- package/docs/templates/frontend-eval/fixtures/failures/02-unit-component-test-fail.md +16 -0
- package/docs/templates/frontend-eval/fixtures/failures/03-fixture-schema-drift.md +16 -0
- package/docs/templates/frontend-eval/fixtures/failures/04-missing-loading-empty-error-state.md +16 -0
- package/docs/templates/frontend-eval/fixtures/failures/05-forbidden-write-writeset-expansion.md +16 -0
- package/docs/templates/frontend-eval/fixtures/failures/06-unapproved-dependency-add.md +16 -0
- package/docs/templates/frontend-eval/fixtures/failures/07-mock-production-on.md +21 -0
- package/docs/templates/frontend-eval/fixtures/functional/01-simple-component-style.md +29 -0
- package/docs/templates/frontend-eval/fixtures/functional/02-form-validation.md +28 -0
- package/docs/templates/frontend-eval/fixtures/functional/03-list-detail-page.md +28 -0
- package/docs/templates/frontend-eval/fixtures/functional/04-api-mock.md +29 -0
- package/docs/templates/frontend-eval/fixtures/functional/05-permission-auth-gated-ui.md +27 -0
- package/docs/templates/frontend-eval/fixtures/functional/06-ssr-server-client-boundary.md +28 -0
- package/docs/templates/frontend-eval/fixtures/functional/07-shared-public-component-api.md +28 -0
- package/docs/templates/frontend-eval/fixtures/functional/08-pure-local-no-remote.md +27 -0
- package/docs/templates/frontend-eval/metrics.md +138 -0
- package/docs/templates/frontend-eval/smoke-targets.md +53 -0
- package/docs/templates/frontend-implementation-contract.schema.json +27 -0
- package/docs/templates/frontend-task-constraints.md +10 -0
- package/docs/templates/frontend-task-requirement.md +9 -0
- package/docs/templates/frontend-test-dag.generate-cases.prompt.md +5 -0
- package/docs/templates/frontend-test-dag.json +23 -0
- package/docs/templates/frontend-test-dag.retrieve-context.prompt.md +3 -0
- package/docs/templates/frontend-test-dag.retrospect.prompt.md +3 -0
- package/docs/templates/frontend-test-dag.review-cases.prompt.md +3 -0
- package/docs/templates/frontend-test-dag.review-execution.prompt.md +3 -0
- package/docs/templates/knowledge-graph-bootstrap-dag.json +1 -1
- package/docs/templates/knowledge-sync-dag.json +1 -0
- package/docs/verification-matrix.md +4 -1
- package/examples/decision-gate-agent-dag.json +4 -4
- package/examples/hybrid-loop-agent-dag.json +1 -1
- package/package.json +2 -2
- package/scripts/kb-bootstrap-init-skeleton.sh +2 -1
- package/scripts/kb-graph-incremental-prepare.mjs +19 -5
- package/scripts/kb-graph-promote.mjs +12 -1
- package/skills/ai-engineering-context/SKILL.md +2 -2
- package/skills/analyze-product-dependencies/SKILL.md +67 -0
- package/skills/analyze-product-dependencies/agents/openai.yaml +4 -0
- package/skills/analyze-product-dependencies/references/api-documentation-schema.md +30 -0
- package/skills/analyze-product-dependencies/references/dependency-analysis-schema.md +28 -0
- package/skills/analyze-product-dependencies/references/example.md +76 -0
- package/skills/analyze-product-dependencies/references/forward-test-cases.md +35 -0
- package/skills/analyze-product-dependencies/references/input-contract.md +11 -0
- package/skills/analyze-product-dependencies/references/scouting-rules.md +61 -0
- package/skills/analyze-product-dependencies/scripts/test-validators.mjs +267 -0
- package/skills/analyze-product-dependencies/scripts/validate-api-documentation.mjs +101 -0
- package/skills/analyze-product-dependencies/scripts/validate-dependency-analysis.mjs +142 -0
- package/skills/analyze-product-dependencies/scripts/validate-product-requirement-input.mjs +76 -0
- package/skills/analyze-product-dependencies/scripts/validation-helpers.mjs +146 -0
- package/skills/analyze-product-requirements/SKILL.md +90 -0
- package/skills/analyze-product-requirements/agents/openai.yaml +4 -0
- package/skills/analyze-product-requirements/references/acceptance-criteria.md +91 -0
- package/skills/analyze-product-requirements/references/clarification-and-knowledge.md +56 -0
- package/skills/analyze-product-requirements/references/example.md +86 -0
- package/skills/analyze-product-requirements/references/forward-test-cases.md +66 -0
- package/skills/analyze-product-requirements/references/product-analysis-schema.md +32 -0
- package/skills/analyze-product-requirements/references/product-requirement-schema.md +33 -0
- package/skills/analyze-product-requirements/references/requirement-clarification-schema.md +35 -0
- package/skills/analyze-product-requirements/scripts/test-validators.mjs +193 -0
- package/skills/analyze-product-requirements/scripts/validate-product-analysis.mjs +69 -0
- package/skills/analyze-product-requirements/scripts/validate-product-requirement.mjs +97 -0
- package/skills/analyze-product-requirements/scripts/validate-requirement-clarification.mjs +98 -0
- package/skills/analyze-product-requirements/scripts/validation-helpers.mjs +156 -0
- package/skills/browser-tools/SKILL.md +196 -0
- package/skills/browser-tools/browser-content.js +103 -0
- package/skills/browser-tools/browser-cookies.js +35 -0
- package/skills/browser-tools/browser-eval.js +53 -0
- package/skills/browser-tools/browser-hn-scraper.js +108 -0
- package/skills/browser-tools/browser-nav.js +44 -0
- package/skills/browser-tools/browser-pick.js +162 -0
- package/skills/browser-tools/browser-screenshot.js +34 -0
- package/skills/browser-tools/browser-start.js +86 -0
- package/skills/browser-tools/package-lock.json +2556 -0
- package/skills/browser-tools/package.json +19 -0
- package/skills/frontend-design-review/SKILL.md +6 -1
- package/skills/frontend-design-review/references/review-checklist.md +25 -4
- package/skills/frontend-implementation/SKILL.md +25 -30
- package/skills/frontend-implementation/references/code-standards.md +20 -22
- package/skills/frontend-implementation/references/node-contracts.md +17 -53
- package/skills/frontend-review/SKILL.md +10 -4
- package/skills/frontend-review/references/review-findings.md +8 -3
- package/skills/frontend-verification/SKILL.md +22 -9
- package/skills/frontend-verification/references/verification-checklist.md +17 -5
- package/skills/grill-with-docs/SKILL.md +5 -5
- package/skills/grill-with-docs/adr-format.md +3 -3
- package/skills/init-capability-evolution/SKILL.md +5 -5
- package/skills/loop-agent/SKILL.md +5 -5
- package/skills/loop-agent/references/README.md +3 -3
- package/skills/loop-agent/references/command-reference.md +98 -24
- package/skills/loop-agent/references/docs-converge.md +15 -15
- package/skills/loop-agent/references/harness-policy.md +2 -2
- package/skills/loop-agent/references/hybrid-dag.md +32 -22
- package/skills/loop-agent/references/multi-worktree.md +1 -1
- package/skills/loop-agent/references/orchestrator-and-interventions.md +8 -8
- package/skills/loop-agent/references/task-workflow.md +1 -1
- package/skills/loop-agent/references/verification-and-failure-handling.md +6 -4
- package/skills/playwright-cli/SKILL.md +420 -0
- package/skills/playwright-cli/references/element-attributes.md +23 -0
- package/skills/playwright-cli/references/playwright-tests.md +39 -0
- package/skills/playwright-cli/references/request-mocking.md +87 -0
- package/skills/playwright-cli/references/running-code.md +241 -0
- package/skills/playwright-cli/references/session-management.md +225 -0
- package/skills/playwright-cli/references/storage-state.md +275 -0
- package/skills/playwright-cli/references/test-generation.md +433 -0
- package/skills/playwright-cli/references/tracing.md +139 -0
- package/skills/playwright-cli/references/video-recording.md +143 -0
- package/skills/playwright-cli-case-generator/SKILL.md +74 -0
- package/skills/requesting-code-review/SKILL.md +1 -1
- package/skills/systematic-debugging/CREATION-LOG.md +3 -3
- package/skills/systematic-debugging/SKILL.md +1 -1
- package/skills/systematic-debugging/test-academic.md +1 -1
- package/skills/systematic-debugging/test-pressure-1.md +1 -1
- package/skills/systematic-debugging/test-pressure-2.md +1 -1
- package/skills/systematic-debugging/test-pressure-3.md +1 -1
- package/skills/verification-before-completion/SKILL.md +1 -1
|
@@ -0,0 +1,353 @@
|
|
|
1
|
+
import { createHash } from "node:crypto";
|
|
2
|
+
import { accessSync, constants, existsSync } from "node:fs";
|
|
3
|
+
import { readFile } from "node:fs/promises";
|
|
4
|
+
import path from "node:path";
|
|
5
|
+
import { z } from "zod";
|
|
6
|
+
import { writeDagRunJsonArtifact } from "../../infrastructure/harness/artifact-store.js";
|
|
7
|
+
import { ENV_VAR_NAME_PATTERN } from "./types.js";
|
|
8
|
+
export const BACKEND_TEST_EXECUTION_SCHEMA_ID = "backend-test-execution-v1";
|
|
9
|
+
export const BACKEND_TEST_EXECUTION_DEFAULT_TEST_ROOT = "testcase";
|
|
10
|
+
const SECRET_KEY = /(?:password|passwd|secret|token|api[_-]?key|private[_-]?key|credential|authorization)/i;
|
|
11
|
+
const SECRET_VALUE = /(?:-----BEGIN [A-Z ]*PRIVATE KEY-----|\b(?:sk|ghp|github_pat|xox[baprs]|AKIA)[-_A-Za-z0-9]{12,}\b)/;
|
|
12
|
+
const envNameSchema = z
|
|
13
|
+
.string()
|
|
14
|
+
.regex(ENV_VAR_NAME_PATTERN, "env name must match ^[A-Z_][A-Z0-9_]*$");
|
|
15
|
+
const relativePosixPathSchema = z
|
|
16
|
+
.string()
|
|
17
|
+
.min(1)
|
|
18
|
+
.refine((value) => {
|
|
19
|
+
try {
|
|
20
|
+
normalizeRelativePosixPath(value);
|
|
21
|
+
return true;
|
|
22
|
+
}
|
|
23
|
+
catch {
|
|
24
|
+
return false;
|
|
25
|
+
}
|
|
26
|
+
}, { message: "path must be repo-relative posix without .. or absolute form" });
|
|
27
|
+
const fixtureSchema = z
|
|
28
|
+
.object({
|
|
29
|
+
name: z.string().min(1),
|
|
30
|
+
sourcePath: relativePosixPathSchema,
|
|
31
|
+
kind: z.string().min(1),
|
|
32
|
+
})
|
|
33
|
+
.strict();
|
|
34
|
+
const readinessSchema = z
|
|
35
|
+
.object({
|
|
36
|
+
path: relativePosixPathSchema,
|
|
37
|
+
description: z.string().min(1),
|
|
38
|
+
})
|
|
39
|
+
.strict();
|
|
40
|
+
const dataIsolationSchema = z
|
|
41
|
+
.object({
|
|
42
|
+
mode: z.string().min(1),
|
|
43
|
+
evidence: z.string().min(1).optional(),
|
|
44
|
+
})
|
|
45
|
+
.strict();
|
|
46
|
+
const managedCommandSchema = z
|
|
47
|
+
.object({
|
|
48
|
+
start: z.string().min(1).optional(),
|
|
49
|
+
stop: z.string().min(1).optional(),
|
|
50
|
+
sourceRef: z.string().min(1).optional(),
|
|
51
|
+
})
|
|
52
|
+
.strict();
|
|
53
|
+
const runnerSchema = z
|
|
54
|
+
.object({
|
|
55
|
+
commandParts: z.array(z.string().min(1)).optional(),
|
|
56
|
+
frozenCommandHints: z.array(z.string().min(1)).optional(),
|
|
57
|
+
})
|
|
58
|
+
.strict()
|
|
59
|
+
.refine((value) => (value.commandParts?.length ?? 0) > 0 ||
|
|
60
|
+
(value.frozenCommandHints?.length ?? 0) > 0, { message: "runner requires commandParts and/or frozenCommandHints" });
|
|
61
|
+
const evidenceGapSchema = z
|
|
62
|
+
.object({
|
|
63
|
+
description: z.string().min(1),
|
|
64
|
+
sourceRef: z.string().min(1).optional(),
|
|
65
|
+
})
|
|
66
|
+
.strict();
|
|
67
|
+
export const backendTestExecutionContractSchema = z
|
|
68
|
+
.object({
|
|
69
|
+
schemaVersion: z.literal(1),
|
|
70
|
+
framework: z.literal("pytest"),
|
|
71
|
+
runner: runnerSchema,
|
|
72
|
+
testRoot: relativePosixPathSchema,
|
|
73
|
+
workingDirectory: relativePosixPathSchema.default("."),
|
|
74
|
+
report: z
|
|
75
|
+
.object({
|
|
76
|
+
format: z.literal("junit"),
|
|
77
|
+
relativeHint: z.string().min(1),
|
|
78
|
+
})
|
|
79
|
+
.strict(),
|
|
80
|
+
targetMode: z.enum([
|
|
81
|
+
"in-process",
|
|
82
|
+
"external-running-service",
|
|
83
|
+
"managed-command",
|
|
84
|
+
]),
|
|
85
|
+
baseUrlEnvName: envNameSchema.optional(),
|
|
86
|
+
readiness: z.array(readinessSchema).default([]),
|
|
87
|
+
existingFixtures: z.array(fixtureSchema).default([]),
|
|
88
|
+
authenticationMode: z.string().min(1),
|
|
89
|
+
requiredEnvNames: z.array(envNameSchema).default([]),
|
|
90
|
+
dataIsolation: dataIsolationSchema,
|
|
91
|
+
managedCommand: managedCommandSchema.optional(),
|
|
92
|
+
evidenceGaps: z.array(evidenceGapSchema).default([]),
|
|
93
|
+
evidenceRefs: z.array(z.string().min(1)).default([]),
|
|
94
|
+
})
|
|
95
|
+
.strict()
|
|
96
|
+
.superRefine((value, ctx) => {
|
|
97
|
+
if (value.targetMode === "external-running-service" &&
|
|
98
|
+
!value.baseUrlEnvName) {
|
|
99
|
+
ctx.addIssue({
|
|
100
|
+
code: z.ZodIssueCode.custom,
|
|
101
|
+
message: "baseUrlEnvName is required when targetMode is external-running-service",
|
|
102
|
+
path: ["baseUrlEnvName"],
|
|
103
|
+
});
|
|
104
|
+
}
|
|
105
|
+
if (value.targetMode === "managed-command") {
|
|
106
|
+
if (!value.managedCommand) {
|
|
107
|
+
ctx.addIssue({
|
|
108
|
+
code: z.ZodIssueCode.custom,
|
|
109
|
+
message: "managedCommand is required when targetMode is managed-command",
|
|
110
|
+
path: ["managedCommand"],
|
|
111
|
+
});
|
|
112
|
+
}
|
|
113
|
+
else if (!value.managedCommand.sourceRef?.trim()) {
|
|
114
|
+
ctx.addIssue({
|
|
115
|
+
code: z.ZodIssueCode.custom,
|
|
116
|
+
message: "managed-command requires managedCommand.sourceRef evidence from task source; do not default-enable managed mode",
|
|
117
|
+
path: ["managedCommand", "sourceRef"],
|
|
118
|
+
});
|
|
119
|
+
}
|
|
120
|
+
}
|
|
121
|
+
if (value.baseUrlEnvName &&
|
|
122
|
+
!value.requiredEnvNames.includes(value.baseUrlEnvName)) {
|
|
123
|
+
// soft consistency: not a hard fail — preflight will still require env presence
|
|
124
|
+
}
|
|
125
|
+
});
|
|
126
|
+
export function normalizeRelativePosixPath(value) {
|
|
127
|
+
const trimmed = value.trim();
|
|
128
|
+
if (!trimmed)
|
|
129
|
+
throw new Error("path must be non-empty");
|
|
130
|
+
if (path.posix.isAbsolute(trimmed) || path.win32.isAbsolute(trimmed)) {
|
|
131
|
+
throw new Error("absolute paths are forbidden");
|
|
132
|
+
}
|
|
133
|
+
if (trimmed.includes("\\")) {
|
|
134
|
+
throw new Error("path must use posix separators");
|
|
135
|
+
}
|
|
136
|
+
const segments = trimmed.split("/").filter((part, index, arr) => {
|
|
137
|
+
if (part === "" && index > 0 && index < arr.length - 1)
|
|
138
|
+
return false;
|
|
139
|
+
return true;
|
|
140
|
+
});
|
|
141
|
+
const normalizedSegments = [];
|
|
142
|
+
for (const segment of segments) {
|
|
143
|
+
if (segment === "" || segment === ".")
|
|
144
|
+
continue;
|
|
145
|
+
if (segment === "..")
|
|
146
|
+
throw new Error("path traversal is forbidden");
|
|
147
|
+
normalizedSegments.push(segment);
|
|
148
|
+
}
|
|
149
|
+
if (normalizedSegments.length === 0)
|
|
150
|
+
return ".";
|
|
151
|
+
return normalizedSegments.join("/");
|
|
152
|
+
}
|
|
153
|
+
export function secretIssues(value, at = "$", issues = []) {
|
|
154
|
+
if (typeof value === "string" && SECRET_VALUE.test(value)) {
|
|
155
|
+
issues.push(`${at}: secret-like value is forbidden`);
|
|
156
|
+
}
|
|
157
|
+
if (Array.isArray(value)) {
|
|
158
|
+
value.forEach((item, index) => secretIssues(item, `${at}[${index}]`, issues));
|
|
159
|
+
}
|
|
160
|
+
else if (value && typeof value === "object") {
|
|
161
|
+
for (const [key, child] of Object.entries(value)) {
|
|
162
|
+
if (SECRET_KEY.test(key)) {
|
|
163
|
+
issues.push(`${at}.${key}: secret-shaped field name is forbidden`);
|
|
164
|
+
}
|
|
165
|
+
secretIssues(child, `${at}.${key}`, issues);
|
|
166
|
+
}
|
|
167
|
+
}
|
|
168
|
+
return issues;
|
|
169
|
+
}
|
|
170
|
+
export function extractStrictJsonObject(text) {
|
|
171
|
+
const trimmed = text.trim();
|
|
172
|
+
if (trimmed.startsWith("{") && trimmed.endsWith("}")) {
|
|
173
|
+
return JSON.parse(trimmed);
|
|
174
|
+
}
|
|
175
|
+
const blocks = [...trimmed.matchAll(/```json\s*\n([\s\S]*?)\n```/gi)];
|
|
176
|
+
if (blocks.length !== 1 || trimmed.replace(blocks[0][0], "").trim()) {
|
|
177
|
+
throw new Error("execution contract output must be one pure JSON object or one fenced json block with no trailing text");
|
|
178
|
+
}
|
|
179
|
+
return JSON.parse(blocks[0][1]);
|
|
180
|
+
}
|
|
181
|
+
function normalizeContractPaths(contract) {
|
|
182
|
+
const existingFixtures = contract.existingFixtures ?? [];
|
|
183
|
+
const readiness = contract.readiness ?? [];
|
|
184
|
+
const requiredEnvNames = contract.requiredEnvNames ?? [];
|
|
185
|
+
const evidenceGaps = contract.evidenceGaps ?? [];
|
|
186
|
+
const evidenceRefs = contract.evidenceRefs ?? [];
|
|
187
|
+
return {
|
|
188
|
+
...contract,
|
|
189
|
+
testRoot: normalizeRelativePosixPath(contract.testRoot),
|
|
190
|
+
workingDirectory: normalizeRelativePosixPath(contract.workingDirectory ?? "."),
|
|
191
|
+
existingFixtures: existingFixtures.map((fixture) => ({
|
|
192
|
+
...fixture,
|
|
193
|
+
sourcePath: normalizeRelativePosixPath(fixture.sourcePath),
|
|
194
|
+
})),
|
|
195
|
+
readiness: readiness.map((item) => ({
|
|
196
|
+
...item,
|
|
197
|
+
path: normalizeRelativePosixPath(item.path),
|
|
198
|
+
})),
|
|
199
|
+
requiredEnvNames,
|
|
200
|
+
evidenceGaps,
|
|
201
|
+
evidenceRefs,
|
|
202
|
+
};
|
|
203
|
+
}
|
|
204
|
+
export function assertBackendTestExecutionPreflight(input) {
|
|
205
|
+
const errors = [];
|
|
206
|
+
const contract = normalizeContractPaths(input.contract);
|
|
207
|
+
if (contract.framework !== "pytest") {
|
|
208
|
+
errors.push(`unsupported framework: ${String(contract.framework)}`);
|
|
209
|
+
}
|
|
210
|
+
const testRoot = contract.testRoot;
|
|
211
|
+
const workingDirectory = contract.workingDirectory;
|
|
212
|
+
if (input.expectedTestRoot) {
|
|
213
|
+
const expected = normalizeRelativePosixPath(input.expectedTestRoot);
|
|
214
|
+
if (expected !== testRoot) {
|
|
215
|
+
errors.push(`testRoot mismatch: contract=${testRoot} expectedTestRoot=${expected}`);
|
|
216
|
+
}
|
|
217
|
+
}
|
|
218
|
+
if (!input.skipFilesystemChecks) {
|
|
219
|
+
const rootAbs = path.resolve(input.cwd, testRoot);
|
|
220
|
+
const workAbs = path.resolve(input.cwd, workingDirectory);
|
|
221
|
+
if (!rootAbs.startsWith(path.resolve(input.cwd))) {
|
|
222
|
+
errors.push(`testRoot escapes working tree: ${testRoot}`);
|
|
223
|
+
}
|
|
224
|
+
if (!workAbs.startsWith(path.resolve(input.cwd))) {
|
|
225
|
+
errors.push(`workingDirectory escapes working tree: ${workingDirectory}`);
|
|
226
|
+
}
|
|
227
|
+
if (!existsSync(rootAbs)) {
|
|
228
|
+
errors.push(`testRoot does not exist: ${testRoot}`);
|
|
229
|
+
}
|
|
230
|
+
else {
|
|
231
|
+
try {
|
|
232
|
+
accessSync(rootAbs, constants.R_OK);
|
|
233
|
+
}
|
|
234
|
+
catch {
|
|
235
|
+
errors.push(`testRoot is not readable: ${testRoot}`);
|
|
236
|
+
}
|
|
237
|
+
}
|
|
238
|
+
}
|
|
239
|
+
for (const name of contract.requiredEnvNames) {
|
|
240
|
+
const value = input.env[name];
|
|
241
|
+
if (value === undefined || value === "") {
|
|
242
|
+
errors.push(`required env missing or empty: ${name}`);
|
|
243
|
+
}
|
|
244
|
+
}
|
|
245
|
+
if (contract.targetMode === "external-running-service") {
|
|
246
|
+
if (!contract.baseUrlEnvName) {
|
|
247
|
+
errors.push("external-running-service requires baseUrlEnvName before writer/execute");
|
|
248
|
+
}
|
|
249
|
+
else {
|
|
250
|
+
const value = input.env[contract.baseUrlEnvName];
|
|
251
|
+
if (value === undefined || value === "") {
|
|
252
|
+
errors.push(`external base URL env missing or empty: ${contract.baseUrlEnvName}`);
|
|
253
|
+
}
|
|
254
|
+
}
|
|
255
|
+
}
|
|
256
|
+
if (contract.targetMode === "managed-command") {
|
|
257
|
+
if (!contract.managedCommand?.sourceRef?.trim()) {
|
|
258
|
+
errors.push("managed-command is fail-closed without managedCommand.sourceRef task-source evidence");
|
|
259
|
+
}
|
|
260
|
+
}
|
|
261
|
+
if ((contract.evidenceGaps?.length ?? 0) > 0) {
|
|
262
|
+
errors.push(`execution contract has ${contract.evidenceGaps.length} evidenceGaps; fail-closed before writer/execute`);
|
|
263
|
+
}
|
|
264
|
+
if (contract.targetMode === "in-process" &&
|
|
265
|
+
(contract.existingFixtures?.length ?? 0) === 0) {
|
|
266
|
+
errors.push("in-process targetMode requires at least one existingFixtures entry (or record evidenceGaps)");
|
|
267
|
+
}
|
|
268
|
+
return {
|
|
269
|
+
ok: errors.length === 0,
|
|
270
|
+
errors,
|
|
271
|
+
testRoot,
|
|
272
|
+
workingDirectory,
|
|
273
|
+
};
|
|
274
|
+
}
|
|
275
|
+
export async function materializeBackendTestExecutionContract(input) {
|
|
276
|
+
if (!/^[a-z0-9][a-z0-9._-]*\.json$/.test(input.artifactName) ||
|
|
277
|
+
!/^[a-z0-9][a-z0-9._-]*$/.test(input.outputDir)) {
|
|
278
|
+
throw new Error("unsafe structured artifact path");
|
|
279
|
+
}
|
|
280
|
+
const nodePath = path.join(input.runDir, `${input.fromNodeId}.json`);
|
|
281
|
+
const record = JSON.parse(await readFile(nodePath, "utf8"));
|
|
282
|
+
const raw = record.assistantText?.trim() || record.stdout?.trim() || "";
|
|
283
|
+
let parsed;
|
|
284
|
+
try {
|
|
285
|
+
parsed = extractStrictJsonObject(raw);
|
|
286
|
+
}
|
|
287
|
+
catch (error) {
|
|
288
|
+
throw new Error(`invalid-output: ${error instanceof Error ? error.message : String(error)}`);
|
|
289
|
+
}
|
|
290
|
+
const secrets = secretIssues(parsed);
|
|
291
|
+
if (secrets.length) {
|
|
292
|
+
throw new Error(`invalid-output: ${secrets.join("; ")}`);
|
|
293
|
+
}
|
|
294
|
+
const result = backendTestExecutionContractSchema.safeParse(parsed);
|
|
295
|
+
if (!result.success) {
|
|
296
|
+
throw new Error(`invalid-output: ${result.error.issues
|
|
297
|
+
.map((issue) => `${issue.path.join(".")}: ${issue.message}`)
|
|
298
|
+
.join("; ")}`);
|
|
299
|
+
}
|
|
300
|
+
const normalized = normalizeContractPaths(result.data);
|
|
301
|
+
// Incomplete discovery must not become a green gate that unblocks pytest writers.
|
|
302
|
+
if ((normalized.evidenceGaps?.length ?? 0) > 0) {
|
|
303
|
+
throw new Error(`invalid-output: evidenceGaps present (${normalized.evidenceGaps.length}); fail-closed until scout resolves gaps or records an explicit alternate contract`);
|
|
304
|
+
}
|
|
305
|
+
if (normalized.targetMode === "in-process" &&
|
|
306
|
+
(normalized.existingFixtures?.length ?? 0) === 0) {
|
|
307
|
+
throw new Error("invalid-output: in-process targetMode requires existingFixtures evidence");
|
|
308
|
+
}
|
|
309
|
+
const relativePath = path.posix.join(input.outputDir, input.artifactName);
|
|
310
|
+
const artifactPath = await writeDagRunJsonArtifact(input.runDir, relativePath, normalized);
|
|
311
|
+
const serialized = `${JSON.stringify(normalized, null, 2)}\n`;
|
|
312
|
+
return {
|
|
313
|
+
path: artifactPath,
|
|
314
|
+
sha256: createHash("sha256").update(serialized).digest("hex"),
|
|
315
|
+
schemaId: BACKEND_TEST_EXECUTION_SCHEMA_ID,
|
|
316
|
+
};
|
|
317
|
+
}
|
|
318
|
+
/** Shell-friendly preflight body used by execute-backend-pytest-shell. */
|
|
319
|
+
export function buildBackendTestExecutionPreflightShellSnippet(options) {
|
|
320
|
+
const contractRelativePath = options?.contractRelativePath ?? "contracts/backend-test-execution.json";
|
|
321
|
+
const expectedTestRoot = options?.expectedTestRoot ?? BACKEND_TEST_EXECUTION_DEFAULT_TEST_ROOT;
|
|
322
|
+
// Keep this deterministic and free of secret values. Node is available in the harness.
|
|
323
|
+
// Keep node -e and its path argument in ONE shell command (space-separated),
|
|
324
|
+
// not joined by ";" — otherwise "${CONTRACT}" becomes a separate exec attempt.
|
|
325
|
+
const nodePreflight = [
|
|
326
|
+
"node -e ",
|
|
327
|
+
`'const fs=require("fs");const path=require("path");`,
|
|
328
|
+
`const contractPath=process.argv[1];`,
|
|
329
|
+
`const contract=JSON.parse(fs.readFileSync(contractPath,"utf8"));`,
|
|
330
|
+
`const expected=${JSON.stringify(expectedTestRoot)};`,
|
|
331
|
+
`const errors=[];`,
|
|
332
|
+
`if(contract.framework!=="pytest") errors.push("framework must be pytest");`,
|
|
333
|
+
`const testRoot=String(contract.testRoot||"");`,
|
|
334
|
+
`if(!testRoot||testRoot.includes("..")||path.isAbsolute(testRoot)) errors.push("unsafe testRoot");`,
|
|
335
|
+
`if(testRoot.replace(/\\/+$/,"")!==expected.replace(/\\/+$/,"")) errors.push("testRoot mismatch vs frozen command: "+testRoot+" !== "+expected);`,
|
|
336
|
+
`const rootAbs=path.resolve(process.cwd(),testRoot);`,
|
|
337
|
+
`if(!fs.existsSync(rootAbs)) errors.push("testRoot does not exist: "+testRoot);`,
|
|
338
|
+
`if(Array.isArray(contract.evidenceGaps)&&contract.evidenceGaps.length) errors.push("evidenceGaps present: "+contract.evidenceGaps.length);`,
|
|
339
|
+
`if(contract.targetMode==="in-process"&&!(Array.isArray(contract.existingFixtures)&&contract.existingFixtures.length)) errors.push("in-process requires existingFixtures");`,
|
|
340
|
+
`for (const name of (contract.requiredEnvNames||[])) { if(!process.env[name]) errors.push("required env missing: "+name); }`,
|
|
341
|
+
`if(contract.targetMode==="external-running-service"){ const n=contract.baseUrlEnvName; if(!n||!process.env[n]) errors.push("external base URL env missing: "+String(n||"<empty>")); }`,
|
|
342
|
+
`if(contract.targetMode==="managed-command" && !(contract.managedCommand&&contract.managedCommand.sourceRef)) errors.push("managed-command requires sourceRef evidence");`,
|
|
343
|
+
`if(errors.length){ console.error(errors.join("; ")); process.exit(2);} `,
|
|
344
|
+
`console.log("backend-test preflight ok: framework=pytest testRoot="+testRoot+" targetMode="+contract.targetMode);'`,
|
|
345
|
+
' "${CONTRACT}"',
|
|
346
|
+
].join("");
|
|
347
|
+
return [
|
|
348
|
+
'test -n "${HARNESS_DAG_RUN_DIR:-}" || { echo "missing HARNESS_DAG_RUN_DIR for backend pytest preflight" >&2; exit 2; }',
|
|
349
|
+
`CONTRACT="\${HARNESS_DAG_RUN_DIR}/${contractRelativePath}"`,
|
|
350
|
+
'test -f "${CONTRACT}" || { echo "missing backend-test execution contract: ${CONTRACT}" >&2; exit 2; }',
|
|
351
|
+
nodePreflight,
|
|
352
|
+
].join("; ");
|
|
353
|
+
}
|