Skip to content

refactor: migrate VariableTracer to JS-X-Ray workspace#511

Merged
fraxken merged 1 commit intomasterfrom
remove-tracer-ws
Feb 2, 2026
Merged

refactor: migrate VariableTracer to JS-X-Ray workspace#511
fraxken merged 1 commit intomasterfrom
remove-tracer-ws

Conversation

@fraxken
Copy link
Member

@fraxken fraxken commented Feb 2, 2026

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Feb 2, 2026

🦋 Changeset detected

Latest commit: 74e4e5f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@nodesecure/js-x-ray Minor

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/tracer workspace (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/tracer to 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.

@fraxken fraxken merged commit cd46792 into master Feb 2, 2026
8 checks passed
@fraxken fraxken deleted the remove-tracer-ws branch February 2, 2026 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants