Finish pre-release-0.6.1

EliotVU · EliotVU · commit 525e14e83e09 · 2023-01-29T07:04:53.000+01:00
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -1,7 +1,7 @@
 on:
   push:
     tags:
-      - '*'
+      - 'release-*'
 
 name: Deploy Extension
 jobs:
@@ -11,7 +11,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-          node-version: 12
+          node-version: 16
       - run: npm ci
       - name: Publish to Visual Studio Marketplace
         uses: HaaLeo/publish-vscode-extension@v0
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,12 +1,12 @@
 # UnrealScript Language Service
 
-## 0.6.0
+## 0.6.0 (Jan 26, 2023)
 
 - Implemented [LSP Semantic-Tokens #137](https://github.com/EliotVU/UnrealScript-Language-Service/issues/137) (References to a class will now be highlighted as such even where the tmLanguage cannot determine the identifier's type)
   - ![image](https://user-images.githubusercontent.com/808593/211020346-38724ace-2fbe-4d92-b68c-69640ded824f.png)
 
 - Implemented [LSP Workspace Symbols #148](https://github.com/EliotVU/UnrealScript-Language-Service/issues/148)
-  - ![image](./media/workspaceSymbols.png)
+  - ![image](./docs/media/workspaceSymbols.png)
 
 - Added [UnrealScript snippets #149](https://github.com/EliotVU/UnrealScript-Language-Service/issues/149)
 
@@ -23,7 +23,7 @@
   - Fixed LSP/documentSymbol [VSCode's Sticky scroll feature](https://github.com/EliotVU/UnrealScript-Language-Service/issues/148)
   - Fixed an issue that caused the document transformer to abort when trying to build a property with bad type-grammar (actually usually triggered by use of  macros).
 
-## 0.5.0
+## 0.5.0 (Nov 8, 2021)
 
 - Autocomplete and IntelliSense
   - Has been displaced with the help of a third-party library [c3](https://github.com/mike-lischke/antlr4-c3)
diff --git a/client/src/extension.ts b/client/src/extension.ts
@@ -1,8 +1,6 @@
 import * as path from 'path';
 import { ExtensionContext, workspace } from 'vscode';
-import {
-    LanguageClient, LanguageClientOptions, ServerOptions, TransportKind
-} from 'vscode-languageclient/node';
+import { LanguageClient, LanguageClientOptions, ServerOptions, TransportKind } from 'vscode-languageclient/node';
 
 let client: LanguageClient;
 
@@ -34,7 +32,11 @@ export function activate(context: ExtensionContext) {
 		documentSelector: [{ scheme: 'file', language: 'unrealscript' }],
 		synchronize: {
             configurationSection: 'unrealscript',
-			fileEvents: workspace.createFileSystemWatcher('**/*.{uc,uci}')
+			fileEvents: [
+                workspace.createFileSystemWatcher('**/*.{uc,uci}'),
+                // Let's not watch for upk changes, just u files because those are expected to change often.
+                workspace.createFileSystemWatcher('**/*.{u}')
+            ]
 		},
         diagnosticCollectionName: 'UnrealScript',
 		outputChannelName: 'UnrealScript',
diff --git a/grammars/UCLexer.g4 b/grammars/UCLexer.g4
@@ -2,6 +2,10 @@ lexer grammar UCLexer;
 
 channels { MACRO, COMMENTS_CHANNEL }
 
+@lexer::header {
+    // TODO: Create a map of keyword tokens?
+}
+
 @lexer::members {
 	parensLevel: number = 0;
 	braceLevel: number = 0;
diff --git a/grammars/UCParser.g4 b/grammars/UCParser.g4
@@ -21,16 +21,16 @@ options {
         const token = this._input.get(i);
         return token.type === UCParser.NEWLINE;
     }
+
+    isKeywordToken(token: Token): boolean {
+        return token.type >= UCParser.KW_DEFAULT && token.type < UCParser.ID;
+    }
 }
 
 // Class modifier keywords have been commented out, because we are not using them for parsing.
 identifier
     : ID
-    | keyword
-    ;
-
-keyword
-	: 'default'
+    | { this.isKeywordToken(this.currentToken) }? ('default'
 	| 'self'
 	| 'super'
 	| 'global'
@@ -202,8 +202,8 @@ keyword
 	| 'rng'
     | 'arraycount'
     | 'enumcount'
-    | 'sizeof'
-	;
+    | 'sizeof')
+    ;
 
 // Parses the following possiblities.
 // Package.Class
@@ -594,6 +594,7 @@ structCppText
 	;
 
 // UnrealScriptBug: Anything WHATSOEVER can be written after this closing brace as long as it's on the same line!
+// Skips a C++ block of text: "{ ... | { ... }* }
 exportBlockText
 	: OPEN_BRACE (~(OPEN_BRACE | CLOSE_BRACE)+ | exportBlockText)* CLOSE_BRACE
 	;
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -2,7 +2,7 @@
     "name": "uc",
     "description": "UnrealScript Language support",
     "displayName": "UnrealScript",
-    "version": "0.6.0",
+    "version": "0.6.1",
     "author": {
         "name": "Eliot van Uytfanghe",
         "url": "https://EliotVU.com"
@@ -13,12 +13,10 @@
         "type": "git",
         "url": "https://github.com/EliotVU/UnrealScript-Language-Service"
     },
-    "license": "MIT",
-    "capabilities": {
-        "virtualWorkspaces": {
-            "supported": false
-        }
+    "bugs": {
+        "url": "https://github.com/EliotVU/UnrealScript-Language-Service/issues"
     },
+    "license": "MIT",
     "categories": [
         "Programming Languages",
         "Linters"
@@ -28,10 +26,23 @@
         "UnrealScript",
         "IntelliSense"
     ],
+    "pricing": "Free",
+    "sponsor": {
+        "url": "https://github.com/sponsors/eliotvu"
+    },
+    "icon": "Icon.png",
+    "galleryBanner": {
+        "color": "#1c1a42",
+        "theme": "dark"
+    },
     "engines": {
         "vscode": "^1.74.0"
     },
-    "icon": "Icon.png",
+    "capabilities": {
+        "virtualWorkspaces": {
+            "supported": false
+        }
+    },
     "dependencies": {
         "antlr4ts": "0.5.0-alpha.4",
         "syntaxes": "file:syntaxes"
@@ -137,7 +148,7 @@
                 "unrealscript.checkTypes": {
                     "scope": "window",
                     "type": "boolean",
-                    "description": "Checks and reports if an expression's type is a valid one. e.g. assignments and passed arguments.",
+                    "description": "(Experimental) Checks and reports if an expression's type is a valid one. e.g. assignments and passed arguments. Recommended for laboratory mice.",
                     "default": false
                 },
                 "unrealscript.macroSymbols": {
@@ -210,6 +221,24 @@
                             "extends": "Object"
                         }
                     }
+                },
+                "unrealscript.indexPackageExtensions": {
+                    "scope": "resource",
+                    "type": "array",
+                    "description": "A list of package (uc,upk) extensions to index.",
+                    "default": [
+                        "u",
+                        "upk"
+                    ]
+                },
+                "unrealscript.indexDocumentExtensions": {
+                    "scope": "resource",
+                    "type": "array",
+                    "description": "A list of document (uc) extensions to index.",
+                    "default": [
+                        "uc",
+                        "uci"
+                    ]
                 }
             }
         },
@@ -307,4 +336,4 @@
             }
         ]
     }
-}
+}
diff --git a/server/package-lock.json b/server/package-lock.json
diff --git a/server/package.json b/server/package.json
@@ -5,7 +5,7 @@
 	"author": "Eliot van Uytfanghe",
 	"publisher": "EliotVU",
 	"license": "MIT",
