eyeprolog 1.1.24 → 1.1.25

package/package.json

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

package/src/iso.js

@@ -1164,9 +1164,9 @@ function* readTermBuiltin({ solver, goal, env }) {
   yield next;
 }
 function defaultTermWriteOptions(mode) {
-  if (mode === 'writeq') return { quoted: true, ignoreOps: false, numbervars: true, variableNames: new Map() };
-  if (mode === 'canonical') return { quoted: true, ignoreOps: true, numbervars: false, variableNames: new Map() };
-  return { quoted: false, ignoreOps: false, numbervars: true, variableNames: new Map() };
+  if (mode === 'writeq') return { quoted: true, ignoreOps: false, numbervars: true, variableNames: new Map(), compact: true, operatorAtomsAsArgs: true };
+  if (mode === 'canonical') return { quoted: true, ignoreOps: true, numbervars: false, variableNames: new Map(), compact: true, operatorAtomsAsArgs: true };
+  return { quoted: false, ignoreOps: false, numbervars: true, variableNames: new Map(), compact: true, operatorAtomsAsArgs: true };
 }
 
 function writeOptionBoolean(value, env, option) {

package/src/write.js

@@ -124,11 +124,18 @@ function chooseOperator(term, table) {
   return null;
 }
 
-function format(term, env, options, table, maxPriority = 1200) {
+function format(term, env, options, table, maxPriority = 1200, context = 'term') {
   const resolved = deref(term, env);
   if (resolved.type === VAR) return options.variableNames.get(resolved.name) ?? writeVariable(resolved.name);
   if (resolved.type === STRING) return writeString(resolved.name);
-  if (resolved.type === ATOM) return options.quoted ? writeAtom(resolved.name) : resolved.name;
+  if (resolved.type === ATOM) {
+    if (!options.quoted) return resolved.name;
+    // ISO 6.3.3.1 gives functional arguments and list elements a special
+    // `arg` production: an atom that is a current operator is valid there
+    // without quoting. Keep lexical exceptions such as `|` quoted.
+    if (options.operatorAtomsAsArgs && context === 'argument' && table.has(resolved.name)) return operatorName(resolved.name);
+    return writeAtom(resolved.name);
+  }
   if (resolved.type === NUMBER) return resolved.name;
 
   if (options.numbervars && resolved.type === COMPOUND && resolved.name === '$VAR' && resolved.arity === 1) {
@@ -146,9 +153,13 @@ function format(term, env, options, table, maxPriority = 1200) {
     let cursor = resolved;
     while (true) {
       cursor = deref(cursor, env);
-      if (isEmptyList(cursor)) return `[${parts.join(', ')}]`;
-      if (!isCons(cursor)) return `[${parts.join(', ')} | ${format(cursor, env, options, table, 999)}]`;
-      parts.push(format(cursor.args[0], env, options, table, 999));
+      const separator = options.compact ? ',' : ', ';
+      if (isEmptyList(cursor)) return `[${parts.join(separator)}]`;
+      if (!isCons(cursor)) {
+        const tailSeparator = options.compact ? '|' : ' | ';
+        return `[${parts.join(separator)}${tailSeparator}${format(cursor, env, options, table, 999, 'argument')}]`;
+      }
+      parts.push(format(cursor.args[0], env, options, table, 999, 'argument'));
       cursor = cursor.args[1];
     }
   }
@@ -183,8 +194,8 @@ function format(term, env, options, table, maxPriority = 1200) {
   }
 
   const name = options.quoted ? writeAtom(resolved.name) : resolved.name;
-  const args = resolved.args.map((arg) => format(arg, env, options, table, 999));
-  return `${name}(${args.join(', ')})`;
+  const args = resolved.args.map((arg) => format(arg, env, options, table, 999, 'argument'));
+  return `${name}(${args.join(options.compact ? ',' : ', ')})`;
 }
 
 export function formatTermForWrite(term, env = new Env(), options = {}) {
@@ -194,6 +205,8 @@ export function formatTermForWrite(term, env = new Env(), options = {}) {
     numbervars: options.numbervars !== false,
     doubleQuotes: options.doubleQuotes,
     variableNames: options.variableNames instanceof Map ? options.variableNames : new Map(),
+    compact: options.compact === true,
+    operatorAtomsAsArgs: options.operatorAtomsAsArgs === true,
   };
   return format(term, env, normalized, operatorTable(options.operators), 1200);
 }

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

@@ -1,2 +1,2 @@
-pair(First, Second)
+pair(First,Second)
 corrigendum3_write_variable_names.

package/test/run-regression.mjs

@@ -950,6 +950,23 @@ c4 ?- call((!;1)).
         assertEqual(run(source, { goal: 'operator_argument(ok)' }).stdout, 'operator_argument(ok).\n', 'operator argument syntax');
       },
     },
+    {
+      name: 'ISO writeq preserves operator atoms in argument syntax',
+      run: () => {
+        const source = [
+          'emit_operator_arguments :-',
+          "  writeq([:-,-]), put_char('|'),",
+          "  writeq(f(*)), put_char('|'),",
+          "  writeq(f(;,'|',';;')).",
+          '',
+        ].join('\n');
+        assertEqual(
+          run(source, { goal: 'emit_operator_arguments' }).stdout,
+          "[:-,-]|f(*)|f(;,'|',';;')emit_operator_arguments.\n",
+          'operator argument output',
+        );
+      },
+    },
     {
       name: 'ISO query operator and quad infix extension are visible through current_op/3',
       run: () => {
@@ -1020,7 +1037,7 @@ c4 ?- call((!;1)).
         ].join('\n');
         assertEqual(
           run(source, { goal: 'emit' }).stdout,
-          "hello world|'hello world'|a + b * c|'+'(a, *(b, c))|hello world|'hello world'|+(a, b)|a + b|A|$VAR(0)|pair(Left, Right)emit.\n",
+          "hello world|'hello world'|a + b * c|'+'(a,*(b,c))|hello world|'hello world'|+(a,b)|a + b|A|$VAR(0)|pair(Left,Right)emit.\n",
           'stdout',
         );
       },

package/the-art-of-eyeprolog.md

@@ -5131,7 +5131,12 @@ parsing of subsequent text, place them before their first use. ISO argument
 syntax also permits an atom that is currently an operator to appear directly
 as a functional argument or list element, so forms such as
 `current_op(Priority, Specifier, :-)` and `[:-,-]` are valid without quoting
-or parenthesizing those operator atoms. The ISO initial operator table also
+or parenthesizing those operator atoms. Term output observes the same `arg`
+rule: with `quoted(true)`, an operator atom is not quoted merely because it is
+an operator when it occurs as a functional argument or list element. Thus
+`writeq([:-,-])` emits `[:-,-]`, and `writeq(f(;,'|',';;'))` emits
+`f(;,'|',';;')`; the bar stays quoted because ISO treats the unquoted `|`
+token as a list separator rather than an atom. The ISO initial operator table also
 contains `?-` at priority 1200 with specifier `fx`, so
 `current_op(1200, fx, ?-)` succeeds. EyeProlog's embedded quad syntax permits
 an optional label before the query marker (`Label ?- Query.`), so while quad
@@ -5171,7 +5176,10 @@ write_event(Path, Event) :-
 
 The period is essential when another Prolog processor will read the result as
 a term. `write/1-2` uses readable conventional syntax, `writeq/1-2` quotes
-where needed, and `write_canonical/1-2` exposes canonical structure.
+where needed, and `write_canonical/1-2` exposes canonical structure. ISO term
+output uses only the separator characters needed by the syntax, so functional
+arguments and list elements are emitted compactly; for example `writeq([a,b])`
+outputs `[a,b]`.
 `write_term/2-3` supports `quoted/1`, `ignore_ops/1`, `numbervars/1`, and
 `variable_names/1`.