javascript-obfuscator / javascript-obfuscator/react-native-obfuscating-transformer

lots of TypeScript errors

Open
#69 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
TypeScript
Stars
211
Forks
102
PR merge metrics
No merged PRs in 30d

Description

I'm experiencing many TypeScript errors (I cropped the cmd output).
As pointed out in https://github.com/microsoft/TypeScript/issues/40426 , you can not disable type checking on *.ts files from packages in node_modules, only on *.d.ts files.
Can the files be changed from *.ts files to *.d.ts files?
```
C:\project>yarn web
override webpack with config-overrides.js
(node:8468) [DEP_WEBPACK_DEV_SERVER_ON_AFTER_SETUP_MIDDLEWARE] DeprecationWarning: 'onAfterSetupMiddleware' option is deprecated. Please use the 'setupMiddlewares' option.
(Use `node --trace-deprecation ...` to show where the warning was created)
(node:8468) [DEP_WEBPACK_DEV_SERVER_ON_BEFORE_SETUP_MIDDLEWARE] DeprecationWarning: 'onBeforeSetupMiddleware' option is deprecated. Please use the 'setupMiddlewares' option.
Starting the development server...
Failed to compile.

[eslint] Plugin "react" was conflicted between ".eslintrc.js » @react-native-community/eslint-config" and "BaseConfig » C:\project\node_modules\react-scripts\node_modules\eslint-config-react-app\base.js".
ERROR in [eslint] Plugin "react" was conflicted between ".eslintrc.js » @react-native-community/eslint-config" and "BaseConfig » C:\project\node_modules\react-scripts\node_modules\eslint-config-react-app\base.js".

webpack compiled with 1 error
ERROR in node_modules/javascript-obfuscator/src/analyzers/stack-trace-analyzer/callee-data-extractors/AbstractCalleeDataExtractor.ts:9:23
TS1219: Experimental support for decorators is a feature that is subject to change in a future release. Set the 'experimentalDecorators' option in your 'tsconfig' or 'jsconfig' to remove this warning.
7 |
8 | @injectable()
> 9 | export abstract class AbstractCalleeDataExtractor implements ICalleeDataExtractor {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
10 | /**
11 | * @param {Node[]} blockScopeBody
12 | * @param {Node} callee

ERROR in node_modules/javascript-obfuscator/src/analyzers/stack-trace-analyzer/callee-data-extractors/FunctionDeclarationCalleeDataExtractor.ts:13:14
TS1219: Experimental support for decorators is a feature that is subject to change in a future release. Set the 'experimentalDecorators' option in your 'tsconfig' or 'jsconfig' to remove this warning.
11 |
12 | @injectable()
> 13 | export class FunctionDeclarationCalleeDataExtractor extends AbstractCalleeDataExtractor {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
14 | /**
15 | * @param {NodeGuards[]} blockScopeBody
16 | * @param {Identifier} callee

ERROR in node_modules/javascript-obfuscator/src/analyzers/stack-trace-analyzer/callee-data-extractors/FunctionDeclarationCalleeDataExtractor.ts:49:67
TS18047: 'node.id' is possibly 'null'.
47 | estraverse.traverse(targetNode, {
48 | enter: (node: ESTree.Node): any => {
> 49 | if (NodeGuards.isFunctionDeclarationNode(node) && node.id.name === name) {
| ^^^^^^^
50 | calleeBlockStatement = node.body;
51 |
52 | return estraverse.VisitorOption.Break;

ERROR in node_modules/javascript-obfuscator/src/analyzers/stack-trace-analyzer/callee-data-extractors/FunctionExpressionCalleeDataExtractor.ts:13:14
TS1219: Experimental support for decorators is a feature that is subject to change in a future release. Set the 'experimentalDecorators' option in your 'tsconfig' or 'jsconfig' to remove this warning.
11 |
12 | @injectable()
> 13 | export class FunctionExpressionCalleeDataExtractor extends AbstractCalleeDataExtractor {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
14 | /**
15 | * @param {NodeGuards[]} blockScopeBody
16 | * @param {Identifier} callee

ERROR in node_modules/javascript-obfuscator/src/analyzers/stack-trace-analyzer/callee-data-extractors/FunctionExpressionCalleeDataExtractor.ts:30:43
TS2339: Property 'body' does not exist on type 'never'.
The intersection 'Identifier & FunctionExpression' was reduced to 'never' because property 'type' has conflicting types in some constituents.
28 |
29 | if (NodeGuards.isFunctionExpressionNode(callee)) {
> 30 | calleeBlockStatement = callee.body;
| ^^^^
31 | }
32 |
33 | if (!calleeBlockStatement) {

ERROR in node_modules/javascript-obfuscator/src/analyzers/stack-trace-analyzer/callee-data-extractors/ObjectExpressionCalleeDataExtractor.ts:15:14
TS1219: Experimental support for decorators is a feature that is subject to change in a future release. Set the 'experimentalDecorators' option in your 'tsconfig' or 'jsconfig' to remove this warning.
13 |
14 | @injectable()
> 15 | export class ObjectExpressionCalleeDataExtractor extends AbstractCalleeDataExtractor {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
16 | /**
17 | * @param {NodeGuards[]} blockScopeBody
18 | * @param {MemberExpression} callee
...
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Reproduce the failure with `yarn web`, separating the ESLint plugin conflict from the TypeScript diagnostics. Inspect the reported `node_modules/javascript-obfuscator/src/...` files alongside `tsconfig` or `jsconfig`; done means the dependency errors no longer appear during compilation.

Written by the indexing model from the issue text.

Assessment

Tech stack
react-native, typescript
Domain
build-system
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.