Skip to content

[CI 03] ci: build and test workflow for backend and frontend - #232

Merged
willchen96 merged 1 commit into
Open-Legal-Products:mainfrom
amal66:olp-pr/ci-workflows
Jul 22, 2026
Merged

[CI 03] ci: build and test workflow for backend and frontend#232
willchen96 merged 1 commit into
Open-Legal-Products:mainfrom
amal66:olp-pr/ci-workflows

Conversation

@amal66

@amal66 amal66 commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

A CI workflow (.github/workflows/ci.yml) that builds and tests both packages and runs the eval harness on every PR and push to main. Ported from the reference index (#205, row amal66#46) and extended for this tree. Complements the e2e workflow in #220 — this one is the fast gate (~2-3 min), that one the full-stack gate.

Changes

Three jobs:

  • backendnpm ci, npm test --if-present, npm run build (tsc, so the backend is typechecked on every PR).
  • frontendnpm ci, npm test --if-present, npm run lint (advisorycontinue-on-error, because main currently carries 23 eslint errors; flip it to blocking once that backlog is burned down), and a production next build with placeholder NEXT_PUBLIC_* env (values are only inlined at build time; nothing is contacted — verified locally that the build succeeds with placeholders).
  • evals — runs node evals/run.mjs --threshold 1.0 when evals/ exists, else skips with a message.

Why

The repo has no CI besides the e2e workflow proposed in #220 today: a PR that breaks the backend build, fails tsc, or breaks a unit test reports green. Test steps use --if-present and the evals job checks for evals/run.mjs, so this is safe to merge before or after the other testing PRs in any order — on today's tree it still gates both builds.

Testing

  • Every step verified locally against this branch's tree: backend npm ci && npm run build passes; frontend production build with the exact placeholder env in the workflow passes; actionlint-style read-through of the YAML (the workflow mirrors the fork's, which ran green on the fork's Actions).
  • To make it merge-blocking, add the backend and frontend checks to branch protection alongside e2e / playwright (see docs/e2e-ci.md in [Testing 12] test: Playwright e2e suite + CI merge gate (auth, chat, projects, tabular, workflows) #220).

Based directly on main; no shared commits with the other testing PRs.

🤖 Generated with Claude Code

Ported from #46 onto current main, extended for this tree:
backend job runs npm ci, tests (--if-present, so it is safe to merge in any
order relative to the test-harness PR) and tsc build; frontend job runs
tests, eslint as an advisory step (main currently carries 23 lint errors —
flip to blocking once burned down), and a production next build with
placeholder NEXT_PUBLIC_* env (verified locally that the build succeeds and
contacts nothing); evals job runs node evals/run.mjs when present, else
skips.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


QA Runner seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@amal66 amal66 changed the title ci: build and test workflow for backend and frontend [Test 03] ci: build and test workflow for backend and frontend Jul 21, 2026
@amal66 amal66 changed the title [Test 03] ci: build and test workflow for backend and frontend [Testing 03] [Test 03] ci: build and test workflow for backend and frontend Jul 21, 2026
@amal66 amal66 changed the title [Testing 03] [Test 03] ci: build and test workflow for backend and frontend [Testing 03] ci: build and test workflow for backend and frontend Jul 21, 2026
@amal66 amal66 changed the title [Testing 03] ci: build and test workflow for backend and frontend [CI 03] ci: build and test workflow for backend and frontend Jul 21, 2026
@willchen96
willchen96 merged commit 73b9cd1 into Open-Legal-Products:main Jul 22, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants