fix(lint): clear lib/config baseline and make CI lint blocking #63

Merged
varutasu merged 1 commit from chore/fix-lint-lib-and-blocking-ci into main 2026-06-02 02:03:42 -04:00
varutasu commented 2026-06-02 01:52:10 -04:00 (Migrated from github.com)

Summary

  • Fix remaining ESLint issues in lib/theme-context.js, lib/use-auth.js, postcss.config.js, and tailwind.config.js
  • Remove the non-blocking lint wrapper from .github/workflows/ci.yml so ESLint failures block merge

Merge order

Merge #61 and #62 first, then this PR. Once all three land, npm run lint exits 0 repo-wide.

Test plan

  • npx eslint lib/ postcss.config.js tailwind.config.js → exit 0
  • npm run test:run → 53/53 pass (on combined tree with #61+#62)
  • After #61+#62 merge: npm run lint → exit 0 on main + this branch

Made with Cursor

## Summary - Fix remaining ESLint issues in `lib/theme-context.js`, `lib/use-auth.js`, `postcss.config.js`, and `tailwind.config.js` - Remove the non-blocking lint wrapper from `.github/workflows/ci.yml` so ESLint failures block merge ## Merge order **Merge #61 and #62 first**, then this PR. Once all three land, `npm run lint` exits 0 repo-wide. ## Test plan - [x] `npx eslint lib/ postcss.config.js tailwind.config.js` → exit 0 - [x] `npm run test:run` → 53/53 pass (on combined tree with #61+#62) - [ ] After #61+#62 merge: `npm run lint` → exit 0 on main + this branch Made with [Cursor](https://cursor.com)
vercel[bot] commented 2026-06-02 01:52:12 -04:00 (Migrated from github.com)

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tcg-vault Ready Ready Preview, Comment Jun 2, 2026 5:52am

Request Review

[vc]: #sYntC+iKUG07kx/+6BfYPPjo/ubFB7EHj6smfJduSBg=:eyJpc01vbm9yZXBvIjp0cnVlLCJ0eXBlIjoiZ2l0aHViIiwicHJvamVjdHMiOlt7Im5hbWUiOiJ0Y2ctdmF1bHQiLCJwcm9qZWN0SWQiOiJwcmpfRjZXOEVvRkd3Y0g3aWVGcnRvRlNlOXdVVkFhNSIsImluc3BlY3RvclVybCI6Imh0dHBzOi8vdmVyY2VsLmNvbS9yYW5kYWxsLXN0aWxsd2VsbHMtcHJvamVjdHMvdGNnLXZhdWx0L0dqZ0FxOTV2Q1A0WE5LRXRXWHhyaVA0ZmdxczIiLCJwcmV2aWV3VXJsIjoidGNnLXZhdWx0LWdpdC1jaG9yZS1maXgtbGludC04MzczNDItcmFuZGFsbC1zdGlsbHdlbGxzLXByb2plY3RzLnZlcmNlbC5hcHAiLCJuZXh0Q29tbWl0U3RhdHVzIjoiREVQTE9ZRUQiLCJsaXZlRmVlZGJhY2siOnsicmVzb2x2ZWQiOjAsInVucmVzb2x2ZWQiOjAsInRvdGFsIjowLCJsaW5rIjoidGNnLXZhdWx0LWdpdC1jaG9yZS1maXgtbGludC04MzczNDItcmFuZGFsbC1zdGlsbHdlbGxzLXByb2plY3RzLnZlcmNlbC5hcHAifSwicm9vdERpcmVjdG9yeSI6bnVsbH1dLCJyZXF1ZXN0UmV2aWV3VXJsIjoiaHR0cHM6Ly92ZXJjZWwuY29tL3ZlcmNlbC1hZ2VudC9yZXF1ZXN0LXJldmlldz9vd25lcj12YXJ1dGFzdSZyZXBvPXRjZy12YXVsdCZwcj02MyJ9 The latest updates on your projects. Learn more about [Vercel for GitHub](https://vercel.link/github-learn-more). | Project | Deployment | Actions | Updated (UTC) | | :--- | :----- | :------ | :------ | | [tcg-vault](https://vercel.com/randall-stillwells-projects/tcg-vault) | ![Ready](https://vercel.com/static/status/ready.svg) [Ready](https://vercel.com/randall-stillwells-projects/tcg-vault/GjgAq95vCP4XNKEtWXxriP4fgqs2) | [Preview](https://tcg-vault-git-chore-fix-lint-837342-randall-stillwells-projects.vercel.app), [Comment](https://vercel.live/open-feedback/tcg-vault-git-chore-fix-lint-837342-randall-stillwells-projects.vercel.app?via=pr-comment-feedback-link) | Jun 2, 2026 5:52am | <a href="https://vercel.com/vercel-agent/request-review?owner=varutasu&repo=tcg-vault&pr=63" rel="noreferrer"><picture><source media="(prefers-color-scheme: dark)" srcset="https://agents-vade-review.vercel.sh/request-review-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://agents-vade-review.vercel.sh/request-review-light.svg"><img src="https://agents-vade-review.vercel.sh/request-review-light.svg" alt="Request Review"></picture></a>
github-actions[bot] commented 2026-06-02 01:52:20 -04:00 (Migrated from github.com)

Pipeline Health

Build + CI gates

Gate Status
Vercel build (Preview) pass
CI: Lint failure
CI: Schema map fresh skipped
Preview smoke pass
Visual diff pass

Build runs on Vercel; this CI runs lint and schema-map drift only (no duplicate build).

Role reports

Role Status
Reviewer report pending
A11y audit pending
Design system audit pending

See individual comments above for details. This rollup updates automatically.

<!-- pipeline-rollup --> ## Pipeline Health ### Build + CI gates | Gate | Status | | --- | --- | | Vercel build (Preview) | ✅ pass | | CI: Lint | ❌ failure | | CI: Schema map fresh | ❌ skipped | | Preview smoke | ✅ pass | | Visual diff | ✅ pass | _Build runs on Vercel; this CI runs lint and schema-map drift only (no duplicate build)._ ### Role reports | Role | Status | | --- | --- | | Reviewer report | ⏳ pending | | A11y audit | ⏳ pending | | Design system audit | ⏳ pending | See individual comments above for details. This rollup updates automatically.
github-actions[bot] commented 2026-06-02 01:53:48 -04:00 (Migrated from github.com)

Visual Diff

Screenshots and diffs uploaded as artifacts: view run

If intentional changes: update snapshots locally with npx playwright test --project=visual --update-snapshots and commit.

## Visual Diff Screenshots and diffs uploaded as artifacts: [view run](https://github.com/varutasu/tcg-vault/actions/runs/26801239339) If intentional changes: update snapshots locally with `npx playwright test --project=visual --update-snapshots` and commit.
Sign in to join this conversation.
No description provided.