Inferred array elements omit Partial evidence applications
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 58/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- javascript, node.js, rust
- Ambito
- compilers, testing-qa
Direzione di ricerca
Start in compiler-frontend/checking/src/source/terms/collections.rs, especially array_core in ArrayMode::Infer, then inspect expression-aware instantiation or subtyping and the evidence handling in forms.rs. Reproduce the issue with the described temporary fixture using just t compiler 1790461440 --verbose and the Node assertion. Done means inferred array elements no longer retain an extra Partial dictionary and the runtime coverage passes without removing the #605 abstractions.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
A runtime regression discovered while independently reviewing #605: inferred array elements retain a dictionary abstraction without applying its evidence. This is distinct from Granite’s duplicate-Partial inline finding on that PR.
Confirmed on PR #605 head ef33df090. The PR is currently open; this report does not claim the regression is present on main or in a released version.
Minimal reproducer
module Main where
data Choice = Chosen Int | Other
fs = [\(Chosen value) -> value]
Iris infers:
fs :: Partial => Array (Choice -> Int)
Against freshly generated output, this Node assertion fails:
import { strictEqual } from "node:assert/strict";
import * as Main from "./output/Main/index.js";
strictEqual(Main.fs({})[0](Main.Chosen(17)), 17);
Expected: 17.
Actual on the PR head:
AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
+ actual - expected
+ [Function: $closure$1]
- 17
The semantic tree exposes the missing application:
fs = \{partialDict} -> [\{partialDict1} -> \(Chosen value) -> value]
There is one external Partial dictionary, but each array element incorrectly still expects another dictionary before accepting its Choice argument.
Verification and isolation
I created a temporary compiler fixture with the source above and a verify.mjs containing the assertion, then ran:
just t compiler 1790461440 --verbose
- PR head: Node verification fails with the function-versus-17 assertion above.
- With only the six added lines in
compiler-frontend/checking/src/source/terms/forms.rsrestored to their pre-fix version: the identical fixture passes (1 passed, no pending snapshots).
This isolates the regression to the new evidence abstractions interacting with array-element elaboration. The PR head was restored afterward. The temporary investigation fixture was removed; the timestamp above records the executed command, not a fixture currently committed to the repository.
Cause and proposed direction
In compiler-frontend/checking/src/source/terms/collections.rs, array_core in ArrayMode::Infer infers each expression, calls type-only unification::subtype, and retains the original expression. Constraint handling at the type level does not add an EvidenceApplication to that expression.
Before #605, an inferred partial lambda lacked its required abstraction and happened to work in this path. After #605 correctly supplies that abstraction, the array path leaves it unapplied.
Investigate expression-aware instantiation/subtyping for inferred array elements and add compiler-fixture runtime coverage. Do not remove the evidence abstractions from #605: that would restore the original unsafePartial case-section crash.
Reference result from purs 0.15.15
Compiled the exact five-line Main.purs above with the official Linux release of purs 0.15.15, with no library dependencies:
purs --version
# 0.15.15
purs compile Main.purs --output output
Compilation succeeds. Its only warning is MissingTypeDeclaration, reporting the same inferred type:
Partial => Array (Choice -> Int)
Actual generated fs from output/Main/index.js:
var fs = function (dictPartial) {
return [ function (v) {
if (v instanceof Chosen) {
return v.value0;
};
throw new Error("Failed pattern match at Main (line 5, column 7 - line 5, column 31): " + [ v.constructor.name ]);
} ];
};
The array element accepts Choice directly; it has no extra dictionary parameter. Executed with Node.js v22.23.2:
node --input-type=module -e '
import { strictEqual } from "node:assert/strict";
import * as Main from "./output/Main/index.js";
const result = Main.fs({})[0](new Main.Chosen(17));
strictEqual(result, 17);
console.log(result);
'
Actual stdout is 17; exit status is 0. The constructor call uses purs’s own representation (new Main.Chosen(17)), rather than Iris’s Main.Chosen(17), so neither runtime receives a value from the other compiler’s representation.
Investigation: https://ampcode.com/threads/T-01a0df1f-526b-71ab-a932-9dac42222d89
- Lingua principale
- Rust
- Stelle
- 117
- Fork
- 11
- Merge medio
- 4h 22m
- PR unite (30g)
- 126
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di purefunctor/purescript-iris
-
bug language-server
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
purefunctor/purescript-iris#552 ·
I maintainer di solito rispondono entro 1 giorno
-
bug language-server
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
purefunctor/purescript-iris#551 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
purefunctor/purescript-iris#613 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
purefunctor/purescript-iris#560 ·
I maintainer di solito rispondono entro 1 giorno
-
tooling
Difficoltà 4/5 3-5 giorni Idoneità per principianti 58/100
purefunctor/purescript-iris#507 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di purefunctor/purescript-iris
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
area: cli bug priority: P2 ready-for-agent
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno