goto noRecord; noRecord:

As it currently stands, goto statements are parsed and processed properly, but there is no intellIsense support yet, i.e. "noRecord" is never found.

Describe the solution you'd like

Is it technically possible to add some basic, default snippets here, without creating a standalone snippet extension? Something like these: https://github.com/InsultingPros/LazyKFWiki/blob/main/docs/_content/unrealscript.json

They are really helpful, not mandatory and if you add them, people will be able to easily modify them instead of writing from scratch.

Code blocks of a state's declaration are not yet supported.

When a method's parameter has a default value assignment, the highlighter will fail to detect the ending of said parameter, and thus incorrectly assign the parameter's identifier as a Type.

As seen in this screenshot:
http://prntscr.com/om1sxz

Is your feature request related to a problem? Please describe

Let's say I'm very happy with #148 fixed, and I can build from a fork for myself :d But lot's of people (at least some of my friends) will be lazy to set up a build system just to get the fixed-beta-minor version.

Describe the solution you'd like

VSCode has a pre-release feature, which allows to easily pick the latest, not tested version and check it. And it will allow more people to do it.

/**
 * Removes the values for all configurable properties in this object's class from the .ini file.
 *
 * @param	PropertyName		if specified, only this property's value will be removed.
 */
native static final function StaticClearConfig( optional string PropertyName );
*/

The UC compiler obviously considers this fine, so perhaps downgrade it to a suggestion or something?

Currently any qualified identifier is interpreted as Package.Class.Field or Class.Field.SubField, but depending on the scope the UnrealScript will interpret these differently.

E.g. a qualified identifier as Package.Class is in most cases only valid when extending a class, or implementing a class.

Valid:
class MyObject extends Core.Object;

Invalid:
var Core.Object AnObject;

It appears that when declaring a variable, a qualified identifier will always be interpreted as Class.EnumOrStruct, deeply referencing nested enums or structs are also invalid.

var class<Core.Object> MyMetaClass; is also invalid and thus inconsistent with the extends semantic.

As it currently stands the LSP interprets everything as PackageOrAny?.AnyField?.AnySubField?....

The 'intrinsic' keyword is not recognized as a field modifier. It works fine to use 'intrinsic' instead of 'native' for classes though.

https://puu.sh/GINmQ/2b417829ba.png

Currently, the suggestions are basically non-existent. I wrote WOTCStrategyOverhaul/CovertInfiltration@61b5780 using this and I remember that I had to basically type out everything manually. I'll try to list as many cases as I can.

Keywords

Types

Static calls

(the function is GetCovertActionSquad)

ERROR in ./src/UC/Parser/CommonTokenStreamExt.ts

Module not found: Error: Can't resolve '../../antlr/UCLexer' in '/home/travis/build/EliotVU/UnrealScript-Language-Service/server/src/UC/Parser'

 @ ./src/UC/Parser/CommonTokenStreamExt.ts 6:18-48

 @ ./src/UC/document.ts

 @ ./src/UC/indexer.ts

 @ ./src/server.ts

Travis appears to skip the ANTLR step when building the extension for production.

As shown, a comment on the same line is not being highlighted.

Describe the bug

The server is missing support for qualified object literals that don't have specified a type:

defaultproperties
{
     // Not supported
     Texture=Engine.S_Ammo
     // Supported
     Texture=Texture'Engine.S_Ammo'
}

Screenshots

Describe the bug
The intrinsic Vect function is not being recognized as a valid function input when put directly in the function params.

Screenshots

States with local field declarations are not yet supported.

Generic arguments are not recognized but non-generic (eg. Remove) are fine

There is also a RandomizeOrder function which isn't recognized. I suspect that it's a Firaxis addition since I can find uses only inside XComGame

Right now everything works wonders except for the error squiggles. Since the Intellisense doesn't know how to deal with native functions/classes that don't have declarations in the immediate workspace, nearly every script has errors all over the place. Can we get an option to turn this off?

Describe the solution you'd like

VSCode introduced sticky scroll feature in V1.71.

It's a killer feature when you browse huge functions and especially with several classes opened at the same time.

Aaand it doesn't work your extension which is a unfortunate.

This issue serves as a form to track the progress towards implementing the SemanticTokens feature of the LSP API.

UnrealScript has some quirky special const tokens such as "sizeof", "nameof", and "arraycount". Where "sizeof" in particular can only be used within a const declaration.

const CONST_SIZEOF = sizeof(Object);

const CONST_ARRAYCOUNT = arraycount(staticArray);

const CONST_CLASS = class'Object';

const CONST_VECT = vect(0,0,0);

const CONST_ROT = rot(0,0,0);

const CONST_RNG = rng(0,0);

