Skip to content

Variables preserve transformer improvements #552

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
Merged
merged 35 commits into from
Feb 23, 2020
Merged

Variables preserve transformer improvements #552

merged 35 commits into from
Feb 23, 2020

Conversation

sanex3339
Copy link
Member

Fixed conflicts between generated names and names from untouched identifiers from source code.

Fixed #550
Fixed #549

This was referenced Feb 8, 2020
Closed
Closed
@sanex3339
Copy link
Member Author

This is WIP PR, still should optimize a few things

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 98.62% when pulling db908f7 on variable-preserve-improvements into 43b8fc9 on master.

@coveralls
Copy link

coveralls commented Feb 8, 2020 •
edited
Loading

Coverage Status

Coverage decreased (-0.03%) to 98.732% when pulling e1a2424 on variable-preserve-improvements into 43b8fc9 on master.

sanex3339 and others added 24 commits February 9, 2020 01:49
Prevented transformation of object keys in sequence expression that h…
4e716ed 
…as `super` call
Additional improvements of variables preserving
7872278
Added more fixes of identifier names conflicts when using `identifier…
ac259da 
…sPrefix`
Added more tests of identifier names conflicts when using `identifier…
029d595 
…sPrefix`
Improved mangled identifier names generator. Improved variable preser…
002697c 
…ving.
Removed unused NodeGuards code
b95b279
Dependencies update
7ff6721
Added basic tests for mangled lexical scope identifiers generation
eb233be
More stable way to preserve identifiers with scope identifiers traverser
89e8a24
Added more tests
ca1545b
Fix of duplicate identifiers error
2ee1a3b
More runtime tests variants
837e743
Custom nodes refactoring. CustomNodeObfuscator class.
9dde522
Added fixed seed for the first obfuscation pass of the CustomNodeObfu…
a507743 
…scator
Removed getPreservedNames method from AbstractIdentifierNamesGenerator
16a7656
@sanex3339
Merge pull request #555 from javascript-obfuscator/new-mangled-names-…
daba144 
…generator

Improved mangled identifier names generator
Fixed wrong names generation for the custom nodes
dd7c1df
Refactoring of custom nodes: split on custom nodes and custom code he…
51523c0 
…lpers
Fixed errors related to identifiers. Changed self defending helper
a2e9c05
Fixed string array name preserving inside string array helpers
499c2b1
Improved new self defending code
e3f8291
Attempt to fix done() called multiple times error on node 10
65c41c2
Attempt to fix done() called multiple times error on node 10 #2
e900171
Attempt to fix done() called multiple times error on node 10 #3
b2f6549
@sanex3339 sanex3339 mentioned this pull request Feb 18, 2020
Closed
@sanex3339 sanex3339 requested a review from zamotkin February 20, 2020 21:06
@sanex3339 sanex3339 mentioned this pull request Feb 21, 2020
Closed
@sanex3339 sanex3339 merged commit fbab814 into master Feb 23, 2020
@sanex3339
Copy link
Member Author

Finally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zamotkin zamotkin Awaiting requested review from zamotkin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

transformObjectKeys - duplicate variable declaration Duplicate identifier when using identifierNamesGenerator: 'mangled'
2 participants
@sanex3339 @coveralls