Skip to content

Syntax Highlighter to Janino Scripts #249

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.

Sign up for GitHub

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

Jump to bottom
Open
KAMKEEL wants to merge 325 commits into
base: dev
Choose a base branch
from
Open

Syntax Highlighter to Janino Scripts #249

KAMKEEL wants to merge 325 commits into from

Conversation

@KAMKEEL
Copy link
Owner

@KAMKEEL KAMKEEL commented Jan 9, 2026

No description provided.

@bigguy345
Fixed wrapText not splitting at \n
c1e4e82
@bigguy345
Added external type constructor validation
9aa23ef
@bigguy345
Captured full/type/name offsets of method declarations
b7333bc
@bigguy345
Method declaration error validation + helper classes needed for that
3e21757
@bigguy345
Added error "Cannot return a value from a method with void result type"
6d20f5f
@bigguy345
Added error "method signature already defined in this scope"
e2a5d25
@bigguy345
Refactored error curly underline rendering to its own helper class
534ec7b
@bigguy345
Small package refactor
c2e0bbe
@bigguy345
Expression resolving/type resolution framework with operator parsing
941cc5e
@bigguy345
oops
574c893
@bigguy345
Fixed members of script types being unable to reference their type on…
3a2bcd2 
… creation
@bigguy345
Unified isTypeCompatible across classes
595efb5
@bigguy345
"null" is now a valid assignment/argument type
2ab4812
@bigguy345
Fetched best method overload according to arg types provided
7e25b1c
@bigguy345
resolveExpressionType now accounts for method arg types when resolvin…
790cfae 
…g a method, to fetch the matching overload
@bigguy345
MethodInfo now stores it's java Method
96d12c3
@bigguy345
Proper numeric promotion between java types for fetching the correct …
bc05164 
…overload.

i.ie Math.max(20f,20) checks if the second int can be promoted to the nearest matching overload max(float, float), and fetches it
@bigguy345
Casting completely functional
efd7d96
@bigguy345
Delegated Casting resolution to a helper class
c46d61b
@bigguy345
Deprecated CastExpressionResolver as ExpressionResolver API already h…
c9e50f2 
…as all this built in
@bigguy345
DONT FORGET TO TURN ON!
36e0e11 
 Turned off script error logging
@bigguy345
Added error: "Cannot access non-static field from static context"
0e2375e
@bigguy345
TokenErrorMessage impl
d6ca60b
@bigguy345
Commented out UNDEFINED_VAR hover info declaration
d6b2512
@bigguy345
applyTokenMetadata
bd0044a
@bigguy345
Code optimization (for erorring static access of non static members)
5899186
@bigguy345
Improved static access detection logic
cd506cb
@bigguy345
Fixed class type detection using Character.isUpperCase in most contex…
b84ada3 
…ts instead of more robust resolution
@bigguy345
Fixed class detection patterns only searching for capital case words
91140d8
@bigguy345
Small refactor
8240cba
@bigguy345
Merge branch 'dev' into syntax-highlighter-unified
0d1520d
@bigguy345
Loaded all .d.ts files in assets/modId/api directory of ALL loaded mods
b86c028
@bigguy345
Got bridges working for new JSTypeRegistry domains (.d.ts files of ea…
f1a968d 
…ch loaded mod)
@bigguy345
Allowed for .d.ts patches to be applied and merge into existing types
f43b9ae 
i.e IPlayer.d.ts patch for getDBCPlayer() by npcdbc addon to return IDBCAddon instead of IDBCPlayer
@bigguy345
Bridged .d.ts patch overrides to java side to show in autocomplete/ho…
46b62ee 
…verinfo
@bigguy345
Merge branch 'dev' into syntax-highlighter-unified
39a43a3
@bigguy345
Properly parsed enclosed namespace types like INpcEvent.UpdateEvent a…
df0d46e 
…nd added them to JSTypeRegistry.types
@bigguy345
Added DataScript global definitions i.e. npc, world, event to syntax …
aae2fdb 
…highlighter as global vars
@bigguy345
Added API global engine objects and DataScript global definitions as …
76b6dae 
…JS autocomplete suggestions
@bigguy345
Penalised keyword items in autocomplete suggestions
bd1743d
@bigguy345
Removed redundant fullScreenButton.draw calls, already drawn in GuiSc…
3f204d4 
…riptTextArea
@bigguy345
Moved fullscreen button mouseClicked into GuiScriptTextArea
8fa22e1
@bigguy345
Properly ordered mouse clicked components
ac8228e
@bigguy345
Improved fullscreen mode for legacy GuiScript
4245f5b
@bigguy345
init'd JSTypeRegistry on game load
6f1fc08
@bigguy345
On Autocomplete insertion, set cursor carat inside () if method has p…
720d0dc 
…arameters
@bigguy345
DELETE_LINE, MOVE_LINE_UP and MOVE_LINE_DOWN KeyPresets
940dc01
@bigguy345
Copy with no selection now copies whole line, and pastes to a new line
8b5d3a2
@bigguy345
Atomic UNDO/REDO for all typed text (operates on entire words at once…
1863038 
…, not just characters)
@bigguy345
CTRL + Click on fields/methods/type to go to definition
ce7566e
@bigguy345
forgot this from autocomplete insertion carat between () commit
83a54c1
@bigguy345
Prevented auto pair insertion of '', "", [], () blocks in excluded ra…
ea8cc51 
…nges (comments,strings)
@bigguy345
Improved comment/string excluded range detection for auto-pair insertion
235fd3b
@bigguy345
Added JS overloads to autocomplete properly
1e96a21
@bigguy345
Completely parsed type generics.
5795cc9 
List<IPlayer> list;
list.get(0) NOW returns IPlayer, instead of T or Object.

Nested generics are also handled and functional:
Map<String, List<IPlayer>>

Implemented TokenHoverInfo/Autocomplete rendering and handling for generics as well.
@bigguy345
wtf
19cdf19
@bigguy345
Optimised and refined type generics parsing. Removed bloat and added …
125a799 
…edge case handling
@bigguy345
Fixed visual hover info bugs for generics
969d82d
@bigguy345
Fixed Autocomplete suggestions not working when whitespaces exist bet…
6c5a269 
…ween it and receiver like "player. get <- doesnt work due to WS"
@bigguy345
Lambda impl Phase 1: Track inner callables as nested scopes so parame…
9e04a11 
…ters and locals resolve correctly inside the callable body,
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@KAMKEEL @bigguy345