-	"version": "0.6.0",
+	"version": "0.6.1",
 	"repository": {
 		"type": "git",
 		"url": "https://github.com/EliotVU/UnrealScript-Language-Service"
diff --git a/server/src/UC/Parser/Parser.utils.ts b/server/src/UC/Parser/Parser.utils.ts
@@ -1,15 +1,17 @@
-import { ParserRuleContext, Token } from 'antlr4ts';
+import { Parser, ParserRuleContext, Token } from 'antlr4ts';
 
-export function getTokenDebugInfo(token?: Token): string {
+export function getTokenDebugInfo(token: Token | undefined, parser?: Parser): string {
     if (typeof token === 'undefined') {
-        return '';
+        return 'null';
     }
-    return `(${token.line}:${token.charPositionInLine}) "${token.text}"`;
+    const typeTree = parser ? parser.vocabulary.getSymbolicName(token.type) : token.type;
+    return `(${token.line}:${token.charPositionInLine}) [${typeTree}] ${JSON.stringify(token.text)}`;
 }
 
-export function getCtxDebugInfo(ctx?: ParserRuleContext): string {
+export function getCtxDebugInfo(ctx: ParserRuleContext | undefined, parser?: Parser): string {
     if (typeof ctx === 'undefined') {
-        return '';
+        return 'null';
     }
-    return `(${ctx.start.line}:${ctx.start.charPositionInLine}) "${ctx.text}"`;
+    const typeTree = parser ? parser.ruleNames[ctx.ruleIndex] : ctx.ruleIndex;
+    return `(${ctx.start.line}:${ctx.start.charPositionInLine}) [${typeTree}]`;
 }
diff --git a/server/src/UC/Symbols/ISymbol.ts b/server/src/UC/Symbols/ISymbol.ts
@@ -101,4 +101,14 @@ export function getOuter<T extends ISymbol = ISymbol>(symbol: ISymbol, kind: UCS
 
 export function hasNoKind(symbol: { kind: UCNodeKind }): boolean {
     return typeof symbol.kind === 'undefined';
+}
+
+export function getDebugSymbolInfo(symbol?: ISymbol): string {
+    if (typeof symbol === 'undefined') {
+        return 'null';
+    }
+
+    const range = symbol.getRange();
+    const path = symbol.getName().text;
+    return `(${range.start.line + 1}:${range.start.character} - ${range.end.line + 1}:${range.end.character}) [${path}]`;
 }
diff --git a/server/src/UPK/UnrealPackage.ts b/server/src/UPK/UnrealPackage.ts
@@ -0,0 +1,17 @@
+/**
+ * Implementation based on UnrealPackage.cs from https://github.com/EliotVU/Unreal-Library
+ */
+import { UCPackage } from '../UC/Symbols';
+
+export type UnrealPackageSummary = {
+    version: number;
+    licenseeVersion: number;
+}
+
+export class UnrealPackage {
+    summary: UnrealPackageSummary;
+
+    constructor(public rootPackage: UCPackage) {
+
+    }
+}
diff --git a/server/src/completion.ts b/server/src/completion.ts
diff --git a/server/src/server.ts b/server/src/server.ts
diff --git a/server/src/settings.ts b/server/src/settings.ts

Original file line number	Diff line number	Diff line change
`@@ -1,15 +1,17 @@`
`1`		`-import { ParserRuleContext, Token } from 'antlr4ts';`
	`1`	`+import { Parser, ParserRuleContext, Token } from 'antlr4ts';`
`2`	`2`
`3`		`-export function getTokenDebugInfo(token?: Token): string {`
	`3`	`+export function getTokenDebugInfo(token: Token \| undefined, parser?: Parser): string {`
`4`	`4`	`if (typeof token === 'undefined') {`
`5`		`- return '';`
	`5`	`+ return 'null';`
`6`	`6`	`}`
`7`		- return `(${token.line}:${token.charPositionInLine}) "${token.text}"`;
	`7`	`+ const typeTree = parser ? parser.vocabulary.getSymbolicName(token.type) : token.type;`
	`8`	+ return `(${token.line}:${token.charPositionInLine}) [${typeTree}] ${JSON.stringify(token.text)}`;
`8`	`9`	`}`
`9`	`10`
`10`		`-export function getCtxDebugInfo(ctx?: ParserRuleContext): string {`
	`11`	`+export function getCtxDebugInfo(ctx: ParserRuleContext \| undefined, parser?: Parser): string {`
`11`	`12`	`if (typeof ctx === 'undefined') {`
`12`		`- return '';`
	`13`	`+ return 'null';`
`13`	`14`	`}`
`14`		- return `(${ctx.start.line}:${ctx.start.charPositionInLine}) "${ctx.text}"`;
	`15`	`+ const typeTree = parser ? parser.ruleNames[ctx.ruleIndex] : ctx.ruleIndex;`
	`16`	+ return `(${ctx.start.line}:${ctx.start.charPositionInLine}) [${typeTree}]`;
`15`	`17`	`}`