Skip to content

Add baseline test for PE056 error in AP242 schema parsing#10

Merged
starseeker merged 2 commits intotreatfrom
copilot/add-schema-scanner-test
Feb 6, 2026
Merged

Add baseline test for PE056 error in AP242 schema parsing#10
starseeker merged 2 commits intotreatfrom
copilot/add-schema-scanner-test

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 6, 2026 •
edited
Loading

PE056 (QUERY_REQUIRES_AGGREGATE) triggers at line 34148 in 242_mim_lf.exp during schema parsing. This adds instrumentation to lock the regression for later fixes.

Changes

  • Test: test_ap242_pe056_baseline in src/express/test/CMakeLists.txt

    • Runs check-express parser on data/ap242/242_mim_lf.exp
    • Set to WILL_FAIL true (expects PE056 until fixed)
    • Execution time: ~170ms (focused parse, no schema build)
    • Labels: parser, ap242, baseline

  • Documentation: README_AP242_BASELINE.md

    • Error location and context (styled_item entity WR3)
    • Quick local run: ./bin/check-express ../data/ap242/242_mim_lf.exp

Error Details

data/ap242/242_mim_lf.exp:34148: --ERROR PE056: Query expression source must be an aggregate.

The QUERY expression QUERY(it <* item | ...) fails because item is not recognized as an aggregate type. Once resolved, flip WILL_FAIL to false.

Original prompt

Task 0 — Baseline instrumentation (optional but recommended)

Purpose: Ensure we can pinpoint exactly where PE056 is triggered in AP242 and lock a regression test around it.

Agent input:

Add a CMake/CTest entry (or whatever test harness exists) that runs the schema scanner (or libexpress parser) against data/ap242/242_mim_lf.exp and captures failure location + error code.
The test should:
    fail on current treat branch state
    pass after later tasks
Prefer a focused parse/resolve test rather than a full build if possible (faster CI).

Acceptance criteria:

A new test exists that reproduces the failure deterministically (PE056 at styled_item or PE052 for treat, whichever is current).
Running it locally is quick and documented.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@starseeker
Add baseline instrumentation test for PE056 in AP242 schema
af218d5 
Co-authored-by: starseeker <238416+starseeker@users.noreply.github.com>
Copilot AI changed the title [WIP] Add CMake entry for schema scanner test in AP242 Add baseline test for PE056 error in AP242 schema parsing Feb 6, 2026
Copilot AI requested a review from starseeker February 6, 2026 22:18
@starseeker starseeker marked this pull request as ready for review February 6, 2026 22:19
@starseeker starseeker merged commit 159cd3c into treat Feb 6, 2026
@starseeker starseeker deleted the copilot/add-schema-scanner-test branch February 6, 2026 22:19
starseeker added a commit that referenced this pull request Feb 9, 2026
@starseeker
Add baseline test for PE056 error in AP242 schema parsing (#10)
93369ce 
* Initial plan

* Add baseline instrumentation test for PE056 in AP242 schema

Co-authored-by: starseeker <238416+starseeker@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: starseeker <238416+starseeker@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@starseeker starseeker starseeker approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@starseeker starseeker

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@starseeker