Skip to content

refactor(move): relocate daemon Maestro replay tests - #3034

Merged
thymikee merged 1 commit into
codex/2663-replay-repair-testsfrom
codex/2663-replay-maestro-tests
Sep 28, 2026
Merged

thymikee merged 1 commit into
codex/2663-replay-repair-testsfrom
codex/2663-replay-maestro-tests

Conversation

@thymikee

@thymikee thymikee commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Move 12 daemon-backed Maestro replay test and fixture files to src/daemon/__tests__/replay-maestro; only relative module specifiers change.
  • Keep these integration tests with the daemon while the production replay adapter moves to @agent-device/replay-port in later stack layers.

Part of #2663. This PR depends on #3032. Rename-aware review diff is 484 additions and 484 deletions, 968 gross lines.

Validation

  • Exact head b61939f4654699969b2459257b4da46c2c0b3536: pnpm check:affected --run passed (613 related files, 5,419 tests; provider integration and wire compatibility passed).
  • All 61 moved tests passed directly; pnpm check:quick, pnpm check:layering, and pnpm check:fallow --base origin/main passed.
  • GitHub Coverage and Integration Tests remain CI-owned. Live .ad and Maestro characterization will run on the completed package migration commit.

@thymikee
thymikee added this pull request to stack #3020 September 28, 2026 12:40
@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
Installed (including dependencies) 4.87 MB 4.87 MB 0 B
Package (unpacked) 4.87 MB 4.87 MB 0 B
Package (download) 1.46 MB 1.46 MB -3 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 25.3 ms 25.6 ms +0.2 ms
CLI --help 72.8 ms 72.0 ms -0.8 ms

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 12 files

Re-trigger cubic

@thymikee

Copy link
Copy Markdown
Member Author

Reviewed at b61939f. This is a clean move: the daemon Maestro replay tests are renamed with only import-path edits, no test is lost, and no production code changes. All checks pass.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Sep 28, 2026
@thymikee
thymikee force-pushed the codex/2663-replay-maestro-tests branch from b61939f to 08ca077 Compare September 28, 2026 16:35
@thymikee
thymikee merged commit 7260fda into main Sep 28, 2026
17 of 18 checks passed
@thymikee
thymikee deleted the codex/2663-replay-maestro-tests branch September 28, 2026 16:51
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