Skip to content

fix(consumer): preserve compatibility with workshop-rs rule events - #163

Merged
Teakowa merged 3 commits into
mainfrom
agent/workshop-rs-event-compat
Aug 18, 2026
Merged

fix(consumer): preserve compatibility with workshop-rs rule events#163
Teakowa merged 3 commits into
mainfrom
agent/workshop-rs-event-compat

Conversation

@Teakowa

@Teakowa Teakowa commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Consumes the released crates.io workshop-rs =0.1.1 contract and preserves Wright's consumer compatibility for the expanded public Event surface.

  • Cargo.toml uses the exact registry requirement workshop-rs = "=0.1.1".
  • Cargo.lock resolves workshop-rs from crates.io with the recorded checksum; no GitHub source remains for this dependency.
  • Preserves parameterless Event::EachPlayer as the legacy/default representation.
  • Treats explicit EachPlayerWithFilters { All, All } as semantically equivalent in analyzer, OPY reconstruction, and OSTW reconstruction.
  • Keeps filtered/player event forms outside the current consumer reconstruction surfaces as explicit structured unsupported cases.
  • Keeps parser collision expectations for the canonical event-only enum domains.

Validation

Validated against the released workshop-rs v0.1.1 package:

  • cargo info [email protected]: registry package available, Rust 1.85 minimum;
  • cargo fmt --all --check: pass;
  • cargo clippy --locked --workspace --all-targets --all-features -- -D warnings: pass;
  • cargo test --locked --workspace --all-targets --all-features: pass;
  • Rust 1.85 fmt, clippy, and full workspace tests: pass;
  • cargo build --locked --release -p wright-cli -p wright-lsp -p wright-bench: pass;
  • python3 scripts/v1-gates.py --wright target/release/wright: 6/6 passed;
  • python3 scripts/run-scenarios.py --wright target/release/wright: 7/7 passed;
  • OPY and OSTW differential tests: pass;
  • target/release/wright-bench: no regressions, peak RSS 8.72 MB;
  • locked Cargo metadata and lockfile both resolve workshop-rs 0.1.1 from the crates.io registry;
  • PR CI run 32065747320: 12/12 checks passed, including stable/MSRV, LPP, differential, release gates, and all distribution platforms.

The existing event consumer changes remain unchanged; this follow-up only replaces the temporary source pin with the released crates.io contract. Ready for merge; all current CI checks pass.

Fixes #163

@Teakowa
Teakowa merged commit a0b4914 into main Aug 18, 2026
12 checks passed
@Teakowa
Teakowa deleted the agent/workshop-rs-event-compat branch August 18, 2026 03:38
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Make Workshop Great Again Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant