Skip to content

feat(ci): validate PPL lint across grammar surfaces - #7

Closed
Hanyu-W wants to merge 2 commits into
codex/ppl-lint-13-rule-ci-implementationfrom
codex/ppl-lint-multi-surface-compatibility-clean
Closed

feat(ci): validate PPL lint across grammar surfaces#7
Hanyu-W wants to merge 2 commits into
codex/ppl-lint-13-rule-ci-implementationfrom
codex/ppl-lint-multi-surface-compatibility-clean

Conversation

@Hanyu-W

@Hanyu-W Hanyu-W commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Description

Clean replacement for #6. Builds on #5 and implements the multi-surface compatibility CI design.

  • Plans exactly three configurations: OpenSearch 2.19.6 with OSD's compiled-simplified grammar, the highest official GA release at or below the normalized PR target with its runtime bundle, and the SQL PR build with its runtime bundle.
  • Replaces the fixed-version/discovery workflow with the six-job observation and fan-in shape.
  • Retires only the legacy Analytics Engine Compatibility workflow; SQL Java CI and Security Plugin IT are unchanged from the base.
  • Emits a complete schema-v3 12 x 3 compatibility report, preserving missing legs as inconclusive columns and publishing artifacts before enforcement.
  • Applies surface, version, then engine-mode exclusions before detector execution.
  • Distinguishes detector regression, full engine relaxation, partial engine relaxation, and contract drift with query-level evidence and remediation.

Testing

  • node --test scripts/ppl-lint/__tests__/*.test.mjs (218 passing)
  • actionlint .github/workflows/ppl-lint-multiversion-validation.yml .github/workflows/ppl-lint-rule-validation.yml
  • node --check scripts/ppl-lint/aggregate-compatibility.mjs
  • node --check scripts/ppl-lint/plan-compatibility.mjs
  • node --check scripts/ppl-lint/run-frontend-contract.mjs
  • git diff --check

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • Commits are signed per the DCO using --signoff or -s.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hanyu Wei added 2 commits August 4, 2026 20:00
Signed-off-by: Hanyu Wei <weihanyu@amazon.com>
Signed-off-by: Hanyu Wei <weihanyu@amazon.com>
@Hanyu-W

Hanyu-W commented Aug 5, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #8, which restores the upstream-owned Analytics Engine Compatibility workflow, leaves every inherited workflow unchanged, includes the latest multi-surface lint CI, and labels the added checks as Linter workflows.

@Hanyu-W Hanyu-W closed this Aug 5, 2026
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