chore(deps): bump @d-zero/dealer to 1.13.1 - #325
Merged
Conversation
1.13.1 resets a countdown's tracked start time once its placeholder leaves the display stack. That fixes the crawl progress line freezing at `Opening page… 0s` after a navigation retry reuses the `openPage_<url>` countdown id, and the per-lane interval countdown reading 0 from the second page onward.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
@d-zero/dealerを1.13.0→1.13.1に上げる（5パッケージ + yarn.lock）。1.13.1はDisplayがカウントダウン ID ごとの開始時刻を破棄する経路を持たなかった問題（d-zero-dev/tools#959）を修正している。同じ ID が再登場すると前回の開始時刻を引き継ぎ、経過時間がすでに満了時間を超えているため残り 0 に張り付いていた。crawl 表示への影響
nitpicker から見える症状が2つ直る。
Opening page… 0sが 0 のまま動かない —@d-zero/beholderがopenPage_<url>をカウントダウン ID にしているため、同一 URL のナビゲーションリトライで ID が衝突していたdeal()の<index>_intervalとcrawler.tsの同等コードがレーン番号だけを ID にしているため、そのレーンが2件目のアイテムを処理する時点で衝突していたあわせて、
Displayが保持するカウントダウン ID のMapがクロール中ずっと肥大化していた点も解消される（ユニーク URL 数だけエントリが積み上がっていた）。変更内容
依存バージョンの更新のみ。nitpicker 側のコード変更はない。
@nitpicker/cli@nitpicker/core@nitpicker/crawler@nitpicker/viewer@nitpicker/report-google-sheets検証
yarn build（13 projects）/yarn test（4864 passed, 4 skipped / 689 files）/yarn lint（0 errors, cspell 0 issues）publish 済みの
@d-zero/dealer@1.13.1を使ってdeal()を2アイテム・interval: 300で実行し、レンダリングされたカウントダウン値の列を確認。2件目が満了時間から数え直すことを確認した