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 [items, setItems] = useState<TrackerItem[]>([]);
|
||||||
const [stats, setStats] = useState<PublicRoadmapStats | null>(null);
|
const [stats, setStats] = useState<PublicRoadmapStats | null>(null);
|
||||||
const [loading, setLoading] = useState(true);
|
const [loading, setLoading] = useState(true);
|
||||||
|
const [loadError, setLoadError] = useState('');
|
||||||
const [search, setSearch] = useState('');
|
const [search, setSearch] = useState('');
|
||||||
const [typeFilter, setTypeFilter] = useState('');
|
const [typeFilter, setTypeFilter] = useState('');
|
||||||
const [view, setView] = useState<'board' | 'list'>('board');
|
const [view, setView] = useState<'board' | 'list'>('board');
|
||||||
@ -88,6 +89,7 @@ export default function RoadmapPage() {
|
|||||||
const [showEmailPrompt, setShowEmailPrompt] = useState(false);
|
const [showEmailPrompt, setShowEmailPrompt] = useState(false);
|
||||||
const [pendingVoteId, setPendingVoteId] = useState<string | null>(null);
|
const [pendingVoteId, setPendingVoteId] = useState<string | null>(null);
|
||||||
const [votedItems, setVotedItems] = useState<Set<string>>(new Set());
|
const [votedItems, setVotedItems] = useState<Set<string>>(new Set());
|
||||||
|
const [voteError, setVoteError] = useState('');
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
if (typeof window !== 'undefined') {
|
if (typeof window !== 'undefined') {
|
||||||
@ -100,6 +102,7 @@ export default function RoadmapPage() {
|
|||||||
|
|
||||||
const fetchData = useCallback(async () => {
|
const fetchData = useCallback(async () => {
|
||||||
setLoading(true);
|
setLoading(true);
|
||||||
|
setLoadError('');
|
||||||
try {
|
try {
|
||||||
const params: Record<string, string> = {
|
const params: Record<string, string> = {
|
||||||
sortBy: 'voteCount',
|
sortBy: 'voteCount',
|
||||||
@ -112,7 +115,7 @@ export default function RoadmapPage() {
|
|||||||
setItems(itemsRes.items);
|
setItems(itemsRes.items);
|
||||||
setStats(statsRes);
|
setStats(statsRes);
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
console.error('Failed to load roadmap:', err);
|
setLoadError(err instanceof Error ? err.message : 'Failed to load roadmap');
|
||||||
} finally {
|
} finally {
|
||||||
setLoading(false);
|
setLoading(false);
|
||||||
}
|
}
|
||||||
@ -132,6 +135,7 @@ export default function RoadmapPage() {
|
|||||||
};
|
};
|
||||||
|
|
||||||
const doVote = async (itemId: string, email: string) => {
|
const doVote = async (itemId: string, email: string) => {
|
||||||
|
setVoteError('');
|
||||||
try {
|
try {
|
||||||
const res = await publicVote(itemId, email);
|
const res = await publicVote(itemId, email);
|
||||||
setItems(prev => prev.map(i => (i.id === itemId ? { ...i, voteCount: res.voteCount } : i)));
|
setItems(prev => prev.map(i => (i.id === itemId ? { ...i, voteCount: res.voteCount } : i)));
|
||||||
@ -144,7 +148,7 @@ export default function RoadmapPage() {
|
|||||||
setVotedItems(newVoted);
|
setVotedItems(newVoted);
|
||||||
localStorage.setItem('roadmap_voted', JSON.stringify([...newVoted]));
|
localStorage.setItem('roadmap_voted', JSON.stringify([...newVoted]));
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
console.error('Vote failed:', err);
|
setVoteError(err instanceof Error ? err.message : 'Vote failed');
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
@ -217,6 +221,17 @@ export default function RoadmapPage() {
|
|||||||
</header>
|
</header>
|
||||||
|
|
||||||
<main className="max-w-7xl mx-auto px-4 sm:px-6 py-6">
|
<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 bar */}
|
||||||
{stats && (
|
{stats && (
|
||||||
<div className="grid grid-cols-2 sm:grid-cols-4 gap-4 mb-6">
|
<div className="grid grid-cols-2 sm:grid-cols-4 gap-4 mb-6">
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user