Skip to content

[TS PBT] Preserve runtime limitation diagnostics in property search - #560

Open
CaelmBleidd wants to merge 1 commit into
mainfrom
caelmbleidd/pbt-runtime-limits-555
Open

CaelmBleidd wants to merge 1 commit into
mainfrom
caelmbleidd/pbt-runtime-limits-555

Conversation

@CaelmBleidd

Copy link
Copy Markdown
Member

A sparse indexed write can exclude every symbolic path while PBT reports an unexplained engine failure, or hide excluded paths behind a successful no-violation result. Subscribe to the existing runtime-limitation observer and retain the reason, original source range and EtsIR statement in the existing diagnostics field.

Keep at most 64 distinct runtime diagnostics plus an explicit truncation record. A search with excluded paths and no candidate reports UNSUPPORTED; reached candidates, property errors, timeouts and solver UNKNOWN keep their existing classifications and retain the diagnostic. No result schema or selector API changes.

Validation: 35 focused PBT/DSL tests passed, including native TypeScript confirmation of the retained candidate. All three new DSL regressions fail on the original searcher and pass with the fix. PBT Detekt main/test, FastCheck Detekt test and git diff --check pass. Campaign artifact serialization uses the same diagnostics field in #542; the integrated roundtrip remains a separate #542 verification.

Refs #555. Sparse indexed growth itself remains #388.

@Test
fun `supported counterexample and its native replay survive an excluded path`() {
val program = defineProperty(id = "runtime.mixed-counterexample") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported counterexample and its native replay survive an excluded path`() {
val program = defineProperty(id = "runtime.mixed-counterexample") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported counterexample and its native replay survive an excluded path`() {
val program = defineProperty(id = "runtime.mixed-counterexample") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported counterexample and its native replay survive an excluded path`() {
val program = defineProperty(id = "runtime.mixed-counterexample") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported counterexample and its native replay survive an excluded path`() {
val program = defineProperty(id = "runtime.mixed-counterexample") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported successful paths do not hide excluded model paths`() {
val program = defineProperty(id = "runtime.mixed-no-violation") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported successful paths do not hide excluded model paths`() {
val program = defineProperty(id = "runtime.mixed-no-violation") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported successful paths do not hide excluded model paths`() {
val program = defineProperty(id = "runtime.mixed-no-violation") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported successful paths do not hide excluded model paths`() {
val program = defineProperty(id = "runtime.mixed-no-violation") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
@Test
fun `supported successful paths do not hide excluded model paths`() {
val program = defineProperty(id = "runtime.mixed-no-violation") {
val write = function<(Double) -> Double>(module = "runtime-limitations.ts", exportName = "supportedOrSparse")
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.

2 participants