Guard unsupported reaction slice admission - #183
Conversation
Extends SemanticSliceKind enum to support Automation and Translation slice types that Stage #79 needs to render reactor-based slices. Changes: - SemanticModel.cs: Added Automation (2) and Translation (3) enum values - SemanticModelBinder.cs: Added switch cases to map SliceType.Automation and SliceType.Translate to the new semantic kinds This is the Screenplay side of a cross-repository change. Stage will implement the rendering side using the existing ReactionSliceRenderer. Verified: 2,302 specs pass, clean Debug build
|
Parent source review found an incomplete contract change. This PR is now draft, not merge-ready.
Regression specs are being run against the newly admitted bare slice kinds before correcting the admission boundary. The earlier passing existing suite had no coverage of this new behavior. No dependency or release action is being taken. Independent Sol review attempts exhausted provider quota without a verdict; these are parent source findings, not a completed independent approval. |
The enum-only addition admitted slice kinds the canonical semantic serializer cannot represent. Preserve the existing unsupported-syntax diagnostic until portable occurrence and effect semantics exist, rather than failing later as invalid binding. Retain eight regression facts for bare Automation and Translate inputs. Strict Debug/Release rebuilds, 2,288 language specs and 22 canonical checks pass on net10.0. This corrects the earlier branch commit without rewriting history.
|
Corrective commit Red/green evidence:
The PR remains draft because independent review has not returned a verdict. This does not implement Automation/Translation semantics or resolve Stage #79. Future work needs actual portable occurrence/effect contracts and a semantic renderer; simply routing to the legacy syntax renderer is not sufficient. |
Added