pll-python 0.5.1 → 0.6.0

package/README.md

@@ -34,9 +34,9 @@ print(greeting)
 | Line in the file | What it does |
 | --- | --- |
 | `#level raw` | Nothing checked. Plain Python plus PLL's libraries. This is what you get with no line at all. |
-| `#level beginner` | Warns about reassigning a variable, hiding another name (a built-in like `list`, or one of PLL's own like `circle`), and `global` / `nonlocal`. Type annotations are checked as the program runs. If it finds a problem, the file does **not** run. |
+| `#level beginner` | Warns about reassigning a variable, hiding another name (a built-in like `list`, or one of PLL's own like `circle`), `global` / `nonlocal`, and removing a name with `del` (not an item: `del d[key]` is fine). Type annotations are checked as the program runs. If it finds a problem, the file does **not** run. |
 | `#level intermediate` | Same, but reassignment is allowed inside a function. |
-| `#level advanced` | No pre-run checks; annotations still checked, by Python's own rules. |
+| `#level advanced` | One check before the run: names used where they have no value, which every level but `raw` makes. Annotations still checked, by Python's own rules. |
 
 There is deliberately **no flag to override the level**. A file behaves the
 same way everywhere, which is the point of putting it in the file.
@@ -46,7 +46,8 @@ same way everywhere, which is the point of putting it in the file.
 `test_*` functions in the same file run once the program finishes, against
 what it defined, as they do in the editor. A test that prints shows what it
 printed under its result; `@pytest.mark.skip`, `skipif` and `xfail` work as
-in pytest:
+in pytest. A test written with parameters is not run: what is wrong with it
+is said above `Tests:`, apart from the tests that ran.
 
 ```bash
 pll hw.py            # the program, then its tests
@@ -203,7 +204,7 @@ be fast and honest rather than secret.
 | 0 | Ran, and any tests passed |
 | 1 | The program raised, or Ctrl+C stopped it (even if it caught the `KeyboardInterrupt`; its tests are then not run) |
 | 2 | Level checks found problems, so it was not run |
-| 3 | A test failed |
+| 3 | A test failed, or could not be run (it takes parameters) |
 | 64 | Bad usage, or Python could not start (it is given two minutes) |
 | 130 | A second Ctrl+C gave up waiting for the program to stop |
 | 141 | Its output could no longer be read - `pll hw.py \| head` |

package/cli.cjs

@@ -137,19 +137,17 @@ var HEADER_RE = /^#\s*level\s+([a-z]+)\s*$/;
 function parseLevel(source) {
   const lines = source.split(/\r?\n/);
   for (const raw of lines) {
-    const line = raw.trim();
-    if (line.length === 0) {
+    if (raw.trim().length === 0) {
       continue;
     }
-    const match = HEADER_RE.exec(line);
-    if (!match) {
-      return DEFAULT_LEVEL;
-    }
-    const name = match[1];
-    return isLevel(name) ? name : DEFAULT_LEVEL;
+    return headerLevel(raw) ?? DEFAULT_LEVEL;
   }
   return DEFAULT_LEVEL;
 }
+function headerLevel(line) {
+  const match = HEADER_RE.exec(line.trim());
+  return match !== null && isLevel(match[1]) ? match[1] : null;
+}
 function levelHeaderProblem(source) {
   const lines = source.split(/\r?\n/);
   const valid = new RegExp(`^#\\s*level\\s+(?:${LEVEL_NAMES.join("|")})\\s*$`);
@@ -274,7 +272,7 @@ function closestLevel(named) {
   return best !== null && bestDistance <= Math.max(2, Math.floor(best.length / 3)) ? best : null;
 }
 function levelHasStaticChecks(level) {
-  return level === LEVEL_BEGINNER || level === LEVEL_INTERMEDIATE;
+  return level !== LEVEL_RAW;
 }
 
 // src/run/finding.ts
@@ -303,11 +301,13 @@ function findingLocation(finding) {
   };
 }
 function formatFriendlyError(finding) {
+  return serializedFindingLines(serializeFinding(finding));
+}
+function serializedFindingLines(finding) {
   const lines = [];
   lines.push(`${finding.errorType}: ${finding.headline}`);
-  const location = findingLocation(finding);
-  if (location) {
-    lines.push(`  at ${location.label}`);
+  if (finding.location) {
+    lines.push(`  at ${finding.location.label}`);
   }
   if (finding.howToFix.length > 0) {
     lines.push("");
@@ -345,7 +345,12 @@ function explainTestReport(event, fileName, level) {
     ...event,
     tests: event.tests.map(
       ({ error, ...test }) => error ? { ...test, finding: serializeFinding(findRuntimeFinding(fileName, level, error)) } : test
-    )
+    ),
+    ...event.malformed ? {
+      malformed: event.malformed.map(
+        ({ problem, ...test }) => problem ? { ...test, finding: serializeFinding({ ...problem, fileName: problem.fileName ?? fileName, level }) } : test
+      )
+    } : {}
   };
 }
 
@@ -1273,7 +1278,7 @@ var Tally = class {
       } else if (event.kind === "testReport") {
         this.testsRan = true;
         this.testsStopped = event.stopped === true;
-        this.testFailures += event.failed + event.errors;
+        this.testFailures += event.failed + event.errors + (event.malformed?.length ?? 0);
       } else if (event.kind === "done") {
         if (event.exitCode !== void 0) this.exitCode = event.exitCode;
         if (event.stopped) this.stopReached = true;
@@ -2338,6 +2343,16 @@ var CliView = class {
     }
   }
   testReport(event) {
+    for (const test of event.malformed ?? []) {
+      if (test.finding) {
+        const lines = serializedFindingLines(test.finding);
+        this.problem(this.red(lines[0]));
+        for (const line of lines.slice(1)) {
+          this.problem(line);
+        }
+        this.problem("");
+      }
+    }
     const bad = event.failed + event.errors;
     const parts = [
       ...event.passed ? [`${event.passed} passed`] : [],
@@ -2566,7 +2581,7 @@ ${USAGE_HINT}
     return EXIT.ok;
   }
   if (args.version) {
-    process.stdout.write("0.5.1\n");
+    process.stdout.write("0.6.0\n");
     return EXIT.ok;
   }
   if (args.error !== void 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pll-python",
-  "version": "0.5.1",
+  "version": "0.6.0",
   "description": "Run Python with PLL's language levels from the command line: level checks, in-file tests, friendly errors and the built-in image and table libraries, all on Pyodide. No Python install needed.",
   "license": "MIT",
   "repository": {