From 738cc93cef00fc37ce9a7070f567302ecc767e39 Mon Sep 17 00:00:00 2001 From: hyperpolymath <6759885+hyperpolymath@users.noreply.github.com> Date: Tue, 2 Jun 2026 12:20:07 +0100 Subject: [PATCH] ci(language-policy): delegate Java/Kotlin check to standards reusable MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Deletes the local `Check for Java/Kotlin files` step in `.github/workflows/language-policy.yml`. The estate-wide `hyperpolymath/standards` `governance-reusable.yml language-policy` job (which neurophone already calls via `governance.yml`) covers the same ban, AND honours the new `android/**/src/**/*.java` carve-out for platform-required JVM shims (standards#341). Keeping a local copy: - duplicates maintenance burden, - creates drift risk (local check could lag the estate carve-out), - gives a false sense of independent gating (both gates share the same ban list semantics). Single source of truth: the standards reusable. Path-2B per the RFC at `docs/migrations/RFC-ANDROID-KOTLIN-TO-RUST.adoc` (sub-PR #2). Companion to standards#341. This PR can merge AFTER standards#341 merges. If it lands first, the estate-wide check still blocks Kotlin files (which is correct — the neurophone migration still has Kotlin to delete in sub-PRs #5/#6/#7/#9). Co-Authored-By: Claude Opus 4.7 (1M context) --- .github/workflows/language-policy.yml | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/.github/workflows/language-policy.yml b/.github/workflows/language-policy.yml index 2332211..fadada5 100644 --- a/.github/workflows/language-policy.yml +++ b/.github/workflows/language-policy.yml @@ -84,14 +84,11 @@ jobs: fi echo "✓ No npm runtime dependencies found" - - name: Check for Java/Kotlin files - run: | - if find . -name "*.java" -o -name "*.kt" -o -name "*.kts" | head -1 | grep -q .; then - echo "::error::Java/Kotlin files found. Use Rust/Tauri/Dioxus instead." - find . -name "*.java" -o -name "*.kt" -o -name "*.kts" - exit 1 - fi - echo "✓ No Java/Kotlin files found" + # Java/Kotlin check is delegated to hyperpolymath/standards + # governance-reusable.yml `language-policy` job (which honours the + # estate-wide android/**/src/**/*.java carve-out for platform-required + # JVM shims — see standards#341). Keeping a local copy here drifts + # the policy. Single source of truth. - name: Check for Swift files run: |