IntersectMBO/plutus

Data.AssocMap and Data.Value should use Data.List

Open

#6.933 geöffnet am 10. März 2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Haskell (1.637 Stars) (511 Forks)batch import
Good first issueInternalPlinthPlutus Ledger APIstatus: triaged

Beschreibung

After https://github.com/IntersectMBO/plutus/pull/6931 is merged, you can search for uses of toSOPList or unsafeFromSOPList and you'll see that lots of code in the ledger API or the standard library does conversions to and from SOP lists which is not ideal. This is because when we implemented Data-backed Map and Value, we didn't have a Data-backed List yet. We should refactor all of these cases where a Data-backed List would be more appropriate.

Contributor Guide

Tech Stack
Keine
Domain
data
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
blocked
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
PR #6931 mergedKnowledge of Plutus Data typesFamiliarity with ASSOCMAP and VALUE modulesUnderstanding of SOP lists
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
Search for uses of `toSOPList` and `unsafeFromSOPList` in the ledger API and standard library. After PR #6931 is merged, refactor each occurrence to use Data backed List types where appropriate. Check the Plutus repository's Data module for available list types. Ensure that the refactoring passes all existing tests.