const CONST_NAMEOF = nameof(staticArray);

Describe the bug

If you have multiple utility classes across different packages with the same name:

Workspace
|-- US Sources of the game
|   `-- ...
|-- MYMod_1
|   |-- ...
|   `-- Settings.uc
`-- MYMod_2
    |-- ...
    `-- Settings.uc

and try to access default variables:

class'Settings'.default.bHidePortraits

extension will show class hints only for one of the packages. For others it will warn variable does not exist on type MYMod_2.Settings.

Screenshots

As described in the title, UE1, and UE2 do coerce the return type to its first parameter class type. However in UE3 this logic is only applied to functions that have "coerce" as a return modifier.

e.g. "function coerce Actor Spawn(class class);"

Describe the bug

After copying files (outside of VSCode) into the workspace's directory they do not get indexed - no definitions are exported from them until VSCode restarts. You can't view the parents, super code or anything else.

Enabling-disabling this option, or editing-saving new files to trigger the reindexing doesn't fix the issue.

After writing a struct on multiple lines in DefaultProperties, everything else below it loses colors. It only happens on 0.5.0.

It doesn't happen when writing it on a single line without the curly brackets.

Seems like the closing bracket causes it, since removing it (aside from causing a syntax error) restores the colors:

References from the Killing Floor 2 SDK cannot be found since they reside somewhere else than the mod folder.

Source files for the the mod packages are located in:
<drive:>\Users\<user>\Documents\My Games\KillingFloor2\KFGame\Src\MyPackage

However, SDK source files are located in:
<drive:>\Steam\SteamApps\common\killingfloor2\development\Src

Hence the missing references.

Source

Describe the bug

After 0.6.0 symbols started to appear in Outline tab. But they behave quite strangely:

• For some classes they just work fine and instantly.
• For others it will warn you no symbols found in document and show nothing. It doesn't depend on a file size - huge hud files can work and small utility classes will bug out.

You can fix it by editing-saving the file or switching to other classes several times.

Screenshots

Not a screenshot but a short video demonstration: https://youtu.be/h22MRGMKCwI

Describe the bug
Object literals that reference asset files emit errors that the internal reference string is a missing type.

Expected behavior
The type lookups should only be done for class and enum object literals. The extension should make no attempt to do sanity checks on the internal reference string for any other object literal.

Screenshots

Please give me the ability to turn off error highlighting. I'm using this extension on unrealscript 1 files and it completely breaks down. All comparison operators are "invalid". Most functions from external packages are considered to not exist.

This extension has most of the necessary highlighting to be fully functional (it still lacks proper highlighting for Class and Name type), however the error highlighting has never worked since the beginning and it can just go with no value loss.

Context:
Extension settings: All 0.4.3 default settings with Unrealscript generation set to 1 (changing the generation DOES NOT fix this).
Workspace: All folders for the package sources are included.

Please, this is a great extension and it helps a lot. It does not need error highlighting, and error highlighting is the only thing that's wrong with this extension, please turn it off.

The current indexer will fail to link the type "delegate", as seen in this line of code:
var array<delegate<CanUnpause> > Pausers;

Hello, thank you for making this extension!
I am tiring to use this extension to write some modes for xcom 2
But can not figure out how to add external definitions for game specific code to the intellisense
Is there are some config option for this?

The UnrealScript grammar currently does not properly support meta data bodies, that have an extra meta tag.

The following leads to a parsing error, stating: "Extraneous input '|'":
var(Display) const interp float DrawScale <UIMin=0.1 | UIMax=4.0>;

Describe the bug

The Language-Server will sometimes hang and stop responding all together, upon further re-search this occurs if the auto-completion parser hits a block like cpptext { code... }, this appears to cause an infinite loop in the third-party library antlr4-c3.

I have submitted an issue for this at antlr4-c3/issues/74.

Little sneak peek:

#34

Before in [0.4.4]:

Now in [0.5.0]:

(Awesome to see a new release by the way!)

Basically assignment operators within a return statement are not being parsed (as intended). However it appears this is valid UnrealScript. Need to investigate its boundaries.

native final function SetPos(float PosX, float PosY, float PosZ=0.0);

Canvas.SetPos(4, out_YPos);

The UC compiler obviously considers this fine, so perhaps downgrade it to a suggestion or something?

E.g.

const A = 0;
function Test() {
       Log(A);
       Log(B);

       const C = 0;
}

function Test2() {
    const B = 0;
    Log(C);
}

Is valid code but currently not supported if a const is declared after an expression, nor is Log(C) valid as C cannot be found.

Describe the bug

The server is missing support for assigning an object using a string literal, and reports the error Type 'String' is not assignable to type 'Object if type checking is enabled.

