Use core's blob fetch-summary formatter in apply's fetch_stage instead of a drifted copy
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 78/100
Rechercherichtung
Lies crates/socket-patch-cli/src/commands/fetch_stage.rs zusammen mit den Core-Formatierern in crates/socket-patch-core/src/api/blob_fetcher.rs; repair wird als bestehender Aufrufer angeführt. Aktualisiere die Fetch-Stage so, dass sie die Core-Formatierer für Erfolg und Fehler verwendet und dabei ihre Zeile total == 0 beibehält. Aktualisiere anschließend die genannten fetch_stage- und apply_render-Tests. Fertig ist die Änderung, wenn die Erfolgsausgabe unverändert bleibt und die Fehlerausgabe dem sortierten, knappen Format des Core ohne abschließenden Doppelpunkt entspricht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
[agent] Filed by the scheduled architecture audit (audit-cli-tests). Register: register comment.
Kind: refactor. Register ID: T04.
Problem
Two private functions in fetch_stage.rs L77-L128,`` format_fetch_summary and `format_fetch_failures`, copy the public formatters in core: `api/blob_fetcher.rs` L210-L264 (`format_fetch_result_for`, `format_fetch_successes`, `format_fetch_failures`). `repair` already uses the core ones.
The copies have drifted:
| core | fetch_stage | |
|---|---|---|
| failures sorted by hash | yes (results come from a HashSet) |
no, so the order is nondeterministic |
| header | Failed to download N blobs |
Failed to download N blobs: |
| id | noun.display_id (no ... for ids of 12 chars or fewer) |
always chars().take(12) + ... |
| error text | concise_fetch_error |
raw error |
"All blobs are present locally." and the ... and N more cap are written in both copies. The cap is also open-coded in fetch_stage::format_purl_list (L77-L83) and in repair.rs.
Proposed change
- Delete
fetch_stage::format_fetch_summaryandformat_fetch_failures. - Call
blob_fetcher::format_fetch_successes/format_fetch_failureswithArtifactNounblobs, keeping thetotal == 0line.
Size
1 file. About −45 / +5 lines.
Acceptance criteria
applyprints the same success lines.- Failure lines take core's form: sorted, concise, and without the trailing colon.
fetch_stageandapply_rendertests are updated where they pinned the old form.
- Vorherrschende Sprache
- Rust
- Sterne
- 8
- Forks
- 2
- Ø Merge
- 19 Std. 12 Min.
- Gemergte PRs (30 T.)
- 709
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus SocketDev/socket-patch
-
agent:triaged arch-audit priority:p3 refactor
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
SocketDev/socket-patch#1860 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Share one scoped env-var test guard: EnvGuard is copy-pasted 11 times across core unit and integration testsEvtl. vergeben @mikolalysenko hat das heute übernommen. Offenagent:claimed agent:triaged arch-audit priority:p3 refactor
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
SocketDev/socket-patch#1889 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Memoize env-keyed subprocess probes through one helper: composer, python and ruby each define the same env key and drifted on when an answer is keptEvtl. vergeben @mikolalysenko hat das heute übernommen. Offenagent:claimed agent:triaged arch-audit priority:p1 refactor
Schwierigkeit 4/5 1-2 Tage Anfängerfreundlichkeit 25/100
SocketDev/socket-patch#1888 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
agent:triaged arch-audit priority:p3 refactor
Schwierigkeit 4/5 1-2 Tage Anfängerfreundlichkeit 55/100
SocketDev/socket-patch#1887 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
agent:needs-human agent:triaged arch-audit priority:p3 refactor
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100
SocketDev/socket-patch#1881 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in SocketDev/socket-patch
Ähnliche Issues
-
bug priority: low ui
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
storytold/photocraft#10370 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:doc bug in progress needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
bherbruck/solvecraft#9634 · 2 Kommentare · 1 Reaktion ·
-
inflate: total_in uses plain += on c_ulong (32-bit on Windows), panics in debug for streams > 4 GiBOffen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
trifectatechfoundation/zlib-rs#570 · 1 Kommentar ·
-
bug discord-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
LanternOps/breeze#10755 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
aranya-project/aranya#803 ·