chore(tracker-web): clear roadmap lint warnings
What changed: - Replaced roadmap console error calls with page-level error state. - Removed the stale eslint.config.js now that eslint.config.mjs is present. Warning impact: - @bytelyst/tracker-web scoped lint: 2 warnings + module warning -> 0 warnings. - Workspace lint: 95 -> 90 warnings. Verification: - pnpm --filter @bytelyst/tracker-web typecheck - pnpm --filter @bytelyst/tracker-web test - pnpm --filter @bytelyst/tracker-web exec eslint . --ext .ts,.tsx - pnpm --filter @bytelyst/tracker-web build - pnpm lint
This commit is contained in:
parent
db4257fd76
commit
04d239848e
@ -1,104 +0,0 @@
|
||||
import js from '@eslint/js';
|
||||
import tseslint from '@typescript-eslint/eslint-plugin';
|
||||
import tsparser from '@typescript-eslint/parser';
|
||||
|
||||
export default [
|
||||
{
|
||||
ignores: [
|
||||
'dist/**',
|
||||
'node_modules/**',
|
||||
'coverage/**',
|
||||
'*.config.js',
|
||||
'*.config.ts',
|
||||
'.next/**',
|
||||
'out/**',
|
||||
],
|
||||
},
|
||||
js.configs.recommended,
|
||||
{
|
||||
files: ['**/*.ts', '**/*.tsx'],
|
||||
languageOptions: {
|
||||
parser: tsparser,
|
||||
parserOptions: {
|
||||
ecmaVersion: 2022,
|
||||
sourceType: 'module',
|
||||
},
|
||||
globals: {
|
||||
process: 'readonly',
|
||||
Buffer: 'readonly',
|
||||
__dirname: 'readonly',
|
||||
__filename: 'readonly',
|
||||
Headers: 'readonly',
|
||||
TextEncoder: 'readonly',
|
||||
TextDecoder: 'readonly',
|
||||
RequestInit: 'readonly',
|
||||
HeadersInit: 'readonly',
|
||||
Response: 'readonly',
|
||||
Request: 'readonly',
|
||||
fetch: 'readonly',
|
||||
console: 'readonly',
|
||||
performance: 'readonly',
|
||||
AbortSignal: 'readonly',
|
||||
expect: 'readonly',
|
||||
describe: 'readonly',
|
||||
it: 'readonly',
|
||||
test: 'readonly',
|
||||
beforeEach: 'readonly',
|
||||
afterEach: 'readonly',
|
||||
beforeAll: 'readonly',
|
||||
afterAll: 'readonly',
|
||||
crypto: 'readonly',
|
||||
localStorage: 'readonly',
|
||||
sessionStorage: 'readonly',
|
||||
ReactNode: 'readonly',
|
||||
React: 'readonly',
|
||||
JSX: 'readonly',
|
||||
window: 'readonly',
|
||||
document: 'readonly',
|
||||
navigator: 'readonly',
|
||||
setTimeout: 'readonly',
|
||||
clearTimeout: 'readonly',
|
||||
setInterval: 'readonly',
|
||||
clearInterval: 'readonly',
|
||||
URL: 'readonly',
|
||||
URLSearchParams: 'readonly',
|
||||
confirm: 'readonly',
|
||||
alert: 'readonly',
|
||||
HTMLDivElement: 'readonly',
|
||||
HTMLInputElement: 'readonly',
|
||||
HTMLButtonElement: 'readonly',
|
||||
MessageEvent: 'readonly',
|
||||
},
|
||||
},
|
||||
plugins: {
|
||||
'@typescript-eslint': tseslint,
|
||||
},
|
||||
rules: {
|
||||
// TypeScript specific rules
|
||||
'@typescript-eslint/no-unused-vars': [
|
||||
'error',
|
||||
{ argsIgnorePattern: '^_', varsIgnorePattern: '^_', caughtErrorsIgnorePattern: '^_' },
|
||||
],
|
||||
'@typescript-eslint/explicit-function-return-type': 'off',
|
||||
'@typescript-eslint/explicit-module-boundary-types': 'off',
|
||||
'@typescript-eslint/no-explicit-any': 'warn',
|
||||
|
||||
// General rules
|
||||
'no-console': 'warn',
|
||||
'no-debugger': 'error',
|
||||
'prefer-const': 'error',
|
||||
'no-var': 'error',
|
||||
'no-unused-vars': 'off', // Use TypeScript version
|
||||
|
||||
// Import rules - disabled for now
|
||||
// 'sort-imports': ['error', { ignoreDeclarationSort: true }],
|
||||
},
|
||||
},
|
||||
{
|
||||
files: ['next.config.ts'],
|
||||
rules: {
|
||||
'@typescript-eslint/no-var-requires': 'off',
|
||||
'@typescript-eslint/no-require-imports': 'off',
|
||||
},
|
||||
},
|
||||
];
|
||||
@ -67,6 +67,7 @@ export default function RoadmapPage() {
|
||||
const [items, setItems] = useState<TrackerItem[]>([]);
|
||||
const [stats, setStats] = useState<PublicRoadmapStats | null>(null);
|
||||
const [loading, setLoading] = useState(true);
|
||||
const [loadError, setLoadError] = useState('');
|
||||
const [search, setSearch] = useState('');
|
||||
const [typeFilter, setTypeFilter] = useState('');
|
||||
const [view, setView] = useState<'board' | 'list'>('board');
|
||||
@ -88,6 +89,7 @@ export default function RoadmapPage() {
|
||||
const [showEmailPrompt, setShowEmailPrompt] = useState(false);
|
||||
const [pendingVoteId, setPendingVoteId] = useState<string | null>(null);
|
||||
const [votedItems, setVotedItems] = useState<Set<string>>(new Set());
|
||||
const [voteError, setVoteError] = useState('');
|
||||
|
||||
useEffect(() => {
|
||||
if (typeof window !== 'undefined') {
|
||||
@ -100,6 +102,7 @@ export default function RoadmapPage() {
|
||||
|
||||
const fetchData = useCallback(async () => {
|
||||
setLoading(true);
|
||||
setLoadError('');
|
||||
try {
|
||||
const params: Record<string, string> = {
|
||||
sortBy: 'voteCount',
|
||||
@ -112,7 +115,7 @@ export default function RoadmapPage() {
|
||||
setItems(itemsRes.items);
|
||||
setStats(statsRes);
|
||||
} catch (err) {
|
||||
console.error('Failed to load roadmap:', err);
|
||||
setLoadError(err instanceof Error ? err.message : 'Failed to load roadmap');
|
||||
} finally {
|
||||
setLoading(false);
|
||||
}
|
||||
@ -132,6 +135,7 @@ export default function RoadmapPage() {
|
||||
};
|
||||
|
||||
const doVote = async (itemId: string, email: string) => {
|
||||
setVoteError('');
|
||||
try {
|
||||
const res = await publicVote(itemId, email);
|
||||
setItems(prev => prev.map(i => (i.id === itemId ? { ...i, voteCount: res.voteCount } : i)));
|
||||
@ -144,7 +148,7 @@ export default function RoadmapPage() {
|
||||
setVotedItems(newVoted);
|
||||
localStorage.setItem('roadmap_voted', JSON.stringify([...newVoted]));
|
||||
} catch (err) {
|
||||
console.error('Vote failed:', err);
|
||||
setVoteError(err instanceof Error ? err.message : 'Vote failed');
|
||||
}
|
||||
};
|
||||
|
||||
@ -217,6 +221,17 @@ export default function RoadmapPage() {
|
||||
</header>
|
||||
|
||||
<main className="max-w-7xl mx-auto px-4 sm:px-6 py-6">
|
||||
{loadError ? (
|
||||
<div className="mb-4 rounded-lg border border-red-200 bg-red-50 px-4 py-3 text-sm text-red-700 dark:border-red-900 dark:bg-red-900/20 dark:text-red-300">
|
||||
{loadError}
|
||||
</div>
|
||||
) : null}
|
||||
{voteError ? (
|
||||
<div className="mb-4 rounded-lg border border-red-200 bg-red-50 px-4 py-3 text-sm text-red-700 dark:border-red-900 dark:bg-red-900/20 dark:text-red-300">
|
||||
{voteError}
|
||||
</div>
|
||||
) : null}
|
||||
|
||||
{/* Stats bar */}
|
||||
{stats && (
|
||||
<div className="grid grid-cols-2 sm:grid-cols-4 gap-4 mb-6">
|
||||
|
||||
Loading…
Reference in New Issue
Block a user