defaultproperties
{
     // Not supported
     DefaultTexture="EngineResources.WhiteSquareTexture"
     // Supported
     DefaultTexture=Texture2D'EngineResources.WhiteSquareTexture'
}

Screenshots

No response

Reported by BeavisOfArabia

"[Error - 3:47:10 PM] Request textDocument/completion failed.
Message: Request textDocument/completion failed with message: Cannot read properties of undefined (reading 'addChild')
Code: -32603 "

I've tried 0.4.0 and I've been met with endless parsing.

Mod: https://github.com/WOTCStrategyOverhaul/X2WOTCCommunityHighlander/tree/development
I've also added SrcOrig as per your recommendation as "Add folder to workspace"

Output: https://gist.github.com/Xymanek/cb912eb539f3d13c49fd9a3ae7c7b05d Note the following:

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 00007FF7EBA076CF 
 2: 00007FF7EA69CA46 
 3: 00007FF7EA69CC56 
 4: 00007FF7E9883828 
 5: 00007FF7E98837C9 
 6: 00007FF7E98FC110 
 7: 00007FF7E98FAF14 
 8: 00007FF7E98F8AC9 
 9: 00007FF7E99043FB 
10: 00007FF7E99044A9 
11: 00007FF7E951DC4D 
12: 00007FF7E96CFC69 
13: 00007FF7E9AEFDF2 
[Info  - 11:00:03 AM] Connection to server got closed. Server will restart.

This happens in an endless loop. At the point when this happens at least 4-5 GB of RAM is free

As seen above, "case:" is highlighted but "default:" never is!

Github issues are great for tracking and coordinating on long-term tasks but isn't really suitable for generic discussion. I suggest making a room on gitter.im so that we could discuss/coordinate(1) things easier

(1) For example, the question I would like to ask right now is "How far away is the next release? Should I make issues now or wait till next release (since I saw that there were a few commits already)"

I'm having this error repeatedly until server stops restarting. The codebase I'm working on is pretty huge

c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:68
*/Object.defineProperty(e,"__esModule",{value:!0});e.RecognitionException=class extends Error{constructor(t,e,r,n){super(n),this._offendingState=-1,this._recognizer=t,this.input=e,this.ctx=r,t&&(this._offendingState=t.state)}get offendingState(){return this._offendingState}setOffendingState(t){this._offendingState=t}get expectedTokens(){if(this._recognizer)return this._recognizer.atn.getExpectedTokens(this._offendingState,this.ctx)}get context(){return this.ctx}get inputStream(){return this.input}getOffendingToken(t){if(!t||t===this._recognizer)return this.offendingToken}setOffendingToken(t,e){t===this._recognizer&&(this.offendingToken=e)}get recognizer(){return this._recognizer}}},function(t,e,r){"use strict";Object.defineProperty(e,"__esModule",{value:!0});const n=r(278),i={};class s{constructor(t,e){this.hash=t,this.text=e,i[t]=this}toString(){return this.text}}function o(t){const e=n.str(t.toLowerCase());return i[e]||new s(e,t)}

TypeError: e.findSuperSymbol is not a function
at l.index (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:364:5417)
at e.UCQualifiedTypeSymbol.index (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:364:2600)
at e.UCObjectLiteral.index (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:364:21347)
at e.UCDefaultAssignmentExpression.index (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:364:16405)
at e.UCBlock.index (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:600:2504)
at e.DocumentIndexer.visitDefaultPropertiesBlock (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:600:4913)
at e.UCDefaultPropertiesBlock.accept (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:600:20680)
at e.DocumentIndexer.visitStructBase (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:600:4642)
at e.DocumentIndexer.visitClass (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:600:4690)
at e.UCDocumentClassSymbol.accept (c:\Users\mssol.vscode\extensions\eliotvu.uc-0.4.1\server\out\server.js:600:19023)
[Info - 7:32:11 PM] Connection to server got closed. Server will restart.

The usage of macros can lead to severe parsing and indexing issues, like the following code:

return ( `TimeSince( LastRenderTime ) < 0.3 );

The usage of `TimeSince here will tell the lexer that this is a PP command, when this is occurred the input will be thrown out thus leaving the parser with:

return ( < 0.3 );

Which leads to even bigger issues down the road, so as the end block of a method not being terminated.

One workaround for this issue could be to safely parse binary operators that are missing an expression on the left or right, however due the nature of UnrealScript allowance of custom defined pre and post operators, "< 0.3" could in theory be a post operator! Thus it cannot be assumed to be an incomplete binary operator (albeit it would be totally unexpected for "<" symbol, that cannot be said of other symbols).

The proper solution would be not to skip any PP commands and instead fully parse and evaluate any macro prior to parsing the document, this is however not an easy task!