Files
callstack__agent-device/scripts/fuzz/validation-arbitraries-maestro.ts
Michał Pierzchała 021fe2aa1d feat(maestro): support assertTrue phase 1 - literal/${VAR} truthiness (#1295) (#2010)
* feat(maestro): support assertTrue phase 1 - literal/${VAR} truthiness (#1295)

Adds the assertTrue command, scoped to literal values and bare ${VAR}
lookups per the #1292 lookup-only decision; JS expressions keep
failing loud at parse time with a runScript hint. Truthiness on a
looked-up value is evaluated against a pinned falsy-string table
("", "false", "0", "null", "undefined") since flow config/env/
runScript-output values are always stored as strings, rather than
native JS truthiness (which would treat "false" as truthy).

Wires assertTrue through the parser, interpreter, optional/warning
composition, and the layer-1 conformance oracle, narrowing the
067_assertTrue_pass divergence to the JS-expression case and removing
the now-satisfied 076_optional_assertion entry. Also materializes
scrollUntilVisible's default direction in the conformance canonical
projection, a latent gap only exposed once 076 could fully compare.

* fix(maestro): correct assertTrue truthiness claim in CLI help/docs

The support-matrix text said assertTrue is "evaluated with JS
truthiness", but the engine actually uses a pinned falsy-string table
("", "false", "0", "null", "undefined") since looked-up values always
arrive as strings — native JS truthiness would treat "false" as
truthy. Spell out the actual rule instead of the misleading claim.

* fix(maestro): fix oxfmt quote-style violation in expected-divergence.ts

CI's format gate failed on a single-quoted string containing an
apostrophe; oxfmt prefers double quotes there.

* fix(maestro): bump eager-closure-budget pin for the new truthiness module

engine-truthiness.ts is a genuinely new module on the core interpreter
path (assertTrue is dispatched unconditionally by
replay-plan-step-execution.ts), so packages/maestro/src/index.ts now
eagerly evaluates 105 modules instead of 104 — a deliberate growth,
not a laziness regression.

---------

Co-authored-by: Claude <noreply@anthropic.com>
2026-08-24 20:15:26 +02:00

148 lines
6.1 KiB
TypeScript

// Maestro flow case generation for the `maestro-validation` fuzz target (#1781 B2).
//
// Cases are built from the command shapes the converter accepts, then one shape rule is violated,
// so the failure surfaces in command-shape validation rather than in the YAML tokenizer. The
// shapes are rendered here because Maestro's accepted surface is a parser table rather than
// exported data; the test asserts this file covers `SUPPORTED_MAESTRO_COMMAND_NAMES`, so the copy
// cannot silently drift from the converter.
import fc from 'fast-check';
import { encodeValidationCase } from './validation-case.ts';
import { ACCEPT, SAFE_VALUES } from './validation-values.ts';
// ---------------------------------------------------------------------------------------------
const yamlText = (salt: number): string =>
JSON.stringify(SAFE_VALUES[salt % SAFE_VALUES.length] || 'Login');
/** One valid command rendered as YAML list-entry lines. */
function validMaestroCommand(pick: number, salt: number): string[] {
const text = yamlText(salt);
const options: (() => string[])[] = [
() => ['- back'],
() => ['- hideKeyboard'],
() => ['- stopApp'],
() => ['- scroll'],
() => ['- waitForAnimationToEnd'],
() => ['- eraseText'],
() => [`- eraseText: ${1 + (salt % 9)}`],
() => ['- launchApp'],
() => [`- launchApp: ${text}`],
() => [`- tapOn: ${text}`],
() => ['- tapOn:', ` id: ${text}`],
() => ['- tapOn:', ` text: ${text}`],
() => [`- doubleTapOn: ${text}`],
() => [`- longPressOn: ${text}`],
() => [`- inputText: ${text}`],
() => ['- openLink: "https://example.com/page"'],
() => [`- assertVisible: ${text}`],
() => [`- assertNotVisible: ${text}`],
() => ['- assertTrue: true'],
() => [`- takeScreenshot: ${text}`],
() => ['- swipe:', ` direction: ${['UP', 'DOWN', 'LEFT', 'RIGHT'][salt % 4]}`],
() => [`- pressKey: ${['back', 'enter', 'return', 'home'][salt % 4]}`],
() => ['- extendedWaitUntil:', ` visible: ${text}`, ' timeout: 500'],
() => ['- scrollUntilVisible:', ' element:', ` text: ${text}`],
() => ['- repeat:', ' times: 2', ' commands:', ' - back'],
() => ['- runFlow: other.yaml'],
];
return options[pick % options.length]!();
}
const FAKE_MAESTRO_COMMANDS = [
'clickOn',
'tapOnPoint',
'evalScript',
'launchActivity',
'inputTextt',
] as const;
/** `code` is per class, like the CLI table: a class whose contract changes moves alone. */
type MaestroMutation = { name: string; code: string; lines: (salt: number) => string[] };
const MAESTRO_MUTATIONS: readonly MaestroMutation[] = [
{
// The B2 headline: an unknown command name must die in command validation, not the
// YAML tokenizer — mutated names used to be unreachable because the YAML never parsed.
name: 'unsupported-command',
code: 'INVALID_ARGS',
lines: (salt) => {
const name = FAKE_MAESTRO_COMMANDS[salt % FAKE_MAESTRO_COMMANDS.length]!;
return salt % 2 === 0 ? [`- ${name}`] : [`- ${name}: ${yamlText(salt)}`];
},
},
{
name: 'unsupported-field',
code: 'INVALID_ARGS',
lines: (salt) =>
salt % 2 === 0
? ['- tapOn:', ` bogusField: ${yamlText(salt)}`]
: ['- launchApp:', ` appId: ${yamlText(salt)}`, ' bogus: 1'],
},
{
name: 'multi-key-command',
code: 'INVALID_ARGS',
lines: (salt) => [`- tapOn: ${yamlText(salt)}`, ` inputText: ${yamlText(salt + 1)}`],
},
{
name: 'missing-required',
code: 'INVALID_ARGS',
lines: (salt) =>
salt % 2 === 0 ? ['- inputText:'] : ['- extendedWaitUntil:', ' timeout: 500'],
},
{ name: 'bad-press-key', code: 'INVALID_ARGS', lines: () => ['- pressKey: sleep'] },
{ name: 'scroll-options', code: 'INVALID_ARGS', lines: () => ['- scroll:', ' direction: UP'] },
];
/** Declared classes plus the config-level variant `unsupported-field` renders for a salt slice. */
export const MAESTRO_MUTATION_NAMES: readonly string[] = [
...MAESTRO_MUTATIONS.map((mutation) => mutation.name),
'config-unknown-key',
];
type MaestroBase = { commandPicks: number[]; salt: number; withConfig: boolean };
/** `planted` travels as one value: a mutation's lines and where they go are never separable. */
function renderMaestroFlow(base: MaestroBase, planted?: { lines: string[]; at: number }): string {
const commands = base.commandPicks.map((pick, index) =>
validMaestroCommand(pick, base.salt + index),
);
if (planted) commands.splice(Math.min(planted.at, commands.length), 0, planted.lines);
const body = commands.flat().join('\n');
const config = base.withConfig ? `appId: ${yamlText(base.salt)}\n---\n` : '';
return `${config}${body}\n`;
}
const maestroBaseArb: fc.Arbitrary<MaestroBase> = fc.record({
commandPicks: fc.array(fc.nat({ max: 100 }), { minLength: 1, maxLength: 4 }),
salt: fc.nat({ max: 10_000 }),
withConfig: fc.boolean(),
});
/** Encoded Maestro validation cases: ~1/3 valid flows, the rest one planted shape violation. */
export const maestroValidationArb: fc.Arbitrary<string> = fc
.record({ base: maestroBaseArb, mutationIndex: fc.nat(), insertAt: fc.nat({ max: 4 }) })
.map(({ base, mutationIndex, insertAt }) => {
if (mutationIndex % (MAESTRO_MUTATIONS.length + 3) >= MAESTRO_MUTATIONS.length) {
return encodeValidationCase({
payload: renderMaestroFlow(base),
mutation: 'valid',
expect: ACCEPT,
});
}
const mutation = MAESTRO_MUTATIONS[mutationIndex % MAESTRO_MUTATIONS.length]!;
// A config-level violation replaces the flow body mutation for a slice of the space.
if (mutation.name === 'unsupported-field' && base.salt % 3 === 0) {
return encodeValidationCase({
payload: `appId: com.example.app\nbogusKey: 1\n---\n${renderMaestroFlow({ ...base, withConfig: false })}`,
mutation: 'config-unknown-key',
expect: { outcome: 'reject', code: 'INVALID_ARGS' },
});
}
return encodeValidationCase({
payload: renderMaestroFlow(base, { lines: mutation.lines(base.salt), at: insertAt }),
mutation: mutation.name,
expect: { outcome: 'reject', code: mutation.code },
});
});