Skip to content

test(compatibility): link opcode coverage to behavior evidence - #708

Merged
Mathih13 merged 1 commit into
mainfrom
feat/vanilla-compatibility-first-pass
Oct 15, 2026
Merged

Mathih13 merged 1 commit into
mainfrom
feat/vanilla-compatibility-first-pass

Conversation

@Mathih13

@Mathih13 Mathih13 commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Opcode dispatch coverage does not establish that gameplay works. The compatibility inventory now links message rows to nine curated behavior entries, each with prerequisites, test locations, evidence level and remaining work.

The scanner checks named Rust test declarations and records whether they are ignored. It explicitly reports that it did not execute them. Real-client status remains unverified.

What you verified

All nine applicable CI checks passed at 41197289, including the full durable suite. Retargeting to main preserved the tested base tree.

Local checks ran on the combined Core tree at 5003aca5, based on 9be23cd1. The reordered stack at 8d411c26 has the identical tree. The current stack at 41197289 adds only the three test-fixture repairs in #709; all other files are identical. The six affected durable tests passed separately on #709.

  • Gateway suite: 2,131 passed, 32 ignored, including all nine inventory example tests.
  • cargo run --locked --quiet -p lyracore-gateway --example benilla_inventory produced the JSON inventory with nine behavior entries.
  • Workspace formatting and strict Clippy passed. Tests cover missing or ambiguous test names, ignored tests and opcode direction.
  • Independent adversarial review found no defect.

Comment review passed for the inventory source and tests. Comments in scanner test strings remain fixture data.

What you did not verify

This is a curated starting set, not a complete gameplay audit. Test discovery does not prove a passing test or real-client behavior.

Merged Core changes: fixture timing → gossip Gates → Hunter pet rename → behavior evidence inventory. The Transfer correction #705 and the Deadmines tests have merged.

This PR targets main and contains its own change.

Model: GPT-6 Astra, using Codex through T3 Code.

@Mathih13
Mathih13 force-pushed the feat/vanilla-compatibility-first-pass branch from 8d411c2 to 4119728 Compare October 15, 2026 13:27
@Mathih13
Mathih13 force-pushed the feat/hunter-pet-rename branch from dd4b462 to 980ad20 Compare October 15, 2026 13:27
@Mathih13
Mathih13 changed the base branch from feat/hunter-pet-rename to main October 15, 2026 13:51
@Mathih13
Mathih13 merged commit 44b0487 into main Oct 15, 2026
9 of 18 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