eyeprolog 1.1.19 → 1.1.20

package/conformance-report.md

@@ -11,7 +11,7 @@ This report summarizes the file-based conformance corpus under `test/conformance
 | builtins | 11 | 0 | 0 | 0 | 11 |
 | context | 11 | 0 | 0 | 0 | 11 |
 | control | 15 | 0 | 0 | 0 | 15 |
-| iso | 164 | 209 | 0 | 0 | 373 |
+| iso | 165 | 209 | 0 | 0 | 374 |
 | lists | 52 | 3 | 0 | 0 | 55 |
 | modules | 2 | 0 | 0 | 0 | 2 |
 | negation | 8 | 0 | 19 | 0 | 27 |
@@ -23,4 +23,4 @@ This report summarizes the file-based conformance corpus under `test/conformance
 | terms | 26 | 3 | 0 | 0 | 29 |
 | unification | 18 | 0 | 0 | 0 | 18 |
 | variables | 16 | 9 | 0 | 0 | 25 |
-| **Total** | **479** | **260** | **19** | **21** | **779** |
+| **Total** | **480** | **260** | **19** | **21** | **780** |

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "1.1.19",
+  "version": "1.1.20",
   "description": "EyeProlog turns facts and rules into answers and proofs.",
   "type": "module",
   "main": "./index.js",

package/src/iso.js

