Nightly e2e_bun_lockb red: offline takeover test predates #1648
メンテナーはふだん 1 日以内に返信
関連するプルリクエストがすでにマージされています。
- #1771 @mikolalysenko による — マージ済み
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 35/100
調査の方向性
Start at crates/socket-patch-cli/tests/e2e_bun_lockb.rs:928 in native_binary_hosted_vendored_takeover_roundtrip, where the assertion still expects redirect_revert_failed. Run that test with bun 1.0.36 on main to see vendor --offline --dry-run exit 1 with the top-level offline_eject_unavailable error, then update the step to assert that error, exit code 1, and no files written. Check the remaining roundtrip assertions still hold. Note PR #1771 is already open against this, so coordinate before starting.
索引モデルが issue の本文から書いたものです。
説明
[agent] The nightly e2e-extended legs e2e_bun_lockb (bun 1.0.36 and 1.1.45) have been red on main since #1648 merged.
Evidence
- Run https://github.com/SocketDev/socket-patch/actions/runs/40639505834 (schedule, main
b787454, 2026-10-28), jobs 122654524514 and 122654524565. native_binary_hosted_vendored_takeover_roundtrippanics atcrates/socket-patch-cli/tests/e2e_bun_lockb.rs:928:expected redirect_revert_failed, butvendor --offline --dry-runnow returns the top-level erroroffline_eject_unavailable("ejecting 1 hosted package needs 1 patch record(s) from the Socket API, and this run is offline; re-run without --offline").- Reproduces locally with bun 1.0.36 on
origin/main.
Cause
#1648 (vendor hosted patches directly from lockfiles) made vendor --offline refuse up front whenever a hosted pin is in scope, which its description records as the chosen behaviour. This test still expects the older per-pin redirect_revert_failed event. It only runs on the bun 1.0/1.1 legs, which are nightly-only, so the PR and merge-queue runs never exercised it.
Fix
Update the test's offline-refusal step to the new contract (exit 1, offline_eject_unavailable, nothing written), and check that the rest of the roundtrip still holds.
Generated by Claude Code
- 主要言語
- Rust
- スター
- 8
- フォーク
- 2
- 平均マージ
- 19時間 24分
- マージ済み PR（30日）
- 669
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
SocketDev/socket-patch のほかの issue
-
agent:triaged arch-audit priority:p3 refactor
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
SocketDev/socket-patch#1860 ·
メンテナーはふだん 1 日以内に返信
-
agent:triaged arch-audit priority:p3 refactor
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
SocketDev/socket-patch#1838 ·
メンテナーはふだん 1 日以内に返信
-
Vendored npm rollback / remove / vendor --revert on a lockfileVersion 2 lock fail after any `npm install`, yet half-revert the lock: `packages` goes back to the registry while the legacy mirror keeps the patched integrity, and npm 6 then fails EINTEGRITY対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbug bughunt pm:npm
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
SocketDev/socket-patch#1937 ·
メンテナーはふだん 1 日以内に返信
-
arch-audit refactor
難易度 4/5 3〜5日 初心者へのやさしさ 22/100
SocketDev/socket-patch#1935 ·
メンテナーはふだん 1 日以内に返信
-
arch-audit bug
難易度 3/5 半日 初心者へのやさしさ 55/100
SocketDev/socket-patch#1934 ·
メンテナーはふだん 1 日以内に返信
SocketDev/socket-patch の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
longbridge/nsh#804 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
devex good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
calcit-lang/calcit#2538 ·
メンテナーはふだん 1 日以内に返信