ai-workspace-lab/xworkmate-app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ci): drop ripgrep dependency from check-no-app-ffi.sh

Browse Source 
The Flutter verification lane runs on Ubuntu 22.04 without ripgrep
installed, so the FFI integration guard silently fell through and
printed 'No app-side Codex FFI integration artifacts found' on every
run. Replace rg with the POSIX grep -RInE that ships with the runner,
keep the same excludes (check-no-app-ffi.sh, Pods, ephemeral, build,
.dart_tool) and emit the actual offending matches so the gate fails
loudly when a forbidden reference reappears.
This commit is contained in:
Haitao Pan 2026-06-05 19:29:39 +08:00
parent 53875ff37b
commit a0aec84615
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
scripts/check-no-app-ffi.sh
View File

@ -22,16 +22,23 @@ for relative_path in "${forbidden_paths[@]}"; do
fi fi
done done
if rg -n \ forbidden_ffi_pattern='copy_ffi_framework|generate_ffi_bindings|integrate_rust_flutter|flutter_rust_bridge|libcodex_ffi|codex_ffi_generated|ffi-(copy|generate|integrate)|build-macos-ffi'
"copy_ffi_framework|generate_ffi_bindings|integrate_rust_flutter|flutter_rust_bridge|libcodex_ffi|codex_ffi_generated|ffi-(copy|generate|integrate)|build-macos-ffi" \
matches="$(grep -RInE \
--binary-files=without-match \
--exclude='check-no-app-ffi.sh' \
--exclude-dir='Pods' \
--exclude-dir='ephemeral' \
--exclude-dir='build' \
--exclude-dir='.dart_tool' \
"$forbidden_ffi_pattern" \
"$ROOT_DIR/Makefile" \ "$ROOT_DIR/Makefile" \
"$ROOT_DIR/scripts" \ "$ROOT_DIR/scripts" \
"$ROOT_DIR/lib" \ "$ROOT_DIR/lib" \
"$ROOT_DIR/macos/Runner.xcodeproj" \ "$ROOT_DIR/macos/Runner.xcodeproj" || true)"
--glob '!scripts/check-no-app-ffi.sh' \
--glob '!**/Pods/**' \ if [[ -n "$matches" ]]; then
--glob '!**/Flutter/ephemeral/**' \ echo "$matches" >&2
--glob '!**/build/**'; then
echo "Forbidden app-side FFI integration reference found." >&2 echo "Forbidden app-side FFI integration reference found." >&2
failures=$((failures + 1)) failures=$((failures + 1))
fi fi