@@ -1814,7 +1814,7 @@ function* catchBuiltin({ solver, goal, env }) {
     const ball = error instanceof ThrownTerm
       ? error.term
       : error instanceof PrologError
-        ? formalErrorTerm(error)
+        ? freshCopy(formalErrorTerm(error), new Env())
         : null;
     if (ball == null) throw error;
     const recovered = env.clone();
@@ -1827,7 +1827,9 @@ function* catchBuiltin({ solver, goal, env }) {
 function* throwBuiltin({ goal, env }) {
   const ball = deref(goal.args[0], env);
   if (ball.type === VAR) throw new PrologError('instantiation_error');
-  throw new ThrownTerm(copyResolved(ball, env));
+  // ISO throw/1 copies the thrown term before control unwinds. Freshen
+  // variables so the catcher cannot retain aliases to the protected goal.
+  throw new ThrownTerm(freshCopy(ball, env));
 }
 function* onceBuiltin({ solver, goal, env }) {
   const child = solver.cloneForInnerGoal(1);

package/src/repl.js

@@ -57,7 +57,7 @@ export async function runRepl(engine, options = {}) {
           exitCode = error.code;
           break;
         }
-        output.write(`   ${formatError(error)}\n`);
+        output.write(`   ${formatError(engine, state, error)}\n`);
       }
     }
   } catch (error) {
@@ -368,13 +368,15 @@ function queryVariables(goal) {
 
 function formatAnswer(engine, state, variables, env) {
   const bindings = [];
+  const queryVariableNames = new Set(variables.map((variable) => variable.name));
   const names = new Map(variables.map((variable) => [variable.name, variable.name]));
   let generated = 0;
 
   for (const variable of variables) collectUnboundVariables(engine, variable, env, names, () => `_${letterName(generated++)}`);
   for (const variable of variables) {
     const value = engine.deref(variable, env);
-    if (value.type === 'var' && value.name === variable.name) continue;
+    if (value.type === 'var' &&
+        (value.name === variable.name || !queryVariableNames.has(value.name))) continue;
     bindings.push(`${variable.name} = ${engine.formatTermForWrite(value, env, {
       quoted: true,
       operators: [...state.program.operators.values()],
@@ -406,7 +408,21 @@ function letterName(index) {
   return suffix === 0 ? letter : `${letter}${suffix}`;
 }
 
-function formatError(error) {
+function formatError(engine, state, error) {
+  if (error?.name === 'PrologError') {
+    const env = new engine.Env();
+    const variableNames = new Map();
+    let generated = 0;
+    if (error.culprit != null) {
+      collectUnboundVariables(engine, error.culprit, env, variableNames, () => `_${letterName(generated++)}`);
+    }
+    const culprit = error.culprit == null ? '' : `, ${engine.formatTermForWrite(error.culprit, env, {
+      quoted: true,
+      operators: [...state.program.operators.values()],
+      variableNames,
+    })}`;
+    return `error(${error.formal}${culprit}).`;
+  }
   const message = error?.message ?? String(error);
   return message.endsWith('.') ? message : `${message}.`;
 }

package/test/conformance/ISO-MATRIX.md

@@ -9,7 +9,7 @@ case totals.
 | --- | --- | --- |
 | Clause 6 lexical and term syntax | tokenizer, operator parser, lists, curly terms, quotes, numeric syntax, comments | `scryer_lexical_terms`, `lexical_and_curly_terms`, `double_quoted_lists`, `corrigendum1_double_quote_operator`, syntax error cases |
 | Clause 7 term order and unification | finite-tree unification, identity, standard order, errors | `unification_control_information`, `swipl_occurs_check`, `term_modes_and_ordering`, `logtalk_compare_standard_order` |
-| Clause 7 control and exceptions | call, cut, conjunction, disjunction, if-then-else, catch and throw | `cut_control`, `control_and_terms`, `exceptions_and_flags`, `corrigenda_catch_callability` |
+| Clause 7 control and exceptions | call, cut, conjunction, disjunction, if-then-else, catch and throw | `cut_control`, `control_and_terms`, `exceptions_and_flags`, `corrigenda_catch_callability`, `throw_copies_ball` |
 | 8.2-8.5 term predicates | unification, Corrigendum 2 tests, comparison, sorting, creation and decomposition | `corrigenda_term_predicates`, `corrigenda_sort_keysort`, `logtalk_arg_unification`, `logtalk_univ`, associated error cases |
 | 8.6-8.7 arithmetic predicates | `is/2` and all six arithmetic comparisons | `arithmetic`, comparison cases, isolated evaluation errors |
 | 8.8-8.10 database and solutions | logical update view, dynamic mutation, all-solutions grouping | `dynamic_database`, `trealla_logical_update_view`, `corrigenda_retractall`, `grouped_solutions_and_clauses` |

package/test/conformance/README.md

@@ -94,7 +94,7 @@ Selected cases are adapted from the ISO and standard-core suites of Logtalk,
 Scryer Prolog, Trealla Prolog, and SWI-Prolog. Their upstream identifiers and licenses
 are recorded in [THIRD_PARTY.md](THIRD_PARTY.md).
 
-The corpus has 373 cases in `iso/` and 779 file-based conformance cases in
+The corpus has 374 cases in `iso/` and 780 file-based conformance cases in
 total. The generated `conformance-report.md` is the authoritative source for
 current category totals. Together with regression, documentation-sync, API,
 example, and book-example checks, `npm test` is the release gate.

package/test/conformance/cases/iso/throw_copies_ball.pl

@@ -0,0 +1,17 @@
+% ISO 7.12.1: throw/1 copies the ball before searching for a catcher.
+%% goal: user_throw_copy
+%% goal: error_throw_copy
+
+user_throw_copy :-
+    catch(throw(g(X)), g(V), true),
+    X = protected,
+    V = caught.
+
+error_throw_copy :-
+    catch(
+        number_chars(_, [1, [], X|2]),
+        error(type_error(list, Culprit), _),
+        true
+    ),
+    X = protected,
+    Culprit = [1, [], caught|2].

package/test/conformance/expected/iso/throw_copies_ball.pl

@@ -0,0 +1,2 @@
+user_throw_copy.
+error_throw_copy.

package/test/run-regression.mjs

@@ -496,6 +496,32 @@ c4 ?- call((!;1)).
         assertEqual(result.stderr, '', 'stderr');
       },
     },
+    {
+      name: 'REPL hides aliases to fresh throw variables while preserving query aliases',
+      run: () => {
+        const result = runCli([], {
+          input: 'catch(throw(g(X)),g(V),true).\nX = Y.\nhalt.\n',
+        });
+        assertEqual(result.status, 0, 'exit status');
+        assertEqual(result.stdout, '?-    true.\n?-    X = Y.\n?- ', 'stdout');
+        assertEqual(result.stderr, '', 'stderr');
+      },
+    },
+    {
+      name: 'REPL freshens variables displayed in uncaught ISO errors',
+      run: () => {
+        const result = runCli([], {
+          input: 'number_chars(V,[1,[],X|2]).\nnumber_chars(V,[1,[],Xx|2]).\nhalt.\n',
+        });
+        assertEqual(result.status, 0, 'exit status');
+        assertEqual(result.stdout,
+          '?-    error(type_error(list), [1, [], _A | 2]).\n' +
+          '?-    error(type_error(list), [1, [], _A | 2]).\n' +
+          '?- ',
+          'stdout');
+        assertEqual(result.stderr, '', 'stderr');
+      },
+    },
     {
       name: 'REPL enumerates and stops answers like the Scryer top level',
       run: () => {

package/the-art-of-eyeprolog.md

@@ -5749,8 +5749,14 @@ instances whose message contains the corresponding Prolog error term.
 | *syntax_error(number)*, *syntax_error(read_term)* | Lexical number conversion or streamed term parsing failed. |
 
 `catch/3` converts a `PrologError` into a catchable
-`error(Formal,eyeprolog)` term. A user ball from `throw/1` is delivered as-is.
-An unmatched ball or error continues outward.
+`error(Formal,eyeprolog)` term. `throw/1` copies its ball before unwinding: bound
+parts are preserved, repeated variables remain shared within the copied ball,
+and unbound variables are fresh with respect to the protected goal and catcher.
+Catchable error terms follow the same variable-freshening rule. At the interactive
+top level, variables that occur in an uncaught ISO error are rendered with fresh
+answer names such as `_A` rather than reusing query-variable spellings such as
+`X` or `Xx`; this keeps the displayed error consistent with the copied exception
+term. An unmatched ball or error continues outward.
 
 Streams belong to one solver run and are shared by nested calls, exceptions,
 and solution collectors. `user_input` and `user_output` are always present.
@@ -6815,7 +6821,7 @@ node test/run-conformance-report.mjs
 ```
 
 The complete suite must pass before release. The file-based conformance corpus
-contains 779 cases, including 373 focused ISO
+contains 780 cases, including 374 focused ISO
 cases derived from the success, failure, mode, and error behavior in
 ISO/IEC 13211-1 clauses 7 and 8, Part 2 modules, and Part 3 grammar rules.
 Separate exact-output suites check 189 normal