@thepalaceproject/circulation-admin 1.41.0-post.33 → 1.41.0-post.34
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CLAUDE.md +2 -1
- package/dist/circulation-admin.js +1 -1
- package/eslint.config.mjs +11 -19
- package/jest.polyfills.js +0 -1
- package/package.json +2 -1
- package/scripts/syncPatronBlockingDocs.js +0 -1
- package/testReporter.js +0 -2
- package/tests/browser/navigate.js +114 -124
- package/tests/browser/redirect.js +26 -41
- package/tests/jest/components/AdvancedSearchBuilder.test.tsx +1 -1
- package/tests/jest/components/CustomLists.test.tsx +0 -1
- package/tests/jest/components/DebugAuthentication.test.tsx +0 -2
- package/tests/jest/components/InventoryReportRequestModal.test.tsx +0 -2
- package/tests/jest/components/JsonField.test.tsx +0 -1
- package/tests/jest/components/LaneEditor.test.tsx +1 -1
- package/tests/jest/components/PatronBlockingRulesHelpModal.test.tsx +6 -8
- package/tests/jest/components/QuicksightDashboard.test.tsx +0 -1
- package/tests/jest/components/Stats.test.tsx +7 -13
- package/tests/jest/features/book.test.ts +2 -7
- package/webpack.common.js +0 -2
- package/webpack.dev-server.config.js +0 -2
package/CLAUDE.md
CHANGED
|
@@ -15,6 +15,7 @@ npm run dev # Watch build (for use with local Circulation Man
|
|
|
15
15
|
npm run dev-server -- --env=backend=https://your-cm-url # Dev server against remote backend
|
|
16
16
|
npm run prod # Production build
|
|
17
17
|
npm run lint # Run tslint + sass-lint
|
|
18
|
+
npm run lint:js # Run ESLint over the whole tree (eslint . --max-warnings 0)
|
|
18
19
|
npm test # Full test suite (mocha legacy + jest)
|
|
19
20
|
npm run test-jest # Jest tests only
|
|
20
21
|
npm run test-jest-file <path> # Single jest test: npm run test-jest-file tests/jest/components/MyTest.test.tsx
|
|
@@ -70,7 +71,7 @@ The dev server can also read the backend URL from `.env` or `.env.local` with `B
|
|
|
70
71
|
## Code Style
|
|
71
72
|
|
|
72
73
|
- Prettier: double quotes, semicolons, trailing commas (es5), 80 char width
|
|
73
|
-
- ESLint with `jsx-a11y/strict` —
|
|
74
|
+
- ESLint (flat config, `eslint.config.mjs`) with `jsx-a11y/strict` — runs on the whole tree in CI via `npm run lint:js` (`eslint . --max-warnings 0`, so warnings fail the build) and on staged files via the husky pre-commit hook. Separate from `npm run lint`, which runs tslint + sass-lint.
|
|
74
75
|
- `@typescript-eslint/no-explicit-any` is disabled (any is allowed)
|
|
75
76
|
- Prefer template literals over string concatenation for building strings with variables
|
|
76
77
|
- When an arrow function only passes its arguments through to another function with the same signature, pass the function directly instead of wrapping it (e.g., `onClick: this.handleClick` not `onClick: (e) => this.handleClick(e)`)
|