Skip to content

enhance(relay-compiler): remove unnecessary stuff and cleanup deps#8013

Merged
ardatan merged 3 commits intomasterfrom
cleanup-deps
Mar 5, 2026
Merged

enhance(relay-compiler): remove unnecessary stuff and cleanup deps#8013
ardatan merged 3 commits intomasterfrom
cleanup-deps

Conversation

@ardatan
Copy link
Copy Markdown
Owner

@ardatan ardatan commented Mar 5, 2026

No description provided.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 5, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: 7cb97d9

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

This PR includes changesets to release 2 packages
Name Type
@ardatan/relay-compiler Major
@graphql-tools/relay-operation-optimizer Patch

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 5, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9dee723f-d84f-4861-822e-1c60554f4121

📥 Commits

Reviewing files that changed from the base of the PR and between 9410e77 and 7cb97d9.

⛔ Files ignored due to path filters (2)
  • package-lock.json is excluded by !**/package-lock.json
  • packages/relay-compiler/relay-compiler.min.js is excluded by !**/*.min.js
📒 Files selected for processing (27)
  • .changeset/@ardatan_relay-compiler-8013-dependencies.md
  • .changeset/funky-bags-matter.md
  • packages/relay-compiler/bin/relay-compiler
  • packages/relay-compiler/lib/codegen/NormalizationCodeGenerator.js
  • packages/relay-compiler/lib/codegen/ReaderCodeGenerator.js
  • packages/relay-compiler/lib/codegen/RelayCodeGenerator.js
  • packages/relay-compiler/lib/codegen/RelayFileWriter.js
  • packages/relay-compiler/lib/codegen/compileRelayArtifacts.js
  • packages/relay-compiler/lib/codegen/writeRelayGeneratedFile.js
  • packages/relay-compiler/lib/core/GraphQLWatchmanClient.js
  • packages/relay-compiler/lib/core/RelayGraphQLEnumsGenerator.js
  • packages/relay-compiler/lib/index.js
  • packages/relay-compiler/lib/language/RelayLanguagePluginInterface.js
  • packages/relay-compiler/lib/language/javascript/FindGraphQLTags.js
  • packages/relay-compiler/lib/language/javascript/RelayFlowGenerator.js
  • packages/relay-compiler/lib/reporters/ConsoleReporter.js
  • packages/relay-compiler/lib/runner/Artifacts.js
  • packages/relay-compiler/lib/transforms/ConnectionTransform.js
  • packages/relay-compiler/lib/transforms/DeclarativeConnectionMutationTransform.js
  • packages/relay-compiler/lib/transforms/FieldHandleTransform.js
  • packages/relay-compiler/lib/transforms/MatchTransform.js
  • packages/relay-compiler/lib/transforms/ReactFlightComponentTransform.js
  • packages/relay-compiler/lib/transforms/RequiredFieldTransform.js
  • packages/relay-compiler/lib/transforms/query-generators/NodeQueryGenerator.js
  • packages/relay-compiler/lib/util/TimeReporter.js
  • packages/relay-compiler/package.json
  • packages/relay-compiler/relay-compiler.js
📝 Walkthrough

Summary by CodeRabbit

Refactor

  • Removed the relay-compiler command-line tool.
  • Removed code generation and Relay IR transformation features, including normalization, reader code generation, and flow type generation.
  • Removed connection and field transformation logic.
  • Streamlined package dependencies by removing unused libraries.

Walkthrough

This pull request removes extensive code from the @ardatan/relay-compiler package, including code generation modules, transform implementations, file writing utilities, and reporter classes. Dependencies on Babel, Watchman, and Relay runtime are also removed, along with the relay-compiler binary entry point and related exports from the main index.

Changes

Cohort / File(s) Summary
Changeset Documentation
.changeset/@ardatan_relay-compiler-8013-dependencies.md, .changeset/funky-bags-matter.md		 Changeset entries documenting the major version bump for @ardatan/relay-compiler and removal of dependencies and compilation features.
Code Generators
packages/relay-compiler/lib/codegen/NormalizationCodeGenerator.js, packages/relay-compiler/lib/codegen/ReaderCodeGenerator.js, packages/relay-compiler/lib/codegen/RelayCodeGenerator.js		 Entire code generation pipeline modules removed, eliminating Relay IR-to-AST conversion capabilities (normalization, reader, and dispatch logic).
Artifact Compilation
packages/relay-compiler/lib/codegen/compileRelayArtifacts.js, packages/relay-compiler/lib/codegen/writeRelayGeneratedFile.js		 Core artifact compilation and file writing functionality removed, including context creation, code generation orchestration, and module serialization.
Language-specific Codegen
packages/relay-compiler/lib/language/javascript/FindGraphQLTags.js, packages/relay-compiler/lib/language/javascript/RelayFlowGenerator.js		 GraphQL tag parsing and Flow type generation modules removed, eliminating schema extension and type annotation capabilities.
Transform Modules
packages/relay-compiler/lib/transforms/ConnectionTransform.js, packages/relay-compiler/lib/transforms/DeclarativeConnectionMutationTransform.js, packages/relay-compiler/lib/transforms/FieldHandleTransform.js, packages/relay-compiler/lib/transforms/MatchTransform.js, packages/relay-compiler/lib/transforms/ReactFlightComponentTransform.js, packages/relay-compiler/lib/transforms/RequiredFieldTransform.js		 Six transform implementations removed, eliminating support for @connection, @stream_connection, @match, @module, mutation directives, field handles, flight components, and @required validation.
Reporter & Utility Modules
packages/relay-compiler/lib/reporters/ConsoleReporter.js, packages/relay-compiler/lib/core/GraphQLWatchmanClient.js, packages/relay-compiler/lib/core/RelayGraphQLEnumsGenerator.js, packages/relay-compiler/lib/util/TimeReporter.js		 Removed console reporting, Watchman file watching client, GraphQL enum generator, and timing utilities.
Import Path Updates
packages/relay-compiler/lib/codegen/RelayFileWriter.js, packages/relay-compiler/lib/runner/Artifacts.js, packages/relay-compiler/lib/transforms/query-generators/NodeQueryGenerator.js		 Three modules updated to import nullthrows from local ../util/nullthrowsOSS instead of external 'nullthrows' package.
API & Dependency Updates
packages/relay-compiler/lib/index.js, packages/relay-compiler/lib/language/RelayLanguagePluginInterface.js, packages/relay-compiler/package.json		 Removed RelayCodeGenerator and TimeReporter exports from main API; removed header comments from language plugin interface; removed 7 dependencies (@babel/generator, @babel/parser, chalk, fb-watchman, nullthrows, relay-runtime, signedsource) and relay-compiler binary entry; added sideEffects and bob fields.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

  • Upcoming Release Changes #7024: Updates consumer packages to depend on a modified @ardatan/relay-compiler version, directly impacted by the substantial API and dependency removals in this PR.

Suggested reviewers

  • enisdenjo
  • n1ru4l

Poem

🐰 Hopping through the codebase with delight,
Removing giants that no longer ignite,
Generators gone, transforms take flight,
A leaner relay-compiler shines so bright! ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cleanup-deps

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
@ardatan/relay-compiler 13.0.0-alpha-20260305154116-7cb97d91ac89f0799c5c12554f46e977ce528bfa npm ↗︎ unpkg ↗︎
@graphql-tools/relay-operation-optimizer 7.1.1-alpha-20260305154116-7cb97d91ac89f0799c5c12554f46e977ce528bfa npm ↗︎ unpkg ↗︎

@ardatan ardatan merged commit 48592e1 into master Mar 5, 2026
14 of 15 checks passed
@ardatan ardatan deleted the cleanup-deps branch March 5, 2026 15:43
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

💻 Website Preview

The latest changes are available as preview in: https://pr-8013.graphql-tools.pages.dev

@github-actions github-actions bot mentioned this pull request Mar 5, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ardatan