Skip to content

Commit bdda3b8

Browse files
authored
Modernize the repo toolchain and remove dead legacy config (#75)
This PR cleans up a large amount of leftover Nx/Jest/Babel-era repo config and updates the workspace to a current Vite/Vitest/TypeScript setup. It removes dead files and unused config, including old Jest root config, Babel config, Browserslist files, EditorConfig, commitlint/commitizen config, the unused tools/ directory, stale root index.html, vercel.json, .nvmrc, and other no-longer-used repo metadata. It also modernizes the workspace toolchain: - standardizes on Vitest with test and test:ci - adds typecheck scripts across packages/apps and requires typecheck in release CI - updates TypeScript targets to ES2024 - switches modern app/config TS surfaces to moduleResolution: "bundler" - updates root/workspace package scripts and CI workflows to the new commands - cleans up ESLint usage so ignore behavior lives in config - On the package side, it updates @trrack/core and @trrack/vis-react build metadata and Vite configs, removes UMD output, tightens externals, and aligns package exports/types with the current build output. It also includes follow-on correctness fixes needed by the stricter toolchain: - preserves the synchronous TrrackActionFunction contract - fixes core/example type issues so repo-wide typecheck passes - updates vis-react internals away from Mantine/FontAwesome-heavy UI dependencies toward lighter native UI - improves vis-react accessibility by properly associating the annotation label and textarea - expands vis-react test coverage for node navigation, bookmarks, annotations, and current-node extras
2 parents e3fe8bb + af4cabc commit bdda3b8

124 files changed

Lines changed: 5151 additions & 4423 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.commitlintrc.json

Lines changed: 0 additions & 4 deletions
This file was deleted.

.czrc

Lines changed: 0 additions & 3 deletions
This file was deleted.

.editorconfig

Lines changed: 0 additions & 13 deletions
This file was deleted.

.eslintignore

Lines changed: 0 additions & 3 deletions
This file was deleted.

.eslintrc.json

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
11
{
22
"root": true,
3-
"ignorePatterns": ["**/*"],
4-
"globals": {
5-
"vi": true
6-
},
3+
"ignorePatterns": ["node_modules", "**/node_modules/**/*", "dist", "coverage", ".next"],
74
"overrides": [
85
{
9-
"files": ["*.ts", "*.tsx"],
6+
"files": ["**/*.ts", "**/*.tsx"],
107
"extends": ["plugin:@typescript-eslint/recommended"],
118
"parser": "@typescript-eslint/parser",
12-
"rules": {}
13-
},
14-
{
15-
"files": ["*.js", "*.jsx"],
9+
"parserOptions": {
10+
"ecmaVersion": 2024,
11+
"sourceType": "module",
12+
"ecmaFeatures": {
13+
"jsx": true
14+
}
15+
},
1616
"rules": {}
1717
}
1818
]

.github/workflows/build_test.yml

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,23 @@ jobs:
3636
- name: Install dependencies
3737
run: yarn install
3838

39+
- name: Typecheck
40+
run: yarn typecheck
41+
3942
- name: Build packages
40-
run: yarn build:packages
43+
run: yarn build
44+
45+
- name: Test with coverage
46+
run: yarn test:coverage
4147

42-
- name: Test
43-
run: yarn test:all
48+
- name: Publish coverage summary
49+
if: always()
50+
run: node scripts/summarize-coverage.mjs >> "$GITHUB_STEP_SUMMARY"
51+
52+
- name: Upload coverage reports
53+
if: always()
54+
uses: actions/upload-artifact@v4
55+
with:
56+
name: coverage-reports
57+
path: coverage
58+
if-no-files-found: warn

.github/workflows/build_test_release.yml

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,14 @@ jobs:
3333
- name: Install dependencies
3434
run: yarn install
3535

36+
- name: Typecheck
37+
run: yarn typecheck
38+
3639
- name: Build packages
37-
run: yarn build:packages
40+
run: yarn build
3841

3942
- name: Test
40-
run: yarn test:all
43+
run: yarn test:ci
4144

4245
- name: Resolve release metadata
4346
id: release_meta
@@ -103,8 +106,16 @@ jobs:
103106
104107
npm pkg set version="${RELEASE_VERSION}" --prefix "${dist_dir}"
105108
npm pkg delete scripts --prefix "${dist_dir}"
109+
npm pkg delete devDependencies --prefix "${dist_dir}"
106110
done
107111
112+
- name: Verify package contents
113+
run: |
114+
set -euo pipefail
115+
116+
npm pack --dry-run packages/core/dist
117+
npm pack --dry-run packages/vis-react/dist
118+
108119
- name: Publish packages
109120
env:
110121
DIST_TAG: ${{ steps.release_meta.outputs.dist_tag }}

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ Thumbs.db
4040

4141
# Next.js
4242
.next
43+
*.tsbuildinfo
4344

4445
# Nx
4546
migrations.json

.husky/pre-commit

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
#!/usr/bin/env sh
22
. "$(dirname -- "$0")/_/husky.sh"
33

4-
yarn build:packages && yarn lint:packages
4+
yarn build && yarn lint

.nvmrc

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)