Fix workflow syntax validation to handle Python files correctly

[Copilot]

The PR checks workflow was using node -c to validate all file types including Python, which fails silently due to || true. Additionally, the user requested this fix be applied via a new PR, but I clarified that I can only modify the current PR.

Changes needed:

• Separate JS/TS validation (using node -c) from Python validation (using python -m py_compile)
• Remove || true that masks all validation errors
• Add conditional checks to only run validation when files of each type exist

Current issue in .github/workflows/pr-checks.yml:

find . -name "*.js" -o -name "*.py" -o -name "*.ts" | xargs -I {} node -c {} || true

Proposed fix:

js_ts_files=$(find . \( -name "*.js" -o -name "*.ts" \))
if [ -n "$js_ts_files" ]; then
  echo "$js_ts_files" | xargs -r -I {} node -c {}
fi

py_files=$(find . -name "*.py")
if [ -n "$py_files" ]; then
  echo "$py_files" | xargs -r -I {} python -m py_compile {}
fi

Note: I cannot create new pull requests. Awaiting confirmation to apply this fix to the current PR or provide guidance for manual PR creation.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Copilot wasn't able to review any files in this pull request.