Chore/lint type cleanup main by DhineshPonnarasan · Pull Request #781 · CodeGraphContext/CodeGraphContext

DhineshPonnarasan · 2026-03-31T10:41:19Z

Summary

This PR performs a comprehensive cleanup of the codebase to ensure a 100% error-free lint, type-check, and build process.

What was fixed and implemented

TypeScript & ESLint Errors

Removed all TypeScript type errors and ESLint errors across the codebase
Fixed all any usages, missing/incorrect types, and interface/type issues
Removed or replaced all empty interfaces/types to comply with @typescript-eslint/no-empty-object-type
Fixed unnecessary escape character errors in regexes (e.g., in LocalUploader.tsx, parser.worker.ts, and API handlers)
Addressed block statement and empty block errors

Regex & Validation

Cleaned up regex patterns to remove unnecessary escapes
Ensured cross-platform compatibility
Updated path splitting and validation logic in file upload and parsing utilities

Component & API Refactoring

Ensured all React components and API handlers are properly typed
Updated imports and exports for better Fast Refresh compatibility
(Note: warnings remain, but no errors)

Dependency Management

Installed missing dev dependencies (e.g., Next.js types for API handlers)
Ensured all required packages are present for lint/type/build

Build & Lint Verification

Ran npm run lint and npm run build to confirm zero errors
Only non-blocking warnings remain (mostly Fast Refresh/component export warnings)

Testing

No test script was present
Verification limited to linting and build process

Remaining Warnings

Some Fast Refresh/component export warnings remain
(e.g., exporting constants/functions alongside components)
No blocking issues — project is ready for further development or deployment

Motivation

Ensures codebase is maintainable and type-safe
Provides a clean baseline for future PRs
Unblocks CI/CD and contributor workflows

Closes #770

Hi @Shashankss1205 ,
Please review these changes and let me know if any further modifications are needed. If you notice any issues, please leave a comment below and I’ll address them. Thank you!

vercel · 2026-03-31T10:41:24Z

@DhineshPonnarasan is attempting to deploy a commit to the shashankss1205's projects Team on Vercel.

A member of the Team first needs to authorize it.

…anylinux build step

… workflow

…ras for PyInstaller build

…for PyInstaller build

… and falkordblite only

…dows

…on Windows

…sion checks in PyInstaller spec

…in cgc.spec

… bracket

…rom cgc.spec

…e_sitter_c_sharp/kuzu before build

…ptional

…B tests with importorskip

…manylinux build

…=5.13 install failure

Shashankss1205 · 2026-07-25T21:50:01Z

👋 Thanks for contributing to CodeGraphContext! Since this PR was opened, main has moved forward significantly (we just shipped v0.5.2 and merged ~50 PRs), so this one now has merge conflicts. Could you please rebase onto the latest main and resolve the conflicts? Once it is conflict-free and CI is green, we will review and merge it. Really appreciate your work — thank you! 🙏

Shashankss1205 · 2026-08-13T18:01:39Z

Thanks for the lint/type cleanup effort. Unfortunately this is the kind of PR that conflicts badly with time: it touches 25 files with broad mechanical changes, and against current main it now produces ~21 conflict regions spanning CI workflows (test.yml, e2e-tests.yml), pyproject.toml, cli/main.py and several website files — all of which have been modified independently since March.

Rebasing a cleanup PR of this shape is usually more work than redoing it, because the underlying code has changed and the original fixes may no longer apply where they did.

If you're still interested, the most effective approach would be to run the linter against current main and open a small PR per area (e.g. one for cli/, one for tools/languages/). Those review and land quickly, and they don't go stale while waiting.

