Remove @grpc/proto-loader from @grpc/grpc-js
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 42/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- node.js, typescript
調査の方向性
まず、@grpc/grpc-js の依存関係宣言と、issue で言及されている channelZ 関連の import またはエントリーポイントを調べます。@grpc/proto-loader なしでパッケージがどのようにビルドされ、使用されるかを確認し、その後、サポート対象の grpc-js の使用やバンドリングを壊すことなく依存関係を削除できることを検証します。
索引モデルが issue の本文から書いたものです。
説明
Is your feature request related to a problem?
Right now, the grpc-js package is install the proto-loader package (which I'm 100% sure it doesn't need). I suggest removing the proto-loader package from the grpc-js dependencies.
Describe the solution you'd like
To import channelZ (if someone needs it), grpc-js can use code generation of any of dozens of other packages that create physical files and can work in environments where there is no node_modules folder. In this way, we will fix two problems - unnecessary dependency and work in production environments (where the use of node_modules is prohibited and there are requirements for bundling the application into a single js file)
Describe alternatives you've considered
protobufjs-cli, protobuf-ts, @bufbuild/protobuf and many others.
Additional context
See how many unnecessary dependencies I need to install: https://npmgraph.js.org/?q=%40grpc%2Fgrpc-js. And every such unnecessary dependency is a potential place to attack.
Notes
For me, i can "omit" via overrides:
"@grpc/proto-loader": "npm:dry-uninstall"
- 主要言語
- TypeScript
- スター
- 4.8k
- フォーク
- 717
- 平均マージ
- 1日 18時間
- マージ済み PR(30日)
- 17
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
grpc/grpc-node のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 2 日以内に返信
-
package: @grpc/grpc-js
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
grpc/grpc-node#2993 · コメント 3 件 · リアクション 4 件 ·
メンテナーはふだん 2 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
grpc/grpc-node#3091 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
feature request
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
grpc/grpc-node#3077 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 76/100
grpc/grpc-node#3068 · コメント 2 件 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
似ている issue
-
area/frontend good first issue kind/cooldown
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
voidzero-dev/oxc-angular-compiler#511 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
langchain-ai/deepagentsjs#898 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
anomalyco/models.dev#8509 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
bug documentation P2 UI/UX
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信