Skip to content

Remove toml j0.4 and use Polyscript TOML parser#2442

Merged
WebReflection merged 3 commits intopyscript:mainfrom
WebReflection:polyscript-toml
Feb 3, 2026
Merged

Remove toml j0.4 and use Polyscript TOML parser#2442
WebReflection merged 3 commits intopyscript:mainfrom
WebReflection:polyscript-toml

Conversation

@WebReflection
Copy link
Copy Markdown
Contributor

Description

This MR brings in the latest polyscript with its ability to understand a config-parser attribute from the main thread and use it whenever it's desired instead of its "native" basic-toml parser.

The parser specified on main will propagate out of the box to workers too, which is why there are manual tests around that.

Changes

  • updated polyscript to its latest with propagated custom parser abilities
  • changed the main logic to never bring in the deprecated (defunct) toml j0.4 module
  • created a test that shows both main thread and workers are able to use such parser

Checklist

  • I have checked make build works locally.
  • I have created / updated documentation for this change (if applicable).

@WebReflection WebReflection marked this pull request as draft January 27, 2026 17:47
@ntoll
Copy link
Copy Markdown
Member

ntoll commented Jan 27, 2026

Pending the changes we discussed.

@WebReflection WebReflection marked this pull request as ready for review January 28, 2026 18:05
@WebReflection
Copy link
Copy Markdown
Contributor Author

WebReflection commented Jan 29, 2026

published on npm as:

  • JS https://cdn.jsdelivr.net/npm/@pyscript/[email protected]/dist/core.js
  • CSS https://cdn.jsdelivr.net/npm/@pyscript/[email protected]/dist/core.css

@WebReflection WebReflection merged commit 8ace483 into pyscript:main Feb 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants