Skip to content

Pylon-dev-rework #66

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Pylon-dev-rework #66

wants to merge 28 commits into from

Conversation

schettn
Copy link
Contributor

@schettn schettn commented Feb 6, 2025

This pull request includes several changes to enhance the functionality and maintainability of the @getcronit/pylon and @getcronit/pylon-dev packages. The most notable changes include the addition of a new plugin build hook, the extension of the plugin system, and the replacement of treekill with ps-tree. Additionally, there are updates to the build process to use esbuild watch mode and improvements to the handling of fallback pages and unhandled routes.

Enhancements and New Features:

  • Added a new plugin build hook to allow custom esbuild builds, which is called before the main build process. This hook does not re-run during watch mode.
  • Extended the plugin system to support middleware and app setup, integrating the viewer via a built-in useViewer plugin. Custom plugins can now access the app instance and register middleware and setup functions.

Build Process Improvements:

  • Integrated the builder in the development server to simplify maintenance and future development.
  • Updated the build process to use esbuild watch mode instead of rebuilding from scratch.

Dependency and Configuration Updates:

  • Replaced treekill with ps-tree in the @getcronit/pylon-dev package.
  • Added a fallback page for the landing page and unhandled routes (404s), which can be disabled via the pylon config.

Copy link

changeset-bot bot commented Feb 6, 2025

🦋 Changeset detected

Latest commit: 5fd3a97

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@getcronit/pylon-dev Major
@getcronit/pylon Major
create-pylon Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Feb 6, 2025

🚀 Snapshot Release (canary)

The latest changes of this PR are available as canary on npm (based on the declared changesets):

Package Version Info
create-pylon 1.2.0-canary-20250218210337.5fd3a977d589c14cc224729b00d712f57339cb3d npm ↗︎ unpkg ↗︎
@getcronit/pylon 3.0.0-canary-20250218210337.5fd3a977d589c14cc224729b00d712f57339cb3d npm ↗︎ unpkg ↗︎
@getcronit/pylon-dev 2.0.0-canary-20250218210337.5fd3a977d589c14cc224729b00d712f57339cb3d npm ↗︎ unpkg ↗︎

@harrytran998
Copy link

Hi @schettn - Do you estimate the time you can finish this PR? Thanks for great package

@schettn
Copy link
Contributor Author

schettn commented Feb 14, 2025

Hi @harrytran998, what feature from this PR do you need?

@schettn
Copy link
Contributor Author

schettn commented Feb 14, 2025

This PR will result in a major release which includes:

  • A better process / build management for the dev server
  • Improve the plugin system to add custom build time / runtime plugins
  • useAuth plugin that works with all OIDC providers and adds builtin routes for authentication with tokens stored in the cookies.
  • usePages plugin for developing full stack applications Pylon’s New Pages Router – Full-Stack Support Coming! Feedback Wanted! #69
  • Removing the GQty integration from the dev server in favor of using the GQty CLI with --watch mode.

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