heroshot 0.0.2-alpha.1 → 0.0.2-alpha.2

package/eslint.config.js

@@ -1,327 +0,0 @@
-// @ts-check
-import eslint from '@eslint/js';
-import barrelFiles from 'eslint-plugin-barrel-files';
-import eslintComments from 'eslint-plugin-eslint-comments';
-import importPlugin from 'eslint-plugin-import';
-import promisePlugin from 'eslint-plugin-promise';
-import regexp from 'eslint-plugin-regexp';
-import security from 'eslint-plugin-security';
-import sonarjs from 'eslint-plugin-sonarjs';
-import svelte from 'eslint-plugin-svelte';
-import unicorn from 'eslint-plugin-unicorn';
-import unusedImports from 'eslint-plugin-unused-imports';
-import prettierConfig from 'eslint-config-prettier';
-import svelteParser from 'svelte-eslint-parser';
-import tseslint from 'typescript-eslint';
-
-export default tseslint.config(
-  eslint.configs.recommended,
-  ...tseslint.configs.strictTypeChecked,
-  ...tseslint.configs.stylisticTypeChecked,
-  importPlugin.flatConfigs.recommended,
-  importPlugin.flatConfigs.typescript,
-  // @ts-expect-error - flat config exists but types are incomplete
-  promisePlugin.configs['flat/recommended'],
-  unicorn.configs.recommended,
-  sonarjs.configs.recommended,
-  prettierConfig, // Must be last to disable conflicting formatting rules
-  {
-    plugins: {
-      'barrel-files': barrelFiles,
-      regexp,
-      security,
-      'eslint-comments': eslintComments,
-      'unused-imports': unusedImports,
-    },
-    languageOptions: {
-      parserOptions: {
-        project: true,
-        tsconfigRootDir: import.meta.dirname,
-      },
-    },
-    settings: {
-      'import/resolver': {
-        node: true,
-        typescript: true,
-      },
-    },
-    rules: {
-      // ===== TypeScript Strict Rules =====
-      '@typescript-eslint/no-explicit-any': 'error',
-      '@typescript-eslint/no-unsafe-assignment': 'error',
-      '@typescript-eslint/no-unsafe-call': 'error',
-      '@typescript-eslint/no-unsafe-member-access': 'error',
-      '@typescript-eslint/no-unsafe-return': 'error',
-      '@typescript-eslint/no-unsafe-argument': 'error',
-
-      // Allow ONLY `as const`; ban all other assertions (`as Type`, `<Type>expr`)
-      '@typescript-eslint/consistent-type-assertions': 'off',
-      '@typescript-eslint/prefer-as-const': 'error',
-      '@typescript-eslint/no-unnecessary-type-assertion': 'error',
-      'no-restricted-syntax': [
-        'error',
-        {
-          selector: 'TSAsExpression',
-          message:
-            'Type assertions (as Type) are disallowed. Use type guards/predicates instead. ' +
-            'Exception: `as const` is allowed for literal narrowing.',
-        },
-        {
-          selector: 'TSTypeAssertion',
-          message:
-            'Angle-bracket type assertions (<Type>expr) are disallowed. Use type guards/predicates instead.',
-        },
-        {
-          selector: 'ExportAllDeclaration',
-          message:
-            'Re-exporting with export * is disallowed. ' +
-            'Import the symbols and export them explicitly at their definition instead.',
-        },
-        {
-          selector: 'ExportNamedDeclaration[source]',
-          message:
-            'Re-exporting with export { } from is disallowed. ' +
-            'Import the symbols and export them explicitly at their definition instead.',
-        },
-        {
-          // bans `arr.map(f => f.prop)` - prefer destructuring `arr.map(({ prop }) => prop)`
-          selector:
-            'ArrowFunctionExpression[params.length=1][params.0.type=Identifier][body.type=MemberExpression][body.computed=false]',
-          message: 'Use destructuring in callback: prefer ({ prop }) => prop over f => f.prop',
-        },
-      ],
-
-      // Let TypeScript infer return types
-      '@typescript-eslint/explicit-function-return-type': 'off',
-      '@typescript-eslint/explicit-module-boundary-types': 'off',
-
-      // Forbid unnecessary type annotations
-      '@typescript-eslint/no-inferrable-types': [
-        'error',
-        {
-          ignoreParameters: false,
-          ignoreProperties: false,
-        },
-      ],
-
-      // Forbid .ts extensions in type-only imports side effects
-      '@typescript-eslint/no-import-type-side-effects': 'error',
-
-      // ===== Promise Handling =====
-      '@typescript-eslint/no-floating-promises': 'error',
-      '@typescript-eslint/promise-function-async': 'error',
-      '@typescript-eslint/await-thenable': 'error',
-      '@typescript-eslint/return-await': ['error', 'in-try-catch'],
-
-      // ===== TypeScript Import/Export =====
-      '@typescript-eslint/consistent-type-exports': 'error',
-      '@typescript-eslint/consistent-type-imports': 'error',
-      '@typescript-eslint/consistent-type-definitions': ['error', 'interface'],
-      '@typescript-eslint/no-require-imports': 'error',
-      'no-restricted-imports': [
-        'error',
-        {
-          patterns: [
-            {
-              regex: '.*',
-              importNamePattern: '^\\*$',
-              message:
-                'Namespace imports (import * as) are not allowed. Use named imports instead.',
-            },
-            {
-              regex: '\\.js$',
-              message: 'Do not use .js extension in imports. TypeScript handles module resolution.',
-            },
-          ],
-        },
-      ],
-
-      // Variable scoping
-      'no-shadow': 'off',
-      '@typescript-eslint/no-shadow': 'error',
-
-      // ===== Import Organization =====
-      'import/order': [
-        'error',
-        {
-          groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
-          'newlines-between': 'never',
-          alphabetize: { order: 'asc', caseInsensitive: true },
-        },
-      ],
-      'sort-imports': ['error', { ignoreDeclarationSort: true }],
-
-      // ===== Import Hygiene =====
-      'import/namespace': 'off', // Slow, TypeScript already validates imports
-      'import/no-extraneous-dependencies': ['error', { devDependencies: true }],
-
-      // ===== Unused Imports =====
-      'no-unused-vars': 'off',
-      '@typescript-eslint/no-unused-vars': 'off',
-      'unused-imports/no-unused-imports': 'error',
-      'unused-imports/no-unused-vars': [
-        'error',
-        {
-          vars: 'all',
-          varsIgnorePattern: '^_',
-          args: 'after-used',
-          argsIgnorePattern: '^_',
-        },
-      ],
-
-      // ===== Security =====
-      '@typescript-eslint/no-implied-eval': 'error',
-      '@typescript-eslint/restrict-template-expressions': [
-        'error',
-        {
-          allowNumber: true,
-          allowBoolean: true,
-          allowAny: false,
-          allowNullish: false,
-        },
-      ],
-      'no-eval': 'error',
-      'no-new-func': 'error',
-      'no-script-url': 'error',
-      'no-inner-declarations': 'error',
-
-      // Regex DoS Prevention (eslint-plugin-regexp)
-      'regexp/no-super-linear-backtracking': 'error',
-      'regexp/no-dupe-characters-character-class': 'warn',
-      'regexp/confusing-quantifier': 'warn',
-      'regexp/optimal-quantifier-concatenation': 'warn',
-
-      // General Security Hotspots (eslint-plugin-security)
-      'security/detect-non-literal-regexp': 'warn',
-      'security/detect-child-process': 'error',
-      'security/detect-new-buffer': 'error',
-      'security/detect-non-literal-fs-filename': 'warn',
-      'security/detect-object-injection': 'off', // Too many false positives
-
-      // ===== Code Quality =====
-      'max-params': ['error', { max: 5 }],
-      complexity: ['error', { max: 20 }],
-
-      // ===== Code Style =====
-      'object-shorthand': ['error', 'always'],
-      'prefer-destructuring': [
-        'error',
-        {
-          VariableDeclarator: { array: true, object: true },
-          AssignmentExpression: { array: false, object: false },
-        },
-        { enforceForRenamedProperties: true },
-      ],
-
-      // ===== Barrel Files =====
-      'barrel-files/avoid-barrel-files': 'error',
-
-      // ===== ESLint Comments =====
-      'eslint-comments/no-use': [
-        'error',
-        {
-          allow: ['eslint-disable-next-line', 'eslint-disable'],
-        },
-      ],
-      'eslint-comments/require-description': [
-        'error',
-        {
-          ignore: [],
-        },
-      ],
-      'eslint-comments/disable-enable-pair': [
-        'error',
-        {
-          allowWholeFile: false,
-        },
-      ],
-
-      // ===== Unicorn Overrides =====
-      'unicorn/filename-case': ['error', { case: 'camelCase' }],
-      'unicorn/no-null': 'off', // Allow null - used by external APIs
-      'unicorn/no-array-callback-reference': 'off', // Callback references are fine and readable
-      'unicorn/prefer-number-properties': 'off', // isNaN is fine
-
-      // ===== SonarJS Overrides =====
-      'sonarjs/cognitive-complexity': ['error', 35],
-      'sonarjs/no-nested-template-literals': 'off',
-      'sonarjs/no-dead-store': 'off', // False positives on loop control variables
-      'sonarjs/redundant-type-aliases': 'off', // Type aliases improve readability
-      'sonarjs/no-in-misuse': 'off',
-      'sonarjs/todo-tag': 'off', // TODO comments are useful
-      'sonarjs/deprecation': 'off', // Expensive check
-      'sonarjs/aws-restricted-ip-admin-access': 'off', // Not relevant
-
-      // Allow || for default values
-      '@typescript-eslint/prefer-nullish-coalescing': 'off',
-      // Allow void expressions in arrow functions
-      '@typescript-eslint/no-confusing-void-expression': 'off',
-      // Allow unnecessary conditions (helps with optional chaining)
-      '@typescript-eslint/no-unnecessary-condition': 'off',
-    },
-  },
-  {
-    // Toolbar folder - uses its own tsconfig
-    files: ['toolbar/**/*.ts'],
-    languageOptions: {
-      parserOptions: {
-        project: './toolbar/tsconfig.json',
-        tsconfigRootDir: import.meta.dirname,
-      },
-    },
-    rules: {
-      // Disable prefer-destructuring - false positives with renamed destructuring
-      'prefer-destructuring': 'off',
-    },
-  },
-  // Svelte files configuration
-  ...svelte.configs['flat/recommended'],
-  {
-    files: ['toolbar/**/*.svelte'],
-    languageOptions: {
-      parser: svelteParser,
-      parserOptions: {
-        parser: tseslint.parser,
-        project: './toolbar/tsconfig.json',
-        tsconfigRootDir: import.meta.dirname,
-        extraFileExtensions: ['.svelte'],
-      },
-      globals: {
-        document: 'readonly',
-        globalThis: 'readonly',
-      },
-    },
-    rules: {
-      // Svelte specific rules
-      'svelte/no-at-html-tags': 'error',
-      'svelte/require-each-key': 'error',
-      'svelte/valid-each-key': 'error',
-      // Disable rules that conflict with Svelte
-      'import/no-mutable-exports': 'off',
-      '@typescript-eslint/no-unsafe-assignment': 'off', // Svelte bindings cause issues
-      '@typescript-eslint/no-unsafe-member-access': 'off',
-      '@typescript-eslint/no-unsafe-call': 'off',
-      // Svelte uses PascalCase for component files
-      'unicorn/filename-case': ['error', { case: 'pascalCase' }],
-      // Allow Props abbreviation in Svelte - common pattern
-      'unicorn/prevent-abbreviations': [
-        'error',
-        { replacements: { props: false, e: { event: true } } },
-      ],
-      // Disable prefer-destructuring - false positives with svelte-eslint-parser
-      'prefer-destructuring': 'off',
-    },
-  },
-  {
-    // Ignore generated files, config files, and tests (tests only need to pass typecheck)
-    ignores: [
-      '**/node_modules/**',
-      '**/dist/**',
-      '*.config.js',
-      '*.config.ts',
-      'toolbar/*.config.ts',
-      '**/*.test.ts',
-      '**/tests/**',
-    ],
-  }
-);

