diff --git a/capabilities/testdrive-jsui b/capabilities/testdrive-jsui index a9a27b86..c79cfd19 160000 --- a/capabilities/testdrive-jsui +++ b/capabilities/testdrive-jsui @@ -1 +1 @@ -Subproject commit a9a27b86bb726a75d22ed013f5c4db1b59a7faa5 +Subproject commit c79cfd19c96bd088227b0bc6181e8f960e988978 diff --git a/workplans/MARKITECT-WP-0002-testdrive-jsui-publication.md b/workplans/MARKITECT-WP-0002-testdrive-jsui-publication.md index 544fec86..19f6760f 100644 --- a/workplans/MARKITECT-WP-0002-testdrive-jsui-publication.md +++ b/workplans/MARKITECT-WP-0002-testdrive-jsui-publication.md @@ -114,6 +114,10 @@ state_hub_task_id: "94dd2a30-fa38-471e-b19a-ed0913579524" 2026-10-02 (update, standalone `a9a27b8`, pin bumped): markitect-tool fixed (2)-(4): CJS now emitted as `.cjs`, `exports` map added (types/import/require/browser, `./style.css`), `npm pkg fix` applied, CHANGELOG in the tarball (15 files), `npm publish --dry-run` no longer warns about package.json. Also fixed a second bug found by that test: bundles threw `ControlBase is not defined` on load because controls skipped their `window.X =` assignment under a bundler. Independently re-verified here from a fresh `npm ci` and packed tarball: Jest 68/68; ESM `import` and CJS `require` load with a browser-like `window` (=globalThis, jsdom document); the min UMD bundle loads in a jsdom window and exposes `TestDriveJSUI`, `ControlBase`, `EditControl`. Caveats: the library is browser-only (plain Node import still throws; `indexedDB` unhandled rejection after load); Jest tests source, not `dist`, and no tarball regression test exists yet. **Still open for T05:** lint (1,172 problems, 860 `indent`); stale `github.com/markitect` URLs in package.json/pyproject.toml; no real-browser smoke test. T06 must not tag until lint is resolved or explicitly waived. +2026-10-02 (update, standalone `c79cfd1`, pin bumped): Bernd approved the format fix. `eslint --fix` (semi, indent 4) applied in `652cb4e`, cross-file browser globals declared in the eslint config (`de47bf1`): lint 1,172 → 134 problems (**47 errors**, 87 warnings). The remaining 47 are not formatting: unused variables and `jest/no-conditional-expect` / `valid-expect` in tests, plus `new-cap`, `no-cond-assign`, `no-prototype-builtins`. Lint is therefore still not a clean gate; fix or waive before T06. Jest 68/68, build ok, pack 15 files, `publish --dry-run` completes. + +**Browser smoke test (done, headless Chromium via Playwright, `markitect md-render --edit` output):** page loads with no page errors; `marked`, `ControlBase`, `EditControl`, `MarkitectDebugSystem` defined; clicking a section opens the editor with the right markdown; Accept re-renders the section (edited text and `` present), editor closes. The test found two pre-existing bugs (present at the original pin `b8f13b4`), both fixed in the standalone repo: `172660c` `js/core/debug-system.js` ended with an ES `export`, a SyntaxError as a classic `