-
Notifications
You must be signed in to change notification settings - Fork 10
Eliminate TypeScript Errors and Convert foundational .js files to .ts #413
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Changes from 94 commits
Commits
Show all changes
108 commits
Select commit
Hold shift + click to select a range
7b442e5
fix: Reduce TypeScript errors across the codebase
kriszyp 861be3d
fix: Further reduce TypeScript errors across core files
kriszyp 7100c23
fix: Strengthen types and remove 'any' casts
kriszyp 7fb4686
fix: Properly wrap DatabaseTransaction in Context and support primiti…
kriszyp c0d043f
fix: Resolve intermittent LMDB segfault by providing default CommitOp…
kriszyp ca7ea69
fix: Resolve LMDBTransaction properties and continue Table.ts typing
kriszyp 38384a5
fix: Strengthen types and resolve TS errors across multiple modules
kriszyp 091f010
fix: Strengthen types and resolve TS errors in Table.ts, search.ts, R…
kriszyp 5a4add0
fix: Further refine and strengthen types for serialization and http c…
kriszyp cb15a2e
Update resources/Resource.ts
kriszyp 22087ef
Update resources/Resource.ts
kriszyp e1e685e
fix: Restore original create method signature and resolve export.js t…
kriszyp 4fc0d70
fix: Resolve CRL verification test failure and TS errors
kriszyp 7d52c92
Formatting
kriszyp b6411cc
fix: Restore missing hasContent: false caching directives and fix tes…
kriszyp 08e71d8
fix: Resolve LMDBTransaction pooling and caching regressions
kriszyp 4fe9eb0
fix: Resolve 250+ TS errors and fix transaction/caching regressions
kriszyp a959e1a
fix: Resolve all remaining TypeScript errors and stabilize test suite
kriszyp 4292fac
fix: Resolve missing TypeScript errors in secondary files
kriszyp be10340
chore: Remove errors.txt tracking file
kriszyp db86dd6
Merge origin/main into ts-fixes, resolving all conflicts and TS regre…
kriszyp a590460
style: Fix remaining ESLint 'no-unused-vars' warnings and format code…
kriszyp e1875db
fix: Resolve LMDB testing errors and resource connect typing
kriszyp 2646aa3
style: Clean up unused CompartmentOptions import in jsLoader.ts
kriszyp ed60382
Merge origin/main into ts-fixes and resolve regressions
kriszyp d9b9538
fix: Address PR #413 review feedback
kriszyp 618d30d
fix: Revert unintentional argument parsing behavioral changes
kriszyp 4c1b1f7
Formatting
kriszyp 8dacd0e
refactor: Convert security/cryptoHash.js to TypeScript
kriszyp b1cb11e
refactor: Convert foundational utility/errors directory to TypeScript
kriszyp b875428
refactor: Convert security/fastifyAuth.js and data_objects to TypeScript
kriszyp a900330
refactor: Convert security/keys.js and environmentManager.js to TypeS…
kriszyp 5fbb450
fix: Resolve initialization order error for util in security/keys.ts
kriszyp 54c6b02
refactor: Convert multiple root utility scripts to TypeScript
kriszyp 71e97df
fix: Resolve runtime TypeError in logger and initialization order for…
kriszyp 276ab1a
fix: Consolidate remaining uncommitted style changes
kriszyp 9708185
fix: Resolve runtime TypeErrors during Harper startup
kriszyp 37b7a7e
fix: Resolve runtime module resolution failures for harperConfigEnvVars
kriszyp f0035d8
chore: Regenerate package-lock.json to include missing optional nativ…
kriszyp 82eedc6
fix: Resolve ESM module transpilation issues in dataLayer/bulkLoad.js
kriszyp 9aa43fc
fix: Resolve mock mapping failures in unit tests caused by module tra…
kriszyp 5e64361
fix: Resolve test runner failures for mount_hdb and PermissionRespons…
kriszyp 20c2752
Revert back to main's package state
kriszyp 8818e7f
Formatting
kriszyp 91f0ee3
refactor: Convert multiple root utility scripts to TypeScript
kriszyp f25506e
refactor: Convert utility/common_utils.js to TypeScript
kriszyp dc37d4a
fix: Commit dangling TS resolution adjustments for OperationFunctionC…
kriszyp c38cf85
refactor: Convert utility/globalSchema.js to TypeScript
kriszyp 3e7716d
fix: Resolve missing databases.ts compilation output module references
kriszyp e50d36c
refactor: Convert security/role.js to TypeScript
kriszyp fc0569d
refactor: Convert utility/operation_authorization.js to TypeScript
kriszyp 331ddaf
refactor: Convert dataLayer Objects to TypeScript
kriszyp 600355d
fix: Resolve ESM module transpilation export mapping across DataLayer…
kriszyp fc0b2ff
refactor: Convert all remaining root DataLayer modules to TypeScript
kriszyp cf3c8e3
fix: Update explicit .ts requires to .js extensions for ESM compatibi…
kriszyp 683beaa
refactor: Convert core LMDB utility bridge structures and tools to Ty…
kriszyp 62145ed
fix: Resolve missing prototype methods, mock bridges, and undefined d…
kriszyp e3518ac
refactor: Convert root validation endpoints and rulesets to TypeScript
kriszyp d842414
fix: Resolve function name assertions and missing ESM module exports
kriszyp e5f8922
fix: adjust installValidator require logic and other exports to handl…
kriszyp a185955
refactor: Convert foundational logging functions logRotator, readLog,…
kriszyp e5bfd56
refactor: Convert sqlTranslator endpoints to TypeScript
kriszyp 79d2062
refactor: Convert core utility/logging/harper_logger.js to TypeScript
kriszyp 60d1162
refactor: Convert server/jobs to TypeScript and resolve AST translati…
kriszyp 520959e
refactor: Convert upgrade components to TypeScript and adapt associat…
kriszyp 84561c7
refactor: Convert bin/ scripts to TypeScript and fix global logger ex…
kriszyp 32b8faf
fix: resolve SelectValidator not a constructor in search.ts
kriszyp b4fdb2f
fix: resolve SQLSearch not a constructor in search.ts
kriszyp bb7ae9d
chore: remove leftover duplicate .js files in upgrade directory
kriszyp 34a1fdc
fix: safely handle missing system database during keys initialization
kriszyp 5ceced5
fix: resolve harperBridge missing method errors caused by circular Co…
kriszyp 52a6376
fix: resolve papaParse module extension assignment error by explicitl…
kriszyp 938e62e
Merge origin/main
kriszyp 7c6cfdc
Fix callback conversion
kriszyp 20db20b
Merge origin/main
kriszyp 7e9165e
fix(blob): re-apply missing blob cleanup fixes after origin/main merg…
kriszyp 307d871
fix: resolve lint:required errors by cleaning up unused imports and v…
kriszyp 6730e77
Formatting
kriszyp a8b5999
Try to restore package-lock.json
kriszyp 0ca6f49
Some cleanup
kriszyp b62f156
fix(merge): resolve TS typing conflicts and infinite db recursion dur…
kriszyp b0b21ab
chore: run formatter
kriszyp 9a44c07
fix(merge): successfully resolve all origin/main merge conflicts and …
kriszyp 5996e51
test: stabilize unit tests after NodeNext migration
kriszyp 8ada7e8
test: fix keys.test.js mock certificate pollution and state leaks
kriszyp 47a80be
fix: resolve test failures and Bun startup crash from PR #413 TypeScr…
kriszyp 9e69554
fix: address PR review comments - remove debug code, fix test auth flow
kriszyp 72f3348
fix: remove premature getDatabases() calls from test utilities
kriszyp 89fb93c
fix: restore loadedDatabases guard before definedDatabases reset
kriszyp ac0a111
fix: explicitly load system database before overriding test paths in …
kriszyp 3df2946
fix: restore eager session table initialization in auth.ts
kriszyp ec563fa
fix: silence pre-existing lint warnings and fix auth bypass for API t…
kriszyp a9483ba
fix: restore bunServeConfigs export to fix Bun integration test crash
kriszyp dea242a
Merge origin/main into ts-fixes
kriszyp 7057f53
fix: strengthen auth bypass to not depend on IP check in test environ…
kriszyp 22029cf
fix: restore after:'authentication' ordering in REST middleware regis…
kriszyp d217a33
fix: scope auth bypass to no-credential requests only
kriszyp c7dca38
fix(test): restore hdb_certificate after keys test to prevent MQTT TL…
kriszyp 672bd43
fix(imports): correct module extension references for node type-strip…
kriszyp df85073
fix(imports): convert require(.ts) to imports; strip extensions from …
kriszyp 7e8f58a
fix(imports): convert package require() to import in TypeScript modules
kriszyp 172897b
Merge remote-tracking branch 'origin/main' into merge-main-into-ts-fixes
kriszyp a752a7d
fix(imports): revert require→import where unit tests use rewire mocking
kriszyp 5b962b0
fix(tests): align rewire variable names with compiled TS import names
kriszyp 8f02671
fix: revert packageUtils to CJS .js + fix TS build errors
kriszyp be86b60
fix: restore .ts extensions in require() calls referencing TS source
kriszyp 55eaa57
chore: apply prettier formatting
kriszyp 1a81ccc
chore: format subscriptionReplay test
kriszyp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| import { startHTTPThreads } from '../server/threads/socketRouter.js'; | ||
| startHTTPThreads(1); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocker:
no-evalglobally disabled.Disabling
no-evalin the shared lint config means future code introducingeval()will never be flagged — it's a security rule. Similarly,no-dupe-keys(line 28) is a correctness rule: duplicate object keys silently overwrite each other.If these rules are firing on pre-existing code that won't be cleaned up in this PR, prefer targeted inline suppressions (
// oxlint-disable-next-line no-eval) on the specific call sites rather than a global disable that covers all future code too.