Worth knowing: there are meta-tests that enforce some of this automatically now — test_parser_encoding_resilience (every open( in tools/languages/*.py needs errors=) and test_command_delegation_and_exit_codes (CLI abbreviations must forward every Typer option). Those are good targets if you want cleanups that are guaranteed not to be controversial.

Happy to review focused follow-ups. If you'd prefer to drop this one, just let me know and I'll close it.

chore: lint, type, and build cleanup (no errors, only warnings remain)

efe1284

DhineshPonnarasan added 17 commits March 31, 2026 06:48

fix(ci): use system Python with shared libraries for PyInstaller in m…

ab9567b

…anylinux build step

fix(ci): install parsing extras for tree-sitter dependencies in build…

8d0f69c

… workflow

fix(ci): add missing tree-sitter and kuzu dependencies to parsing ext…

9360115

…ras for PyInstaller build

fix(ci): add redislite, falkordb, and falkordblite to parsing extras …

fbdf27a

…for PyInstaller build

fix(ci): explicitly install missing packages before PyInstaller build

4f102b2

fix(ci): remove pip install for tree_sitter_* modules, keep redislite…

b0483af

… and falkordblite only

fix(ci): remove falkordblite from pip install, keep only redislite

5df46e8

fix(ci): skip redislite install on Windows runners

90466de

fix(ci): set shell: bash for Windows/Mac dependency install step

7ee0a34

fix(ci): move redislite install to its own step with runner.os check

ed5a7e7

fix: remove redislite from [parsing] extras to prevent install on Win…

1aeeecb

…dows

fix: add platform marker to falkordblite in [parsing] extras to skip …

0c11b08

…on Windows

fix: wrap falkordblite, kuzu, and tree_sitter_c_sharp in platform/ver…

c5e2960

…sion checks in PyInstaller spec

fix: remove unmatched bracket and stray 'atexit' from hidden_imports …

d103467

…in cgc.spec

fix: move 'atexit' inside hidden_imports and close list, remove stray…

36a0d8a

… bracket

fix: remove duplicate/invalid falkordblite append and stray bracket f…

717e36b

…rom cgc.spec

fix: add datas = [] initialization to resolve NameError in cgc.spec

d01cb48

DhineshPonnarasan force-pushed the chore/lint-type-cleanup-main branch from d60dbfd to d01cb48 Compare March 31, 2026 14:36

DhineshPonnarasan added 10 commits March 31, 2026 10:44

ci: ensure falkordb/falkordblite install and directory checks for tre…

b37915f

…e_sitter_c_sharp/kuzu before build

ci: remove pip install falkordb (not on PyPI), only install falkordblite

ac3ede7

ci: fix Linux build shell block quoting and clarify falkordblite is o…

29e7fd5

…ptional

fix: make FalkorDB dependencies fully optional

bee0e9f

ci: remove falkordblite install, add kuzu to test envs, guard FalkorD…

802f866

…B tests with importorskip

ci: upgrade pyinstaller to >=5.13 for Python 3.12 compatibility

b071860

ci: copy workspace to /tmp/cgc to avoid Linux path-too-long error in …

3ad3fe7

…manylinux build

ci: use Python 3.11 and manylinux pre-built cp311 to fix pyinstaller>…

1195905

…=5.13 install failure

ci: skip kuzu in manylinux pyinstaller build

bb0dc50

ci: run Linux PyInstaller build on ubuntu runner

b8a171c

DhineshPonnarasan added 2 commits April 6, 2026 03:19

Merge branch 'main' into chore/lint-type-cleanup-main

283bc98

Merge branch 'main' into chore/lint-type-cleanup-main

741a279

Navigation Menu

Uh oh!

Uh oh!

Chore/lint type cleanup main - #781

Chore/lint type cleanup main#781
DhineshPonnarasan wants to merge 30 commits into
CodeGraphContext:mainfrom
DhineshPonnarasan:chore/lint-type-cleanup-main

DhineshPonnarasan commented Mar 31, 2026

Uh oh!

vercel Bot commented Mar 31, 2026

Uh oh!

Shashankss1205 commented Jul 25, 2026

Uh oh!

Shashankss1205 commented Aug 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Uh oh!

Conversation

DhineshPonnarasan commented Mar 31, 2026

Summary

What was fixed and implemented

TypeScript & ESLint Errors

Regex & Validation

Component & API Refactoring

Dependency Management

Build & Lint Verification

Testing

Remaining Warnings

Motivation

Uh oh!

vercel Bot commented Mar 31, 2026

Uh oh!

Shashankss1205 commented Jul 25, 2026

Uh oh!

Shashankss1205 commented Aug 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants