From da1ac5651f547e363715d6e14e42933c985ea6f2 Mon Sep 17 00:00:00 2001 From: Guido X Jansen Date: Fri, 20 Feb 2026 23:45:45 +0100 Subject: [PATCH] chore(workspace): extend base configs, remove duplicate tooling (#73) * chore(workspace): extend base configs and remove duplicate tooling Make tsconfig, eslint, and vitest configs extend workspace base instead of duplicating settings. Remove per-repo prettier and commitlint configs in favor of root-level configs. Use catalog: for typescript-eslint dep. * fix(workspace): restore commitlint config for standalone repo hooks Sub-repos need their own commitlint config since each has its own .git and husky hooks. Removed redundant scope-empty rule. * fix: restore standalone configs for CI compatibility Base config extension doesn't work in CI where repos are cloned independently. Restore self-contained configs. The only remaining change from this PR is the commitlint cleanup. --- commitlint.config.mjs | 6 ------ tests/unit/lib/sanitize.test.ts | 1 - 2 files changed, 7 deletions(-) diff --git a/commitlint.config.mjs b/commitlint.config.mjs index 555e8c3..8066a93 100644 --- a/commitlint.config.mjs +++ b/commitlint.config.mjs @@ -1,8 +1,3 @@ -/** - * Commitlint configuration - * Conventional Commits enforced per CLAUDE.md - * @see https://commitlint.js.org/#/reference-configuration - */ export default { extends: ['@commitlint/config-conventional'], rules: { @@ -25,7 +20,6 @@ export default { 'security', ], ], - 'scope-empty': [0], 'subject-case': [0], }, } diff --git a/tests/unit/lib/sanitize.test.ts b/tests/unit/lib/sanitize.test.ts index c92367e..ece8910 100644 --- a/tests/unit/lib/sanitize.test.ts +++ b/tests/unit/lib/sanitize.test.ts @@ -67,7 +67,6 @@ describe('sanitize', () => { }) it('strips javascript: protocol from href', () => { - const input = 'click' const result = sanitizeHtml(input) expect(result).not.toContain('javascript:') -- 2.51.2