chore: add ui test runner, run tests in CI, fix stale agent docs - #157
Merged
Merged
Conversation
- Add Vitest, jsdom and Testing Library to @eqtylab/equality, with a root `pnpm test` that runs every package's tests. The ui script passes with no tests until the first suite lands. - Run `pnpm test` in the Code Quality workflow, so the docs package's existing tests run in CI for the first time. - AGENTS.md: correct the docs path and the llms.txt route, finish the truncated tokens sentence, and fix the build/dev instructions. - create-component skill: correct the docs path. - equality-design-system skill: add the name translation table that AGENTS.md refers to. - Demo tsconfig: drop `baseUrl`, deprecated in TypeScript 6. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
matthewmr-eqty
approved these changes
Sep 24, 2026
matthewmr-eqty
left a comment
Contributor
There was a problem hiding this comment.
Thanks Tucker. Nice to have the tests running in CI now.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tooling and docs housekeeping, split out of
feat/select-searchso that PR can stay focused on the feature. Nothing here changes the published library, so there's no changelog entry or version bump.Tests
@eqtylab/equality, with config inpackages/ui/vitest.config.tsand tests underpackages/ui/tests/. Tests live outsidesrc, so they're never bundled or published (filesisdistonly).pnpm test(pnpm -r run test) and a Test step in the Code Quality workflow. The docs package's 103 existing tests have never run in CI; now they do.vitest run --passWithNoTestsfor now, because the first suite lands withfeat/select-search. That PR drops the flag.Agent docs
packages/demo/src/content/docs/components(it was missingdocs/);llms.txtand.mdroutes now point atpackages/docs/src/runtime/routes(the old path no longer exists);pnpm buildandpnpm devare described correctly.Dialog, drawer →Sheet, chip →Badgeand so on), each checked against an actual export. Also fixes a stray backtick onDateRangePicker.Config
baseUrlfrom the demo tsconfig (deprecated in TypeScript 6) and makes thepathsentries relative. The demo type errors are the same before and after (17 existing ones inastro.config.mjs), and the Astro build resolves every alias.Verification
pnpm lint,pnpm format:check,pnpm buildandpnpm testall pass locally on Node 24. The docs tests use Node's built-in TypeScript support, which needs Node 22.18 or later; CI'snode-version: 22should resolve to a new enough release, but it's worth confirming on this PR's run.🤖 Generated with Claude Code