CI Build issue patch - 'patch' is possibly 'undefined'.
メンテナーはふだん 4 日以内に返信
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 25/100
- issue の種類
- バグ
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- typescript
- 領域
- build-system, ci-cd
調査の方向性
immer/src/core/immerClass.ts:181 で報告されている pnpm のビルド失敗と、パッチに示されている immer/src/utils/common.ts の関連行から始めてください。この依存関係がリポジトリでどのように扱われているかを確認し、依存関係レベルの修正を行った後にビルドを検証してください。pnpm build が報告された TypeScript エラーなしで完了すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Hi! 👋
I was getting following error while doing build
./node_modules/.pnpm/[email protected]/node_modules/immer/src/core/immerClass.ts:181:8
Type error: 'patch' is possibly 'undefined'.
179 | for (i = patches.length - 1; i >= 0; i--) {
180 | const patch = patches[i]
> 181 | if (patch.path.length === 0 && patch.op === "replace") {
| ^
182 | base = patch.value
183 | break
184 | }
ELIFECYCLE Command failed with exit code 1.
Error: Command "pnpm build" exited with 1
Here is the diff that solved my problem:
diff --git a/node_modules/immer/src/core/immerClass.ts b/node_modules/immer/src/core/immerClass.ts
index f827361..37bb8c9 100644
--- a/node_modules/immer/src/core/immerClass.ts
+++ b/node_modules/immer/src/core/immerClass.ts
@@ -1,30 +1,30 @@
import {
- IProduceWithPatches,
+ DRAFT_STATE,
+ Draft,
+ Drafted,
IProduce,
+ IProduceWithPatches,
ImmerState,
- Drafted,
- isDraftable,
- processResult,
- Patch,
+ NOTHING,
Objectish,
- DRAFT_STATE,
- Draft,
+ Patch,
PatchListener,
- isDraft,
- isMap,
- isSet,
createProxyProxy,
- getPlugin,
+ current,
die,
enterScope,
- revokeScope,
- leaveScope,
- usePatchesInScope,
- getCurrentScope,
- NOTHING,
freeze,
- current
-} from "../internal"
+ getCurrentScope,
+ getPlugin,
+ isDraft,
+ isDraftable,
+ isMap,
+ isSet,
+ leaveScope,
+ processResult,
+ revokeScope,
+ usePatchesInScope
+} from "../internal";
interface ProducersFns {
produce: IProduce
@@ -178,7 +178,7 @@ export class Immer implements ProducersFns {
let i: number
for (i = patches.length - 1; i >= 0; i--) {
const patch = patches[i]
- if (patch.path.length === 0 && patch.op === "replace") {
+ if (patch?.path?.length === 0 && patch?.op === "replace") {
base = patch.value
break
}
diff --git a/node_modules/immer/src/utils/common.ts b/node_modules/immer/src/utils/common.ts
index 692157b..8db10ba 100644
--- a/node_modules/immer/src/utils/common.ts
+++ b/node_modules/immer/src/utils/common.ts
@@ -1,14 +1,14 @@
import {
- DRAFT_STATE,
- DRAFTABLE,
- Objectish,
- Drafted,
- AnyObject,
AnyMap,
+ AnyObject,
AnySet,
- ImmerState,
ArchType,
die,
+ DRAFT_STATE,
+ DRAFTABLE,
+ Drafted,
+ ImmerState,
+ Objectish,
StrictMode
} from "../internal"
@@ -160,14 +160,14 @@ export function shallowCopy(base: any, strict: StrictMode) {
for (let i = 0; i < keys.length; i++) {
const key: any = keys[i]
const desc = descriptors[key]
- if (desc.writable === false) {
+ if (desc?.writable === false) {
desc.writable = true
desc.configurable = true
}
// like object.assign, we will read any _own_, get/set accessors. This helps in dealing
// with libraries that trap values, like mobx or vue
// unlike object.assign, non-enumerables will be copied as well
- if (desc.get || desc.set)
+ if (desc?.get || desc?.set)
descriptors[key] = {
configurable: true,
writable: true, // could live with !!desc.set as well here...
This issue body was partially generated by patch-package.
- 主要言語
- JavaScript
- スター
- 29k
- フォーク
- 905
- 平均マージ
- 3日 16時間
- マージ済み PR（30日）
- 22
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
immerjs/immer のほかの issue
-
Shrinking an array with `length =` and growing it back loses the patches for the truncated indices対応中かも @RClaudiuM が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 4 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
メンテナーはふだん 4 日以内に返信
-
Docs: Chinese (zh-CN) website translation is outdated vs English docs再び着手できるかも @lijiayou0728 が 91 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
immerjs/immer#1242 · コメント 5 件 ·
メンテナーはふだん 4 日以内に返信
-
question
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
メンテナーはふだん 4 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
immerjs/immer#1217 · コメント 1 件 ·
メンテナーはふだん 4 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Automattic/node-canvas#2633 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
capricorn86/happy-dom#2594 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
microsoft/monaco-editor#5575 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信