Skip to content

Commit

Permalink
Merge branch 'master' into dependabot/npm_and_yarn/micromatch-4.0.8
Browse files Browse the repository at this point in the history
  • Loading branch information
@TwitchBronBron
TwitchBronBron authored Nov 26, 2024
2 parents 6b95e0e + 062faf7 commit 5a27dc1
Show file tree
Hide file tree
Showing 6 changed files with 338 additions and 52 deletions.
57 changes: 57 additions & 0 deletions .github/workflows/create-package.yml
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
name: create-package
on:
pull_request:
types: [labeled, unlabeled, synchronize]
jobs:
create-package:
runs-on: ubuntu-latest
if: contains(github.event.pull_request.labels.*.name, 'create-package')
env:
GH_TOKEN: ${{ github.token }}
steps:
- uses: actions/checkout@master
- uses: actions/setup-node@master
with:
node-version: "16.20.2"
# Get a bot token so the bot's name shows up on all our actions
- name: Get Token From roku-ci-token Application
uses: tibdex/github-app-token@v1
id: generate-token
with:
app_id: ${{ secrets.BOT_APP_ID }}
private_key: ${{ secrets.BOT_PRIVATE_KEY }}
- run: echo "TOKEN=${{ steps.generate-token.outputs.token }}" >> $GITHUB_ENV
- name: Compute variables
run: |
CURRENT_VERSION=$(grep -o '\"version\": *\"[^\"]*\"' package.json | awk -F'\"' '{print $4}')
SANITIZED_BRANCH_NAME=$(echo "$GITHUB_HEAD_REF" | sed 's/[^0-9a-zA-Z-]/-/g')
BUILD_VERSION="$CURRENT_VERSION-$SANITIZED_BRANCH_NAME.$(date +%Y%m%d%H%M%S)"
NPM_PACKAGE_NAME=$(grep -o '\"name\": *\"[^\"]*\"' package.json | awk -F'\"' '{print $4}' | sed -e 's/@//g' -e 's#/#-#g')
ARTIFACT_NAME=$(echo "$NPM_PACKAGE_NAME-$BUILD_VERSION.tgz" | tr '/' '-')
ARTIFACT_URL="${{ github.server_url }}/${{ github.repository }}/releases/download/v0.0.0-packages/${ARTIFACT_NAME}"
echo "BUILD_VERSION=$BUILD_VERSION" >> $GITHUB_ENV
echo "ARTIFACT_URL=$ARTIFACT_URL" >> $GITHUB_ENV
- run: npm ci
- run: npm version "$BUILD_VERSION" --no-git-tag-version
- run: npm pack

# create the release if not exist
- run: gh release create v0.0.0-packages --title "v0.0.0-packages" --latest=false --prerelease --notes "catchall release for temp packages" -R ${{ github.repository }}
continue-on-error: true

# upload this artifact to the "packages" github release
- run: gh release upload v0.0.0-packages *.tgz -R ${{ github.repository }}

