Skip to content

Add integration test workflow for mngr#48

Merged
DanverImbue merged 4 commits intomainfrom
sculptor/add-github-ci-test-gate
Feb 23, 2026
Merged

Add integration test workflow for mngr#48
DanverImbue merged 4 commits intomainfrom
sculptor/add-github-ci-test-gate

Conversation

@DanverImbue
Copy link
Collaborator

Adds a separate CI job that tests offload against a golden version of the imbue-ai/mngr repository. The workflow:

  1. Checks out both offload (current) and mngr (golden SHA)
  2. Installs Rust, just, and uv for dependencies
  3. Installs the current version of offload via just install
  4. Installs mngr dependencies
  5. Runs just test-offload in the mngr repo

Configuration:

  • MNGR_SHA env var at top of file controls the golden version
  • Requires MNGR_ACCESS_TOKEN secret for private repo access

🤖 Generated with Claude Code

Adds a separate CI job that tests offload against a golden version of
the imbue-ai/mngr repository. The workflow:

1. Checks out both offload (current) and mngr (golden SHA)
2. Installs Rust, just, and uv for dependencies
3. Installs the current version of offload via `just install`
4. Installs mngr dependencies
5. Runs `just test-offload` in the mngr repo

Configuration:
- MNGR_SHA env var at top of file controls the golden version
- Requires MNGR_ACCESS_TOKEN secret for private repo access

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Sculptor <sculptor@imbue.com>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Vet found 0 issues.

@danverbraganza danverbraganza force-pushed the sculptor/add-github-ci-test-gate branch from 1ccbd87 to 3584535 Compare February 19, 2026 04:19
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Vet found 0 issues.

@danverbraganza danverbraganza force-pushed the sculptor/add-github-ci-test-gate branch from 3584535 to ec23de0 Compare February 20, 2026 00:04
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Vet found 0 issues.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Vet found 0 issues.

@danverbraganza danverbraganza force-pushed the sculptor/add-github-ci-test-gate branch from d70b677 to b0c4c84 Compare February 20, 2026 00:36
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Vet found 0 issues.

@danverbraganza danverbraganza force-pushed the sculptor/add-github-ci-test-gate branch from b0c4c84 to c4c3461 Compare February 20, 2026 06:44
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Vet found 0 issues.

@github-actions
Copy link

Vet found 1 issue.


scripts/modal_sandbox.py:78

[commit_message_mismatch] (severity 3/5) (confidence 0.85)

The user request was to 'Add integration test workflow for mngr', but the diff also includes significant changes to scripts/modal_sandbox.py (simplifying copy_from_sandbox from tar-based to direct file read) and adds a new Rust integration test (modal_download_junit_xml) in src/provider/default.rs. These changes are not part of the requested CI workflow addition and represent unauthorized scope expansion.

@DanverImbue DanverImbue merged commit 37ff576 into main Feb 23, 2026
5 checks passed
@DanverImbue DanverImbue deleted the sculptor/add-github-ci-test-gate branch February 23, 2026 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant