Skip to content

fix: make harness example self-contained#21

Merged
ForeverAngry merged 1 commit into
mainfrom
fix/self-contained-harness-example
May 27, 2026
Merged

fix: make harness example self-contained#21
ForeverAngry merged 1 commit into
mainfrom
fix/self-contained-harness-example

Conversation

@ForeverAngry

Copy link
Copy Markdown
Owner

Summary

  • vendor the example-only harness record helper into rig-mcp/examples/common
  • repoint the harness example at the local helper so isolated CI checkouts do not need a sibling rig-compose repo

Validation

  • cargo fmt --all -- --check
  • just check

Copilot AI review requested due to automatic review settings May 27, 2026 13:40

Copilot AI 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.

Pull request overview

This PR makes the harness_record example self-contained by vendoring the shared “harness record” helper into this repository’s examples/ tree, removing the need for a sibling rig-compose checkout in isolated CI environments.

Changes:

  • Repoint examples/harness_record.rs to use a local examples/common/harness_record.rs module via #[path = ...].
  • Add a new examples/common/harness_record.rs with serde-serializable mirror types and a helper constructor for converting from rig-compose invocation/result types.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
examples/harness_record.rs Updates module path to the vendored example helper to avoid external relative paths.
examples/common/harness_record.rs Adds the shared example-only harness record schema/types and conversion helper.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ForeverAngry ForeverAngry merged commit 274894b into main May 27, 2026
9 checks passed
@ForeverAngry ForeverAngry deleted the fix/self-contained-harness-example branch May 27, 2026 13:43
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.

3 participants