Inferred array elements omit Partial evidence applications
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 58/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, node.js, rust
- Área
- compilers, testing-qa
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Rust
- Estrellas
- 123
- Forks
- 12
- Merge medio
- 7 h 34 min
- PR fusionados (30 d)
- 89
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de purefunctor/purescript-iris
-
diagnostics enhancement language-server
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
purefunctor/purescript-iris#648 ·
Los mantenedores suelen responder en 1 día
-
ideas
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
purefunctor/purescript-iris#620 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
purefunctor/purescript-iris#613 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
purefunctor/purescript-iris#560 ·
Los mantenedores suelen responder en 1 día
-
tooling
Dificultad 4/5 3-5 días Aptitud para principiantes 58/100
purefunctor/purescript-iris#507 ·
Los mantenedores suelen responder en 1 día
Todos los issues de purefunctor/purescript-iris
Issues similares
-
bug llm translation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
accounts bug decoding
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
wboayue/rust-ibapi#1088 ·
Los mantenedores suelen responder en 1 día
-
chmod -v: missing 'could not be accessed' line for operands that can't be stat'edPosiblemente ocupada @emirmrkaljevic la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
gtk-rs/gtk-rs-core#2068 ·
Los mantenedores suelen responder en 6 días