Skip to content

fix: pin live scenario dispatch command#43

Merged
galligan merged 1 commit into
mainfrom
fix/live-scenario-project-dispatch-bin
Jun 11, 2026
Merged

fix: pin live scenario dispatch command#43
galligan merged 1 commit into
mainfrom
fix/live-scenario-project-dispatch-bin

Conversation

@galligan

@galligan galligan commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Make the live scenario runner default to uv run --project <repo> dispatch so it always exercises the checkout that owns the scenario file, even after the runner changes into an isolated temp work directory.
  • Print the resolved dispatch command in dry-run output.
  • Add a regression assertion for the resolved in-tree command.

Testing

  • uv run pytest tests/test_scenarios.py -q
  • python scripts/run_scenario.py --dry-run tests/scenarios/basic_coordination.toml
  • python /Users/mg/Developer/outfitter/dispatch/scripts/run_scenario.py /Users/mg/Developer/outfitter/dispatch/tests/scenarios/basic_coordination.toml from /tmp
  • just scenario -- tests/scenarios/basic_coordination.toml
  • just check

Notes

This was found while dogfooding the new live scenario harness. The old default passed under just because it inherited the repo uv context, but it could resolve a different installed dispatch when run from outside that context.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@galligan galligan marked this pull request as ready for review June 11, 2026 18:27

galligan commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Jun 11, 9:42 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 11, 9:42 PM UTC: @galligan merged this pull request with Graphite.

@galligan galligan merged commit 042addc into main Jun 11, 2026
5 checks passed
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