Carry hosted stale-install and takeover warnings as RunWarning, not serde_json::Value round-tripped through run_warning_of
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Read scan/hosted.rs first, especially run_warning_of and run_redirect_selected_untracked, then trace the warning fields and constructors in scan/hosted/python.rs, scan/hosted/nuget.rs, scan/hosted/takeover.rs, and vlt_heal.rs. Convert these paths to carry RunWarning, mapping RewriteWarning once. Run the named hosted and redirect end to end suites; done when their human and JSON warning output is unchanged and no listed file contains a json! warning literal.
索引モデルが issue の本文から書いたものです。
説明
[agent] Filed by the scheduled architecture audit (audit-cli-tests). Register: register comment.
Kind: refactor. Register ID: T27 (part of T21, the warning sink).
Problem
Several hosted helpers build their warnings as untyped serde_json::json!({"code": .., "detail": ..}) values:
- the gem probe: scan/hosted.rs L300, L309;
- scan/hosted/python.rs L201;``
- scan/hosted/nuget.rs L75;````
- vlt_heal.rs L373, L417;
- scan/hosted/takeover.rs L161, L238, L286, L382, L492, L609, L623. takeover even converts typed core
RewriteWarnings into JSON values.
The caller then parses them back with run_warning_of, scan/hosted.rs L162-L167:`````
RunWarning::new(
w["code"].as_str().unwrap_or_default(),
w["detail"].as_str().unwrap_or_default(),
)
A typo in a key silently becomes an empty code. The fold in run_redirect_selected_untracked (hosted.rs L1863-L1898) juggles four warning shapes at once: RewriteWarning, Value, the go audit's (String, String), and RunWarning.
Proposed change
- Make
StaleInstallOutcome.warningsandtakeover::Finished.warnings(and the vlt heal outcome) aVec<RunWarning>. - Build those warnings with
RunWarning::newat each site. - Map
RewriteWarning→RunWarningonce with aFromimpl. - Delete
run_warning_of.
Size
5 files (scan/hosted.rs, scan/hosted/{python,nuget,takeover}.rs, vlt_heal.rs). Roughly −40 / +20.
Acceptance criteria
--jsonwarnings[]and the human output are unchanged. The e2e hosted suites (e2e_hosted_production.rs,e2e_redirect_*) guard them.- No
json!({"code"warning literal remains in these files.
- 主要言語
- Rust
- スター
- 9
- フォーク
- 2
- 平均マージ
- 20時間 47分
- マージ済み PR（30日）
- 681
環境構築
- 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 日以内に返信
-
arch-audit refactor
難易度 4/5 3〜5日 初心者へのやさしさ 54/100
SocketDev/socket-patch#1993 ·
メンテナーはふだん 1 日以内に返信
-
arch-audit refactor
難易度 3/5 1〜2日 初心者へのやさしさ 66/100
SocketDev/socket-patch#1992 ·
メンテナーはふだん 1 日以内に返信
-
agent:triaged arch-audit priority:p3 refactor
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
SocketDev/socket-patch#1987 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
SocketDev/socket-patch の issue をすべて見る
似ている issue
-
A-linter
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
oxc-project/oxc#28783 ·
メンテナーはふだん 1 日以内に返信
-
Substrait consumer doesn't recognise `shift_left` / `shift_right` from functions_arithmetic.yamlオープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
apache/datafusion#27059 ·
メンテナーはふだん 1 日以内に返信
-
Bump calamine to 0.38: RUSTSEC-2026-0436 (CFB/OLE2 panic and unbounded allocation) fails cargo auditオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
docling-project/docling.rs#979 ·
メンテナーはふだん 1 日以内に返信
-
llm translation
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信