package/knip.json

@@ -1,6 +0,0 @@
-{
-  "ignoreDependencies": ["sharp"],
-  "ignore": ["toolbar/**", "src/capture.ts"],
-  "ignoreExportsUsedInFile": true,
-  "entry": ["src/config.ts"]
-}

package/scripts/pre-commit.sh

@@ -1,71 +0,0 @@
-#!/bin/sh
-
-# Create temp directory for outputs (gitignored via .* pattern)
-mkdir -p .pre-commit-logs
-
-# First: auto-fix formatting and re-stage any changed files
-echo "Running Prettier auto-fix..."
-pnpm format > .pre-commit-logs/format.log 2>&1
-if [ $? -ne 0 ]; then
-  echo "FAIL: Prettier failed:"
-  cat .pre-commit-logs/format.log
-  exit 1
-fi
-# Re-stage any files that were formatted
-git add -u
-
-# Run remaining quality checks in parallel, save outputs to files
-echo "Running pre-commit checks in parallel..."
-
-pnpm lint > .pre-commit-logs/lint.log 2>&1 &
-P1=$!
-pnpm typecheck > .pre-commit-logs/typecheck.log 2>&1 &
-P2=$!
-pnpm knip > .pre-commit-logs/knip.log 2>&1 &
-P3=$!
-pnpm test:run > .pre-commit-logs/tests.log 2>&1 &
-P4=$!
-
-# Wait for all jobs
-wait $P1; R1=$?
-wait $P2; R2=$?
-wait $P3; R3=$?
-wait $P4; R4=$?
-
-# Check results and show output only on failure
-FAILED=0
-
-if [ $R1 -ne 0 ]; then
-  echo "FAIL: ESLint check failed:"
-  cat .pre-commit-logs/lint.log
-  FAILED=1
-fi
-
-if [ $R2 -ne 0 ]; then
-  echo "FAIL: TypeScript check failed:"
-  cat .pre-commit-logs/typecheck.log
-  FAILED=1
-fi
-
-if [ $R3 -ne 0 ]; then
-  echo "FAIL: Knip check failed:"
-  cat .pre-commit-logs/knip.log
-  FAILED=1
-fi
-
-if [ $R4 -ne 0 ]; then
-  echo "FAIL: Tests failed:"
-  cat .pre-commit-logs/tests.log
-  FAILED=1
-fi
-
-if [ $FAILED -eq 0 ]; then
-  echo "OK: All pre-commit checks passed!"
-  # Clean up logs on success
-  rm -rf .pre-commit-logs
-  exit 0
-else
-  echo ""
-  echo "Tip: Check .pre-commit-logs/ for detailed output"
-  exit 1
-fi