Skip to content

docs: retire Maestro compatibility tracker#1350

Merged
thymikee merged 1 commit into
mainfrom
agent/retire-maestro-compat-tracker
Jul 20, 2026
Merged

docs: retire Maestro compatibility tracker#1350
thymikee merged 1 commit into
mainfrom
agent/retire-maestro-compat-tracker

Conversation

@thymikee

@thymikee thymikee commented Jul 20, 2026

Copy link
Copy Markdown
Member

Summary

Retire the stale #558 Maestro compatibility umbrella while keeping a concise versioned capability and limitations reference in agent-device help maestro and the website docs.
Treat conformance divergences as a mechanical parity record, with focused issue links only when implementation work is planned, and preserve #1342's intentional we-are-lenient cases.

Validation

  • pnpm format:check
  • pnpm typecheck
  • pnpm lint
  • pnpm exec vitest run src/compat/maestro/__tests__/support-matrix.test.ts src/cli/parser/__tests__/cli-help-command-usage.test.ts src/cli/parser/__tests__/cli-help-topics.test.ts (61 tests)
  • pnpm maestro:conformance (46 tests)
  • Broad related run: 863/868 passed in parallel; all five fixed-timeout failures passed when their four files were rerun with one worker (23 tests)

@thymikee

Copy link
Copy Markdown
Member Author

Two blockers before this can be ready:

  1. The branch conflicts with merged feat(maestro): numeric option fields accept ${VAR} lookup interpolation (#1293) #1342 in scripts/maestro-conformance/expected-divergence.ts. Rebase while retaining feat(maestro): numeric option fields accept ${VAR} lookup interpolation (#1293) #1342’s we-are-lenient model and removing the obsolete upstream/042_extended_wait divergence; otherwise the tracker cleanup would restore a divergence feat(maestro): numeric option fields accept ${VAR} lookup interpolation (#1293) #1342 made identical. As part of that reconciliation, remove the completed Maestro compat: numeric option fields accept ${VAR} lookup interpolation (extendedWaitUntil.timeout, tapOn.index, siblings) #1293 links from the four new leniency records if tracking should exist only for planned work.
  2. The PR deletes the only versioned user-facing supported-subset and operational-boundary guidance, replacing it with “focused subset” plus ADR 0015. That ADR does not enumerate supported commands/grammar/platform constraints, env precedence, or the runScript trust/network caveat. This conflicts with the repository rule that versioned CLI help is the agent-facing source of truth, and the weakened tests no longer protect those claims. Preserve a concise versioned capability/limitations reference (or an equivalent generated user-facing source) and test substantive coverage.

The current head has no full post-rebase CI evidence.

@thymikee
thymikee force-pushed the agent/retire-maestro-compat-tracker branch from 9f62347 to 6e30969 Compare July 20, 2026 18:14
@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-20 19:09 UTC

@github-actions

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.8 MB 1.8 MB +1.1 kB
JS gzip 574.2 kB 574.5 kB +382 B
npm tarball 688.6 kB 689.0 kB +424 B
npm unpacked 2.4 MB 2.4 MB +1.1 kB

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 29.4 ms 28.1 ms -1.3 ms
CLI --help 62.6 ms 62.1 ms -0.5 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/cli-help.js +2.1 kB +906 B
dist/src/registry.js -1.0 kB -524 B

@thymikee
thymikee marked this pull request as ready for review July 20, 2026 18:24
@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Jul 20, 2026
@thymikee

Copy link
Copy Markdown
Member Author

Re-reviewed 6e30969d: both prior blockers are resolved. The branch now cleanly preserves #1342’s we-are-lenient model, keeps the obsolete extended-wait divergence removed, and drops completed #1293 tracking links. Substantive versioned guidance is restored through agent-device help maestro, shared capability/limitation data, website parity, and tests covering commands, platform/launch boundaries, environment precedence, and the runScript trust/network caveat. CI is fully green; no remaining findings.

@thymikee
thymikee merged commit d9f16de into main Jul 20, 2026
24 checks passed
@thymikee
thymikee deleted the agent/retire-maestro-compat-tracker branch July 20, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant