Skip to content

feat(ci): reconnect PPL lint validation to the candidate runtime grammar #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Hanyu-W wants to merge 10 commits into
base: main
Choose a base branch
from
Open

feat(ci): reconnect PPL lint validation to the candidate runtime grammar #2

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
9a35e76
feat(ci): add PPL lint rule validation check (eventstats PoC)
Jul 19, 2026
738672e
style(ci): use $(...) instead of legacy backticks in ppl-lint workflow
Jul 20, 2026
74fcfaa
ci: re-trigger PPL lint rule validation after Actions recovery
Jul 20, 2026
9aca44c
fix(ci): split ppl-lint validation into frontend/backend jobs
Jul 20, 2026
1745ed9
feat(ci): extend PPL lint contract to all reachable rules (schema v2)
Jul 20, 2026
5b03f4f
feat(ci): reconnect PPL lint validation to candidate runtime grammar
Jul 20, 2026
9b799e7
fix(ci): pipefail so detector runner failure is not masked by tee
Jul 20, 2026
eef3110
feat(ci): allow workflow_dispatch to validate against an OSD fork
Jul 21, 2026
e13ff80
fix(ci): resolve OSD target via repo variables so PR check can valida…
Jul 22, 2026
b1fc00c
ci(ppl-lint): document sibling-workflow reuse; harden detector job
Jul 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
374 changes: 374 additions & 0 deletions .github/workflows/ppl-lint-rule-validation.yml
View file
Open in desktop

Large diffs are not rendered by default.

10 changes: 9 additions & 1 deletion .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,4 +59,12 @@ http-client.env.json
!.claude/harness/
.claude/settings.local.json
.clinerules
memory-bank
memory-bank
# PPL lint rule validation contract run artifacts (uploaded in CI, not committed)
backend-report.json
backend-report-nightly.json
detector-report.json
detector-contract.log
ppl-grammar-bundle.json
target.json
run-manifest.json
15 changes: 15 additions & 0 deletions integ-test/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,21 @@ tasks.withType(licenseHeaders.class) {
additionalLicense 'AL ', 'Apache', 'Licensed under the Apache License, Version 2.0 (the "License")'
}

// Forward the PPL lint rule validation contract knobs to every integ test JVM
// (PplLintRuleValidationIT reads them): which schedule to run (pr|nightly), an
// optional path to write the observed backend report, and — while the cluster is
// alive — optional paths to export the candidate runtime grammar bundle and its
// target manifest for the detector-validation job. Applied globally so every
// RestIntegTestTask that runs the class picks it up without per-task edits.
tasks.withType(Test).configureEach {
systemProperty "ppl.lint.schedule", System.getProperty("ppl.lint.schedule", "pr")
["ppl.lint.report", "ppl.lint.grammar.bundle", "ppl.lint.target"].each { prop ->
if (System.getProperty(prop) != null) {
systemProperty prop, System.getProperty(prop)
}
}
}

validateNebulaPom.enabled = false
loggerUsageCheck.enabled = false

Expand Down
Loading
Loading