Skip to content

fix: restore test suite, CI and repo hygiene lost in 78edd57 - #4

Merged
FranGuh merged 2 commits into
mainfrom
fix/restore-testing-ci
Jun 11, 2026
Merged

fix: restore test suite, CI and repo hygiene lost in 78edd57#4
FranGuh merged 2 commits into
mainfrom
fix/restore-testing-ci

Conversation

@FranGuh

Copy link
Copy Markdown
Owner

Summary

  • Restores the entire testing/CI layer accidentally deleted by commit 78edd57 ("feat: UX/UI polish"): 72 tests, vitest config, GitHub Actions workflow, husky pre-commit hook, Phase 2 planning doc and conductor track docs
  • Fixes repo hygiene: UTF-16-corrupted .gitignore entry (which caused coverage/ to be tracked), dual-lockfile conflict (npm + pnpm)
  • Consolidates tooling on pnpm end to end (CI, pre-commit, docs, README)

Changes

AreaChange
src/__tests__/4 test files + setup restored; minimal drift fixes so tests match current UI (no source code touched)
vitest.config.jsRestored (jsdom env, setup file, globals)
.github/workflows/ci.ymlRestored and converted npm → pnpm (pnpm/action-setup@v4, frozen lockfile)
.husky/pre-commitRestored; now pnpm exec lint-staged
package.jsonRe-added prepare script, husky + lint-staged devDeps; added packageManager; dropped deprecated git add from lint-staged
.gitignoreRewritten as clean UTF-8 (was UTF-16-corrupted); coverage/ and package-lock.json now ignored
coverage/Untracked and removed (stale report from deleted tests)
package-lock.jsonDeleted — pnpm is the single package manager
README.mdCI badge added; install instructions npm → pnpm
docs/, conductor/Phase 2 plan + track docs restored; commands updated to pnpm

Test Plan

  • pnpm run test — 71 passed, 1 todo, 0 failed
  • pnpm run build — success
  • pnpm run lint — clean
  • Pre-commit hook verified live (ran on this PR's commit)

🤖 Generated with Claude Code

Commit 78edd57 (UI polish) accidentally deleted the testing/CI layer.
This restores it and fixes repo hygiene:
- Restore 72 tests, vitest.config.js, CI workflow, husky pre-commit,
Phase 2 planning doc and conductor track docs from 78edd57~1
- Adjust restored tests minimally for UI drift (tests only, no source)
- Fix UTF-16-corrupted .gitignore entry; untrack stale coverage/
- Consolidate on pnpm: drop package-lock.json, CI uses pnpm with
frozen lockfile, pre-commit uses pnpm exec lint-staged
- Add packageManager field, drop deprecated 'git add' from lint-staged
- Update README (pnpm instructions + CI badge) and track docs to pnpm
@vercel

vercelBot commented Jun 11, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
computer-selector-helperReadyReadyPreview, CommentJun 11, 2026 5:19am

@FranGuh
FranGuh merged commit 70d7673 into mainJun 11, 2026
3 checks passed
@FranGuh
FranGuh deleted the fix/restore-testing-ci branch June 14, 2026 07:17
Sign up for freeto 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.

1 participant

@FranGuh