Brummel
e25580e3a3
Wires the #55-cutover fieldtest into a durable regression net. The
fieldtest exercised the post-cutover ParamMode={Own,Borrow} surface as a
downstream LLM author would and produced 12 fixtures + a report; left as
loose files they would be dead fixtures that drift. This commits them as
a protected property.
crates/ail/tests/cut55_cutover_surface.rs — one table-driven test running
each fixture through `ail check` and asserting accept (exit 0) or reject
at the right pipeline stage. Reject rows key on the bracketed diagnostic
CODE (consume-while-borrowed, borrow-over-value, borrow-return-not-permitted,
surface-parse-error), NOT message text, so the test is decoupled from the
diagnostic-wording improvement tracked separately.
examples/fieldtest/cut55_*.ail — the corpus. cut55_2c is the #58 repro
(now correctly rejected). cut55_2b was misanalysed by the fieldtest as a
consume-while-borrowed reject; in fact bump's recursive param is
(borrow List) so the tail is only borrowed, never consumed into an own
slot — `ail check` correctly accepts it. Renamed to
cut55_2b_borrow_traversal_clean and recast as the #58 false-positive
guard (a borrow-position use of a heap sub-binder must stay accepted).
cut55_1b similarly does not fire over-strict-mode (its recursive call
consumes the tail, so the lint's consume_count==0 premise fails) — comment
corrected; it is a plain accept. cut55_1c is the genuine over-strict-mode
example (accepts exit 0, emits the warning).
docs/specs/0065-eliminate-implicit-mode fieldtest report — moved 2c to
the reject set, added the 2b false-positive-guard section, marked the
double-free spec_gap RESOLVED (it shipped as the #58 fix).
Relates to #55.
21 lines
688 B
Plaintext
21 lines
688 B
Plaintext
; Fieldtest cut55-3b — decision (c), WRONG mode on purpose.
|
|
;
|
|
; An author who reasons "n is only read, never consumed, so borrow it"
|
|
; writes `(borrow (con Int))`. The ledger says this is illegal: borrow on
|
|
; a value type is a check error, own is the only legal mode there. I
|
|
; expect a `borrow-over-value` error that NAMES the param and tells me to
|
|
; use `own`.
|
|
;
|
|
; Expected: ail check -> error [borrow-over-value], exit 1.
|
|
|
|
(module cut55_3b_borrow_over_value
|
|
|
|
(fn double
|
|
(doc "Borrow a value-typed Int — illegal, own is the only legal mode.")
|
|
(type
|
|
(fn-type
|
|
(params (borrow (con Int)))
|
|
(ret (own (con Int)))))
|
|
(params n)
|
|
(body (app + n n))))
|