refactor: migrate VariableTracer to JS-X-Ray workspace#511
Conversation
🦋 Changeset detectedLatest commit: 74e4e5f The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
315665b to
c2391a7
Compare
There was a problem hiding this comment.
Pull request overview
This pull request migrates the VariableTracer class from the standalone @nodesecure/tracer workspace into the main @nodesecure/js-x-ray workspace. This refactoring consolidates the codebase by moving variable tracing functionality directly into the primary analysis package, eliminating an external dependency.
Changes:
- Removed the entire
@nodesecure/tracerworkspace (package.json, tsconfig, README, LICENSE, CHANGELOG) - Moved VariableTracer implementation and related utilities into
@nodesecure/js-x-ray/src/ - Migrated all VariableTracer tests to
workspaces/js-x-ray/test/VariableTracer/ - Updated all imports from
@nodesecure/tracerto internal paths - Added comprehensive documentation for VariableTracer functionality
Reviewed changes
Copilot reviewed 38 out of 51 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
workspaces/tracer/* |
Complete removal of the tracer workspace |
workspaces/js-x-ray/src/VariableTracer.ts |
Added with enhanced JSDoc documentation for interfaces |
workspaces/js-x-ray/src/utils/* |
Added utility functions migrated from tracer (isEvilIdentifier, stripNodePrefix, getSubMemberExpressionSegments, makePrefixRemover) |
workspaces/js-x-ray/src/index.ts |
Added VariableTracer exports |
workspaces/js-x-ray/test/VariableTracer/* |
Migrated test files with updated imports |
workspaces/js-x-ray/test/probes/* |
Updated test helper imports from "../utils/index.ts" to "../helpers.ts" |
workspaces/js-x-ray/test/helpers.ts |
Updated import paths to use relative internal paths |
workspaces/js-x-ray/package.json |
Removed @nodesecure/tracer dependency |
workspaces/js-x-ray/docs/VariableTracer.md |
Added comprehensive API documentation |
workspaces/js-x-ray/docs/estree/patterns-french.md |
Added detailed French documentation on ESTree patterns |
package.json |
Removed tracer from workspaces list |
README.md |
Updated documentation links, removed tracer workspace reference |
AGENTS.md |
Removed tracer workspace reference |
.changeset/breezy-canyons-kiss.md |
Added changeset for minor version bump |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
c2391a7 to
74e4e5f
Compare
No description provided.