- name: Fetch build artifact
uses: actions/github-script@v7
with:
github-token: ${{ env.TOKEN }}
script: |
return github.rest.issues.createComment({
issue_number: context.issue.number,
owner: context.repo.owner,
repo: context.repo.repo,
body: "Hey there! I just built a new temporary npm package based on ${{ github.event.pull_request.head.sha }}. You can download it [here](${{ env.ARTIFACT_URL }}) or install it by running the following command: \n```bash\nnpm install ${{ env.ARTIFACT_URL }}\n```"
});
14 changes: 7 additions & 7 deletions package-lock.json
View file

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
"dependencies": {
"@xml-tools/ast": "^5.0.5",
"@xml-tools/parser": "1.0.10",
"brighterscript": "^0.67.8",
"brighterscript": "^0.68.0",
"del": "6.0.0",
"fs-extra": "9.1.0",
"glob-all": "3.2.1",
Expand Down
168 changes: 129 additions & 39 deletions src/prefixer/File.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,15 @@ import { buildAst } from '@xml-tools/ast';
import type { RopmOptions } from '../util';
import { util } from '../util';
import * as path from 'path';
import type { BrsFile, Position, Program, Range, XmlFile } from 'brighterscript';
import { ParseMode, createVisitor, isCallExpression, isCustomType, isDottedGetExpression, isDottedSetStatement, isIndexedGetExpression, isIndexedSetStatement, WalkMode, util as bsUtil } from 'brighterscript';
import type { BrsFile, Position, Program, Range, XmlFile, NamespaceStatement } from 'brighterscript';
import { ParseMode, createVisitor, isCallExpression, isCustomType, isDottedGetExpression, isDottedSetStatement, isIndexedGetExpression, isIndexedSetStatement, WalkMode, util as bsUtil, isNamespaceStatement, DeclarableTypes } from 'brighterscript';

/**
* List of all declaralbe types in BrighterScript (i.e. the native types), stored in lower case for reference
*/
const nativeTypes = new Set(
DeclarableTypes.map(x => x.toLowerCase())
);

export class File {
constructor(
Expand Down Expand Up @@ -79,7 +86,7 @@ export class File {
endOffset: number;
}>;

public classDeclarations = [] as Array<{
public prefixableDeclarations = [] as Array<{
name: string;
nameOffset: number;
hasNamespace: boolean;
Expand All @@ -94,9 +101,9 @@ export class File {
}>;

/**
* Anywhere that a class is used as a type (like in class `extends` or function parameters)
* Anywhere that a prefixable reference (i.e. class, enum, etc...) is used as a type (like in class `extends` or function parameters)
*/
public classReferences = [] as Array<{
public prefixableReferences = [] as Array<{
fullyQualifiedName: string;
offsetBegin: number;
offsetEnd: number;
Expand Down Expand Up @@ -241,24 +248,54 @@ export class File {
this.findComponentInterfaceFunctions();
this.findComponentFieldOnChangeFunctions();
}

}

private addClassRef(className: string, containingNamespace: string | undefined, range: Range) {
//look up the class. If we can find it, use it
const cls = (this.bscFile as BrsFile).getClassFileLink(className, containingNamespace)?.item;
private tryAddPrefixableRef(options: {
name: string | undefined;
containingNamespace: string | undefined;
range: Range | undefined;
}) {
//skip if we don't have a name or a range
if (!options.name || !options.range) {
return;
}

let fullyQualifiedName: string;
if (cls) {
fullyQualifiedName = bsUtil.getFullyQualifiedClassName(cls.getName(ParseMode.BrighterScript), cls.namespaceName?.getName(ParseMode.BrighterScript));
} else {
fullyQualifiedName = bsUtil.getFullyQualifiedClassName(className, containingNamespace);
//skip if this is a native type, don't prefix it
if (options.name.trim().length > 0 && nativeTypes.has(options.name.toLowerCase())) {
return;
}

this.classReferences.push({
fullyQualifiedName: fullyQualifiedName,
offsetBegin: this.positionToOffset(range.start),
offsetEnd: this.positionToOffset(range.end)
const lowerName = options.name.toLowerCase();
const lowerContainingNamespace = options.containingNamespace?.toLowerCase();

const scopes = this.bscFile.program.getScopesForFile(this.bscFile);
let fullyQualifiedName: string | undefined;

//find the first item in the first scope that has it
for (let scope of scopes) {
const enumLink = scope.getEnumFileLink(lowerName, lowerContainingNamespace);
if (enumLink) {
fullyQualifiedName = enumLink.item.fullName;
break;
}

const link =
scope.getClassFileLink(lowerName, lowerContainingNamespace) ??
scope.getInterfaceFileLink(lowerName, lowerContainingNamespace) ??
scope.getConstFileLink(lowerName, lowerContainingNamespace);
if (link) {
fullyQualifiedName = bsUtil.getFullyQualifiedClassName(
link.item.getName(ParseMode.BrighterScript),
link.item.namespaceName?.getName(ParseMode.BrighterScript)
);
break;
}
}

this.prefixableReferences.push({
fullyQualifiedName: fullyQualifiedName ?? bsUtil.getFullyQualifiedClassName(options.name, options.containingNamespace),
offsetBegin: this.positionToOffset(options.range.start),
offsetEnd: this.positionToOffset(options.range.end)
});
}

Expand Down Expand Up @@ -306,7 +343,7 @@ export class File {
//track class declarations (.bs and .d.bs only)
ClassStatement: (cls) => {
const annotations = cls.annotations ?? [];
this.classDeclarations.push({
this.prefixableDeclarations.push({
name: cls.name.text,
nameOffset: this.positionToOffset(cls.name.range.start),
hasNamespace: !!cls.namespaceName,
Expand All @@ -317,32 +354,84 @@ export class File {
endOffset: this.positionToOffset(cls.end.range.end)
});

if (cls.parentClassName) {
this.addClassRef(
cls.parentClassName.getName(ParseMode.BrighterScript),
cls.namespaceName?.getName(ParseMode.BrighterScript),
cls.parentClassName.range!
);
}
this.tryAddPrefixableRef({
name: cls.parentClassName?.getName(ParseMode.BrighterScript),
containingNamespace: cls.namespaceName?.getName(ParseMode.BrighterScript),
range: cls.parentClassName?.range
});
},
FieldStatement: (node) => {
this.tryAddPrefixableRef({
name: node.type?.text,
containingNamespace: node.findAncestor<NamespaceStatement>(isNamespaceStatement)?.getName(ParseMode.BrighterScript),
range: node.type?.range
});
},
InterfaceFieldStatement: (node) => {
this.tryAddPrefixableRef({
name: node.tokens.type?.text,
containingNamespace: node.findAncestor<NamespaceStatement>(isNamespaceStatement)?.getName(ParseMode.BrighterScript),
range: node.tokens.type?.range
});
},
//track enum declarations (.bs and .d.bs only)
EnumStatement: (node) => {
const annotations = node.annotations ?? [];
this.prefixableDeclarations.push({
name: node.tokens.name.text,
nameOffset: this.positionToOffset(node.tokens.name.range.start),
hasNamespace: !!node.namespaceName,
//Use annotation start position if available, otherwise use class keyword
startOffset: this.positionToOffset(
(annotations?.length > 0 ? annotations[0] : node.tokens.enum).range.start
),
endOffset: this.positionToOffset(node.tokens.endEnum.range.end)
});
},
//track enum declarations (.bs and .d.bs only)
ConstStatement: (node) => {
const annotations = node.annotations ?? [];
this.prefixableDeclarations.push({
name: node.tokens.name.text,
nameOffset: this.positionToOffset(node.tokens.name.range.start),
hasNamespace: !!node.findAncestor(isNamespaceStatement),
//Use annotation start position if available, otherwise use class keyword
startOffset: this.positionToOffset(
(annotations?.length > 0 ? annotations[0] : node.tokens.const).range.start
),
endOffset: this.positionToOffset(node.value.range!.end ?? node.tokens.equals.range.end)
});
},
//track enum declarations (.bs and .d.bs only)
InterfaceStatement: (node) => {
const annotations = node.annotations ?? [];
this.prefixableDeclarations.push({
name: node.tokens.name.text,
nameOffset: this.positionToOffset(node.tokens.name.range.start),
hasNamespace: !!node.findAncestor(isNamespaceStatement),
//Use annotation start position if available, otherwise use class keyword
startOffset: this.positionToOffset(
(annotations?.length > 0 ? annotations[0] : node.tokens.interface).range.start
),
endOffset: this.positionToOffset(node.tokens.endInterface.range.end)
});
},
FunctionExpression: (func) => {
const namespaceName = func.namespaceName?.getName(ParseMode.BrighterScript);
//any parameters containing custom types
for (const param of func.parameters) {
if (isCustomType(param.type)) {
this.addClassRef(
param.type.name,
namespaceName,
param.typeToken!.range
);
}
this.tryAddPrefixableRef({
name: param.typeToken?.text,
containingNamespace: namespaceName,
range: param.typeToken?.range
});
}
if (isCustomType(func.returnType)) {
this.addClassRef(
func.returnType.name,
namespaceName,
func.returnTypeToken!.range
);
this.tryAddPrefixableRef({
name: func.returnTypeToken?.text,
containingNamespace: namespaceName,
range: func.returnTypeToken?.range
});
}
},
FunctionStatement: (func) => {
Expand All @@ -353,7 +442,7 @@ export class File {
hasNamespace: !!func.namespaceName,
//Use annotation start position if available, otherwise use keyword
startOffset: this.positionToOffset(
(annotations?.length > 0 ? annotations[0] : func.func.functionType)!.range.start
(annotations?.length > 0 ? annotations[0] : func.func.functionType)!.range.start as Position
),
endOffset: this.positionToOffset(func.func.end.range.end)
});
Expand Down Expand Up @@ -646,6 +735,7 @@ export class File {
});
}
}

}

export interface Edit {
Expand Down
Loading

0 comments on commit 5a27dc1

Please sign in to comment.