Skip to content

Add roger-roger to e2e:canary tests#702

Merged
nodo merged 6 commits intomainfrom
nodo/external-agents-e2e-canary
Mar 16, 2026
Merged

Add roger-roger to e2e:canary tests#702
nodo merged 6 commits intomainfrom
nodo/external-agents-e2e-canary

Conversation

@nodo
Copy link
Copy Markdown
Contributor

@nodo nodo commented Mar 16, 2026

Use mise to install roger-roger and entire-agent-roger-roger, this simplify a lot the setup and allow tests to run without changing the PATH manually.

Following up from #700

Use `mise` to install roger-roger and entire-agent-roger-roger, this
simplify a lot the setup and allow tests to run without changing the
PATH manually.

Entire-Checkpoint: b1019a97fb96
@nodo nodo requested a review from a team as a code owner March 16, 2026 14:11
Copilot AI review requested due to automatic review settings March 16, 2026 14:11
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR shifts installation of the roger-roger E2E agent binaries from ad-hoc go install steps (scripts + CI workflows) to being managed centrally via mise tooling, and updates the canary E2E task to exercise both built-in and external deterministic agents.

Changes:

  • Add roger-roger and entire-agent-roger-roger as mise-managed Go tools.
  • Remove inline go install of roger-roger binaries from E2E task scripts and GitHub workflows.
  • Expand the E2E canary task to run once with vogon and once with roger-roger, producing separate artifacts/reports.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
mise.toml Adds roger-roger Go tool installs under [tools] so mise provisions the binaries.
mise-tasks/test/e2e/roger-roger Removes manual GitHub install of roger-roger binaries; now assumes mise provides them.
mise-tasks/test/e2e/canary Runs canary E2Es twice (vogon + roger-roger) with separate gotestsum event files and reports.
.github/workflows/e2e.yml Stops installing roger-roger via go install, relying on mise instead.
.github/workflows/e2e-isolated.yml Same as above for isolated E2E workflow.

nodo and others added 5 commits March 16, 2026 15:20
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Entire-Checkpoint: a2ab33567fa0
@nodo nodo merged commit fd546b8 into main Mar 16, 2026
3 checks passed
@nodo nodo deleted the nodo/external-agents-e2e-canary branch March 16, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants