OpenConceptLab/ocl_issues#2838 | Pull requests run the Eslint job, and npm run eslint lints .jsx

[paynejd]

Part of OpenConceptLab/ocl_issues#2838 (oclweb3). The same change as OpenConceptLab/oclmap#84.

Summary

Lint ran only in build.yml, on pushes to main, so a pull request showed no lint result before it merged. A lint failure surfaced only after the merge, where it blocks the build and deploy.

Change

• .github/workflows/pr.yml (new): runs build.yml's "Eslint" job steps, unchanged, on every pull_request:

  • the start-prod.sh secret check
  • Node 22 with the npm cache, and npm ci
  • ./node_modules/eslint/bin/eslint.js --ext .jsx,.js src/

  It has a read-only token, and a newer push to the same PR cancels the older run. build.yml is untouched, deploy jobs included. oclweb3's build has no test job, so there's none to mirror.

• package.json: the eslint script adds --ext .jsx,.js. Without it, ESLint 8 lints only .js files, so npm run eslint skipped every .jsx file.

Test plan

• The workflow's steps are identical to build.yml's "Eslint" job (diffed).
• Locally, after npm ci: CI's lint command is clean on this branch.
• npm run eslint now reports an unused import in a throwaway .jsx file (since removed); before, it skipped .jsx.
• The new "PR checks / Eslint" check ran on this PR and passed, every step.
• Codex adversarial review, posted on this PR: clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Lkm1wDY1zKSDTm2apzPJLi

[paynejd]

Codex adversarial review, pass 1 (codex-cli 0.158, commit 0533942)

Scope: origin/main...HEAD, including build.yml and the new pr.yml. Read-only run. The report below is verbatim.

Outcome: Clean.

---

No real issues found. The lint jobs are structurally identical, both YAML files parse, and build.yml is unchanged. Repository searches found no callers broken by the eslint script change. No files were modified.